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.

Sign up for GitHub

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

Jump to bottom

#7553 Fix release workflow #7562

Merged
merged 12 commits into from
Mar 25, 2023
Merged

#7553 Fix release workflow #7562

merged 12 commits into from
Mar 25, 2023

Conversation

planetf1
Copy link
Member

@planetf1 planetf1 commented Mar 24, 2023 •
edited
Loading

Signed-off-by: Nigel Jones nigel.l.jones+git@gmail.com

Description

This PR will capture any additional commits needed to fix the release 4 build pipelines & add them to main:

  • Fixes release yml - path to assembly was incorrect
  • Updated release & merge yml to path naming consistent with gradle usage
  • Update maven metadata to match v3
  • Skip interstitial directories (ie do not publish access-services, access-services/asset-owner as they are just top levels)
  • Fix extraction of version from build - used in release pipeline
  • disable cache (to ensure results are totally cleanly built from source - in case of any caching errors)
  • enforce series builds (needed to publish to sonatype staging repositories!)
  • ensure PR build does not publish - main/release only
  • correct files to upload and attach as build artifact

Related Issue(s)

Testing

Release Notes & Documentation

Additional notes

@planetf1
odpi#7533 fix incorrect path in release workflow
e1a81ea 
Signed-off-by: Nigel Jones <nigel.l.jones+git@gmail.com>
@planetf1 planetf1 marked this pull request as draft March 24, 2023 08:01
@planetf1
odpi#7533 Update maven metadata to match v3 pom
9bf0750 
Signed-off-by: Nigel Jones <nigel.l.jones+git@gmail.com>
@planetf1
odpi#7533 Only configure gradle project if build file exists - remove…
6d8986a 
… interstitials

Signed-off-by: Nigel Jones <nigel.l.jones+git@gmail.com>
@planetf1
Merge branch 'main' of https://github.com/odpi/egeria
d73cef2
@planetf1
odpi#7533 fix extraction of version, for publishing of container image
9b3a109 
Signed-off-by: Nigel Jones <nigel.l.jones+git@gmail.com>
@planetf1
Ensure artifacts are signed, and correct name in pom for bom
c737b58 
Signed-off-by: Nigel Jones <nigel.l.jones+git@gmail.com>
@planetf1
Ensure artifacts are signed, and correct name in pom for bom
e214104 
Signed-off-by: Nigel Jones <nigel.l.jones+git@gmail.com>
@planetf1
Disable parallelism for release (needed for sonatype) & disable caching
6a800ed 
Signed-off-by: Nigel Jones <nigel.l.jones+git@gmail.com>
@planetf1
Correct gradle build action (line deleted)
7cd23ee 
Signed-off-by: Nigel Jones <nigel.l.jones+git@gmail.com>
@planetf1
Merge branch 'main' of https://github.com/odpi/egeria
7298d0a
@planetf1
Disable artefact publish from pr build (no signatory)
c700c41 
Signed-off-by: Nigel Jones <nigel.l.jones+git@gmail.com>
@planetf1
Correct file location of assembly to upload as build artifact
6db97df 
Signed-off-by: Nigel Jones <nigel.l.jones+git@gmail.com>
@planetf1 planetf1 marked this pull request as ready for review March 25, 2023 00:26
@planetf1 planetf1 enabled auto-merge March 25, 2023 00:26
@planetf1 planetf1 merged commit d92b7d7 into odpi:main Mar 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@planetf1