Skip to content

coordinator: fix equality checks for manifest properties#777

Merged
daniel-weisse merged 2 commits intomasterfrom
dw/fix-equality-checks
Dec 16, 2024
Merged

coordinator: fix equality checks for manifest properties#777
daniel-weisse merged 2 commits intomasterfrom
dw/fix-equality-checks

Conversation

@daniel-weisse
Copy link
Copy Markdown
Member

two missing checks were preventing manifest updates from being properly applied if only these values changed

Proposed changes

  • fix missing check for DisableSecretBinding in manifest.Marble.Equal
  • fix missing check for AcceptedTCBStatuses and AcceptedAdvisories in quote.PackageProperties.Equal

@daniel-weisse daniel-weisse added the bug fix This change fixes a user facing bug from the previous release label Dec 12, 2024
@netlify
Copy link
Copy Markdown

netlify bot commented Dec 12, 2024

Deploy Preview for marblerun-docs canceled.

Name Link
🔨 Latest commit 0607107
🔍 Latest deploy log https://app.netlify.com/sites/marblerun-docs/deploys/675c523949f1970008fc5b7d

@daniel-weisse daniel-weisse merged commit e7ee0c1 into master Dec 16, 2024
@daniel-weisse daniel-weisse deleted the dw/fix-equality-checks branch December 16, 2024 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug fix This change fixes a user facing bug from the previous release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants