Skip to content

fix: update connection method#93

Merged
matttrach merged 2 commits into
rancher:mainfrom
matttrach:verify-webhook
May 13, 2025
Merged

fix: update connection method#93
matttrach merged 2 commits into
rancher:mainfrom
matttrach:verify-webhook

Conversation

@matttrach
Copy link
Copy Markdown
Collaborator

There are a few things in this one:

  1. We update the server module to the newest version
  2. We break out the downstream cluster creation in examples to their own local module
  3. We use a different method of getting the downstream cluster to talk to the Rancher cluster
    • rather than modifying the DNS address to have a 10... IP we allow the public addresses of the nodes
    • so now the public IPs of the downstream cluster are allowed to talk to the Rancher load balancer
  4. We added an additional check to make sure deployments are complete before calling Rancher ready
    • the Rancher webhook deployment was sometimes not ready when we were just checking nodes and pods

matttrach added 2 commits May 12, 2025 22:35
Signed-off-by: matttrach <matt.trachier@suse.com>
Signed-off-by: matttrach <matt.trachier@suse.com>
@matttrach matttrach self-assigned this May 13, 2025
@matttrach matttrach requested a review from a team as a code owner May 13, 2025 03:54
@matttrach matttrach merged commit 624c182 into rancher:main May 13, 2025
7 checks passed
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