| *** alee has quit IRC | 00:02 | |
| *** alee has joined #puppet-openstack | 00:03 | |
| openstackgerrit | Merged openstack/puppet-designate master: Set minimal-responses in BIND backend configuration https://review.openstack.org/578881 | 01:01 |
|---|---|---|
| *** sai_p has quit IRC | 01:02 | |
| *** knifefork has joined #puppet-openstack | 01:07 | |
| *** knifefork has quit IRC | 01:08 | |
| *** knifefork has joined #puppet-openstack | 01:09 | |
| *** ykarel|away has joined #puppet-openstack | 01:56 | |
| *** colonwq has quit IRC | 01:57 | |
| *** ykarel|away has quit IRC | 02:02 | |
| *** knifefork has quit IRC | 02:08 | |
| *** knifefork has joined #puppet-openstack | 02:24 | |
| *** knifefork has quit IRC | 02:24 | |
| *** knifefork has joined #puppet-openstack | 02:25 | |
| *** knifefork has quit IRC | 03:05 | |
| *** jaganathan has joined #puppet-openstack | 03:09 | |
| *** skramaja has joined #puppet-openstack | 03:09 | |
| *** ykarel|away has joined #puppet-openstack | 03:47 | |
| *** ykarel|away is now known as ykare | 03:47 | |
| *** ykare is now known as ykarel | 03:47 | |
| openstackgerrit | Merged openstack/puppet-neutron master: Add support for dynamic routing bpg dragent https://review.openstack.org/578774 | 04:23 |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration master: Remove base fedora repos as fedora-stable is enabled https://review.openstack.org/579884 | 05:38 |
| *** armaan has joined #puppet-openstack | 05:57 | |
| *** skramaja_ has joined #puppet-openstack | 06:00 | |
| *** skramaja has quit IRC | 06:02 | |
| *** ykarel_ has joined #puppet-openstack | 06:07 | |
| *** ykarel has quit IRC | 06:10 | |
| *** ykarel__ has joined #puppet-openstack | 06:32 | |
| *** ykarel_ has quit IRC | 06:35 | |
| *** shardy has joined #puppet-openstack | 06:40 | |
| *** dpawlik has joined #puppet-openstack | 06:43 | |
| *** amoralej|off is now known as amoralej | 06:44 | |
| *** ykarel__ is now known as ykarel | 06:52 | |
| *** rmart04 has joined #puppet-openstack | 06:59 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-neutron stable/queens: Add support for dynamic routing bpg dragent https://review.openstack.org/580056 | 07:07 |
| *** ykarel is now known as ykarel|mtg | 07:11 | |
| *** rcernin_ has quit IRC | 07:13 | |
| *** ykarel_ has joined #puppet-openstack | 07:13 | |
| *** vnogin has joined #puppet-openstack | 07:15 | |
| *** ykarel|mtg has quit IRC | 07:16 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-horizon stable/queens: Fix redirect to https bug and testing https://review.openstack.org/580062 | 07:19 |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration master: Remove base fedora repos as fedora-stable is enabled https://review.openstack.org/579884 | 07:25 |
| *** tesseract has joined #puppet-openstack | 07:28 | |
| *** ykarel__ has joined #puppet-openstack | 07:34 | |
| *** ykarel_ has quit IRC | 07:37 | |
| *** tosky has joined #puppet-openstack | 07:37 | |
| *** jpena|off is now known as jpena | 08:04 | |
| *** derekh has joined #puppet-openstack | 08:14 | |
| *** ykarel__ is now known as ykarel | 08:27 | |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration master: Remove base fedora repos as fedora-stable is enabled https://review.openstack.org/579884 | 08:32 |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration master: [DNM] Testing fedora py3 on RDO CI https://review.openstack.org/579126 | 08:36 |
| *** ispp has joined #puppet-openstack | 08:45 | |
| *** jaganathan has quit IRC | 08:48 | |
| *** armaan has quit IRC | 09:02 | |
| *** jaganathan has joined #puppet-openstack | 09:04 | |
| openstackgerrit | Iago Santos proposed openstack/puppet-keystone master: Service_token_roles_required missing in authtoken config file Edit https://review.openstack.org/579799 | 09:08 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-neutron stable/queens: Add support for dynamic routing bpg dragent https://review.openstack.org/580056 | 09:12 |
| *** alee has quit IRC | 09:32 | |
| *** armaan has joined #puppet-openstack | 09:38 | |
| openstackgerrit | Numan Siddique proposed openstack/puppet-ovn master: Set mac-table-size for provider bridges defined in 'ovn_bridge_mappings' https://review.openstack.org/580114 | 09:51 |
| *** armaan has quit IRC | 09:56 | |
| *** armaan has joined #puppet-openstack | 09:57 | |
| *** armaan has quit IRC | 09:57 | |
| *** armaan has joined #puppet-openstack | 09:59 | |
| *** jaganathan has quit IRC | 10:02 | |
| *** alee has joined #puppet-openstack | 10:13 | |
| *** jaganathan has joined #puppet-openstack | 10:15 | |
| *** knifefork has joined #puppet-openstack | 10:31 | |
| *** armaan has quit IRC | 10:42 | |
| *** armaan has joined #puppet-openstack | 10:43 | |
| *** jaganathan has quit IRC | 10:44 | |
| *** armaan has quit IRC | 10:48 | |
| *** jpena is now known as jpena|lunch | 10:59 | |
| *** dtantsur|afk is now known as dtantsur | 11:10 | |
| *** armaan has joined #puppet-openstack | 11:11 | |
| *** vnogin has quit IRC | 11:34 | |
| *** vnogin has joined #puppet-openstack | 11:35 | |
| *** armaan has quit IRC | 12:03 | |
| *** armaan has joined #puppet-openstack | 12:04 | |
| *** amoralej is now known as amoralej|lunch | 12:06 | |
| *** armaan has quit IRC | 12:13 | |
| *** jpena|lunch is now known as jpena | 12:16 | |
| *** vabada has joined #puppet-openstack | 12:25 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: Add VNC SSL options https://review.openstack.org/580153 | 12:31 |
| *** ispp has quit IRC | 12:32 | |
| tobasco | mwhahaha: is there any policy about what should be placed in module's ::init and what should go into separate classes? Should we try to have as much as possible in ::init or not? For example in nova we have use_syslog etc in nova::init but it's been moved out to nova::logging and then there is this pick https://github.com/openstack/puppet-nova/blob/e211fcfd65f3e1144ca9c29ff7314597e0989abc/manifes | 12:34 |
| tobasco | ts/logging.pp#L122-L130 | 12:34 |
| tobasco | https://github.com/openstack/puppet-nova/blob/e211fcfd65f3e1144ca9c29ff7314597e0989abc/manifests/logging.pp#L122-L130 | 12:34 |
| tobasco | I dont like picks, it becomes kind of messy and IMO should only be used when we should support backwards compatibility. | 12:35 |
| tobasco | zigo: we moved so puppet4 jobs only run on <= Queens branches and master runs puppet5 since there was too many jobs | 12:35 |
| tobasco | IIRC debian was puppet4 only and not available in queens branches | 12:36 |
| tobasco | or we accidentally removed it, we can prob add it back in | 12:36 |
| zigo | tobasco: I don't think there's any issue with it running with Puppet 5. | 12:36 |
| zigo | I don't see why there would be any. | 12:37 |
| zigo | In fact, I'd prefer to also have jobs testing puppet 5 for Debian. | 12:37 |
| zigo | Especially that Buster will be shipped with Pupppet 5. | 12:37 |
| tobasco | let me check | 12:37 |
| tobasco | and add puppet5 jobs for debian if they're not there yet | 12:37 |
| zigo | tobasco: Any idea why it's always doing a timeout, btw? https://review.openstack.org/#/c/577281 | 12:38 |
| zigo | I don't get it ... | 12:38 |
| zigo | I'm trying to check if re-activating FWaaS works with the added upstream patch. | 12:38 |
| tobasco | debian puppet5 jobs is in experimental, so if you give a "check experimental" comment on the change you'll get zuul running those jobs | 12:40 |
| tobasco | should maybe do that once and see how it goes and then we can move them to check pipeline to run on each commit after that | 12:41 |
| tobasco | zigo: hm don't know, probably have to investigate with a disposable vm | 12:42 |
| tobasco | have only seen timeouts before when heat looped for a long time (when trying to get magnum in gate) and i assume there is some timeout value on runs in zuul | 12:43 |
| zigo | Will try on a VM, ok. | 12:43 |
| tobasco | that seems to be an issue with something never finishing or get stuck in the puppet run | 12:44 |
| tobasco | considering its fwaas probably neutron related right | 12:44 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-horizon stable/queens: Fix redirect to https bug and testing https://review.openstack.org/580158 | 12:53 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-neutron stable/queens: Add support for dynamic routing bpg dragent https://review.openstack.org/580161 | 13:01 |
| tobasco | EmilienM: sorry about that :( | 13:01 |
| EmilienM | tobasco: no worries | 13:01 |
| openstackgerrit | Merged openstack/puppet-ovn master: Add release note link in README https://review.openstack.org/578383 | 13:01 |
| *** amoralej|lunch is now known as amoralej | 13:03 | |
| openstackgerrit | Merged openstack/puppet-qdr master: Add release note link in README https://review.openstack.org/578420 | 13:05 |
| tobasco | EmilienM: do you have any feedback on my question to mwhahaha some ~30 mins ago if you scroll up? :) | 13:08 |
| EmilienM | tobasco: if it's a lot of parameters usually it's good to create a separated class | 13:09 |
| EmilienM | we like classes for visibility/readability | 13:09 |
| EmilienM | but there is no strict rule | 13:09 |
| tobasco | so since nova::logging is there would it make sense to deprecate those in ::nova? I haven't checked if the other modules has had those options moved out to <name>::logging but makes sense | 13:16 |
| *** hjensas has quit IRC | 13:22 | |
| *** armaan has joined #puppet-openstack | 13:36 | |
| *** armaan has quit IRC | 13:41 | |
| *** hjensas has joined #puppet-openstack | 13:55 | |
| *** hjensas has quit IRC | 13:55 | |
| *** hjensas has joined #puppet-openstack | 13:55 | |
| *** ykarel is now known as ykarel|away | 13:55 | |
| *** skramaja_ has quit IRC | 13:58 | |
| *** alee has quit IRC | 13:59 | |
| *** alee has joined #puppet-openstack | 13:59 | |
| *** ykarel|away has quit IRC | 14:01 | |
| openstackgerrit | Merged openstack/puppet-openstacklib master: Making immutable config setting when using <_IMMUTABLE_>. https://review.openstack.org/579183 | 14:21 |
| *** ykarel|away has joined #puppet-openstack | 14:31 | |
| *** ykarel|away is now known as ykarel | 14:32 | |
| openstackgerrit | Merged openstack/puppet-murano master: Add release note link in README https://review.openstack.org/578406 | 14:37 |
| *** dpawlik has quit IRC | 14:43 | |
| *** armaan has joined #puppet-openstack | 14:56 | |
| *** rmart04 has quit IRC | 14:58 | |
| *** armaan has quit IRC | 15:23 | |
| *** armaan has joined #puppet-openstack | 15:24 | |
| openstackgerrit | Merged openstack/puppet-ironic master: Expose power sync parameters https://review.openstack.org/579893 | 15:26 |
| openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic stable/queens: Expose power sync parameters https://review.openstack.org/580197 | 15:29 |
| *** armaan has quit IRC | 15:52 | |
| *** armaan has joined #puppet-openstack | 15:52 | |
| *** dgurtner has joined #puppet-openstack | 15:55 | |
| *** dgurtner has quit IRC | 15:55 | |
| *** dgurtner has joined #puppet-openstack | 15:55 | |
| *** armaan has quit IRC | 15:57 | |
| *** shardy has quit IRC | 15:58 | |
| *** alee has quit IRC | 15:59 | |
| *** alee has joined #puppet-openstack | 16:00 | |
| *** ykarel has quit IRC | 16:08 | |
| *** vnogin has quit IRC | 16:11 | |
| *** alee has quit IRC | 16:30 | |
| *** alee has joined #puppet-openstack | 16:30 | |
| *** tesseract has quit IRC | 16:30 | |
| *** ykarel has joined #puppet-openstack | 16:32 | |
| *** ykarel has quit IRC | 16:39 | |
| openstackgerrit | Numan Siddique proposed openstack/puppet-ovn master: Set mac-table-size for provider bridges defined in 'ovn_bridge_mappings' https://review.openstack.org/580114 | 16:50 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: Deprecate nova::logging params in ::nova https://review.openstack.org/580213 | 16:51 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: Deprecate nova::logging params in ::nova https://review.openstack.org/580213 | 16:52 |
| *** armaan has joined #puppet-openstack | 16:54 | |
| *** dtantsur is now known as dtantsur|afk | 16:56 | |
| *** dpawlik has joined #puppet-openstack | 16:59 | |
| *** dpawlik has quit IRC | 17:03 | |
| *** jpena is now known as jpena|off | 17:03 | |
| *** derekh has quit IRC | 17:10 | |
| *** amoralej is now known as amoralej|off | 17:11 | |
| mgagne | Is there a way to relocate the openrc file used by Puppet resources for OpenStack? Puppet currently assumes it's located in /root/openrc | 17:15 |
| mgagne | it also seems to fallback to parsing the keystone_authtoken section in the service config file. This user usually doesn't have admin role and therefore cannot manage resources. | 17:16 |
| mgagne | so I'm wondering if relocating it to ${settings::confdir}/openrc would be a good idea | 17:23 |
| *** armaan has quit IRC | 17:32 | |
| *** armaan has joined #puppet-openstack | 17:32 | |
| *** armaan has quit IRC | 17:36 | |
| *** alee has quit IRC | 17:45 | |
| *** alee has joined #puppet-openstack | 17:46 | |
| mwhahaha | It assumes home/openrc I thought | 18:05 |
| mwhahaha | So non-root should still work | 18:05 |
| mnaser | mwhahaha: you are right | 18:33 |
| mnaser | https://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack/auth.rb#L6 | 18:33 |
| mnaser | mgagne: ^ | 18:33 |
| mgagne | yes, I would like for Puppet to look somewhere else by default or in priority. somewhere a "user" (root or not) cannot interfere with Puppet by messing with its openrc | 18:34 |
| mnaser | can you run puppet with environment vars? | 18:35 |
| mgagne | we had some issue where loading openrc in your terminal and running Puppet results in weird behavior | 18:35 |
| mnaser | yes | 18:36 |
| mnaser | because it uses that | 18:36 |
| mnaser | https://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack/auth.rb#L35-L46 | 18:36 |
| mnaser | it uses credentials in the env. first then falls back to /root/openrc | 18:36 |
| mnaser | err | 18:36 |
| mnaser | $HOME/openrc | 18:36 |
| mgagne | yes, I'm not sure if it's the best thing because as a user, you might source openrc for stuff and then run Puppet in your terminal without knowing it will read your envvars | 18:37 |
| mgagne | so I was wondering if a more "puppet" location would be better, like /etc/puppetlabs/puppet/openrc | 18:38 |
| *** vnogin has joined #puppet-openstack | 18:43 | |
| *** vnogin has quit IRC | 18:48 | |
| *** vnogin has joined #puppet-openstack | 20:58 | |
| *** vnogin has quit IRC | 21:02 | |
| *** alee has quit IRC | 21:02 | |
| *** alee has joined #puppet-openstack | 21:03 | |
| *** vnogin has joined #puppet-openstack | 21:08 | |
| *** alee has quit IRC | 21:17 | |
| *** alee has joined #puppet-openstack | 21:18 | |
| *** alee has quit IRC | 21:29 | |
| *** alee has joined #puppet-openstack | 21:30 | |
| *** alee has quit IRC | 21:34 | |
| *** alee has joined #puppet-openstack | 21:35 | |
| *** alee has quit IRC | 21:36 | |
| *** alee has joined #puppet-openstack | 21:36 | |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-openstack-integration master: WIP: Debian noop test https://review.openstack.org/576416 | 22:21 |
| *** rcernin has joined #puppet-openstack | 22:27 | |
| *** alee has quit IRC | 22:43 | |
| *** vnogin has quit IRC | 23:31 | |
| *** alee has joined #puppet-openstack | 23:38 | |
| openstackgerrit | Numan Siddique proposed openstack/puppet-ovn master: Set mac-table-size for provider bridges defined in 'ovn_bridge_mappings' https://review.openstack.org/580114 | 23:44 |
| *** alee has quit IRC | 23:48 | |
| *** tosky has quit IRC | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!