Skip to content

Replace maven publish step with updated credentials - #8

Merged
korsveien merged 4 commits into
mainfrom
upgrade-maven-central-publish
Aug 8, 2025
Merged

Replace maven publish step with updated credentials#8
korsveien merged 4 commits into
mainfrom
upgrade-maven-central-publish

Conversation

@korsveien

Copy link
Copy Markdown
Contributor
  • Replace publish step in order to use new signing key
  • Bump dependency on super-pom to release using new signing key

- Replace publish step in order to use new signing key
- Bump dependency on super-pom to release using new signing key
@korsveien

Copy link
Copy Markdown
Contributor Author

Version her er satt til SNAPSHOT i pom-en. Burde dette endres på? Noe annet som mangler? @post-svejk

@post-svejk post-svejk left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kjør på og prøv! Kan bli noen utfordringer med versjonen av plugins i denne POM-en vs. de oppdaterte versjonene i parent-POM-en. Den nye Sonatype-pluginen stiller noen krav til versjoner av andre plugins, såvidt jeg husker. Det er derfor jeg ikke tar noen sjanser og oppdaterer plugins i POM-ene, også.

@post-svejk

Copy link
Copy Markdown

Version her er satt til SNAPSHOT i pom-en. Burde dette endres på? Noe annet som mangler? @post-svejk

Nei, mvn versions:set ... ordner det. Vi må for øvrig bruke den her, kan ikke bruke alternativet med system properties: https://maven.apache.org/guides/mini/guide-maven-ci-friendly.html, da Sonatype sin plugin ikke forstår versjoner fra system properties.

@korsveien

korsveien commented Aug 8, 2025

Copy link
Copy Markdown
Contributor Author

Kjør på og prøv! Kan bli noen utfordringer med versjonen av plugins i denne POM-en vs. de oppdaterte versjonene i parent-POM-en. Den nye Sonatype-pluginen stiller noen krav til versjoner av andre plugins, såvidt jeg husker. Det er derfor jeg ikke tar noen sjanser og oppdaterer plugins i POM-ene, også.

Satt bare <version> til 1. Bygget er grønt men finner ikke igjen versjonen på Maven Central (MC). Men ser nå at artefaktet heter jo http-client-builder**5** og har versjon 5.0.0 som siste på MC, men ser ikke dette i POM-en. Er dette noe som har blitt publisert manuelt?

@post-svejk

post-svejk commented Aug 8, 2025

Copy link
Copy Markdown

Kjør på og prøv! Kan bli noen utfordringer med versjonen av plugins i denne POM-en vs. de oppdaterte versjonene i parent-POM-en. Den nye Sonatype-pluginen stiller noen krav til versjoner av andre plugins, såvidt jeg husker. Det er derfor jeg ikke tar noen sjanser og oppdaterer plugins i POM-ene, også.

Satt bare <version> til 1. Bygget er grønt men finner ikke igjen versjonen på Maven Central (MC). Men ser nå at artefaktet heter jo http-client-builder**5** og har versjon 5.0.0 som siste på MC, men ser ikke dette i POM-en. Er dette noe som har blitt publisert manuelt?

Du må logge deg inn og sjekke i portalen: https://central.sonatype.com/publishing/deployments. Jeg ser ingenting her.
Versjonene settes dynamisk, som nevnt over, så du ser bare den konkrete versjonen i det som lastes opp for deployment i portalen, og etter å ha publisert, på Maven Central (browsing etter artefakter). Vanligvis står versjonen på LOCAL-SNAPSHOT, siden det er praktisk for lokal testing i POM-ene, endres aldri, kun dynamisk i byggeskript.

@post-svejk

Copy link
Copy Markdown

Ser greit ut nå. Bare rull tilbake POM-versjonen. Så er det bare lage en release og sjekke at deploymentet validerer og dukker opp i portalen. Det du har gjort hittil er jo snapshot-bygging, og da ser vi loggene at det stopper opp på install-steget til Maven, og snapshotet ser ikke ut til å bli deployet. Det er ett kjent problem vi se nærmere på etterpå. Men release-bygging fungerte jo for parent-POM-en, så det bør funke, her også.

@korsveien

Copy link
Copy Markdown
Contributor Author

Kjør på og prøv! Kan bli noen utfordringer med versjonen av plugins i denne POM-en vs. de oppdaterte versjonene i parent-POM-en. Den nye Sonatype-pluginen stiller noen krav til versjoner av andre plugins, såvidt jeg husker. Det er derfor jeg ikke tar noen sjanser og oppdaterer plugins i POM-ene, også.

Satt bare <version> til 1. Bygget er grønt men finner ikke igjen versjonen på Maven Central (MC). Men ser nå at artefaktet heter jo http-client-builder**5** og har versjon 5.0.0 som siste på MC, men ser ikke dette i POM-en. Er dette noe som har blitt publisert manuelt?

Du må logge deg inn og sjekke i portalen: https://central.sonatype.com/publishing/deployments. Jeg ser ingenting her. Versjonene settes dynamisk, som nevnt over, så du ser bare den konkrete versjonen i det som lastes opp for deployment i portalen, og etter å ha publisert, på Maven Central (browsing etter artefakter). Vanligvis står versjonen på LOCAL-SNAPSHOT, siden det er praktisk for lokal testing i POM-ene, endres aldri, kun dynamisk i byggeskript.

Skjønner. Hvor ligger brukernavn og passord?

@post-svejk

Copy link
Copy Markdown

Skjønner. Hvor ligger brukernavn og passord?

Posta på Slack. Grunnen til at du ikke ser noe på Maven Central er at snapshots ikke funker (noop, hvis du der på Actions-loggen).

@korsveien

Copy link
Copy Markdown
Contributor Author

Ser ut som det gikk. Lagde en release på denne branchen og ser at det dukker opp en ny verifisert release på maven central: https://central.sonatype.com/publishing/deployments. Her ser det ut som man må manuelt inn og trykke publish?

@post-svejk

Copy link
Copy Markdown

Ser ut som det gikk. Lagde en release på denne branchen og ser at det dukker opp en ny verifisert release på maven central: https://central.sonatype.com/publishing/deployments. Her ser det ut som man må manuelt inn og trykke publish?

Bra! Bare publiser. Merk at det tar lang tid før den dukker opp (går fra PUBLISHING til PUBLISHED i portalen). Tror det tok over en halv time på parent-POM-en.

@korsveien
korsveien merged commit e916294 into main Aug 8, 2025
8 checks passed
@korsveien
korsveien deleted the upgrade-maven-central-publish branch August 8, 2025 10:35
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