UNCLASSIFIED - NO CUI

Skip to content
Snippets Groups Projects

SKIP UPGRADE fix: add autogenControllers, background, and failurePolicy to all policies

Merged Jonathan Braswell requested to merge 48-consistent-use into main

Addresses #48 (closed) by adding autogenControllers, background, and failurePolicy values to all kind: ClusterPolicy.

Tested by validating the policy changes successfully deploy to my cluster.

Note there is a bug in the upgrade tests. After a lot of debugging, I figured out the pod from the pre-upgrade wait-for-ready job is stuck in the Terminating state. I spent a few hours trying to figure out what is causing it, and ultimately gave up. I did verify that this MR did not introduce the issue, as an MR with no changes (only a version bump) reproduces the same failure. I opened #51 (closed) for this issue. I did verify an upgrade works without issue in a dev cluster, so it does seem to only be an issue in CI.

Edited by Jonathan Braswell

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading