Skip to content

[e2e] Fix an error when "env_vars" is not set.#22234

Merged
rkooo567 merged 1 commit intoray-project:masterfrom
fishbone:e2e-env-vars
Feb 9, 2022
Merged

[e2e] Fix an error when "env_vars" is not set.#22234
rkooo567 merged 1 commit intoray-project:masterfrom
fishbone:e2e-env-vars

Conversation

@fishbone
Copy link
Copy Markdown
Contributor

@fishbone fishbone commented Feb 9, 2022

Why are these changes needed?

To fix error in session https://buildkite.com/ray-project/periodic-ci/builds/2699#c532ed2b-ee89-48ad-a7db-fd4211ef8bd9

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 :(

@rkooo567 rkooo567 merged commit 8b1bbfe into ray-project:master Feb 9, 2022
mwtian added a commit that referenced this pull request Feb 15, 2022
* [e2e] Update e2e test to use redisless ray by default. (#22189)

As title, after infra got updated, we need to merge the PR so that test can run ray without redis.

* [e2e] Fix an error when "env_vars" is not set. (#22234)

To fix error in session https://buildkite.com/ray-project/periodic-ci/builds/2699#c532ed2b-ee89-48ad-a7db-fd4211ef8bd9

Co-authored-by: Yi Cheng <74173148+iycheng@users.noreply.github.com>
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.

2 participants