| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: metal: sync all containers to add is_metal https://review.openstack.org/648292 | 00:00 |
|---|---|---|
| *** luksky has quit IRC | 00:06 | |
| *** DanyC has quit IRC | 00:26 | |
| *** cmart has quit IRC | 00:45 | |
| *** dxiri has quit IRC | 01:01 | |
| *** gyee has quit IRC | 01:16 | |
| *** djhankb has joined #openstack-ansible | 01:26 | |
| *** jhesketh has joined #openstack-ansible | 02:12 | |
| *** DanyC has joined #openstack-ansible | 02:26 | |
| *** raukadah is now known as chandankumar | 02:27 | |
| openstackgerrit | Chandan Kumar (raukadah) proposed openstack/openstack-ansible-os_tempest master: Update role for new source build process https://review.openstack.org/648026 | 02:29 |
| *** DanyC has quit IRC | 02:31 | |
| *** cshen has joined #openstack-ansible | 02:51 | |
| *** cshen has quit IRC | 02:55 | |
| *** kmadac4 has joined #openstack-ansible | 03:01 | |
| *** kmadac3 has quit IRC | 03:04 | |
| openstackgerrit | Chandan Kumar (raukadah) proposed openstack/openstack-ansible-os_tempest master: Update role for new source build process https://review.openstack.org/648026 | 03:54 |
| *** ivve has joined #openstack-ansible | 04:01 | |
| *** udesale has joined #openstack-ansible | 04:05 | |
| *** ivve has quit IRC | 04:24 | |
| *** shyamb has joined #openstack-ansible | 04:44 | |
| *** shyamb has quit IRC | 04:53 | |
| *** shyamb has joined #openstack-ansible | 05:03 | |
| *** markvoelker has joined #openstack-ansible | 05:04 | |
| openstackgerrit | Guilherme Steinmuller Pimentel proposed openstack/openstack-ansible-os_rally master: Update role for new source build process https://review.openstack.org/648236 | 05:42 |
| *** jbadiapa has quit IRC | 06:07 | |
| *** SmearedBeard has joined #openstack-ansible | 06:10 | |
| *** shyamb has quit IRC | 06:27 | |
| *** SmearedBeard has quit IRC | 06:30 | |
| *** shyamb has joined #openstack-ansible | 06:38 | |
| *** phasespace has quit IRC | 07:02 | |
| *** chandankumar has quit IRC | 07:06 | |
| *** chandankumar has joined #openstack-ansible | 07:08 | |
| *** jbadiapa has joined #openstack-ansible | 07:14 | |
| *** DanyC has joined #openstack-ansible | 07:16 | |
| *** DanyC has quit IRC | 07:17 | |
| *** pcaruana has joined #openstack-ansible | 07:20 | |
| *** cshen has joined #openstack-ansible | 07:32 | |
| *** shyamb has quit IRC | 07:36 | |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_panko master: Update role for new source build process https://review.openstack.org/648335 | 07:40 |
| *** shyamb has joined #openstack-ansible | 07:46 | |
| *** phasespace has joined #openstack-ansible | 07:46 | |
| *** kopecmartin|off is now known as kopecmartin | 07:52 | |
| fnpanic | morning | 07:54 |
| *** shyamb has quit IRC | 07:56 | |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_cloudkitty master: Update role for new source build process https://review.openstack.org/648338 | 07:59 |
| noonedeadpunk | mornings | 07:59 |
| *** hamzaachi has joined #openstack-ansible | 08:03 | |
| *** tosky has joined #openstack-ansible | 08:04 | |
| *** rgogunskiy has joined #openstack-ansible | 08:07 | |
| noonedeadpunk | odyssey4me seems like neutron role is not really ready... https://github.com/openstack/openstack-ansible-os_neutron/blob/master/vars/source_install.yml#L56 | 08:08 |
| *** luksky has joined #openstack-ansible | 08:11 | |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_cloudkitty master: Update role for new source build process https://review.openstack.org/648338 | 08:45 |
| *** shyamb has joined #openstack-ansible | 08:47 | |
| *** DanyC has joined #openstack-ansible | 09:08 | |
| *** oanson has quit IRC | 09:10 | |
| *** DanyC has quit IRC | 09:12 | |
| *** DanyC has joined #openstack-ansible | 09:13 | |
| *** SmearedBeard has joined #openstack-ansible | 09:14 | |
| *** luksky has quit IRC | 09:37 | |
| *** shyamb has quit IRC | 09:37 | |
| *** oanson has joined #openstack-ansible | 09:41 | |
| *** shyamb has joined #openstack-ansible | 09:44 | |
| *** aedc has joined #openstack-ansible | 09:54 | |
| *** priteau has joined #openstack-ansible | 09:54 | |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_tacker master: Update role for new source build process https://review.openstack.org/648361 | 09:55 |
| *** shyamb has quit IRC | 10:03 | |
| *** luksky has joined #openstack-ansible | 10:09 | |
| *** shyamb has joined #openstack-ansible | 10:12 | |
| *** rgogunskiy has quit IRC | 10:29 | |
| admin0 | hi .. does anyone have a generic heat templae they use to test their cloud .. like something that creates a server, volume, router, floating ip, etc | 10:31 |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_tacker master: Update role for new source build process https://review.openstack.org/648361 | 10:45 |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_tacker master: Update role for new source build process https://review.openstack.org/648361 | 10:47 |
| openstackgerrit | Chandan Kumar (raukadah) proposed openstack/openstack-ansible-os_tempest master: Reuse tempest_tempest_conf_overrides in tempestconf extra args https://review.openstack.org/648137 | 10:50 |
| *** aedc has quit IRC | 10:53 | |
| *** ansmith_ has quit IRC | 10:55 | |
| *** udesale has quit IRC | 11:01 | |
| CeeMac_ | morning | 11:10 |
| *** CeeMac_ is now known as CeeMac | 11:10 | |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_cloudkitty master: Update role for new source build process https://review.openstack.org/648338 | 11:12 |
| *** savvas has quit IRC | 11:20 | |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_tacker master: Update role for new source build process https://review.openstack.org/648361 | 11:21 |
| *** rgogunskiy has joined #openstack-ansible | 11:27 | |
| *** rgogunskiy has quit IRC | 11:32 | |
| fnpanic | admin0: why not use the openstack_metrics from rallly-openstack? | 11:47 |
| *** cshen has quit IRC | 11:47 | |
| fnpanic | done in no time and you have a nice report :-) | 11:48 |
| fnpanic | https://github.com/openstack/rally-openstack | 11:48 |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_cloudkitty master: Update role for new source build process https://review.openstack.org/648338 | 11:51 |
| *** rgogunskiy has joined #openstack-ansible | 11:56 | |
| *** weshay_pto is now known as weshay | 11:59 | |
| *** cshen has joined #openstack-ansible | 12:02 | |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_zun master: Update role for new source build process https://review.openstack.org/648380 | 12:05 |
| CeeMac | i'm seeing some odd behaviour under the Orchestration | Template Generator area of Horizon. I get an error about not being able to resolve qos policies | 12:08 |
| CeeMac | the qos extension driver isn't listed in the agents, and I'm not aware of any user variables I would have set that would require this | 12:08 |
| CeeMac | is this a known issue? | 12:08 |
| *** markvoelker has quit IRC | 12:20 | |
| fnpanic | qos? cinder oder neutron? | 12:29 |
| fnpanic | CeeMac: ? | 12:29 |
| *** electrofelix has joined #openstack-ansible | 12:30 | |
| CeeMac | hi fnpanic | 12:38 |
| *** shyamb has quit IRC | 12:38 | |
| CeeMac | neutron | 12:38 |
| CeeMac | "Unable to retrieve resources Qos polices: the resource could not be found. Neutron server returns request_ids:" | 12:40 |
| *** ansmith has joined #openstack-ansible | 12:40 | |
| CeeMac | Neutron logs say "No controller found for: qos - returning response code 404: PecanNotFound" | 12:40 |
| fnpanic | do you load the qos? | 12:41 |
| fnpanic | neutron_plugin_base: | 12:42 |
| fnpanic | it is not enabled by default | 12:42 |
| fnpanic | https://docs.openstack.org/openstack-ansible-os_neutron/rocky/ | 12:42 |
| CeeMac | i havent configured anything for qos | 12:42 |
| fnpanic | so this could be the error i guess | 12:43 |
| CeeMac | heat template must just try to arbitrarily pull all extensions? | 12:43 |
| fnpanic | not sure | 12:44 |
| CeeMac | im not even trying to configure a template, just clicking that tab brings up the error | 12:44 |
| fnpanic | where are you clicking? | 12:45 |
| fnpanic | i see no error in the template generator and i enabled the qos | 12:45 |
| CeeMac | log in to horizon | 12:47 |
| CeeMac | expand orchestration then click template generator | 12:47 |
| fnpanic | works | 12:48 |
| CeeMac | error message bubble appears in top right corner | 12:48 |
| CeeMac | you have qos enabled though? | 12:48 |
| fnpanic | yes | 12:48 |
| CeeMac | ok, i'll give that a go | 12:48 |
| CeeMac | just seems odd. if the functinality requires qos, i would have expected qos to be enabled | 12:48 |
| CeeMac | neutron_plugin_base right? | 12:49 |
| *** udesale has joined #openstack-ansible | 12:50 | |
| *** shyamb has joined #openstack-ansible | 12:50 | |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_zun master: Update role for new source build process https://review.openstack.org/648380 | 12:54 |
| *** rgogunskiy has quit IRC | 12:56 | |
| fnpanic | CeeMac: yes | 12:57 |
| CeeMac | fnpanic, thats done it, thanks | 12:59 |
| *** rgogunskiy has joined #openstack-ansible | 13:00 | |
| *** fghaas has joined #openstack-ansible | 13:02 | |
| mnaser | bonjour | 13:09 |
| openstackgerrit | Natal Ngétal proposed openstack/openstack-ansible-plugins master: Correct spelling mistakes https://review.openstack.org/619531 | 13:09 |
| CeeMac | hey | 13:09 |
| mnaser | oooooou | 13:09 |
| mnaser | we broke gate with os_neutron change oops | 13:09 |
| openstackgerrit | Natal Ngétal proposed openstack/openstack-ansible-plugins master: Correct spelling mistakes https://review.openstack.org/619531 | 13:10 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_neutron master: vars: eliminate developer mode vars https://review.openstack.org/648395 | 13:17 |
| *** rgogunskiy has quit IRC | 13:18 | |
| mnaser | cores ^ please review, that should unbreak gates | 13:18 |
| *** jhesketh has quit IRC | 13:18 | |
| *** rgogunskiy has joined #openstack-ansible | 13:21 | |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_cloudkitty master: Updated from OpenStack Ansible Tests https://review.openstack.org/639525 | 13:23 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_cloudkitty master: fix tox python3 overrides https://review.openstack.org/606819 | 13:23 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_cloudkitty master: missing document start "---" https://review.openstack.org/614100 | 13:24 |
| *** rgogunskiy has quit IRC | 13:25 | |
| *** shyamb has quit IRC | 13:26 | |
| *** electrofelix has quit IRC | 13:26 | |
| *** electrofelix has joined #openstack-ansible | 13:26 | |
| guilhermesp | mnaser: should we wait +1 from zuul? | 13:27 |
| guilhermesp | also, does it makes sense https://review.openstack.org/#/c/648236/2? | 13:27 |
| mnaser | guilhermesp: let me look, also, if you +2+W and Zuul fails, it won't merge, so not that big of a deal | 13:28 |
| guilhermesp | all right | 13:28 |
| mnaser | guilhermesp: makes sense to me | 13:29 |
| mnaser | ill wait to see what noonedeadpunk has to comment about it | 13:29 |
| guilhermesp | yeah... we don't rely on git shas to install rally anymore. Let me know if you agree noonedeadpunk | 13:29 |
| jamesdenton | LGTM | 13:29 |
| *** electrofelix has quit IRC | 13:32 | |
| guilhermesp | also https://review.openstack.org/#/c/648231/1 | 13:35 |
| guilhermesp | this is new for me, RETRY_LIMIT | 13:36 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_heat master: Update role for new source build process https://review.openstack.org/648398 | 13:36 |
| guilhermesp | for the functional tests | 13:36 |
| mnaser | yea I saw that for octavia | 13:36 |
| mnaser | guilhermesp: the trick for that is to do a "recheck", wait for it to appear in zuul | 13:36 |
| mnaser | when the job appears, click on it to view the console and wait until it fails | 13:36 |
| guilhermesp | all right | 13:37 |
| jrosser | all of those with RETRY_LIMIT are where we lose the upstream network | 13:37 |
| jrosser | i've already dug at this | 13:37 |
| jrosser | the thing that happens right before that is we reload systemd_networkd, and then boom everything is brok | 13:38 |
| mnaser | jrosser: wonder how come it affects only certain roles | 13:38 |
| mnaser | and not all of them | 13:38 |
| jrosser | becasue there is data in the role tests that drives how the interfaces are set up, i think. | 13:38 |
| jrosser | and my hunch is that for swift/octaivia etc we ask for some extra networking to be made that goes a bit wrong | 13:39 |
| noonedeadpunk | guilhermesp: Yep ,sorry, didn't get the point at the first time | 13:39 |
| guilhermesp | noonedeadpunk np :) | 13:39 |
| jrosser | mnaser: tbh, i wonder if there is more value in pushing the current changes through for those broken roles | 13:40 |
| noonedeadpunk | just comment started from building with repo_build which is not the case, so this distracted me) | 13:40 |
| jrosser | and then fixing up the integrated test, becasue otherwise we will spend a bunch of time on tests that we are ditching | 13:40 |
| *** udesale has quit IRC | 13:41 | |
| noonedeadpunk | mnaser: I'm really very bad in licensing... | 13:41 |
| *** udesale has joined #openstack-ansible | 13:41 | |
| jrosser | guilhermesp: the only way to see what happens with those RETRY_LIMIT jobs is to kick one off then go watch the console at zuul.oopenstack.org | 13:43 |
| guilhermesp | yeah, I'm just keeping my eyes on it | 13:43 |
| *** fghaas has left #openstack-ansible | 13:44 | |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_cloudkitty master: Update role for new source build process https://review.openstack.org/648338 | 13:46 |
| chandankumar | jrosser: guilhermesp https://review.openstack.org/#/c/648137/ please have a look at this when free! | 13:48 |
| guilhermesp | hum jrosser seems to be stuck in TASK [systemd_networkd : Enable and start systemd-networkd] | 13:54 |
| *** SmearedBeard has quit IRC | 13:55 | |
| jrosser | yes, thats the point at which the upstream connection is lost, if you dig in eavesdrop i had a chat with the #infra people about this and the conclusion was we just lose the connection to the instance | 13:55 |
| jrosser | but like i say, there is agood chance that this is all just an aretefact of having moved the role tests to using systemd-networkd a while ago | 13:56 |
| *** SmearedBeard has joined #openstack-ansible | 14:04 | |
| odyssey4me | o/ | 14:06 |
| odyssey4me | mnaser maybe we should bite the bullet and release https://review.openstack.org/647919 to merge to help get this stuff done, then just fix up any roles that were broken once they're using the integrated build for testing? | 14:08 |
| odyssey4me | mnaser thanks for https://review.openstack.org/648395 - apologies for breaking everything :( | 14:14 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_nova master: Update role for new source build process https://review.openstack.org/648071 | 14:15 |
| odyssey4me | once neutron and nova are done, I can get on with cleaning up the legacy build implementation :) | 14:16 |
| *** gillesMo has joined #openstack-ansible | 14:20 | |
| guilhermesp | odyssey4me mnaser noonedeadpunk what do we do about roles marked as possible retired roles? https://etherpad.openstack.org/p/osa-python-build-simplification | 14:24 |
| odyssey4me | guilhermesp https://docs.openstack.org/infra/manual/drivers.html#retiring-a-project | 14:25 |
| guilhermesp | k so, almanach, molteniron, monasca*, searchlight, watcher, zaqar and zun are going to be retired | 14:28 |
| odyssey4me | however, given that not zun | 14:28 |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_tacker master: Update role for new source build process https://review.openstack.org/648361 | 14:28 |
| odyssey4me | oops - not zun | 14:28 |
| odyssey4me | but yes to all the others IMO | 14:29 |
| noonedeadpunk | hate retiring smth, but agree with Jesse | 14:29 |
| *** cmart has joined #openstack-ansible | 14:30 | |
| noonedeadpunk | btw, odyssey4me tacker is really tricky as well. Probably I had to place several patches for it... | 14:31 |
| odyssey4me | note that molteniron is still one of the integrated services for ocata/pike/queens (it got removed in rocky), so we won't be able to fully retire it until those go EOL... but the rest have never integrated, so they should be good | 14:31 |
| odyssey4me | yeah, I'm not sure if anyone's using tacker - but it has not had much activity, so if none of the maintainers are using it then perhaps it should also be up for retirement | 14:32 |
| guilhermesp | so let me mark then in the etherpad as a reminder | 14:32 |
| odyssey4me | retiring it is not forever - it can be reverted if someone wants to take on the maintenance | 14:33 |
| odyssey4me | but having the repositories around is not free for the core maintainers | 14:33 |
| odyssey4me | I find myself wondering whether swift should be retired too. It seems that most of our users are using Ceph RGW instead. | 14:34 |
| noonedeadpunk | it will be ok now for a while, I think | 14:34 |
| cloudnull | mornings all | 14:34 |
| gillesMo | Hello ! Anyone is using gnocchi and had a problem with a wrong endpoint URL in the response ? https://bugs.launchpad.net/openstack-ansible/+bug/1822113 | 14:34 |
| openstack | Launchpad bug 1822113 in openstack-ansible "gnocchi : wrong endpoint URL in responses" [Undecided,New] | 14:34 |
| cloudnull | https://review.openstack.org/#/c/647888/ - anyone want to give that a review? | 14:34 |
| odyssey4me | cloudnull I saw that - does it work? | 14:35 |
| cloudnull | cjloader has it running now | 14:35 |
| odyssey4me | cloudnull ok, there are many ways to skin that cat - but if you're happy with it and it's functional, I'm happy | 14:36 |
| cloudnull | cjloader mind giving that PR a review ? | 14:36 |
| *** phasespace has quit IRC | 14:36 | |
| cjloader | done | 14:37 |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-os_manila master: Test using integrated build https://review.openstack.org/647856 | 14:40 |
| openstackgerrit | Gabriele Santomaggio proposed openstack/openstack-ansible-rabbitmq_server master: Improve Mnesia IO performances https://review.openstack.org/648426 | 14:43 |
| noonedeadpunk | Btw, is it ok for zun to run tests only against ubuntu 16 and 18? | 14:46 |
| odyssey4me | noonedeadpunk for stein only 18 is needed, and I don't think anyone's put the effort it to add suse/centos support | 14:46 |
| odyssey4me | it's a new role | 14:46 |
| noonedeadpunk | i'll probably remove xenial and add centos/rhel (at least non-voting) | 14:48 |
| noonedeadpunk | or we will move it to the integrated build? | 14:48 |
| cloudnull | i did a lot of the work on the initial zun role, if I can help out there let me know | 14:52 |
| cloudnull | i had been trying to land https://review.openstack.org/#/c/571400/ for a while. | 14:53 |
| cloudnull | however looking at the pr seems we need to update some things structurally | 14:53 |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_zun master: Update role for new source build process https://review.openstack.org/648380 | 14:56 |
| noonedeadpunk | cloudnull got it:) | 14:57 |
| odyssey4me | noonedeadpunk I wouldn't bother adding any new platforms at the moment. Next step will be to make it use the integrated build for testing, so yeah - we'll have to add it there, similar to https://review.openstack.org/647898 | 14:57 |
| odyssey4me | ah yeah cloudnull - perhaps use https://review.openstack.org/647898 as a reference for the other bits that need doing | 14:58 |
| openstackgerrit | Merged openstack/openstack-ansible-plugins master: Correct spelling mistakes https://review.openstack.org/619531 | 15:01 |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_tacker master: Update role for new source build process https://review.openstack.org/648361 | 15:04 |
| *** cmart has quit IRC | 15:04 | |
| mnaser | odyssey4me: im worried we will end up with a branch with almost no testing.. I was thinking about this today, how do you feel about dropping that patch from a dependency, running both integrated and functional and then we can push in that afterwards again? | 15:08 |
| *** bgmccollum has quit IRC | 15:09 | |
| mnaser | so: run both functional and integrated and once we land all the changes we're happy with, we pull that change in | 15:09 |
| *** hamzy has quit IRC | 15:10 | |
| odyssey4me | mnaser part of the reason for it is that several of the role tests are broken in the roles, so to keep things simple I just got rid of them from the equation | 15:11 |
| mnaser | odyssey4me: ahhh. I see. | 15:11 |
| mnaser | I was rebasing things for those ones, I was thinking of like watching the topic and once we have all integrated things in, release that -2, +2 it and merge them all in | 15:12 |
| mnaser | let me make an etherpad to see what that looks like right now | 15:13 |
| odyssey4me | well, if we have the integrated tests implemented for the main service roles then we should be good - are there any roles in particular that are concerning you and you'd like the patches for them done? | 15:13 |
| odyssey4me | yeah - good idea, then we can get the rest of them done | 15:13 |
| odyssey4me | hmm, I think I realise now where the problem lies - this is all good for the os_ roles... but if we ditch the functional role test for the infra roles then we have nothing | 15:14 |
| openstackgerrit | Merged openstack/openstack-ansible-os_congress master: Update the congress service egg name to openstack_congress https://review.openstack.org/648125 | 15:14 |
| chandankumar | odyssey4me: hello | 15:15 |
| chandankumar | odyssey4me: yesterday test failures were weired after multiple rechecks it passed | 15:15 |
| chandankumar | the same tests one time passes on one job and failed on another | 15:16 |
| odyssey4me | chandankumar yeah, I'm not sure what's going on there and unfortunately I don't have acess to a public cloud any more to build a test system to inspect things | 15:16 |
| openstackgerrit | Gabriele Santomaggio proposed openstack/openstack-ansible-rabbitmq_server master: Improve Mnesia IO performances https://review.openstack.org/648437 | 15:17 |
| openstackgerrit | Merged openstack/openstack-ansible-os_rally master: Update role for new source build process https://review.openstack.org/648236 | 15:18 |
| openstackgerrit | Merged openstack/openstack-ansible-os_neutron master: vars: eliminate developer mode vars https://review.openstack.org/648395 | 15:22 |
| chandankumar | odyssey4me: if it still pops up, I can spin a vm on rdocloud | 15:24 |
| chandankumar | currently it is passing https://review.openstack.org/#/c/648026/ | 15:24 |
| mnaser | odyssey4me: ah, yes, I think that's a good point as well | 15:27 |
| mnaser | so in that case I would say lets drop that dependency and *worst* case scenario let's just move the functional tests to non-voting | 15:27 |
| mnaser | we can do that inside the role | 15:27 |
| *** jhesketh has joined #openstack-ansible | 15:28 | |
| odyssey4me | mnaser yeah, ok - makes sense, sorry about that - I forgot about the impact to all the infra roles | 15:30 |
| mnaser | all good, I didn't think of it till you brought it up :P | 15:30 |
| odyssey4me | We have a little work to get done to make the use of the integrated build for testing good there. | 15:31 |
| mnaser | I mean we can just run a base aio and call it a day, yes we got a bunch of extra services running too but *shrug* | 15:32 |
| *** jhesketh has quit IRC | 15:34 | |
| *** hamzy has joined #openstack-ansible | 15:35 | |
| *** cshen has quit IRC | 15:45 | |
| odyssey4me | mnaser the thing is that currently the functional test for keystone does a 3 node cluster, so does galera and so does rabbitmq... testing a cluster is very valuable, but I think I have figured out how we could make that possible for the integrated repo... it's just not been the highest priority just yet. | 15:50 |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_zun master: Update role for new source build process https://review.openstack.org/648380 | 15:53 |
| openstackgerrit | Merged openstack/openstack-ansible-os_tempest master: Reuse tempest_tempest_conf_overrides in tempestconf extra args https://review.openstack.org/648137 | 15:55 |
| *** udesale has quit IRC | 16:01 | |
| *** jbadiapa has quit IRC | 16:02 | |
| *** gyee has joined #openstack-ansible | 16:12 | |
| *** chandankumar is now known as raukadah | 16:13 | |
| *** cshen has joined #openstack-ansible | 16:19 | |
| mnaser | odyssey4me: yeah I think we can probably look into multinode tests at that point | 16:25 |
| *** cmart has joined #openstack-ansible | 16:30 | |
| *** cshen has quit IRC | 16:32 | |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_heat master: Update role for new source build process https://review.openstack.org/648398 | 16:32 |
| mnaser | odyssey4me: if you have a second, this is a bit more of a tricky one I'd like you to examine if you can - https://review.openstack.org/#/c/648026/ | 16:34 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_ironic master: Update role for new source build process https://review.openstack.org/648463 | 16:39 |
| raukadah | mnaser: How do we monitor OSA periodic jobs? DO we have ML to track it? | 16:39 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_swift master: Update role for new source build process https://review.openstack.org/648465 | 16:42 |
| mnaser | raukadah: I don't really know how to monitor periodic jobs right now, dunno if anyone puts eyes on them right now :\ | 16:43 |
| *** dcdamien has joined #openstack-ansible | 16:44 | |
| mnaser | noonedeadpunk: awesome work on os_tacker | 16:44 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_tacker master: Correct notification driver https://review.openstack.org/635682 | 16:45 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_tacker master: config: add linebreak after oslo_messaging notif. driver https://review.openstack.org/646045 | 16:45 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_tacker master: Updated from OpenStack Ansible Tests https://review.openstack.org/639551 | 16:45 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_tacker master: fix tox python3 overrides https://review.openstack.org/606785 | 16:45 |
| noonedeadpunk | mnaser: I'd probably had to split it into separate commits, but feel like had problems with dependencies, as role was a bit broken... | 16:48 |
| noonedeadpunk | thanks) | 16:49 |
| *** luksky has quit IRC | 16:49 | |
| raukadah | mnaser: ack! thanks! | 16:49 |
| mnaser | noonedeadpunk: yeah, that's fine, that's what happens when our stuff just gets behind | 16:49 |
| odyssey4me | mnaser they're easy to monitor - just look at http://zuul.openstack.org/builds?project=openstack%2Fopenstack-ansible&pipeline=periodic | 16:50 |
| odyssey4me | raukadah ^ | 16:51 |
| mnaser | oh TIL | 16:51 |
| raukadah | odyssey4me: may we can extend this dashboard to rssfeed or mail stuff to notify following jobs failed today | 16:52 |
| raukadah | I will check with tristin tomorrow | 16:52 |
| * raukadah is lazy to open an url | 16:53 | |
| noonedeadpunk | seems that only os_manila stays without patch. It almost ready, but there are some neats. And cloudnull actively work with it | 16:54 |
| noonedeadpunk | so I'll probably won't get in to it | 16:55 |
| odyssey4me | raukadah that's a zuul dashboard, and zuul is open source, so yes it is possible if it's not been done already ;) | 16:57 |
| openstackgerrit | Merged openstack/openstack-ansible-os_tacker master: Add source code repository notes link to README https://review.openstack.org/591264 | 16:58 |
| raukadah | mnaser: odyssey4me from one of the today's discussion one topic came out how to test stuff with tempest where complex test requirements is required | 17:00 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_manila master: venv: fix venv build https://review.openstack.org/648468 | 17:01 |
| raukadah | like octavia in osa, we have a playbook to test octavia amphora image | 17:01 |
| raukadah | and how projects can calloborate together to solve these issues | 17:01 |
| mnaser | noonedeadpunk: it looks like it was done but there's a small bug -- https://review.openstack.org/#/c/648468/ | 17:01 |
| mnaser | may interest cloudnull ^ | 17:02 |
| mnaser | raukadah: could you put this on the pig list? :D | 17:02 |
| mnaser | ptg** | 17:02 |
| raukadah | each projects have needs but not broadcasted with in community leading to evolution of different tools | 17:02 |
| odyssey4me | dang, noonedeadpunk is killing it! | 17:02 |
| raukadah | mnaser: I am not going to ptg due to visa issues | 17:02 |
| raukadah | mnaser: do we have etherpad where I can put my thoughts? | 17:03 |
| mnaser | gdi | 17:04 |
| mnaser | I was cleaning up the f'n ether pad and it all got messed up and now we have some missing roles in the non integrated list | 17:04 |
| mnaser | GAH | 17:04 |
| mnaser | raukadah: https://etherpad.openstack.org/p/osa-train-ptg | 17:05 |
| raukadah | mnaser: I will put that! | 17:06 |
| raukadah | odyssey4me: thanks! | 17:08 |
| odyssey4me | mnaser ctrl-z ? | 17:08 |
| mnaser | odyssey4me: that wasn't fixing it, I saw a lot of duplicate data, cleaned up, refreshed, and it all changed weirdly | 17:09 |
| mnaser | almost done backfilling | 17:09 |
| mnaser | boo | 17:09 |
| odyssey4me | mnaser https://media.giphy.com/media/xUKTfpLS9BYUT2vprT/giphy.gif | 17:10 |
| mnaser | YUP | 17:10 |
| odyssey4me | oh man, this is a beautiful thing to behold: https://media.giphy.com/media/aBfSGPDRDSdqw/giphy.gif | 17:11 |
| *** hamzaachi has quit IRC | 17:11 | |
| odyssey4me | I know evrardjp will love it. | 17:11 |
| guilhermesp | lol odyssey4me hahaha | 17:11 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_blazar master: Update role for new source build process https://review.openstack.org/647835 | 17:11 |
| *** hamzaachi has joined #openstack-ansible | 17:14 | |
| evrardjp | odyssey4me: hahahah | 17:14 |
| evrardjp | glrlrlrlrlrlrllrrllrlrlr | 17:14 |
| mnaser | I think we might be able to land all python_venv_build changes before Saturday and branching | 17:16 |
| mnaser | dis makes me happy | 17:16 |
| mnaser | we're 8 away from being able to rip out repo_build | 17:19 |
| mnaser | ill start working on that even if it fails I guess | 17:19 |
| odyssey4me | mnaser once the nova and tempest ones merge, I'll be getting on it | 17:24 |
| cloudnull | woot! | 17:24 |
| cloudnull | that's fairly awesome ! repo-build is no more :) | 17:25 |
| odyssey4me | I have time this evening to get it done. | 17:25 |
| cloudnull | *almost* | 17:25 |
| odyssey4me | cloudnull :) it's been a journey | 17:25 |
| *** nwonknu has quit IRC | 17:25 | |
| raukadah | It's night in India :-) | 17:25 |
| odyssey4me | of course, that also means we can pivot to py3 straight after | 17:25 |
| cloudnull | very cool to think about the evolution everything | 17:25 |
| *** cshen has joined #openstack-ansible | 17:30 | |
| odyssey4me | noonedeadpunk mnaser we have a circular dependency between https://review.openstack.org/#/c/648361/ and https://review.openstack.org/#/c/635682/ | 17:32 |
| odyssey4me | the first depends on the second in the commit msg, but the second is rebased on the first | 17:33 |
| odyssey4me | lemme just merge them | 17:33 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: repo: stop using repo_build https://review.openstack.org/648477 | 17:33 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tacker master: Update role for new source build process https://review.openstack.org/648361 | 17:34 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tacker master: Correct notification driver https://review.openstack.org/635682 | 17:34 |
| *** hamzaachi has quit IRC | 17:34 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tacker master: fix tox python3 overrides https://review.openstack.org/606785 | 17:34 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tacker master: Updated from OpenStack Ansible Tests https://review.openstack.org/639551 | 17:34 |
| odyssey4me | that should do it | 17:35 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: repo: stop using repo_build https://review.openstack.org/648477 | 17:35 |
| noonedeadpunk | odyssey4me: yep, sure. | 17:35 |
| raukadah | odyssey4me: mnaser one more stupid question, do we have job which checks to all OSA repos when a new ansible release lands whats are things getting broken? | 17:35 |
| *** dcdamien has quit IRC | 17:36 | |
| *** nwonknu has joined #openstack-ansible | 17:36 | |
| odyssey4me | raukadah there is an experimental pipeline job on the integrated repo, but it'd be nice to have something check for a new version and automatically propose an update to it | 17:36 |
| noonedeadpunk | just didn't want to include Kevin's patch to mine just not to stole it:) | 17:36 |
| odyssey4me | then it'd be in the review queue and therefore test things :) | 17:37 |
| raukadah | odyssey4me: job link? | 17:37 |
| odyssey4me | raukadah https://github.com/openstack/openstack-ansible/blob/master/zuul.d/jobs.yaml#L50-L74 | 17:37 |
| noonedeadpunk | odyssey4me I just guess, that https://review.openstack.org/#/c/648361/ shoudl be rebased on top of https://review.openstack.org/#/c/635682/ | 17:39 |
| noonedeadpunk | oh, no, disregard | 17:40 |
| *** gillesMo has quit IRC | 17:40 | |
| *** aedc has joined #openstack-ansible | 17:43 | |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_placement master: Update role for new source build process https://review.openstack.org/648483 | 17:44 |
| *** cshen has quit IRC | 17:44 | |
| mnaser | ok great | 17:44 |
| mnaser | I think we have changes for all roles now | 17:44 |
| guilhermesp | would be good to mark placement to be integrated too | 17:45 |
| mnaser | guilhermesp: I added a note, placement is complicated | 17:45 |
| mnaser | we need to have an upgrade path | 17:45 |
| mnaser | and I don't think we can land it this change | 17:45 |
| mnaser | this cycle** | 17:45 |
| guilhermesp | yes I agree | 17:45 |
| mnaser | unless we release with broken upgrades :-P | 17:46 |
| odyssey4me | mnaser upgrades aren't normally in a working state on release | 17:46 |
| odyssey4me | are nova keeping placement in the code base for stein's release? | 17:46 |
| odyssey4me | it would make sense to get all roles testing with the integrated build first, so that we can also implement upgrade tests for check/gate - then we can get placement integrated properly | 17:47 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: repo: stop using repo_build + pip_install https://review.openstack.org/648477 | 17:49 |
| *** spatel has joined #openstack-ansible | 17:50 | |
| mnaser | odyssey4me: nova will be keeping placement in-tree for stein | 17:51 |
| mnaser | being ripped out soon during pig timeframe | 17:51 |
| openstackgerrit | Merged openstack/openstack-ansible-os_panko master: Update role for new source build process https://review.openstack.org/648335 | 17:54 |
| *** jbadiapa has joined #openstack-ansible | 17:54 | |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_blazar master: Update role for new source build process https://review.openstack.org/647835 | 17:54 |
| openstackgerrit | Merged openstack/openstack-ansible-os_cloudkitty master: Update role for new source build process https://review.openstack.org/648338 | 17:55 |
| *** SmearedBeard has quit IRC | 17:55 | |
| odyssey4me | mnaser ok, I think we may still have pip_install implemented in openstack_hosts, so that will also need to be pulled out | 17:55 |
| odyssey4me | and https://review.openstack.org/648477 should depend on the patch for that, along with depending on https://review.openstack.org/648071 and https://review.openstack.org/648026, then we'll have a real test of the whole thing | 17:56 |
| odyssey4me | I need to run for now - I'll check back later to see what it all looks like - thanks for picking that up! | 17:57 |
| mnaser | odyssey4me: ok this whole switching to integrated with broken role jobs is being rough. I think I will push up a patch to add a new project template called openstack-ansible-linters which runs the lint job, then push a patch removing the Ansible role jobs and adding lintters, then adding the integrated jobs | 18:01 |
| goldenfri | Hi, sorry to bother you guys does anyone a few minutes today to pretty please help me troubleshoot my deployment? I'm not sure how to troubleshoot any further, I'm getting the following error and not sure what the problem is: http://paste.openstack.org/show/748502/ | 18:03 |
| goldenfri | I can run that command just fine on the ceph monitors | 18:03 |
| *** kopecmartin is now known as kopecmartin|off | 18:11 | |
| jrosser | goldenfri: i would expect that task to be executed on your ceph monitor, not the glance container | 18:12 |
| jrosser | here is why https://github.com/openstack/openstack-ansible-ceph_client/blob/master/tasks/ceph_auth.yml#L26 | 18:13 |
| goldenfri | right and I can execute it by hand just fine on the monitor | 18:13 |
| goldenfri | So I'm confused why it is complaining | 18:13 |
| openstackgerrit | Merged openstack/openstack-ansible-os_manila master: venv: fix venv build https://review.openstack.org/648468 | 18:14 |
| goldenfri | ceph_mon_host is the ceph-mon_container right? | 18:15 |
| jrosser | goldenfri: and silly question but 172.29.236.11 is your ceph monitor? | 18:15 |
| goldenfri | its the host | 18:15 |
| goldenfri | that the monitor container runs on | 18:15 |
| openstackgerrit | Merged openstack/openstack-ansible-os_nova master: Update role for new source build process https://review.openstack.org/648071 | 18:15 |
| goldenfri | oh... | 18:15 |
| jrosser | hmm, yes | 18:15 |
| goldenfri | should it be pointing to the container ip not the host ip? | 18:16 |
| jrosser | well its a bit more complicated than that, becasue the connection plugin will ssh to the required host then attach to the container | 18:17 |
| *** DanyC_ has joined #openstack-ansible | 18:17 | |
| jrosser | there is much more info about that if you run openstack-ansible with -vvvv and can see the decisions the connection plugin made | 18:17 |
| *** DanyC has quit IRC | 18:20 | |
| *** DanyC has joined #openstack-ansible | 18:21 | |
| goldenfri | There is a lot there, but its saying container_name: "172.29.236.11" and physical_host: "infra1" That doesn't seem correct | 18:21 |
| *** hamzaachi has joined #openstack-ansible | 18:25 | |
| goldenfri | oh man that was it! I shouldn't have listed the mon hosts in the user_variables.yml I commented them out and that fixed it. THANK YOU! jrosser | 18:26 |
| jrosser | aaaahhh! yes thats for if you integrate with an external ceph cluster | 18:27 |
| jrosser | but if you deploy ceph with OSA there is an ansible host group | 18:27 |
| goldenfri | ahhh yea that wasn't super clear in the docs, thank you again for your great help I really appreciate it!! | 18:28 |
| jrosser | there is an example of that here https://github.com/openstack/openstack-ansible/blob/master/playbooks/ceph-rgw-install.yml#L17 | 18:28 |
| openstackgerrit | Merged openstack/openstack-ansible-os_tempest master: Update role for new source build process https://review.openstack.org/648026 | 18:28 |
| *** cmart has quit IRC | 18:29 | |
| jrosser | ceph-rgw is the host group, or ceph_rgws is a list of externally deployed rgw | 18:29 |
| goldenfri | ahhh :) | 18:29 |
| mnaser | cores: https://review.openstack.org/#/c/648483/ is ready to land to fix up placement | 18:29 |
| mnaser | also Blazar ready to land: https://review.openstack.org/#/c/647835/ | 18:31 |
| mnaser | and heat ready to land too https://review.openstack.org/#/c/648398/ | 18:31 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-tests master: zuul: create openstack-ansible-linters project-template https://review.openstack.org/648502 | 18:34 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_cinder master: Test using integrated build https://review.openstack.org/647847 | 18:37 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_neutron master: Test using integrated build https://review.openstack.org/647843 | 18:38 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_heat master: Replace role tests with integrated build tests https://review.openstack.org/647862 | 18:38 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_nova master: Test using integrated build https://review.openstack.org/647842 | 18:39 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_keystone master: Test using integrated build https://review.openstack.org/647844 | 18:39 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_glance master: Test using integrated build https://review.openstack.org/647848 | 18:40 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_masakari master: Clean up testing https://review.openstack.org/648183 | 18:40 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_barbican master: Test using integrated build https://review.openstack.org/648104 | 18:40 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_blazar master: Test using integrated build https://review.openstack.org/648106 | 18:41 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_congress master: Test using integrated build https://review.openstack.org/648108 | 18:41 |
| *** DanyC has quit IRC | 18:42 | |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_aodh master: Test using integrated build https://review.openstack.org/647851 | 18:42 |
| *** aedc has quit IRC | 18:48 | |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: repo: stop using repo_build + pip_install https://review.openstack.org/648477 | 18:49 |
| mnaser | tracking progress here too for this https://etherpad.openstack.org/p/osa-use-integrated-build | 18:51 |
| openstackgerrit | Merged openstack/openstack-ansible-os_tacker master: Update role for new source build process https://review.openstack.org/648361 | 18:52 |
| openstackgerrit | Merged openstack/openstack-ansible-os_tacker master: Correct notification driver https://review.openstack.org/635682 | 18:52 |
| openstackgerrit | Merged openstack/openstack-ansible-os_tacker master: Updated from OpenStack Ansible Tests https://review.openstack.org/639551 | 18:52 |
| *** aedc has joined #openstack-ansible | 18:52 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_tacker master: fix tox python3 overrides https://review.openstack.org/606785 | 18:59 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_swift master: Test using integrated build https://review.openstack.org/648506 | 19:01 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_swift master: Update role for new source build process https://review.openstack.org/648465 | 19:02 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_swift master: Updated from OpenStack Ansible Tests https://review.openstack.org/639550 | 19:02 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_octavia master: Test using integrated build https://review.openstack.org/648509 | 19:09 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_octavia master: Updated from OpenStack Ansible Tests https://review.openstack.org/639544 | 19:11 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_octavia master: Update role for new source build process https://review.openstack.org/648231 | 19:11 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_octavia master: Break out config files for each octavia service https://review.openstack.org/603228 | 19:12 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_octavia master: Fix old docs listing octavia as experimental https://review.openstack.org/636776 | 19:12 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_ironic master: Test using integrated build https://review.openstack.org/648512 | 19:15 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_octavia master: Test using integrated build https://review.openstack.org/648509 | 19:15 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_ironic master: Update role for new source build process https://review.openstack.org/648463 | 19:17 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_ironic master: Replace usage of netloc filters to urlsplit https://review.openstack.org/648190 | 19:17 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_ironic master: Updated from OpenStack Ansible Tests https://review.openstack.org/639533 | 19:17 |
| *** cshen has joined #openstack-ansible | 19:17 | |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_ironic master: missing document start "---" https://review.openstack.org/614107 | 19:18 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_ceilometer master: Test using integrated build https://review.openstack.org/648513 | 19:23 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_panko master: Test using integrated build https://review.openstack.org/648514 | 19:24 |
| openstackgerrit | James Denton proposed openstack/openstack-ansible-os_neutron master: [WIP] Adds DPDK support for Open vSwitch https://review.openstack.org/590810 | 19:26 |
| *** luksky has joined #openstack-ansible | 19:26 | |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_gnocchi master: Test using integrated build https://review.openstack.org/648515 | 19:26 |
| *** priteau has quit IRC | 19:27 | |
| mnaser | cloudnull: have you ever seen this behavior? "ERROR! The requested handler 'systemd service changed' was not found in either the main handlers list nor in the listening handlers list" | 19:30 |
| mnaser | http://logs.openstack.org/56/647856/6/check/openstack-ansible-deploy-aio_metal_manila-ubuntu-bionic/f984b76/job-output.txt.gz | 19:30 |
| mnaser | this landed not long ago but I would imagine ti would expose it then https://review.openstack.org/#/c/648071/7 | 19:30 |
| *** cshen has quit IRC | 19:34 | |
| *** cshen has joined #openstack-ansible | 19:34 | |
| *** aedc has quit IRC | 19:44 | |
| *** cshen has quit IRC | 19:46 | |
| *** SmearedBeard has joined #openstack-ansible | 19:47 | |
| *** fghaas has joined #openstack-ansible | 19:49 | |
| *** cshen has joined #openstack-ansible | 19:56 | |
| *** cshen has quit IRC | 20:08 | |
| *** pcaruana has quit IRC | 20:08 | |
| *** cshen has joined #openstack-ansible | 20:09 | |
| jrosser | mnaser: i wonder if that ^ is due to the include/import tasks change | 20:12 |
| *** cshen has quit IRC | 20:26 | |
| *** ansmith has quit IRC | 20:29 | |
| *** cmart has joined #openstack-ansible | 20:30 | |
| *** cshen has joined #openstack-ansible | 20:31 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_blazar master: Update role for new source build process https://review.openstack.org/647835 | 20:39 |
| mnaser | jrosser: im curious as well | 20:53 |
| jrosser | mnaser: i was just reading this https://medium.com/opsops/include-role-import-role-and-handlers-in-ansible-b32a5386a555 | 20:53 |
| mnaser | I have to do $day_job things for a little bit now drafting up some proposal but I'd appreciate if anyone can dig into it | 20:53 |
| mnaser | if not ill get back at it | 20:53 |
| jrosser | but it is unclear if that is whats going on | 20:53 |
| jrosser | for now putting the import_role back to include_role might give a clue | 20:53 |
| *** hamzy has quit IRC | 20:54 | |
| mnaser | jrosser: AH | 20:54 |
| mnaser | jrosser: https://review.openstack.org/#/c/648071/7/tasks/main.yml L141 | 20:54 |
| jrosser | yeah, i'm slightly struggling to match those results to our failure though, it doesnt quite look the same | 20:54 |
| mnaser | but it's the only change that relates to systemd_service | 20:54 |
| mnaser | and we've only always used include_role with systemd_service so.. | 20:55 |
| jrosser | but we are playbook->role->role and things have been known to go all a bit wonky with several levels | 20:55 |
| mnaser | yeah {import, include}_{role, playbook,file} is all a bit of magic now | 20:55 |
| jrosser | i'll edit that quickly now if you are busy | 20:56 |
| mnaser | jrosser: writing commit msg :P | 20:56 |
| jrosser | hah too quick :) | 20:56 |
| mnaser | jrosser: but if you don't mind doing a depends-on change in integrated to test it | 20:56 |
| mnaser | once I push it up | 20:56 |
| mnaser | that'd be nice | 20:56 |
| jrosser | sure | 20:56 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_nova master: systemd: move to include_role https://review.openstack.org/648534 | 20:57 |
| mnaser | jrosser: ^ | 20:57 |
| jrosser | will be interesting to see if ceph_client suffers in a similar way | 20:58 |
| *** cmart has quit IRC | 20:59 | |
| *** fghaas has quit IRC | 21:01 | |
| openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: Test nova [include|import]_role for systemd_networkd https://review.openstack.org/648536 | 21:05 |
| *** ansmith has joined #openstack-ansible | 21:06 | |
| *** DanyC has joined #openstack-ansible | 21:07 | |
| *** openstackgerrit has quit IRC | 21:07 | |
| cloudnull | mnaser i have not | 21:10 |
| jrosser | cloudnull: this looks related-ish https://medium.com/opsops/include-role-import-role-and-handlers-in-ansible-b32a5386a555 | 21:11 |
| *** cmart has joined #openstack-ansible | 21:16 | |
| cloudnull | hum yea that looks like the issue | 21:18 |
| cloudnull | TIL | 21:18 |
| cloudnull | hows it jrosser? | 21:18 |
| *** cmart has quit IRC | 21:19 | |
| *** cmart has joined #openstack-ansible | 21:19 | |
| jrosser | cloudnull: keeping busy.... got some gpus working | 21:19 |
| cloudnull | nice! | 21:20 |
| cloudnull | pass through or vgpu ? | 21:20 |
| jrosser | passthrough, i need to look how expensive vgpu might be | 21:20 |
| jrosser | iirc there is licencing on nvidia, but ive not really investigated that much | 21:21 |
| cloudnull | how have you liked the passthrough things ? | 21:25 |
| cloudnull | working well? | 21:25 |
| *** cmart has quit IRC | 21:28 | |
| jrosser | we've had a little trouble with migrating vm | 21:29 |
| *** cmart has joined #openstack-ansible | 21:29 | |
| jrosser | but not really looked at whats going on there yet | 21:30 |
| jrosser | tbh the most time spent getting hold of a bios that didnt just freak out during POST when the GPU was present | 21:31 |
| cloudnull | oh, was the BIOS a nightmare ? | 21:31 |
| jrosser | just a lot of rounds with the server vendor till they sent over a new bios | 21:32 |
| jrosser | and whaddya know its right there in the release notes! | 21:32 |
| jrosser | just not available for public download yet | 21:32 |
| cloudnull | ha! gotta love that | 21:33 |
| *** cshen has quit IRC | 21:36 | |
| *** hamzaachi has quit IRC | 21:36 | |
| *** spatel has quit IRC | 21:47 | |
| mnaser | gah | 22:02 |
| mnaser | failed on ceph_client | 22:02 |
| *** openstackgerrit has joined #openstack-ansible | 22:02 | |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_nova master: systemd: move to include_role https://review.openstack.org/648534 | 22:02 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_nova master: Test using integrated build https://review.openstack.org/647842 | 22:03 |
| mnaser | gonna keep an eye on https://review.openstack.org/#/c/647842/ jrosser | 22:04 |
| jrosser | it's jsut failed | 22:04 |
| jrosser | fatal: [aio1]: FAILED! => {"msg": "The conditional check 'nova_install_method == 'source'' failed. The error was: error while evaluating conditional (nova_install_method == 'source'): 'nova_install_method' is undefined\n\nThe error appears to have been in '/etc/ansible/roles/python_venv_build/handlers/main.yml' | 22:04 |
| jrosser | so it's the same story with import_role on python_venv_build i think | 22:04 |
| mnaser | yep, saw that | 22:04 |
| mnaser | orly? I thought that failed with ceph_client | 22:05 |
| mnaser | ah yes its on RUNNING HANDLER [python_venv_build : venv changed] ***************************** | 22:06 |
| mnaser | oh I think I know why | 22:06 |
| jrosser | i think this https://github.com/openstack/openstack-ansible-os_nova/blob/master/tasks/nova_install.yml#L49 | 22:06 |
| mnaser | ok so I think what happens is | 22:07 |
| mnaser | when venv changed handler is called | 22:07 |
| mnaser | is does something with filtered_nova_services ? | 22:07 |
| mnaser | or the handler is running inside the scope of the role | 22:07 |
| mnaser | does that mean we don't have handlers for venv in our other roles? | 22:08 |
| mnaser | hmm, we do have venv changed handler being called in other places | 22:08 |
| mnaser | cause all the other roles seem to be ok with this.. | 22:08 |
| jrosser | did the others all get changed to import_role too? | 22:08 |
| mnaser | yeah | 22:09 |
| jrosser | curious | 22:09 |
| mnaser | BUT the others we moved them up to main.yaml | 22:09 |
| mnaser | and I didn't move from include_tasks to import_tasks | 22:09 |
| *** cshen has joined #openstack-ansible | 22:09 | |
| mnaser | this moved the python venv build to import_role and the file it belongs to with a import_task | 22:09 |
| mnaser | I wonder if https://github.com/openstack/openstack-ansible-os_nova/blob/master/tasks/main.yml#L88 is the issue | 22:10 |
| mnaser | I think that's more likely | 22:10 |
| mnaser | I will move it from import_tasks to include_tasks. I think that's likely a difference maker imho | 22:11 |
| *** SmearedBeard has quit IRC | 22:12 | |
| jrosser | mnaser: look at this http://codesearch.openstack.org/?q=venv%20changed&i=nope&files=&repos= | 22:14 |
| *** cshen has quit IRC | 22:14 | |
| jrosser | the hander in python_venv_build is a no-op and there are handlers in the calling roles | 22:14 |
| jrosser | so there is still a chance that we have import/include_role changing the handler behaviour | 22:15 |
| *** cmart has quit IRC | 22:15 | |
| mnaser | jrosser: I wonder if the integrated repo does something odd | 22:19 |
| mnaser | jrosser: neutron uses these with no problems | 22:21 |
| mnaser | there you og | 22:21 |
| mnaser | we use | 22:21 |
| mnaser | include_role for nova in integrated | 22:21 |
| mnaser | the only one which we use it for | 22:22 |
| mnaser | bet that's doing it | 22:22 |
| mnaser | https://github.com/openstack/openstack-ansible/blob/master/playbooks/common-playbooks/nova.yml#L142-L160 | 22:23 |
| mnaser | we totally don't need that anymore because with the new stuff we include "../defaults/repo_packages/openstack_services.yml" anyways | 22:23 |
| mnaser | so adding those extra vars is useless | 22:23 |
| jrosser | we could also copy the pattern from the other common_playbooks with pre_tasks: roles: post_tasks: rather than mess with include/import_role magic behaviours | 22:25 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: repo: stop using repo_build + pip_install https://review.openstack.org/648477 | 22:28 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: nova: refactor common playbook to use role instead of include_role https://review.openstack.org/648551 | 22:28 |
| mnaser | jrosser: I think that's the trick (first patch) | 22:28 |
| mnaser | err or second one | 22:28 |
| jrosser | and this one is good https://review.openstack.org/#/c/648534/ | 22:30 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: repo: stop using repo_build + pip_install https://review.openstack.org/648477 | 22:33 |
| mnaser | jrosser: I think that is no longer necessary imho | 22:33 |
| mnaser | we'll wait and see | 22:33 |
| spotz | BOOM!!!! | 22:33 |
| mnaser | jrosser: mind checking https://review.openstack.org/#/c/648502/ ? | 22:35 |
| mnaser | if we get that in, I can later tonight land code (restoring +2s on the same patches that we voted in to add integration testing) | 22:35 |
| jrosser | lgtm | 22:36 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: nova: refactor common playbook to use role instead of include_role https://review.openstack.org/648551 | 22:36 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: repo: stop using repo_build + pip_install https://review.openstack.org/648477 | 22:36 |
| mnaser | thank you sir | 22:37 |
| mnaser | https://etherpad.openstack.org/p/osa-python-build-simplification and https://etherpad.openstack.org/p/osa-use-integrated-build looking good | 22:37 |
| jrosser | no worries - you guys have done awesome work here | 22:38 |
| mnaser | https://review.openstack.org/#/c/648398/ and https://review.openstack.org/#/c/648483/ ready to ship too | 22:39 |
| *** fabstao has joined #openstack-ansible | 22:40 | |
| *** fabstao has quit IRC | 22:41 | |
| jrosser | mnaser: one query on https://review.openstack.org/#/c/648398/2 | 22:47 |
| *** Pandry has joined #openstack-ansible | 22:48 | |
| *** Pandry has quit IRC | 22:51 | |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-repo_server master: pypiserver: drop pypi server https://review.openstack.org/648556 | 22:51 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_heat master: Update role for new source build process https://review.openstack.org/648398 | 22:53 |
| mnaser | jrosser: dang, good catch | 22:53 |
| *** luksky has quit IRC | 23:05 | |
| openstackgerrit | Merged openstack/openstack-ansible-tests master: zuul: create openstack-ansible-linters project-template https://review.openstack.org/648502 | 23:18 |
| *** DanyC has quit IRC | 23:24 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_manila master: Updated from OpenStack Ansible Tests https://review.openstack.org/642843 | 23:27 |
| openstackgerrit | Merged openstack/openstack-ansible-os_placement master: Update role for new source build process https://review.openstack.org/648483 | 23:33 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: repo: stop using repo_build + pip_install https://review.openstack.org/648477 | 23:38 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-repo_server master: pypiserver: drop pypi server https://review.openstack.org/648556 | 23:40 |
| mnaser | jrosser: sweet, it just passed os_nova | 23:41 |
| * mnaser will start updating patches to add depends-on | 23:42 | |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_nova master: Test using integrated build https://review.openstack.org/647842 | 23:44 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_octavia master: Test using integrated build https://review.openstack.org/648509 | 23:45 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_panko master: Test using integrated build https://review.openstack.org/648514 | 23:45 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_gnocchi master: Test using integrated build https://review.openstack.org/648515 | 23:46 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_ceilometer master: Test using integrated build https://review.openstack.org/648513 | 23:46 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_swift master: Test using integrated build https://review.openstack.org/648506 | 23:47 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_ironic master: Test using integrated build https://review.openstack.org/648512 | 23:47 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_glance master: Test using integrated build https://review.openstack.org/647848 | 23:47 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_barbican master: Test using integrated build https://review.openstack.org/648104 | 23:48 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_masakari master: Clean up testing https://review.openstack.org/648183 | 23:48 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_neutron master: Test using integrated build https://review.openstack.org/647843 | 23:48 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_keystone master: Test using integrated build https://review.openstack.org/647844 | 23:49 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_aodh master: Test using integrated build https://review.openstack.org/647851 | 23:49 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_heat master: Replace role tests with integrated build tests https://review.openstack.org/647862 | 23:50 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_cinder master: Test using integrated build https://review.openstack.org/647847 | 23:50 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_blazar master: Test using integrated build https://review.openstack.org/648106 | 23:50 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_congress master: Test using integrated build https://review.openstack.org/648108 | 23:50 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_manila master: Test using integrated build https://review.openstack.org/647856 | 23:50 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: Add zero-container AIO distro install jobs https://review.openstack.org/647840 | 23:51 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: Remove experimetal scenarios https://review.openstack.org/647897 | 23:51 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: Add panko to integrated build https://review.openstack.org/647898 | 23:51 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_heat master: Update role for new source build process https://review.openstack.org/648398 | 23:54 |
| *** gyee has quit IRC | 23:57 | |
| *** jhesketh has joined #openstack-ansible | 23:58 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
