Skip to content

Rteco alpine#3561

Open
naveenku-jfrog wants to merge 33 commits into
masterfrom
RTECO-alpine
Open

Rteco alpine#3561
naveenku-jfrog wants to merge 33 commits into
masterfrom
RTECO-alpine

Conversation

@naveenku-jfrog

@naveenku-jfrog naveenku-jfrog commented Jun 19, 2026

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

naveenku-jfrog and others added 2 commits June 23, 2026 11:44
@naveenku-jfrog @cursoragent
RTECO-945 - Wire Alpine APK commands in jfrog-cli
e1cfd15 
Register jf apk / jf apk config / jf apk upload in jfrog-cli and update
module dependencies to pick up the new Alpine implementation:

- buildtools/cli.go: Add ApkCmd dispatcher with subcommand routing for
  config, upload, and native apk passthrough; extract --server-id, --repo,
  --alpine-version, --user, --password, --branch, --arch, --apply from args;
  builder-pattern construction of ApkCommand / ApkConfigCommand / ApkUploadCommand
- docs/buildtools/apkcommand/help.go: Usage, description, and argument
  reference strings for jf apk --help
- utils/cliutils/commandsflags.go: Add Apk constant, apkAlpineVersion /
  apkArch flag definitions, and Apk command flag list
- go.mod: Upgrade jfrog-cli-artifactory to v0.8.1-0.20260621212942-c4258681e69a
  and build-info-go to v1.13.1-0.20260621212604-330cfe272ba6

Co-authored-by: Cursor <cursoragent@cursor.com>
@naveenku-jfrog @cursoragent
RTECO-945 - Fix ineffassign: discard unused args after ExtractBoolFla…
213f8f6 
…gFromArgs

Co-authored-by: Cursor <cursoragent@cursor.com>
@naveenku-jfrog @cursoragent
RTECO-945 - Register apk in Ghost Frog SupportedTools
72addaf 
Add "apk" to SupportedTools so that when JFROG_CLI_GHOST_FROG=true and
the apk binary is symlinked to jf, invoking `apk add ...` is transparently
intercepted and rewritten to `jf apk add ...`.

Co-authored-by: Cursor <cursoragent@cursor.com>
@naveenku-jfrog @cursoragent
RTECO-945 - Add Alpine APK e2e test cases
98397ae 
29 e2e tests covering all jf apk subcommands: add, upgrade, del,
update, info, search, fetch, fix, audit, version, stats, config,
and upload.

Tests validate build-info structure, dependency ID format,
prod/transitive scopes, requestedBy chains, checksums, module
override, multiple-build isolation, JF flag stripping, Ghost Frog
wiring, env secret filtering, and the --alpine-version flag.

The TestApk flag and Alpine repo infrastructure (constants, repo
templates, main_test.go wiring) are in the prerequisite CI PR so
that the workflow can compile cleanly before this file is present.

Co-authored-by: Cursor <cursoragent@cursor.com>
@naveenku-jfrog @cursoragent
RTECO-945 - Rename TestApk→TestAlpine, ApkBuildName→AlpineBuildName
b6ad46e 
Address PR #3590 review: rename test flag and build-name constant to
use 'Alpine' prefix for consistency with AlpineLocalRepo/RemoteRepo/VirtualRepo.
Also update skip message to reference '-test.alpine=true'.

Co-authored-by: Cursor <cursoragent@cursor.com>
udaykb2 and others added 3 commits July 8, 2026 11:21
@udaykb2 @naveenku-jfrog
RTECO-1074 - Remove redundendent functions in agent namespace-1 (#3566)
d20ff7a 
* Remove redundant functions in agent namespace

* Update go.mod
# Conflicts:
#	go.mod
#	go.sum
@dependabot @naveenku-jfrog
Bump github.com/containerd/containerd from 1.7.32 to 1.7.33 (#3565)
49d07ae 
Bumps [github.com/containerd/containerd](https://github.com/containerd/containerd) from 1.7.32 to 1.7.33.
- [Release notes](https://github.com/containerd/containerd/releases)
- [Changelog](https://github.com/containerd/containerd/blob/main/RELEASES.md)
- [Commits](containerd/containerd@v1.7.32...v1.7.33)

---
updated-dependencies:
- dependency-name: github.com/containerd/containerd
  dependency-version: 1.7.33
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@udaykb2 @naveenku-jfrog
RTECO-1074 - Remove redundant functions in agent namespace-2 (#3567)
99ba219 
* Remove redundant functions in agent namespace

* Update go.mod and go.sum file
ehl-jf and others added 9 commits July 8, 2026 11:21
@ehl-jf @naveenku-jfrog
Fix ghost frog install to honour default ModePass tools (#3579)
3a90fdb 
* JGC-476 - Fix ghost frog install to honour default ModePass tools

* JGC-476 - Fix ghost frog workflow upload path and setup version mismatch
@ehl-jf @naveenku-jfrog
JGC-476 - Remove go from CI package-alias-tools to prevent test runne…
3693603 
…r interception (#3580)
@IL-Automation @jfrog-ecosystem @naveenku-jfrog
Bump version to 2.112.0 (#3583)
86d70fa 
Co-authored-by: IL-Automation <eco-system@jfrog.com>
@attiasas @naveenku-jfrog
E2e tests rt vcs (#3584)
a16f0e2
@udaykb2 @naveenku-jfrog
RTECO-1560 - Skip agentplugins tests (#3587)
c16f84c
@ehl-jf @naveenku-jfrog
JGC-596 - Hide survey link when invoked by an AI agent (#3588)
5d0d445 
Extend ShouldHideSurveyLink to also check ExecutionContext.IsAgent
from jfrog-cli-core, so the post-help survey prompt isn't shown when
jfrog-cli is invoked by an AI coding agent (Claude Code, Cursor,
Gemini CLI, etc.). Also clear agent-detector env vars in existing
survey tests so they stay deterministic when run from within an
agent shell.
@attiasas @naveenku-jfrog
E2e tests rt vsc more tools (#3585)
d3ac423
@naveenku-jfrog
Rteco alpine tests (#3590)
14ad79c 
* RTECO-945 - Add CI workflow to run Alpine APK tests
@naveenku-jfrog
RTECO-945 - Wire Alpine APK commands in jfrog-cli
c9d93e6 
Register jf apk / jf apk config / jf apk upload in jfrog-cli and update
module dependencies to pick up the new Alpine implementation:

- buildtools/cli.go: Add ApkCmd dispatcher with subcommand routing for
  config, upload, and native apk passthrough; extract --server-id, --repo,
  --alpine-version, --user, --password, --branch, --arch, --apply from args;
  builder-pattern construction of ApkCommand / ApkConfigCommand / ApkUploadCommand
- docs/buildtools/apkcommand/help.go: Usage, description, and argument
  reference strings for jf apk --help
- utils/cliutils/commandsflags.go: Add Apk constant, apkAlpineVersion /
  apkArch flag definitions, and Apk command flag list
- go.mod: Upgrade jfrog-cli-artifactory to v0.8.1-0.20260621212942-c4258681e69a
  and build-info-go to v1.13.1-0.20260621212604-330cfe272ba6

# Conflicts:
#	go.mod
#	go.sum
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

5 out of 6 committers have signed the CLA.
✅ (naveenku-jfrog)[https://github.com/naveenku-jfrog]
✅ (udaykb2)[https://github.com/udaykb2]
✅ (christophermiJfrog)[https://github.com/christophermiJfrog]
✅ (ehl-jf)[https://github.com/ehl-jf]
✅ (attiasas)[https://github.com/attiasas]
❌ @IL-Automation
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

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

@naveenku-jfrog @cursoragent
RTECO-945 - Fix golangci-lint errors in apk_test.go and buildtools/cl…
668c5f0 
…i.go

- errcheck: replace defer os.RemoveAll with clientTestUtils.RemoveAllAndAssert
  so the error return is not silently dropped (apk_test.go lines 658, 761)
- ineffassign: reassigned args is never read after ExtractBoolFlagFromArgs;
  use _ to make the intent explicit (buildtools/cli.go line 2215)

Co-authored-by: Cursor <cursoragent@cursor.com>
attiasas and others added 3 commits July 13, 2026 09:38
@attiasas @naveenku-jfrog
E2e tests rt vsc containers (#3586)
18d761a 
# Conflicts:
#	go.mod
#	go.sum
@IL-Automation @jfrog-ecosystem @naveenku-jfrog
Bump version to 2.113.0 (#3594)
f73b088 
Co-authored-by: IL-Automation <eco-system@jfrog.com>
@ehl-jf @naveenku-jfrog
JGC-525 - Add OpenAPI spec embed for jf api operation discovery (#3595)
e3dee5b 
- Add docs/api-spec/stub/ with 10 trimmed real operations (Access
  users/groups/permissions/tokens/roles, Artifactory system, Workers) extracted
  from JFrog's platform API reference
- Add the apispec package (docs/api-spec/) that lazily parses the embedded
  OpenAPI YAML once per process; ships a stub bundle by default and a full
  bundle behind the "full" build tag
- Add Makefile stub/full targets and a JF_BUILD_TAGS passthrough in
  build/build.sh for the internal release pipeline to opt into the full bundle
@naveenku-jfrog @cursoragent
RTECO-945 - Remove redundant Ghost Frog test from apk_test.go
a9cb1e1 
Ghost Frog behaviour is already comprehensively covered by the dedicated
ghostfrog_test.go suite (47 tests covering install, dispatch, kill-switch,
recursion-prevention, passthrough, and more). The per-package-manager smoke
test in apk_test.go adds no new coverage and is removed.

Co-authored-by: Cursor <cursoragent@cursor.com>
@naveenku-jfrog @cursoragent
RTECO-945 - Consolidate 10 passthrough tests into one table-driven test
bfcf51b 
Replace TestApkUpdate_Passthrough, TestApkUpdate_PassthroughWithBuildFlags,
TestApkDel_Passthrough, TestApkInfo_Passthrough, TestApkSearch_Passthrough,
TestApkFetch_Passthrough, TestApkFix_Passthrough, TestApkAudit_Passthrough,
TestApkVersion_Passthrough, and TestApkStats_Passthrough with a single
TestApkPassthroughCommands table-driven test. Same coverage, far less
boilerplate.

Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@naveenku-jfrog @udaykb2 @christophermiJfrog @IL-Automation @ehl-jf @attiasas