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

feature: Add flags, and update the necessary packages #2169

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

cubxxw
Copy link
Collaborator

@cubxxw cubxxw commented Apr 4, 2023

Signed-off-by: Xinwei Xiong(cubxxw) [email protected]

Describe what this PR does / why we need it

Does this pull request fix one issue?

Describe how you did it

Describe how to verify it

❯ go version
go version go1.18.1 linux/amd64
❯ sealer version
{"gitVersion":"unknown","gitCommit":"21fa921db","buildDate":"2023-04-20 12:07:40","goVersion":"go1.18.1","compiler":"gc","platform":"linux/amd64"}

❯ go version
go version go1.17 linux/amd64
❯ sealer version
{"gitVersion":"unknown","gitCommit":"1065d4f66","buildDate":"2023-04-20 12:12:44","goVersion":"go1.17","compiler":"gc","platform":"linux/amd64"}

Special notes for reviews

Signed-off-by: Xinwei Xiong(cubxxw) <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Apr 4, 2023

Codecov Report

Patch coverage has no change and project coverage change: -6.70 ⚠️

Comparison is base (e885fa8) 18.89% compared to head (1065d4f) 12.20%.

📣 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    #2169      +/-   ##
==========================================
- Coverage   18.89%   12.20%   -6.70%     
==========================================
  Files          96      263     +167     
  Lines        8918    22420   +13502     
==========================================
+ Hits         1685     2736    +1051     
- Misses       7003    19302   +12299     
- Partials      230      382     +152     
Flag Coverage Δ
e2e-tests 7.44% <ø> (?)
unit-tests 19.26% <0.00%> (+0.36%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
build/kubefile/parser/parse.go 56.21% <ø> (ø)
pkg/cluster-runtime/hook.go 3.75% <ø> (ø)
pkg/imageengine/buildah/build.go 0.00% <0.00%> (ø)
pkg/imageengine/buildah/load.go 0.00% <0.00%> (ø)

... and 176 files with indirect coverage changes

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 in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

flags.BoolVarP(&loadOpts.Quiet, "quiet", "q", false, "Suppress the output")
flags.StringVar(&loadOpts.TmpDir, "tmp-dir", "", "set temporary directory when load image. if not set, use system`s temporary directory")
flags.StringVar(&loadOpts.TmpDir, "tmp-dir", "", "Set temporary directory when load image. use system temporary directory is not (/var/tmp/)at present ")
Copy link
Member

@kakaZhou719 kakaZhou719 Apr 6, 2023

Choose a reason for hiding this comment

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

Set temporary directory when load image. use system temporary directory(/tmp/) if not present.

@starnop
Copy link
Collaborator

starnop commented Apr 18, 2023

@cubxxw please make sure the CI passed

@github-actions github-actions bot added the ImageBuilding related to all staff with image building label Apr 20, 2023
@cubxxw cubxxw changed the title fix: code format feature: Add flags, and update the necessary packages Apr 20, 2023
@cubxxw cubxxw requested a review from kakaZhou719 April 20, 2023 04:19
Signed-off-by: Xinwei Xiong(cubxxw) <[email protected]>

Adjust the pkg package

Signed-off-by: Xinwei Xiong(cubxxw) <[email protected]>

Adjust the pkg package

Signed-off-by: Xinwei Xiong(cubxxw) <[email protected]>

backup stash@2

Signed-off-by: Xinwei Xiong(cubxxw) <[email protected]>

feature: Add flags, and update the necessary packages

Signed-off-by: Xinwei Xiong(cubxxw) <[email protected]>

fix: avoid useless packets affecting cicd

Signed-off-by: Xinwei Xiong(cubxxw) <[email protected]>
@cubxxw cubxxw requested a review from starnop May 7, 2023 13:58
@kakaZhou719
Copy link
Member

@cubxxw , i see a lot of changes. could you pls split it to make our pr more pure?

  1. doc: add some description for cmd flag and const.
  2. code optimization
  3. seems it is some new code that i could not know its background . is it a bugfix or feature ? pls add more description .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ImageBuilding related to all staff with image building
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants