community icon indicating copy to clipboard operation
community copied to clipboard

Errors while generating olm bundle for `bedrockagent-controller-v0.0.2`

Open ack-bot opened this issue 7 months ago • 4 comments

Errors while generating olm bundle for bedrockagent-controller-v0.0.2

stdout for olm-create-bundle.sh bedrockagent v0.0.2:

Building release artifacts for bedrockagent-v0.0.2
Generating common custom resource definitions
Generating custom resource definitions for bedrockagent
Generating RBAC manifests for bedrockagent
Generating operator lifecycle manager bundle assets for bedrockagent
Generating bundle version 0.0.2
Generating bundle manifests

stderr for olm-create-bundle.sh bedrockagent v0.0.2:

time="2025-05-30T19:27:38Z" level=fatal msg="Error generating bundle manifests: error reading CSV base: error reading existing ClusterServiceVersion base /home/prow/go/src/github.com/aws-controllers-k8s/bedrockagent-controller/config/manifests/bases/ack-bedrockagent-controller.clusterserviceversion.yaml: no ClusterServiceVersion manifest in /home/prow/go/src/github.com/aws-controllers-k8s/bedrockagent-controller/config/manifests/bases/ack-bedrockagent-controller.clusterserviceversion.yaml"

Steps for closing this issue:

  1. Checkout code-generator and bedrockagent-controller repositories
  2. From code-generator repo, execute ./scripts/olm-create-bundle.sh bedrockagent v0.0.2
  3. Checkout community-operators and community-operators-prod repositories
  4. Copy manifests, metadata and tests directories from bedrockagent-controller/olm/bundle/ into community-operators/operators/ack-bedrockagent-controller/0.0.2
  5. Copy bundle.Dockerfile from bedrockagent-controller/olm/ into community-operators/operators/ack-bedrockagent-controller/0.0.2
  6. Create a new pull request with these changes for community-operators
  7. Copy contents from bedrockagent-controller/olm/bundle/ into community-operators-prod/operators/ack-bedrockagent-controller/0.0.2
  8. Copy bundle.Dockerfile from bedrockagent-controller/olm/ into community-operators-prod/operators/ack-bedrockagent-controller/0.0.2
  9. Create a new pull request with these changes for community-operators-prod
  10. Mention this issue number in the both pull requests
  11. When both PRs are merged, close this issue

For more guidance, create an issue or reach out on Slack.

ack-bot avatar May 27 '25 17:05 ack-bot

Hello @ack-bot 👋 Thank you for opening an issue in ACK! A maintainer will triage this issue soon.

We encourage community contributions, so if you're interested in tackling this yourself or suggesting a solution, please check out our Contribution and Code of Conduct guidelines.

You can find more information about ACK on our website.

github-actions[bot] avatar May 27 '25 17:05 github-actions[bot]

@acornett21: It looks like this is the same error seen in this eventbridge issue. However, after filling out the olmconfig.yaml to include the details added to the eventbridge's olmconfig.yaml I'm still seeing the same error (at least locally). I wondering if you might have any insights?

knottnt avatar May 29 '25 17:05 knottnt

Hi @knottnt It seems that ack-generate runs alot slower then it used to, making troubleshooting very hard, do you know why it's so slow and is using 150% CPU for ~12 mins?

Without being able to test some thoughts in the olm file

  • the 's
  • the in the url (which seems to be incorrect i believe what you want is https://aws.amazon.com/bedrock/resources/

It's the 's you can see this in the rendered out /config/manifests/bases/*csv.yaml file, since the description is already '' this breaks the description block making the yaml invalid.

acornett21 avatar May 29 '25 22:05 acornett21

@acornett21 Thank you very much for the help! Will add those fixes.

As far as the ack-generate performance. I don't know the specific cause of the slowdown, but the team is aware of it.

knottnt avatar May 30 '25 16:05 knottnt

@acornett21: The both the community-operators PR associated with this issue and a PR for a newer release of the bedrockagent-controller have been open with all checks passing for quite awhile. Is there anything on our end that needs to happen for those PRs to be merged?

knottnt avatar Jun 30 '25 19:06 knottnt

@knottnt It looks like the maintainers of those repos added the necessary labels and the PR's have been approved/merged/etc.

acornett21 avatar Jul 07 '25 15:07 acornett21

@acornett21 Great, thanks for the help!

knottnt avatar Jul 07 '25 16:07 knottnt