Skip to content

Automation: Jenkins fix when handling complex variable values with corral#17059

Merged
izaac merged 1 commit into
rancher:masterfrom
izaac:imported_cluster_kubeconfig_fix
Mar 28, 2026
Merged

Automation: Jenkins fix when handling complex variable values with corral#17059
izaac merged 1 commit into
rancher:masterfrom
izaac:imported_cluster_kubeconfig_fix

Conversation

@izaac
Copy link
Copy Markdown
Contributor

@izaac izaac commented Mar 28, 2026

Summary

Fixes rancher/qa-tasks#1867

Occurred changes and/or fixed issues

The clean_corral() function was making the base64 kubeconfig invalid after striping characters. The intention was to clean the logs not the variable values.

Areas or cases that should be tested

Jenkins CI

Areas which could experience regressions

Jenkins CI

Checklist

  • The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
  • The PR has a Milestone
  • The PR template has been filled out
  • The PR has been self reviewed
  • The PR has a reviewer assigned
  • The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
  • The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes
  • The PR has been reviewed in terms of Accessibility
  • The PR has considered, and if applicable tested with, the three Global Roles Admin, Standard User and User Base

@izaac izaac added this to the v2.15.0 milestone Mar 28, 2026
@izaac izaac self-assigned this Mar 28, 2026
@izaac izaac added QA/dev-automation Issues that engineers have written automation around so QA doesn't have look at this area/e2e labels Mar 28, 2026
@izaac izaac merged commit 7df6ef7 into rancher:master Mar 28, 2026
67 of 68 checks passed
@izaac izaac deleted the imported_cluster_kubeconfig_fix branch March 28, 2026 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/e2e QA/dev-automation Issues that engineers have written automation around so QA doesn't have look at this

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[UI] - Replace Corral with Ansible for the UI tests infra.

2 participants