Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[release/v1.1] scripts: use coordinator rules/settings for bare metal #1000

Merged
merged 2 commits into from
Nov 13, 2024

Conversation

edgelessci
Copy link
Contributor

Backport of #999 to release/v1.1.

Original description:


This fixes a bug where we would publish a coordinator deployment with a policy that can only be deployed in the default namespace.

Bug found by @3u13r

Copy link
Contributor

@burgerdev burgerdev left a comment

Choose a reason for hiding this comment

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

Congrats on #1000!

This fixes a bug where we would publish a coordinator deployment with a policy that can only be deployed in the default namespace.

Signed-off-by: Paul Meyer <[email protected]>
(cherry picked from commit a92ed86)
Signed-off-by: Paul Meyer <[email protected]>
(cherry picked from commit b41b8a6)
@katexochen katexochen force-pushed the backport-999-to-release/v1.1 branch from a0a682d to f127c2e Compare November 13, 2024 13:27
@katexochen katexochen merged commit 0f654c6 into release/v1.1 Nov 13, 2024
9 checks passed
@katexochen katexochen deleted the backport-999-to-release/v1.1 branch November 13, 2024 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fix Fixing a user facing bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants