| *** xarses_ has joined #puppet-openstack | 02:36 | |
| *** ykarel|away has joined #puppet-openstack | 04:24 | |
| *** ykarel|away is now known as ykarel | 04:47 | |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration master: [DNM] Testing https://review.openstack.org/#/c/614536/ with latest cinder https://review.openstack.org/614988 | 05:19 |
|---|---|---|
| *** ykarel is now known as ykarel|afk | 06:21 | |
| *** dpawlik has joined #puppet-openstack | 06:26 | |
| *** dpawlik has quit IRC | 06:26 | |
| *** dpawlik has joined #puppet-openstack | 06:27 | |
| *** ykarel|afk has quit IRC | 06:31 | |
| *** ykarel|afk has joined #puppet-openstack | 06:31 | |
| *** salmankhan has joined #puppet-openstack | 07:25 | |
| *** salmankhan has quit IRC | 07:26 | |
| *** salmankhan1 has joined #puppet-openstack | 07:26 | |
| *** salmankhan1 is now known as salmankhan | 07:28 | |
| *** salmankhan has quit IRC | 07:40 | |
| *** ykarel|afk is now known as ykarel | 08:11 | |
| *** salmankhan has joined #puppet-openstack | 08:29 | |
| *** salmankhan has quit IRC | 08:30 | |
| *** salmankhan has joined #puppet-openstack | 08:30 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: Use puppet4 functions-api https://review.openstack.org/614241 | 08:32 |
| *** salmankhan has quit IRC | 08:34 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-neutron master: Use puppet4 functions-api https://review.openstack.org/614246 | 08:52 |
| *** salmankhan has joined #puppet-openstack | 09:12 | |
| *** salmankhan has quit IRC | 09:19 | |
| *** ykarel_ has joined #puppet-openstack | 09:20 | |
| *** ykarel has quit IRC | 09:23 | |
| *** derekh has joined #puppet-openstack | 09:25 | |
| *** dpawlik has quit IRC | 09:28 | |
| *** ykarel_ is now known as ykarel | 09:31 | |
| *** ykarel_ has joined #puppet-openstack | 09:34 | |
| *** ykarel_ has quit IRC | 09:35 | |
| *** ykarel has quit IRC | 09:36 | |
| *** salmankhan has joined #puppet-openstack | 09:40 | |
| *** salmankhan has quit IRC | 09:46 | |
| *** dtantsur|afk is now known as dtantsur | 09:48 | |
| *** ykarel has joined #puppet-openstack | 09:55 | |
| *** salmankhan has joined #puppet-openstack | 10:19 | |
| *** EmilienM is now known as EvilienM | 10:25 | |
| tobias-urdin | EvilienM: can you review and ack https://review.openstack.org/#/c/614577/ that unblocks CI for nova | 10:26 |
| EvilienM | tobias-urdin: of course | 10:26 |
| tobias-urdin | ty | 10:29 |
| *** owalsh_ is now known as owalsh | 10:30 | |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration master: [DNM] Testing https://review.openstack.org/#/c/614536/ with latest cinder https://review.openstack.org/614988 | 10:34 |
| *** iurygregory is now known as iury|lunch | 10:59 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: Use puppet4 functions-api https://review.openstack.org/614241 | 11:00 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-neutron master: Use puppet4 functions-api https://review.openstack.org/614246 | 11:04 |
| openstackgerrit | Merged openstack/puppet-openstack_spec_helper stable/ocata: Remove duplicate puppet declaration https://review.openstack.org/614882 | 11:42 |
| *** morazi has joined #puppet-openstack | 12:02 | |
| *** trown|outtypewww is now known as trown | 12:25 | |
| chandankumar | EvilienM: tobias-urdin please have a look at this failure http://logs.openstack.org/36/609936/9/check/puppet-openstack-integration-5-scenario001-tempest-ubuntu-bionic-mimic/0cc54c7/logs/testr_results.html.gz | 12:28 |
| chandankumar | it is a known issue? | 12:28 |
| openstackgerrit | Chandan Kumar proposed openstack/puppet-glance master: Add puppet-glance-tripleo-standalone job https://review.openstack.org/609936 | 12:30 |
| tobias-urdin | chandankumar: yes, probably a race condition, you can recheck that for now | 12:31 |
| tobias-urdin | see here https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/nova.pp#L73 | 12:32 |
| tobias-urdin | we can probably increase that sleep unless nova-compute doesn't startup and register itself in those 10 seconds on first startup | 12:33 |
| chandankumar | tobias-urdin: I will push a new patch | 12:33 |
| chandankumar | thanks for the insight | 12:34 |
| tobias-urdin | you can push a patch that increases that to 30 seconds, that should hopefully resolve it | 12:34 |
| *** iury|lunch is now known as iurygregory | 12:36 | |
| openstackgerrit | Chandan Kumar proposed openstack/puppet-openstack-integration master: Increase the sleep to 30 sec for discover_hosts in Nova https://review.openstack.org/615178 | 12:39 |
| chandankumar | tobias-urdin: EvilienM mwhahaha iurygregory ^^ | 12:39 |
| iurygregory | chandankumar, ack | 12:39 |
| tobias-urdin | will approve right away | 12:39 |
| openstackgerrit | Chandan Kumar proposed openstack/puppet-glance master: Add puppet-glance-tripleo-standalone job https://review.openstack.org/609936 | 12:39 |
| iurygregory | tobias-urdin, my +2 was slow lol | 12:40 |
| tobias-urdin | :) | 12:40 |
| chandankumar | thanks guys :-) | 12:40 |
| tobias-urdin | iurygregory: could you review https://review.openstack.org/#/c/614234/ let me know if you'd like us to deprecate os_any2array or remove it, was only intended to compatibility in horizon module and not functionality exposed to end users, but somebody might use it | 12:42 |
| tobias-urdin | EvilienM: mwhahaha ^ perhaps it might be used in tripleo somewhere | 12:42 |
| chandankumar | tobias-urdin: standalone job has coverage of horizon, | 12:43 |
| iurygregory | tobias-urdin, ack will look | 12:43 |
| iurygregory | if ooo use we prob will deprecate | 12:43 |
| iurygregory | but lets wait for EvilienM and mwhahaha =) | 12:43 |
| tobias-urdin | chandankumar: cool, so standalone job passed so it's probably clear then | 12:43 |
| chandankumar | but we can confirm with mwhahaha and EvilienM Once | 12:44 |
| tobias-urdin | ack | 12:44 |
| iurygregory | at least http://codesearch.openstack.org/?q=os_any2array&i=nope&files=&repos= only show puppet-horizon \o/ | 12:44 |
| openstackgerrit | Merged openstack/puppet-nova master: Convert some spec testing to rspec-puppet-facts https://review.openstack.org/614577 | 12:52 |
| *** ykarel_ has joined #puppet-openstack | 13:10 | |
| *** ykarel has quit IRC | 13:10 | |
| *** hjensas has quit IRC | 13:29 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstacklib master: Use puppet4 functions-api https://review.openstack.org/614231 | 13:37 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-neutron master: Use puppet4 functions-api https://review.openstack.org/614246 | 13:44 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: Deprecate nova-network code https://review.openstack.org/613522 | 13:47 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: Deprecate neutron options and add alternatives https://review.openstack.org/613529 | 13:47 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: nova::quota fix correct conf group and rename params https://review.openstack.org/614183 | 13:48 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-nova stable/rocky: Split off metadata vendordata plugin in own class https://review.openstack.org/614083 | 13:50 |
| *** ykarel_ is now known as ykarel | 13:53 | |
| *** xarses_ has quit IRC | 13:54 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-integration master: Update pupppet 4 unit to use 4.10 https://review.openstack.org/613439 | 13:59 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: Add cells v1 deprecation notice and remove broken code https://review.openstack.org/612187 | 13:59 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack_extras master: Convert spec testing to use rspec-puppet-facts https://review.openstack.org/614782 | 14:02 |
| tobias-urdin | iurygregory: https://review.openstack.org/#/c/614782/2 last part to unblock CI, see my comment on the last PS, please approve if ok | 14:03 |
| tobias-urdin | last piece to unblock unit testing that got broken in CI | 14:04 |
| iurygregory | tobias-urdin, sure | 14:04 |
| openstackgerrit | Merged openstack/puppet-openstack_spec_helper master: Bump version to 14.0.0 https://review.openstack.org/613435 | 14:08 |
| chandankumar | mwhahaha: iurygregory can we disable this http://logs.openstack.org/36/609936/11/check/puppet-openstack-integration-5-scenario003-tempest-centos-7/5d9750f/logs/testr_results.html.gz horizon test? | 14:33 |
| mwhahaha | yes plz | 14:35 |
| * mwhahaha tries to find the spot | 14:36 | |
| chandankumar | mwhahaha: sending the patch | 14:36 |
| mwhahaha | oh yea the excludes in run_tests.sh in p-o-i | 14:38 |
| ykarel | tobias-urdin, good to recheck and ur votes, looks like u arleady have fixes in place https://review.openstack.org/#/q/topic:enable-py3+status:open? | 14:43 |
| ykarel | mwhahaha, ^^ | 14:44 |
| ykarel | Thanks | 14:45 |
| openstackgerrit | Chandan Kumar proposed openstack/puppet-openstack-integration master: Blacklist tempest_horizon.tests test in Scenario003 CentOS CI https://review.openstack.org/615202 | 14:47 |
| chandankumar | mwhahaha: EvilienM tobias-urdin iurygregory ^^ | 14:48 |
| ykarel | mwhahaha, tobias-urdin please also review https://review.openstack.org/#/c/614536/ this is required to get puppet ci promotion | 14:48 |
| iurygregory | chandankumar, will look in a few i have many mtgs today XD | 14:50 |
| chandankumar | iurygregory: np, see ya | 14:51 |
| iurygregory | 2 mtgs in a row starting in 10 mtg XD (1hr each) | 14:51 |
| *** iurygregory is now known as iurygregory|mtg | 14:55 | |
| *** chandankumar is now known as chkumar|off | 15:14 | |
| openstackgerrit | Merged openstack/puppet-openstack_extras master: Convert spec testing to use rspec-puppet-facts https://review.openstack.org/614782 | 15:21 |
| openstackgerrit | Chandan Kumar proposed openstack/puppet-openstack-integration master: Enable debug logging for horizon tests https://review.openstack.org/607918 | 15:25 |
| chkumar|off | mwhahaha: iurygregory|mtg ^^ we also need this patch to find more info related to horizon test failure | 15:25 |
| chkumar|off | rdopiera is looking into that | 15:26 |
| ykarel | chkumar|off, good to check with rdopiera first, iirc he said something is still missing ^^ | 15:26 |
| ykarel | with the review | 15:26 |
| chkumar|off | ykarel: based on discussion I have padded this patch | 15:27 |
| chkumar|off | *passed | 15:27 |
| ykarel | chkumar|off, okk u discussed with rdopiera? | 15:28 |
| ykarel | i was asking re. debug patch | 15:28 |
| chkumar|off | ykarel: yes check #rhos-dev internally | 15:28 |
| ykarel | okk | 15:28 |
| ykarel | okk so he is not sure, that's was the same i rememeber from last discussion | 15:29 |
| ykarel | so good to check if Debug logs are available before merging | 15:29 |
| ykarel | and try to reproduce as well | 15:29 |
| ykarel | chkumar|off, looks something wrong with EXCLUDES, --black-regex=^(?!neutron_dynamic_routing.tests.tempest.scenario.*)(?!tempest_horizon.tests.scenario.test_dashboard_basic_ops.*$ | 15:37 |
| ykarel | it looks like part for regex, | 15:38 |
| ykarel | black-regex should include !? with it's name it looks like just need to pass the tests to skip | 15:38 |
| ykarel | will check results from job once available, but just by looking seems wrong | 15:39 |
| ykarel | i commented on patch just to confirm | 15:43 |
| *** jaosorior has quit IRC | 15:58 | |
| *** bnemec is now known as beekneemech | 16:00 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstacklib master: Use puppet4 functions-api https://review.openstack.org/614231 | 16:01 |
| openstackgerrit | Chandan Kumar proposed openstack/puppet-openstack-integration master: Blacklist tempest_horizon.tests test in Scenario003 CI https://review.openstack.org/615202 | 16:13 |
| chkumar|off | ykarel: tobias-urdin ^^ done | 16:13 |
| openstackgerrit | Nagasai Vinaykumar Kapalavai proposed openstack/puppet-qdr master: Add a broker connector configuration https://review.openstack.org/613992 | 16:16 |
| tobias-urdin | chkumar|off: awesome ty | 16:21 |
| tobias-urdin | mwhahaha: any idea why the File stubs are not doing anything? been at it for a while now https://review.openstack.org/#/c/614246/8/spec/functions/convert_cert_to_string_spec.rb | 16:27 |
| ykarel | chkumar|off, ack | 16:27 |
| mwhahaha | tobias-urdin: not really i've found that the new version seems to not like the ruby stubs anymore. i ran into similar issues when we were mocking it out via ruby and had to write a puppet function instead | 16:28 |
| * mwhahaha goes to find the patch for context | 16:28 | |
| mwhahaha | https://review.openstack.org/#/c/613447/1/spec/classes/nova_migration_libvirt_spec.rb | 16:28 |
| mwhahaha | that thing where i had to like replace the override with a regular function | 16:29 |
| mwhahaha | tobias-urdin: are you sure it isn't before (:each) ? | 16:31 |
| mwhahaha | or is that before each context | 16:31 |
| * mwhahaha can never remember | 16:31 | |
| tobias-urdin | dont know, i haven't tried it atleast, can't get the stupid rspec testing to run locally either would be easier to test more :| | 16:32 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-designate master: Install python3-keystoneclient in Fedora or RedHat > 7 https://review.openstack.org/615235 | 16:33 |
| *** iurygregory|mtg has quit IRC | 16:36 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-neutron master: Use puppet4 functions-api https://review.openstack.org/614246 | 16:46 |
| *** ykarel is now known as ykarel|away | 16:48 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: Deprecate neutron options and add alternatives https://review.openstack.org/613529 | 17:10 |
| *** ykarel_ has joined #puppet-openstack | 17:10 | |
| *** ykarel|away has quit IRC | 17:13 | |
| *** dtantsur is now known as dtantsur|afk | 17:19 | |
| *** trown is now known as trown|lunch | 17:21 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: nova::quota fix correct conf group and rename params https://review.openstack.org/614183 | 17:27 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-placement master: Initial cookiecutter and import from nova::placement https://review.openstack.org/604182 | 17:42 |
| tobias-urdin | EvilienM: the two final patches for the rabbit_* removal https://review.openstack.org/#/q/topic:removed-oslo-rabbitmq-options+is:open | 17:44 |
| tobias-urdin | ykarel_: sorry I might have checked this with you earlier, but in this patch I suggest removal of puppet5 nightly build testing | 17:47 |
| tobias-urdin | and fedora uses that https://review.openstack.org/#/c/612201/3/run_tests.sh | 17:47 |
| tobias-urdin | is that something we want to do or is it something we can/should change? | 17:47 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-ceph master: WIP: Ceph providers https://review.openstack.org/579369 | 17:52 |
| EvilienM | tobias-urdin: nice work | 17:59 |
| *** derekh has quit IRC | 18:02 | |
| openstackgerrit | Radomir Dopieralski proposed openstack/puppet-openstack-integration master: Enable debug logging for horizon tests https://review.openstack.org/607918 | 18:04 |
| ykarel_ | tobias-urdin, checkig | 18:08 |
| *** salmankhan has quit IRC | 18:09 | |
| -openstackstatus- NOTICE: OpenStack infra's mirror nodes stopped accepting connections on ports 8080, 8081, and 8082. We will notify when this is fixed and jobs can be rechecked if they failed to communicate with a mirror on these ports. | 18:10 | |
| ykarel_ | tobias-urdin, at first instance removal of PUPPET_PKG export not looking good, we are using it to override package name | 18:11 |
| openstackgerrit | Merged openstack/puppet-openstack-guide master: Update CI jobs information in guide https://review.openstack.org/613440 | 18:11 |
| ykarel_ | and that is_fedora is true for both centos and fedora | 18:11 |
| openstackgerrit | Carlos Goncalves proposed openstack/puppet-octavia master: Set common configuration options across services https://review.openstack.org/612395 | 18:25 |
| *** trown|lunch is now known as trown | 18:33 | |
| openstackgerrit | Carlos Goncalves proposed openstack/puppet-octavia master: Set common configuration options across services https://review.openstack.org/612395 | 18:34 |
| *** ykarel_ is now known as ykarel | 18:37 | |
| tobias-urdin | ykarel: hm where is puppet_pkg overriden? | 18:45 |
| tobias-urdin | EvilienM: thanks | 18:45 |
| ykarel | tobias-urdin, we override in RDO testing of puppet modules from rpm | 18:45 |
| ykarel | tobias-urdin, https://github.com/rdo-infra/ansible-role-weirdo-puppet-openstack/blob/master/tasks/main.yml#L108 | 18:46 |
| tobias-urdin | ack, i'll change that back, everything else looked ok? | 18:47 |
| ykarel | yes rest looks fine | 18:52 |
| ykarel | the is_fedora is true for both centos/fedora so seeing other jobs passing it looks fine | 18:52 |
| tobias-urdin | ack, i'll push fix when infra reports mirrors are ok again | 18:53 |
| *** EvilienM is now known as EmilienM | 18:53 | |
| ykarel | but we soon to update puppet on our side as we have puppet 5 | 18:53 |
| ykarel | okk | 18:53 |
| ykarel | so the failure i just seen in one of the job is infra | 18:53 |
| ykarel | damn i recheced it | 18:54 |
| mwhahaha | they are good again | 18:54 |
| ykarel | okk then fine | 18:54 |
| tobias-urdin | yay | 18:54 |
| ykarel | i rechecked after checking the mirror url was working | 18:54 |
| ykarel | so thought it was a network glich | 18:54 |
| tobias-urdin | EmilienM: haha just got an enlightenment, your friday name is so suiting during halloween times | 18:55 |
| ykarel | but good to know it was a real issue and now fixed | 18:55 |
| *** ykarel is now known as ykarel|away | 18:55 | |
| ykarel|away | happy weekend everyone | 18:55 |
| -openstackstatus- NOTICE: The firewall situation with ports 8080, 8081, and 8082 on mirror nodes has been resolved. You can recheck jobs that have failed to communicate to the mirrors on those ports now. | 18:55 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-integration master: Add puppet6 testing https://review.openstack.org/612201 | 18:55 |
| *** ykarel|away has quit IRC | 19:01 | |
| openstackgerrit | Merged openstack/puppet-octavia master: Do not set inexistent haproxy_amphora/key_path https://review.openstack.org/611875 | 19:20 |
| *** salmankhan has joined #puppet-openstack | 19:24 | |
| *** salmankhan has quit IRC | 19:28 | |
| *** jistr has quit IRC | 19:35 | |
| *** jistr has joined #puppet-openstack | 19:37 | |
| *** hjensas has joined #puppet-openstack | 19:41 | |
| *** morazi has quit IRC | 20:07 | |
| *** salmankhan has joined #puppet-openstack | 20:20 | |
| *** salmankhan has quit IRC | 20:20 | |
| *** salmankhan has joined #puppet-openstack | 20:21 | |
| *** salmankhan has quit IRC | 20:47 | |
| *** salmankhan has joined #puppet-openstack | 21:11 | |
| *** salmankhan has quit IRC | 21:16 | |
| openstackgerrit | Merged openstack/puppet-horizon master: Remove os_any2array function https://review.openstack.org/614234 | 21:40 |
| openstackgerrit | Merged openstack/puppet-cinder master: Deprecate several iscsi_XXX parameters https://review.openstack.org/614536 | 22:02 |
| openstackgerrit | Merged openstack/puppet-swift master: Remove deprecated rabbit opts in swift::proxy::ceilometer https://review.openstack.org/612189 | 22:17 |
| *** salmankhan has joined #puppet-openstack | 22:20 | |
| *** salmankhan has quit IRC | 22:29 | |
| *** salmankhan has joined #puppet-openstack | 22:40 | |
| *** salmankhan has quit IRC | 22:42 | |
| openstackgerrit | Merged openstack/puppet-neutron master: Allow the networking_ansible plugin to set manage_vlans https://review.openstack.org/611297 | 23:08 |
| *** salmankhan has joined #puppet-openstack | 23:09 | |
| openstackgerrit | Carlos Goncalves proposed openstack/puppet-octavia master: Set common configuration options across services https://review.openstack.org/612395 | 23:11 |
| *** owalsh has quit IRC | 23:16 | |
| *** owalsh has joined #puppet-openstack | 23:17 | |
| *** shardy_ has quit IRC | 23:18 | |
| *** shardy has quit IRC | 23:18 | |
| *** jtomasek has quit IRC | 23:31 | |
| *** salmankhan has quit IRC | 23:35 | |
| *** markvoelker has joined #puppet-openstack | 23:41 | |
| *** markvoelker has quit IRC | 23:46 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstacklib master: Use puppet4 functions-api https://review.openstack.org/614231 | 23:49 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!