Skip to content

Feature: Moving to pure Dagger based CI/CD Pipeline from Goreleaser#547

Merged
bupd merged 24 commits into
goharbor:mainfrom
NucleoFusion:feat/dagger-release
Jan 20, 2026
Merged

Feature: Moving to pure Dagger based CI/CD Pipeline from Goreleaser#547
bupd merged 24 commits into
goharbor:mainfrom
NucleoFusion:feat/dagger-release

Conversation

@NucleoFusion

@NucleoFusion NucleoFusion commented Sep 13, 2025

Copy link
Copy Markdown
Contributor

Completed Release

The release binaries are here,
https://github.com/NucleoFusion/harbor-cli/releases/tag/v0.2.54

image

We only miss the SBOM's from the current release structure.

Apt repo

The apt repo files are here,
https://github.com/NucleoFusion/harbor-cli/tree/gh-pages

PS: Now updated so that only Packages.gz is there

Publish and Image

The images are at,
https://hub.docker.com/repository/docker/nucleofusion/harbor-cli

PS: Now re-use binaries instead of re-creation

Homebrew

The homebrew formula/PR for cron is at,
goharbor/homebrew-tap#5

PS: Now supports mac and linux both

Tracking Issue

#553

Releases

  • Build binaries - linux, windows, mac, bsd - amd64, arm64, armv6
  • Archives (tar.gz) w/ Checksum
  • .deb, .rpm w/ nFPM
  • Release Publish
  • Brew Publish

Misc

  • Doc Gen
  • Code Coverage
  • Lint & Lint Report
  • Tests
    • Run Tests
    • Test Coverage
    • Test Report

@NucleoFusion NucleoFusion changed the title Feature: Moving to pure Dagger based CI/CD Pipeline from Goreleaser [WIP] Feature: Moving to pure Dagger based CI/CD Pipeline from Goreleaser Sep 13, 2025
Comment thread dagger.json
@NucleoFusion NucleoFusion changed the title [WIP] Feature: Moving to pure Dagger based CI/CD Pipeline from Goreleaser Feature: Moving to pure Dagger based CI/CD Pipeline from Goreleaser Oct 5, 2025
@NucleoFusion NucleoFusion marked this pull request as ready for review October 5, 2025 05:59
@codecov

codecov Bot commented Oct 5, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 6.44%. Comparing base (60ad0bd) to head (19bbdb7).
⚠️ Report is 72 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##             main    #547      +/-   ##
=========================================
- Coverage   10.99%   6.44%   -4.55%     
=========================================
  Files         173     260      +87     
  Lines        8671   12842    +4171     
=========================================
- Hits          953     828     -125     
- Misses       7612   11908    +4296     
  Partials      106     106              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bupd bupd left a comment

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 have commented improvements

Comment thread .dagger/pipeline/brew.go Outdated
Comment thread .dagger/README.md
Comment thread dagger.json
Comment thread .github/workflows/default.yaml Outdated
@NucleoFusion NucleoFusion force-pushed the feat/dagger-release branch 2 times, most recently from cb31d76 to bd0b3db Compare October 13, 2025 04:27
@NucleoFusion NucleoFusion requested a review from bupd October 13, 2025 04:28

@bupd bupd left a comment

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.

@NucleoFusion added my suggestions

Comment thread .dagger/pipeline/brew.go Outdated
Comment thread .dagger/pipeline/checksum.go Outdated
Comment thread .dagger/pipeline/release.go Outdated
Comment thread .dagger/README.md
Comment thread .github/workflows/default.yaml

@bupd bupd left a comment

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.

lgtm

@bupd

bupd commented Oct 21, 2025

Copy link
Copy Markdown
Member

Missing docker publish.

Comment thread .github/workflows/default.yaml
@NucleoFusion

Copy link
Copy Markdown
Contributor Author

