Skip to content

ROX-15724: preserve infra kubeconfig contexts during merge - #118

Open
parametalol wants to merge 1 commit into
masterfrom
michael/ROX-15724-fix-kubeconfig-merge
Open

parametalol wants to merge 1 commit into
masterfrom
michael/ROX-15724-fix-kubeconfig-merge

Conversation

@parametalol

Copy link
Copy Markdown
Contributor

Summary

  • Replace the kubeconfig merge in infra-sync.sh with a Python helper while retaining infractl for artifact downloads.
  • Name infra contexts after their referenced clusters and always prefix infra user names with the artifact cluster name; avoid silent cluster, user, or context collisions.
  • Back up the prior kubeconfig and write the merged file privately, preserving it if the merge fails.

Supersedes #113.

Validation

  • python3 -m unittest discover -s scripts/dev -p test_infra_sync_merge.py -v (5 tests)
  • bash -n scripts/dev/infra-sync.sh
  • git diff --check

@coderabbitai

coderabbitai Bot commented Oct 20, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Central YAML (base), Organization UI (inherited)
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: fe6fb03a-6c28-47e1-8be2-0c778ced3153

Comment @coderabbitai help to get the list of available commands.

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