profile
viewpoint
Timothy R. Chavez timrchavez @digitalocean Austin, TX

timrchavez/google-plus-hangout-livecoding 7

A live coding app for Google+ Hangouts

timrchavez/ansible-modules-extras 3

Ansible extra modules - these modules ship with ansible

timrchavez/gitflow 2

Git extensions to provide high-level repository operations for Vincent Driessen's branching model.

timrchavez/asciiquariumxss 1

An xscreensaver of the brilliant and timeless asciiquarium program!

timrchavez/gristle 1

Stream GitHub repository events to SSH-connected clients similar to gerrit stream-events

timrchavez/amazon-cognito-auth-js 0

The Amazon Cognito Auth SDK for JavaScript simplifies adding sign-up, sign-in with user profile functionality to web apps.

timrchavez/amplify-cli 0

A CLI toolchain for simplifying serverless web and mobile development.

timrchavez/ansible-modules-core 0

Ansible modules - these modules ship with ansible

timrchavez/apache-log-parser 0

Parses log lines from an apache log

timrchavez/atc 0

build scheduler and web interface

pull request commentkubernetes-sigs/kubespray

Move some approvers to emeritus status

I haven't been able to spend time on kubespray because of home renovations i'm afraid. Hope to have time again after December/January. :-)

No problem at all, and again not really a removal more, as said in the doc 👍

It is inevitable, but there are times when someone may shift focuses, change jobs or step away from a specific area in the project for a time. These people may be domain experts over certain areas of the codebase, but can no longer dedicate the time needed to handle the responsibilities of reviewing and approving changes. They are encouraged to add themselves as an "emeritus" approver under the emeritus_approvers key.

GitHub usernames listed under the emeritus_approvers key can no longer approve code (use the /approve command) and will be ignored by prow for assignment. However, it can still be referenced by a person looking at the OWNERS file for a possible second or more informed opinion.

When a contributor returns to being more active in that area, they may be promoted back to a regular approver at the discretion of the current approvers.

😃

floryut

comment created time in 16 minutes

pull request commentkubernetes-sigs/kubespray

Move some approvers to emeritus status

The following users are mentioned in OWNERS file(s) but are untrusted for the following reasons. One way to make the user trusted is to add them as members of the kubernetes-sigs org. You can then trigger verification by writing /verify-owners in a comment.

  • jjungnickel
    • User is not a member of the org. Satisfy at least one of these conditions to make the user trusted.
  • verwilst
    • User is not a member of the org. Satisfy at least one of these conditions to make the user trusted.
  • archifleks
    • User is not a member of the org. Satisfy at least one of these conditions to make the user trusted.
floryut

comment created time in 17 minutes

issue commentnetbox-community/netbox

Restore ability to collapse prefix hierarchy to uppermost level

@jeremystretch the comment from @martinkiska is relevant. It is what I wrote in the use case: Also opening Prefixes in collapsed form is more lucid and we do see the hierarchy of most parent subnets.

marbarmar

comment created time in 39 minutes

issue openeddenoland/deno

Cross-platform compilation support?

Now that a basic deno compile subcommand has been merged into deno it might be interesting to consider implementing support for a --target flag so that one could compile to all supported platforms to all supported platforms. Because of the approach taken for embedding the code into the binary it would probably be pretty easy to implement, the only slight problem being getting the binaries for the other platforms but those could just be fetched from releases.

created time in 42 minutes

pull request commentkubernetes-sigs/kubespray

Move some approvers to emeritus status

I haven't been able to spend time on kubespray because of home renovations i'm afraid. Hope to have time again after December/January. :-)

floryut

comment created time in 43 minutes

issue closedHygieia/Hygieia

UCD(uDeploy) collector showing only deployments done by admin user

Hi All,

We have configured Hygieia uDeploy collector and it is working fine for only the deployments done by "admin" user, if any other user, for example "xyz" or "xyz@domain.com" done the deployment then the deployment list is not showing.

Do we need to specify in application.properties like get the list of deployments done by specific user?

The below is my application.properties for uDeploy currently using.

dbname=dashboarddb #Database HostName - default is localhost dbhost=localhost #Database Port - default is 27017 dbport=27017 #Database Password - default is blank dbpassword=dbpassword #Logging File location logging.file=./logs/udeploy.log #Collector schedule (required) udeploy.cron=0 0/2 * * * * #uDeploy server (required) - Can provide multiple udeploy.servers=https://uDeploy-server/ #uDeploy user name (required) udeploy.username="xyz" or "xyz@domain.com" #udeploy.username= #uDeploy password (required) #udeploy.password= #uDeploy token can be used instead of username and password udeploy.token=<token>

closed time in an hour

HaasyaDhanvika

issue closedHygieia/Hygieia

Inconsitent info of Team Dashboard in Product Dashboard

Find enclosed "Team Dashboard" view. Also check another screenshot of "Team Dashboards" list shown while creating Product Dashboard. == Why Product Dashboard creation does not show correct list of Team Dashboards?

Btw. In fact, if you do not choose right dashboard then one may get "Exception RestApiExceptionHandler - null"

image

image

================================================

closed time in an hour

VivekBuzruk

pull request commentlxc/lxd

lxd/rbac: Filter storage UsedBy

Testsuite passed

stgraber

comment created time in an hour

push eventcodemirror/CodeMirror

Marijn Haverbeke

commit sha 5e11705588c69925dcd8531bc605854bb379150b

[clojure mode] Fix exponential-complexity regexp

view details

push time in an hour

push eventkubernetes-sigs/kubespray

Pasquale Toscano

commit sha f1231bb97d1701da61741d3477f80ee20d61e5e5

Add molecule for Kata Containers with Containerd (#6905)

view details

push time in an hour

PR merged kubernetes-sigs/kubespray

Reviewers
Add molecule for Kata Containers with Containerd approved cncf-cla: yes kind/feature lgtm size/M

<!-- Thanks for sending a pull request! Here are some tips for you:

  1. If this is your first time, please read our contributor guidelines: https://git.k8s.io/community/contributors/guide/first-contribution.md and developer guide https://git.k8s.io/community/contributors/devel/development.md
  2. Please label this pull request according to what type of issue you are addressing, especially if this is a release targeted pull request. For reference on required PR/issue labels, read here: https://git.k8s.io/community/contributors/devel/sig-release/release.md#issuepr-kind-label
  3. Ensure you have added or ran the appropriate tests for your PR: https://git.k8s.io/community/contributors/devel/sig-testing/testing.md
  4. If you want faster PR reviews, read how: https://git.k8s.io/community/contributors/guide/pull-requests.md#best-practices-for-faster-reviews
  5. Follow the instructions for writing a release note: https://git.k8s.io/community/contributors/guide/release-notes.md
  6. If the PR is unfinished, see how to mark it: https://git.k8s.io/community/contributors/guide/pull-requests.md#marking-unfinished-pull-requests -->

What type of PR is this?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:

/kind api-change /kind bug /kind cleanup /kind design /kind documentation /kind failing-test

/kind feature

/kind flake

What this PR does / why we need it:

This PR provides molecule testing for kata-containers role.

Which issue(s) this PR fixes: <!-- Automatically closes linked issue when PR is merged. Usage: Fixes #<issue number>, or Fixes (paste link of issue). If PR is about failing-tests or flakes, please post the related issues/tests in a comment and do not use Fixes --> Fixes #

Special notes for your reviewer:

Test this PR running the molecule tests inside the kata-containers directory:

cd roles/container-engine/kata-containers
molecule test

Does this PR introduce a user-facing change?: <!-- If no, just write "NONE" in the release-note block below. If yes, a release note is required: Enter your extended release note in the block below. If the PR requires additional action from users switching to the new release, include the string "action required". -->

NONE
+90 -0

5 comments

4 changed files

pasqualet

pr closed time in an hour

pull request commentkubernetes-sigs/kubespray

Helm v3 only

Ok do we unhold this and remove Helm2 support ? WDYT @Miouge1 @EppO @oomichi @hafe ?

champtar

comment created time in an hour

pull request commentkubernetes-sigs/kubespray

Add molecule for Kata Containers with Containerd

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: <a href="https://github.com/kubernetes-sigs/kubespray/pull/6905#issuecomment-736281446" title="Approved">floryut</a>, <a href="https://github.com/kubernetes-sigs/kubespray/pull/6905#" title="Author self-approved">pasqualet</a>

The full list of commands accepted by this bot can be found here.

The pull request process is described here

<details > Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment </details> <!-- META={"approvers":[]} -->

pasqualet

comment created time in an hour

pull request commentkubernetes-sigs/kubespray

Add molecule for Kata Containers with Containerd

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: <a href="https://github.com/kubernetes-sigs/kubespray/pull/6905#pullrequestreview-541549908" title="Approved">floryut</a>, <a href="https://github.com/kubernetes-sigs/kubespray/pull/6905#" title="Author self-approved">pasqualet</a>

The full list of commands accepted by this bot can be found here.

The pull request process is described here

<details > Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment </details> <!-- META={"approvers":[]} -->

pasqualet

comment created time in an hour

pull request commentkubernetes-sigs/kubespray

Add molecule for Kata Containers with Containerd

/approve

pasqualet

comment created time in an hour

pull request commentkubernetes-sigs/kubespray

Helm v3 only

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: <a href="https://github.com/kubernetes-sigs/kubespray/pull/6846#" title="Author self-approved">champtar</a>, <a href="https://github.com/kubernetes-sigs/kubespray/pull/6846#issuecomment-736276103" title="Approved">mattymo</a>

The full list of commands accepted by this bot can be found here.

The pull request process is described here

<details > Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment </details> <!-- META={"approvers":[]} -->

champtar

comment created time in an hour

pull request commentkubernetes-sigs/kubespray

Helm v3 only

/lgtm /approve

champtar

comment created time in 2 hours

pull request commentkubernetes-sigs/kubespray

Move some approvers to emeritus status

also seems @verwilst is left kubernetes org and therefore i think he can be removed not moved to emeritus

floryut

comment created time in 2 hours

pull request commentkubernetes-sigs/kubespray

Move some approvers to emeritus status

you can move me also for now, don't have much time to do contributions for now, maybe at some point it will change, but for now hard to say.

floryut

comment created time in 2 hours

PR opened denoland/deno

feat(std/node): Add `setImmediate` and `clearImmediate` to global scope

Fixes #8500

+39 -0

0 comment

3 changed files

pr created time in 2 hours

issue openedpennersr/django-allauth

Salesforce: "Social Network Login Failure"

I have a Django backend with django-allauth, and I'm trying to add Salesforce as a login method.

I've followed the instructions here:

  • Created a Salesforce Connected App with id and openid scopes (along with some others), and set the callback URL to https://www.mywebdomain.com/accounts/salesforce/login/callback/ (not my real domain, obviously)
  • Created a SocialApplication in Django with client ID, secret, and login URL (https://login.salesforce.com/)
  • Included allauth.socialaccount.providers.salesforce in INSTALLED_APPS Create a Social application in Django admin, with client id, client key, and login_url (in “key” field)

After entering Salesforce login credentials, the Salesforce system redirected to the callback URL, resulting in a page that contains the following text:

Social Network Login Failure
An error occurred while attempting to login via your social network account.

The URL in the address bar on this page looks like this:

https://www.mywebdomain.com/accounts/salesforce/login/callback/#access_token=00D3t000004QWRm%21ARwAQPfHWiM6jdB43dlyW6qjEw._34mjzGi_Jv6YCXp0QssT.9F9lCge5_YaH8gqTy3Od6SywCs8X9zOGv145SyviBVeGdn0&instance_url=https%3A%2F%2Fna123.salesforce.com&id=https%3A%2F%2Flogin.salesforce.com%2Fid%2F00D3t000004QWRmEAO%2F0053t000008QBetAAG&issued_at=1606802917608&signature=KvxAX0WBCFQYY%2BO25id9%2FXxpbh2q2d2vWdQ%2FFV5FCBw%3D&state=jsforce0.popup.c0ockgct29g&scope=id+api+web+refresh_token+openid&token_type=Bearer

I tried to probe the error (auth_error.code and auth_error.exception), but they were empty.

Am I missing something, or is the Salesforce provider experiencing an issue as currently implemented?

created time in 2 hours

issue commentlxc/lxd

Proper support for moving between pools

Hi, Im working with vpranav5 on this issue. We have a few questions if that is ok. We want to know what files we are modifying to create the Move Instance method and the MoveCustomVolume method and want to know if we should create a whole new branch for development locally. Unfortunately, whenever we try to follow the Adding Your Fork Remote instruction we get a permission denied error. Thanks! @stgraber

toby63

comment created time in 3 hours

issue commentnetbox-community/netbox

Enable custom fields for organizational models

Can I add a vote for custom fields of Interfaces too?

jeremystretch

comment created time in 4 hours

issue openeddenoland/deno

fetch function allows passing non-dictionary object as second parameter

In Chromium, calling fetch with a second parameter that isn't an object causes the promise to reject:

const not_an_object = 0;

fetch(url, not_an_object);
TypeError: Failed to execute 'fetch' on 'Window': cannot convert to dictionary.

Deno lets it pass without error.

created time in 6 hours

startedSamVerschueren/tsd-typescript

started time in 6 hours

Pull request review commentdenoland/deno

refactor(cli): DRY cli/ast.rs

 export function getIndex(c) {-    return "\x00\r\n\x85\u2028\u2029".indexOf(c);+    return "\u{0}\r\n\u{85}\u{2028}\u{2029}".indexOf(c);

See: https://github.com/denoland/deno/pull/8162#issuecomment-731719973

bartlomieju

comment created time in 6 hours

Pull request review commentdenoland/deno

refactor(cli): DRY cli/ast.rs

 export function getIndex(c) {-    return "\x00\r\n\x85\u2028\u2029".indexOf(c);+    return "\u{0}\r\n\u{85}\u{2028}\u{2029}".indexOf(c);

Something is wrong if you need to do this now... @kdy1 fixed this, so it shouldn't modify this...

bartlomieju

comment created time in 6 hours

pull request commentdenoland/deno

docs(runtime): Fix typo in load_module's docstring

CLA assistant check <br/>Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.<br/><sub>You have signed the CLA already but the status is still pending? Let us recheck it.</sub>

pakoito

comment created time in 6 hours

PR opened denoland/deno

docs(runtime): Fix typo in load_module's docstring

One small fix

+1 -1

0 comment

1 changed file

pr created time in 6 hours

pull request commentkubernetes-sigs/kubespray

[fix] change result conditions

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: <a href="https://github.com/kubernetes-sigs/kubespray/pull/6973#" title="Author self-approved">yonwon01</a> To complete the pull request process, please assign mirwan after the PR has been reviewed. You can assign the PR to them by writing /assign @mirwan in a comment when ready.

The full list of commands accepted by this bot can be found here.

<details open> Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment </details> <!-- META={"approvers":["mirwan"]} -->

yonwon01

comment created time in 6 hours

more