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

Add catalog index file and script to generate it #82

Merged
merged 1 commit into from
Aug 1, 2024

Conversation

OlivierCazade
Copy link
Contributor

The script generate a catalog index file using the current bundle.

The bundle must be reachable from a repository, it can not only be built localy.

@anfredette anfredette added the hold label Aug 1, 2024
@anfredette
Copy link
Contributor

@OlivierCazade, you'll need to sign off on your commit to pass the DCO checks.

Makefile Outdated
@@ -361,6 +361,10 @@ ifneq ($(origin CATALOG_BASE_IMG), undefined)
FROM_INDEX_OPT := --from-index $(CATALOG_BASE_IMG)
endif

.PHONY: catalog-update
catalog-update:
Copy link
Contributor

Choose a reason for hiding this comment

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

pls add ## Generate catalog yaml for konflux or something like that to show in
make help

@@ -5,9 +5,9 @@ ENTRYPOINT ["/bin/opm"]
CMD ["serve", "/configs", "--cache-dir=/tmp/cache"]

# Copy declarative config root into image at /configs and pre-populate serve cache
ADD bpfman-operator-catalog /configs
ADD catalog /configs
Copy link
Contributor

Choose a reason for hiding this comment

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

if u will use catalog as folder then pls delete bpfman-operator-catalog folder

Copy link
Contributor

@msherif1234 msherif1234 left a comment

Choose a reason for hiding this comment

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

Thanks @OlivierCazade for helping out with this

cat <<EOF >"${CATALOG_FILE}"
---
defaultChannel: latest
icon:
Copy link
Contributor

Choose a reason for hiding this comment

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

What icon are you using? Does it need to be embedded in the script like this, or could you upload and point to the svg file?

Copy link
Contributor

Choose a reason for hiding this comment

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

this bpfman log and it can be verified using https://base64.guru/converter/decode/image
and other operators doing the same the also embedded that way so we are following alone

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I used the one from the bundle:

I can include it in a separate file and make the script use it if you prefer.

Copy link
Contributor

Choose a reason for hiding this comment

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

no that is good enough Thanks

Copy link
Contributor

Choose a reason for hiding this comment

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

This is good. Thanks.

@anfredette anfredette removed the hold label Aug 1, 2024
@OlivierCazade OlivierCazade force-pushed the fbc-index branch 2 times, most recently from 281911e to b33f2cd Compare August 1, 2024 16:27
Signed-off-by: Olivier Cazade <ocazade@redhat.com>
Copy link

codecov bot commented Aug 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 27.61%. Comparing base (02bb7c6) to head (73c01fb).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #82   +/-   ##
=======================================
  Coverage   27.61%   27.61%           
=======================================
  Files          81       81           
  Lines        6999     6999           
=======================================
  Hits         1933     1933           
  Misses       4877     4877           
  Partials      189      189           
Flag Coverage Δ
unittests 27.61% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mergify mergify bot merged commit d64898a into bpfman:main Aug 1, 2024
12 of 13 checks passed
msherif1234 pushed a commit to msherif1234/bpfman-operator that referenced this pull request Nov 15, 2024
…/ocp-bpfman-operator

chore(deps): update ocp-bpfman-operator to ebeda02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants