Skip to content

ROX-33792: optimize operator builds #19417

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
janisz wants to merge 34 commits into from
Closed

ROX-33792: optimize operator builds #19417

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
34 commits
Select commit Hold shift + click to select a range
efe108a
perf(ci): build operator binary once outside Docker for faster builds
janisz Mar 18, 2026
3704d9f
perf(ci): build operator binaries once per arch instead of per branding
janisz Mar 18, 2026
20109b4
perf(ci): consolidate operator build into main-build-nodeps
janisz Mar 19, 2026
f71ff1a
perf(ci): optimize operator build by extracting bundle to separate job
janisz Mar 19, 2026
d0c0468
fix(ci): run pre-build-operator-bundle on ubuntu-latest instead of co…
janisz Mar 19, 2026
46370a4
chore(ci): remove unnecessary Docker Buildx setup from bundle job
janisz Mar 19, 2026
ddc1953
perf(ci): compute operator tag once in bundle job
janisz Mar 19, 2026
d04befd
fix(operator): only use prebuilt binaries in CI to avoid stale builds
janisz Mar 19, 2026
a76549d
fix(ci): sync .containerignore with .dockerignore
janisz Mar 19, 2026
6c50ea5
perf(ci): remove container from push-operator-manifests job
janisz Mar 20, 2026
45b8f15
perf(ci): consolidate build-and-push-operator to single job with loops
janisz Mar 20, 2026
49dcf98
fix(ci): download artifacts to separate directories to avoid conflicts
janisz Mar 20, 2026
1922332
perf(ci): merge operator build and manifest jobs into one
janisz Mar 20, 2026
160eeef
fix(ci): address shellcheck warnings in operator build workflow
janisz Mar 20, 2026
da07c95
refactor(ci): extract operator build logic to dedicated script
janisz Mar 20, 2026
748345f
revert stackrox-operator cmd rename
janisz Mar 23, 2026
dadf683
Update .github/workflows/build.yaml
janisz Mar 24, 2026
7c312c6
refactor(ci): address code review comments for operator build
janisz Mar 24, 2026
ab0e532
cleanup operator-bundle
janisz Mar 25, 2026
a016547
perf(ci): optimize checkout times by pre-computing main tag
janisz Mar 25, 2026
d889496
fix git ownership
janisz Mar 25, 2026
28d5326
perf(ci): move tag computation to define-job-matrix
janisz Mar 25, 2026
74e6b32
chore(ci): remove redundant fetch-depth: 1 settings
janisz Mar 25, 2026
0e29168
fix(ci): use BUILD_TAG and SHORTCOMMIT for shallow clones
janisz Mar 25, 2026
7588692
Apply suggestion from @porridge
janisz Mar 25, 2026
8168ac0
refactor(ci): use consistent naming for short commit output
janisz Mar 25, 2026
4a9fedd
refactor(ci): rename main-tag to build-tag for consistency
janisz Mar 25, 2026
ded96e5
remove unused matrix job
janisz Mar 25, 2026
756bd07
remove pre-
janisz Mar 25, 2026
9a7e19f
fixes
janisz Mar 25, 2026
1b82680
refactor(ci): use registry_from_branding helper function
janisz Mar 25, 2026
c2253ef
revert scafolding changes
janisz Mar 25, 2026
49a5817
fix(ci): use prebuilt operator binary and fix bash syntax error
janisz Mar 25, 2026
30c697d
fix(build): only set GOTAGS=release for actual release tags
janisz Mar 25, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .containerignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,9 @@
# Having this file enables us to have different exclusion list for RHTAP builds than in GHA.
# To regenerate, replace the below contents with the output of the following command (please keep these comments).
# grep -vF -e '/.git/' -e '/image/' -e '/qa-tests-backend/' .dockerignore

**/node_modules/
/bin/
!/bin/linux_*/stackrox-operator
/linux-gocache/
/operator/*/bin/
.dccache
1 change: 1 addition & 0 deletions .dockerignore
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
**/node_modules/
/.git/
/bin/
!/bin/linux_*/stackrox-operator
Copy link
Copy Markdown
Contributor

@davdhacs davdhacs Mar 25, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Locally, if I have the stackrox-operator binary it will get included in my other docker built images from here, won't it? Maybe this could go in a different .dockerignore?

Copy link
Copy Markdown
Contributor Author

@janisz janisz Mar 25, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It will be included in builder context but will not appear in final image unless you COPY or ADD it. https://docs.docker.com/build/concepts/context/

/image/bin/
/image/rhel/bin/
/linux-gocache/
Expand Down
Loading
Loading