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

release ocb container image #295

Open
frzifus opened this issue Feb 16, 2023 · 4 comments
Open

release ocb container image #295

frzifus opened this issue Feb 16, 2023 · 4 comments
Assignees

Comments

@frzifus
Copy link
Member

frzifus commented Feb 16, 2023

It seems that there has already been some effort to make a ocb container image available. But its unclear where it should live?

According to the discussion in the collector repo, should it be released here?

@jpkrohling
Copy link
Member

I believe the first step would then to move the release of ocb to this repo. It's currently being released from the core repository: https://github.com/open-telemetry/opentelemetry-collector/releases/tag/cmd%2Fbuilder%2Fv0.72.0

@bogdandrutu bogdandrutu reopened this May 23, 2023
@frzifus frzifus self-assigned this May 24, 2023
@frzifus
Copy link
Member Author

frzifus commented May 24, 2023

note: As discussed in the collector sig call - we create the image here.

@jackgopack4
Copy link
Contributor

Hi all, going to work on adding the ocb release pipeline to the opentelemetry-collector-releases repo and go from there to take this up.

@frzifus
Copy link
Member Author

frzifus commented Jul 31, 2024

FTR: This topic was discussed during the SIG Call on March 1, 2023, 17:00 UTC SIG mtg. At that time I think recordings have been published on youtube. Unfortunately its not listed on the meeting recording list.

dmitryax pushed a commit to open-telemetry/opentelemetry-collector that referenced this issue Aug 9, 2024
After discussion with the collector SIG call this week, July
31st, I have made small tweaks to the artifact release that will not
require an "extra step" of pushing tags. If these changes are accepted,
the distributions will be released under tag "vX.XXX.X" and the ocb
binaries will be released under "cmd/builder/vX.XXX.X" but both under
this repo, opentelemetry-collector-releases. The workflow
`builder-release.yaml` in releases repo will push the additional tags
necessary for the builder release.

Refactoring the release process as mentioned in past issues/PRs. ocb
binary is currently released in a separate release tag under this
repository, which makes it harder to find and inconsistent with the rest
of the binaries and images releasing in the
opentelemetry-collector-releases repository.

Releasing the ocb binaries in opentelemetry-collector-releases makes
this process more standardized and also opens the opportunity to release
more tools alongside ocb binary down the road, like corresponding docker
images (as discussed in #5712).

#### Link to tracking issue
Fixes #10662
Mentioned in [releases issue #
295](open-telemetry/opentelemetry-collector-releases#295)
[Corresponding PR in
opentelemetry-collector-releases](open-telemetry/opentelemetry-collector-releases#608)

#### Testing
Sample binary release in [forked
jackgopack4/opentelemetry-collector-releases
repo](https://github.com/jackgopack4/opentelemetry-collector-releases/releases/tag/cmd%2Fbuilder%2Fv0.106.1)

#### Documentation
Updated corresponding documentation to reflect new release process

---------

Co-authored-by: Pablo Baeyens <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants