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.

Sign up for GitHub

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

Jump to bottom

sync manager task properties with CRD fields #303

Closed
zimnx opened this issue Dec 15, 2020 · 0 comments · Fixed by #305
Closed

sync manager task properties with CRD fields #303

zimnx opened this issue Dec 15, 2020 · 0 comments · Fixed by #305
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@zimnx
Copy link
Collaborator

zimnx commented Dec 15, 2020

Describe the bug

parallel flag for 2.2 repair is missing, check if CRD is in sync.
@zimnx zimnx added the kind/bug Categorizes issue or PR as related to a bug. label Dec 15, 2020
@zimnx zimnx added this to the 1.0 milestone Dec 15, 2020
zimnx added a commit that referenced this issue Dec 16, 2020
@zimnx
manager: synchronize task fileds with Manager 2.2 (#303)
2e7648d 
Intensity in Manager is float64, but k8s runtime controller doesn't support floats.
To workaround it, intensity becomes a string value, and validating webbook checks if
provided value is a correct float string.

Fixes #303
zimnx added a commit that referenced this issue Dec 16, 2020
@zimnx
manager: synchronize task fileds with Manager 2.2 (#303)
303d00e 
Intensity in Manager is float64, but k8s runtime controller doesn't support floats.
To workaround it, intensity becomes a string value, and validating webbook checks if
provided value is a correct float string.

Fixes #303
zimnx added a commit that referenced this issue Dec 16, 2020
@zimnx
manager: synchronize task fileds with Manager 2.2 (#303)
902c367 
Intensity in Manager is float64, but k8s runtime controller doesn't support floats.
To workaround it, intensity becomes a string value, and validating webhook checks if
provided value is a correct float string.

Fixes #303
@zimnx zimnx mentioned this issue Dec 16, 2020
Merged
2 tasks
zimnx added a commit that referenced this issue Dec 17, 2020
@zimnx
manager: synchronize task fileds with Manager 2.2 (#303)
043eb7a 
Intensity in Manager is float64, but k8s runtime controller doesn't support floats.
To workaround it, intensity becomes a string value, and validating webhook checks if
provided value is a correct float string.

Fixes #303
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

manager: synchronize task fileds with Manager 2.2 (#303) scylladb/scylla-operator
1 participant
@zimnx