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

Rename enduser and tab repos #735

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

mrbobbytables
Copy link
Member

The end user repos are being restructured to more closely mirror the TOC.

A summary of changes:

  • cncf/tab -> cncf/tab-private - used for private discussions, meetings etc
  • cncf/enduser-public -> cncf/tab - used for public facing issues, policies, discussions, meetings
  • cncf/enduser -> cncf/enduser-private - end user members only items, discussions etc

Copy link

clowarden bot commented Dec 2, 2024

Validation failed

‼️ Some errors were found validating the configuration files


Errors:

  • invalid directory configuration
    • error parsing permissions file
      • did not find expected key at line 797 column 6, while parsing a block mapping at line 792 column 5

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

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

Copy link

clowarden bot commented Dec 2, 2024

Validation succeeded

✅ The proposed configuration changes are valid!

Configuration changes

Github

  • repository enduser-private has been added (visibility: private)
    • Teams
      • cncf-end-users: write
      • cncf-projects: admin
      • cncf-tab: write
    • Collaborators
      • Swil78: admin
  • repository tab-private has been added (visibility: private)
    • Teams
      • cncf-projects: admin
      • cncf-tab: write
    • Collaborators
      • Swil78: admin
  • user Swil78 is now a collaborator (role: admin) of repository tab
  • repository tab visibility has been updated to public

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

@mrbobbytables mrbobbytables added the ok-to-test Apply if PR is validated safe to test label Dec 2, 2024
@mrbobbytables mrbobbytables merged commit 3583656 into cncf:main Dec 2, 2024
5 of 6 checks passed
Copy link

clowarden bot commented Dec 2, 2024

Reconciliation completed

‼️ Some errors were found during the reconciliation

Github

  • Error applying change: CollaboratorAdded("glossary", "Giulia-dipietro", Write)
HTTP Error. Code: 403 Forbidden, message: {"message":"Resource not accessible by integration","documentation_url":"https://docs.github.com/rest/collaborators/collaborators#add-a-repository-collaborator","status":"403"}

@tegioz
Copy link
Contributor

tegioz commented Dec 3, 2024

User Giulia-dipietro does no longer exist. This error should be fixed by merging #734.

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