| opendevreview | Takashi Kajinami proposed openstack/puppet-manila master: Add support for cephfs_cached_allocated_capacity_update_interval https://review.opendev.org/c/openstack/puppet-manila/+/942706 | 02:11 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: hpe3par: Fix wrong option name https://review.opendev.org/c/openstack/puppet-cinder/+/942756 | 05:22 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: hpe3par: Replace redundant hard-coded default values https://review.opendev.org/c/openstack/puppet-cinder/+/942757 | 05:23 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: hpe3par: Remove own hard-coded defaults https://review.opendev.org/c/openstack/puppet-cinder/+/942758 | 05:30 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: hpe3par: Fix wrong option name https://review.opendev.org/c/openstack/puppet-cinder/+/942756 | 07:55 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: hpe3par: Replace redundant hard-coded default values https://review.opendev.org/c/openstack/puppet-cinder/+/942757 | 07:55 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: hpe3par: Remove own hard-coded defaults https://review.opendev.org/c/openstack/puppet-cinder/+/942758 | 07:55 |
| opendevreview | Merged openstack/puppet-manila master: Add support for cephfs_cached_allocated_capacity_update_interval https://review.opendev.org/c/openstack/puppet-manila/+/942706 | 08:35 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: hpe3par: Support an array value for hpe3par_iscsi_ips https://review.opendev.org/c/openstack/puppet-cinder/+/942768 | 09:53 |
| d1nuc0m | Hi, a quick question, when using puppet-designate with bind9, designate fails because it does not find /etc/rndc.conf, am I supposed to manage it manually? | 10:22 |
| tkajinam | d1nuc0m, yeah you have to create the file. I guess it can even be an empty file | 11:15 |
| tkajinam | d1nuc0m, I made usage of rndc.conf optional by https://review.opendev.org/c/openstack/puppet-designate/+/941029 but that may not be backported given its impact | 11:15 |
| tkajinam | we used to create the file externally but removed it by https://review.opendev.org/c/openstack/puppet-openstack-integration/+/941057 | 11:16 |
| d1nuc0m | tkajinam, thank you, I had it in production as I installed it a long time ago, while now reinstalling the testbed it was missing, fixed with a file resource https://paste.openstack.org/show/b2RdOGmpRF1bAkiddgkj/ | 11:20 |
| d1nuc0m | It can't be empty AFAIK as it needs to reference the secret key | 11:21 |
| tkajinam | d1nuc0m, if you have rndc_key_file in pool.yaml then you don't need it, IIUC | 11:21 |
| tkajinam | but you can have it in rndc.conf without adding it to pool.yaml and pool data in designate db | 11:22 |
| d1nuc0m | d1nuc0m, thanks, didn't have it | 11:22 |
| d1nuc0m | *tk | 11:22 |
| d1nuc0m | * tkajinam | 11:22 |
| tkajinam | :-) | 11:22 |
| d1nuc0m | Noob question, can I use resource providers inside lib/puppet or are these supposed to be for internal use only? Like https://github.com/openstack/puppet-nova/blob/master/lib/puppet/provider/nova_flavor/openstack.rb | 11:56 |
| tobias-urdin | d1nuc0m: you can use them (in your example the nova_flavor resource) | 12:40 |
| d1nuc0m | tobias-urdin, thanks, should these also be documented? In case I'll open a quick PR | 12:42 |
| tobias-urdin | i will assume you mean in the readme file, probably yes - the documentation part is probably lacking in most modules, feel free to improve it :) | 12:44 |
| d1nuc0m | In the header of the file? Dunno, where should the proper place for documentation be | 12:46 |
| tobias-urdin | im not actually sure | 12:54 |
| d1nuc0m | Does the project use/plan to use Puppet strings? | 12:55 |
| tobias-urdin | we haven't talked about or had a plan about that to my knowledge | 12:56 |
| tkajinam | if there is a good way to document the interfaces within the code then we can evaluate it, but I'm a bit hesitant to maintain independent doc, given the fact that these can be quite easily outdated | 12:56 |
| tkajinam | examples files we have kept in each repo are a good "example" of such pain | 12:57 |
| d1nuc0m | I was thinking of Puppet strings because of this - it means having the parameters/resources documented in the head of the Puppet file they're declared in (so no separate files) and allows to generate a reference. Disadvantages - one more thing to set up | 13:29 |
| d1nuc0m | Well probably you know it better than me XD | 13:29 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: Deprecate support for bare SAN driver https://review.opendev.org/c/openstack/puppet-cinder/+/942827 | 16:17 |
| *** amoralej_ is now known as amoralej | 16:54 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!