Skip to content
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.

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

Build Docker manifest only when all dependencies are ready #6621

Merged
merged 1 commit into from
Aug 31, 2021
Merged

Build Docker manifest only when all dependencies are ready #6621

merged 1 commit into from
Aug 31, 2021

Conversation

omus
Copy link
Contributor

@omus omus commented Aug 30, 2021

Fixes #6541. The push-linux-amd64-images step is the same as the push-images step but only used the linux/amd64 platform.

@alexec alexec enabled auto-merge (squash) August 31, 2021 00:58
@codecov
Copy link

codecov bot commented Aug 31, 2021

Codecov Report

Merging #6621 (1bc8aea) into master (e10f644) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #6621   +/-   ##
=======================================
  Coverage   48.49%   48.49%           
=======================================
  Files         261      261           
  Lines       18955    18978   +23     
=======================================
+ Hits         9192     9204   +12     
- Misses       8740     8746    +6     
- Partials     1023     1028    +5     
Impacted Files Coverage Δ
workflow/sync/chain_throttler.go 68.75% <0.00%> (-31.25%) ⬇️
workflow/metrics/server.go 15.78% <0.00%> (-3.51%) ⬇️
workflow/sync/throttler.go 93.20% <0.00%> (-3.39%) ⬇️
server/workflow/workflow_server.go 44.41% <0.00%> (-2.40%) ⬇️
workflow/controller/operator.go 71.47% <0.00%> (-0.10%) ⬇️
cmd/argo/commands/get.go 59.18% <0.00%> (+0.87%) ⬆️
workflow/controller/controller.go 22.46% <0.00%> (+1.48%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e10f644...1bc8aea. Read the comment docs.

@alexec alexec merged commit d4fe873 into argoproj:master Aug 31, 2021
@sarabala1979 sarabala1979 mentioned this pull request Sep 2, 2021
61 tasks
@sarabala1979 sarabala1979 mentioned this pull request Sep 9, 2021
68 tasks
@omus omus deleted the cv/build-manifest-once branch September 9, 2021 19:37
JPZ13 pushed a commit to JPZ13/argo-workflows that referenced this pull request Sep 26, 2021
…rgoproj#3523, argoproj#2063

Signed-off-by: Alex Collins <[email protected]>

mre

Signed-off-by: Alex Collins <[email protected]>

bits

Signed-off-by: Alex Collins <[email protected]>

fix(executor): Disambiguate PNS executor initialization log (argoproj#6582)

Signed-off-by: Yuan Tang <[email protected]>

ci: Disable builds on forks (argoproj#6589)

Signed-off-by: Yuan Tang <[email protected]>

fix!: Enable authentication by default on Argo Server `/metrics` endpoint. Fixes argoproj#6592 (argoproj#6595)

Signed-off-by: Alex Collins <[email protected]>

ok

Signed-off-by: Alex Collins <[email protected]>

ok

Signed-off-by: Alex Collins <[email protected]>

ok

Signed-off-by: Alex Collins <[email protected]>

chore: Upgrade cobra to v1.2.1  (argoproj#6597)

Signed-off-by: Alex Collins <[email protected]>

ok

Signed-off-by: Alex Collins <[email protected]>

ok

Signed-off-by: Alex Collins <[email protected]>

docs: Document auth rate limiting. Fixes argoproj#5217

docs: Document IP address logging. Fixes argoproj#5216

fix: Fix `gosec` warnings, disable pprof by default. Fixes argoproj#6594 (argoproj#6596)

Signed-off-by: Alex Collins <[email protected]>

ok

Signed-off-by: Alex Collins <[email protected]>

fix tests

Signed-off-by: Alex Collins <[email protected]>

fix/skip tests

Signed-off-by: Alex Collins <[email protected]>

ok

Signed-off-by: Alex Collins <[email protected]>

ok

Signed-off-by: Alex Collins <[email protected]>

o

Signed-off-by: Alex Collins <[email protected]>

ok

Signed-off-by: Alex Collins <[email protected]>

ok

Signed-off-by: Alex Collins <[email protected]>

docs: Fix template-defaults duplicated in docs and add missing whitespace in h1 (argoproj#6601)

Signed-off-by: Michael Pöllath <[email protected]>

docs: fix continue on failure dag example (argoproj#6609)

Signed-off-by: Siebren Zwerver <[email protected]>

fix: manifests/quick-start/sso for running locally PROFILE=sso (argoproj#6503)

Signed-off-by: Tetsuya Shiota <[email protected]>

chore: Run `make codegen`

Signed-off-by: Alex Collins <[email protected]>

build: disable UI by default for `make start`

docs: Document argoproj#6297 breaking change (argoproj#6616)

docs: Remove sym-links from docs (argoproj#6617)

Signed-off-by: Alex Collins <[email protected]>

upgrade to v0.0.9

Signed-off-by: Alex Collins <[email protected]>

fix test

Signed-off-by: Alex Collins <[email protected]>

fix tests

Signed-off-by: Alex Collins <[email protected]>

add missing label selector

Signed-off-by: Alex Collins <[email protected]>

add transport wrappers

Signed-off-by: Alex Collins <[email protected]>

fix clean-up keys

Signed-off-by: Alex Collins <[email protected]>

ok

Signed-off-by: Alex Collins <[email protected]>

ok

Signed-off-by: Alex Collins <[email protected]>

fix: quay.io stuffs

Signed-off-by: Alex Collins <[email protected]>

fix: support in-cluster correctly

Signed-off-by: Alex Collins <[email protected]>

ok

Signed-off-by: Alex Collins <[email protected]>

fix(controller): Initialize throttler during starting workflow-controller. Fixes: argoproj#6599 (argoproj#6608)

Signed-off-by: smile-luobin <[email protected]>

 docs: Add slack exit handler example. Resolves argoproj#4152  (argoproj#6612)

Signed-off-by: J.P. Zivalich <[email protected]>

fix: Argo Workflow specs link to not go to raw content (argoproj#6624)

Signed-off-by: Andrey Melnikov <[email protected]>

ci: Build Docker manifest with complete dep list (argoproj#6621)

Signed-off-by: Curtis Vogt <[email protected]>

fix: Upgrade Dataflow to v0.0.96 (argoproj#6622)

Signed-off-by: Alex Collins <[email protected]>

ok

Signed-off-by: Alex Collins <[email protected]>

docs: Fix incorrect link to examples (argoproj#6630)

Signed-off-by: Yuan Tang <[email protected]>

fix: Fixed typo in clusterrole (argoproj#6626)

Signed-off-by: Saravanan Balasubramanian <[email protected]>

build: Fix repository prefix (argoproj#6636)

Signed-off-by: Alex Collins <[email protected]>

feat: Upgrade dataflow to v0.0.98 (argoproj#6637)

Signed-off-by: Alex Collins <[email protected]>

docs: correct https://bit.ly/book-30m-with-argo-team URL

feat(controller): Add a shared index informer for ConfigMaps (argoproj#6644)

Signed-off-by: Yuan Tang <[email protected]>

Fix duplicated import

Signed-off-by: J.P. Zivalich <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

argoproj/argoexec:latest sometimes only contains linux/amd64
2 participants