Skip to content

Conversation

@pavolloffay
Copy link
Member

Related to hypertrace/hypertrace#144

Remove deprecated CCI release job to GHA.

I haven't tested if the credentials are correct, hence this PR might need some more love after is merged and release triggered.

@ryandens
Copy link
Member

ryandens commented Jul 8, 2021

Fantastic, really excited about this! 🎉 I recently did something like this on a different closed-source projects. One of things I found useful to do prior to merging was temporarily adding a step to the PR pipeline with the gradle task publishToMavenLocal to make sure I pulled in all of the signing key properties were correct and produced a valid signature. Additionally, to prevent some issues popping up when we go to release this next, we could test the publish task in GHA by simply publishing to a staging repo, but without the closeAndReleaseRepository task. When we've verified it works properly, we can simply manually drop the staging repo in nexus and remove the steps used to test the publish workflow. ultimately, nothing will perfectly test publishing until we actually publish, but I think doing something like I described would help us reduce risk.

@ryandens
Copy link
Member

ryandens commented Jul 8, 2021

Nit: could you also update the badge in the README to point to GHA rather than CircleCI? Additionally, I believe the bullet points describing how to add an SSH key for CircleCI in RELEASE.md#Configuration are no longer necessary and can be deleted. Mainly, I'm just looking to cleanup references to CircleCI.

@pavolloffay
Copy link
Member Author

I will update docs, however I don't have the secrets for the local dry run. Would you like to try it yourself?

@ryandens
Copy link
Member

ryandens commented Jul 8, 2021

I will update docs, however I don't have the secrets for the local dry run. Would you like to try it yourself?

I don't think I'll have the time to get to it today, and I wrote this suggestion before seeing your message about cutting a patch release to test it out, that's probably a better way to go! Thanks for the PR 😃

@pavolloffay
Copy link
Member Author

I will merge, cut the release, and hope 😅

Signed-off-by: Pavol Loffay <[email protected]>
Signed-off-by: Pavol Loffay <[email protected]>
Signed-off-by: Pavol Loffay <[email protected]>
Signed-off-by: Pavol Loffay <[email protected]>
@pavolloffay pavolloffay merged commit c415f52 into hypertrace:main Jul 12, 2021
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.

2 participants