Skip to content
This repository has been archived by the owner on Jul 10, 2024. It is now read-only.

SUBMARINE-1396. Split experiment-prehandler test into separate action #1093

Closed
wants to merge 4 commits into from

Conversation

cdmikechen
Copy link
Contributor

@cdmikechen cdmikechen commented Jul 29, 2023

What is this PR for?

The image build of experiment-prehandler resulted in a no space left on device exception. Now split experiment-prehandler build to a standalone script.
Currently, the e2e test already contains experiment-prehandler experiment related codes (a new issue has been created to handle unit test https://issues.apache.org/jira/browse/SUBMARINE-1397), so for now remove pyhton test.

What type of PR is it?

Hot Fix

Todos

  • - Strip experiment-prehandler from python tests

What is the Jira issue?

https://issues.apache.org/jira/browse/SUBMARINE-1396

How should this be tested?

CI test

Screenshots (if appropriate)

NA

Questions:

  • Do the license files need updating? No
  • Are there breaking changes for older versions? No
  • Does this need new documentation? No

@codecov
Copy link

codecov bot commented Jul 29, 2023

Codecov Report

Merging #1093 (31f058d) into master (0a554c4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1093   +/-   ##
=======================================
  Coverage   67.27%   67.27%           
=======================================
  Files         128      128           
  Lines        6224     6224           
=======================================
  Hits         4187     4187           
  Misses       2037     2037           
Flag Coverage Δ
python-integration 54.20% <ø> (+0.58%) ⬆️
python-unit 48.00% <ø> (ø)

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

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@cdmikechen cdmikechen requested a review from pingsutw July 29, 2023 13:45
@asfgit asfgit closed this in 5a5e3a6 Jul 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants