Skip to content

Provisioning CAPI downstream cluster#543

Merged
lscalabrini01 merged 8 commits into
rancher:mainfrom
lscalabrini01:qa-issue_2137
Jul 17, 2026
Merged

Provisioning CAPI downstream cluster#543
lscalabrini01 merged 8 commits into
rancher:mainfrom
lscalabrini01:qa-issue_2137

Conversation

@lscalabrini01

Copy link
Copy Markdown
Contributor

No description provided.

@lscalabrini01 lscalabrini01 self-assigned this Mar 3, 2026
@lscalabrini01 lscalabrini01 added the team/pit-crew slack notifier for pit crew label Mar 3, 2026
@lscalabrini01
lscalabrini01 requested review from khushboo-rancher and removed request for rancher-max July 13, 2026 19:59
@lscalabrini01
lscalabrini01 marked this pull request as ready for review July 14, 2026 17:00

@slickwarren slickwarren left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

just wondering if this is exclusive to harvester / behind the vpn or if you want it to work with other providers

Comment thread validation/pipeline/qainfra/Jenkinsfile.capi.e2e Outdated

@floatingman floatingman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm mostly concerned with the cleanup of AWS resources.

def capiAnsibleConfig = env.CAPI_ANSIBLE_CONFIG
def capiConfigFile = "capiconfig.yaml"
def capiClusterConfigFile = "capiclusterconfig.yaml"
def harvesterConfigfilename = "local.yaml"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think this is used anywhere.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

}
}

stage('Cleanup Rancher Environment') {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Does this stage cleanup any clusters that may have been made with CAPI?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just following up on this question.

@lscalabrini01 lscalabrini01 Jul 17, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, this stage cleanup the CAPI resources, the turtles clean the resources

@lscalabrini01
lscalabrini01 force-pushed the qa-issue_2137 branch 2 times, most recently from 89bd426 to 5421c14 Compare July 16, 2026 23:01
@@ -0,0 +1,13 @@
#!/bin/bash

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think it makes sense to have a script to run a single command that basically only works in Jenkins. Why not just run the command with sh?

@lscalabrini01 lscalabrini01 Jul 17, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

To keep the code organized and follow the standard

@hamistao hamistao Jul 17, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Which standard are you referring to? Perhaps I am unaware of this

@lscalabrini01
lscalabrini01 requested a review from hamistao July 17, 2026 17:25
@lscalabrini01
lscalabrini01 merged commit 7399dc1 into rancher:main Jul 17, 2026
4 checks passed
@lscalabrini01
lscalabrini01 deleted the qa-issue_2137 branch July 17, 2026 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team/pit-crew slack notifier for pit crew

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants