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

Moving machinelearningservices(1), maintenance(3), managed(1) to v1beta1 version #365

Conversation

svscheg
Copy link
Contributor

@svscheg svscheg commented Feb 16, 2023

Description of your changes

machinelearningservices group:

maintenance group:

managed group:

Fixes #270

I have:

  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Manually
maintenance group:

Screenshot 2023-02-14 at 17 09 41

Screenshot 2023-02-20 at 17 49 32

  • azurerm_maintenance_configuration - Unstable resource. Passed on manual runs with True condition and kubectl annotate managed --all upjet.upbound.io/test=true --overwrite

Screenshot 2023-02-20 at 20 41 36

Uptest
machinelearningservices group:

managed group:

@svscheg svscheg force-pushed the logz_machinelearningservices_maintenance_managed branch 2 times, most recently from 6546c7c to 0843448 Compare February 16, 2023 20:46
@svscheg
Copy link
Contributor Author

svscheg commented Feb 16, 2023

/test-examples="examples/machinelearningservices/synapsespark.yaml"

@svscheg
Copy link
Contributor Author

svscheg commented Feb 17, 2023

/test-examples="examples/compute/manageddisksastoken.yaml"

@svscheg svscheg force-pushed the logz_machinelearningservices_maintenance_managed branch 2 times, most recently from 4eccba8 to 8c5391f Compare February 20, 2023 12:49
@svscheg svscheg changed the title Moving machinelearningservices(4), maintenance(3), managed(1) to v1beta1 version Moving machinelearningservices(1), maintenance(3), managed(1) to v1beta1 version Feb 20, 2023
@svscheg svscheg force-pushed the logz_machinelearningservices_maintenance_managed branch 2 times, most recently from d8b8152 to 5df3134 Compare February 20, 2023 13:00
@svscheg
Copy link
Contributor Author

svscheg commented Feb 20, 2023

/test-examples="examples/compute/manageddisksastoken.yaml"

@svscheg
Copy link
Contributor Author

svscheg commented Feb 20, 2023

/test-examples="examples/machinelearningservices/workspace.yaml"

@svscheg
Copy link
Contributor Author

svscheg commented Feb 20, 2023

/test-examples="examples/maintenance/maintenanceassignmentvirtualmachine.yaml"

@svscheg svscheg requested a review from sergenyalcin February 20, 2023 14:07
@svscheg
Copy link
Contributor Author

svscheg commented Feb 20, 2023

/test-examples="examples/maintenance/maintenanceconfiguration.yaml"

1 similar comment
@svscheg
Copy link
Contributor Author

svscheg commented Feb 20, 2023

/test-examples="examples/maintenance/maintenanceconfiguration.yaml"

@svscheg svscheg force-pushed the logz_machinelearningservices_maintenance_managed branch from 5df3134 to f4b4b08 Compare February 20, 2023 16:59
@svscheg
Copy link
Contributor Author

svscheg commented Feb 20, 2023

/test-examples="examples/maintenance/maintenanceconfiguration.yaml"

@svscheg svscheg force-pushed the logz_machinelearningservices_maintenance_managed branch 2 times, most recently from 50ec59f to 74234c8 Compare February 20, 2023 19:00
Copy link
Collaborator

@sergenyalcin sergenyalcin left a comment

Choose a reason for hiding this comment

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

Thanks @svscheg LGTM!

@svscheg svscheg force-pushed the logz_machinelearningservices_maintenance_managed branch from 74234c8 to 4be48a9 Compare February 21, 2023 21:44
@svscheg svscheg merged commit 63388a2 into crossplane-contrib:main Feb 21, 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.

Moving machinelearningservices(1), maintenance(3), managed(1) to v1beta1 version
3 participants