Skip to content

FISH-13115 update nexus registry creds#8034

Open
abdulrahim458 wants to merge 8 commits intopayara:mainfrom
abdulrahim458:FISH-13115-update-dependabot-private-registry-creds
Open

FISH-13115 update nexus registry creds#8034
abdulrahim458 wants to merge 8 commits intopayara:mainfrom
abdulrahim458:FISH-13115-update-dependabot-private-registry-creds

Conversation

@abdulrahim458
Copy link
Copy Markdown
Contributor

Description

Important Info

Blockers

Testing

New tests

Testing Performed

Testing Environment

Documentation

Notes for Reviewers

Copy link
Copy Markdown
Member

@Pandrex247 Pandrex247 left a comment

Choose a reason for hiding this comment

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

Community shouldn't require access to the private Enterprise repo

Pandrex247
Pandrex247 previously approved these changes Apr 1, 2026
@Pandrex247 Pandrex247 dismissed their stale review April 1, 2026 08:33

Fat fingers

Comment thread .github/dependabot.yml Outdated
Comment on lines +14 to +15
registries:
- payara-private-nexus
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It's not trying to check this repo, it's trying to check the private staging repo

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We should either exclude this repo (as that repo should be disabled by default anyway), and we don't want dependabot prematurely creating PRs for things we're trying to patch - it shouldn't be creating PRs for *.payara-px anyway, and that's the only thing which should really be in there

Comment thread .github/dependabot.yml Outdated
- dependency-name: "fish.payara.*"
update-types: ["version-update:semver-major"]

- dependency-name: "*payara-px*"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm not sure this would match anything?
We don't have any dependencies following that naming convention unless I'm overlooking something

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

else i can just ignore the failing dependencies to check

  • dependency-name: "javax.xml.stream:stax-api"
  • dependency-name: "javax.jbi:jbi"

this prevents from failures or else add the staging registry details to dependabot file

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Those dependencies will just be the first ones being polled - there's nothing special about them

@abdulrahim458
Copy link
Copy Markdown
Contributor Author

abdulrahim458 commented Apr 20, 2026

Before: Dependabot was returning 401 Unauthorized on every request to payara-staging, causing private_source_authentication_failure for javax.jbi:jbi and javax.xml.stream:stax-api. Because of these auth failures, the job was exiting with a non-zero exit code meaning not all dependencies were being evaluated some updates were being silently skipped and no PRs were created for those dependencies.

After: With the registry credentials added, payara-staging now correctly returns 404 where artifacts aren't present and falls back to payara-artifacts for Payara-patched dependencies (e.g. *.payara-p1) and Maven Central for everything else. All dependencies are now fully evaluated.

The job completes successfully and update PRs are created correctly.

Test run: https://github.com/abdulrahim458/Payara/actions/runs/24636034657/job/72031532827

CC: @Pandrex247

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