Provide compatibility for dns_zone module with openstack.cloud collection #105
Conversation
![otc-zuul[bot]](https://avatars.githubusercontent.com/in/11628?s=60&v=4){kind=small}
| description: | ||
| - Cache duration (in second) on a local DNS server | ||
| type: int | ||
| zone_type: |
There was a problem hiding this comment.
let's use "alias" for keeping it backward compatible (https://opendev.org/openstack/ansible-collections-openstack/src/branch/master/plugins/modules/server.py#L498)
There was a problem hiding this comment.
'type' means whether primary or secondary dns zone (line 56), did you mean alias 'type'?
There was a problem hiding this comment.
no. Its actually a recordset that is in openstack has "recordset_type", and on our side "type". For that we should on our side rename "type" attribute into "recordset_type" and with aliases: ['type'] provide backward compatibility.
Later: Well, we actually need to drop recordset module completely and only provide routing to the openstack one
There was a problem hiding this comment.
let's rename it not to type, but "visibility"
| description: | ||
| - Cache duration (in second) on a local DNS server | ||
| type: int | ||
| zone_type: |
There was a problem hiding this comment.
no. Its actually a recordset that is in openstack has "recordset_type", and on our side "type". For that we should on our side rename "type" attribute into "recordset_type" and with aliases: ['type'] provide backward compatibility.
Later: Well, we actually need to drop recordset module completely and only provide routing to the openstack one
| - dns_fl.ptr.description is defined | ||
|
|
||
| - name: Creating a public DNS Zone - check mode | ||
| opentelekomcloud.cloud.dns_zones: |
There was a problem hiding this comment.
hmm, how was this working before? Weird
| - This parameter is disabled, it only provides compatibility with openstack.cloud colection. | ||
| choices: [primary, secondary] | ||
| type: str | ||
| masters: |
There was a problem hiding this comment.
you add "masters", but do not use it
gtema
dismissed stale reviews from tischrei, anton-sidelnikov, SebastianGode, and otc-zuul[bot]
via
070f58d
| description: | ||
| - Cache duration (in second) on a local DNS server | ||
| type: int | ||
| zone_type: |
There was a problem hiding this comment.
let's rename it not to type, but "visibility"
| choices: [public, private] | ||
| default: public | ||
| type: |
There was a problem hiding this comment.
must be zone_type (as currently in openstack module)
| description: Cache duration (in second) on a local DNS server | ||
| type: int | ||
| sample: 300 | ||
| zone_type: |
There was a problem hiding this comment.
I think when we return result we need to explicitly rename zone_type to visibility
| opentelekomcloud.cloud.dns_zone: | ||
| name: "test.com." | ||
| state: present | ||
| zone_type: private |
There was a problem hiding this comment.
zone_type: primary
visibility: private
| ttl=dict(required=False, type='int'), | ||
| zone_type=dict(type='str', choices=['public', 'private'], default='public') | ||
| masters=dict(required=False, type='list', elements='str'), | ||
| type=dict(required=False, choices=['primary', 'secondary'], type='str') |
There was a problem hiding this comment.
this is "visibilty"
and zone_type parameter as such is still described here, hust not used further in the code
| attrs = {} | ||
| query = { | ||
| 'type': self.params['zone_type'], | ||
| 'type': self.params['type'], |
There was a problem hiding this comment.
'zone_type': self.params['visibilty'],
| if not needs_update: | ||
| # Check if VPC exists | ||
| if self.params['zone_type'] == 'private': | ||
| if self.params['type'] == 'private': |
| if self.params['zone_type']: | ||
| attrs['zone_type'] = self.params['zone_type'] | ||
|
|
||
| if self.params['type']: |
There was a problem hiding this comment.
if self.params['visibility']:
attrs['zone_type'] = self.params['visibility']
5 participants
As a user of OTC switching between openstack module and otc module in the current form requires changes in the parameters (zone_type=private is not supported by openstack vs zone_type=public not supported by OTC).
As such we should have following: