Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OPRUN-3076: Update to Upstream v0.7.0 #31

Merged
merged 27 commits into from Oct 31, 2023

Conversation

dtfranz
Copy link
Contributor

@dtfranz dtfranz commented Oct 26, 2023

Currently released operator-controller does not yet use catalogd@v0.8.0 so keeping this at the currently vendored version for now.

everettraven and others added 22 commits July 18, 2023 15:09
to ensure we only list resources belonging to the 
`Catalog` resource currently being reconciled.
Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
Fix link and misspelling

The operator-controller docs link was 404ing, now it links to the
correct folder.

Signed-off-by: Justin Kulikauskas <jkulikau@redhat.com>
Signed-off-by: Andy Goldstein <andy.goldstein@redhat.com>
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>
closes #113

Signed-off-by: Anik <anikbhattacharya93@gmail.com>
Signed-off-by: Jordan Keister <jordan@nimblewidget.com>
Closes #113

Signed-off-by: Anik <anikbhattacharya93@gmail.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>
* (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

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>
closes #119

Signed-off-by: Anik <anikbhattacharya93@gmail.com>
@openshift-ci 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 Oct 26, 2023
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 26, 2023

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@dtfranz
Copy link
Contributor Author

dtfranz commented Oct 26, 2023

/test unit

@dtfranz dtfranz marked this pull request as ready for review October 27, 2023 00:05
@openshift-ci 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 Oct 27, 2023
@dtfranz
Copy link
Contributor Author

dtfranz commented Oct 27, 2023

/test all

Signed-off-by: dtfranz <dfranz@redhat.com>
@everettraven
Copy link
Contributor

/test openshift-e2e-aws

dtfranz and others added 4 commits October 30, 2023 15:10
Signed-off-by: dtfranz <dfranz@redhat.com>
Signed-off-by: dtfranz <dfranz@redhat.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>
…g e2e

Signed-off-by: Bryce Palmer <everettraven@gmail.com>
Copy link
Contributor

@everettraven everettraven left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@everettraven
Copy link
Contributor

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Oct 31, 2023
@ncdc
Copy link
Contributor

ncdc commented Oct 31, 2023

/lgtm
/approve

/override ci/prow/verify-commits
/override ci/prow/verify-commits-shared
(these never pass on rebase PRs)

Copy link
Contributor

openshift-ci bot commented Oct 31, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dtfranz, everettraven, ncdc

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 31, 2023
Copy link
Contributor

openshift-ci bot commented Oct 31, 2023

@ncdc: Overrode contexts on behalf of ncdc: ci/prow/verify-commits, ci/prow/verify-commits-shared

In response to this:

/lgtm
/approve

/override ci/prow/verify-commits
/override ci/prow/verify-commits-shared
(these never pass on rebase PRs)

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Copy link
Contributor

openshift-ci bot commented Oct 31, 2023

@dtfranz: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@openshift-ci openshift-ci bot merged commit db893cc into openshift:main Oct 31, 2023
10 checks passed
@dtfranz dtfranz changed the title Update to Upstream v0.7.0 OPRUN-3076: Update to Upstream v0.7.0 Nov 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants