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

Add new module to deploy Azure Cognitive Search 'azure_rm_cognitivesearch' #372

Merged
merged 27 commits into from
Apr 12, 2021
Merged

Add new module to deploy Azure Cognitive Search 'azure_rm_cognitivesearch' #372

merged 27 commits into from
Apr 12, 2021

Conversation

next-davidduquehernandez
Copy link
Contributor

SUMMARY

This module allows to deploy, update and delete an Azure Cognitive Search.

ISSUE TYPE
  • New Module Pull Request
COMPONENT NAME
  • azure_rm_search
  • azure_rm_search_info
ADDITIONAL INFORMATION

All dev info could be found at https://docs.microsoft.com/en-us/rest/api/searchservice/

@Fred-sun Fred-sun added medium_priority Medium priority new_module_pr Add new modules work in In trying to solve, or in working with contributors labels Dec 30, 2020
@Fred-sun
Copy link
Collaborator

@next-davidduquehernandez Please add module name "azure_rm_search.py" to pr-pipeline.yml. Thank you very much!

Copy link
Collaborator

@Fred-sun Fred-sun left a comment

Choose a reason for hiding this comment

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

@next-davidduquehernandez Please help to adjust your test case --main.yml, thank you!

plugins/modules/azure_rm_search_info.py Outdated Show resolved Hide resolved
plugins/modules/azure_rm_search_info.py Outdated Show resolved Hide resolved
plugins/modules/azure_rm_search.py Outdated Show resolved Hide resolved
plugins/modules/azure_rm_search.py Outdated Show resolved Hide resolved
plugins/modules/azure_rm_search.py Outdated Show resolved Hide resolved
plugins/modules/azure_rm_search.py Outdated Show resolved Hide resolved
plugins/modules/azure_rm_search.py Outdated Show resolved Hide resolved
plugins/modules/azure_rm_search.py Outdated Show resolved Hide resolved
plugins/modules/azure_rm_search.py Outdated Show resolved Hide resolved
tests/integration/targets/azure_rm_search/tasks/main.yml Outdated Show resolved Hide resolved
@Fred-sun
Copy link
Collaborator

Fred-sun commented Feb 7, 2021

@next-davidduquehernandez Please fixes follow sanity error! Thank you very much!

ERROR: plugins/modules/azure_rm_search.py:0:0: nonexistent-parameter-documented: Argument 'append_tags' is listed in DOCUMENTATION.options, but not accepted by the module argument_spec
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'hosting_mode' in argument_spec defines type as 'str' but documentation doesn't define type
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'identity' in argument_spec defines type as 'str' but documentation doesn't define type
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'location' in argument_spec defines type as 'str' but documentation doesn't define type
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'name' in argument_spec defines type as 'str' but documentation doesn't define type
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'partition_count' in argument_spec defines type as 'int' but documentation doesn't define type
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'public_network_access' in argument_spec defines type as 'str' but documentation doesn't define type
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'replica_count' in argument_spec defines type as 'int' but documentation doesn't define type
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'resource_group' in argument_spec defines type as 'str' but documentation doesn't define type
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'sku' in argument_spec defines type as 'str' but documentation doesn't define type
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'state' in argument_spec defines type as 'str' but documentation doesn't define type

@next-davidduquehernandez
Copy link
Contributor Author

@next-davidduquehernandez Please fixes follow sanity error! Thank you very much!

ERROR: plugins/modules/azure_rm_search.py:0:0: nonexistent-parameter-documented: Argument 'append_tags' is listed in DOCUMENTATION.options, but not accepted by the module argument_spec
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'hosting_mode' in argument_spec defines type as 'str' but documentation doesn't define type
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'identity' in argument_spec defines type as 'str' but documentation doesn't define type
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'location' in argument_spec defines type as 'str' but documentation doesn't define type
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'name' in argument_spec defines type as 'str' but documentation doesn't define type
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'partition_count' in argument_spec defines type as 'int' but documentation doesn't define type
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'public_network_access' in argument_spec defines type as 'str' but documentation doesn't define type
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'replica_count' in argument_spec defines type as 'int' but documentation doesn't define type
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'resource_group' in argument_spec defines type as 'str' but documentation doesn't define type
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'sku' in argument_spec defines type as 'str' but documentation doesn't define type
ERROR: plugins/modules/azure_rm_search.py:0:0: parameter-type-not-in-doc: Argument 'state' in argument_spec defines type as 'str' but documentation doesn't define type

