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

Replace XGBoost image for E2E with community hosted #1922

Merged
merged 1 commit into from
Sep 29, 2023

Conversation

tenzen-y
Copy link
Member

What this PR does / why we need it:
I replaced the image with community hosted one.
follow up: #1913

Which issue(s) this PR fixes (optional, in Fixes #<issue number>, #<issue number>, ... format, will close the issue(s) when PR gets merged):
Fixes #

Checklist:

  • Docs included if any changes are user facing

@tenzen-y tenzen-y changed the title Replace XGBoost image for E2E with community hosted WIP: Replace XGBoost image for E2E with community hosted Sep 28, 2023
@tenzen-y tenzen-y force-pushed the update-xgboost--e2e-image branch 2 times, most recently from 445e5a7 to bf7f5aa Compare September 28, 2023 12:03
@coveralls
Copy link

coveralls commented Sep 28, 2023

Pull Request Test Coverage Report for Build 6339877052

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.01%) to 42.805%

Totals Coverage Status
Change from base Build 6275548472: 0.01%
Covered Lines: 3745
Relevant Lines: 8749

💛 - Coveralls

@tenzen-y tenzen-y force-pushed the update-xgboost--e2e-image branch 3 times, most recently from 8ec9ed5 to 082d6a3 Compare September 28, 2023 12:39
@google-oss-prow google-oss-prow bot added size/S and removed size/XS labels Sep 28, 2023
@tenzen-y tenzen-y force-pushed the update-xgboost--e2e-image branch 2 times, most recently from 812e3b7 to 4f49c30 Compare September 28, 2023 13:53
Signed-off-by: Yuki Iwai <yuki.iwai.tz@gmail.com>
@tenzen-y tenzen-y force-pushed the update-xgboost--e2e-image branch from 4f49c30 to b4023e2 Compare September 28, 2023 14:29
Comment on lines +58 to +72
# This step is a Workaround to avoid the "No space left on device" error.
# ref: https://github.com/actions/runner-images/issues/2840
- name: Remove unnecessary files
shell: bash
run: |
sudo rm -rf /usr/share/dotnet
sudo rm -rf /opt/ghc
sudo rm -rf "/usr/local/share/boost"
sudo rm -rf "$AGENT_TOOLSDIRECTORY"
sudo rm -rf /usr/local/lib/android
sudo rm -rf /usr/local/share/powershell
sudo rm -rf /usr/share/swift

echo "Disk usage after cleanup:"
df -h
Copy link
Member Author

Choose a reason for hiding this comment

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

@tenzen-y tenzen-y changed the title WIP: Replace XGBoost image for E2E with community hosted Replace XGBoost image for E2E with community hosted Sep 28, 2023
@tenzen-y
Copy link
Member Author

/assign @johnugeorge @terrytangyuan

@tenzen-y
Copy link
Member Author

/assign @kuizhiqing

@kuizhiqing
Copy link
Member

lgtm

@tenzen-y
Copy link
Member Author

lgtm

Could you say /lgtm?

Copy link
Member

@terrytangyuan terrytangyuan left a comment

Choose a reason for hiding this comment

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

Thanks!

/lgtm

@google-oss-prow google-oss-prow bot added the lgtm label Sep 29, 2023
@google-oss-prow
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: tenzen-y, terrytangyuan

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • OWNERS [tenzen-y,terrytangyuan]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@google-oss-prow google-oss-prow bot merged commit 04936a0 into kubeflow:master Sep 29, 2023
29 checks passed
@tenzen-y tenzen-y deleted the update-xgboost--e2e-image branch September 29, 2023 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants