Skip to content

Conversation

@petr-muller
Copy link
Member

@petr-muller petr-muller commented Aug 9, 2023

Just ran an existing Makefile target /shrug:

$ make bundle VERSION=5.0.2-dev
cd config/manager && /usr/bin/kustomize edit set image controller=controller:latest
/usr/bin/kustomize build config/manifests | operator-sdk generate bundle -q --overwrite --version 5.0.2-dev --channels=v1 --default-channel=v1
INFO[0000] Creating bundle.Dockerfile                   
INFO[0000] Creating bundle/metadata/annotations.yaml    
INFO[0000] Bundle metadata generated successfully       
operator-sdk bundle validate ./bundle
INFO[0000] All validation tests have completed successfully

The bundle will not work by itself: we are not publishing the operator images anywhere (except the productized images ofc) and we are not publishing cincinnati images anywhere suitable either (quay.io/cincinnati/cincinnati:latest is stale and quay.io/app-sre/cincinnati does not have floating or even version tags).

So for now this work should just support building the bundles from pull request code for the purpose of testing operator and operand PRs in CI and by QE, see openshift/release#42053

Just ran an existing `Makefile` target /shrug:

```console
$ make bundle VERSION=5.0.2-dev
cd config/manager && /usr/bin/kustomize edit set image controller=controller:latest
/usr/bin/kustomize build config/manifests | operator-sdk generate bundle -q --overwrite --version 5.0.1 --channels=v1 --default-channel=v1
INFO[0000] Creating bundle.Dockerfile
INFO[0000] Creating bundle/metadata/annotations.yaml
INFO[0000] Bundle metadata generated successfully
operator-sdk bundle validate ./bundle
INFO[0000] All validation tests have completed successfully
```
@openshift-ci openshift-ci bot requested review from PratikMahajan and wking August 9, 2023 15:54
@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 9, 2023
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Aug 9, 2023

@petr-muller: all tests passed!

Full PR test history. Your PR dashboard.

Details

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.

Copy link
Contributor

@PratikMahajan PratikMahajan left a comment

Choose a reason for hiding this comment

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

/lgtm

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

openshift-ci bot commented Aug 10, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: petr-muller, PratikMahajan

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

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [PratikMahajan,petr-muller]

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

@openshift-merge-robot openshift-merge-robot merged commit 4a4b0c0 into openshift:master Aug 10, 2023
@petr-muller petr-muller changed the title Include bundle manifests and Dockerfile OTA-1007: Include bundle manifests and Dockerfile Aug 11, 2023
wking added a commit to wking/cincinnati-operator that referenced this pull request Nov 22, 2023
Generated with:

  $ cp config/crd/bases/updateservice.operator.openshift.io_updateservices.yaml bundle/manifests/updateservice.operator.openshift.io_updateservices.yaml

to catch up with 6b266d2 (config: Regenerate UpdateService CRD,
2023-10-03, openshift#176).  24c7382 (Include bundle manifests and
Dockerfile, 2023-08-08, openshift#173) suggests I could have done this with
'make bundle VERSION=5.0.3-dev' or some such, but I don't have
operator-sdk installed at the moment.
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.

3 participants