Skip to content

Jfrogpipelines#3422

Open
naveenku-jfrog wants to merge 13 commits intomasterfrom
jfrogpipelines
Open

Jfrogpipelines#3422
naveenku-jfrog wants to merge 13 commits intomasterfrom
jfrogpipelines

Conversation

@naveenku-jfrog
Copy link
Copy Markdown
Collaborator

@naveenku-jfrog naveenku-jfrog commented Apr 9, 2026

  • All tests have passed. If this feature is not already covered by the tests, new tests have been added.
  • The pull request is targeting the master branch.
  • The code has been validated to compile successfully by running go vet ./....
  • The code has been formatted properly using go fmt ./....

Updated pipelines.yml and access.test so that these tests become compatible with tests running in GHE runners.

@naveenku-jfrog naveenku-jfrog added the safe to test Approve running integration tests on a pull request label Apr 9, 2026
@naveenku-jfrog naveenku-jfrog added safe to test Approve running integration tests on a pull request and removed safe to test Approve running integration tests on a pull request labels Apr 9, 2026
@naveenku-jfrog naveenku-jfrog added safe to test Approve running integration tests on a pull request and removed safe to test Approve running integration tests on a pull request labels Apr 9, 2026
@naveenku-jfrog naveenku-jfrog added safe to test Approve running integration tests on a pull request and removed safe to test Approve running integration tests on a pull request labels Apr 9, 2026
@naveenku-jfrog naveenku-jfrog added safe to test Approve running integration tests on a pull request and removed safe to test Approve running integration tests on a pull request labels Apr 9, 2026
…vior

Add cloud-compatible username fallback for access token creation and add eventual-consistency polling for Maven artifact cleanup checks to reduce flaky failures on jfrogdev.org.

Made-with: Cursor
Publish aggregated workflow run status to a result resource and add a separate jfrog_cli_tests_status step so overall test outcome and failed workflow list are always surfaced clearly in the pipeline.

Made-with: Cursor
Quote echo commands containing colons so pipeline schema treats them as string commands instead of object mappings during source sync validation.

Made-with: Cursor
Update jfrog-cli pipeline configuration to dispatch GitHub workflow runs from jfrog-cli-workflows master by default now that the workflow changes are merged.

Made-with: Cursor
Add JFROG_CLI_GITHUB_REF pipeline variable (default jfrogpipelines) and prefer it over resource commit SHA when passing jfrog_cli_ref to workflow dispatch.

Made-with: Cursor
Update pipeline defaults to dispatch GitHub workflows from feature/RTECO-1003-add-podman-pipelines-workflows while podman fixes are validated there.

Made-with: Cursor
- Add ERR trap to catch unhandled script errors and write failure
  status before exit
- Write initial "running" status at step start so platform kills
  leave an informative state
- Add write_output to onFailure for onStart failures and edge cases
- Use guard variable to prevent double-reporting between trap and
  explicit fail_jf_cli_tests calls
- Make jfrog_cli_tests_status step fail with exit 1 when overall
  status is not success

Made-with: Cursor
Rootless Podman requires kernel user-namespace support which is
unavailable on DinD scale-set runners. The containers block
newuidmap/nested user namespaces unless started with --privileged.
Remove podmanTests.yml from dispatch list until runner infra supports it.

Made-with: Cursor
…sts, fix Gradle eventual consistency

- Remove jfrog_cli_tests_status step and jfrog_cli_tests_result PropertyBag
  (status reporting via dedicated step is no longer needed)
- Re-include podmanTests.yml in pipeline dispatch (rootful workaround in workflows repo)
- Fix TestGradleBuildPublishWithCIVcsProps: retry GetBuildInfo with Eventuallyf
  to handle eventual consistency, guard against nil publishedBuildInfo

Made-with: Cursor
…nsistency

The search index on pipeline-provisioned Artifactory can take longer
than 30s to reflect deletions. Increase the Eventuallyf timeout from
30s to 90s in assertMavenArtifactsEventuallyDeleted.

Made-with: Cursor
npm 10 lowercases env-var config keys, which breaks the scoped
_authToken env var set by CreateTempNpmrc. Write file-based auth
directly into the project .npmrc to preserve case. Also guard
against panic in TestNpmPublishWithWorkspacesRunNative when modules
array is empty.

Made-with: Cursor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe to test Approve running integration tests on a pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant