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

Fix fips-bundle-test artifactId (it is for github repo lib-fips-bundle-test, looks confusing but in fact it's not :) #4089

Merged
merged 1 commit into from
Sep 13, 2024

Conversation

olamy
Copy link
Contributor

@olamy olamy commented Sep 13, 2024

Signed-off-by: Olivier Lamy olamy@apache.org

Link to GitHub repository

When modifying release permission

List the GitHub usernames of the users who should have commit permissions below:

  • @username1
  • @username2

This is needed in order to cut releases of the plugin or component.

If you are modifying the release permission of your plugin or component, fill out the following checklist:

Release permission checklist (for submitters)

When enabling automated releases (cd: true)

Follow the documentation to ensure, your pull request is set up properly. Don't merge it yet.
In case of changes requested by the hosting team, an open PR facilitates future reviews, without derailing work across multiple PRs.

Link to the PR enabling CD in your plugin

CD checklist (for submitters)

Reviewer checklist

There are IRC Bot commands for it.

…e-test, looks confusing but in fact it's not :)

Signed-off-by: Olivier Lamy <olamy@apache.org>
@olamy olamy requested a review from a team as a code owner September 13, 2024 08:00
@olamy
Copy link
Contributor Author

olamy commented Sep 13, 2024

@timja timja merged commit 4707328 into jenkins-infra:master Sep 13, 2024
3 checks passed
@olamy olamy deleted the fix-artifactId-fips-provider-test branch September 13, 2024 08:18
@@ -2,7 +2,7 @@
name: "lib-fips-bundle-test"
github: "jenkinsci/lib-fips-bundle-test"
paths:
- "io/jenkins/test/fips/lib-fips-bundle-test"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

weird

Caused by: org.apache.maven.wagon.authorization.AuthorizationException: authorization failed for https://repo.jenkins-ci.org/releases/io/jenkins/test/fips/fips-bundle-test/20.vb_e5df22f62ea_/fips-bundle-test-20.vb_e5df22f62ea_.pom, status: 403 Forbidden

here https://github.com/jenkinsci/lib-fips-bundle-test/actions/runs/10845721879/job/30097173717

@timja

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh maybe I need to learn patience? :)

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

no worries, Friday arvo I will have a beer ;)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I got few but still a problem (not sure I can get more now :) )
I will wait tomorrow.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I hope there is no mistake somewhere as I may have been confused by the naming convention :)

  • github is lib-fips-bundle-test
  • artifactId fips-bundle-test
  • file here is component-fips-bundle-test.yml

You guys made it hard to me to not make any mistake :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

issue created here jenkins-infra/helpdesk#4290

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