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

Improve the Netlify access model #9850

Closed
cahillsf opened this issue Dec 11, 2023 · 12 comments · Fixed by #10122
Closed

Improve the Netlify access model #9850

cahillsf opened this issue Dec 11, 2023 · 12 comments · Fixed by #10122
Labels
area/release Issues or PRs related to releasing kind/documentation Categorizes issue or PR as related to documentation. triage/accepted Indicates an issue or PR is ready to be actively worked on.

Comments

@cahillsf
Copy link
Member

Right now only limit people have access to the Netlify and for doing any thing regarding Netlify the people with access have to do some manual work. There is scope to improve this, either through more automation or more shared access.

Carryover from: #9104

@k8s-ci-robot k8s-ci-robot added the needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. label Dec 11, 2023
@k8s-ci-robot
Copy link
Contributor

This issue is currently awaiting triage.

If CAPI contributors determines this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@cahillsf
Copy link
Member Author

cahillsf commented Dec 11, 2023

@nawazkh @furkatgofurov7 you're listed for this one, want to continue pushing this forward? also would appreciate your feedback on prioritization for the issue board 🙇

@cahillsf
Copy link
Member Author

/area release

@k8s-ci-robot k8s-ci-robot added the area/release Issues or PRs related to releasing label Dec 11, 2023
@fabriziopandini
Copy link
Member

My two cents is to not over-engineering this but just to document the process because ultimately we need to access Netlify only one or two times a year (or even less)

@sbueringer
Copy link
Member

sbueringer commented Jan 19, 2024

Just want to mention that Netlify access is out of our control. I think you'll have to talk to sig-contribex if you want to change something about how all of this works.

@cahillsf
Copy link
Member Author

re the docs, for a start might be helpful to provide some direction for who currently has access (docs seem to just say "someone with access to Netlify" src). @sbueringer @fabriziopandini any ideas for better phrasing here? do all the maintainers have access?

last cycle i believe Cecile helped us with this (#6017 (comment)) but i am aware she is taking a step back from the project

@sbueringer
Copy link
Member

sbueringer commented Jan 25, 2024

I have access. IIRC @fabriziopandini as well. Very likely @vincepri as well.

@nawazkh
Copy link
Member

nawazkh commented Jan 31, 2024

@furkatgofurov7 and I were looking into this issue earlier, and the TL;DR is that "Today there is no automation for Netlify access, however there's a handful of people to handle access / requests for "all sites" (spanning SIGs / companies / contributors - to handle redundancy). It would be awesome to build something to automate it."

Reference conversation: https://kubernetes.slack.com/archives/C8TSNPY4T/p1697826615480259?thread_ts=1697826166.088039&cid=C8TSNPY4T

@fabriziopandini
Copy link
Member

+1 to improve the doc with the list of people having access (or just maintainers, the list ~overlaps) for now.
/kind documentation
/triage accepted

@k8s-ci-robot k8s-ci-robot added kind/documentation Categorizes issue or PR as related to documentation. triage/accepted Indicates an issue or PR is ready to be actively worked on. and removed needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Feb 6, 2024
@sbueringer
Copy link
Member

I think neither Killian nor Christian have Netlify access. I suspect Alberto also doesn't have it.

@fabriziopandini
Copy link
Member

Just checked
There is Cecile, Vince, Me, Stefan, Jack, Matt (Invite Pending)
We can add more by asking to ContribEx

@cahillsf
Copy link
Member Author

cahillsf commented Feb 8, 2024

just opened a small PR, lmk if this is sufficient

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/release Issues or PRs related to releasing kind/documentation Categorizes issue or PR as related to documentation. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Projects
Development

Successfully merging a pull request may close this issue.

5 participants