Testing: Save pod logs and resources after e2e test run #234

jmprusi · 2023-05-29T14:57:20Z

Description

Generate a summary of logs/resources after e2e runs.

Disabled by default, to enable, pass the ARTIFACT_PATH env var when running the e2e tests: ARTIFACT_PATH=/tmp/e2e/ make test
Enabled on GitHub CI, you can access the artifacts via the "e2e / e2e-kind (pull_request)" check/action.

Current output of a e2e test run:

.
├── argocd-operator-system
│   └── argocd-operator-controller-manager-deployment.yaml
├── catalogd-system
│   ├── catalogd-controller-manager-77b88f9f46-85hg2-kube-rbac-proxy-logs.txt
│   ├── catalogd-controller-manager-77b88f9f46-85hg2-manager-logs.txt
│   └── catalogd-controller-manager-deployment.yaml
├── cert-manager
│   ├── cert-manager-5f76f6fd49-4926c-cert-manager-logs.txt
│   ├── cert-manager-cainjector-68698c5958-nvxzg-cert-manager-logs.txt
│   ├── cert-manager-cainjector-deployment.yaml
│   ├── cert-manager-deployment.yaml
│   ├── cert-manager-webhook-5c9c98975c-pplrr-cert-manager-logs.txt
│   └── cert-manager-webhook-deployment.yaml
├── crdvalidator-system
│   ├── crd-validation-webhook-67795fdc9d-nvz57-crd-validation-webhook-logs.txt
│   └── crd-validation-webhook-deployment.yaml
├── default
├── local-path-storage
│   ├── local-path-provisioner-684f458cdd-5vsfh-local-path-provisioner-logs.txt
│   └── local-path-provisioner-deployment.yaml
├── operator-controller-system
│   ├── operator-controller-controller-manager-5fdb4997d8-l8sqp-kube-rbac-proxy-logs.txt
│   ├── operator-controller-controller-manager-5fdb4997d8-l8sqp-manager-logs.txt
│   └── operator-controller-controller-manager-deployment.yaml
├── operator-qztk2xvw-bundleDeployment.yaml
├── operator-qztk2xvw-bvww52-bundle.yaml
├── operator-qztk2xvw-operator.yaml
├── rukpak-system
│   ├── core-66d8b5494d-vr494-kube-rbac-proxy-logs.txt
│   ├── core-66d8b5494d-vr494-manager-logs.txt
│   ├── core-deployment.yaml
│   ├── helm-provisioner-68b8df478-p8gp7-kube-rbac-proxy-logs.txt
│   ├── helm-provisioner-68b8df478-p8gp7-manager-logs.txt
│   ├── helm-provisioner-deployment.yaml
│   ├── rukpak-webhooks-5cd9dddf98-f6gx8-webhooks-logs.txt
│   └── rukpak-webhooks-deployment.yaml
└── test-catalog-catalogsource.yaml

9 directories, 29 files

TODO:

Enable/Disable
Custom e2e artifacts path
Download artifact on GitHub actions.

Reviewer Checklist

API Go Documentation
Tests: Unit Tests (and E2E Tests, if appropriate)
Comprehensive Commit Messages
Links to related GitHub Issue(s)

.github/workflows/e2e.yaml

test/e2e/install_test.go

ncdc · 2023-05-30T14:23:04Z

test/e2e/install_test.go

+
+	// Get all namespaces
+	namespaces, err := kubeClient.CoreV1().Namespaces().List(ctx, metav1.ListOptions{})
+	Expect(err).To(Not(HaveOccurred()))


Global: I can't remember - does Expect abort the test immediately if the expectation fails? If so, I'd recommend distinguishing between a hard-failure (e.g. couldn't create kubeClient, couldn't create artifactPath) and soft failures (couldn't list something, couldn't marshal something, etc.). For the soft failures, I'd suggest logging the issue and continuing, so you try to capture as much as you can.

Expect does abort immediately

I don't see a useful logger... fmt.Printf is good enough? what do you propose?

https://onsi.github.io/ginkgo/#logging-output has details on what's available

fixed, PTAL.

test/e2e/install_test.go

tmshort · 2023-05-30T18:09:56Z

test/e2e/install_test.go

+				buf := make([]byte, 1024)
+				for {
+					_, err := logs.Read(buf)
+					if err != nil {
+						break
+					}
+				}
+
+				// Save logs to artifact path
+				err = os.WriteFile(filepath.Join(namespaceArtifactPath, pod.Name+"-"+container.Name+"-logs.txt"), buf, 0644)


My gut is telling me this will only read 1024 bytes of the log at a time, and only write out the last 1024 bytes? (Although golang surprises me sometimes.)

So my understanding is that this will read from the logs 1024 bytes at a time until it encounters an error (ideally and EOF error to signal the end of the file, but the error check doesn't seem to differentiate - and maybe there isn't a need to) and then will break and write the output. I would personally prefer to substitute this for loop with a buf, err := io.ReadAll(logs) as it should handle all the reading for us.

Logs can be very long and artifact gathers that get OOMKilled write nothing :)

That's why I previously suggested a small buffer and io.Copy...

I could be reading it wrong, but it looks like io.ReadAll() internally reads a small buffer at a time: https://cs.opensource.google/go/go/+/refs/tags/go1.20.4:src/io/io.go;drc=dc8e2a6a8ec94f2c98ba20edd57932eba284efb1;l=690

Definitely agree with the suggestion of io.Copy for writing as the stream is read

It starts small, but because this function returns the entire contents, io.ReadAll continually increases the size of the buffer as it reads more data.

using io.Copy now, let me know if that's better

everettraven · 2023-05-30T20:48:22Z

.github/workflows/e2e.yaml

+        ARTIFACT_PATH=/tmp/artifacts make e2e
+
+    - uses: cytopia/upload-artifact-retry-action@v0.1.7
+      if: ${{ always() }}
+      with:
+        name: e2e-artifacts
+        path: /tmp/artifacts/


Might be nice to make the artifact path a variable we define once and use in multiple places. Some docs on github action (environment) variables: https://docs.github.com/en/actions/learn-github-actions/variables#about-variables

test/e2e/install_test.go

everettraven · 2023-05-30T21:06:49Z

test/e2e/install_test.go

+
+	// Get all namespaces
+	namespaces, err := kubeClient.CoreV1().Namespaces().List(ctx, metav1.ListOptions{})
+	Expect(err).To(Not(HaveOccurred()))


Expect does abort immediately

everettraven · 2023-05-30T21:13:02Z

test/e2e/install_test.go

+				buf := make([]byte, 1024)
+				for {
+					_, err := logs.Read(buf)
+					if err != nil {
+						break
+					}
+				}
+
+				// Save logs to artifact path
+				err = os.WriteFile(filepath.Join(namespaceArtifactPath, pod.Name+"-"+container.Name+"-logs.txt"), buf, 0644)


So my understanding is that this will read from the logs 1024 bytes at a time until it encounters an error (ideally and EOF error to signal the end of the file, but the error check doesn't seem to differentiate - and maybe there isn't a need to) and then will break and write the output. I would personally prefer to substitute this for loop with a buf, err := io.ReadAll(logs) as it should handle all the reading for us.

test/e2e/install_test.go

codecov · 2023-07-19T08:45:09Z

Codecov Report

Patch coverage has no change and project coverage change: -4.42% ⚠️

Comparison is base (1ec4a23) 81.64% compared to head (b9824db) 77.23%.

❗ Current head b9824db differs from pull request most recent head 29b8758. Consider uploading reports for the commit 29b8758 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #234      +/-   ##
==========================================
- Coverage   81.64%   77.23%   -4.42%     
==========================================
  Files          21       18       -3     
  Lines         937      795     -142     
==========================================
- Hits          765      614     -151     
- Misses        119      146      +27     
+ Partials       53       35      -18

Flag	Coverage Δ
e2e	`?`
unit	`77.23% <ø> (ø)`

Flags with carried forward coverage won't be shown. Click here to find out more.

see 6 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

joelanford · 2023-08-16T19:09:13Z

What's the status of this PR? Functionality like this would help shed light on this apparent e2e code coverage change. #336 (comment)

ncdc · 2023-09-07T14:46:14Z

