Skip to content

Support nested path for docker build - #3330

Merged
agrasth merged 4 commits into
jfrog:masterfrom
fluxxBot:dockernested
Feb 9, 2026
Merged

agrasth merged 4 commits into
jfrog:masterfrom
fluxxBot:dockernested

Conversation

@fluxxBot

@fluxxBot fluxxBot commented Jan 29, 2026 •

Copy link
Copy Markdown
Contributor
  • 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 ./....

Description:
It was found that uploading docker images inside a nested path caused failure in attaching build properties in the artifacts due to invalid path construction:
eg: docker-repo/myorg/image:latest when uploaded, cli tried to attach properties on docker-repo/image:latest, as a result attaching properties caused 404 not found failure.

Solution:
instead of using image full name which returned image:latest, we are using image full name without repo which returns myorg/image:latest

depends on:

  1. Support nested path for docker build jfrog-cli-artifactory#354

@fluxxBot fluxxBot added improvement Automatically generated release notes safe to test Approve running integration tests on a pull request labels Jan 29, 2026
@fluxxBot
fluxxBot requested a review from bhanurp January 30, 2026 04:16
@fluxxBot fluxxBot 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 Jan 30, 2026
@fluxxBot fluxxBot 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 Feb 2, 2026
@fluxxBot fluxxBot 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 Feb 3, 2026
agrasth and others added 2 commits February 9, 2026 12:06
@agrasth
agrasth enabled auto-merge (squash) February 9, 2026 06:38
@github-actions

github-actions Bot commented Feb 9, 2026

Copy link
Copy Markdown
Contributor

👍 Frogbot scanned this pull request and did not find any new security issues.


@agrasth agrasth 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 Feb 9, 2026
@agrasth
agrasth merged commit e5c4b62 into jfrog:master Feb 9, 2026
78 of 79 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Automatically generated release notes safe to test Approve running integration tests on a pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants