Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update config.yaml #399

Merged
merged 2 commits into from
Mar 14, 2024
Merged

Update config.yaml #399

merged 2 commits into from
Mar 14, 2024

Conversation

raravena80
Copy link
Contributor

Update new TAG-Runtime co-chairs access

Update new TAG-Runtime co-chairs access

Signed-off-by: Ricardo Aravena <[email protected]>
Copy link

clowarden bot commented Mar 14, 2024

Validation failed

‼️ Some errors were found validating the configuration files


Errors:

  • invalid github service configuration
    • error parsing permissions file
      • repositories[127].external_collaborators.miao0miao: unknown variant amdin, expected one of read, triage, write, maintain, admin at line 864 column 18

For more details about the configuration files format please see the documentation.

🔺 These errors must be addressed before this PR can be merged 🔺

Signed-off-by: Ricardo Aravena <[email protected]>
Copy link

clowarden bot commented Mar 14, 2024

Validation succeeded

✅ The proposed configuration changes are valid!

Configuration changes

Github

  • user helayoty is no longer a collaborator of repository tag-runtime
  • user quinton-hoole is no longer a collaborator of repository tag-runtime
  • user RichiH is no longer a collaborator of repository tag-runtime
  • user srust is now a collaborator (role: admin) of repository tag-runtime
  • user miao0miao is now a collaborator (role: admin) of repository tag-runtime

🔸 Please review the changes detected as they will be applied immediately once this PR is merged 🔸

Copy link
Member

@onlydole onlydole left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍🏼

@onlydole onlydole added the ok-to-test Apply if PR is validated safe to test label Mar 14, 2024
@onlydole onlydole merged commit bd6b28e into cncf:main Mar 14, 2024
4 of 5 checks passed
Copy link

clowarden bot commented Mar 14, 2024

Reconciliation completed

✅ The reconciliation completed successfully and all changes have been applied across the services!

Changes applied

Github

  • user helayoty is no longer a collaborator of repository tag-runtime
  • user RichiH is no longer a collaborator of repository tag-runtime
  • user quinton-hoole is no longer a collaborator of repository tag-runtime
  • user srust is now a collaborator (role: admin) of repository tag-runtime
  • user miao0miao is now a collaborator (role: admin) of repository tag-runtime

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test Apply if PR is validated safe to test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants