Skip to content

[release] move release testing end to end script to main ray repo#17070

Merged
ericl merged 4 commits intoray-project:masterfrom
krfricke:release-testing
Jul 14, 2021
Merged

[release] move release testing end to end script to main ray repo#17070
ericl merged 4 commits intoray-project:masterfrom
krfricke:release-testing

Conversation

@krfricke
Copy link
Copy Markdown
Contributor

Why are these changes needed?

cc @ericl @simon-mo @rkooo567

Related issue number

Checks

  • I've run scripts/format.sh to lint the changes in this PR.
  • I've included any doc changes needed for https://docs.ray.io/en/master/.
  • I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
  • Testing Strategy
    • Unit tests
    • Release tests
    • This PR is not tested :(

@krfricke krfricke marked this pull request as ready for review July 14, 2021 09:28
@krfricke
Copy link
Copy Markdown
Contributor Author

As can be seen here (branch release-testing) https://buildkite.com/ray-project/periodic-ci-ray-repo this works.

After merging we can disable the old releaser repo.

@krfricke
Copy link
Copy Markdown
Contributor Author

FYI, contents of the files are the same as in the releaser repo, with just minor alterations (linting and one change to the default RAY_PROJECT)

@krfricke krfricke requested a review from ericl July 14, 2021 11:30
@krfricke krfricke added the tests-ok The tagger certifies test failures are unrelated and assumes personal liability. label Jul 14, 2021
@ericl ericl added the @author-action-required The PR author is responsible for the next step. Remove tag to send back to the reviewer. label Jul 14, 2021
@@ -0,0 +1,214 @@
<!doctype html>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

should we upgrade this to buildkite input step?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Let's do this in a separate PR

@ericl ericl merged commit ed131f8 into ray-project:master Jul 14, 2021
@krfricke krfricke deleted the release-testing branch July 14, 2021 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

@author-action-required The PR author is responsible for the next step. Remove tag to send back to the reviewer. tests-ok The tagger certifies test failures are unrelated and assumes personal liability.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants