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

Releases, versioning #11

Open
seemk opened this issue Mar 30, 2021 · 13 comments
Open

Releases, versioning #11

seemk opened this issue Mar 30, 2021 · 13 comments

Comments

@seemk
Copy link
Contributor

seemk commented Mar 30, 2021

How should we release different contrib projects?

nginx now (#10) builds release binaries for different platforms and I'd like expose these artifacts via Github releases.
Should each project have it's own version tag? E.g. nginx-0.1.0 and httpd-0.1.0 so when the tag is created a GH release will be available.

@lalitb
Copy link
Member

lalitb commented Mar 30, 2021

Should each project have it's own version tag? E.g. nginx-0.1.0 and httpd-0.1.0 so when the tag is created a GH release will be available.

I will vote for this. Each project within repo should have separate version tags and releases.

@seemk
Copy link
Contributor Author

seemk commented Apr 6, 2021

If we are ok with this, I'll do a release for nginx-0.1.0, it will provide the nginx module binaries as part of the release.

@lalitb
Copy link
Member

lalitb commented Apr 6, 2021

Looks good to me.

@seemk
Copy link
Contributor Author

seemk commented Apr 7, 2021

Looks like I can't do it in this repo - no Create a new release button available 🙁 Access rights I presume

@lalitb
Copy link
Member

lalitb commented Apr 7, 2021

Ok, I think @maxgolov is the admin of this repo and can give you write access to create a release. And probably I am getting write access through the cpp-contrib-approvers group. I did try adding you to the group earlier but seems need to be a member of opentelemetry org for that.
You can raise a ticket to be a member (example) : open-telemetry/community#647 . I can be one of the sponsors ( we need two ). Once this is done, I can add you to the approver's group. Hopefully, that should give you permission to create a release.

@seemk
Copy link
Contributor Author

seemk commented Apr 7, 2021

Thanks! Created a request: open-telemetry/community#699

@lalitb
Copy link
Member

lalitb commented Apr 8, 2021

Have added you to cpp-contrib-approvers. Don't think if that will help, need to wait for @maxgolov next week for right permissions.

@maxgolov
Copy link
Contributor

@seemk @lalitb

I vaguely remember that when we initially got it set up, we had to ask for the open-telemetry org admins to set permissions on it.. Can I help creating a tag? Would you be able to provide me with release notes, and how do you want to label it - v0.1.0 , or do we align it with the mainline tag?

Let's discuss it on slack.

@maxgolov
Copy link
Contributor

I created the tag v0.1.0. Hope that it addresses your immediate needs.

@TomRoSystems
Copy link
Member

Yes, thanks. We would like to include also some artefacts (assets) as well during this process. I've contacted @SergeyKanzhelev and he suggested for me to file an issue which I did - open-telemetry/community#718

@maxgolov
Copy link
Contributor

maxgolov commented Apr 26, 2021

@TomRoSystems - once you get permissions, you may be able to edit the release and simply attach additional artifacts. It would be great though if we somehow script the creation of binaries. That is to avoid attaching prebuilt binaries that have not been created by the GitHub Actions, or by some other secure process. Attaching source or documentation in any form should be fine. Just need to be careful with binaries.

@SergeyKanzhelev
Copy link
Member

Knowing GH permissions nonsense, if somebody will need to get Admin on repo to set everything up - let me know (ideally by adding details in open-telemetry/community#718).

@seemk
Copy link
Contributor Author

seemk commented Apr 27, 2021

@maxgolov Thanks for this, and my bad for not noticing this discussion here earlier 🙂

We decided to tag different instrumentation releases with each their own tag, e.g. nginx-0.1.0, but I can edit these now.

I was hoping the artifacts would show up under the releases, but CI will need to package and publish them separately (via https://github.com/softprops/action-gh-release), going to add it soon.

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

No branches or pull requests

5 participants