| *** ade_lee has quit IRC | 04:26 | |
| *** ykarel has joined #puppet-openstack | 04:40 | |
| *** cgfbee has joined #puppet-openstack | 05:47 | |
| *** gfidente has joined #puppet-openstack | 06:36 | |
| *** chkumar|off is now known as chandankumar | 06:41 | |
| chandankumar | tobias-urdin: https://review.openstack.org/609936 needs +2 and +w on this | 06:43 |
|---|---|---|
| *** dpawlik has joined #puppet-openstack | 07:12 | |
| *** jaosorior has joined #puppet-openstack | 07:13 | |
| *** dpawlik has quit IRC | 07:16 | |
| *** jaosorior has quit IRC | 07:24 | |
| *** jaosorior has joined #puppet-openstack | 07:27 | |
| *** iurygregory has joined #puppet-openstack | 07:29 | |
| iurygregory | good morning | 07:29 |
| chandankumar | iurygregory: Good morning | 07:31 |
| chandankumar | iurygregory: needs +2 and +w on this https://review.openstack.org/#/c/609936/ | 07:31 |
| chandankumar | dependent patches are merged now | 07:32 |
| iurygregory | chandankumar, let me take a look =) | 07:32 |
| iurygregory | done | 07:33 |
| chandankumar | iurygregory: Thanks :-) | 07:40 |
| iurygregory | yw | 07:41 |
| *** dpawlik has joined #puppet-openstack | 07:42 | |
| jaosorior | could I get some reviews for this backport https://review.openstack.org/#/c/614083/ ? | 07:43 |
| iurygregory | sure | 07:43 |
| jaosorior | thanks iurygregory! | 07:43 |
| iurygregory | yw | 07:47 |
| *** dpawlik has quit IRC | 07:47 | |
| *** agurenko has joined #puppet-openstack | 08:02 | |
| *** jtomasek has joined #puppet-openstack | 08:13 | |
| tobias-urdin | chandankumar: this exclude didn't seem to work on debian based https://review.openstack.org/#/c/615202/ | 08:16 |
| tobias-urdin | still saw some failures after that was merged | 08:16 |
| *** dpawlik has joined #puppet-openstack | 08:17 | |
| chandankumar | tobias-urdin: will push a patch up for them | 08:21 |
| tobias-urdin | ty | 08:22 |
| jaosorior | tobias-urdin: if you get some time, could you check this out https://review.openstack.org/#/c/614083/ ? | 08:31 |
| tobias-urdin | already done :) | 08:34 |
| *** jtomasek has quit IRC | 08:36 | |
| jaosorior | ohoo :D | 08:36 |
| jaosorior | tobias-urdin: thanks! | 08:36 |
| *** jtomasek has joined #puppet-openstack | 08:44 | |
| *** jpena|off is now known as jpena | 08:56 | |
| chandankumar | tobias-urdin: Hello | 09:07 |
| chandankumar | tobias-urdin: do we have link to failed job when debian regex is not working? | 09:07 |
| tobias-urdin | example https://review.openstack.org/#/c/604182/ | 09:11 |
| tobias-urdin | puppet-openstack-integration-5-scenario003-tempest-ubuntu-bionic | 09:11 |
| chandankumar | tobias-urdin: got it and let me push a patch | 09:12 |
| *** hjensas has joined #puppet-openstack | 09:16 | |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration master: Add trailing slash to fernet key_repository https://review.openstack.org/615513 | 09:20 |
| openstackgerrit | Merged openstack/puppet-glance master: Add puppet-glance-tripleo-standalone job https://review.openstack.org/609936 | 09:20 |
| openstackgerrit | Chandan Kumar proposed openstack/puppet-openstack-integration master: [tempest][scenario003] Fixed black_regex for debian https://review.openstack.org/615514 | 09:25 |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration master: [DNM] Testing https://review.openstack.org/#/c/615513/ with latest keystone https://review.openstack.org/614988 | 09:30 |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration master: Add trailing slash to fernet key_repository https://review.openstack.org/615513 | 09:36 |
| *** ykarel is now known as ykarel|lunch | 09:41 | |
| chandankumar | tobias-urdin: https://review.openstack.org/615514 this might fix the issue | 09:43 |
| chandankumar | for debian | 09:43 |
| tobias-urdin | ack | 09:43 |
| *** derekh has joined #puppet-openstack | 09:49 | |
| *** trown has quit IRC | 09:55 | |
| *** salmankhan has joined #puppet-openstack | 09:55 | |
| *** salmankhan1 has joined #puppet-openstack | 09:56 | |
| *** trown has joined #puppet-openstack | 09:58 | |
| *** salmankhan has quit IRC | 10:00 | |
| *** salmankhan1 is now known as salmankhan | 10:00 | |
| openstackgerrit | Chandan Kumar proposed openstack/puppet-mistral master: Add puppet-mistral-tripleo-standalone job https://review.openstack.org/615520 | 10:04 |
| *** ykarel|lunch is now known as ykarel | 10:05 | |
| *** chem has joined #puppet-openstack | 10:10 | |
| openstackgerrit | Merged openstack/puppet-nova stable/rocky: Split off metadata vendordata plugin in own class https://review.openstack.org/614083 | 10:11 |
| *** shardy has joined #puppet-openstack | 10:21 | |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration master: [DNM] Testing https://review.openstack.org/#/c/615513/ with latest keystone https://review.openstack.org/614988 | 10:29 |
| *** dtantsur|afk is now known as dtantsur\ | 10:35 | |
| *** dtantsur\ is now known as dtantsur | 10:35 | |
| openstackgerrit | Chandan Kumar proposed openstack/puppet-openstack-integration master: Use blacklist_file for skipping tests in debian https://review.openstack.org/615514 | 10:59 |
| chandankumar | tobias-urdin: I am switching to blacklist file instead of constructing long regex and --black-regex is missing in debian package | 11:00 |
| *** iurygregory is now known as iury|lunch | 11:01 | |
| openstackgerrit | Chandan Kumar proposed openstack/puppet-openstack-integration master: Use blacklist_file for skipping tests in debian https://review.openstack.org/615514 | 11:04 |
| ykarel | mwhahaha, tobias-urdin, with re. to horizon skip in scenario003, should also need to think about telemetry test in scenario001 as it fails intermittently like: http://logs.openstack.org/13/615513/2/check/puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous/b47e6e2/logs/testr_results.html.gz | 11:17 |
| ykarel | it maily fails at Teardown | 11:17 |
| ykarel | Details: {u'message': u'Unable to complete operation on subnet 02d9d6a4-5227-45f9-8e75-a9cdec1032ee: One or more ports have an IP allocation from this subnet.', u'type': u'SubnetInUse', u'detail': u''} | 11:17 |
| ykarel | long ago iirc i was able to reproduce it and i found that heat stack was not deleted when this issue occurred | 11:18 |
| ykarel | so some cleanup is missing | 11:19 |
| ykarel | may be chandankumar have some idea here as it's seems related to gabbi and tempest | 11:19 |
| chandankumar | ykarel: it is still the same issue we were hitting earlier in tripleo ci | 11:25 |
| chandankumar | ykarel: in Tripleo CI ,we skipped that test and we never looked on it | 11:25 |
| ykarel | chandankumar, can u lookup at the cleaning aspect i am referring ^^ | 11:26 |
| ykarel | i think u will find the cause there and fix too | 11:26 |
| ykarel | chandankumar, is https://bugs.launchpad.net/tripleo/+bug/1792862 the bug | 11:33 |
| openstack | Launchpad bug 1792862 in tripleo "[master] Telemetry Tempest integration tests failed giving Unable to complete operation on subnet b13c76ec-c851-48d0-91a9-245a2fdcad9b: One or more ports have an IP allocation from this subnet." [Critical,Fix committed] - Assigned to Mehdi Abaakouk (sileht) | 11:33 |
| ykarel | as per this https://review.openstack.org/#/c/603329/ also needed | 11:33 |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration master: ceilometer: Use new archive policies https://review.openstack.org/603329 | 11:34 |
| ykarel | chandankumar, tobias-urdin i rebased it | 11:34 |
| chandankumar | ykarel: ack | 11:36 |
| *** dpawlik has quit IRC | 11:38 | |
| *** iury|lunch is now known as iurygregory | 11:40 | |
| zigo | We're trying to define a new custom puppet type: barbican_type | 11:41 |
| zigo | But when we're doing it, we're getting: | 11:41 |
| zigo | Error: /Stage[main]/Oci::Controller/Barbican_secret[swift-barbican-secret]: Could not evaluate: wrong number of arguments (given 1, expected 0) | 11:41 |
| zigo | Does anyone know where fix this? | 11:41 |
| zigo | barbican_secret, not barbican_type, sorry. | 11:41 |
| *** beekneemech has quit IRC | 11:53 | |
| *** bnemec has joined #puppet-openstack | 11:57 | |
| tobias-urdin | zigo: what i know there is no barbican_secret resource | 12:08 |
| tobias-urdin | zigo: now that i have you here, do you have any plans to finish the debian testing? we really need it to pass or move it to experimental since it fails all the time, we shouldn't waste the resources for testing | 12:09 |
| tobias-urdin | i know we have announced support, and we really want to keep it, but to keep it we must have testing working | 12:09 |
| *** dpawlik has joined #puppet-openstack | 12:16 | |
| *** dpawlik has quit IRC | 12:18 | |
| *** dpawlik has joined #puppet-openstack | 12:18 | |
| *** jpena is now known as jpena|lunch | 12:20 | |
| *** dpawlik has quit IRC | 12:23 | |
| *** dpawlik has joined #puppet-openstack | 12:26 | |
| *** dpawlik has quit IRC | 12:27 | |
| *** dpawlik has joined #puppet-openstack | 12:28 | |
| *** dpawlik_ has joined #puppet-openstack | 12:30 | |
| *** dpawlik has quit IRC | 12:31 | |
| *** dpawlik_ has quit IRC | 12:32 | |
| *** dpawlik has joined #puppet-openstack | 12:32 | |
| *** dpawlik_ has joined #puppet-openstack | 12:35 | |
| *** dpawlik has quit IRC | 12:37 | |
| *** dpawlik_ has quit IRC | 12:42 | |
| *** dpawlik has joined #puppet-openstack | 12:42 | |
| zigo | tobias-urdin: Yeah, I'm writting one ! :) | 12:46 |
| zigo | tobias-urdin: I will work on it, ok. | 12:46 |
| zigo | tobias-urdin: The thing is, it's hard for me to know what's going on, because it times out. | 12:46 |
| zigo | We also need to switch it to rocky. | 12:47 |
| openstackgerrit | Derek Higgins proposed openstack/puppet-neutron stable/rocky: Allow the networking_ansible plugin to set manage_vlans https://review.openstack.org/615553 | 12:48 |
| *** dtantsur is now known as dtantsur|brb | 12:51 | |
| tobias-urdin | ok | 12:57 |
| chandankumar | tobias-urdin: can we ask someone to update openstack-tempest package in debian? | 12:57 |
| chandankumar | it is very old | 12:58 |
| tobias-urdin | on ubuntu you mean? | 12:58 |
| chandankumar | tobias-urdin: yes | 12:58 |
| tobias-urdin | i've asked the canonical guys about that and they won't be maintaining it since they dont use it from packaging | 12:58 |
| zigo | chandankumar: I can update it in Debian, I think it's up-to-date, no? | 12:58 |
| tobias-urdin | i forgot to write that down, but we need to install tempest from pypi | 12:58 |
| chandankumar | tobias-urdin: let me tweak my changes then | 12:59 |
| zigo | Debian Sid got 1:19.0.0-2 | 12:59 |
| tobias-urdin | chandankumar: thanks! | 12:59 |
| zigo | That's the latest for Rocky, no? | 12:59 |
| tobias-urdin | zigo: it's ubuntu, debian is prob okay :) | 12:59 |
| zigo | Indeed, Ubuntu is lagging 2 versions behind. | 13:00 |
| zigo | You should ping jamespage or coreycb. | 13:00 |
| chandankumar | zigo: but the tempest run output have some missing params | 13:00 |
| tobias-urdin | 13:08 < tobias-urdin> jamespage: any plans on packaging the tempest plugins that has been moved out of project repo trees? | 13:00 |
| tobias-urdin | 13:08 < jamespage> tobias-urdin: for all of our tempest testing we make use of venvs and install from source | 13:00 |
| tobias-urdin | we need to install from source or pypi | 13:01 |
| tobias-urdin | that was some months ago i asked them | 13:01 |
| chandankumar | tobias-urdin: let me update the same patch then | 13:01 |
| zigo | Not for Debian please. | 13:01 |
| zigo | I very much prefer if the Debian package is in use there. | 13:01 |
| chandankumar | zigo: tobias-urdin https://review.openstack.org/#/c/615514/ please have a look at this patch | 13:02 |
| tobias-urdin | chandankumar: s/blaclist/blacklist/g | 13:02 |
| zigo | chandankumar: Why excluding tempest_horizon.tests.scenario.test_dashboard_basic_ops ? | 13:03 |
| tobias-urdin | mwhahaha: came across some stuff this weekend about puppet 4 was being eol (only security updates) in oct 2018 https://groups.google.com/forum/#!topic/puppet-users/78ZcVVyuIMQ | 13:03 |
| tobias-urdin | should we carry the puppet 4 unit for stein and remove support for puppet 4 in T release? | 13:04 |
| openstackgerrit | Chandan Kumar proposed openstack/puppet-openstack-integration master: Use blacklist_file for skipping tests in debian https://review.openstack.org/615514 | 13:05 |
| chandankumar | zigo: it was failing in centos so we blacklisted last friday i think at both place | 13:05 |
| chandankumar | zigo: removed in new review | 13:05 |
| *** agurenko has quit IRC | 13:16 | |
| *** agurenko has joined #puppet-openstack | 13:18 | |
| zigo | tobias-urdin: Could we increase the run's timeout to 2h instead of 1h30, so that I have a chance to catch what the trouble is? | 13:21 |
| tobias-urdin | i dont know where that is done, but if it's in our in-repo zuul config just push a do-not-merge patch to test it | 13:23 |
| *** jpena|lunch is now known as jpena | 13:25 | |
| *** jtomasek has quit IRC | 13:27 | |
| *** jtomasek has joined #puppet-openstack | 13:28 | |
| *** jistr is now known as jistr|call | 13:32 | |
| mwhahaha | tobias-urdin: yeah it's probably time to depecate 4. Tripleo will be moving to 5 in late Stein | 13:37 |
| tobias-urdin | mwhahaha: how about adding a notice in some common file for stein and officially not support it in T release | 13:38 |
| tobias-urdin | and we'll just keep the puppet 4 unit in for stein | 13:38 |
| tobias-urdin | i tried updating puppet 4 unit from 4.8 to 4.10 but shit just timed out, don't feel like it's something we should care about, what u think? | 13:39 |
| *** hjensas has quit IRC | 13:40 | |
| mwhahaha | tobias-urdin: leave it at 4.8. I think for the 3 depreciation we did a ml post and a release note in like openstacklib | 13:41 |
| gfidente | tobias-urdin this guy is green now https://review.openstack.org/#/c/613213/ thanks | 13:42 |
| tobias-urdin | gfidente: done | 13:42 |
| tobias-urdin | mwhahaha: cool, i'll look into it and do the same and update some docs and post to ML | 13:43 |
| zigo | tobias-urdin: mwhahaha: Now, with my new barbican_secret provider, I get: openstack secret store: error: unrecognized arguments: --nameswift-barbican-secret--payload-content-type="application/octet-stream"--payload-content-encoding="base64"--algorithmaes--bit-length 256--modectr--secret-typesymmetric--payloadYV9iaWdfc2VjcmV0Cg=='. Retrying for 167 more seconds | 13:49 |
| zigo | What's the bit that is adding correct spacing ? | 13:49 |
| zigo | Probably I should make it a patch so you can see better? | 13:50 |
| zigo | (it's currently only in my PoC) | 13:50 |
| openstackgerrit | Lee Yarwood proposed openstack/puppet-placement master: WIP Add missing params https://review.openstack.org/615563 | 13:52 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-barbican master: Add a barbican_secret provider https://review.openstack.org/615564 | 13:52 |
| zigo | mwhahaha: tobias-urdin: Any comment on this patch? What's the way so that puppet correctly passes the params? | 13:54 |
| zigo | (ie: with spaces, etc.) | 13:54 |
| tobias-urdin | how you mean? there seems to be a lot of hardcoded stuff in there that would need to be moved out to parameters so others can use it | 14:00 |
| tobias-urdin | some unit testing would be good as well | 14:00 |
| openstackgerrit | Lee Yarwood proposed openstack/puppet-nova stable/ocata: WIP Remove Placement service from puppet-nova https://review.openstack.org/615567 | 14:01 |
| openstackgerrit | Lee Yarwood proposed openstack/puppet-openstack-integration master: WIP Extract the Placement service from Nova https://review.openstack.org/615568 | 14:02 |
| zigo | tobias-urdin: Sure, I know that ! :) | 14:04 |
| zigo | tobias-urdin: But somehow, I'm trying to make it work first... :P | 14:04 |
| zigo | Then I can tweak, improve, etc. | 14:04 |
| zigo | tobias-urdin: Right now, all parameters are concatenated without adding space between them, that's the problem. | 14:04 |
| openstackgerrit | Lee Yarwood proposed openstack/puppet-nova master: WIP Remove Placement service from puppet-nova https://review.openstack.org/615569 | 14:08 |
| *** jistr|call is now known as jistr | 14:08 | |
| zigo | I think I need to write def self.request(service, action, properties=nil) in barbican.rb ... | 14:12 |
| *** morazi has joined #puppet-openstack | 14:12 | |
| *** dtantsur|brb is now known as dtantsur | 14:19 | |
| *** hjensas has joined #puppet-openstack | 14:27 | |
| *** ade_lee has joined #puppet-openstack | 14:48 | |
| *** jistr is now known as jistr|call | 15:04 | |
| *** jistr|call is now known as jistr | 15:10 | |
| openstackgerrit | Douglas Mendizábal proposed openstack/puppet-barbican master: Add HSM related parameters https://review.openstack.org/608339 | 15:19 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-octavia master: Certificate changes should restart services https://review.openstack.org/615586 | 15:40 |
| chandankumar | tobias-urdin: for ubuntu I think I need to switch to tempest git and debian I donot know what is broken | 15:56 |
| chandankumar | zigo: http://logs.openstack.org/14/615514/4/check/puppet-openstack-integration-4-scenario003-tempest-debian-stable/7b43269/job-output.txt.gz | 15:57 |
| chandankumar | zigo: please have a look on debian | 15:57 |
| tobias-urdin | chandankumar: yea | 15:59 |
| zigo | chandankumar: It sometimes fails if we're updating the repository as it runs... | 16:02 |
| zigo | chandankumar: Can you do a recheck? | 16:02 |
| zigo | (I wont rebuild anything there...) | 16:02 |
| *** dpawlik has quit IRC | 16:05 | |
| *** dpawlik_ has joined #puppet-openstack | 16:05 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration master: Revert "Checkout stable/pike for spec_helper" https://review.openstack.org/615451 | 16:05 |
| *** iurygregory is now known as iurygregory|home | 16:08 | |
| openstackgerrit | Chandan Kumar proposed openstack/puppet-openstack-integration master: Use blacklist_file for skipping tests in debian https://review.openstack.org/615514 | 16:08 |
| chandankumar | zigo: tobias-urdin ^^ please have a look at ubuntu changes | 16:09 |
| zigo | chandankumar: lgtm ! | 16:11 |
| *** dpawlik_ has quit IRC | 16:12 | |
| chandankumar | let's see what CI says :-) | 16:12 |
| *** dpawlik has joined #puppet-openstack | 16:12 | |
| *** dpawlik has quit IRC | 16:17 | |
| *** dpawlik has joined #puppet-openstack | 16:28 | |
| *** agurenko has quit IRC | 16:28 | |
| *** dpawlik has quit IRC | 16:32 | |
| *** openstackgerrit has quit IRC | 16:48 | |
| *** trown is now known as trown|lunch | 17:00 | |
| *** ykarel has quit IRC | 17:23 | |
| *** jpena is now known as jpena|off | 17:48 | |
| *** derekh has quit IRC | 18:00 | |
| *** salmankhan has quit IRC | 18:02 | |
| *** trown|lunch is now known as trown | 18:36 | |
| *** openstackgerrit has joined #puppet-openstack | 18:50 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstacklib master: Deprecate Puppet 4 https://review.openstack.org/615627 | 18:50 |
| openstackgerrit | Rajini Karthik proposed openstack/puppet-nova stable/queens: Support for libvirt volume multipath https://review.openstack.org/614625 | 18:54 |
| *** morazi has quit IRC | 19:08 | |
| *** dtantsur is now known as dtantsur|afk | 19:24 | |
| *** shardy has quit IRC | 19:52 | |
| *** shardy has joined #puppet-openstack | 19:52 | |
| *** morazi has joined #puppet-openstack | 20:08 | |
| *** jtomasek has quit IRC | 20:23 | |
| *** jtomasek has joined #puppet-openstack | 20:24 | |
| *** jtomasek has quit IRC | 20:24 | |
| *** salmankhan has joined #puppet-openstack | 20:28 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-cinder master: Convert spec testing to use rspec-puppet-facts https://review.openstack.org/615648 | 20:38 |
| *** gfidente is now known as gfidente|afk | 20:52 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstacklib master: Deprecate Puppet 4 https://review.openstack.org/615627 | 21:46 |
| *** gfidente|afk has quit IRC | 22:08 | |
| *** trown is now known as trown|outtypewww | 22:08 | |
| *** salmankhan has quit IRC | 23:18 | |
| *** chem has quit IRC | 23:43 | |
| openstackgerrit | Merged openstack/puppet-cinder stable/queens: Set rbd_cluster_name to cope with custom cluster names https://review.openstack.org/613213 | 23:49 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!