-
Notifications
You must be signed in to change notification settings - Fork 355
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
Bugfix multi image support #2019
Bugfix multi image support #2019
Conversation
Codecov ReportBase: 18.40% // Head: 18.34% // Decreases project coverage by
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more Additional details and impacted files@@ Coverage Diff @@
## main #2019 +/- ##
==========================================
- Coverage 18.40% 18.34% -0.06%
==========================================
Files 101 101
Lines 9201 9225 +24
==========================================
- Hits 1693 1692 -1
- Misses 7277 7302 +25
Partials 231 231
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
59b037e
to
b37db58
Compare
pls rebase commits |
83519b0
to
b790c77
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
// if arch is amd64 | ||
// `COPY ${ARCH}/* .` will be mutated to `COPY amd64/* .` | ||
// `COPY $ARCH/* .` will be mutated to `COPY amd64/* .` | ||
_, arch, _, err := parse2.Platform(kp.platform) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could we add unit tests for these logic?
@@ -53,6 +53,6 @@ func NewPushCmd() *cobra.Command { | |||
// tls-verify is not working currently | |||
pushCmd.Flags().BoolVar(&pushOpts.TLSVerify, "tls-verify", true, "require HTTPS and verify certificates when accessing the registry. TLS verification cannot be used when talking to an insecure registry. (not work currently)") | |||
pushCmd.Flags().BoolVarP(&pushOpts.Quiet, "quiet", "q", false, "don't output progress information when pushing images") | |||
pushCmd.Flags().BoolVar(&pushOpts.All, "all", false, "also push the images in the list") | |||
pushCmd.Flags().BoolVar(&pushOpts.All, "all", true, "also push the images in the list") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this compatible?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it is just push command, i thought there will be no impact.
Could you please add e2e tests for this PR? |
test/suites/build/build.go
Outdated
return filepath.Join(settings.DefaultTestEnvDir, "suites", "build", "fixtures") | ||
// GetBuildImageName return specific image name for sealer build test | ||
func GetBuildImageName() string { | ||
return "docker.io/sealerio/forbuildtest:v1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
docker.io/sealerio/build-test:v1 is OK
bugfix: not panic if user wite invalid copy fix lint add e2e test for sealer build disable wrapper.IgnoreFile for build
cd7ea89
to
8855b23
Compare
/test all |
Describe what this PR does / why we need it
Kubefile:
FROM registry.cn-qingdao.aliyuncs.com/sealer-io/ackdistro-multi:v0.4.0 COPY ${ARCH}/bin bin APP myapp local://app.yaml LAUNCH myapp
build context
build multi image:
sealer build -f Kubefile -t my-kubernetes:1.19.8 --platform linux/amd64,linux/arm64
tag and push all images to docker hub:
Does this pull request fix one issue?
Describe how you did it
Describe how to verify it
Special notes for reviews