You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kie.apache.org by "raboof (via GitHub)" <gi...@apache.org> on 2024/12/20 12:41:51 UTC

[PR] Check Maven Central before Apache-specific repositories [incubator-kie-tools]

raboof opened a new pull request, #2822:
URL: https://github.com/apache/incubator-kie-tools/pull/2822

   GitHub Actions builds are hitting the rule from
   https://infra.apache.org/infra-ban.html that disallows excessive 404's on repository.apache.org, causing them to get banned.
   
   This change should make various builds check Maven Central before checking `repository.apache.org`, which hopefully should help.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org
For additional commands, e-mail: commits-help@kie.apache.org

Re: [PR] kie-issues#1750: Try Maven Central before hitting `repository.apache.org` [incubator-kie-tools]

Posted by "yesamer (via GitHub)" <gi...@apache.org>.
yesamer commented on PR #2822:
URL: https://github.com/apache/incubator-kie-tools/pull/2822#issuecomment-2732545409

   @tiagobento I guess we can merge this, WDYT?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org
For additional commands, e-mail: commits-help@kie.apache.org

Re: [PR] NO-ISSUE: Check Maven Central before Apache-specific repositories [incubator-kie-tools]

Posted by "tiagobento (via GitHub)" <gi...@apache.org>.
tiagobento commented on PR #2822:
URL: https://github.com/apache/incubator-kie-tools/pull/2822#issuecomment-2557872418

   Most likely yes! @ricardozanini or @wmedvede might now more assertively. Let's wait for their assessment on this matter... Thanks @raboof!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org
For additional commands, e-mail: commits-help@kie.apache.org

Re: [PR] Check Maven Central before Apache-specific repositories [incubator-kie-tools]

Posted by "tiagobento (via GitHub)" <gi...@apache.org>.
tiagobento commented on PR #2822:
URL: https://github.com/apache/incubator-kie-tools/pull/2822#issuecomment-2557862870

   If I'm not mistaken, even though apache.snapshots is listed first, it has `releases` disabled, so I thought Maven only tries to download from it when it finds a dependency with a SNAPSHOT version. Following this logic, and because our 1st party dependencies (drools,jbpm,kogito) are the only ones with a SNAPSHOT version, we're not overloading the apache.snapshots repository with 404 requests for all other 3rd party dependencies... That's at least my theory 😅
   
   Do you see it differently or do you have any logs/material that could help us get a more assertive assessment on this issue?
   
   In any case, thanks for the PR and proactively helping us figure it out!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org
For additional commands, e-mail: commits-help@kie.apache.org

Re: [PR] kie-issues#1750: Try Maven Central before hitting `repository.apache.org` [incubator-kie-tools]

Posted by "tiagobento (via GitHub)" <gi...@apache.org>.
tiagobento merged PR #2822:
URL: https://github.com/apache/incubator-kie-tools/pull/2822


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org
For additional commands, e-mail: commits-help@kie.apache.org

Re: [PR] NO-ISSUE: Check Maven Central before Apache-specific repositories [incubator-kie-tools]

Posted by "raboof (via GitHub)" <gi...@apache.org>.
raboof commented on PR #2822:
URL: https://github.com/apache/incubator-kie-tools/pull/2822#issuecomment-2557867198

   Yes, that's definitely true for `packages/dev-deployment-kogito-quarkus-blank-app/pom.xml`, `packages/maven-base/settings.xml` and `packages/stunner-editors/pom.xml` - I guess I may have gotten a bit carried away :)
   
   The changes to the generated poms and to `packages/sonataflow-image-common/resources/modules/kogito-maven/common/maven/settings.xml` may still be (more) relevant, though?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org
For additional commands, e-mail: commits-help@kie.apache.org

Re: [PR] kie-issues#1750: Try Maven Central before hitting `repository.apache.org` [incubator-kie-tools]

Posted by "yesamer (via GitHub)" <gi...@apache.org>.
yesamer commented on PR #2822:
URL: https://github.com/apache/incubator-kie-tools/pull/2822#issuecomment-2710129278

   I agree with @tiagobento, I would remove the changes where the apache repo is used for snapshots only.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org
For additional commands, e-mail: commits-help@kie.apache.org

Re: [PR] kie-issues#1750: Try Maven Central before hitting `repository.apache.org` [incubator-kie-tools]

Posted by "raboof (via GitHub)" <gi...@apache.org>.
raboof commented on PR #2822:
URL: https://github.com/apache/incubator-kie-tools/pull/2822#issuecomment-2710736779

   Updated accordingly!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org
For additional commands, e-mail: commits-help@kie.apache.org