test/e2e/install_test.go

+const (
+	defaultTimeout = 30 * time.Second
+	defaultPoll    = 1 * time.Second
+	testCatalogRef = "localhost/testdata/catalogs/test-catalog:e2e"


This needs to be removed - e2e_suite_test.go currently has consts for this, and a helper function getCatalogImageRef which supports optionally changing the catalog image via an env var.

ncdc · 2023-09-07T14:46:34Z

test/e2e/install_test.go

+// - bundle
+// - bundledeployments
+// - catalogsources
+


Suggested change

ncdc · 2023-09-07T14:46:59Z

test/e2e/install_test.go

 })
+
+// getArtifactsOutput gets all the artifacts from the test run and saves them to the artifact path.
+// right now it will save:


Suggested change

// right now it will save:

// Currently it saves:

ncdc · 2023-09-07T14:50:41Z

@jmprusi if you can rebase and make my requested changes, we can merge

Signed-off-by: Joaquim Moreno Prusi <joaquim@redhat.com>

…work#234) Bumps google.golang.org/protobuf from 1.31.0 to 1.33.0. --- updated-dependencies: - dependency-name: google.golang.org/protobuf dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

#1542) * Makes codecov-action optional We do not want to fail the job if codecov fails to upload the report due to rate limiting. Signed-off-by: Mikalai Radchuk <mradchuk@redhat.com> * Enable dependabot (#125) Signed-off-by: Andy Goldstein <andy.goldstein@redhat.com> * Enable merge queue (#131) Signed-off-by: Andy Goldstein <andy.goldstein@redhat.com> * Switch to using tilt-support repo Signed-off-by: Andy Goldstein <andy.goldstein@redhat.com> * Add DCO information Signed-off-by: Andy Goldstein <andy.goldstein@redhat.com> * Add CI for tilt (#134) Signed-off-by: Andy Goldstein <andy.goldstein@redhat.com> * Enable CatalogMetadataAPI via explicit flag, fix syncing issues (#138) * Setup CODEOWNERS (#140) Signed-off-by: Andy Goldstein <andy.goldstein@redhat.com> * Deprecate the `CatalogMetadataAPI` feature gate (#141) Marking the CatalogMetadataAPI feature gate as Deprecated before introducing the HTTP server as a default mechanism to serve catalog contents Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com> * Remove Package and BundleMetadata APIs (#149) Signed-off-by: Joe Lanford <joe.lanford@gmail.com> * add go-apidiff action (#151) Signed-off-by: Joe Lanford <joe.lanford@gmail.com> * Store FBC in local directory (#144) closes #113 Signed-off-by: Anik <anikbhattacharya93@gmail.com> * update demo based on API changes (#154) Signed-off-by: Jordan Keister <jordan@nimblewidget.com> * Serve locally stored fbc content via an http server (#148) Closes #113 Signed-off-by: Anik <anikbhattacharya93@gmail.com> * Add metrics to the catalog server (#156) * add metrics to catalogd http server that can be used for calculating the Apdex Score and assess the health of the http server that is serving catalog contents to clients Signed-off-by: Bryce Palmer <bpalmer@redhat.com> * quick fixes from review comments Signed-off-by: Bryce Palmer <bpalmer@redhat.com> * rename package from server --> metrics Signed-off-by: Bryce Palmer <bpalmer@redhat.com> * rename package from server --> metrics Signed-off-by: Bryce Palmer <bpalmer@redhat.com> --------- Signed-off-by: Bryce Palmer <bpalmer@redhat.com> * (docs): Add docs on fetching contents via HTTP server (#166) * (docs): Add docs on fetching contents via HTTP server Signed-off-by: Bryce Palmer <bpalmer@redhat.com> * add curl examples Signed-off-by: Bryce Palmer <everettraven@gmail.com> --------- Signed-off-by: Bryce Palmer <bpalmer@redhat.com> Signed-off-by: Bryce Palmer <everettraven@gmail.com> * (refactor): Remove deprecated CatalogMetadata and associated code (#169) * (refactor): Remove deprecated CatalogMetadata and associated code Signed-off-by: Bryce Palmer <bpalmer@redhat.com> * go mod tidy Signed-off-by: Bryce Palmer <bpalmer@redhat.com> --------- Signed-off-by: Bryce Palmer <bpalmer@redhat.com> * (server) Expose content URL on CR status (#168) closes #119 Signed-off-by: Anik <anikbhattacharya93@gmail.com> * deps: bump kubernetes and operator-registry (#173) Signed-off-by: Joe Lanford <joe.lanford@gmail.com> * Add RELEASING.md (#176) Add release guide Signed-off-by: Catherine Chan-Tse <cchantse@redhat.com> * Add contribution guide (#165) * Add contributing guide to catalogd Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com> * Address review comments Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com> --------- Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com> * Update README with simplified make target ref (#178) Signed-off-by: kevinrizza <krizza@redhat.com> * add default printer columns (#174) Add default printer columns Signed-off-by: kevinrizza <krizza@redhat.com> * (cleanup) Kustomization (#183) Closes #155, #160 Signed-off-by: Anik Bhattacharjee <anbhatta@redhat.com> * (feature): add direct image registry client `Unpacker` implementation (#145) (feat): direct image registry client unpacker Signed-off-by: Bryce Palmer <bpalmer@redhat.com> * (techdebt): refactor catalog controller unit tests (#196) * (techdebt): refactor catalog controller unit tests to no longer use Ginkgo and instead use the native Go testing and testify Signed-off-by: Bryce Palmer <everettraven@gmail.com> * remove rebase detritus, unnecessary IIFE, and featuregate comments/blocks. goimports. Signed-off-by: Bryce Palmer <everettraven@gmail.com> --------- Signed-off-by: Bryce Palmer <everettraven@gmail.com> * Bump golang.org/x/net from 0.10.0 to 0.17.0 (#197) Bumps [golang.org/x/net](https://github.com/golang/net) from 0.10.0 to 0.17.0. - [Commits](https://github.com/golang/net/compare/v0.10.0...v0.17.0) --- updated-dependencies: - dependency-name: golang.org/x/net dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump github.com/docker/docker from 23.0.1+incompatible to 23.0.3+incompatible (#195) Bump github.com/docker/docker Bumps [github.com/docker/docker](https://github.com/docker/docker) from 23.0.1+incompatible to 23.0.3+incompatible. - [Release notes](https://github.com/docker/docker/releases) - [Commits](https://github.com/docker/docker/compare/v23.0.1...v23.0.3) --- updated-dependencies: - dependency-name: github.com/docker/docker dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * (feature) Implement polling image source in intervals (#185) Implements https://docs.google.com/document/d/1iWSrWL9pYRJ5Ua3VYErkK1Q2lAusBUeDCh66Ew4lDbQ/edit?usp=sharing Closes #180 Signed-off-by: Anik Bhattacharjee <anbhatta@redhat.com> * Add GoDoc for Phase (#199) GoDoc for Phase Signed-off-by: Anik Bhattacharjee <anbhatta@redhat.com> * (feature): add skip-tls-verify option for image sources (#201) * (feature): add skip-tls-verify option for image sources Signed-off-by: Bryce Palmer <everettraven@gmail.com> * re-generate manifests Signed-off-by: Bryce Palmer <everettraven@gmail.com> --------- Signed-off-by: Bryce Palmer <everettraven@gmail.com> * 🌱 Add PR template with icons & verify job (#205) Add PR template with icons & verify job Signed-off-by: Andy Goldstein <andy.goldstein@redhat.com> * 🌱 Bump github.com/docker/docker from 23.0.3+incompatible to 24.0.7+incompatible (#203) Bump github.com/docker/docker Bumps [github.com/docker/docker](https://github.com/docker/docker) from 23.0.3+incompatible to 24.0.7+incompatible. - [Release notes](https://github.com/docker/docker/releases) - [Commits](https://github.com/docker/docker/compare/v23.0.3...v24.0.7) --- updated-dependencies: - dependency-name: github.com/docker/docker dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :bug: attempt to remove cache entry if unpack fails (#207) (bugfix): attempt to remove cache entry if unpack fails and add a new test case that ensures if the required label is missing from the image, thus failing image unpacking, an error is consistently returned fixes #206 Signed-off-by: Bryce Palmer <everettraven@gmail.com> * 🌱 Prefix dependabot GH actions PRs (#208) Prefix dependabot GH actions PRs Signed-off-by: Andy Goldstein <andy.goldstein@redhat.com> * :seedling: replace e2e Job with client-go ProxyGet() call (#210) (e2e): replace e2e Job with client-go ProxyGet() call Signed-off-by: Bryce Palmer <everettraven@gmail.com> * ✨ Add Ingress overlay to access `Catalog` contents outside of the cluster (#209) * Add Ingress overlay to access `Catalog` contents outside of the cluster Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com> * Address review feedback Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com> * Address review feedback #1 Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com> * Address feedback #2 Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com> --------- Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com> * 🌱 bump to k8s 1.27.7 (#211) bump to k8s 1.27.7 to ensure catalogd is fully remediated and unaffected by the rapid reset CVE Signed-off-by: Bryce Palmer <everettraven@gmail.com> * :seedling: Remove cert-manager from release note install process (#215) Remove cert-manager from release note install process as cert-manager has not been a requirement for installing catalogd for quite some time. Signed-off-by: Bryce Palmer <everettraven@gmail.com> * :bug: Add enum validation for source type (#214) Add enum validation for source type Signed-off-by: Bryce Palmer <everettraven@gmail.com> * 🌱 updates to enable autogenerated demo, hosted out-of-tree (#212) * updates to support automated demo generation Signed-off-by: Jordan Keister <jordan@nimblewidget.com> * review updates Signed-off-by: Jordan Keister <jordan@nimblewidget.com> --------- Signed-off-by: Jordan Keister <jordan@nimblewidget.com> * :seedling: demo install from published tag (#217) * demo install from published tag Signed-off-by: Jordan Keister <jordan@nimblewidget.com> * review updates Signed-off-by: Jordan Keister <jordan@nimblewidget.com> --------- Signed-off-by: Jordan Keister <jordan@nimblewidget.com> * Bump golang.org/x/crypto from 0.14.0 to 0.17.0 (#218) Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.14.0 to 0.17.0. - [Commits](https://github.com/golang/crypto/compare/v0.14.0...v0.17.0) --- updated-dependencies: - dependency-name: golang.org/x/crypto dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump github.com/go-git/go-git/v5 from 5.4.2 to 5.11.0 (#220) Bumps [github.com/go-git/go-git/v5](https://github.com/go-git/go-git) from 5.4.2 to 5.11.0. - [Release notes](https://github.com/go-git/go-git/releases) - [Commits](https://github.com/go-git/go-git/compare/v5.4.2...v5.11.0) --- updated-dependencies: - dependency-name: github.com/go-git/go-git/v5 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump github.com/containerd/containerd from 1.6.22 to 1.6.26 (#219) Bumps [github.com/containerd/containerd](https://github.com/containerd/containerd) from 1.6.22 to 1.6.26. - [Release notes](https://github.com/containerd/containerd/releases) - [Changelog](https://github.com/containerd/containerd/blob/main/RELEASES.md) - [Commits](https://github.com/containerd/containerd/compare/v1.6.22...v1.6.26) --- updated-dependencies: - dependency-name: github.com/containerd/containerd dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * (bugfix): bump the catalogserver write timeout to 5 minutes (#223) to allow for fully writing large catalogs contents in the response Signed-off-by: everettraven <everettraven@gmail.com> * (bugfix): add validation for pollInterval duration units (#225) Signed-off-by: everettraven <everettraven@gmail.com> * (chore): bump supported k8s version to 1.28 (#227) bumps controller-runtime --> v0.16.3 bumps k8s.io deps --> v0.28.5 bumps operator-registry --> v1.34.0 updated kind version --> v0.20.0 made kind node image configurable and 1.28 by default updated ginkgo version --> v2.12.0 Signed-off-by: everettraven <everettraven@gmail.com> * Bump github.com/containerd/containerd from 1.7.6 to 1.7.11 (#228) Bumps [github.com/containerd/containerd](https://github.com/containerd/containerd) from 1.7.6 to 1.7.11. - [Release notes](https://github.com/containerd/containerd/releases) - [Changelog](https://github.com/containerd/containerd/blob/main/RELEASES.md) - [Commits](https://github.com/containerd/containerd/compare/v1.7.6...v1.7.11) --- updated-dependencies: - dependency-name: github.com/containerd/containerd dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump golang.org/x/crypto from 0.16.0 to 0.17.0 (#230) Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.16.0 to 0.17.0. - [Commits](https://github.com/golang/crypto/compare/v0.16.0...v0.17.0) --- updated-dependencies: - dependency-name: golang.org/x/crypto dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * (bugfix): make garbage collection a runnable (#231) and add it to the controller manager. Make it log errors instead of exiting. This prevents crashlooping when there are errors in the garbage collection process. Signed-off-by: everettraven <everettraven@gmail.com> * bumping codecov-action to v4 (#232) * GH Actions: add workflow to add epic issues to OLMv1 project (#233) Signed-off-by: Joe Lanford <joe.lanford@gmail.com> * Bump google.golang.org/protobuf from 1.31.0 to 1.33.0 (#234) Bumps google.golang.org/protobuf from 1.31.0 to 1.33.0. --- updated-dependencies: - dependency-name: google.golang.org/protobuf dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump Go to 1.21 (#237) Signed-off-by: Mikalai Radchuk <mradchuk@redhat.com> * bump docker deps to 25.0.5 (#236) * bump kind to 0.22.0 (#238) Signed-off-by: Joe Lanford <joe.lanford@gmail.com> * remove terminating space in dependabot config file (#245) Signed-off-by: Jordan Keister <jordan@nimblewidget.com> * make golangci-lint config consistent across repos (#255) Signed-off-by: Joe Lanford <joe.lanford@gmail.com> * Bump golang.org/x/net from 0.19.0 to 0.23.0 (#244) Bumps [golang.org/x/net](https://github.com/golang/net) from 0.19.0 to 0.23.0. - [Commits](https://github.com/golang/net/compare/v0.19.0...v0.23.0) --- updated-dependencies: - dependency-name: golang.org/x/net dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump docker/login-action from 2 to 3 (#246) Bumps [docker/login-action](https://github.com/docker/login-action) from 2 to 3. - [Release notes](https://github.com/docker/login-action/releases) - [Commits](https://github.com/docker/login-action/compare/v2...v3) --- updated-dependencies: - dependency-name: docker/login-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump actions/checkout from 3 to 4 (#247) Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](https://github.com/actions/checkout/compare/v3...v4) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump github.com/go-logr/logr from 1.3.0 to 1.4.1 (#253) Bumps [github.com/go-logr/logr](https://github.com/go-logr/logr) from 1.3.0 to 1.4.1. - [Release notes](https://github.com/go-logr/logr/releases) - [Changelog](https://github.com/go-logr/logr/blob/master/CHANGELOG.md) - [Commits](https://github.com/go-logr/logr/compare/v1.3.0...v1.4.1) --- updated-dependencies: - dependency-name: github.com/go-logr/logr dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump actions/add-to-project from 0.5.0 to 1.0.1 (#249) Bumps [actions/add-to-project](https://github.com/actions/add-to-project) from 0.5.0 to 1.0.1. - [Release notes](https://github.com/actions/add-to-project/releases) - [Commits](https://github.com/actions/add-to-project/compare/v0.5.0...v1.0.1) --- updated-dependencies: - dependency-name: actions/add-to-project dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump actions/setup-go from 4 to 5 (#250) Bumps [actions/setup-go](https://github.com/actions/setup-go) from 4 to 5. - [Release notes](https://github.com/actions/setup-go/releases) - [Commits](https://github.com/actions/setup-go/compare/v4...v5) --- updated-dependencies: - dependency-name: actions/setup-go dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump sigs.k8s.io/yaml from 1.3.0 to 1.4.0 (#254) Bumps [sigs.k8s.io/yaml](https://github.com/kubernetes-sigs/yaml) from 1.3.0 to 1.4.0. - [Release notes](https://github.com/kubernetes-sigs/yaml/releases) - [Changelog](https://github.com/kubernetes-sigs/yaml/blob/master/RELEASE.md) - [Commits](https://github.com/kubernetes-sigs/yaml/compare/v1.3.0...v1.4.0) --- updated-dependencies: - dependency-name: sigs.k8s.io/yaml dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump kubernetes-sigs/kubebuilder-release-tools (#248) Bumps [kubernetes-sigs/kubebuilder-release-tools](https://github.com/kubernetes-sigs/kubebuilder-release-tools) from 0.4.0 to 0.4.3. - [Release notes](https://github.com/kubernetes-sigs/kubebuilder-release-tools/releases) - [Changelog](https://github.com/kubernetes-sigs/kubebuilder-release-tools/blob/master/RELEASE.md) - [Commits](https://github.com/kubernetes-sigs/kubebuilder-release-tools/compare/v0.4.0...v0.4.3) --- updated-dependencies: - dependency-name: kubernetes-sigs/kubebuilder-release-tools dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * OCPBUGS-28230: enforce termination message policy on all platform pods (#260) Signed-off-by: Joe Lanford <joe.lanford@gmail.com> * bump controller-gen to v0.14.0 (#262) Signed-off-by: Tayler Geiger <tayler@redhat.com> * ⚠️ Serve catalog over HTTPS (#263) * make catalog server serve catalog contents over HTTPS adds cert-manager as a dependency again to create self-signed certs for the catalog server Signed-off-by: everettraven <everettraven@gmail.com> * fix e2e Signed-off-by: everettraven <everettraven@gmail.com> * Reorganize manifests for cert-manager overlay This allows the use of alternate certificate managers. Signed-off-by: Tayler Geiger <tayler@redhat.com> * Reconfigure TLS functionality to use Listener Fix a few manifest issues as well. Signed-off-by: Tayler Geiger <tayler@redhat.com> * Add certwatcher for TLS cert and key from controller-runtime - Add error for missing either tls-key or tls-cert arguments. - Move server creation and configuration to serverutil Signed-off-by: Tayler Geiger <tayler@redhat.com> * Update README and docs for HTTPS --------- Signed-off-by: everettraven <everettraven@gmail.com> Signed-off-by: Tayler Geiger <tayler@redhat.com> Co-authored-by: everettraven <everettraven@gmail.com> * Makefile: fix issues with repeated evaluations (#267) * rename Catalog to ClusterCatalog (#268) Signed-off-by: everettraven <everettraven@gmail.com> * ✨ Bump k8s deps to 0.30.0 and go 1.22 (#269) * bump to go 1.22 and k8s 0.30.0 Signed-off-by: everettraven <everettraven@gmail.com> * bump o-reg to tag Signed-off-by: everettraven <everettraven@gmail.com> --------- Signed-off-by: everettraven <everettraven@gmail.com> * :seedling: Bump k8s.io/client-go from 0.30.0 to 0.30.1 (#276) Bumps [k8s.io/client-go](https://github.com/kubernetes/client-go) from 0.30.0 to 0.30.1. - [Changelog](https://github.com/kubernetes/client-go/blob/master/CHANGELOG.md) - [Commits](https://github.com/kubernetes/client-go/compare/v0.30.0...v0.30.1) --- updated-dependencies: - dependency-name: k8s.io/client-go dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump k8s.io/component-base from 0.28.5 to 0.30.1 (#275) Bumps [k8s.io/component-base](https://github.com/kubernetes/component-base) from 0.28.5 to 0.30.1. - [Commits](https://github.com/kubernetes/component-base/compare/v0.28.5...v0.30.1) --- updated-dependencies: - dependency-name: k8s.io/component-base dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump k8s.io/apiserver from 0.28.5 to 0.30.1 (#274) Bumps [k8s.io/apiserver](https://github.com/kubernetes/apiserver) from 0.28.5 to 0.30.1. - [Commits](https://github.com/kubernetes/apiserver/compare/v0.28.5...v0.30.1) --- updated-dependencies: - dependency-name: k8s.io/apiserver dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Fix XDG_DATA_HOME build issue (#264) This mirrors the same fix made previously in operator-controller: https://github.com/operator-framework/operator-controller/pull/790 Signed-off-by: Tayler Geiger <tayler@redhat.com> * Change default namespace to olmv1-system (#283) * :seedling: Bump k8s.io/apiextensions-apiserver from 0.30.0 to 0.30.2 (#284) Bumps [k8s.io/apiextensions-apiserver](https://github.com/kubernetes/apiextensions-apiserver) from 0.30.0 to 0.30.2. - [Release notes](https://github.com/kubernetes/apiextensions-apiserver/releases) - [Commits](https://github.com/kubernetes/apiextensions-apiserver/compare/v0.30.0...v0.30.2) --- updated-dependencies: - dependency-name: k8s.io/apiextensions-apiserver dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump github.com/containerd/containerd from 1.7.16 to 1.7.18 (#282) Bumps [github.com/containerd/containerd](https://github.com/containerd/containerd) from 1.7.16 to 1.7.18. - [Release notes](https://github.com/containerd/containerd/releases) - [Changelog](https://github.com/containerd/containerd/blob/main/RELEASES.md) - [Commits](https://github.com/containerd/containerd/compare/v1.7.16...v1.7.18) --- updated-dependencies: - dependency-name: github.com/containerd/containerd dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Fix codecov-action params (#287) * `functionalities` param is no longer exist. It was used to enable file fixes to ignore common lines from coverage. This feature is now seems to be on by default. * Adding `disable_search` because we do not need for the codecov action to search for coverage files: we explicitly provide files. Signed-off-by: Mikalai Radchuk <mradchuk@redhat.com> * Use url.Scheme to determine port number (#289) Signed-off-by: Todd Short <todd.short@me.com> * Fixing the labels for better identify metrics (#292) As the label selector used for both catalogd and operator-controller metrics services is "control-plane: controller-manager". Hence changing the labels in both operator-controller and catalogd to make sure we do not overlap. Refer https://github.com/operator-framework/operator-controller/issues/955 for details Signed-off-by: Lalatendu Mohanty <lmohanty@redhat.com> * adding gzip content support (#293) Signed-off-by: Jordan Keister <jordan@nimblewidget.com> * :seedling: Bump actions/add-to-project from 1.0.1 to 1.0.2 (#291) Bumps [actions/add-to-project](https://github.com/actions/add-to-project) from 1.0.1 to 1.0.2. - [Release notes](https://github.com/actions/add-to-project/releases) - [Commits](https://github.com/actions/add-to-project/compare/v1.0.1...v1.0.2) --- updated-dependencies: - dependency-name: actions/add-to-project dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump github.com/prometheus/client_golang (#286) Bumps [github.com/prometheus/client_golang](https://github.com/prometheus/client_golang) from 1.18.0 to 1.19.1. - [Release notes](https://github.com/prometheus/client_golang/releases) - [Changelog](https://github.com/prometheus/client_golang/blob/main/CHANGELOG.md) - [Commits](https://github.com/prometheus/client_golang/compare/v1.18.0...v1.19.1) --- updated-dependencies: - dependency-name: github.com/prometheus/client_golang dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump github.com/google/go-containerregistry (#285) Bumps [github.com/google/go-containerregistry](https://github.com/google/go-containerregistry) from 0.19.1 to 0.19.2. - [Release notes](https://github.com/google/go-containerregistry/releases) - [Changelog](https://github.com/google/go-containerregistry/blob/main/.goreleaser.yml) - [Commits](https://github.com/google/go-containerregistry/compare/v0.19.1...v0.19.2) --- updated-dependencies: - dependency-name: github.com/google/go-containerregistry dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump github.com/go-logr/logr from 1.4.1 to 1.4.2 (#279) Bumps [github.com/go-logr/logr](https://github.com/go-logr/logr) from 1.4.1 to 1.4.2. - [Release notes](https://github.com/go-logr/logr/releases) - [Changelog](https://github.com/go-logr/logr/blob/master/CHANGELOG.md) - [Commits](https://github.com/go-logr/logr/compare/v1.4.1...v1.4.2) --- updated-dependencies: - dependency-name: github.com/go-logr/logr dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump github.com/onsi/ginkgo/v2 from 2.17.3 to 2.19.0 (#277) Bumps [github.com/onsi/ginkgo/v2](https://github.com/onsi/ginkgo) from 2.17.3 to 2.19.0. - [Release notes](https://github.com/onsi/ginkgo/releases) - [Changelog](https://github.com/onsi/ginkgo/blob/master/CHANGELOG.md) - [Commits](https://github.com/onsi/ginkgo/compare/v2.17.3...v2.19.0) --- updated-dependencies: - dependency-name: github.com/onsi/ginkgo/v2 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump github.com/operator-framework/operator-registry (#281) Bumps [github.com/operator-framework/operator-registry](https://github.com/operator-framework/operator-registry) from 1.42.0 to 1.43.1. - [Release notes](https://github.com/operator-framework/operator-registry/releases) - [Commits](https://github.com/operator-framework/operator-registry/compare/v1.42.0...v1.43.1) --- updated-dependencies: - dependency-name: github.com/operator-framework/operator-registry dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * check the underlying storage for existing cluster catalog content (#290) Signed-off-by: Igor Troyanovsky <itroyano@redhat.com> * :seedling: Bump github.com/operator-framework/operator-registry (#294) Bumps [github.com/operator-framework/operator-registry](https://github.com/operator-framework/operator-registry) from 1.43.1 to 1.44.0. - [Release notes](https://github.com/operator-framework/operator-registry/releases) - [Commits](https://github.com/operator-framework/operator-registry/compare/v1.43.1...v1.44.0) --- updated-dependencies: - dependency-name: github.com/operator-framework/operator-registry dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Share common CA with OLMv1 in overlays/cert-manager (#296) Use kustomization Components to share a common ClusterIssuer with operator-controller. Fixes #295 Signed-off-by: Todd Short <todd.short@me.com> * :seedling: Bump github.com/containerd/containerd from 1.7.18 to 1.7.19 (#297) Bumps [github.com/containerd/containerd](https://github.com/containerd/containerd) from 1.7.18 to 1.7.19. - [Release notes](https://github.com/containerd/containerd/releases) - [Changelog](https://github.com/containerd/containerd/blob/main/RELEASES.md) - [Commits](https://github.com/containerd/containerd/compare/v1.7.18...v1.7.19) --- updated-dependencies: - dependency-name: github.com/containerd/containerd dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * reduce cpu requests (#298) Signed-off-by: Jordan Keister <jordan@nimblewidget.com> * Remove second installation of cert-manager (#300) Signed-off-by: Mikalai Radchuk <mradchuk@redhat.com> * Bump ginkgo CLI (#301) Signed-off-by: Mikalai Radchuk <mradchuk@redhat.com> * :seedling: Bump github.com/google/go-containerregistry (#302) Bumps [github.com/google/go-containerregistry](https://github.com/google/go-containerregistry) from 0.19.2 to 0.20.0. - [Release notes](https://github.com/google/go-containerregistry/releases) - [Changelog](https://github.com/google/go-containerregistry/blob/main/.goreleaser.yml) - [Commits](https://github.com/google/go-containerregistry/compare/v0.19.2...v0.20.0) --- updated-dependencies: - dependency-name: github.com/google/go-containerregistry dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump google.golang.org/grpc from 1.64.0 to 1.64.1 (#303) Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.64.0 to 1.64.1. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](https://github.com/grpc/grpc-go/compare/v1.64.0...v1.64.1) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * shift gzip lib to active fork (#311) Signed-off-by: Jordan Keister <jordan@nimblewidget.com> * Move all packages from /pkg to /internal (#310) Ahead of an initial 1.0.0 release, we shouldn't preemptively expose any packages publicly. Instead, let's start my making any packages internal, and we can expose them publicly later on. This commit moves everything currently in /pkg to /internal for now. Signed-off-by: kevinrizza <krizza@redhat.com> * :seedling: Bump github.com/klauspost/compress from 1.17.8 to 1.17.9 (#312) Bumps [github.com/klauspost/compress](https://github.com/klauspost/compress) from 1.17.8 to 1.17.9. - [Release notes](https://github.com/klauspost/compress/releases) - [Changelog](https://github.com/klauspost/compress/blob/master/.goreleaser.yml) - [Commits](https://github.com/klauspost/compress/compare/v1.17.8...v1.17.9) --- updated-dependencies: - dependency-name: github.com/klauspost/compress dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump github.com/google/go-containerregistry (#315) Bumps [github.com/google/go-containerregistry](https://github.com/google/go-containerregistry) from 0.20.0 to 0.20.1. - [Release notes](https://github.com/google/go-containerregistry/releases) - [Changelog](https://github.com/google/go-containerregistry/blob/main/.goreleaser.yml) - [Commits](https://github.com/google/go-containerregistry/compare/v0.20.0...v0.20.1) --- updated-dependencies: - dependency-name: github.com/google/go-containerregistry dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump github.com/operator-framework/operator-registry (#319) Bumps [github.com/operator-framework/operator-registry](https://github.com/operator-framework/operator-registry) from 1.44.0 to 1.45.0. - [Release notes](https://github.com/operator-framework/operator-registry/releases) - [Commits](https://github.com/operator-framework/operator-registry/compare/v1.44.0...v1.45.0) --- updated-dependencies: - dependency-name: github.com/operator-framework/operator-registry dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump k8s.io/apiextensions-apiserver from 0.30.2 to 0.30.3 (#320) Bumps [k8s.io/apiextensions-apiserver](https://github.com/kubernetes/apiextensions-apiserver) from 0.30.2 to 0.30.3. - [Release notes](https://github.com/kubernetes/apiextensions-apiserver/releases) - [Commits](https://github.com/kubernetes/apiextensions-apiserver/compare/v0.30.2...v0.30.3) --- updated-dependencies: - dependency-name: k8s.io/apiextensions-apiserver dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Include OperatorHub.io ClusterCatalog in releases (#321) Also adds optional instructions for adding the OperatorHub.io ClusterCatalog to a cluster. Signed-off-by: Tayler Geiger <tayler@redhat.com> * ✨ json-lines interface: set content-type, enforce format, document interface (#313) * set content-type for ndjson Signed-off-by: Jordan Keister <jordan@nimblewidget.com> * generate json-lines validation data Signed-off-by: Jordan Keister <jordan@nimblewidget.com> * added docs for interface format Signed-off-by: Jordan Keister <jordan@nimblewidget.com> --------- Signed-off-by: Jordan Keister <jordan@nimblewidget.com> * :seedling: Bump github.com/containerd/containerd from 1.7.19 to 1.7.20 (#324) Bumps [github.com/containerd/containerd](https://github.com/containerd/containerd) from 1.7.19 to 1.7.20. - [Release notes](https://github.com/containerd/containerd/releases) - [Changelog](https://github.com/containerd/containerd/blob/main/RELEASES.md) - [Commits](https://github.com/containerd/containerd/compare/v1.7.19...v1.7.20) --- updated-dependencies: - dependency-name: github.com/containerd/containerd dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Add dependabot K8s dependencies group (#322) Signed-off-by: Mikalai Radchuk <mradchuk@redhat.com> * add catalogd upgrade test (#314) Signed-off-by: Ankita Thomas <ankithom@redhat.com> * Add installation script (#325) This follows the release pattern used in operator-controller where an install.sh script is used for a multi-step installation process. The OperatorHub ClusterCatalog is also included in the release artifacts. Also adjusted the Makefile to use the install.sh for deploy. Signed-off-by: Tayler Geiger <tayler@redhat.com> * Update run-latest-release Makefile target (#326) * 🌱 Revert 2 previous PRs (#327) * Revert "Update run-latest-release Makefile target (#326)" This reverts commit 74de15aa8e50e0764eb7f97c1a667dc41b471a9a. * Revert "Add installation script (#325)" This reverts commit bad488ef1e8245db4e2ca7bb90b6ef7a7a3e4f63. * Add installation script to releases (#328) This follows the release pattern used in operator-controller where an install.sh script is used for a multi-step installation process. The OperatorHub ClusterCatalog is also included in the release artifacts. Also adjusted the Makefile to use the install.sh for deploy. Signed-off-by: Tayler Geiger <tayler@redhat.com> * Edit Makefile targets to use install.sh (#329) Adjusts several Makefile targets to use the new install.sh script. Includes adjustments to test-upgrade-e2e to use a new target that only installs the catalogd manifest so cert-manager can still be installed first for use in the image registry. Signed-off-by: Tayler Geiger <tayler@redhat.com> * capturing gzip catalog demo as asciicast (#323) Signed-off-by: Jordan Keister <jordan@nimblewidget.com> * :seedling: Bump github.com/onsi/gomega from 1.33.1 to 1.34.0 (#332) Bumps [github.com/onsi/gomega](https://github.com/onsi/gomega) from 1.33.1 to 1.34.0. - [Release notes](https://github.com/onsi/gomega/releases) - [Changelog](https://github.com/onsi/gomega/blob/master/CHANGELOG.md) - [Commits](https://github.com/onsi/gomega/compare/v1.33.1...v1.34.0) --- updated-dependencies: - dependency-name: github.com/onsi/gomega dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump github.com/onsi/ginkgo/v2 from 2.19.0 to 2.19.1 (#331) Bumps [github.com/onsi/ginkgo/v2](https://github.com/onsi/ginkgo) from 2.19.0 to 2.19.1. - [Release notes](https://github.com/onsi/ginkgo/releases) - [Changelog](https://github.com/onsi/ginkgo/blob/master/CHANGELOG.md) - [Commits](https://github.com/onsi/ginkgo/compare/v2.19.0...v2.19.1) --- updated-dependencies: - dependency-name: github.com/onsi/ginkgo/v2 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump github.com/onsi/gomega from 1.34.0 to 1.34.1 (#333) Bumps [github.com/onsi/gomega](https://github.com/onsi/gomega) from 1.34.0 to 1.34.1. - [Release notes](https://github.com/onsi/gomega/releases) - [Changelog](https://github.com/onsi/gomega/blob/master/CHANGELOG.md) - [Commits](https://github.com/onsi/gomega/compare/v1.34.0...v1.34.1) --- updated-dependencies: - dependency-name: github.com/onsi/gomega dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * default-catalogs.yaml: add 10m poll (#335) * ✨ initial api audit type alignment (#330) * initial api audit Signed-off-by: Jordan Keister <jordan@nimblewidget.com> * review resolutions Signed-off-by: Jordan Keister <jordan@nimblewidget.com> --------- Signed-off-by: Jordan Keister <jordan@nimblewidget.com> * :seedling: Bump github.com/onsi/ginkgo/v2 from 2.19.1 to 2.20.0 (#338) Bumps [github.com/onsi/ginkgo/v2](https://github.com/onsi/ginkgo) from 2.19.1 to 2.20.0. - [Release notes](https://github.com/onsi/ginkgo/releases) - [Changelog](https://github.com/onsi/ginkgo/blob/master/CHANGELOG.md) - [Commits](https://github.com/onsi/ginkgo/compare/v2.19.1...v2.20.0) --- updated-dependencies: - dependency-name: github.com/onsi/ginkgo/v2 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump github.com/google/go-containerregistry (#337) Bumps [github.com/google/go-containerregistry](https://github.com/google/go-containerregistry) from 0.20.1 to 0.20.2. - [Release notes](https://github.com/google/go-containerregistry/releases) - [Changelog](https://github.com/google/go-containerregistry/blob/main/.goreleaser.yml) - [Commits](https://github.com/google/go-containerregistry/compare/v0.20.1...v0.20.2) --- updated-dependencies: - dependency-name: github.com/google/go-containerregistry dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump github.com/prometheus/client_golang (#344) Bumps [github.com/prometheus/client_golang](https://github.com/prometheus/client_golang) from 1.19.1 to 1.20.0. - [Release notes](https://github.com/prometheus/client_golang/releases) - [Changelog](https://github.com/prometheus/client_golang/blob/v1.20.0/CHANGELOG.md) - [Commits](https://github.com/prometheus/client_golang/compare/v1.19.1...v1.20.0) --- updated-dependencies: - dependency-name: github.com/prometheus/client_golang dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump the k8s-dependencies group with 6 updates (#343) * :seedling: Bump the k8s-dependencies group with 6 updates Bumps the k8s-dependencies group with 6 updates: | Package | From | To | | --- | --- | --- | | [k8s.io/api](https://github.com/kubernetes/api) | `0.30.3` | `0.31.0` | | [k8s.io/apiextensions-apiserver](https://github.com/kubernetes/apiextensions-apiserver) | `0.30.3` | `0.31.0` | | [k8s.io/apimachinery](https://github.com/kubernetes/apimachinery) | `0.30.3` | `0.31.0` | | [k8s.io/apiserver](https://github.com/kubernetes/apiserver) | `0.30.3` | `0.31.0` | | [k8s.io/client-go](https://github.com/kubernetes/client-go) | `0.30.3` | `0.31.0` | | [k8s.io/component-base](https://github.com/kubernetes/component-base) | `0.30.3` | `0.31.0` | Updates `k8s.io/api` from 0.30.3 to 0.31.0 - [Commits](https://github.com/kubernetes/api/compare/v0.30.3...v0.31.0) Updates `k8s.io/apiextensions-apiserver` from 0.30.3 to 0.31.0 - [Release notes](https://github.com/kubernetes/apiextensions-apiserver/releases) - [Commits](https://github.com/kubernetes/apiextensions-apiserver/compare/v0.30.3...v0.31.0) Updates `k8s.io/apimachinery` from 0.30.3 to 0.31.0 - [Commits](https://github.com/kubernetes/apimachinery/compare/v0.30.3...v0.31.0) Updates `k8s.io/apiserver` from 0.30.3 to 0.31.0 - [Commits](https://github.com/kubernetes/apiserver/compare/v0.30.3...v0.31.0) Updates `k8s.io/client-go` from 0.30.3 to 0.31.0 - [Changelog](https://github.com/kubernetes/client-go/blob/master/CHANGELOG.md) - [Commits](https://github.com/kubernetes/client-go/compare/v0.30.3...v0.31.0) Updates `k8s.io/component-base` from 0.30.3 to 0.31.0 - [Commits](https://github.com/kubernetes/component-base/compare/v0.30.3...v0.31.0) --- updated-dependencies: - dependency-name: k8s.io/api dependency-type: direct:production update-type: version-update:semver-minor dependency-group: k8s-dependencies - dependency-name: k8s.io/apiextensions-apiserver dependency-type: direct:production update-type: version-update:semver-minor dependency-group: k8s-dependencies - dependency-name: k8s.io/apimachinery dependency-type: direct:production update-type: version-update:semver-minor dependency-group: k8s-dependencies - dependency-name: k8s.io/apiserver dependency-type: direct:production update-type: version-update:semver-minor dependency-group: k8s-dependencies - dependency-name: k8s.io/client-go dependency-type: direct:production update-type: version-update:semver-minor dependency-group: k8s-dependencies - dependency-name: k8s.io/component-base dependency-type: direct:production update-type: version-update:semver-minor dependency-group: k8s-dependencies ... Signed-off-by: dependabot[bot] <support@github.com> * Bump controller-runtime from 0.18.4 to 0.19.0 Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> --------- Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Per Goncalves da Silva <pegoncal@redhat.com> * :seedling: change GVK to use olm.operatorframework.io (#342) Signed-off-by: Jordan Keister <jordan@nimblewidget.com> * Removing the catalogd from app.kubernetes.io/name (#347) As this namespace is shared between catalogd operator-framework-catalogd Fixes issue #346 Signed-off-by: Lalatendu Mohanty <lmohanty@redhat.com> * :seedling: Bump github.com/operator-framework/operator-registry from 1.45.0 to 1.46.0 (#345) * :seedling: Bump github.com/operator-framework/operator-registry Bumps [github.com/operator-framework/operator-registry](https://github.com/operator-framework/operator-registry) from 1.45.0 to 1.46.0. - [Release notes](https://github.com/operator-framework/operator-registry/releases) - [Commits](https://github.com/operator-framework/operator-registry/compare/v1.45.0...v1.46.0) --- updated-dependencies: - dependency-name: github.com/operator-framework/operator-registry dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * bump controller-gen to v0.16.1 Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> --------- Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Per Goncalves da Silva <pegoncal@redhat.com> * ✨ Capture catalog content timestamp in ClusterCatalogStatus (#334) * Capture catalog content timestamp in ClusterCatalogStatus Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com> * address reviews (partial) Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com> * address review feedback Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com> * Address review comments Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com> --------- Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com> * :seedling: Bump github.com/onsi/ginkgo/v2 from 2.20.0 to 2.20.1 (#350) Bumps [github.com/onsi/ginkgo/v2](https://github.com/onsi/ginkgo) from 2.20.0 to 2.20.1. - [Release notes](https://github.com/onsi/ginkgo/releases) - [Changelog](https://github.com/onsi/ginkgo/blob/master/CHANGELOG.md) - [Commits](https://github.com/onsi/ginkgo/compare/v2.20.0...v2.20.1) --- updated-dependencies: - dependency-name: github.com/onsi/ginkgo/v2 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump github.com/prometheus/client_golang (#351) Bumps [github.com/prometheus/client_golang](https://github.com/prometheus/client_golang) from 1.20.0 to 1.20.2. - [Release notes](https://github.com/prometheus/client_golang/releases) - [Changelog](https://github.com/prometheus/client_golang/blob/main/CHANGELOG.md) - [Commits](https://github.com/prometheus/client_golang/compare/v1.20.0...v1.20.2) --- updated-dependencies: - dependency-name: github.com/prometheus/client_golang dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump github.com/containerd/containerd from 1.7.20 to 1.7.21 (#352) Bumps [github.com/containerd/containerd](https://github.com/containerd/containerd) from 1.7.20 to 1.7.21. - [Release notes](https://github.com/containerd/containerd/releases) - [Changelog](https://github.com/containerd/containerd/blob/main/RELEASES.md) - [Commits](https://github.com/containerd/containerd/compare/v1.7.20...v1.7.21) --- updated-dependencies: - dependency-name: github.com/containerd/containerd dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Catalog Priority (#353) Adds an int32 field to allow users to optionally set priority of catalogs. This priority value may be taken into account during bundle selection from multiple catalogs to remove ambiguity. Signed-off-by: dtfranz <dfranz@redhat.com> * add v1.0 and v1.x issues to OLMv1 project automatically (#354) Signed-off-by: Joe Lanford <joe.lanford@gmail.com> * :seedling: Bump bingo + tools (#358) * Add bingo-upgrade target to Makefile Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> * Upgrade bingo tools Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> * Regenerate manifests Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> --------- Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: Per Goncalves da Silva <pegoncal@redhat.com> * bugfix: fix printcolumn for lastUnpacked based on json tag and not field name (#361) Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com> * :seedling: Bump github.com/onsi/ginkgo/v2 from 2.20.1 to 2.20.2 (#360) Bumps [github.com/onsi/ginkgo/v2](https://github.com/onsi/ginkgo) from 2.20.1 to 2.20.2. - [Release notes](https://github.com/onsi/ginkgo/releases) - [Changelog](https://github.com/onsi/ginkgo/blob/master/CHANGELOG.md) - [Commits](https://github.com/onsi/ginkgo/compare/v2.20.1...v2.20.2) --- updated-dependencies: - dependency-name: github.com/onsi/ginkgo/v2 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * :seedling: Bump github.com/onsi/gomega from 1.34.1 to 1.34.2 (#359) Bumps [github.com/onsi/gomega](https://github.com/onsi/gomega) from 1.34.1 to 1.34.2. - [Release notes](https://github.com/onsi/gomega/releases) - [Changelog](https://github.com/onsi/gomega/blob/master/CHANGELOG.md) - [Commits](https://github.com/onsi/gomega/compare/v1.34.1...v1.34.2) --- updated-dependencies: - dependency-name: github.com/onsi/gomega dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Update cert-manager to v1.15.3 (#367) Signed-off-by: Todd Short <todd.short@me.com> * Remove status.phase field from the code and doc references (#368) Signed-off-by: Todd Short <todd.short@me.com> * 🌱 Add webhook to add a name label to ClusterCatalogs (#356) * Add webhook to name label ClusterCatalogs * Linter fixups * Added bit more webhook test coverage * Use catalogd version & clean up TLS args Signed-off-by: Brett Tofel <btofel@redhat.com> * Fix label name Signed-off-by: Brett Tofel <btofel@redhat.com> * From stand-alone webhook to operator internal Signed-off-by: Brett Tofel <btofel@redhat.com> * Think this fixes e2e-upgrade not sure why PR changes might necessitate this, but it seems legit to delete deployment first Signed-off-by: Brett Tofel <btofel@redhat.com> * PR review changes so far... - get MutaingWebhookConfig from kubebuilder; - stop deleting deployment on only-deploy; - use the existing matchLabel, stop adding webhook label to operator; - rename the name label to metadata.name (but not in API yet). Signed-off-by: Brett Tofel <btofel@redhat.com> * Restore CA issuer config Signed-off-by: Brett Tofel <btofel@redhat.com> * Fix unit tests Signed-off-by: Brett Tofel <btofel@redhat.com> * Addresses review comments around TLS cert config Signed-off-by: Brett Tofel <btofel@redhat.com> * Move RBAC for webhook to kb directive Signed-off-by: Brett Tofel <btofel@redhat.com> * Move allow-direct-injection to secretTemplate Signed-off-by: Brett Tofel <btofel@redhat.com> * Make metadata.name an API constant Signed-off-by: Brett Tofel <btofel@redhat.com> * Use MetaDataNameLabel API constant in log line Signed-off-by: Brett Tofel <btofel@redhat.com> * Combine cert watchers Signed-off-by: Brett Tofel <btofel@redhat.com> * Move cert watching back to ctr-rt manager Signed-off-by: Brett Tofel <btofel@redhat.com> * Address review comments Signed-off-by: Brett Tofel <btofel@redhat.com> * Address review comments 2 Signed-off-by: Brett Tofel <btofel@redhat.com> --------- Signed-off-by: Brett Tofel <btofel@redhat.com> * Move cert-manager.io patch into tls component (#372) Signed-off-by: Todd Short <todd.short@me.com> * :seedling: Bump github.com/prometheus/client_golang (#375) Bumps [github.com/prometheus/client_golang](https://github.com/prometheus/client_golang) from 1.20.2 to 1.20.3. - [Release notes](https://github.com/prometheus/client_golang/releases) - [Changelog](https://github.com/prometheus/client_golang/blob/v1.20.3/CHANGELOG.md) - [Commits](https://github.com/prometheus/client_golang/compare/v1.20.2...v1.20.3) --- updated-dependencies: - dependency-name: github.com/prometheus/client_golang dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Add label creation checking in the e2e (#374) Signed-off-by: Todd Short <todd.short@me.com> * Add matchCondition to webhook config (#376) Should add some efficiency as it only lets the webhook fire when needed Signed-off-by: Brett Tofel <btofel@redhat.com> * Fix matchConditions to be compatible with GenerateName (#382) Signed-off-by: Todd Short <todd.short@me.com> * Force a rename of the catalogd certificate (#386) This will cause the catalogd deployment to restart, meaning that the catalogd will update quicker (rather than waiting on k8s to update the certificates via the volume) Signed-off-by: Todd Short <todd.short@me.com> * :seedling: Bump github.com/operator-framework/operator-registry (#387) Bumps [github.com/operator-framework/operator-registry](https://github.com/operator-framework/operator-registry) from 1.46.0 to 1.47.0. - [Release notes](https://github.com/operator-framework/operator-registry/releases) - [Commits](https://github.com/operator-framework/operator-registry/compare/v1.46.0...v1.47.0) --- updated-dependencies: - dependency-name: github.com/operator-framework/operator-registry dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * 📖 [Docs] Document tilt-support install (#388) * Document tilt-support install Signed-off-by: Brett Tofel <btofel@redhat.com> * Improve tilt-support text Signed-off-by: Brett Tofel <btofel@redhat.com> --------- Signed-off-by: Brett Tofel <btofel@redhat.com> * :seedling: Bump github.com/containerd/containerd from 1.7.21 to 1.7.22 (#391) Bumps [github.com/containerd/containerd](https://github.com/containerd/containerd) from 1.7.21 to 1.7.22. - [Release notes](https://github.com/containerd/containerd/releases) - [Changelog](https://github.com/containerd/containerd/blob/main/RELEASES.md) - [Commits](https://github.com/containerd/containerd/compare/v1.7.21...v1.7.22) --- updated-dependencies: - dependency-name: github.com/containerd/containerd dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * ✨ Ensure docker registry CA is trusted in e2e tests (#377) * Ensure docker registry CA is trusted in e2e tests * Restructure based on feedback * Fix FS cache for image refs with tags (#394) Signed-off-by: Mikalai Radchuk <mradchuk@redhat.com> * Fixes name of httputil test file (#395) Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: Per Goncalves da Silva <pegoncal@redhat.com> * Add .idea to .gitignore (#393) Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: Per Goncalves da Silva <pegoncal@redhat.com> * Remove TLSVerify/PullSecret fields (#369) Remove fields that will not have long term support committed to them to prepare for the v1.0 release. Removed Fields: - spec.source.image.insecureSkipTLSVerify - spec.source.image.pullSecret GH Issue Ref: #355 Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> * Avoid a network call for digest based images (#396) Signed-off-by: Mikalai Radchuk <mradchuk@redhat.com> * :seedling: Bump the k8s-dependencies group with 6 updates (#398) Bumps the k8s-dependencies group with 6 updates: | Package | From | To | | --- | --- | --- | | [k8s.io/api](https://github.com/kubernetes/api) | `0.31.0` | `0.31.1` | | [k8s.io/apiextensions-apiserver](https://github.com/kubernetes/apiextensions-apiserver) | `0.31.0` | `0.31.1` | | [k8s.io/apimachinery](https://github.com/kubernetes/apimachinery) | `0.31.0` | `0.31.1` | | [k8s.io/apiserver](https://github.com/kubernetes/apiserver) | `0.31.0` | `0.31.1` | | [k8s.io/client-go](https://github.com/kubernetes/client-go) | `0.31.0` | `0.31.1` | | [k8s.io/component-base](https://github.com/kubernetes/component-base) | `0.31.0` | `0.31.1` | Updates `k8s.io/api` from 0.31.0 to 0.31.1 - [Commits](https://github.com/kubernetes/api/compare/v0.31.0...v0.31.1) Updates `k8s.io/apiextensions-apiserver` from 0.31.0 to 0.31.1 - [Release notes](https://github.com/kubernetes/apiextensions-apiserver/releases) - [Commits](https://github.com/kubernetes/apiextensions-apiserver/compare/v0.31.0...v0.31.1) Updates `k8s.io/apimachinery` from 0.31.0 to 0.31.1 - [Commits](https://github.com/kubernetes/apimachinery/compare/v0.31.0...v0.31.1) Updates `k8s.io/apiserver` from 0.31.0 to 0.31.1 - [Commits](https://github.com/kubernetes/apiserver/compare/v0.31.0...v0.31.1) Updates `k8s.io/client-go` from 0.31.0 to 0.31.1 - [Changelog](https://github.com/kubernetes/client-go/blob/master/CHANGELOG.md) - [Commits](https://github.com/kubernetes/client-go/compare/v0.31.0...v0.31.1) Updates `k8s.io/component-base` from 0.31.0 to 0.31.1 - [Commits](https://github.com/kubernetes/component-base/compare/v0.31.0...v0.31.1) --- updated-dependencies: - dependency-name: k8s.io/api dependency-type: direct:production update-type: version-update:semver-patch dependency-group: k8s-dependencies - dependency-name: k8s.io/apiextensions-apiserver dependency-type: direct:production update-type: version-update:semver-patch dependency-group: k8s-dependencies - dependency-name: k8s.io/apimachinery dependency-type: direct:production update-type: version-update:semver-patch dependency-group: k8s-dependencies - dependency-name: k8s.io/apiserver dependency-type: direct:production update-type: version-update:semver-patch dependency-group: k8s-dependencies - dependency-name: k8s.io/client-go dependency-type: direct:production update-type: version-update:semver-patch dependency-group: k8s-dependencies - dependency-name: k8s.io/component-base dependency-type: direct:production update-type: version-update:semver-patch dependency-group: k8s-dependencies ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Fix unnecessary lastUnpacked status updates (#397) We should not be updating lastUnpacked status fields when we read from the cache Signed-off-by: Mikalai Radchuk <mradchuk@redhat.com> * ✨ unpacker: switch from google/go-containerregistry to containers/image (#399) * copy containers_image.go from operator-controller Signed-off-by: Joe Lanford <joe.lanford@gmail.com> * initial modifications for naming and types Signed-off-by: Joe Lanford <joe.lanford@gmail.com> * unpacker: switch to containers/image-based implementation Signed-off-by: Joe Lanford <joe.lanford@gmail.com> --------- Signed-off-by: Joe Lanford <joe.lanford@gmail.com> * :seedling: Align ClusterCatalog API godoc with OpenShift API conventions (#390) * Update godoc strings for ClusterCatalog API Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> * Regenerate manifests Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> * Apply suggestions from code review Text fixes Co-authored-by: Bryce Palmer <everettraven@gmail.com> * Address reviewer comments Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> * Update api/core/v1alpha1/clustercatalog_types.go Co-authored-by: Bryce Palmer <everettraven@gmail.com> * Apply suggestions from code review Co-authored-by: Joe Lanford <joe.lanford@gmail.com> Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> --------- Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: Bryce Palmer <everettraven@gmail.com> Co-authored-by: Joe Lanford <joe.lanford@gmail.com> * Add CI to check if golang version updated (#401) Signed-off-by: Todd Short <todd.short@me.com> * Combine the catalogd services (#402) Only need to define one service for: * metrics * webhook * catalogd itself Clean up the name of the service (now `catalogd-service`) and any corresponding resources Renumber the ports to 7443/8443/9443. The external port for the catalog server is either 80 or 443. Signed-off-by: Todd Short <todd.short@me.com> * Update documentation and default to new service name. (#406) Signed-off-by: Todd Short <todd.short@me.com> * fix: make post-upgrade-e2e checks wait for ClusterCatalog to be reconciled (#407) Signed-off-by: Joe Lanford <joe.lanford@gmail.com> * fix: unpack cache cleanup should ignore missing files (#404) Signed-off-by: Joe Lanford <joe.lanford@gmail.com> * use controller-runtime Terminal error instead of custom Unrecoverable error (#405) Signed-off-by: Joe Lanford <joe.lanford@gmail.com> * Refer to the correct location for go-verdiff (#408) The location is different than the operator-controller repo. Signed-off-by: Todd Short <todd.short@me.com> * :seedling: Bump github.com/prometheus/client_golang (#403) Bumps [github.com/prometheus/client_golang](https://github.com/prometheus/client_golang) from 1.20.3 to 1.20.4. - [Release notes](https://github.com/prometheus/client_golang/releases) - [Changelog](https://github.com/prometheus/client_golang/blob/main/CHANGELOG.md) - [Commits](https://github.com/prometheus/client_golang/compare/v1.20.3...v1.20.4) --- updated-dependencies: - dependency-name: github.com/prometheus/client_golang dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * 🐛 Handle finalizers (creation and deletion) better (#411) * 🐛 Handle finalizers (creation and deletion) better Closes [409](https://github.com/operator-framework/catalogd/issues/409) * Move the finalizer setup Move the finalizer setup out from main.go, and into the controller code. This allows the finalizers to be initialized and used in the test code. Signed-off-by: Todd Short <todd.short@me.com> * Undo rename of fbcDeletionFinalizer Signed-off-by: Todd Short <todd.short@me.com> --------- Signed-off-by: Todd Short <todd.short@me.com> Co-authored-by: Todd Short <todd.short@me.com> * ⚠ Status condition cl…

openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 29, 2023

jmprusi force-pushed the jmprusi/test_logs branch 2 times, most recently from 790d618 to 98e238d Compare May 30, 2023 13:39

jmprusi marked this pull request as ready for review May 30, 2023 13:56

openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 30, 2023

jmprusi changed the title ~~Expose logs/resources after test run~~ Testing: Save pod logs and resources after e2e test run May 30, 2023