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

feat(updates.jenkins.io) restrict SA to only a few administrative IPs #570

Merged

Conversation

dduportal
Copy link
Contributor

This PR is related to both jenkins-infra/helpdesk#2649 (comment) and jenkins-infra/helpdesk#3875

It restricts the Azure SA used for updates.jenkins.io to only a few administrative IPs including trusted.ci's 2 networks.

It also allows trusted.ci new network (sponsored) agents to access publick8s controler (for update center)

@dduportal dduportal requested a review from a team January 6, 2024 08:27
Signed-off-by: Damien Duportal <[email protected]>
Copy link
Contributor Author

@dduportal dduportal left a comment

Choose a reason for hiding this comment

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

Plan: 0 to add, 2 to change, 0 to destroy. with plan as expected: self merging

@dduportal dduportal merged commit 29c783c into jenkins-infra:main Jan 6, 2024
2 checks passed
@dduportal dduportal deleted the feat/updates.jenkins.io/SAS-restrict branch January 6, 2024 08:39
dduportal added a commit that referenced this pull request Jan 6, 2024
… and should not disable public access to apply restriction (#571)

Fixup of #570 

Related to jenkins-infra/helpdesk#2649 and
jenkins-infra/helpdesk#3875

Signed-off-by: Damien Duportal <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant