Skip to content

Conversation

@eusebiu-constantin-petu-dbk
Copy link
Collaborator

Signed-off-by: Petu Eusebiu [email protected]

What type of PR is this?

Which issue does this PR fix:

What does this PR do / Why do we need it:

If an issue # is not available please add repro steps and logs showing the issue:

Testing done on this change:

Automation added to e2e:

Will this break upgrades or downgrades?

Does this PR introduce any user-facing change?:


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@eusebiu-constantin-petu-dbk
Copy link
Collaborator Author

@codecov
Copy link

codecov bot commented Apr 12, 2022

Codecov Report

Merging #499 (85e0762) into main (5e35dfa) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #499   +/-   ##
=======================================
  Coverage   85.07%   85.07%           
=======================================
  Files          52       52           
  Lines       10682    10682           
=======================================
  Hits         9088     9088           
  Misses       1236     1236           
  Partials      358      358           

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@rchincha
Copy link
Contributor

This looks good, but where the "build-args" changes made to stacker-build-push-action?

@eusebiu-constantin-petu-dbk
Copy link
Collaborator Author

This looks good, but where the "build-args" changes made to stacker-build-push-action?

I made a PR there and merged it, look there.

@rchincha
Copy link
Contributor

`
$ ls Dockerfile*

Dockerfile Dockerfile-conformance Dockerfile-minimal Dockerfile-zb Dockerfile-zxp
`

Why not the -conformance one?

@rchincha rchincha force-pushed the test_stacker branch 2 times, most recently from df36d1b to 42c29d7 Compare April 15, 2022 18:06
@eusebiu-constantin-petu-dbk eusebiu-constantin-petu-dbk force-pushed the test_stacker branch 15 times, most recently from b731673 to bf46990 Compare April 18, 2022 17:22
@eusebiu-constantin-petu-dbk eusebiu-constantin-petu-dbk force-pushed the test_stacker branch 4 times, most recently from 5ccacbe to 2961499 Compare April 19, 2022 12:53
@eusebiu-constantin-petu-dbk eusebiu-constantin-petu-dbk force-pushed the test_stacker branch 2 times, most recently from f3e24fe to 85e0762 Compare April 19, 2022 13:09
Copy link
Contributor

@rchincha rchincha left a comment

Choose a reason for hiding this comment

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

lgtm

@rchincha rchincha merged commit ad90a49 into project-zot:main Apr 19, 2022
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.

2 participants