UNCLASSIFIED - NO CUI

Skip to content
Snippets Groups Projects

Update gatekeeper to 3.6.0

Merged Jordan Olachea requested to merge bb-747-retry into main
1 unresolved thread

update "chart" (https://github.com/open-policy-agent/gatekeeper) from "v3.5.2" (5733cf33442d2c9485f59c181117c09e588da6cb) to "v3.6.0" (9503ef2307225c5f6e8d97e312efb008a26e36bb)

https://repo1.dso.mil/platform-one/big-bang/bigbang/-/issues/747

Edited by Tunde Oladipupo

Merge request reports

Merge request pipeline #502275 passed

Merge request pipeline passed for 77b7b12e

Merged by Tunde OladipupoTunde Oladipupo 3 years ago (Sep 29, 2021 4:18am UTC)

Merge details

  • Changes merged into main with 8276ee6a (commits were squashed).
  • Deleted the source branch.

Pipeline #504583 passed

Pipeline passed for 8276ee6a on main

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Michael McLeroy
    • One of the big changes for 3.6.0 is ConstraintTemplate CRD moved to v1. How hard would it be to upgrade our ConstraintTemplates to use the v1 API spec? If it is easy, lets include it in this MR. If not, I can open up a new issue for it.

    • Author Contributor

      Due to the time it took to figure out the bug, determine we are accomplishing it with flux, and bypass it, perhaps this should be a new task? Although it may not be possible to make it a new task with the current error in upgrade

    • I was able to move our ConstraintTemplates to v1(had to add type to fix validation error) so install works. For upgrade to work locally, I had to k apply -f chart/crds. I believe the only missing piece for upgrade to work is mounting the crds to gatekeeper-update-crds-hook container. I can also take a look at this later today.

    • Please register or sign in to reply
  • Jordan Olachea added 1 commit

    added 1 commit

    • c25b51bb - Dependencies removed by kpt added back to chart

    Compare with previous version

  • Jordan Olachea added 2 commits

    added 2 commits

    • f75bfb18 - removed repeat appVersion line chart
    • 4dae6532 - appVersion placement chart

    Compare with previous version

  • Michael McLeroy added statusdoing label and removed statusreview label

    added statusdoing label and removed statusreview label

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    • 012fdcf1 - Trying to create crdRepository image values

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    • ad34e248 - testing crdRepositoryversions with tag

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    • d269fb71 - removed tag from crdRepository -wasnt working

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    • a633967e - udpated crdHook to accept new tag

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    • 6c02b518 - new kubectl version for crdRelease

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    • 8cea6ab2 - testing original values with new syntax

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

    • Author Contributor
      Resolved by Jordan Olachea

      kubectl image and gatekeeeper-crd image output when "version" is passed

      `[kubectl:v1.21.1] Client Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.1", GitCommit:"5e58841cce77d4bc13713ad2b91fa0d961e69192", GitTreeState:"archive", BuildDate:"2021-06-18T09:32:04Z", GoVersion:"go1.16.3", Compiler:"gc", Platform:"linux/amd64"} The connection to the server localhost:8080 was refused - did you specify the right host or port?

      [Gatekeeper-crd] Client Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.1", GitCommit:"5e58841cce77d4bc13713ad2b91fa0d961e69192", GitTreeState:"clean", BuildDate:"2021-05-12T14:18:45Z", GoVersion:"go1.16.4", Compiler:"gc", Platform:"linux/amd64"} The connection to the server localhost:8080 was refused - did you specify the right host or port?`

  • Jordan Olachea added 1 commit

    added 1 commit

    • 071ab56e - Changed to full URL for docker crd to test paths

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    • 738c0ac7 - set crdRepo back to reg1 kubectl

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    • 5d9368e8 - added image pull secrets to crds

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 2 commits

    added 2 commits

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    • b30bb4c5 - removed string and left value crd imagepullsec

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    • 521ac1dc - added volumes/sec/imagepullsec for crd

    Compare with previous version

  • Jordan Olachea added 2 commits

    added 2 commits

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    • 46a085a0 - added imagePullSec to crd-hook

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    • 70f5826b - private-registry added to values

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    • 2ea00a19 - setting pullSecret "private-registry" for crd

    Compare with previous version

  • Jordan Olachea added 2 commits

    added 2 commits

    • cd060ac0 - Added image creddentials to values
    • 51b50c32 - updated helpers ant upgrade-crds-hook with sec

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    • 04ac90a3 - Removed upgrade-crds-hook secret added secret.yaml

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    • bd6264de - updated secrets and workaround for crd upgrades

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    • 67e8d07b - updated k8sPSPCap to v1 - from v1beta

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • added 1 commit

    • 3f3580f5 - updated constraint template version

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    • 776963c1 - Added volumes and enabled upgrade

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • Jordan Olachea added 1 commit

    added 1 commit

    Compare with previous version

  • The upgrade test successfully passed. As a preliminary step, I made an MR to merge into BB. Once that is done, we can make a follow up MR from [bb-747-retry-test] or similar to close out the upgrade. Some of the changes made include;

    • Updated selector and deployment strategy to fix this error
    • Added configmap to load crd directory
  • Tunde Oladipupo mentioned in merge request !105 (merged)

    mentioned in merge request !105 (merged)

  • Tunde Oladipupo added 75 commits

    added 75 commits

    Compare with previous version

  • Tunde Oladipupo mentioned in merge request !104 (closed)

    mentioned in merge request !104 (closed)

  • Tunde Oladipupo added statusreview label and removed statusdoing label

    added statusreview label and removed statusdoing label

  • Tunde Oladipupo changed milestone to %1.18.0

    changed milestone to %1.18.0

  • assigned to @echuang

  • LGTM, looking for a second opinion

  • Ernest Chuang approved this merge request

    approved this merge request

  • Branden Cobb approved this merge request

    approved this merge request

  • Tunde Oladipupo mentioned in commit 8276ee6a

    mentioned in commit 8276ee6a

  • Tunde Oladipupo changed the description

    changed the description

  • Please register or sign in to reply
    Loading