@bupd The docker publish is in the Dagger tracker, and I have added that as the highest priority.
I beleive we talked that this (amount of features currently implemented) would be enough but if required I can add the docker stuff

@bupd bupd left a comment

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.

This PR is bleeding out of its scope.

the scope is to replace the goreleaser with dagger. So any parts which were previously not been involved with goreleaser should stay the same as it is.

Comment thread .github/workflows/default.yaml Outdated
Comment thread .github/workflows/default.yaml Outdated
Comment thread .github/actions/publish-and-sign/action.yaml Outdated
Comment thread .github/actions/publish-and-sign/action.yaml Outdated
Comment thread .github/actions/publish-and-sign/action.yaml Outdated
Comment thread .github/actions/publish-and-sign/action.yaml
Comment thread .dagger/main.go Outdated
Comment thread .dagger/main.go Outdated
Comment thread .dagger/main.go Outdated
@NucleoFusion

Copy link
Copy Markdown
Contributor Author

@bupd my mistake sorry, I was so focused on making it fit with the theme/structure I had, I didnt pay attention that it got out of scope. I will fix this.
(The echo cmd was left there from when I was debugging and forgot to remove, my bad)

Comment thread .github/workflows/default.yaml Outdated
Comment thread .github/workflows/default.yaml

@bupd bupd left a comment

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.

lgtm

@bupd bupd left a comment

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.

@NucleoFusion resolve conflicts and remove the v in the tags following the previous release artifact naming

NucleoFusion and others added 23 commits January 10, 2026 11:26
… also removing goreleaser dependency

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
…inaries

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
…t dependency on others

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: env variable name fix

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: adding publish to the pipeline

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: creating release with publish-release and modifying token

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fixed token malformed error

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fixed improper call

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fixed issue with folder being uploaded

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fixing duplicate name error

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: splitting release creation and file upload

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: removing nfpm.yml upload

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: updating name for file

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: quotes on the release title

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: small workflow fixes

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: repo mismatch fix

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: uncomment

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fixing token issue

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: build error

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: resolving 404 error

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: resolving name mismatch error

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: switching to goharbor/harbor-cli

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: readding cosign dependency & updated README.md

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: README fix

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: filepath mismatch for checksum

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: renaming to use checksums.txt

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: renaming to use checksums.txt

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
Signed-off-by: Prasanth Baskar <bupdprasanth@gmail.com>
Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

temp: removing publish dependency on testing

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: binary build error

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: not releasing erropr

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: changing for goharbor/harbor-cli

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: adding back publish-release dependency on test-code

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

test

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

test

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

test

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

test

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

test

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
Co-authored-by: Prasanth Baskar <bupdprasanth@gmail.com>
Signed-off-by: Nucleo Fusion <148791547+NucleoFusion@users.noreply.github.com>
Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fixed build-dev error

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

refactor: refactoring Build and Archive functions to be separate

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

refactor: refactoring other functions

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: temp removal of test dependency

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: adding export in gh actions

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: removing export in release call

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: error in release, nil values

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: source mismatch

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: missing utils fix

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: apt repo

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: docker publish

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: missing dist folder

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: image tags error

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: adding verbosity

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: no such file, ./harbor

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: no such file, ./harbor

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: missing ':'

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: missing flag for build-dir

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: missing ending quotes

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fix

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fix

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fix

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fix

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fix

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fix

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: test

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: test

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: test

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: test

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fox

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fox

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fox

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fox

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fox

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fox

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fox

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fox

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fox

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fox

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fox

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fox

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

fix: fox

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

test

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

test

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>

test

Signed-off-by: NucleoFusion <lakshit.singh.mail@gmail.com>
@bupd bupd merged commit 88093f8 into goharbor:main Jan 20, 2026
6 of 8 checks passed
Vickysomtee pushed a commit to Vickysomtee/harbor-cli that referenced this pull request Feb 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update goreleaser The release process is regenerating binaries.

3 participants