⚠️ [Monorepo] PHASE 1 - Add Catalogd to Operator-Controller Repository by camilamacedo86 · Pull Request #1542 · operator-framework/operator-controller

and others added 30 commits

July 28, 2023 11:55
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>
Signed-off-by: Andy Goldstein <andy.goldstein@redhat.com>
Signed-off-by: Andy Goldstein <andy.goldstein@redhat.com>
Signed-off-by: Andy Goldstein <andy.goldstein@redhat.com>
Signed-off-by: Andy Goldstein <andy.goldstein@redhat.com>
Signed-off-by: Andy Goldstein <andy.goldstein@redhat.com>
Signed-off-by: Andy Goldstein <andy.goldstein@redhat.com>
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>
Signed-off-by: Joe Lanford <joe.lanford@gmail.com>
Signed-off-by: Joe Lanford <joe.lanford@gmail.com>
Signed-off-by: Jordan Keister <jordan@nimblewidget.com>
* 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>
…work#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>
…erator-framework#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>
Signed-off-by: Joe Lanford <joe.lanford@gmail.com>
Add release guide

Signed-off-by: Catherine Chan-Tse <cchantse@redhat.com>
* 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>
Signed-off-by: kevinrizza <krizza@redhat.com>
Add default printer columns

Signed-off-by: kevinrizza <krizza@redhat.com>
…operator-framework#145)

(feat): direct image registry client unpacker

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
…k#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>
Bumps [golang.org/x/net](https://github.com/golang/net) from 0.10.0 to 0.17.0.
- [Commits](golang/net@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>
GoDoc for Phase

Signed-off-by: Anik Bhattacharjee <anbhatta@redhat.com>

@camilamacedo86

go: upgraded golang.org/x/crypto v0.31.0 => v0.32.0
go: upgraded golang.org/x/net v0.30.0 => v0.34.0
go: upgraded golang.org/x/sys v0.28.0 => v0.29.0
go: upgraded golang.org/x/term v0.27.0 => v0.28.0

LalatenduMohanty

Signed-off-by: Camila Macedo <7708031+camilamacedo86@users.noreply.github.com>
Signed-off-by: Camila Macedo <7708031+camilamacedo86@users.noreply.github.com>
Signed-off-by: Camila Macedo <7708031+camilamacedo86@users.noreply.github.com>
Signed-off-by: Camila Macedo <7708031+camilamacedo86@users.noreply.github.com>
Signed-off-by: Camila Macedo <7708031+camilamacedo86@users.noreply.github.com>
- Removed temporary files (`Makefile.orig` and `Makefile.rej`) generated by scripts.
- These files are not required and are cleaned up to maintain a tidy repository.

@camilamacedo86

- Consolidated YAML for both projects into operator-controller.yaml.
- Updated the install script to utilize the unified YAML file.

Co-authored-by: Joe Lanford <joe.lanford@gmail.com>
…uplicates

- Relocated catalogd-specific GitHub actions to the root `.github/workflows` directory.
- Integrated actions to perform catalogd-specific checks at the monorepo level for consistency and centralized management.
- Removed duplicate GitHub actions already present in the operator-controller, ensuring no redundancy.
- Deleted the following obsolete files from the catalogd directory:
  - `.github/dependabot.yml`
  - `.github/pull_request_template.md`
  - `.github/workflows/add-to-project.yaml`
  - `.github/workflows/go-apidiff.yaml`
  - `.github/workflows/go-verdiff.yaml`
  - `.github/workflows/pr-title.yaml`
  - `.github/workflows/release.yaml`
…es not exist

- Resolved failure in the catalogd-crd-diff CI check caused by missing CRDs in the new repository.
- Improved the CI and Makefile target to gracefully skip execution when the original CRD does not exist.

Note: This is a temporary solution, and we may need to revisit and refine this check in the future.
- Unified linting configuration for both projects, now defined in the root directory.
- Addressed Catalogd test file linting issues:
  - Replaced `assert.Nil(t, err)` with `require.NoError(t, err)` where applicable.
  - Replaced `assert.NotNil(t, err)` with `require.Error(t, err)` where appropriate.
- Fixed import sorting ensuring all files have properly organized imports for the lint does not fail with the issue (File is not -ed). Note this was caused by the replace `catalogd "github.com/operator-framework/operator-controller/catalogd/api/v1"` with `catalogd "github.com/operator-framework/operator-controller/catalogd/api/v1"`

LalatenduMohanty

LalatenduMohanty

tmshort

OchiengEd

- Consolidated the release process to use `operator-controller.yaml` for all requirements.
- Removed `catalogd.yaml` as it is no longer necessary.
- Simplified the release action:
  - A single release action now generates images and binaries for both projects.
  - Unified process ensures consistent release management across the monorepo.
- Updated the root `Makefile` with a unified `test-unit` target to execute unit tests for the entire project.
- Removed the `test-unit` target from the `Makefile` under the `catalogd` directory as it is no longer needed.
- Added `.tilt-support` directory at the root, updated to work with both `operator-controller` and `catalogd`.
- Removed the Tilt workflow from the `catalogd` directory, consolidating all configurations into the root Tilt setup.
- Updated configurations to use `catalogd` from the source code instead of fetching it from `github.com/operator-framework/catalogd`.

Co-authored-by: Joe Lanford <joe.lanford@gmail.com>
- Removed `check-go-version.sh` from `catalogd/hack/scripts` as it is no longer needed.
  - Both projects now rely on the `go.mod` file at the root directory for Go version checks.

- Removed duplicated files from the repository:
  - `CODEOWNERS`
  - `CONTRIBUTING.md`
  - `DOC`
  - `LICENSE.md`

- Use hack/boilerplate.go.txt from root and remove duplication
- Standardized Bingo definitions across `catalogd` and `operator-controller`.
- Ensured both projects use the same Bingo config to maintain consistency

camilamacedo86

camilamacedo86

LalatenduMohanty

OchiengEd