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

Feature/add mongo user role resources #859

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

knutejoh
Copy link
Contributor

@knutejoh knutejoh commented Nov 1, 2024

Description of your changes

Added resources
azurerm_cosmosdb_mongo_role_definition
azurerm_cosmosdb_mongo_user_definition

Fixes #858

I have:

  • [ X ] Read and followed Crossplane's [contribution process].
  • [ X ] Run make reviewable to ensure this PR is ready for review.
  • Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Test with kind as local cluster provisioning in Azure.

@knutejoh
Copy link
Contributor Author

knutejoh commented Nov 1, 2024

/test-examples="examples/cosmosdb/v1beta1/mongoroledefinition.yaml"

1 similar comment
@knutejoh
Copy link
Contributor Author

knutejoh commented Nov 1, 2024

/test-examples="examples/cosmosdb/v1beta1/mongoroledefinition.yaml"

@knutejoh
Copy link
Contributor Author

knutejoh commented Nov 1, 2024

/test-examples="examples/cosmosdb/v1beta1/mongouserdefinition.yaml"

@knutejoh
Copy link
Contributor Author

knutejoh commented Nov 1, 2024

@turkenf @ulucinar @sergenyalcin @erhancagirici
New resource ready for code-review. Note the new func mongoDatabaseBasedId in externalname.go which was needed for extracting the actual ID for these two resources.

@turkenf
Copy link
Collaborator

turkenf commented Nov 22, 2024

Hi @knutejoh, thank you for the contribution, could you please rebase your PR to the main and resolve the conflicts to move forward?

@knutejoh knutejoh force-pushed the feature/add-mongo-user-role-resources branch from 35f520a to a1967c0 Compare November 28, 2024 12:58
@knutejoh
Copy link
Contributor Author

/test-examples="examples/cosmosdb/v1beta1/mongoroledefinition.yaml"

@knutejoh
Copy link
Contributor Author

/test-examples="examples/cosmosdb/v1beta1/mongouserdefinition.yaml"

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.

Request for mongodb user and role definition resources
2 participants