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

refactor: Upgrade the build phase of image in CI with gsctl #3142

Merged
merged 2 commits into from
Aug 27, 2023

Conversation

FLYLX
Copy link
Contributor

@FLYLX FLYLX commented Aug 24, 2023

What do these changes do?

Make bash scripts not dependent on bashly and refactor gsctl.

Related issue number

To resolve issue #3074 .
The previous pr #3101 was too messy, so I opened a new pr to submit an issue #3074 .

@codecov-commenter
Copy link

codecov-commenter commented Aug 24, 2023

Codecov Report

Merging #3142 (4db854a) into main (88c5180) will increase coverage by 24.17%.
Report is 1 commits behind head on main.
The diff coverage is 0.00%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #3142       +/-   ##
===========================================
+ Coverage   42.64%   66.81%   +24.17%     
===========================================
  Files         100      102        +2     
  Lines       10817    10979      +162     
===========================================
+ Hits         4613     7336     +2723     
+ Misses       6204     3643     -2561     
Files Changed Coverage Δ
gsctl.py 0.00% <ø> (ø)
python/graphscope/gsctl/gsctl.py 0.00% <0.00%> (ø)

... and 66 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

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

gsctl.py Outdated Show resolved Hide resolved
k8s/Makefile Outdated Show resolved Hide resolved
k8s/dockerfiles/graphscope-dev.Dockerfile Outdated Show resolved Hide resolved
k8s/dockerfiles/manylinux2014.Dockerfile Outdated Show resolved Hide resolved
python/graphscope/gsctl/gsctl.py Outdated Show resolved Hide resolved
python/graphscope/gsctl/gsctl.py Outdated Show resolved Hide resolved
python/graphscope/gsctl/gsctl.py Outdated Show resolved Hide resolved
python/graphscope/gsctl/gsctl.py Outdated Show resolved Hide resolved
python/graphscope/gsctl/gsctl.py Show resolved Hide resolved
python/graphscope/gsctl/gsctl.py Show resolved Hide resolved
@FLYLX FLYLX force-pushed the 3074-refactor-gsctl branch 3 times, most recently from b6b9040 to b149a91 Compare August 26, 2023 05:00
Added gsctl support.

Added gsctl support.

add

.

.

new

test

fix
@FLYLX FLYLX force-pushed the 3074-refactor-gsctl branch 5 times, most recently from bc4e187 to 28a549a Compare August 26, 2023 14:46
@FLYLX FLYLX changed the title [WIP] refactor the build phase of image in CI with gsctl [WIP] Upgrade the build phase of image in CI with gsctl Aug 26, 2023
@FLYLX FLYLX changed the title [WIP] Upgrade the build phase of image in CI with gsctl [WIP] feat(CI) Upgrade the build phase of image in CI with gsctl Aug 27, 2023
@lidongze0629 lidongze0629 changed the title [WIP] feat(CI) Upgrade the build phase of image in CI with gsctl feat(CI) Upgrade the build phase of image in CI with gsctl Aug 27, 2023
@lidongze0629 lidongze0629 changed the title feat(CI) Upgrade the build phase of image in CI with gsctl feat(ci) Upgrade the build phase of image in CI with gsctl Aug 27, 2023
add tempfile in gsctl

fix gsctl.py format

fix gsctl.py format
@lidongze0629 lidongze0629 changed the title feat(ci) Upgrade the build phase of image in CI with gsctl refactor(ci) Upgrade the build phase of image in CI with gsctl Aug 27, 2023
@lidongze0629 lidongze0629 changed the title refactor(ci) Upgrade the build phase of image in CI with gsctl refactor(ci): Upgrade the build phase of image in CI with gsctl Aug 27, 2023
@lidongze0629 lidongze0629 changed the title refactor(ci): Upgrade the build phase of image in CI with gsctl refactor: Upgrade the build phase of image in CI with gsctl Aug 27, 2023
@lidongze0629 lidongze0629 merged commit 6ed225b into alibaba:main Aug 27, 2023
50 of 55 checks passed
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.

3 participants