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

Update RayService CR to integrate with Ray Nightly #322

Merged
merged 61 commits into from
Jul 5, 2022

Conversation

brucez-anyscale
Copy link
Contributor

@brucez-anyscale brucez-anyscale commented Jun 25, 2022

Why are these changes needed?

The ray nightly changes the serve endpoints. This pr applies the new schema.

Manual test results:
Screen Shot 2022-06-28 at 4 36 29 PM

Related issue number

Checks

  • I've made sure the tests are passing.
  • Testing Strategy
    • Unit tests
    • Manual tests
    • This PR is not tested :(

@brucez-anyscale
Copy link
Contributor Author

@edoakes @DmitriGekhtman Please review this pr and see if we can merge it. Thanks

Copy link
Collaborator

@DmitriGekhtman DmitriGekhtman left a comment

Choose a reason for hiding this comment

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

The parts not related to the serve schema look fine to me.
@edoakes @simon-mo or @shrekris-anyscale should review serve-specific content.

Copy link
Contributor

@shrekris-anyscale shrekris-anyscale left a comment

Choose a reason for hiding this comment

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

The updated Serve schemas look good to me. I added a few comments.

Copy link
Contributor

@shrekris-anyscale shrekris-anyscale left a comment

Choose a reason for hiding this comment

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

Approving the Serve schema changes

@brucez-anyscale brucez-anyscale merged commit 2d798aa into master Jul 5, 2022
@Jeffwan Jeffwan deleted the brucez/improveHANewSchema branch October 17, 2022 04:00
lowang-bh pushed a commit to lowang-bh/kuberay that referenced this pull request Sep 24, 2023
* draft for ha

* import fmt

* debug ingress

* Draft service

* update

* fix

* Update service logic

* update

* update

* Logs

* update

* debug

* Update

* Update

* Update

* update

* Fix cluster start flaky issue

* update

* Update service and ingress

* update rbac

* Draft v1

* Update

* address comments

* Address comments and refactor codes

* update

* Fix lint issue

* update

* Fix unit tests

* goImport

* Update unit tests

* Implement unit tests

* Change preparing to pending

* goimports

* update

* Improve the pr to show both statuses

* Improve the pr to show both statuses

* update to align with latest serve status

* update

* Fix ut and imports

* update

* update

* address comments

* update

* update delete ray cluster logic

* update delete ray cluster logic

* update

* Apply new spec schema for serve deployment

* use new schema

* auto generate

* auto generate

* goimports

* Update cpu number

* update ut

* update ut

* update

* update

* update

* update

* address comments
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.

5 participants