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(ci.jenkins.io) allow WinRM and CIFs over TCP from controller to private agent subnets + disable Network ACLs #83

Conversation

dduportal
Copy link
Contributor

@dduportal dduportal commented Jan 15, 2025

Related to jenkins-infra/helpdesk#4316

This PR sets the ground for prototying Windows EC2 agents using WinRM instead of SSH (see jenkins-infra/helpdesk#4316 (comment)) which should be faster.

We need to open a set of ports to allow WinRM protocol (both HTTP and HTTPS channels), along with CIFS over TCP as per the EC2 plugin help:

Capture d’écran 2025-01-15 à 14 44 28

Note: we are removing the Network ACLs here as a first step:

  • If we can succeed in using WinRM, then we'll add them back (with the new rules)
  • If we fail with WinRM or stick to SSH for Windows, then we'll rollback the Network ACL

Signed-off-by: Damien Duportal <[email protected]>
…private agent subnets + disable Network ACLs

Signed-off-by: Damien Duportal <[email protected]>
@dduportal dduportal changed the title Feat/ci.jenkins.io/allow winrm ec2 agents feat(ci.jenkins.io) allow WinRM and CIFs over TCP from controller to private agent subnets + disable Network ACLs Jan 15, 2025
@dduportal
Copy link
Contributor Author

Plan: 8 to add, 0 to change, 2 to destroy.
  • 2 Network ACLs objects to remove
  • 3 privates subnets X 2 SG rules out of controller subnet: 6
    • 2 SG rules in the private subnet for EC2 agents: 8 new objects

@dduportal dduportal merged commit 7b92221 into jenkins-infra:main Jan 15, 2025
3 checks passed
@dduportal dduportal deleted the feat/ci.jenkins.io/allow-winrm-ec2-agents branch January 15, 2025 13:48
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