Hi @Fred-sun,

I'm so sorry, sometimes I forgot to follow all steps before commit and push my code :S.

I've verifyed that there is no sanity error message and integration tests are launched and finished correctly.

Regards!

@Fred-sun
Copy link
Collaborator

@next-davidduquehernandez Thank you for your contribution, please help to fix the plugins/module_utils/azure_rm_common.py file conflict! Thank you very much!

@next-davidduquehernandez
Copy link
Contributor Author

@Fred-sun Merge realized, sanity and integration tests passed correctly after merge. Regards

@Fred-sun Fred-sun removed the medium_priority Medium priority label Mar 15, 2021
@Fred-sun
Copy link
Collaborator

@next-davidduquehernandez Please add this line "plugins/modules/azure_rm_search.py validate-modules:nonexistent-parameter-documented" to "tests/sanity/ignore-2.9.txt". Thanks!

@Fred-sun
Copy link
Collaborator

@next-davidduquehernandez Can you change the "sku:standard3" in the test case to "sku:standard" for convenient test automation? Thank you very much!

Copy link
Collaborator

@Fred-sun Fred-sun left a comment

Choose a reason for hiding this comment

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

Please

plugins/modules/azure_rm_search_info.py Outdated Show resolved Hide resolved
plugins/modules/azure_rm_search.py Outdated Show resolved Hide resolved
@next-davidduquehernandez
Copy link
Contributor Author

@next-davidduquehernandez Can you change the "sku:standard3" in the test case to "sku:standard" for convenient test automation? Thank you very much!

Hi @Fred-sun, accourding to documentation, hosting_mode "highDensity" only accepts sku "standard3". You decide what to do.

https://docs.microsoft.com/en-us/rest/api/searchmanagement/services/createorupdate

Applicable only for the standard3 SKU. You can set this property to enable up to 3 high density partitions that allow up to 1000 indexes, which is much higher than the maximum indexes allowed for any other SKU. For the standard3 SKU, the value is either 'default' or 'highDensity'. For all other SKUs, this value must be 'default'.

Regards!

@Fred-sun
Copy link
Collaborator

@next-davidduquehernandez Yes, I want you to change it to "sku: standard; hosting_mode: default". Thank you!

@Fred-sun Fred-sun added ready_for_review The PR has been modified and can be reviewed and merged and removed work in In trying to solve, or in working with contributors labels Mar 17, 2021
@Fred-sun
Copy link
Collaborator

@next-davidduquehernandez All change is OK, Please make last change, modify module name "azure_rm_search*" to "azure_rm_cognitivesearch", Thank you very much!

@next-davidduquehernandez
Copy link
Contributor Author

Hi @Fred-sun, Modified test in order to change sku to 'standard' and hosting_mode to 'default' and rename module from 'azure_rm_search*' to 'azure_rm_cognitivesearch*'.

Regards

@haiyuazhang haiyuazhang changed the title Add new module to deploy Azure Cognitive Search 'azure_rm_search' Add new module to deploy Azure Cognitive Search 'azure_rm_cognitivesearch' Mar 22, 2021
@Fred-sun
Copy link
Collaborator

@next-davidduquehernandez Please help to resolve the conflict file! I will push for merge! Thank you very much!

@next-davidduquehernandez
Copy link
Contributor Author

Hi @Fred-sun,

I think that update from @haiyuazhang resolves the conflict.

Regards!

@Fred-sun
Copy link
Collaborator

@next-davidduquehernandez Thanks for you update! I will push for merged!

@Ompragash
Copy link
Member

@Fred-sun Could you kindly take a look at it?

@Fred-sun
Copy link
Collaborator

Fred-sun commented Apr 6, 2021

@Fred-sun Could you kindly take a look at it?

@Ompragash This PR has been changed and is waiting to be merged. Thank you very much!

@haiyuazhang haiyuazhang merged commit 1087e6c into ansible-collections:dev Apr 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new_module_pr Add new modules ready_for_review The PR has been modified and can be reviewed and merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants