| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: CI test https://review.openstack.org/380880 | 00:04 |
|---|---|---|
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Enable TestEncryptedCinderVolumes tests (for real) https://review.openstack.org/380374 | 00:06 |
| *** rstarmer has quit IRC | 00:27 | |
| openstackgerrit | Merged openstack/puppet-ceilometer: Move rspec-puppet-facts to spec helper https://review.openstack.org/380508 | 00:37 |
| openstackgerrit | Merged openstack/puppet-zaqar: Move rspec-puppet-facts to spec helper https://review.openstack.org/380549 | 00:37 |
| openstackgerrit | Merged openstack/puppet-barbican: Move rspec-puppet-facts to spec helper https://review.openstack.org/380507 | 00:37 |
| openstackgerrit | Merged openstack/puppet-sahara: Move rspec-puppet-facts to spec helper https://review.openstack.org/380537 | 00:37 |
| openstackgerrit | Merged openstack/puppet-mistral: Move rspec-puppet-facts to spec helper https://review.openstack.org/380525 | 00:37 |
| openstackgerrit | Merged openstack/puppet-openstack_extras: Move rspec-puppet-facts to spec helper https://review.openstack.org/380530 | 00:37 |
| openstackgerrit | Merged openstack/puppet-gnocchi: Move rspec-puppet-facts to spec helper https://review.openstack.org/380516 | 00:37 |
| openstackgerrit | Merged openstack/puppet-oslo: Move rspec-puppet-facts to spec helper https://review.openstack.org/380532 | 00:37 |
| openstackgerrit | Merged openstack/puppet-swift: Move rspec-puppet-facts to spec helper https://review.openstack.org/380538 | 00:37 |
| openstackgerrit | Merged openstack/puppet-neutron: Move rspec-puppet-facts to spec helper https://review.openstack.org/380527 | 00:37 |
| openstackgerrit | Merged openstack/puppet-glance: Move rspec-puppet-facts to spec helper https://review.openstack.org/380514 | 00:38 |
| openstackgerrit | Merged openstack/puppet-ironic: Move rspec-puppet-facts to spec helper https://review.openstack.org/380520 | 00:38 |
| *** ducttape_ has joined #puppet-openstack | 00:46 | |
| *** johnmilton has joined #puppet-openstack | 01:08 | |
| *** AnarchyAo has joined #puppet-openstack | 01:22 | |
| colonwq | EmilienM, dmsimard patch 377158 (cinder backup to rbd) is successful with ipv4 by fails with ipv6. I am manually repeating the settings and test | 01:28 |
| EmilienM | colonwq: looking | 01:29 |
| EmilienM | colonwq: mhh ok | 01:30 |
| colonwq | EmilienM, I can resubmit the patch with ipv6=false if you want to see it in CI. | 01:30 |
| EmilienM | colonwq: it would be great to test cinder backup with rbd backend of scenario001 and swift backend on scenario002 | 01:30 |
| EmilienM | colonwq: no | 01:30 |
| EmilienM | colonwq: I dropped a comment also | 01:32 |
| colonwq | EmilienM, Ok | 01:32 |
| colonwq | EmilienM, I'll make the suggested change. Thanks. | 01:35 |
| openstackgerrit | Keith Schincke proposed openstack/puppet-openstack-integration: Add cinder::backup test to scenario0001 https://review.openstack.org/377158 | 01:42 |
| colonwq | EmilienM, Can I -2 my own patch until it is ship shape? | 01:44 |
| *** ilbot3 has quit IRC | 01:45 | |
| *** ilbot3 has joined #puppet-openstack | 01:48 | |
| *** bfg9000 has quit IRC | 01:49 | |
| *** johnmilton has quit IRC | 02:12 | |
| openstackgerrit | Matt Fischer proposed openstack/puppet-nova: Allow to set allow_resize_to_same_host in nova-api https://review.openstack.org/373648 | 02:12 |
| EmilienM | colonwq: I have -2 on it, don't worry it won't merge ;-) | 02:24 |
| *** johnmilton has joined #puppet-openstack | 02:34 | |
| openstackgerrit | Merged openstack/puppet-nova: Move rspec-puppet-facts to spec helper https://review.openstack.org/380528 | 03:11 |
| openstackgerrit | Merged openstack/puppet-cinder: Move rspec-puppet-facts to spec helper https://review.openstack.org/380510 | 03:40 |
| openstackgerrit | Merged openstack/puppet-keystone: Move rspec-puppet-facts to spec helper https://review.openstack.org/380522 | 03:40 |
| openstackgerrit | Merged openstack/puppet-horizon: Move rspec-puppet-facts to spec helper https://review.openstack.org/380519 | 03:40 |
| openstackgerrit | Merged openstack/puppet-designate: Move rspec-puppet-facts to spec helper https://review.openstack.org/380512 | 03:40 |
| openstackgerrit | Merged openstack/puppet-tempest: Move rspec-puppet-facts to spec helper https://review.openstack.org/380540 | 03:40 |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Pin Ubuntu repositories to stable Newton packaging https://review.openstack.org/380874 | 03:40 |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Revert "Disable Telemetry testing on Ubuntu" https://review.openstack.org/380875 | 03:40 |
| openstackgerrit | Merged openstack/puppet-trove: Move rspec-puppet-facts to spec helper https://review.openstack.org/380541 | 03:40 |
| openstackgerrit | Merged openstack/puppet-heat: Move rspec-puppet-facts to spec helper https://review.openstack.org/380517 | 04:52 |
| *** skramaja has joined #puppet-openstack | 05:03 | |
| *** jaosorior has joined #puppet-openstack | 05:04 | |
| *** jaosorior has quit IRC | 05:31 | |
| *** jaosorior has joined #puppet-openstack | 05:32 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for gnocchi https://review.openstack.org/377645 | 05:32 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for ceilometer https://review.openstack.org/377644 | 05:32 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for aodh https://review.openstack.org/365457 | 05:32 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for keystone https://review.openstack.org/327029 | 05:32 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable internal TLS for MySQL https://review.openstack.org/378471 | 05:32 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Fetch internal certificates for HAProxy based on network https://review.openstack.org/366548 | 05:32 |
| *** ccamacho has joined #puppet-openstack | 05:35 | |
| *** richm has quit IRC | 05:40 | |
| *** rcernin has joined #puppet-openstack | 06:04 | |
| *** dgurtner has joined #puppet-openstack | 06:07 | |
| *** leanderthal|afk is now known as leanderthal | 06:19 | |
| openstackgerrit | Sharat Sharma proposed openstack/puppet-designate: Added the options to be configured for designate-central service https://review.openstack.org/379950 | 06:27 |
| openstackgerrit | Merged openstack/puppet-tripleo: Added X-Forwarded-Proto headers for horizon https://review.openstack.org/359519 | 06:30 |
| *** jtomasek has joined #puppet-openstack | 06:44 | |
| *** hjensas has quit IRC | 06:56 | |
| *** tesseract- has joined #puppet-openstack | 07:03 | |
| *** jpena|off is now known as jpena | 07:20 | |
| *** amoralej|off is now known as amoralej | 07:22 | |
| *** dgurtner has quit IRC | 07:28 | |
| *** danpawlik has joined #puppet-openstack | 07:38 | |
| *** zigo has quit IRC | 07:39 | |
| *** zigo has joined #puppet-openstack | 07:42 | |
| *** zigo is now known as Guest24756 | 07:42 | |
| *** jpich has joined #puppet-openstack | 07:43 | |
| openstackgerrit | Merged openstack/puppet-tripleo: Cinder: Add iSCSI protocol parameter https://review.openstack.org/367542 | 07:49 |
| *** hjensas has joined #puppet-openstack | 07:55 | |
| *** shardy has joined #puppet-openstack | 07:58 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL https://review.openstack.org/380967 | 08:02 |
| *** saneax-_-|AFK is now known as saneax | 08:07 | |
| *** jperry has quit IRC | 08:27 | |
| *** paramite has joined #puppet-openstack | 08:28 | |
| *** dgurtner has joined #puppet-openstack | 08:31 | |
| *** dtantsur|afk is now known as dtantsur | 08:37 | |
| *** dgurtner has quit IRC | 08:42 | |
| *** spredzy has quit IRC | 08:47 | |
| *** derekh has joined #puppet-openstack | 08:47 | |
| *** spredzy has joined #puppet-openstack | 08:48 | |
| *** _degorenko|afk is now known as degorenko | 08:52 | |
| *** Guest24756 is now known as zigo | 08:54 | |
| *** spredzy has quit IRC | 08:56 | |
| *** spredzy has joined #puppet-openstack | 08:59 | |
| *** spredzy has quit IRC | 09:20 | |
| *** spredzy has joined #puppet-openstack | 09:22 | |
| *** gfidente has joined #puppet-openstack | 09:23 | |
| *** jaosorior is now known as jaosorior_lunch | 09:25 | |
| *** igajsin has joined #puppet-openstack | 09:27 | |
| openstackgerrit | Sharat Sharma proposed openstack/puppet-designate: Added the options to be configured for designate-central service https://review.openstack.org/379950 | 09:50 |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL https://review.openstack.org/380967 | 09:50 |
| *** mkarpin has left #puppet-openstack | 09:56 | |
| *** jaosorior_lunch is now known as jaosorior | 10:01 | |
| *** richm has joined #puppet-openstack | 10:09 | |
| *** openstackgerrit has quit IRC | 10:48 | |
| *** openstackgerrit has joined #puppet-openstack | 10:49 | |
| *** cdelatte has joined #puppet-openstack | 10:55 | |
| openstackgerrit | Keith Schincke proposed openstack/puppet-openstack-integration: Add cinder::backup test to scenario0001 https://review.openstack.org/377158 | 10:55 |
| *** mkarpin has joined #puppet-openstack | 11:01 | |
| openstackgerrit | Julie Pichon proposed openstack/puppet-tripleo: Clean out UI httpd configuration file https://review.openstack.org/380152 | 11:14 |
| *** psanchez has joined #puppet-openstack | 11:21 | |
| *** lucas-afk is now known as lucasagomes | 11:26 | |
| openstackgerrit | Peter Zhurba proposed openstack/puppet-glare: Add hooks for external install & svc management https://review.openstack.org/380026 | 11:27 |
| EmilienM | hello | 11:37 |
| *** johnmilton has quit IRC | 11:38 | |
| openstackgerrit | Peter Zhurba proposed openstack/puppet-glare: Add posibility install glare from pip https://review.openstack.org/373805 | 11:54 |
| *** dprince has joined #puppet-openstack | 11:56 | |
| *** johnmilton has joined #puppet-openstack | 12:04 | |
| *** johnmilton has quit IRC | 12:04 | |
| *** johnmilton has joined #puppet-openstack | 12:05 | |
| *** trown|outtypewww is now known as trown | 12:07 | |
| *** pradk has joined #puppet-openstack | 12:14 | |
| *** jayg|g0n3 is now known as jayg | 12:30 | |
| openstackgerrit | Keith Schincke proposed openstack/puppet-openstack-integration: Add cinder backup to scenario002 https://review.openstack.org/381083 | 12:34 |
| *** dgurtner has joined #puppet-openstack | 12:36 | |
| dtantsur | hey EmilienM | 12:41 |
| *** moravec has quit IRC | 12:42 | |
| *** jpena is now known as jpena|lunch | 12:43 | |
| iurygregory | good morning | 12:45 |
| *** fultonj has joined #puppet-openstack | 12:55 | |
| *** degorenko is now known as _degorenko|afk | 12:58 | |
| *** dtantsur is now known as dtantsur|brb | 13:07 | |
| *** ducttape_ has quit IRC | 13:12 | |
| *** morazi has joined #puppet-openstack | 13:17 | |
| *** jperry has joined #puppet-openstack | 13:17 | |
| *** jperry has quit IRC | 13:18 | |
| *** dfisher has joined #puppet-openstack | 13:29 | |
| *** jaosorior has quit IRC | 13:31 | |
| *** jaosorior has joined #puppet-openstack | 13:32 | |
| *** imcsk8 has joined #puppet-openstack | 13:34 | |
| *** ducttape_ has joined #puppet-openstack | 13:39 | |
| *** jpena|lunch is now known as jpena | 13:42 | |
| *** moravec has joined #puppet-openstack | 13:43 | |
| *** jperry has joined #puppet-openstack | 13:43 | |
| *** moravec has quit IRC | 14:00 | |
| EmilienM | iurygregory: can you review https://review.openstack.org/#/c/380492/ ? | 14:02 |
| *** moravec has joined #puppet-openstack | 14:02 | |
| iurygregory | EmilienM, ack | 14:02 |
| openstackgerrit | Peter Zhurba proposed openstack/puppet-glare: Add hooks for external install & svc management https://review.openstack.org/380026 | 14:03 |
| openstackgerrit | Peter Zhurba proposed openstack/puppet-glare: Add posibility install glare from pip https://review.openstack.org/373805 | 14:06 |
| openstackgerrit | Merged openstack/puppet-modulesync-configs: Update managed module list https://review.openstack.org/380492 | 14:10 |
| *** saneax is now known as saneax-_-|AFK | 14:12 | |
| *** andreww has quit IRC | 14:13 | |
| *** beekneemech is now known as bnemec | 14:18 | |
| *** dtantsur|brb is now known as dtantsur | 14:20 | |
| *** _degorenko|afk is now known as degorenko | 14:22 | |
| *** moravec has quit IRC | 14:31 | |
| colonwq | EmilienM, I'll add the tempest check to 381083 after I have it working in the local test environment. | 14:34 |
| *** sharatss has joined #puppet-openstack | 14:36 | |
| *** dgurtner has quit IRC | 14:37 | |
| *** andreww has joined #puppet-openstack | 14:37 | |
| sharatss | https://bugs.launchpad.net/puppet-glare/+bug/1629876 | 14:39 |
| openstack | Launchpad bug 1629876 in puppet-zaqar "Change the link to home-page" [Undecided,New] - Assigned to Sharat Sharma (sharat-sharma) | 14:39 |
| sharatss | please check this bug | 14:40 |
| iurygregory | Note: This bug is under discussion. Once a decision is taken, it will be fixed | 14:41 |
| iurygregory | not sure where is the discussion .-. | 14:41 |
| degorenko | sharatss: what's exactly you want to change? Overview section pointing to openstack? | 14:42 |
| sharatss | iurygregory, i just changed the description before pinging here | 14:42 |
| sharatss | degerenko, all the openstack projects point to their respective developer docs in setup.cfg | 14:43 |
| sharatss | in puppet we are still pointing it to openstack.org | 14:43 |
| degorenko | ah | 14:44 |
| degorenko | looks like you can do it, yeah | 14:44 |
| sharatss | yes | 14:44 |
| degorenko | EmilienM: iurygregory mwhahaha ^ ? | 14:44 |
| sharatss | i thought it should point here: http://docs.openstack.org/developer/puppet-openstack-guide | 14:45 |
| iurygregory | i'm fine with the change but i don't see the need because all readme have the link to puppet-openstack-guide | 14:45 |
| mwhahaha | i thought there was some history as to why we pointed to openstack.org but i'm not against updating it | 14:45 |
| *** moravec has joined #puppet-openstack | 14:48 | |
| sharatss | i saw the same changes going to few other components, so i raised this issue | 14:48 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Fix the timeout for pacemaker systemd resources https://review.openstack.org/380665 | 14:56 |
| *** moravec has quit IRC | 14:57 | |
| *** mkarpin has quit IRC | 14:59 | |
| sharatss | degorenko: iurygregory mwhahaha are we going to fix this? | 15:03 |
| degorenko | no objections from my side | 15:03 |
| sharatss | degorenko: thank u | 15:04 |
| iurygregory | i have no objections too, but not sure what are the benefits =) | 15:04 |
| mwhahaha | consistency i guess :D | 15:05 |
| dmsimard | I'm just going to leave this here http://lists.openstack.org/pipermail/openstack-dev/2016-September/104173.html | 15:06 |
| * dmsimard hides | 15:06 | |
| mwhahaha | i don't think we need to fix it necessarily because we're not a python project | 15:06 |
| mwhahaha | so we don't actually use the setup.cfg for anything other than for the reno stuff | 15:06 |
| dmsimard | mwhahaha: what, you mean the puppet modules aren't published to pypi ? | 15:07 |
| dmsimard | pip install puppet-nova | 15:07 |
| * dmsimard moans | 15:07 | |
| mwhahaha | ಠ_ಠ | 15:07 |
| dmsimard | well I mean | 15:08 |
| dmsimard | there's xstatic stuff on pypi | 15:08 |
| dmsimard | so there's a precedent | 15:08 |
| *** dtantsur is now known as dtantsur|mtg | 15:08 | |
| * iurygregory lunch time | 15:12 | |
| *** mkarpin has joined #puppet-openstack | 15:13 | |
| openstackgerrit | Mykyta Karpin proposed openstack/puppet-nova: [WIP] Add class for migration options configuration in qemu.conf https://review.openstack.org/378754 | 15:17 |
| EmilienM | mwhahaha: do you remember why we don't need to include ::stdlib in the manifests to use functions? | 15:19 |
| *** andreww is now known as xarses | 15:19 | |
| *** rcernin has quit IRC | 15:19 | |
| mwhahaha | nope | 15:20 |
| mwhahaha | other than i thought they just worked | 15:20 |
| *** danpawlik has quit IRC | 15:21 | |
| EmilienM | same :) | 15:21 |
| dmsimard | EmilienM: afaik things like functions, facts and providers/types are loaded automatically when the catalog is compiled | 15:22 |
| mwhahaha | maybe back in like the 0.26 days you needed to include stdlib :D | 15:22 |
| dmsimard | just needs to be in /etc/puppet/modules | 15:22 |
| EmilienM | mhh ok | 15:22 |
| mwhahaha | if you wanted to use the stages from stdlib you needed to do an include stdlib | 15:23 |
| mwhahaha | but not the functions | 15:23 |
| mwhahaha | https://forge.puppet.com/puppetlabs/stdlib#usage | 15:23 |
| *** haplo37__ has joined #puppet-openstack | 15:29 | |
| *** saneax-_-|AFK is now known as saneax | 15:31 | |
| openstackgerrit | Michele Baldessari proposed openstack/puppet-tripleo: Change rabbitmq queues HA mode from ha-all to ha-exactly https://review.openstack.org/379586 | 15:31 |
| *** leanderthal is now known as leanderthal|afk | 15:34 | |
| *** AnarchyAo has quit IRC | 15:35 | |
| *** jtomasek_ has joined #puppet-openstack | 15:41 | |
| *** moravec has joined #puppet-openstack | 15:41 | |
| *** dprince has quit IRC | 15:42 | |
| *** mfisch` is now known as mfisch | 15:43 | |
| *** mfisch is now known as Guest7223 | 15:43 | |
| *** moravec1 has joined #puppet-openstack | 15:45 | |
| *** sharatss has quit IRC | 15:46 | |
| *** moravec has quit IRC | 15:48 | |
| *** Guest7223 is now known as mfisch | 15:49 | |
| *** mfisch has quit IRC | 15:49 | |
| *** mfisch has joined #puppet-openstack | 15:49 | |
| *** tesseract- has quit IRC | 15:56 | |
| *** jtomasek_ has quit IRC | 15:58 | |
| *** moravec1 has quit IRC | 15:58 | |
| *** shardy has quit IRC | 16:00 | |
| mfisch | EmilienM: In 10 years you will be Norweigan actor Vegar Hoel: http://images-mds.staticskynet.be/NewsFolder/w-600_h-450_s-1/occupied_thomas_eriksen_20160105041850.jpg | 16:00 |
| *** moravec has joined #puppet-openstack | 16:01 | |
| mfisch | I saw that watching a show this weekend, I think its you | 16:01 |
| EmilienM | lol | 16:01 |
| EmilienM | becayse of my beard? | 16:02 |
| mfisch | I think he just looks similar but the beard helps | 16:02 |
| EmilienM | interesting | 16:03 |
| EmilienM | you know it's hard to say when it's about yourself. | 16:03 |
| *** hjensas has quit IRC | 16:03 | |
| mfisch | maybe he needs a stunt double | 16:03 |
| *** dtantsur|mtg is now known as dtantsur | 16:05 | |
| *** chem has quit IRC | 16:08 | |
| *** dprince has joined #puppet-openstack | 16:17 | |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-ceph: Split installed packages per role https://review.openstack.org/381228 | 16:27 |
| *** jpich has quit IRC | 16:28 | |
| *** moravec has quit IRC | 16:30 | |
| *** trown is now known as trown|lunch | 16:34 | |
| openstackgerrit | Merged openstack/puppet-vswitch: Move rspec-puppet-facts to spec helper https://review.openstack.org/380545 | 16:39 |
| openstackgerrit | Merged openstack/puppet-tripleo: Add swift proxy for ceilometer middleware https://review.openstack.org/371591 | 16:40 |
| openstackgerrit | Merged openstack/puppet-tripleo: telemetry: normalize coordination_url https://review.openstack.org/380547 | 16:40 |
| openstackgerrit | Merged openstack/puppet-neutron: Change worker defaults to ::os_workers https://review.openstack.org/380286 | 16:41 |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-ceph: Split installed packages per role https://review.openstack.org/381228 | 16:44 |
| *** derekh has quit IRC | 16:55 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/puppet-tripleo: Add swift proxy for ceilometer middleware https://review.openstack.org/381250 | 17:06 |
| *** saneax is now known as saneax-_-|AFK | 17:07 | |
| *** saneax-_-|AFK is now known as saneax | 17:07 | |
| *** moravec has joined #puppet-openstack | 17:07 | |
| *** AnarchyAo has joined #puppet-openstack | 17:12 | |
| *** trown|lunch is now known as trown | 17:21 | |
| openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: Replace proxy concat/template with ini provider https://review.openstack.org/378950 | 17:23 |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-ceph: Split installed packages per role https://review.openstack.org/381228 | 17:23 |
| openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: Replace proxy concat/template with ini provider https://review.openstack.org/378950 | 17:27 |
| *** _ducttape_ has joined #puppet-openstack | 17:29 | |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-ceph: Split installed packages per role https://review.openstack.org/381228 | 17:30 |
| *** ducttape_ has quit IRC | 17:32 | |
| *** degorenko is now known as _degorenko|afk | 17:36 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-guide: new-module: document launchpad project https://review.openstack.org/378939 | 17:43 |
| *** jpena is now known as jpena|away | 17:46 | |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-ceph: Split installed packages per role https://review.openstack.org/381228 | 17:49 |
| *** amoralej is now known as amoralej|off | 17:54 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: firewall: do not exclude empty services https://review.openstack.org/381266 | 17:57 |
| *** dtantsur is now known as dtantsur|afk | 18:02 | |
| *** jaosorior has quit IRC | 18:07 | |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-ceph: Split installed packages per role https://review.openstack.org/381228 | 18:11 |
| openstackgerrit | Keith Schincke proposed openstack/puppet-openstack-integration: Add cinder backup to scenario002 https://review.openstack.org/381083 | 18:13 |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-ceph: Split installed packages per role https://review.openstack.org/381228 | 18:15 |
| *** gfidente has quit IRC | 18:19 | |
| *** _ducttape_ has quit IRC | 18:20 | |
| *** ducttape_ has joined #puppet-openstack | 18:21 | |
| openstackgerrit | Merged openstack/puppet-openstack-guide: new-module: document launchpad project https://review.openstack.org/378939 | 18:27 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Fix the timeout for pacemaker systemd resources https://review.openstack.org/380665 | 18:27 |
| *** rcernin has joined #puppet-openstack | 18:31 | |
| *** ducttape_ has quit IRC | 18:38 | |
| *** xarses has quit IRC | 18:39 | |
| *** moravec has quit IRC | 18:39 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Clean out UI httpd configuration file https://review.openstack.org/380152 | 18:46 |
| *** ducttape_ has joined #puppet-openstack | 18:57 | |
| openstackgerrit | Michele Baldessari proposed openstack/puppet-tripleo: Change rabbitmq queues HA mode from ha-all to ha-exactly https://review.openstack.org/379586 | 19:05 |
| *** haplo37__ has quit IRC | 19:06 | |
| *** AnarchyAo has quit IRC | 19:10 | |
| openstackgerrit | Merged openstack/puppet-tripleo: Use FallbackResource instead of Rewrite for UI https://review.openstack.org/378571 | 19:16 |
| *** johnmilton has quit IRC | 19:21 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Use FallbackResource instead of Rewrite for UI https://review.openstack.org/381276 | 19:24 |
| *** paramite has quit IRC | 19:32 | |
| *** johnmilton has joined #puppet-openstack | 19:35 | |
| *** johnmilton has quit IRC | 19:39 | |
| *** johnmilton has joined #puppet-openstack | 19:41 | |
| mfisch | EmilienM: we have an issue in puppet-nova with regions | 19:41 |
| mfisch | clayton found it | 19:41 |
| clayton | it appears that openstacklib's openstack provider doesn't actually have any region support | 19:41 |
| mwhahaha | orly | 19:42 |
| clayton | am I just missing it? | 19:42 |
| iurygregory | o.o | 19:42 |
| mwhahaha | if you set OS_REGION it should take it | 19:42 |
| mwhahaha | https://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack/auth.rb#L45-L50 | 19:42 |
| EmilienM | that's unfortunate but possible | 19:42 |
| mwhahaha | or OS_REGION_NAME or whatever it is | 19:44 |
| clayton | sure, but to do this properly, it needs to be added to the KEYS arrays here, right? https://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack/credentials.rb | 19:44 |
| clayton | and then the nova provider code changed to pass that through? | 19:44 |
| clayton | the nova provider code already has the region, it's just not being passed along | 19:44 |
| mwhahaha | yea that might be the missing case | 19:45 |
| mfisch | might also need domain there | 19:46 |
| clayton | there are separate lists for v2 and v3 | 19:48 |
| mfisch | puppet's been using v3 for a long time I thought | 19:49 |
| mwhahaha | not all of them | 19:51 |
| mwhahaha | nova still has a bunch that use v2 i think | 19:51 |
| *** dgurtner has joined #puppet-openstack | 19:51 | |
| mwhahaha | we've not been consistent in going back and updating the modules when we make changes like that | 19:51 |
| mfisch | we were on an older puppet-nova until this week | 19:53 |
| mfisch | is mitaka final? | 19:53 |
| EmilienM | mitaka? | 19:53 |
| iurygregory | we are in newton | 19:54 |
| EmilienM | we released newton | 19:54 |
| mfisch | yeah I meant that | 19:54 |
| EmilienM | and yes, it's a final release | 19:54 |
| mfisch | its just that he's such a bad quarterback I cant say his name sorry | 19:54 |
| EmilienM | well, we accept backports to stable/newton for bug fixes I guess | 19:54 |
| mfisch | yeah we'll need it once we figure out the rix | 19:54 |
| mfisch | the fix even | 19:55 |
| clayton | EmilienM: it's broken in mitaka also | 19:55 |
| mwhahaha | did it ever work? | 19:55 |
| clayton | not since the change to make nova use openstacklib for auth | 19:55 |
| mwhahaha | cause i don't think we've ever properly tested the region | 19:55 |
| mfisch | I dont think so | 19:55 |
| clayton | openstacklib has never worked properly with regions | 19:55 |
| clayton | the code just isn't there | 19:55 |
| mwhahaha | no i meant did it work for nova at all | 19:56 |
| clayton | yes, it did, we fixed it :) | 19:56 |
| mfisch | I fixed it I think | 19:56 |
| EmilienM | can you file a bug? | 19:56 |
| mwhahaha | https://bugs.launchpad.net/puppet-nova/+bug/1517220 or i guess we could reopen it :D | 19:57 |
| openstack | Launchpad bug 1517220 in puppet-nova "provider code doesn't account for regions" [High,Fix released] - Assigned to Matt Fischer (mfisch) | 19:57 |
| clayton | the fix is fairly straightforward | 19:57 |
| mfisch | and host aggs is the same way we found it today | 19:58 |
| mfisch | West puppet trying to add to East host agg | 19:58 |
| EmilienM | what would we do without your feedback? :) | 19:59 |
| mwhahaha | be blissfully region free | 20:00 |
| openstackgerrit | Clayton O'Neill proposed openstack/puppet-openstacklib: Add region support to credentials https://review.openstack.org/381288 | 20:01 |
| iurygregory | this the change to fix? ^ O.o | 20:02 |
| mwhahaha | at least it's a minor fix in a single spot | 20:02 |
| clayton | that is the openstacklib change, there is a puppet-nova change to go with it | 20:03 |
| clayton | looking at what tests need to be updated | 20:03 |
| *** jpena|away is now known as jpena | 20:05 | |
| clayton | well, I put that bug in the commit, someone can reopen it if they want | 20:06 |
| clayton | this is basically a regression | 20:07 |
| clayton | oh, this is the best part | 20:08 |
| clayton | mfisch reported and fixed this bug in code that was no longer being used | 20:08 |
| clayton | because it was already switched over to using the broken openstacklib code | 20:08 |
| mfisch | but our branch was so old we just pulled my commit in I guess | 20:09 |
| clayton | it worked for us because we cherry-picked it internally on to an older module that didn't use openstacklib yet | 20:09 |
| mfisch | yep | 20:09 |
| mfisch | clayton: should you make the test something other than Region1 becuase thats default? | 20:10 |
| mfisch | or will it still catch the failure | 20:10 |
| clayton | I don't think it matters, it's not actually reading it out of a config file or anything | 20:10 |
| mwhahaha | RegionOne is techincally the default not Region1 but it should be ok | 20:10 |
| mfisch | ah | 20:11 |
| mfisch | yeah you're right thats fine | 20:11 |
| *** moravec has joined #puppet-openstack | 20:11 | |
| clayton | actually, mfisch your fix probably did fix some code. it appears that not all of the nova providers are using openstacklib | 20:11 |
| *** jtomasek is now known as jtomasek|afk | 20:12 | |
| mwhahaha | yea thats why i said i didn't think nova was using the new stuff | 20:12 |
| mwhahaha | nova has never really been properly migrated | 20:12 |
| clayton | the ones here are: https://github.com/openstack/puppet-nova/commit/08ee5866a6c6440f9202eddbb8f7ae4ec24a1fca#diff-6f83046f529e571b086e130bb0a63c87 | 20:13 |
| colonwq | dmsimard, How do I get clone a specific in review version of p-o-i? I want to clone 381083 and use run_tests.sh. | 20:17 |
| *** dprince has quit IRC | 20:18 | |
| dmsimard | colonwq: not sure what you mean, git fetch https://git.openstack.org/openstack/puppet-openstack-integration refs/changes/83/381083/2 && git checkout FETCH_HEAD ? | 20:18 |
| dmsimard | colonwq: or do you mean with weirdo ? | 20:18 |
| mfisch | clayton: ^ | 20:18 |
| mfisch | sorry wrong window | 20:19 |
| *** lucasagomes is now known as lucas-afk | 20:19 | |
| *** jpena is now known as jpena|off | 20:22 | |
| colonwq | dmsimard, maybe. I am poking with your commands. | 20:22 |
| dmsimard | colonwq: click the download button in gerrit at the top right | 20:22 |
| dmsimard | colonwq: it gives you a command to check out the review | 20:22 |
| dmsimard | colonwq: the one I gave you is the first one | 20:22 |
| colonwq | dmsimard, When did that button appear? Thanks. | 20:23 |
| dmsimard | colonwq: it moved there when gerrit was updated a few months ago :/ | 20:23 |
| dmsimard | before that it was in-line, somewhat close to the CI results.. I sort of forget | 20:23 |
| openstackgerrit | Michele Baldessari proposed openstack/puppet-tripleo: Change rabbitmq queues HA mode from ha-all to ha-exactly https://review.openstack.org/379586 | 20:24 |
| openstackgerrit | Clayton O'Neill proposed openstack/puppet-nova: Fix region support for nova providers https://review.openstack.org/381295 | 20:25 |
| mfisch | clayton: formatting is screwed up in the tests for that one ^ | 20:26 |
| clayton | k, I'll fix. | 20:26 |
| clayton | ah, right, I see. it didnt' cherry-pick cleanly, that's probably why | 20:27 |
| colonwq | dmsimard, I got it to work. Thanks for the help. | 20:27 |
| openstackgerrit | Clayton O'Neill proposed openstack/puppet-nova: Fix region support for nova providers https://review.openstack.org/381295 | 20:27 |
| *** jayg is now known as jayg|g0n3 | 20:27 | |
| dmsimard | colonwq: neat | 20:28 |
| iurygregory | clayton, can you send other patch with release notes for 381288 and 381295? or just upload a new patch in each one | 20:42 |
| clayton | sure | 20:44 |
| iurygregory | tks =D | 20:44 |
| openstackgerrit | Clayton O'Neill proposed openstack/puppet-openstacklib: Add region support to credentials https://review.openstack.org/381288 | 20:49 |
| openstackgerrit | Clayton O'Neill proposed openstack/puppet-nova: Fix region support for nova providers https://review.openstack.org/381295 | 20:52 |
| *** dgurtner has quit IRC | 21:03 | |
| *** trown is now known as trown|outtypewww | 21:06 | |
| *** openstackgerrit has quit IRC | 21:19 | |
| *** openstackgerrit has joined #puppet-openstack | 21:19 | |
| *** ducttape_ has quit IRC | 21:49 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-openstack_spec_helper: Remove add_custom_fact for puppet version https://review.openstack.org/381328 | 21:58 |
| *** jperry has quit IRC | 22:02 | |
| *** rcernin has quit IRC | 22:25 | |
| *** dfisher has quit IRC | 23:03 | |
| *** hjensas has joined #puppet-openstack | 23:13 | |
| *** hjensas has joined #puppet-openstack | 23:13 | |
| *** markvoelker has quit IRC | 23:29 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
