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

feat(helm): add nodeselector and tolerations to init job #23416

Conversation

dim-ops
Copy link
Contributor

@dim-ops dim-ops commented Mar 18, 2023

Add nodeselector and tolerations to init job for this issue: #23109

@dim-ops dim-ops force-pushed the feat/helm-add-nodeselector-and-tolerations-to-init-job branch from 62c7161 to 4111d28 Compare March 18, 2023 10:43
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Congrats on making your first PR and thank you for contributing to Superset! 🎉 ❤️

We hope to see you in our Slack community too! Not signed up? Use our Slack App to self-register.

@dim-ops dim-ops force-pushed the feat/helm-add-nodeselector-and-tolerations-to-init-job branch from 4111d28 to 791a567 Compare March 20, 2023 07:02
@pull-request-size pull-request-size bot added size/M and removed size/S labels Mar 20, 2023
@dim-ops dim-ops requested a review from nytai March 20, 2023 07:17
@dim-ops dim-ops requested review from craig-rueda and nytai and removed request for nytai and craig-rueda March 21, 2023 20:27
@dim-ops dim-ops force-pushed the feat/helm-add-nodeselector-and-tolerations-to-init-job branch from 14288fa to 304dbc2 Compare March 23, 2023 08:03
@dim-ops dim-ops force-pushed the feat/helm-add-nodeselector-and-tolerations-to-init-job branch from 304dbc2 to ec324c2 Compare March 23, 2023 16:26
@dim-ops
Copy link
Contributor Author

dim-ops commented Mar 23, 2023

@craig-rueda is it possible to merge ? Because I fixed my code because which was in conflict with #23452 which you approved and merged while mine is more complete.

@craig-rueda craig-rueda merged commit 0fa4212 into apache:master Mar 24, 2023
@maximdogonov
Copy link

Hi,

I setup:
nodeSelector:
nodetype: core-ccx23

but it applies only for superset and worker.

Redis and Postgresql with out node-selector.

How to fix that?

@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 3.0.0 labels Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels size/M 🚢 3.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants