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

Inconsistent release 1.6.17 #351

Closed
froth opened this issue Jan 28, 2021 · 21 comments
Closed

Inconsistent release 1.6.17 #351

froth opened this issue Jan 28, 2021 · 21 comments
Assignees

Comments

@froth
Copy link

froth commented Jan 28, 2021

Scala steward informed me about a new release 1.6.17.
Looking at https://search.maven.org/search?q=mockito-scala it seems that something went wrong in the process, some packages have the new version some the old. Even mixing for different scala versions.

Not a real problem for me, but I thought you might to know (if you did not already noticed it)

@ultrasecreth
Copy link
Member

I noticed the release job failed yesterday, but on the bintray stage, didn't know that the maven release was corrupted...
As it was a dependency update I didn't pay that much attention, but what you say actually is a problem, gonna have to summon @mockitoguy here, as I'm not really familiar with the release process

Thanks for reporting it!

@mockitoguy
Copy link
Member

Hey guys! This is a little strange.

  • "mockito-scala_2.13:1.16.17" has been published to central (proof) but it does not show up in search.maven.org.
  • "mockito-scala_2.12:1.16.17" (and 2.11) has not been published to Bintray, and therefore is missing from Central.

@bbonanno, what are the expected covariants of Scala we should be publishing?

@omarkilani
Copy link

For whatever reason, the .pom is missing from the published artifact:

https://dl.bintray.com/mockito/maven/org/mockito/mockito-scala_2.13/1.16.17/

Note that the .pom.md5 is there, so that's a little strange.

@mockitoguy
Copy link
Member

The problem is that the CI build failed to publish all artifacts (details). My theory is that it's a bug in Gradle Bintray plugin: when one of the bintrayUpload tasks fails, the bintrayPublish task should not run (e.g. this looks like a bug in plugins handling of Gradle's finalizedBy/dependsOn). I haven't though about a remedy for this yet.

@mockitoguy
Copy link
Member

@bbonanno, [ci skip] does not work, we need to use [skip ci], see the CI configuration. Same goes for skipping releases, it needs to be [skip release] ("skip" goes first ;-)).

@mockitoguy mockitoguy self-assigned this Feb 1, 2021
mockitoguy added a commit that referenced this issue Feb 1, 2021
Until #351 is fixed, let's avoid automated publications to Maven Central
@ultrasecreth
Copy link
Member

@mockitoguy yeah, I noticed that after I merged, but in any case, it should have done a full release right? (I didn't care cause a full release out of that PR was not the end of the world).
I also noticed a PR failed the build on just the bintray upload (which is weird, why is a PR uploading anything)

The full release is a cartesian product of all the modules that produce a Jar x the 3 Scala versions we support

@ultrasecreth
Copy link
Member

I just merged another PR, no comments, so let's see how the release goes

@froth
Copy link
Author

froth commented Feb 2, 2021

went good :) the new release is consistent

@ultrasecreth
Copy link
Member

Nice, let's hope it doesn't happen again!

@ultrasecreth
Copy link
Member

Had to release again (another fix was made to the lib) so if that one's also successful I'll close this issue

@mockitoguy
Copy link
Member

Hey @bbonanno, let's keep this open for a little bit because the root cause is not fixed. I reported a bug to Bintray plugin: bintray/gradle-bintray-plugin#340

@ultrasecreth
Copy link
Member

@mockitoguy oh, fair enough

@ultrasecreth
Copy link
Member

@mockitoguy this thing got worse, the last 3 master builds all hung and got cancelled after being stuck in the release step for a while

@mockitoguy
Copy link
Member

I will look into this. We need to migrate off bintray anyway.

@ultrasecreth
Copy link
Member

@ultrasecreth
Copy link
Member

Hi @mockitoguy, sorry to bother but I noticed that even the PR builds are failing now on the Bintray upload, did something change around that?

@mockitoguy
Copy link
Member

mockitoguy commented Jul 29, 2021 via email

@ultrasecreth
Copy link
Member

Hi @mockitoguy, sorry to be annoying but now no release at all can be made, and PRs don't build anymore as the bintrayUpload always fails... seems to be related to credentials according to the error, but there isn't much I can do here, would you mind to have a look :)

Cheers!!!

@mockitoguy
Copy link
Member

mockitoguy commented Aug 12, 2021 via email

@ultrasecreth
Copy link
Member

@mockitoguy sure, no worries, enjoy your trip :)

@mockitoguy
Copy link
Member

Let's track this in #320

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

4 participants