Skip to content

Generate OLM Bundle for OperatorHub Release#195

Open
MuneebAijaz wants to merge 5 commits intohyperspike:mainfrom
MuneebAijaz:MA/olm-bundle-gen
Open

Generate OLM Bundle for OperatorHub Release#195
MuneebAijaz wants to merge 5 commits intohyperspike:mainfrom
MuneebAijaz:MA/olm-bundle-gen

Conversation

@MuneebAijaz
Copy link
Copy Markdown

@MuneebAijaz MuneebAijaz commented Feb 12, 2025

Refers to: #72

@MuneebAijaz MuneebAijaz marked this pull request as draft February 12, 2025 19:01
@MuneebAijaz
Copy link
Copy Markdown
Author

@dmolik Hi, i have some questions here, $(VERSION) calculation in Makefile currently returns commit hash, can we support git tags there?

While setting up the project, i do get this error while running generators build constraints exclude all Go files in /usr/local/go/src/iter, I need guidance on how to set it up properly.

@dmolik
Copy link
Copy Markdown
Member

dmolik commented Feb 14, 2025

Hey @MuneebAijaz $(VERSION) does support git tags, the behavior is when HEAD matches a tag VERSION is the tag name so if you checkout v0.0.55, $(VERSION) will be v0.0.55. However on any given day HEAD is likely to be untaggged, so VERSION will be the short sha of the most recent commit. There is all so another variable $(SHA) which always gives the most recent short commit sha.

You're other error looks to be related to your go setup, what's $GOROOT and $GOPATH set to?

Sorry for the late response, I've been violently ill this week.

Finally keep up the good work!

@MuneebAijaz
Copy link
Copy Markdown
Author

@dmolik i had some questions about how the operator works, and how can i test it. i saw theres some link in instance name and config maps?

@MuneebAijaz MuneebAijaz marked this pull request as ready for review April 9, 2025 01:49
@ibotty
Copy link
Copy Markdown

ibotty commented Jul 21, 2025

Is there anything missing? Or is it only the maintainers having to find time to properly review?

@MuneebAijaz
Copy link
Copy Markdown
Author

@ibotty its waiting on review

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