Skip to content

Conversation

vincent-pli
Copy link
Contributor

With original command, will hit error:

The directory /root/operator-sdk-samples/go/memcached-operator/deploy/olm-catalog/memcached-operator contains no yaml files

The opm version is v1.21.0

@openshift-ci openshift-ci bot requested review from benluddy and njhale April 26, 2022 07:12
@openshift-ci
Copy link

openshift-ci bot commented Apr 26, 2022

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: vincent-pli
To complete the pull request process, please assign perdasilva after the PR has been reviewed.
You can assign the PR to them by writing /assign @perdasilva in a comment when ready.

The full list of commands accepted by this bot can be found 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


```bash
$ opm alpha bundle generate --directory /etcd --package etcd --channels stable --default stable
$ opm alpha bundle generate --directory ./etcd --package etcd/manifests --channels stable --default stable
Copy link
Contributor

Choose a reason for hiding this comment

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

The input for --package is actually correct here. The input is supposed to be the name of the package, I.e etcd and not the location of the manifests.

As for the value for --directory, it does mention this command is in relation to the bundle example mentioned above in the doc, but I see the value of adding the . before /etcd to be absolutely specific.

@grokspawn grokspawn added the lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. label Sep 11, 2024
@perdasilva perdasilva force-pushed the correct-opm-usage branch 2 times, most recently from 78a6220 to 733287c Compare November 25, 2024 10:07
@perdasilva perdasilva merged commit cfa8043 into operator-framework:master Nov 25, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants