Skip to content

Conversation

@fluxxBot
Copy link
Contributor

@fluxxBot fluxxBot commented Oct 15, 2025

  • 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 ./....

This PR aims to introduce support for docker build and buildx command while maintaing the legacy usage.

Implements docker build, docker buildx and legacy build commands using the Strategy pattern to support different execution modes based on the JFROG_RUN_NATIVE environment variable.

changes:

  1. New Strategy Pattern Implementation: Three strategies for Docker build execution:
    -> RunNativeStrategy: Direct Docker build and buildx execution (JFROG_RUN_NATIVE=true)
    -> LegacyStrategy: Traditional JFrog approach (default)

depends on:

  1. Introduced support for docker build command jfrog-cli-artifactory#283
  2. Added on demand log level to debug for add props jfrog-client-go#1269

@fluxxBot fluxxBot added new feature Automatically generated release notes ignore for release Automatically generated release notes labels Oct 15, 2025
@ehl-jf ehl-jf added the safe to test Approve running integration tests on a pull request label Oct 15, 2025
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Oct 15, 2025
@ehl-jf ehl-jf added the safe to test Approve running integration tests on a pull request label Oct 16, 2025
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Oct 16, 2025
@fluxxBot fluxxBot requested review from agrasth and bhanurp and removed request for bhanurp October 16, 2025 04:55
@RemiBou RemiBou added the jfrog-internal Items created by jfrog employees label Oct 17, 2025
@ehl-jf ehl-jf added the safe to test Approve running integration tests on a pull request label Nov 10, 2025
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Nov 10, 2025
@fluxxBot fluxxBot removed the ignore for release Automatically generated release notes label Nov 10, 2025
@ehl-jf ehl-jf added the safe to test Approve running integration tests on a pull request label Nov 10, 2025
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Nov 10, 2025
@fluxxBot fluxxBot changed the title Added skeleton code for docker build command Introduced support for docker build command Nov 12, 2025
@ehl-jf ehl-jf added the safe to test Approve running integration tests on a pull request label Nov 12, 2025
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Nov 12, 2025
@fluxxBot fluxxBot added the safe to test Approve running integration tests on a pull request label Nov 20, 2025
@bhanurp bhanurp self-requested a review December 4, 2025 06:09
@fluxxBot fluxxBot removed the safe to test Approve running integration tests on a pull request label Dec 4, 2025
@fluxxBot fluxxBot added the safe to test Approve running integration tests on a pull request label Dec 6, 2025
@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 Dec 9, 2025
@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 Dec 9, 2025
@fluxxBot fluxxBot enabled auto-merge (squash) December 9, 2025 11:13
@agrasth agrasth disabled auto-merge December 9, 2025 11:15
@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 Dec 9, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 9, 2025

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


@ehl-jf ehl-jf merged commit 605ffde into jfrog:master Dec 9, 2025
67 of 68 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

jfrog-internal Items created by jfrog employees new feature 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.

5 participants