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

fix: Issue #1391 - Custom labels not being pulled in #1398

Merged
merged 2 commits into from
Sep 8, 2023

Conversation

rxraghu
Copy link
Contributor

@rxraghu rxraghu commented Sep 6, 2023

Why are these changes needed?

fixes #1391 Some organizations have strict labeling requirements for their K8s objects, mainly deployments and pods. kuberay-operator was allowing to add custom labels via values.yaml to pods but not to deployments and kuberay-apiserver was not allowing to add custom labels to either. Added code to pull labels for deployment if present in values.yaml. Also, added code to pull annotations for apiserver as well as is the case with operator.

Related issue number

Closes #1391

Checks

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

@rxraghu
Copy link
Contributor Author

rxraghu commented Sep 6, 2023

I tested this test_sample_rayservice_yamls.py it seems to be failing even without my changes.

Copy link
Member

@kevin85421 kevin85421 left a comment

Choose a reason for hiding this comment

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

LGTM. Would you mind adding more details and a screenshot demonstrating how you tested this PR manually before I merge it? There is no test for this PR. Typically, I would clone your fork and conduct manual tests myself, but I have been very busy with the Ray Summit recently.

@rxraghu
Copy link
Contributor Author

rxraghu commented Sep 8, 2023

Thanks for the review @kevin85421. Here is how I tested the PR. I added the labels and annotations as shown below to the values.yaml of kuberay-operator and kuberay-apiserver:

labels:
  dummy-label: "dummy-value"

annotations:
  dummy-annotation: "dummy-value"

Then installed both using helm install:

# cd into respective directory
helm install kuberay-operator .
helm install kuberay-apiserver .

Then checked that the dummy label and annotation appeared on the pods and only dummy-label appeared on the deployment using Lens. Attached screenshots for apiserver-deployment, operator-deployment, apiserver-pod and operator-pod.
apiserver-deployment
operator-deployment
apiserver-pod
operator-pod

@rxraghu
Copy link
Contributor Author

rxraghu commented Sep 8, 2023

@kevin85421 - Also, can you share when you guys might do the next release? Just to get an estimate of when these changes might be available. Thanks!

@kevin85421
Copy link
Member

Also, can you share when you guys might do the next release? Just to get an estimate of when these changes might be available. Thanks!

The Ray Summit is scheduled to take place from September 18 to September 20. Before the Summit commences, we plan to release the first candidate for KubeRay v1.0. Following this, we anticipate a one-month period of iteration with users before officially releasing the final version of v1.0.

@kevin85421 kevin85421 merged commit f199dc1 into ray-project:master Sep 8, 2023
21 checks passed
z103cb pushed a commit to z103cb/kuberay that referenced this pull request Sep 11, 2023
lowang-bh pushed a commit to lowang-bh/kuberay that referenced this pull request Sep 24, 2023
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.

[Bug] Custom labels are not pulled in for kuberay-operator and kuberay-apiserver
2 participants