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(jenkins::controller) do not use ACP for maven repositories with id local #2821

Merged
merged 2 commits into from May 9, 2023

Conversation

dduportal
Copy link
Contributor

Related to jenkins-infra/helpdesk#3567 (comment)

Needs a twin in jenkins-infra/kubernetes-management

This PR adds exlcusions from the ACP configurations file for any Maven repository which id would be in-project or local to allow loading JAR from source code local files.

@lemeurherve
Copy link
Member

Where does local comes from?

Can't we keep only one unique id for local/in project maven repository?

@dduportal
Copy link
Contributor Author

Where does local comes from?

Can't we keep only one unique id for local/in project maven repository?

I proposed local as an alternative name because in-project is confusing for me.
Also, It does not feel good for me to ask the contributor to change their id in the current pom.xml of their.

Hence the 2 names excluded.

@dduportal
Copy link
Contributor Author

Testing with jenkins-infra/helpdesk#3567 (comment) that it works as expected with the exception set up manually for in-project.

Then let's use only local and we'll ask the user to update their pom.xml

@dduportal dduportal changed the title fix(jenkins::controller) do not use ACP for maven repositories with id in-project nor local fix(jenkins::controller) do not use ACP for maven repositories with id local May 9, 2023
@dduportal dduportal merged commit d6943e2 into production May 9, 2023
8 checks passed
@dduportal dduportal deleted the helpdesk-3567 branch May 9, 2023 16:20
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.

None yet

2 participants