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(vnets) add peering and subnet to allow ci.jenkins.io to spawn VM agents in the new subscription #172

Conversation

dduportal
Copy link
Contributor

@dduportal dduportal commented Nov 17, 2023

Related to jenkins-infra/helpdesk#3818

This PR adds 2 new resources to allow ci.jenkins.io to spin up VM agents in the new "public-jenkins-sponsorship" VNET:

  • Peering between the public vnet and this new vnet to ensure controller and agents can communicate with each other without requiring a public IP (agent -> controller is OK for ci.jenkins.io because it is a public service but using private IP for agents is a requirement so the controller should be able to contact these private VMs).

    • Also required for VPN to reach the new public subscription network for now
  • Add a subnet in the new vnet for ci.jenkins.io agents. Same sizing as the current one: max 512 IPs in the subnet (a BOM build is 300 agents, an ATH build is ~20 agents)

… agents in the new subscription

Signed-off-by: Damien Duportal <[email protected]>
@dduportal dduportal marked this pull request as ready for review November 17, 2023 09:57
@dduportal dduportal requested a review from a team November 17, 2023 09:57
vnets.tf Show resolved Hide resolved
@dduportal
Copy link
Contributor Author

Plan: 3 to add, 0 to change, 0 to destroy. as expected

@dduportal dduportal merged commit 212936f into jenkins-infra:main Nov 17, 2023
7 checks passed
@dduportal dduportal deleted the feat/helpdesk-3818/add-sponsorship-subscription-cijenkinsio-agents branch November 17, 2023 10:13
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.

2 participants