Skip to content
This repository was archived by the owner on Aug 17, 2025. It is now read-only.

feat: Fixed realm name for kube deployments#5333

Merged
stuartwdouglas merged 3 commits intomainfrom
stuartwdouglas/fixed-realm-name
May 5, 2025
Merged

feat: Fixed realm name for kube deployments#5333
stuartwdouglas merged 3 commits intomainfrom
stuartwdouglas/fixed-realm-name

Conversation

@stuartwdouglas
Copy link
Copy Markdown
Collaborator

No description provided.

@stuartwdouglas stuartwdouglas requested review from a team and alecthomas as code owners May 4, 2025 22:27
@stuartwdouglas stuartwdouglas force-pushed the stuartwdouglas/fixed-realm-name branch from 2925177 to aff2815 Compare May 4, 2025 22:30
@stuartwdouglas stuartwdouglas force-pushed the stuartwdouglas/fixed-realm-name branch from aff2815 to b91ef78 Compare May 4, 2025 22:45
@stuartwdouglas stuartwdouglas requested a review from a team as a code owner May 4, 2025 22:45
Comment thread common/schema/schema.go Outdated
ChangesetEvents []*DeploymentRuntimeEvent `protobuf:"3"`
DeploymentEvents []*DeploymentRuntimeEvent `protobuf:"4"`
Realms []*RealmState `protobuf:"5"`
InternalRealm string `protobuf:"6"`
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 would prefer inferring this from existing Realms. I feel that would make this more flexible, if we ever want to support multiple internal realms in the future.

@stuartwdouglas stuartwdouglas force-pushed the stuartwdouglas/fixed-realm-name branch from 6ea6281 to 25877b7 Compare May 4, 2025 23:00
@alecthomas alecthomas mentioned this pull request May 4, 2025
@stuartwdouglas stuartwdouglas force-pushed the stuartwdouglas/fixed-realm-name branch 5 times, most recently from 109f877 to 3ec936b Compare May 5, 2025 06:15
@stuartwdouglas stuartwdouglas force-pushed the stuartwdouglas/fixed-realm-name branch from 3ec936b to c334213 Compare May 5, 2025 06:39
@stuartwdouglas stuartwdouglas merged commit 06c88f1 into main May 5, 2025
77 checks passed
@stuartwdouglas stuartwdouglas deleted the stuartwdouglas/fixed-realm-name branch May 5, 2025 07:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants