Friday, 2021-06-25

michapmajohnsom, we had a convo ages ago about user modification of their SOA records. Using the zone import API appears to allow setting your own retry and expire values, but refresh is generated and negative TTL is always the config value.06:42
michapmajohnsom, I don't think either of those are possible using the normal zone create, so would you consider it a) fine, b) bug in zone create or c) bug in zone import?06:57
opendevreviewMichael Chapman proposed openstack/designate master: [WIP] Zone import and export documentation  https://review.opendev.org/c/openstack/designate/+/79805107:17
johnsommichapma I think zone import is an interesting case. A common use could be to import a zone from outside Designate and you may need to configure it the same as it was prior to import.15:08
michapmajohnsom, in that case maybe the zone minimum should be imported as well.15:45
johnsommichapma Yeah, there is a balance there with allowing settings that can degrade the Designate service.15:46
michapmajohnsom, you can set a TTL of 1 and it will accept it, but the refresh will always be within the configured range15:47
johnsommichapma Zone import is non-admin only right? If that is the case we should error on the side of maintaining the operator configured values15:47
michapmajohnsom regular users can use import. I haven't tested to see if there's any difference if done with admin creds15:48
johnsomYeah, since regular users can trigger it. We should probably enforce the rules you would have on zone create.15:48
johnsomIdeally, there would be operator configured values enforced for user actions, with an admin permission level to allow override. But that isn't how zone create is setup today I think.15:50
opendevreviewMichael Chapman proposed openstack/designate master: [WIP] Zone import and export documentation  https://review.opendev.org/c/openstack/designate/+/79805115:52
michapmajohnsom, ack15:52
johnsommichapma I know it's late for you, let me think about this and we can chat about it on Monday. I'm struggling to context switch back to this at the moment, so I need to go refresh my memory of what we have today.15:54
johnsomMaybe others here will also have input/thoughts on it.15:54
opendevreviewAde Lee proposed openstack/designate master: Replace md5 for fips  https://review.opendev.org/c/openstack/designate/+/79815720:15
opendevreviewAde Lee proposed openstack/designate master: DNM/WIP: Add fips jobs  https://review.opendev.org/c/openstack/designate/+/79815820:24
opendevreviewMichael Johnson proposed openstack/designate master: Replace md5 for fips  https://review.opendev.org/c/openstack/designate/+/79815723:06

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!