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

Bugfix yaml parsing for GKEStartKueueInsideClusterOperator #39234

Merged
merged 2 commits into from May 7, 2024

Conversation

moiseenkov
Copy link
Contributor

Bugfix yaml parsing for GKEStartKueueInsideClusterOperator

@boring-cyborg boring-cyborg bot added area:providers area:system-tests provider:cncf-kubernetes Kubernetes provider related issues provider:google Google (including GCP) related issues labels Apr 24, 2024
@moiseenkov moiseenkov force-pushed the start_kueue_fix_yaml_parsing branch 4 times, most recently from fb279e3 to cf5a92f Compare April 26, 2024 14:59
@VladaZakharova
Copy link
Contributor

Hi @eladkal @potiuk !
Can you please check changes here? Thanks!

@moiseenkov moiseenkov force-pushed the start_kueue_fix_yaml_parsing branch 3 times, most recently from 13a0256 to 86d37d1 Compare May 6, 2024 10:49
@eladkal
Copy link
Contributor

eladkal commented May 6, 2024

can we have unit test to avoid regression?

@moiseenkov moiseenkov force-pushed the start_kueue_fix_yaml_parsing branch from 91b8e56 to cc21e4e Compare May 7, 2024 06:59
@moiseenkov
Copy link
Contributor Author

can we have unit test to avoid regression?

Sure, I added tests. Please, take a look.

@moiseenkov moiseenkov force-pushed the start_kueue_fix_yaml_parsing branch from cc21e4e to 28b3fde Compare May 7, 2024 09:25
@moiseenkov moiseenkov force-pushed the start_kueue_fix_yaml_parsing branch from 28b3fde to 29ef8ca Compare May 7, 2024 12:14
@eladkal eladkal merged commit 287c107 into apache:main May 7, 2024
39 checks passed
RodrigoGanancia pushed a commit to RodrigoGanancia/airflow that referenced this pull request May 10, 2024
)

* Bugfix yaml parsing for GKEStartKueueInsideClusterOperator

* Unit tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:providers area:system-tests provider:cncf-kubernetes Kubernetes provider related issues provider:google Google (including GCP) related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants