| *** Weifan has quit IRC | 00:07 | |
| *** cmart has quit IRC | 00:11 | |
| *** dxiri has quit IRC | 00:20 | |
| *** gyee has quit IRC | 00:24 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_cinder master: Remove deprecated use of tests as filters https://review.opendev.org/657158 | 00:26 |
|---|---|---|
| *** Weifan has joined #openstack-ansible | 00:28 | |
| *** hamzy has joined #openstack-ansible | 00:31 | |
| *** dxiri has joined #openstack-ansible | 00:36 | |
| *** dxiri has quit IRC | 00:40 | |
| openstackgerrit | Merged openstack/openstack-ansible master: Remove the use of 'private' with include_role during bootstrap https://review.opendev.org/656582 | 00:42 |
| openstackgerrit | Merged openstack/openstack-ansible master: Enable standalone Octavia for testing https://review.opendev.org/653849 | 00:48 |
| *** KeithMnemonic has quit IRC | 00:56 | |
| snadge | ok.. i need to consider deploying openstack stein on a test system, ideally using ansible | 01:18 |
| snadge | on Cent 7 | 01:21 |
| *** Weifan has quit IRC | 01:21 | |
| *** dxiri has joined #openstack-ansible | 01:21 | |
| *** dxiri has quit IRC | 01:28 | |
| snadge | im going to try and follow what I see here.. https://github.com/openstack/openstack-ansible | 01:30 |
| *** Weifan has joined #openstack-ansible | 01:34 | |
| *** Weifan has quit IRC | 01:39 | |
| openstackgerrit | Logan V proposed openstack/openstack-ansible master: Add support to deploy placement https://review.opendev.org/656891 | 01:44 |
| openstackgerrit | Nicolas Bock proposed openstack/openstack-ansible-lxc_hosts master: Update (yet again) SUSE lxc container name https://review.opendev.org/657720 | 02:00 |
| nicolasbock | Just a heads up, the location of the openSUSE lxc container image changed again :( I'll try to find out why and why we can't simply use one constant name. | 02:07 |
| *** nicolasbock has quit IRC | 02:15 | |
| *** kplant has quit IRC | 03:23 | |
| openstackgerrit | Logan V proposed openstack/openstack-ansible master: Add support to deploy placement https://review.opendev.org/656891 | 03:32 |
| *** DanyC has joined #openstack-ansible | 03:50 | |
| *** DanyC has quit IRC | 03:54 | |
| *** udesale has joined #openstack-ansible | 03:55 | |
| *** ivve has quit IRC | 04:35 | |
| *** raukadah is now known as chandankumar | 04:40 | |
| *** hwoarang has quit IRC | 05:04 | |
| *** hwoarang has joined #openstack-ansible | 05:05 | |
| *** radeks has joined #openstack-ansible | 05:35 | |
| *** ivve has joined #openstack-ansible | 05:39 | |
| openstackgerrit | Merged openstack/ansible-role-systemd_service stable/stein: Require network-online insetad of network target https://review.opendev.org/657694 | 05:59 |
| *** DanyC has joined #openstack-ansible | 06:03 | |
| *** udesale has quit IRC | 06:05 | |
| *** udesale has joined #openstack-ansible | 06:05 | |
| jrosser | Can we update the review dashboard link in the irc channel topic? It behaves really weirdly with the opendev redirect with IRCCloud/iOS | 06:13 |
| *** hjohnson1 has joined #openstack-ansible | 06:28 | |
| *** hjohnson1 has left #openstack-ansible | 06:28 | |
| evrardjp | good morning | 06:28 |
| openstackgerrit | Merged openstack/openstack-ansible-lxc_hosts master: Update (yet again) SUSE lxc container name https://review.opendev.org/657720 | 06:29 |
| *** yolanda_ has joined #openstack-ansible | 06:31 | |
| *** udesale has quit IRC | 07:03 | |
| *** udesale has joined #openstack-ansible | 07:03 | |
| *** DanyC has quit IRC | 07:17 | |
| *** DanyC has joined #openstack-ansible | 07:29 | |
| *** DanyC has quit IRC | 07:33 | |
| *** vnogin has joined #openstack-ansible | 07:47 | |
| *** starborn has joined #openstack-ansible | 07:52 | |
| *** vnogin has quit IRC | 07:53 | |
| *** oligau has quit IRC | 08:07 | |
| *** hjohnson1 has joined #openstack-ansible | 08:10 | |
| *** vnogin has joined #openstack-ansible | 08:10 | |
| *** oligau has joined #openstack-ansible | 08:13 | |
| *** vnogin has quit IRC | 08:14 | |
| *** DanyC has joined #openstack-ansible | 08:17 | |
| *** hamzaachi has joined #openstack-ansible | 08:21 | |
| *** hamzaachi has quit IRC | 08:23 | |
| *** hamzaachi has joined #openstack-ansible | 08:28 | |
| *** starborn has quit IRC | 08:41 | |
| *** starborn has joined #openstack-ansible | 08:42 | |
| CeeMac | morning | 08:47 |
| _janno | why does openstack-ansible ship its own ansible executable? | 08:47 |
| _janno | Do I have to expect any problems when using another ansible? | 08:48 |
| *** vnogin has joined #openstack-ansible | 08:55 | |
| jrosser | _janno: do you mean the openstack-ansible wrapper script or the ansible installed into /opt/ansible-runtime? | 08:55 |
| *** vnogin has quit IRC | 09:00 | |
| openstackgerrit | Georgina Shippey proposed openstack/openstack-ansible-plugins master: Removing name2int module https://review.opendev.org/657761 | 09:02 |
| _janno | jrosser: I mean /usr/local/bin/openstack-ansible which is linked to /usr/local/bin/ansible | 09:06 |
| *** vnogin has joined #openstack-ansible | 09:06 | |
| _janno | Usually we are running ansible from source, which is located in /opt/ansible | 09:09 |
| *** vnogin has quit IRC | 09:11 | |
| jrosser | _janno: openstack-ansible is a wrapper script, and the actual ansible is installed by the bootstrapping process into /opt/ansible-runtime venv | 09:31 |
| jrosser | a lot of care is taken by OSA to use the correct version of ansible for the branch of OSA you are using, and not to collide with anything you have installed on the system already | 09:31 |
| *** BlackDex has joined #openstack-ansible | 09:37 | |
| BlackDex | Hello there, is it possible to use openstack-ansible with rhel7 instead of centos7? | 09:38 |
| *** electrofelix has joined #openstack-ansible | 09:41 | |
| *** persia has quit IRC | 10:10 | |
| *** persia has joined #openstack-ansible | 10:11 | |
| openstackgerrit | Chason Chan proposed openstack/openstack-ansible master: Update index of project guide for Train https://review.opendev.org/657769 | 10:15 |
| *** CraigBright has joined #openstack-ansible | 10:19 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Add support to deploy placement https://review.opendev.org/656891 | 10:22 |
| openstackgerrit | Matthew Carroll proposed openstack/openstack-ansible-plugins master: Remove bit_length_power_of_2 https://review.opendev.org/657773 | 10:27 |
| odyssey4me | o/ | 10:29 |
| CeeMac | hi odyssey4me | 10:29 |
| noonedeadpunk | BlackDex: I think it should work, however this is something not really tested with zuul | 10:32 |
| BlackDex | noonedeadpunk: thx for the reply :) Ill maybe check that out, since TripleO is not working that well | 10:33 |
| noonedeadpunk | hm, that's strange, as it was designed for RHEL iirc | 10:34 |
| BlackDex | true, but for some reason the baremetal stuff isn't detected that well | 10:34 |
| BlackDex | could be because i have ilo/redfish stuff instead of ipmi | 10:35 |
| BlackDex | although ipmi should work | 10:35 |
| noonedeadpunk | Probably some TripleO folk can help you with that? Probably chandankumar may give some piece of advice... | 10:35 |
| odyssey4me | chandankumar o/ it looks like we're failing builds due to test-requirements in plugin repositories trying to use sphinx 2, which requires python 3.5 ... I wonder whether we even need to install test-requirements.txt any more? | 10:36 |
| *** nicolasbock has joined #openstack-ansible | 10:37 | |
| *** vakuznet has quit IRC | 10:37 | |
| chandankumar | odyssey4me: noonedeadpunk hello | 10:41 |
| chandankumar | odyssey4me: do we have the logs? | 10:41 |
| chandankumar | if we are using tox, we need to install both requirements and test-requirements.txt | 10:43 |
| *** priteau has joined #openstack-ansible | 10:45 | |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-lxc_hosts master: Returning of the Suse LXC image https://review.opendev.org/655851 | 10:45 |
| odyssey4me | chandankumar we aren't using tox though - this is to install the plugins to run tests using tempest as usual | 10:46 |
| odyssey4me | chandankumar see http://logs.openstack.org/19/657019/2/check/openstack-ansible-deploy-aio_lxc-ubuntu-bionic/9a98e17/ | 10:46 |
| chandankumar | odyssey4me: for installing tempest plugin we donot requires test-requirements.txt | 10:52 |
| chandankumar | I have migrated all their requirements to requirements.txt | 10:52 |
| chandankumar | that's why tempest plugin split happens | 10:53 |
| odyssey4me | chandankumar yeah, I think we had that implemented during the transition period - so let me remove those tasks... it'll make it all much simpler too :) | 10:54 |
| openstackgerrit | Matthew Carroll proposed openstack/openstack-ansible-plugins master: Remove bit_length_power_of_2 https://review.opendev.org/657773 | 10:54 |
| chandankumar | odyssey4me: proposing the patch | 10:57 |
| *** priteau has quit IRC | 11:03 | |
| *** priteau has joined #openstack-ansible | 11:03 | |
| openstackgerrit | Chandan Kumar (raukadah) proposed openstack/openstack-ansible-os_tempest master: Remove install_test_requirements flag https://review.opendev.org/657778 | 11:13 |
| chandankumar | odyssey4me: noonedeadpunk ^^ | 11:14 |
| *** udesale has quit IRC | 11:14 | |
| *** phasespace has joined #openstack-ansible | 11:15 | |
| *** jsquare has quit IRC | 11:30 | |
| *** jsquare has joined #openstack-ansible | 11:31 | |
| *** ansmith has quit IRC | 11:34 | |
| noonedeadpunk | chandankumar: commented | 11:37 |
| noonedeadpunk | I guess, that "Install tempest plugin requirements" task makes no sense without test-requirements.txt | 11:39 |
| openstackgerrit | Chandan Kumar (raukadah) proposed openstack/openstack-ansible-os_tempest master: Remove install_test_requirements flag https://review.opendev.org/657778 | 11:40 |
| *** radeks_ has joined #openstack-ansible | 11:49 | |
| odyssey4me | chandankumar LGTM, let's see if it passes | 11:55 |
| *** shyamb has joined #openstack-ansible | 12:07 | |
| *** priteau has quit IRC | 12:07 | |
| *** mgariepy has quit IRC | 12:11 | |
| openstackgerrit | Nicolas Bock proposed openstack/openstack-ansible-lxc_hosts stable/stein: Update (yet again) SUSE lxc container name https://review.opendev.org/657782 | 12:11 |
| evrardjp | should we tem disable centos to get this merged? https://review.opendev.org/#/c/656595/1 it blocks release | 12:14 |
| evrardjp | I guess we can also wait for placement stuff to be fixed | 12:16 |
| noonedeadpunk | evrardjp: I think it was disabled here https://review.opendev.org/#/c/657415/ ? | 12:18 |
| evrardjp | yeah but my concern is for stable which has another placement issue | 12:19 |
| noonedeadpunk | Ah, yeah | 12:19 |
| noonedeadpunk | But I'd personally waited for placement to get it in release as well, as it's pretty big and important thing imo | 12:20 |
| evrardjp | yes. | 12:20 |
| evrardjp | I think upgrade checker complaining about the version is a good enough reason | 12:21 |
| evrardjp | :p | 12:21 |
| *** mgariepy has joined #openstack-ansible | 12:21 | |
| odyssey4me | yeah, I think we're wanting to get placement in, and get upgrades working - most of the patches are up already for master | 12:21 |
| odyssey4me | there is a tempest plugin install issue blocking master, which https://review.opendev.org/657778 should solve - waiting for test feedback | 12:21 |
| evrardjp | odyssey4me: the problem is that stable doesn't deploy | 12:22 |
| odyssey4me | evrardjp yeah, I dunno what the issue with centos is for stable/stein - we'll have to wait for mnaser or guilhermesp to comment on that | 12:23 |
| openstackgerrit | Nicolas Bock proposed openstack/openstack-ansible-lxc_container_create master: Update LXC container config options https://review.opendev.org/657785 | 12:28 |
| *** mgariepy has quit IRC | 12:29 | |
| *** mgariepy has joined #openstack-ansible | 12:34 | |
| *** kplant has joined #openstack-ansible | 12:38 | |
| *** ansmith has joined #openstack-ansible | 12:39 | |
| openstackgerrit | Georgina Shippey proposed openstack/openstack-ansible-plugins master: Remove bit_length_power_of_2 https://review.opendev.org/657773 | 12:42 |
| *** CeeMac_ has joined #openstack-ansible | 12:46 | |
| *** CeeMac has quit IRC | 12:49 | |
| *** radeks_ has quit IRC | 12:53 | |
| *** mgariepy has quit IRC | 12:58 | |
| *** shyamb has quit IRC | 13:03 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Use hashes for the upgrade playbook markers https://review.opendev.org/657041 | 13:05 |
| *** samueldmq has joined #openstack-ansible | 13:09 | |
| *** mgariepy has joined #openstack-ansible | 13:09 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Major Upgrade: Remove pypiserver from repo server https://review.opendev.org/657798 | 13:15 |
| odyssey4me | oh bother, it looks like that tempest test failed and that's likely due to depends-on being broken for the integrated build... at least I recall logan- saying something to that effect | 13:29 |
| jrosser | odyssey4me: we saw depends-on failing to have effect too | 13:31 |
| odyssey4me | jrosser yeah, likely due to the folder name change | 13:32 |
| guilhermesp | morning.. aaaaaah still catching up with things here ( jetlag is almost vanishing :P ) | 13:34 |
| guilhermesp | huum seems that placement is still struggling to merge | 13:34 |
| jrosser | in general do we expect newer versions of the openstack cli tool to be compatible with older deployments? | 13:34 |
| jamesdenton | how old? | 13:35 |
| jamesdenton | The worst I've seen is 404s for API paths that don't exist in older releases, but for the most part things have Just Worked™ | 13:36 |
| jrosser | a rocky cloud with whatever "pip install python-openstackclient" gives you on a users machine | 13:36 |
| jamesdenton | If you trust the notes here, it ought to be OK: https://docs.openstack.org/releasenotes/python-openstackclient/ | 13:38 |
| odyssey4me | noonedeadpunk any chance you could rebase https://review.opendev.org/#/c/653970/4 ? | 13:41 |
| jrosser | it seems that pip install python-openstackclient didnt bring in python-magnumclient - installing that manually then gives errors along the lines of the client passing params that the magnum api doesnt like, so not 404, but the request content having new fields which aren't understood | 13:42 |
| kplant | aren't there environment variables to control api versioning? | 13:42 |
| noonedeadpunk | odyssey4me: sure I can if time for it has come:) | 13:43 |
| kplant | OS_*_API_VERSION | 13:43 |
| jrosser | kplant: ahha yes indeed - but i wonder really how the end user is supposed to know this | 13:44 |
| odyssey4me | jrosser the openstack client shouldn't require any of the service clients as far as I know | 13:44 |
| odyssey4me | unless openstacksdk hasn't implemented the required functions yet | 13:45 |
| jrosser | well, the "openstack coe....." commands just arent present unless python-magnumclient is installed it seems | 13:45 |
| jamesdenton | Same for neutron stuff IIRC. it uses them as libraries | 13:45 |
| odyssey4me | noonedeadpunk I think you'll need to rebase https://review.opendev.org/#/c/653978/8 first though | 13:45 |
| noonedeadpunk | openstackclient requires python-masakariclient in order to support masakari cli. It's not actually required be dependencies, but it's just not including segments section. So yeah you're right | 13:47 |
| odyssey4me | jrosser ah yes, ok - then it's the legacy support implementation which is there to cater for things openstacksdk can't do yet | 13:48 |
| odyssey4me | openstackclient only has the base things done by openstacksdk by default - if the commands are missing, you need to add the applicable service client | 13:48 |
| odyssey4me | there are folks slowly working through each client to add their functionality to the openstacksdk | 13:49 |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible master: Replace git.openstack.org with opendev.org https://review.opendev.org/653970 | 13:54 |
| kplant | jrosser: yeah... that's one of those things that could use some documentation love | 13:55 |
| * kplant not it | 13:55 | |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-tests master: Replace git.openstack.org with opendev.org https://review.opendev.org/653978 | 13:59 |
| logan- | odyssey4me: there is a patch in flight that fixes depends on, one sec I’ll find it | 13:59 |
| logan- | odyssey4me: https://review.opendev.org/#/c/656599/ | 14:00 |
| odyssey4me | logan- ah, thanks | 14:02 |
| odyssey4me | I think we may have to disable the bionic test voting temporarily to get that through, then we'll be able to get the tempest one passing and re-enable the bionic one | 14:02 |
| logan- | K | 14:05 |
| odyssey4me | we should probably merge https://review.opendev.org/#/c/656599/5 into https://review.opendev.org/#/c/653970/ given the latter is more thorough | 14:06 |
| *** phasespace has quit IRC | 14:06 | |
| logan- | I’d rather not fiddle with it anymore just to get depends on working again, and then rebase 656599 later | 14:08 |
| noonedeadpunk | evrardjp: regarding your comments on https://review.opendev.org/#/c/653978 - will placing branch for link really ease backporting? I've just supposed, that it might be backported "as-is" with link to master. | 14:09 |
| odyssey4me | yeah, we can do that - we'll have to disable the following jobs temporarily though: openstack-ansible-deploy-aio_lxc-ubuntu-bionic, openstack-ansible-deploy-aio_metal-ubuntu-bionic, openstack-ansible-deploy-aio_ceph-ubuntu-bionic, openstack-ansible-deploy-aio_proxy-ubuntu-bionic - they're all failing to install tempest plugins | 14:09 |
| noonedeadpunk | s/supposed/was worried/ | 14:09 |
| odyssey4me | the fix for that is https://review.opendev.org/657778 but it's failing the bionic job and the tempestconf job | 14:10 |
| odyssey4me | I suppose we could disable just the tempest repo jobs to get that merged, then the integrated repo should work again | 14:11 |
| noonedeadpunk | btw. how to check the correct paths for zuul? like https://opendev.org/openstack/openstack-ansible-tests/src/branch/master/test-ansible-env-prep.sh#L49 | 14:12 |
| logan- | Yeah that seems better | 14:12 |
| odyssey4me | ah, the tempestconf job is failing because it uses nova master, not a pinned nova | 14:12 |
| odyssey4me | ok, we can fix that | 14:12 |
| chandankumar | ok disabling this | 14:17 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Pin nova_git_install_branch to before placement removal https://review.opendev.org/657816 | 14:19 |
| odyssey4me | chandankumar ^ should fix the os_tempest tempestconf test once it merges | 14:20 |
| odyssey4me | it'll also fix any other failing tests that use the old role test mechanism | 14:20 |
| openstackgerrit | Chandan Kumar (raukadah) proposed openstack/openstack-ansible-os_tempest master: Mov openstack-ansible-functional-tempestconf-centos-7 to nv https://review.opendev.org/657818 | 14:21 |
| chandankumar | odyssey4me: so no need to move nv then | 14:21 |
| odyssey4me | noonedeadpunk as far as I know, that's just a simple switch to opendev as you already have it | 14:21 |
| odyssey4me | it looks like ARA is under a new org though: https://opendev.org/recordsansible/ara | 14:22 |
| evrardjp | it is a new org | 14:25 |
| evrardjp | but redirects are still in place iirc | 14:25 |
| evrardjp | dmsimard: has asked us to test ara with python3 so it would be nice if we could help him on the way and install the right version of ara :p | 14:26 |
| odyssey4me | yeah, I think we need to get the opendev changes out the way first - then work on py3 | 14:27 |
| evrardjp | yeah, opendev + placement stability | 14:28 |
| *** vnogin has joined #openstack-ansible | 14:28 | |
| evrardjp | then we can think about the future | 14:28 |
| evrardjp | here I would like to release stein at least once :p | 14:28 |
| evrardjp | not being stuck in rc things | 14:28 |
| dmsimard | die py2 die | 14:32 |
| dmsimard | but yes, although the new API in ara 1.0 runs py3 only, the callback and API clients can arguably be made to run py2 (allowing a py2 node to send data to a remote api running py3 somewhere, for example) | 14:33 |
| openstackgerrit | Matthew Carroll proposed openstack/openstack-ansible-plugins master: Remove unused Magnum module https://review.opendev.org/657825 | 14:34 |
| *** vnogin has quit IRC | 14:35 | |
| openstackgerrit | Logan V proposed openstack/openstack-ansible-os_nova master: Always deploy placement authtoken config https://review.opendev.org/657827 | 14:37 |
| *** vnogin has joined #openstack-ansible | 14:39 | |
| openstackgerrit | Logan V proposed openstack/openstack-ansible master: Add support to deploy placement https://review.opendev.org/656891 | 14:40 |
| *** vnogin has quit IRC | 14:40 | |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-tests master: Replace git.openstack.org with opendev.org https://review.opendev.org/653978 | 14:50 |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible master: Replace git.openstack.org with opendev.org https://review.opendev.org/653970 | 14:52 |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-tests master: Replace git.openstack.org with opendev.org https://review.opendev.org/653978 | 14:58 |
| *** cmart has joined #openstack-ansible | 14:58 | |
| *** cmart has quit IRC | 15:00 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest master: Temporarily set bionic job to non-voting https://review.opendev.org/657833 | 15:04 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest master: Remove install_test_requirements flag https://review.opendev.org/657778 | 15:05 |
| *** starborn has quit IRC | 15:06 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest master: Revert "Temporarily set bionic job to non-voting" https://review.opendev.org/657834 | 15:06 |
| openstackgerrit | Matthew Carroll proposed openstack/openstack-ansible-plugins master: Remove unused Magnum module https://review.opendev.org/657825 | 15:07 |
| *** ivve has quit IRC | 15:07 | |
| openstackgerrit | Matthew Carroll proposed openstack/openstack-ansible-plugins master: Remove unused Magnum module https://review.opendev.org/657825 | 15:09 |
| *** altlogbot_1 has quit IRC | 15:21 | |
| *** altlogbot_1 has joined #openstack-ansible | 15:21 | |
| *** irclogbot_2 has quit IRC | 15:22 | |
| *** irclogbot_2 has joined #openstack-ansible | 15:23 | |
| noonedeadpunk | odyssey4me: does voting: false works for the gate? For some reason I thought that it's not... probably due to https://review.opendev.org/#/c/657415/1..2/zuul.d/project-templates.yaml | 15:25 |
| odyssey4me | noonedeadpunk typically we would remove it from gate - but in https://review.opendev.org/657833 I'm just doing a temporary voting status change, rather than remove that job from all tests | 15:26 |
| *** altlogbot_1 has quit IRC | 15:27 | |
| odyssey4me | I'm out for the eve - will try and keep tabs on that series of patches which should hopefully unblock the gate so that we can get the opendev changes moving and get depends-on working again. | 15:27 |
| *** altlogbot_2 has joined #openstack-ansible | 15:27 | |
| odyssey4me | FYI https://review.opendev.org/#/q/topic:fix-osa-gates | 15:27 |
| *** hamzaachi has quit IRC | 15:27 | |
| noonedeadpunk | just https://review.opendev.org/657415 had the same purpose, so was wondering | 15:28 |
| *** irclogbot_2 has quit IRC | 15:28 | |
| *** irclogbot_0 has joined #openstack-ansible | 15:29 | |
| *** devx has quit IRC | 15:32 | |
| *** devx has joined #openstack-ansible | 15:33 | |
| *** gyee has joined #openstack-ansible | 15:40 | |
| *** dxiri has joined #openstack-ansible | 15:40 | |
| jamesdenton | Does OSA have a module blacklist capability? | 15:48 |
| *** CraigBright has quit IRC | 15:57 | |
| hjohnson1 | Does anyone know of a way to extract the IP address of a container by including the '/opt/openstack-ansible/inventory/dynamic_inventory.py' script in ansible? I need to get all three designate container IP addresses. | 15:59 |
| logan- | jamesdenton: if it exists i expect it would be in openstack_hosts | 16:01 |
| logan- | hjohnson1: take a look at scripts/inventory-manage.py | 16:02 |
| jamesdenton | thanks logan-. I checked, but didn't see anything obvious. | 16:03 |
| logan- | noonedeadpunk: it's not preferred to run non voting jobs in the queue pipeline since they are simply a waste of CI resources. they cannot influence the gate result so there is no point in running them in that pipeline that's why you'd normally never have nv jobs in the gate pipeline. in odyssey4me's patch, it is ok I guess since it is not a long term change. | 16:04 |
| hjohnson1 | ogan-: Thanks... I take a look! | 16:06 |
| jrosser | hjohnson1: i have a thing for static IP with designate containers | 16:07 |
| *** chandankumar is now known as raukadah | 16:08 | |
| jrosser | hjohnson1: this is a bit work-in-progress but all the things you need to do this are merged https://github.com/jrosser/openstack-ansible-ops/tree/designate-bind/designate-bind | 16:08 |
| jrosser | hjohnson1: this is the key bit https://github.com/jrosser/openstack-ansible-ops/blob/designate-bind/designate-bind/dnsaas-bind.yml#L6-L32 | 16:10 |
| jrosser | hjohnson1: if you define container_extra_networks either at the host_vars or group_vars level (with appropriate logic) then this will combine your interface with a fixed IP with the existing container interfaces https://github.com/jrosser/openstack-ansible-ops/blob/designate-bind/designate-bind/dnsaas-bind.yml#L6-L32 | 16:13 |
| jrosser | whoops this https://github.com/openstack/openstack-ansible-lxc_container_create/blob/master/vars/main.yml#L16-L20 | 16:13 |
| noonedeadpunk | evrardjp: btw, can you re-vote please? https://review.opendev.org/#/c/655851/ | 16:14 |
| hjohnson1 | jrosser: I will definitely look at all of that!!!! Thanks! | 16:16 |
| jrosser | hjohnson1: no worries - i created some extra containers and installed bind into them myself, and gave them dedicated "external" interfaces with fixed IP | 16:16 |
| *** DanyC has quit IRC | 16:44 | |
| *** strattao has joined #openstack-ansible | 16:44 | |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-tests master: Replace git.openstack.org with opendev.org https://review.opendev.org/653978 | 16:49 |
| openstackgerrit | Merged openstack/openstack-ansible-tests master: Pin nova_git_install_branch to before placement removal https://review.opendev.org/657816 | 16:51 |
| noonedeadpunk | odyssey4me cloudnull: can you please check one more time ^ ? Sorry just misunderstood evrardjp comment first time | 16:52 |
| cloudnull | looking now | 16:54 |
| *** vakuznet has joined #openstack-ansible | 16:58 | |
| *** DanyC has joined #openstack-ansible | 17:11 | |
| openstackgerrit | Vadim Kuznetsov proposed openstack/openstack-ansible-os_octavia master: Add support for using distribution packages for OpenStack services https://review.opendev.org/652049 | 17:15 |
| *** DanyC has quit IRC | 17:16 | |
| *** jbadiapa has quit IRC | 17:19 | |
| logan- | cloudnull noonedeadpunk: it looks like https://review.opendev.org/#/c/657778 does not fix openstack-ansible-deploy-aio_metal-ubuntu-bionic as intended so we should not merge it yet imo | 17:21 |
| cloudnull | I pulled +w | 17:22 |
| noonedeadpunk | logan-: that's strange - fail on task that does not exist.... | 17:25 |
| logan- | noonedeadpunk: it fails on tasks/tempest_install_source.yml:- name: Install tempest plugin requirements in the tempest role | 17:26 |
| logan- | http://logs.openstack.org/78/657778/3/check/openstack-ansible-deploy-aio_metal-ubuntu-bionic/640b83f/job-output.txt.gz#_2019-05-08_16_06_19_183124 | 17:26 |
| noonedeadpunk | Yep, I see, but https://review.opendev.org/#/c/657778/3/tasks/tempest_install_source.yml | 17:26 |
| noonedeadpunk | this patch removes this particular task | 17:27 |
| logan- | oh | 17:27 |
| logan- | gotcha | 17:27 |
| logan- | interesting | 17:28 |
| logan- | do we re-clone tempest somewhere | 17:28 |
| logan- | indeed we do | 17:29 |
| *** DanyC has joined #openstack-ansible | 17:29 | |
| noonedeadpunk | somewhere like test repo? | 17:29 |
| *** ianychoi_ has joined #openstack-ansible | 17:29 | |
| noonedeadpunk | so yeah, than we need to merge https://review.opendev.org/#/c/657778 to get gate unlocked, like odyssey4me suggested | 17:30 |
| logan- | yep, agreed, https://review.opendev.org/#/c/656599/ unmerged is why tempest master is used instead of the zuul one | 17:31 |
| logan- | thanks | 17:31 |
| logan- | we can test that in the nv revert patch though | 17:31 |
| *** ansmith has quit IRC | 17:31 | |
| *** ianychoi has quit IRC | 17:31 | |
| noonedeadpunk | +1 | 17:31 |
| openstackgerrit | Logan V proposed openstack/openstack-ansible-os_tempest master: Revert "Temporarily set bionic job to non-voting" https://review.opendev.org/657834 | 17:32 |
| logan- | what a mess of blockers lol | 17:32 |
| noonedeadpunk | logan-: btw, I have https://review.opendev.org/#/c/653970/ as alternative to https://review.opendev.org/#/c/656599/ :) | 17:33 |
| *** jbadiapa has joined #openstack-ansible | 17:33 | |
| *** DanyC has quit IRC | 17:33 | |
| noonedeadpunk | but I'd better rebase on top of https://review.opendev.org/#/c/656599/ | 17:34 |
| logan- | yeah i think that would be preferable since 656599 at least gives us a bare minimum to get depends-on working again, which will unblock a bunch of things. thanks for 653970 though, i'll review it later once we get further towards a working gate ;) | 17:36 |
| noonedeadpunk | and 656599 looks more backportable | 17:38 |
| *** Weifan has joined #openstack-ansible | 17:38 | |
| logan- | ++ yep | 17:38 |
| *** vnogin has joined #openstack-ansible | 17:44 | |
| noonedeadpunk | logan-: I've lost in these dependecies as well... but how https://review.opendev.org/#/c/657778/ will fix https://review.opendev.org/#/c/657834 - as they are not related..? | 17:47 |
| *** strattao has quit IRC | 17:47 | |
| logan- | https://review.opendev.org/#/c/657778/ is meant to fix the openstack-ansible-deploy-aio_metal-ubuntu-bionic job | 17:48 |
| logan- | we set that job non voting to merge 657778 | 17:48 |
| logan- | once that merges, https://review.opendev.org/#/c/657834 will set the job back to voting | 17:48 |
| logan- | ideally we want to see a pass on https://review.opendev.org/#/c/657834 before merging https://review.opendev.org/#/c/657778/ | 17:50 |
| noonedeadpunk | Yep, just you wrote, that we're waiting 657834 to pass in order to merge 657778 | 17:50 |
| noonedeadpunk | Just don't understand the way this might happen:( | 17:50 |
| *** KurtB has joined #openstack-ansible | 17:51 | |
| logan- | the parent commit for 657834 is the sha in 657778 | 17:51 |
| logan- | so they are dependent patches | 17:51 |
| noonedeadpunk | sorry for stupid questions ,just trying to dig into details of processes | 17:51 |
| logan- | yep i understand :) | 17:51 |
| noonedeadpunk | ohhhh | 17:51 |
| noonedeadpunk | now I see | 17:51 |
| noonedeadpunk | ok, thanks:) | 17:52 |
| logan- | yep it is similar to depends-on except you can also do it this way if both patches are in the same repo | 17:52 |
| logan- | and it tends to imply a stronger dependency chain | 17:52 |
| noonedeadpunk | yeah, I just missed that they are for the same repo | 17:53 |
| noonedeadpunk | get used to integrated tests:) | 17:53 |
| noonedeadpunk | and that you've rebased revert | 17:54 |
| *** ivve has joined #openstack-ansible | 17:54 | |
| e-vad | howdy, for stein, is there going to be a 19.0.0 tag in addition to the 'stable/stein' branch? | 17:54 |
| noonedeadpunk | e-vad: for sure there will be such tag | 17:55 |
| e-vad | excellent, i noticed the upgrade doc states to check out the 19.0.0 tag, but it didn't actually exist :-| | 17:56 |
| *** mgariepy has quit IRC | 17:57 | |
| *** mgariepy has joined #openstack-ansible | 17:58 | |
| *** hartripley has joined #openstack-ansible | 18:03 | |
| *** ansmith has joined #openstack-ansible | 18:07 | |
| *** djhankb has quit IRC | 18:08 | |
| *** hartripley has quit IRC | 18:10 | |
| *** strattao has joined #openstack-ansible | 18:15 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_tempest master: Temporarily set bionic job to non-voting https://review.opendev.org/657833 | 18:24 |
| logan- | noonedeadpunk: looks like its going to pass http://zuul.openstack.org/stream/4f00064075634edaa671730b500474cb?logfile=console.log | 18:25 |
| *** vnogin has quit IRC | 18:25 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_tempest master: Remove install_test_requirements flag https://review.opendev.org/657778 | 18:30 |
| *** Weifan has quit IRC | 18:32 | |
| noonedeadpunk | Yep, it's passed. Great news | 18:32 |
| logan- | now to get https://review.opendev.org/#/c/656599/ merged | 18:33 |
| noonedeadpunk | it was merged fast, isn't it?:) | 18:33 |
| logan- | hah, i quickly +3 it since it was already in the gate when I saw the dependent patch was going to pass | 18:34 |
| logan- | otherwise it would have had to go thru check and gate again | 18:34 |
| logan- | once you +W and it enters the gate queue, the job doesn't de-queue if you remove -W, but once it finishes in the gate queue if there is no +W left on the patch, you have to recheck it back thru the check and gate again | 18:35 |
| noonedeadpunk | ok, I see | 18:36 |
| noonedeadpunk | thanks, logan- | 18:36 |
| *** cmart has joined #openstack-ansible | 18:55 | |
| *** Weifan has joined #openstack-ansible | 19:08 | |
| *** Weifan has quit IRC | 19:12 | |
| odyssey4me | o/ | 19:16 |
| logan- | o/ | 19:17 |
| odyssey4me | I've put a -w on https://review.opendev.org/653978 until https://review.opendev.org/656599 merges, otherwise we'll likely find ourselves with a flurry of patches for not much value. | 19:21 |
| logan- | thanks | 19:21 |
| odyssey4me | logan- boo: http://logs.openstack.org/99/656599/5/check/openstack-ansible-deploy-aio_ceph-ubuntu-bionic/adc12fc/job-output.txt.gz#_2019-05-08_19_22_45_146046 | 19:25 |
| openstackgerrit | Logan V proposed openstack/openstack-ansible master: Add Calico networking AIO scenario https://review.opendev.org/645831 | 19:26 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Use opendev links https://review.opendev.org/656599 | 19:26 |
| logan- | argh | 19:26 |
| odyssey4me | ^ rebased to kick off testing gain | 19:26 |
| logan- | thx | 19:27 |
| odyssey4me | no sense in waiting for the others to finish | 19:27 |
| logan- | yep | 19:27 |
| *** kmadac has quit IRC | 19:27 | |
| *** strattao has quit IRC | 19:33 | |
| *** vnogin has joined #openstack-ansible | 19:50 | |
| dirk | evrardjp: can we maybe make the metal_aio jobs test whether the information in meta/vars is matching the current OS? That would simplify things a bit | 19:52 |
| *** vnogin has quit IRC | 19:55 | |
| odyssey4me | dirk not sure what you mean, but the metadata is not available at runtime to a playbook... it's actually only useful for ansible galaxy | 19:57 |
| openstackgerrit | Merged openstack/openstack-ansible-plugins master: Remove unused Magnum module https://review.opendev.org/657825 | 19:57 |
| odyssey4me | dirk it'd be useful to know what you're thinking though - maybe there's another way to do it | 19:57 |
| odyssey4me | e-vad upgrades to stein don't work just yet - there are some patches in flight to get that fixed | 19:59 |
| e-vad | odyssey4me: thanks! | 20:01 |
| *** strattao has joined #openstack-ansible | 20:04 | |
| *** hamzy_ has joined #openstack-ansible | 20:12 | |
| *** mgoddard has quit IRC | 20:13 | |
| *** crazzy has quit IRC | 20:13 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Add support to deploy placement https://review.opendev.org/656891 | 20:14 |
| *** prometheanfire has quit IRC | 20:14 | |
| *** niceplace has quit IRC | 20:14 | |
| *** yolanda_ has quit IRC | 20:15 | |
| *** redrobot has quit IRC | 20:15 | |
| *** vakuznet has quit IRC | 20:15 | |
| *** hamzy has quit IRC | 20:15 | |
| *** mwe1 has quit IRC | 20:15 | |
| *** evrardjp has quit IRC | 20:15 | |
| *** evrardjp has joined #openstack-ansible | 20:16 | |
| *** prometheanfire has joined #openstack-ansible | 20:16 | |
| *** strattao has quit IRC | 20:18 | |
| openstackgerrit | Merged openstack/openstack-ansible-openstack_hosts stable/stein: Switch to Stein packages for openSUSE https://review.opendev.org/655715 | 20:18 |
| *** crazzy has joined #openstack-ansible | 20:20 | |
| *** Weifan has joined #openstack-ansible | 20:20 | |
| odyssey4me | logan- with https://review.opendev.org/657816 merged, do we need the hacky workaround mentioned in https://review.opendev.org/656891 comments? | 20:21 |
| logan- | no - https://review.opendev.org/#/c/657827/ | 20:21 |
| logan- | i already removed it form 656891 | 20:21 |
| *** mgoddard has joined #openstack-ansible | 20:22 | |
| *** niceplace has joined #openstack-ansible | 20:22 | |
| logan- | however it looks like you removed the depends-on fix from the depends-on list in 656891 | 20:22 |
| logan- | so it won't pull in that os_nova change now | 20:22 |
| *** Guest69998 has joined #openstack-ansible | 20:22 | |
| logan- | oh wait | 20:23 |
| logan- | the os_nova fix never got added | 20:23 |
| odyssey4me | did I get the wrong one? I meant to remove the depends-on for https://review.opendev.org/#/c/656599/ because it's not necessary | 20:23 |
| logan- | ah | 20:23 |
| logan- | yeah i think you grabbed the wrong commit id | 20:23 |
| *** Guest69998 is now known as redrobot | 20:23 | |
| logan- | no, you didn't actually | 20:23 |
| logan- | but that patch is required | 20:23 |
| logan- | because we also need to depends-on https://review.opendev.org/#/c/657827/ | 20:23 |
| odyssey4me | ok, will add it | 20:24 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Add support to deploy placement https://review.opendev.org/656891 | 20:25 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Add support to deploy placement https://review.opendev.org/656891 | 20:25 |
| odyssey4me | added the os_nova dependency, and rebased on the opendev patch given it's in the same repo | 20:25 |
| logan- | a question about once we start cleaning up placement from os_nova -- should we continue using the placement service user to obtain an authtoken when talkign to placement, or should we use the nova service user | 20:25 |
| logan- | ty | 20:25 |
| odyssey4me | logan- good question, perhaps best asked of the placement/nova crew... I imagine we could continue to use the nova placement user, or consolidate... I'll dig into the docs tomorrow when I work on the upgrade bits | 20:27 |
| logan- | i looked at their placement removal patch in nova, and they left the [placement] section in place so you can (and must) set keystone auth credentials to reach placement. in nova.conf we have another example (ironic) where we use the ironic service user | 20:28 |
| logan- | so i guess our precedent is to use the other service's credentials | 20:28 |
| *** yolanda_ has joined #openstack-ansible | 20:28 | |
| logan- | seems a little strange though | 20:28 |
| odyssey4me | oh, that does sound a bit odd | 20:29 |
| *** mwe1 has joined #openstack-ansible | 20:29 | |
| odyssey4me | maybe we should understand the best practise and switch everything to whatever that is | 20:30 |
| logan- | yeah | 20:30 |
| guilhermesp | I think the idea of keep the placement user is not to serve only nova, but cinder as well? | 20:32 |
| guilhermesp | at least what I understood, placement was isolated to serve not only nova, am i right? | 20:33 |
| logan- | guilhermesp: I would think you'd use the nova and cinder users since they are API clients of placement | 20:34 |
| odyssey4me | perhaps that is right, but we'd have to assign roles or something | 20:34 |
| logan- | i think we already assign the service role to all service users | 20:35 |
| odyssey4me | good point | 20:35 |
| guilhermesp | this does makes sense logan- . But at this point, this would be just a option: have or not have a placement user for placement role. Maybe asking to placement crew could be a way t o decide which path would be the best one to follow | 20:36 |
| logan- | placement should have its own user since it will need to obtain an authtoken to register its service | 20:37 |
| logan- | nova should not need it | 20:37 |
| *** ansmith has quit IRC | 20:44 | |
| *** hjohnson1 has quit IRC | 20:45 | |
| *** cmart has quit IRC | 20:45 | |
| *** radeks has quit IRC | 20:59 | |
| *** strattao has joined #openstack-ansible | 20:59 | |
| NobodyCam | Good AfterNoon OSA folks, If I wanted to update DNS_DOMAIN on a running cloud what playbooks would I run after updating user_variables.yml? | 21:04 |
| openstackgerrit | Dirk Mueller proposed openstack/openstack-ansible-os_horizon master: Add aio_metal jobs https://review.opendev.org/657540 | 21:08 |
| *** strattao has quit IRC | 21:19 | |
| *** nicolasbock has quit IRC | 21:20 | |
| *** strattao has joined #openstack-ansible | 21:32 | |
| *** partlycloudy has quit IRC | 21:45 | |
| dirk | odyssey4me: there was a comment from evrardjp on one of my reviews that meta/* needs to be updated for thew tests, but the distro was already whitelisted - I was wondering if there is a way to ensrue that when a job is run for distro x in version y, that this combination is listed as elligible in meta/* | 21:51 |
| dirk | it could totally be an additional step from my point of view as long as it is run in the same job | 21:51 |
| *** kplant has quit IRC | 21:56 | |
| openstackgerrit | Logan V proposed openstack/openstack-ansible master: Add Calico networking AIO scenario https://review.opendev.org/645831 | 22:03 |
| openstackgerrit | Merged openstack/openstack-ansible-os_nova master: Always deploy placement authtoken config https://review.opendev.org/657827 | 22:04 |
| *** SmearedBeard has joined #openstack-ansible | 22:05 | |
| *** hamzaachi has joined #openstack-ansible | 22:11 | |
| *** partlycloudy has joined #openstack-ansible | 22:18 | |
| *** strattao has quit IRC | 22:18 | |
| *** KurtB has quit IRC | 22:27 | |
| *** hamzaachi has quit IRC | 22:31 | |
| *** kplant has joined #openstack-ansible | 22:39 | |
| logan- | https://review.opendev.org/#/c/656891/ ready for reviews | 23:00 |
| mnaser | logan-: sweet. | 23:11 |
| *** SmearedBeard has quit IRC | 23:38 | |
| *** timburke has quit IRC | 23:40 | |
| *** hwoarang has quit IRC | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!