*** xdfil has joined #openstack-ansible | 00:33 | |
xdfil | cloudnull: I had a couple other thoughts about macvlan | 00:34 |
---|---|---|
xdfil | Ideally if using macvlan one would want vepa mode for performance/security reasons | 00:35 |
xdfil | but guest/host communication then requires some external solution | 00:36 |
xdfil | guest/host is going to be needed for many reasons. So that probably means bridge/passthrough mode will be used. | 00:37 |
xdfil | I've been thinking of the best way to avoid those modes... currently leaning towards OVS | 00:39 |
xdfil | this is where it would give me peace of mind knowing I have the option of giving the nspawn container a "whole" device vs a device on which it should create a macvlan. | 00:42 |
xdfil | and control over the macvlan mode | 00:43 |
*** gregoryo has joined #openstack-ansible | 01:12 | |
*** pmannidi has joined #openstack-ansible | 01:18 | |
*** pmannidi has quit IRC | 02:54 | |
*** threestrands has quit IRC | 02:57 | |
*** johnny101 has quit IRC | 03:53 | |
*** threestrands has joined #openstack-ansible | 04:00 | |
*** udesale has joined #openstack-ansible | 04:02 | |
*** udesale has quit IRC | 04:06 | |
*** udesale has joined #openstack-ansible | 04:06 | |
*** udesale has quit IRC | 04:06 | |
*** udesale has joined #openstack-ansible | 04:08 | |
*** vnogin has joined #openstack-ansible | 04:32 | |
*** vnogin has quit IRC | 04:37 | |
*** fghaas has joined #openstack-ansible | 04:44 | |
*** fghaas has quit IRC | 04:49 | |
openstackgerrit | lei zhang proposed openstack/openstack-ansible master: Update the title of Admin guide https://review.openstack.org/599286 | 04:57 |
*** shyamb has joined #openstack-ansible | 05:07 | |
*** shyamb has quit IRC | 05:17 | |
*** hamzy has quit IRC | 05:32 | |
*** shyamb has joined #openstack-ansible | 05:32 | |
*** hamzy has joined #openstack-ansible | 05:32 | |
*** DanyC has joined #openstack-ansible | 05:50 | |
*** chkumar|pto is now known as chandankumar | 06:05 | |
*** johnny101 has joined #openstack-ansible | 06:05 | |
*** luksky has joined #openstack-ansible | 06:13 | |
*** DanyC has quit IRC | 06:18 | |
*** rgogunskiy has joined #openstack-ansible | 06:23 | |
*** johnny101 has quit IRC | 06:27 | |
*** _nyloc_ has quit IRC | 06:29 | |
*** nyloc has joined #openstack-ansible | 06:33 | |
*** fghaas has joined #openstack-ansible | 06:35 | |
*** threestrands has quit IRC | 06:48 | |
*** cshen has joined #openstack-ansible | 06:50 | |
hwoarang | good morning | 07:00 |
hwoarang | would anyone have the time to review https://review.openstack.org/#/c/598978/ which we need for opnfv? thank you | 07:00 |
*** Emine has joined #openstack-ansible | 07:01 | |
*** DanyC has joined #openstack-ansible | 07:04 | |
*** shyamb has quit IRC | 07:06 | |
*** shardy has joined #openstack-ansible | 07:09 | |
*** hamzaachi has joined #openstack-ansible | 07:11 | |
*** DanyC has quit IRC | 07:22 | |
*** olivierb_ has joined #openstack-ansible | 07:25 | |
*** tosky has joined #openstack-ansible | 07:36 | |
*** mardim has quit IRC | 07:52 | |
openstackgerrit | Merged openstack/openstack-ansible-os_aodh master: use include_tasks instead of include https://review.openstack.org/585271 | 07:57 |
openstackgerrit | Merged openstack/openstack-ansible-os_horizon master: switch documentation job to new PTI https://review.openstack.org/594663 | 08:03 |
*** DanyC has joined #openstack-ansible | 08:10 | |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible master: [DNM] - Testing OBS master https://review.openstack.org/507451 | 08:22 |
*** flaviosr has quit IRC | 08:23 | |
evrardjp | hwoarang: maybe worth adapting the clouds.yaml too | 08:27 |
*** vnogin has joined #openstack-ansible | 08:33 | |
hwoarang | evrardjp: for openrc you mean? | 08:33 |
evrardjp | hwoarang: yeah a similar thing to the openrc in clouds.yaml -- in a different commit :) | 08:34 |
evrardjp | no urgency nor mandatory thing | 08:34 |
evrardjp | hwoarang: why don't you use clouds.yaml more? | 08:34 |
evrardjp | not sure what's the context in opnfv | 08:34 |
hwoarang | the tools seem to be using openrc | 08:34 |
hwoarang | dont know why | 08:35 |
*** vnogin has quit IRC | 08:37 | |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-openstack_openrc master: templates: openrc: Add missing 'insecure' and 'verify' options https://review.openstack.org/598978 | 08:37 |
*** shyamb has joined #openstack-ansible | 08:37 | |
*** rgogunskiy has quit IRC | 08:41 | |
*** rgogunskiy has joined #openstack-ansible | 08:42 | |
*** gregoryo has quit IRC | 08:43 | |
*** DanyC has quit IRC | 08:49 | |
odyssey4me | hwoarang: asked a question in https://review.openstack.org/#/c/598978/3/templates/openrc.j2 | 08:55 |
hwoarang | odyssey4me: i was told it does not have to have a value. you should export it empty which is interpreted as false | 08:56 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests stable/rocky: Update ansible to 2.5.8 https://review.openstack.org/599266 | 08:56 |
odyssey4me | hwoarang: oh, ok - interesting | 08:56 |
hwoarang | yeah not sure how this thing works internally to be honest. maybe 0 achieves the same thing | 08:58 |
hwoarang | or maybe 'any value' == True | 08:58 |
hwoarang | who knows | 08:58 |
*** DanyC has joined #openstack-ansible | 08:59 | |
odyssey4me | hwoarang: according to http://codesearch.openstack.org/?q=OS_VERIFY&i=nope&files=&repos= though, OS_VERIFY doesn't exist in any openstack git repositories | 09:00 |
hwoarang | yep but it not like that :) | 09:00 |
hwoarang | https://github.com/openstack/openstacksdk/blob/master/openstack/config/cloud_region.py#L180 | 09:00 |
hwoarang | there are 3 options there, insecure, verify and cacert. they have their matching env variables in openrc as OS_INSECURE, OS_CACERT and OS_VERIFY | 09:01 |
hwoarang | so probalby nobody cares about openrc and OS_VERIFY, they either use OS_INSECURE or clouds.yaml | 09:01 |
hwoarang | my guess... | 09:01 |
odyssey4me | fair enough | 09:02 |
hwoarang | according to code, OS_INSECURE should be enough, but for us we also need to export OS_VERIFY= and i am chasing people around to find out what is going on | 09:02 |
*** noonedeadpunk has joined #openstack-ansible | 09:05 | |
*** cshen has quit IRC | 09:06 | |
odyssey4me | hwoarang: ok, fair enough - you have my +2 | 09:09 |
*** pcaruana has joined #openstack-ansible | 09:09 | |
hwoarang | goodie ! | 09:09 |
odyssey4me | evrardjp hwoarang if you have a bit of time for reviews, it'd be nice to have them: https://goo.gl/k9cCfF | 09:09 |
hwoarang | thanks! | 09:10 |
odyssey4me | I did some work on the w/end, and finally found a winning formula for the build simplification | 09:10 |
*** cshen has joined #openstack-ansible | 09:10 | |
evrardjp | odyssey4me: oh? | 09:11 |
evrardjp | you mean in those reviews? | 09:11 |
odyssey4me | well, most of those reviews are converting the roles to use the python_venv_build role | 09:11 |
odyssey4me | the venv_build role was adjusted to be much, much simpler | 09:12 |
evrardjp | I like this :) | 09:12 |
evrardjp | will check the changes on venv_build | 09:12 |
odyssey4me | primarily https://review.openstack.org/598957 | 09:12 |
odyssey4me | but yeah, in https://review.openstack.org/#/q/project:openstack/ansible-role-python_venv_build+branch:master you'll see I had to make a few more adjustments | 09:12 |
noonedeadpunk | morning all | 09:14 |
noonedeadpunk | evrardjp: I've improved this patch a bit https://review.openstack.org/#/c/593092/ | 09:16 |
noonedeadpunk | so your comments are appriciated) | 09:18 |
noonedeadpunk | I think I have to write release notes for this change, as one var from defaults was removed | 09:19 |
hwoarang | oh many patches | 09:22 |
openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_horizon master: Re-written dashboards linking. https://review.openstack.org/593092 | 09:23 |
*** electrofelix has joined #openstack-ansible | 09:33 | |
*** Taseer has joined #openstack-ansible | 09:42 | |
*** shyamb has quit IRC | 09:43 | |
openstackgerrit | caoyuan proposed openstack/openstack-ansible-os_gnocchi master: use include_tasks instead of include https://review.openstack.org/593446 | 09:48 |
*** jbadiapa has quit IRC | 09:49 | |
*** luksky has quit IRC | 09:51 | |
*** shyamb has joined #openstack-ansible | 09:52 | |
evrardjp | odyssey4me: Yes I am browsing those -- quite a series indeed | 09:52 |
*** DanyC has quit IRC | 09:53 | |
odyssey4me | evrardjp: :) a step forward | 09:53 |
*** rgogunskiy has quit IRC | 09:56 | |
evrardjp | odyssey4me: what is _venv_create_extra_options intended for? | 10:00 |
odyssey4me | evrardjp: it sets some CLI parameters, depending on the version of virtualenv: https://github.com/openstack/ansible-role-python_venv_build/blob/master/tasks/python_venv_preflight.yml#L43-L47 | 10:01 |
*** jbadiapa has joined #openstack-ansible | 10:01 | |
odyssey4me | --never-download / --no-download, depending on the version; and <1.7.0 you have to give it --no-site-packages, whereas later versions do that by default | 10:02 |
evrardjp | oh yeah I missed that | 10:02 |
evrardjp | ok | 10:02 |
evrardjp | makes sense | 10:02 |
*** shardy has quit IRC | 10:03 | |
evrardjp | so the pip install packages will hold both the required packages (should there be some) and what to install at the same time. I can't remember if there are still roles that have so pip requirements packages inside venvs... probably none | 10:03 |
evrardjp | it looks generally nice | 10:03 |
evrardjp | odyssey4me: my concern is that if there are some, the wheel build seem to happen BEFORE installing distro packages for venv build | 10:04 |
odyssey4me | evrardjp: um, where are you seeing that? and I don't understand a word of that first sentence :p | 10:05 |
evrardjp | ok, so... | 10:06 |
odyssey4me | oh... do you mean that the mechanism changes to install the pip packages all at once, rather than in two steps | 10:06 |
evrardjp | https://github.com/openstack/ansible-role-python_venv_build/blob/master/tasks/main.yml#L20 happens before https://github.com/openstack/ansible-role-python_venv_build/blob/master/tasks/main.yml#L25 | 10:06 |
odyssey4me | if so, yes | 10:06 |
odyssey4me | yes, but the wheel build installs all the distro packages | 10:06 |
evrardjp | then on the build host (wheel build), you are building all the packages | 10:06 |
evrardjp | ok | 10:07 |
odyssey4me | https://github.com/openstack/ansible-role-python_venv_build/blob/master/tasks/python_venv_wheel_build.yml#L24 all the distro packages are installed | 10:07 |
evrardjp | yeah, so it's fine for when the consumer of the wheel is the same distro as the builder | 10:08 |
evrardjp | else versions of c bindings might conflict | 10:08 |
evrardjp | which can also be a problem if different apt repos for example | 10:08 |
evrardjp | assuming they all ran in openstack_hosts it's just fine | 10:08 |
evrardjp | and openstack_hosts have been properly set up for those apt mirrors | 10:08 |
odyssey4me | well yes, mixed installs won't work with this as it stands - but that's something that'll work later once we can ditch pip.conf on the hosts | 10:08 |
evrardjp | I mean it's not really a concern, it's something we might want to remember. | 10:09 |
evrardjp | https://github.com/openstack/ansible-role-python_venv_build/blob/master/defaults/main.yml#L88 is good enough for cross distro | 10:10 |
evrardjp | my concern is hidden inconsistencies between build host/consumer host 's repositories leading to different packages versions | 10:10 |
odyssey4me | sure, except that we do all our repository changes that matter in two roles - openstack_hosts (done on every host), and galera_client (done on almost every host) | 10:11 |
*** flaviosr has joined #openstack-ansible | 10:12 | |
odyssey4me | in the end, it may make sense for us to ditch wheel building altogether, along with the repo container and move those into a user story for creating a local mirror/cache | 10:14 |
evrardjp | odyssey4me: for your comment on https://review.openstack.org/#/c/557042/2/tasks/ceilometer_install_source.yml L16 -- We could even think of passing an extra requirements instead of building that developer constraints thing: Developers are modifying code so I guess we can probably do a -e . | 10:14 |
evrardjp | in pip install | 10:14 |
odyssey4me | but for now I'm trying to keep it to at least a similar set of functions | 10:14 |
odyssey4me | evrardjp: perhaps, that's a problem I'd like to solve later rather than now - there are many options | 10:15 |
odyssey4me | the nice thing about using a constraints file, is that the requirements list stays constant - no need to do things like replace 'cinder' with '/path/to/src/cinder' | 10:16 |
evrardjp | but we are at the end installing cinder... so it's just to same to say pip install git+https://<cinder location> or pip install git+file:///<cinder location> | 10:18 |
odyssey4me | instead the constraints file just has 'file:///path/to/src/cinder#egg=cinder' | 10:18 |
odyssey4me | evrardjp: yes, that is true for a git source - but not for a pip source | 10:19 |
*** shyamb has quit IRC | 10:20 | |
evrardjp | I am just talking about developer mode here | 10:20 |
odyssey4me | we're always installing cinder - so if we pip install cinder with a constraint of cinder==x; then it comes from pip - but if the constraint is git+https://... then it comes from git - but if the constraint is file:/// then it comes from a folder | 10:20 |
*** fghaas has quit IRC | 10:20 | |
odyssey4me | using the constraint means that the package list never changes, only the constraint does depending on the source | 10:21 |
evrardjp | understood | 10:21 |
odyssey4me | I *think* that'll be the simplest way to handle the variation with minimal code. The only thing that sucks is that constraints have to be in a file. | 10:22 |
evrardjp | could you check if my comment on https://review.openstack.org/#/c/557042/2/tasks/ceilometer_install_source.yml makes sense? | 10:22 |
odyssey4me | that's the default, so why would we do that? | 10:23 |
*** rgogunskiy has joined #openstack-ansible | 10:23 | |
evrardjp | odyssey4me: is it? https://github.com/openstack/ansible-role-python_venv_build/blob/master/defaults/main.yml#L71 | 10:26 |
odyssey4me | evrardjp: well, the question of why is important - what would the purpose be of adding that task? | 10:27 |
*** DanyC has joined #openstack-ansible | 10:28 | |
evrardjp | odyssey4me: if someone wants to build, let's say tempest on his AIO (that has a repo server), using his developer_mode by passing its own information of developer constraints, the file of developer constraints would be built, but it would get not used, as the venv_build_host will not be the same node as where the constraint is dropped | 10:31 |
evrardjp | as only the role knows the developer mode | 10:31 |
evrardjp | odyssey4me: is that correct? | 10:31 |
*** shyamb has joined #openstack-ansible | 10:32 | |
*** luksky has joined #openstack-ansible | 10:34 | |
odyssey4me | evrardjp: Yeah, I just didn't care enough about developer_mode to really think it through. I figured it'd be gone soon, but you make a good point. OK, I'll add a set_fact task there. | 10:36 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ceilometer master: Use a common python build/install role https://review.openstack.org/557042 | 10:40 |
odyssey4me | evrardjp: ^ does that task description make sense? | 10:41 |
*** Taseer has quit IRC | 10:47 | |
*** udesale has quit IRC | 10:56 | |
evrardjp | odyssey4me: yes | 10:56 |
evrardjp | odyssey4me: the name of the task is enough :) | 10:56 |
odyssey4me | ok, awesome, thanks - any other feedback before I work through them all again? | 10:56 |
*** fghaas has joined #openstack-ansible | 10:58 | |
*** Pradiprwt has joined #openstack-ansible | 10:58 | |
*** shyamb has quit IRC | 11:02 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_neutron master: Use a common python build/install role https://review.openstack.org/599238 | 11:04 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Use a common python build/install role https://review.openstack.org/599244 | 11:05 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_barbican master: Use a common python build/install role https://review.openstack.org/557041 | 11:05 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_aodh master: Use a common python build/install role https://review.openstack.org/557039 | 11:06 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_blazar master: Use a common python build/install role https://review.openstack.org/599189 | 11:06 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_swift master: Use a common python build/install role https://review.openstack.org/599256 | 11:07 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_rally master: Use a common python build/install role https://review.openstack.org/599246 | 11:07 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_horizon master: Use a common python build/install role https://review.openstack.org/557053 | 11:08 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_nova master: Use a common python build/install role https://review.openstack.org/599240 | 11:08 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_searchlight master: Use a common python build/install role https://review.openstack.org/599255 | 11:08 |
*** lkoranda has quit IRC | 11:09 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_sahara master: Use a common python build/install role https://review.openstack.org/599247 | 11:09 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_panko master: Use a common python build/install role https://review.openstack.org/599245 | 11:09 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_heat master: Use a common python build/install role https://review.openstack.org/557052 | 11:10 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_designate master: Use a common python build/install role https://review.openstack.org/557049 | 11:10 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: Use a common python build/install role https://review.openstack.org/557043 | 11:11 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_keystone master: Use a common python build/install role https://review.openstack.org/557059 | 11:11 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_glance master: Use a common python build/install role https://review.openstack.org/567692 | 11:12 |
*** lkoranda has joined #openstack-ansible | 11:12 | |
*** vnogin has joined #openstack-ansible | 11:16 | |
evrardjp | odyssey4me: none for now :) | 11:17 |
kaiokmo | mornings | 11:17 |
odyssey4me | evrardjp: of course with those changes, setting developer_mode for any one of the services will make all wheel builds be skipped... at least until I rework developer_mode | 11:18 |
evrardjp | odyssey4me: yeah. But I think it's fine | 11:18 |
evrardjp | for the use case | 11:18 |
evrardjp | kaiokmo: morning :) | 11:19 |
Pradiprwt | How to disable https completely? "openstack_service_publicuri_proto: http" and "haproxy_ssl: false" is not working properly.. It is giving forbiden error after horizon login | 11:20 |
*** hamzaachi has quit IRC | 11:27 | |
*** shyamb has joined #openstack-ansible | 11:34 | |
*** vnogin has quit IRC | 11:40 | |
*** rgogunskiy has quit IRC | 11:40 | |
evrardjp | odyssey4me: you -w https://review.openstack.org/#/c/599240/3 ? | 11:46 |
odyssey4me | evrardjp: yeah, it fails the integrated build - one of the packages being installed doesn't work when installing into a container, so I need to figure out what to do about it | 11:46 |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-openstack_hosts master: Revert "Revert "SUSE: Switch to OBS Cloud Master repository"" https://review.openstack.org/598333 | 11:50 |
guilhermesp | morning! | 11:54 |
openstackgerrit | Merged openstack/openstack-ansible master: Remove molteniron playbook mistakenly left behind https://review.openstack.org/599237 | 11:54 |
kaiokmo | morning guilhermesp | 11:54 |
*** shyamb has quit IRC | 11:55 | |
guilhermesp | mnaser: odyssey4me sorry I was the entire Sunday afk... looking your cc's | 11:55 |
evrardjp | odyssey4me: had a look at all the patches, they all look very similar, no typos seen, and the only few remaining ones with no set_fact are -1 from me, so you can track them easily | 11:55 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Fix issues with infrastructure healthcheck https://review.openstack.org/599360 | 11:57 |
odyssey4me | guilhermesp: no worries at all, sunday is a perfectly good day to rest and have fun with your family :) | 11:57 |
odyssey4me | evrardjp: heh, I guess I missed some - thanks... will adjust those now | 11:58 |
openstackgerrit | Merged openstack/openstack-ansible-os_blazar master: Use a common python build/install role https://review.openstack.org/599189 | 11:58 |
evrardjp | morning guilhermesp :) | 11:58 |
guilhermesp | odyssey4me evrardjp \m/ | 11:59 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_congress master: Use a common python build/install role https://review.openstack.org/557047 | 11:59 |
guilhermesp | ah I really need another cup of coffe... | 11:59 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cloudkitty master: Use a common python build/install role https://review.openstack.org/557046 | 11:59 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_gnocchi master: Use a common python build/install role https://review.openstack.org/557050 | 11:59 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_molteniron master: Use a common python build/install role https://review.openstack.org/557063 | 12:00 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_masakari master: Use a common python build/install role https://review.openstack.org/599197 | 12:00 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_monasca-agent master: Use a common python build/install role https://review.openstack.org/599199 | 12:01 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_monasca master: Use a common python build/install role https://review.openstack.org/599198 | 12:01 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ironic master: Use a common python build/install role https://review.openstack.org/557055 | 12:01 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_magnum master: Use a common python build/install role https://review.openstack.org/557061 | 12:02 |
odyssey4me | evrardjp: the infrastructure healthcheck playbook is very handy, thank you! It needed some fixes though: https://review.openstack.org/599360 | 12:04 |
odyssey4me | the hosts one is failing too - going to work on fixes for that now | 12:04 |
evrardjp | odyssey4me: interesting, those are older, so I didn't work on them recently :) | 12:05 |
evrardjp | odyssey4me: you know what I think now? | 12:06 |
evrardjp | maybe we should, instead of having a role for python venv build, maybe we should have a module :) | 12:07 |
evrardjp | I know it's a big no-no | 12:07 |
evrardjp | but that's what I thought when I saw it. | 12:07 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/rocky: Remove molteniron playbook mistakenly left behind https://review.openstack.org/599362 | 12:08 |
evrardjp | odyssey4me: the absence of pika on the repo server highlights the fact that upstream doesn't rely on it anymore, so maybe this python code should be rewritten | 12:09 |
evrardjp | I was using it because it was simple and in requirements | 12:09 |
guilhermesp | huum seems that this is not enough odyssey4me https://github.com/openstack/openstack-ansible-os_neutron/commit/c0b17e9d89d86d0f9543c401316727cfcd6045f2 going to take a look now | 12:09 |
odyssey4me | guilhermesp: great, thank you | 12:10 |
odyssey4me | evrardjp: yeah, I guess an action plugin could replace the role | 12:11 |
evrardjp | because we are passing all the variables anyway | 12:11 |
evrardjp | :p | 12:11 |
evrardjp | and no problem of include role etc. | 12:11 |
odyssey4me | evrardjp: I will, in time, make changes to the ansible pip module to enable the use of more CLI arguments to eliminate some of those tasks | 12:12 |
evrardjp | good idea | 12:12 |
odyssey4me | I mean, the 'virtualenv_site_packages' argument is now obsolete. And that could be replaced by a more generic 'virtualenv_extra_args' anyway. | 12:13 |
*** Bhujay has joined #openstack-ansible | 12:14 | |
odyssey4me | With 'virtualenv_extra_args' and something which adapts arguments based on the version, we could remove quite a few tasks for the role to simplify it. | 12:14 |
odyssey4me | but yeah, I'll work on that some time in the future | 12:14 |
*** shyamb has joined #openstack-ansible | 12:15 | |
hwoarang | evrardjp: would you like to re-vote on https://review.openstack.org/#/c/598978/ please | 12:22 |
openstackgerrit | Merged openstack/openstack-ansible stable/rocky: Add missing congress_oslomsg_rpc_password https://review.openstack.org/599019 | 12:25 |
openstackgerrit | Merged openstack/openstack-ansible stable/rocky: docs: Add links to very useful Galera recovery docs https://review.openstack.org/599184 | 12:25 |
openstackgerrit | Merged openstack/openstack-ansible stable/queens: docs: Add links to very useful Galera recovery docs https://review.openstack.org/599185 | 12:25 |
openstackgerrit | Merged openstack/openstack-ansible stable/pike: docs: Add links to very useful Galera recovery docs https://review.openstack.org/599186 | 12:25 |
hwoarang | evrardjp: thank you | 12:29 |
hwoarang | odyssey4me: are these build simplification patches going to be in rocky as well? | 12:29 |
openstackgerrit | Merged openstack/openstack-ansible-os_ceilometer master: Use a common python build/install role https://review.openstack.org/557042 | 12:30 |
openstackgerrit | Merged openstack/openstack-ansible-openstack_openrc master: templates: openrc: Add missing 'insecure' and 'verify' options https://review.openstack.org/598978 | 12:38 |
*** udesale has joined #openstack-ansible | 12:39 | |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-openstack_openrc stable/rocky: templates: openrc: Add missing 'insecure' and 'verify' options https://review.openstack.org/599368 | 12:42 |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-openstack_openrc stable/queens: templates: openrc: Add missing 'insecure' and 'verify' options https://review.openstack.org/599369 | 12:42 |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-openstack_openrc stable/pike: templates: openrc: Add missing 'insecure' and 'verify' options https://review.openstack.org/599370 | 12:42 |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-openstack_openrc stable/ocata: templates: openrc: Add missing 'insecure' and 'verify' options https://review.openstack.org/599371 | 12:42 |
odyssey4me | hwoarang: as much as I'd like that, it's a bit late to make such changes - there's still along journey to go | 12:43 |
hwoarang | ok | 12:43 |
odyssey4me | hwoarang: we could perhaps consider it for a .1 release if it all works out nicely and is done relatively soon | 12:43 |
odyssey4me | hwoarang: why do you ask? | 12:43 |
hwoarang | no reason i was just curious | 12:44 |
odyssey4me | there will be some changes that I may port back as I find things that are low impact | 12:44 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Gather facts for host healthcheck https://review.openstack.org/599373 | 12:50 |
odyssey4me | evrardjp: it turns out fixing the host healthcheck was pretty simple: https://review.openstack.org/599373 | 12:51 |
*** pcaruana has quit IRC | 12:52 | |
*** shardy has joined #openstack-ansible | 12:55 | |
guilhermesp | odyssey4me: you think a good approach would be register the results of each task and at the end of the playbook restart the services instead of using handlers for almost each play? https://review.openstack.org/#/c/588647/9/tasks/neutron_post_install.yml | 12:59 |
odyssey4me | guilhermesp: no, not unless absolutely necessary - why? | 13:00 |
evrardjp | odyssey4me: looks reasonable to me | 13:00 |
evrardjp | odyssey4me: you used it for your testing? | 13:01 |
guilhermesp | maybe makes no sense indeed. Actually the problem seems to be that the neutron service is starting without api-paste.ini | 13:01 |
evrardjp | so far it seems only you and me have used it : ) | 13:01 |
*** rgogunskiy has joined #openstack-ansible | 13:01 | |
odyssey4me | evrardjp: I'm using them to validate whether my restored VM's with a full openstack environment are healthy... it's far easier than log trawling :p | 13:02 |
odyssey4me | so yeah, this is really nice stuff - much easier to grok than tempest results | 13:02 |
odyssey4me | I could do with an extra set of eyes to figure out this fail though: http://paste.openstack.org/show/729323/ | 13:02 |
evrardjp | odyssey4me: yeah I think I should make those part of the documentation + more flexible | 13:02 |
odyssey4me | The error is 'The allocation pool 172.29.248.201-172.29.248.255 spans beyond the subnet cidr 10.0.248.0/26' - but I can't for the life of me find where /26 is allocated for that network. It seems it must be outside of the healthcheck playbook. | 13:03 |
evrardjp | odyssey4me: 172.29.248.201-172.29.248.255 spans beyond the subnet cidr 10.0.248.0/26 :) | 13:03 |
evrardjp | oh yoou just pasted it | 13:03 |
evrardjp | yeah they need to match | 13:04 |
evrardjp | on what you are using | 13:04 |
odyssey4me | aha: tempest_public_subnet_cidr: 10.0.248.0/26 | 13:05 |
evrardjp | if you're using mnaio: https://github.com/openstack/openstack-ansible-ops/blob/7ac06c522edc4d3138f0d1aa6f395f54c647034e/multi-node-aio/playbooks/osa/openstack_user_config.yml#L6 | 13:05 |
*** hamzaachi has joined #openstack-ansible | 13:05 | |
odyssey4me | that's an override set in the AIO bootstrap I think | 13:05 |
odyssey4me | oh actually: /etc/openstack_deploy/user_mnaio_variables.yml:tempest_public_subnet_cidr: 10.0.248.0/26 | 13:05 |
evrardjp | yeah which is a part of your flat network | 13:06 |
odyssey4me | easy fix then | 13:06 |
evrardjp | I think we should do a smart vars_files :) | 13:06 |
evrardjp | defaulting to whatever exists now, but overridable by the user | 13:07 |
evrardjp | https://github.com/openstack/openstack-ansible/blob/master/tests/roles/bootstrap-host/templates/user_variables.aio.yml.j2#L23 is what I used I guess | 13:08 |
odyssey4me | well, these are all extra vars, so all we have to do is use the var and default to the vaues that are currently hard coded | 13:08 |
evrardjp | odyssey4me: well, I don't think it's that simple: https://github.com/openstack/openstack-ansible/blob/master/playbooks/defaults/healthchecks-vars.yml#L29-L50 | 13:10 |
evrardjp | if the public is a subset of a /24 it's gonna be a nightmare | 13:10 |
odyssey4me | I'll push a patch up in a bit and you can let me know what you think. | 13:11 |
evrardjp | which is your case | 13:11 |
odyssey4me | oh yeah, that sucks a bit | 13:11 |
openstackgerrit | Merged openstack/openstack-ansible-os_barbican master: Use a common python build/install role https://review.openstack.org/557041 | 13:11 |
openstackgerrit | Merged openstack/openstack-ansible-os_searchlight master: Use a common python build/install role https://review.openstack.org/599255 | 13:12 |
evrardjp | odyssey4me: simply create in MNAIO the override of these vars? | 13:12 |
evrardjp | as you're already overriding the defaults, I guess it's not a big deal? | 13:12 |
openstackgerrit | Merged openstack/openstack-ansible-os_aodh master: Use a common python build/install role https://review.openstack.org/557039 | 13:12 |
odyssey4me | evrardjp: well, ipaddr('-1') is an interesting option to solve at least part of that problem | 13:14 |
openstackgerrit | Merged openstack/openstack-ansible-os_rally master: Use a common python build/install role https://review.openstack.org/599246 | 13:15 |
openstackgerrit | Merged openstack/openstack-ansible-os_panko master: Use a common python build/install role https://review.openstack.org/599245 | 13:18 |
openstackgerrit | Merged openstack/openstack-ansible-os_heat master: Use a common python build/install role https://review.openstack.org/557052 | 13:19 |
odyssey4me | evrardjp: ah ok, I see in the healthcheck we are setting the dhcp range, but in tempest we are not - it may be best to keep them the same, don't you think? | 13:19 |
openstackgerrit | Merged openstack/openstack-ansible-os_neutron master: Use a common python build/install role https://review.openstack.org/599238 | 13:20 |
openstackgerrit | Merged openstack/openstack-ansible-os_horizon master: Use a common python build/install role https://review.openstack.org/557053 | 13:21 |
openstackgerrit | Merged openstack/openstack-ansible-os_keystone master: Use a common python build/install role https://review.openstack.org/557059 | 13:21 |
evrardjp | well this was initially intended to bypass tempest, not use tempest... I think I had a patch for that in the past that was relying on tempest to setup the networks, but it seems futile | 13:22 |
evrardjp | but we can bring that feature back | 13:23 |
openstackgerrit | Merged openstack/openstack-ansible-os_designate master: Use a common python build/install role https://review.openstack.org/557049 | 13:23 |
evrardjp | if tempest_install | bool don't run those setup of networks... | 13:23 |
odyssey4me | evrardjp: well, all I'm working towards here is that the two can co-exist - so if there are overrides/group_vars implemented, use them | 13:24 |
evrardjp | I am okay to change,just propose what you think best: ) | 13:24 |
openstackgerrit | Merged openstack/openstack-ansible-os_sahara master: Use a common python build/install role https://review.openstack.org/599247 | 13:24 |
*** hamzaachi_ has joined #openstack-ansible | 13:25 | |
*** hamzaachi has quit IRC | 13:25 | |
*** Emine has quit IRC | 13:28 | |
*** Emine has joined #openstack-ansible | 13:30 | |
*** schwicht has joined #openstack-ansible | 13:32 | |
openstackgerrit | Merged openstack/openstack-ansible-os_cinder master: Use a common python build/install role https://review.openstack.org/557043 | 13:32 |
openstackgerrit | Merged openstack/openstack-ansible-os_octavia master: Use a common python build/install role https://review.openstack.org/599244 | 13:40 |
openstackgerrit | Merged openstack/openstack-ansible-os_swift master: Use a common python build/install role https://review.openstack.org/599256 | 13:43 |
*** balioune has joined #openstack-ansible | 13:52 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Allow openstack healthcheck and tempest to co-exist https://review.openstack.org/599401 | 13:52 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Allow openstack healthcheck and tempest to co-exist https://review.openstack.org/599401 | 13:54 |
*** Bhujay has quit IRC | 13:58 | |
*** shyamb has quit IRC | 14:00 | |
*** hamzaachi_ has quit IRC | 14:06 | |
*** hamzaachi_ has joined #openstack-ansible | 14:07 | |
odyssey4me | I have to say, trying to debug failures using the journals sucks pretty bad unless you have an ELK stack handy. I'm thinking of making the old log files still be available when 'debug: True'. What do you think, cores? | 14:08 |
odyssey4me | maybe it's just a familiarity issue, but so far 'journalctl -f' on the host is very noisy, and not informative | 14:12 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Only copy the config files/folders required https://review.openstack.org/599402 | 14:24 |
odyssey4me | hwoarang evrardjp got a few more ready if you have a gap for them: https://review.openstack.org/#/q/topic:bp/python-build-install-simplification+status:open | 14:27 |
mnaser | morning all | 14:29 |
mnaser | slow day today, holiday for most of NA | 14:29 |
noonedeadpunk | Guys, have a question. I would like to change an IP address for live migrations in user_variables. For this, I've overrided nova_nova_conf_overrides in this way http://paste.openstack.org/show/729335/ But specified interface in live_migration_inbound_addr exists only for nova_compute. I tried to use ternary filter for checking if inventory_hostname in groups.nova_compute, but task still fails, as it seems, that playbook tries to | 14:31 |
noonedeadpunk | get variable before processing ternary filter | 14:31 |
noonedeadpunk | even tried to do smth like this http://paste.openstack.org/show/729338/ but without any result | 14:35 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Allow openstack healthcheck and tempest to co-exist https://review.openstack.org/599401 | 14:41 |
odyssey4me | noonedeadpunk: the issue is that conf override will be applied to *all* nova.conf files, not just compute | 14:44 |
noonedeadpunk | Ok, resolved it with http://paste.openstack.org/show/729340/ but it's really very weired construction... | 14:44 |
odyssey4me | noonedeadpunk: yep, that's a way to do it - instead of 'inventory_hostname in groups.nova_compute' you can also do 'nova_compute' in groups | 14:45 |
odyssey4me | that will resolve slightly faster | 14:45 |
noonedeadpunk | odyssey4me: yeah, I understand, but I thought, that ternary should do the trick - instead I have to add extra ternary for checking interface, even when I don't need it... | 14:45 |
odyssey4me | noonedeadpunk: however, I think you might find this more interesting: https://evrard.me/group-and-host-variables-overriding-in-openstack-ansible | 14:45 |
odyssey4me | well, it is needed, because there are non-compute hosts which have nova.conf and you're trying to add that override there too - so ansible tries to find that interface | 14:46 |
odyssey4me | noonedeadpunk: are you putting that into user_variables.yml, or are you putting it into /etc/openstack_deploy/group_vars/nova_compute.yml ? | 14:47 |
noonedeadpunk | odyssey4me: into user_variables. And I hoped that this contruction should be enough for resolving the problem http://paste.openstack.org/show/729341/ | 14:48 |
noonedeadpunk | thanks for the link - I'm going to read this post now) | 14:49 |
hwoarang | odyssey4me: done | 14:49 |
hwoarang | i trade you with https://review.openstack.org/#/c/598333/ :) | 14:49 |
odyssey4me | thanks hwoarang | 14:49 |
odyssey4me | done | 14:50 |
noonedeadpunk | yep, you're right. Usage of group_vars will definetelly resolve the problem | 14:51 |
odyssey4me | noonedeadpunk: if you have an inventory.ini file in /etc/openstack_deploy, then you can also put group_var/host_var overrides into /etc/openstack_deploy/{host_vars,group_vars} - that way you can apply an override to only a selected group, rather than globally, which is what the user_variables.yml file is | 14:51 |
odyssey4me | hwoarang: are you afraid of this patch? ;) https://review.openstack.org/557055 | 14:52 |
openstackgerrit | Merged openstack/openstack-ansible stable/rocky: Remove bonds from AIO network config https://review.openstack.org/594913 | 14:52 |
openstackgerrit | Merged openstack/openstack-ansible stable/rocky: Add lxc3 compatibility https://review.openstack.org/594839 | 14:52 |
openstackgerrit | Merged openstack/openstack-ansible stable/rocky: Ensure that tests fail early if network interfaces are down https://review.openstack.org/594912 | 14:52 |
noonedeadpunk | odyssey4me thanks, I've catch the idea) | 14:52 |
hwoarang | must have missed it | 14:54 |
openstackgerrit | Merged openstack/openstack-ansible stable/rocky: Add test for Ubuntu Bionic https://review.openstack.org/594927 | 14:56 |
*** fghaas has quit IRC | 14:57 | |
*** fghaas has joined #openstack-ansible | 15:00 | |
openstackgerrit | Merged openstack/openstack-ansible master: Track master branch for tempest https://review.openstack.org/596761 | 15:00 |
openstackgerrit | Merged openstack/openstack-ansible-openstack_hosts master: Revert "Revert "SUSE: Switch to OBS Cloud Master repository"" https://review.openstack.org/598333 | 15:03 |
*** luksky has quit IRC | 15:05 | |
openstackgerrit | Merged openstack/openstack-ansible-ops master: MNAIO: Only copy the config files/folders required https://review.openstack.org/599402 | 15:05 |
*** cshen has quit IRC | 15:07 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Move bionic jobs to voting https://review.openstack.org/594926 | 15:08 |
guilhermesp | jrosser: odyssey4me mnaser I think i know what happened with os_neutron api-paste.ini issue that is blocking some patches. Hold on a while and then I explain what has been happening :) I'm afk for a moment | 15:20 |
odyssey4me | haha, guilhermesp I was just about to ask | 15:20 |
jrosser | I still have a set of reviews ready for bionic on rocky - evrardjp hwoarang would be good to get those moving | 15:22 |
jrosser | mnaser too ^ | 15:22 |
mnaser | ah sorry | 15:23 |
mnaser | i can look at those now | 15:23 |
hwoarang | jrosser: i think i have reviewed everything. do you have a list? | 15:24 |
openstackgerrit | Merged openstack/openstack-ansible-ops master: Reducing memory footprint on mnaio https://review.openstack.org/599022 | 15:24 |
jrosser | hwoarang: ahha! serves me right for not checking again - looks like theyre done! thanks | 15:25 |
*** rgogunskiy has quit IRC | 15:26 | |
*** udesale has quit IRC | 15:26 | |
jrosser | odyssey4me: i wasn't sure if a release note was needed for the point rocky/bionic becomes real, perhaps that should be on the backport of https://review.openstack.org/#/c/594926/ ? | 15:28 |
*** schwicht has quit IRC | 15:30 | |
hwoarang | mnaser: what do you think of https://review.openstack.org/#/q/topic:opensuse-ceph-distro-job+(status:open+OR+status:merged) ? | 15:31 |
*** schwicht has joined #openstack-ansible | 15:31 | |
hwoarang | especially the 2nd patch since you reworked the AIO jobs | 15:31 |
*** fghaas has quit IRC | 15:31 | |
hwoarang | for me it's helpful since that's the only way to test ceph+opensuse given that the source-based deployment fails due to timeout. | 15:32 |
hwoarang | hmm actually the timeout has been increased so the sourced-based one may pass now | 15:33 |
*** fghaas has joined #openstack-ansible | 15:33 | |
*** fghaas has quit IRC | 15:34 | |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible master: zuul: Restore the openSUSE deployment jobs https://review.openstack.org/599425 | 15:35 |
odyssey4me | jrosser: hmm, that's a good point - yeah, if we're porting it back to rocky, then perhaps we should just add it there | 15:38 |
jrosser | I didn’t know which way to do it, if the reno needs to be in master then get backported.... in which case it should probably go on that “move to voting” you just tended to? | 15:40 |
*** fghaas has joined #openstack-ansible | 15:43 | |
odyssey4me | jrosser: tell you what - how about we just add it to that patch | 15:44 |
mnaser | hwoarang: approach looks ok, i'd like to probably see the jobs a bit more stable | 15:44 |
mnaser | have some experience with what their run times look like and timeout-ability | 15:44 |
*** balioune has quit IRC | 15:44 | |
jrosser | odyssey4me: sounds good | 15:44 |
hwoarang | mnaser: the distro job takes ~ 1h45m, the sourced one takes about ~ 2h10m | 15:45 |
odyssey4me | jrosser: will you do that? | 15:45 |
mnaser | hwoarang: the only concern is that i have noticed the opensuse don't rely as much on upstream mirrors | 15:45 |
mnaser | i'd like to hear other cores opinion, i'm okay with bringing them on if they're relatively stable | 15:45 |
hwoarang | what upstream mirrors? | 15:46 |
hwoarang | like galera? | 15:46 |
jrosser | odyssey4me: not right now, but I can do it later, needs some words about lxc3 and xenial-backports I think | 15:46 |
odyssey4me | mnaser hwoarang if they're that close to the timeout, then perhaps adding them to periodics for now to build a track record would be good? | 15:46 |
mnaser | hwoarang: i think galera was one of them but i cant recall the other one, but anyways, we can fix those as they pop up | 15:46 |
mnaser | yeah, distro jobs seem ok, but the source one feels so close to timeout | 15:46 |
hwoarang | odyssey4me: that's the thing, now that timeout is 2h20m, they are passing | 15:46 |
hwoarang | previously the failed with the 2h timeout | 15:47 |
odyssey4me | jrosser: ok, I'll leave it to you and mark that patch -w for now | 15:47 |
hwoarang | mnaser: but you added a galera mirror in openstack infra right? opensuse uses the same mirror like other distros. same packages provided by upstream | 15:47 |
odyssey4me | I just rebased the patch given it's been in conflict while you've been away, and thought it best to get it moving | 15:47 |
*** fghaas has quit IRC | 15:47 | |
mnaser | i think i did, but im not sure if i updated opensuse to use it because im not that familiar | 15:47 |
hwoarang | actually, everything else is mirrored in infra | 15:47 |
hwoarang | aha | 15:47 |
hwoarang | i can check that | 15:47 |
mnaser | i think rabbitmq is still something we pull direclty | 15:47 |
mnaser | directly | 15:47 |
hwoarang | opensuse uses the rabbitmq from its repos | 15:48 |
hwoarang | the opensuse repos i mean which are mirrored in infra | 15:48 |
mnaser | yeah if you can, i dont really have a suse env nor have i ever used it (i have had some experience with ubuntu in the past so it's a bit easier) | 15:48 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-ops master: Allow using custom publish host https://review.openstack.org/599263 | 15:49 |
*** fghaas has joined #openstack-ansible | 15:50 | |
hwoarang | looks like opensuse is using the same infra mirror for centos for galera | 15:51 |
hwoarang | good | 15:51 |
mnaser | cool cool | 15:51 |
mnaser | jrosser: most of your open patches seem to be -1'd by zuul so am i missing anything? | 15:51 |
mnaser | :p | 15:51 |
*** fghaas has quit IRC | 15:52 | |
*** DanyC has quit IRC | 15:53 | |
odyssey4me | mnaser: I guess we should discuss how we're going to handled 'extended maintenance' at the PTG, given https://review.openstack.org/598164 | 15:54 |
jrosser | mnaser: no not missing anything, hwoarang got in there without me noticing and they’re all done :) | 15:54 |
odyssey4me | see also https://governance.openstack.org/tc/resolutions/20180301-stable-branch-eol.html | 15:54 |
mnaser | ugh | 15:55 |
* mnaser was personally pretty against EM but it is what it is | 15:55 | |
*** vnogin has joined #openstack-ansible | 15:55 | |
mnaser | i feel like this is another thing we need to maintain/handle at our fairly-strapped-for-resources grou | 15:55 |
mnaser | p | 15:55 |
odyssey4me | well, we just need to decide what to do about it | 15:56 |
odyssey4me | we could just remove all tests, making the check/gate a no-op - then any patches just get reviewed but we make no promises, which is the spirit of that process | 15:57 |
mnaser | i guess that's a reasonable thing to do | 15:57 |
odyssey4me | we leave the code in-place, but disable all functional testing | 15:57 |
mnaser | assuming anyone will ever use those branches | 15:58 |
odyssey4me | that way we're not saddled with trying to figure out how the tests worked which we moved on from 3 years ago | 15:58 |
mnaser | i agree | 15:58 |
mnaser | is ptg bot up or not yet | 15:58 |
odyssey4me | and of course anyone pushing patches up to that branch would be responsible for their own testing | 15:58 |
mnaser | yep that's the whole point of it | 15:59 |
odyssey4me | extended maintenance, as much as I don't like it, will be good for RAX - and we will very likely push patches up to it - right now we've got newton forks for several repositories | 15:59 |
mnaser | odyssey4me: that makes me happy to hear | 15:59 |
mnaser | i was worried we'd have zero consumers | 15:59 |
odyssey4me | that said, RAX isn't using Ocata at all - so in this particular case, meh | 15:59 |
mnaser | lol | 16:00 |
mnaser | ..nevermind that thought then | 16:00 |
mnaser | :P | 16:00 |
mnaser | ok looks like the ptg schedule is up so i (should) start scheduling things | 16:00 |
mnaser | how do we feel about a google calendar that i can share? | 16:00 |
odyssey4me | Sure, whatever works. | 16:01 |
*** cshen has joined #openstack-ansible | 16:08 | |
*** dhellmann has quit IRC | 16:39 | |
*** dhellmann has joined #openstack-ansible | 16:39 | |
*** vnogin has quit IRC | 16:40 | |
*** dhellmann has quit IRC | 16:44 | |
*** dhellmann has joined #openstack-ansible | 16:45 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tacker master: Use a common python build/install role https://review.openstack.org/599434 | 16:46 |
*** electrofelix has quit IRC | 16:50 | |
*** szaher has quit IRC | 16:50 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest master: Use a common python build/install role https://review.openstack.org/599436 | 16:52 |
*** DanyC has joined #openstack-ansible | 16:54 | |
*** dhellmann has quit IRC | 16:54 | |
*** dhellmann has joined #openstack-ansible | 16:54 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_trove master: Use a common python build/install role https://review.openstack.org/599437 | 16:57 |
*** DanyC has quit IRC | 16:58 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_watcher master: Use a common python build/install role https://review.openstack.org/599438 | 17:02 |
*** cshen has quit IRC | 17:02 | |
*** cshen has joined #openstack-ansible | 17:06 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_zaqar master: Use a common python build/install role https://review.openstack.org/599439 | 17:07 |
*** cshen has quit IRC | 17:10 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_zun master: Use a common python build/install role https://review.openstack.org/599440 | 17:11 |
*** cshen has joined #openstack-ansible | 17:12 | |
guilhermesp | odyssey4me: so the api-paste.ini thing | 17:17 |
guilhermesp | I noticed that some config files that are in /etc/neutron has soft links to /usr/share/neutron | 17:17 |
odyssey4me | guilhermesp: ok, I think that's working as intended | 17:18 |
*** hamzaachi_ has quit IRC | 17:18 | |
guilhermesp | that was the cause that neutron-server wasn't finding the conf file because it was only in /usr/share/neutron as we defined here https://github.com/openstack/openstack-ansible-os_neutron/commit/c0b17e9d89d86d0f9543c401316727cfcd6045f2 | 17:18 |
guilhermesp | I manually placed the api-paste in /etc/neutron and linked to /usr/share/neutron | 17:18 |
guilhermesp | lemme see how these links are created | 17:19 |
guilhermesp | maybe we don't need mine commit | 17:19 |
guilhermesp | I'm going to run some local tests here to confirm the best solution | 17:19 |
odyssey4me | ok, awesome thanks | 17:19 |
guilhermesp | and i'll submit the PR asap | 17:19 |
*** aicilamd has joined #openstack-ansible | 17:26 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Use a common python build/install role https://review.openstack.org/579021 | 17:32 |
odyssey4me | alright, I'm out for the night - cheers folks! | 17:35 |
guilhermesp | take care odyssey4me ! the fix will be available today | 17:35 |
openstackgerrit | Merged openstack/openstack-ansible master: ansible-role-requirements: Unpin ceph-ansible role https://review.openstack.org/597146 | 17:36 |
*** johnny101 has joined #openstack-ansible | 17:38 | |
openstackgerrit | Merged openstack/openstack-ansible-openstack_openrc stable/ocata: templates: openrc: Add missing 'insecure' and 'verify' options https://review.openstack.org/599371 | 17:41 |
*** vnogin has joined #openstack-ansible | 17:41 | |
openstackgerrit | Merged openstack/openstack-ansible-lxc_hosts stable/rocky: Make apt key import for Ubuntu a uniform process https://review.openstack.org/598964 | 17:43 |
openstackgerrit | Merged openstack/openstack-ansible-openstack_openrc stable/pike: templates: openrc: Add missing 'insecure' and 'verify' options https://review.openstack.org/599370 | 17:45 |
openstackgerrit | Merged openstack/openstack-ansible-openstack_openrc stable/queens: templates: openrc: Add missing 'insecure' and 'verify' options https://review.openstack.org/599369 | 17:47 |
openstackgerrit | Merged openstack/openstack-ansible-openstack_openrc stable/rocky: templates: openrc: Add missing 'insecure' and 'verify' options https://review.openstack.org/599368 | 17:47 |
*** dave-mccowan has quit IRC | 17:47 | |
*** poopcat has joined #openstack-ansible | 18:03 | |
*** tosky has quit IRC | 18:04 | |
*** fghaas has joined #openstack-ansible | 18:10 | |
d34dh0r53 | https://review.openstack.org/#/c/599060/ | 18:14 |
d34dh0r53 | if anyone is around | 18:14 |
*** DanyC has joined #openstack-ansible | 18:19 | |
*** ansmith has joined #openstack-ansible | 18:29 | |
*** olivierb_ has quit IRC | 18:32 | |
*** fghaas has quit IRC | 18:33 | |
*** poopcat has quit IRC | 18:36 | |
openstackgerrit | Merged openstack/openstack-ansible-os_cloudkitty master: Use a common python build/install role https://review.openstack.org/557046 | 18:40 |
openstackgerrit | Merged openstack/openstack-ansible-os_monasca master: Use a common python build/install role https://review.openstack.org/599198 | 18:44 |
*** luksky has joined #openstack-ansible | 18:47 | |
openstackgerrit | Merged openstack/openstack-ansible-os_molteniron master: Use a common python build/install role https://review.openstack.org/557063 | 18:48 |
*** hamzaachi has joined #openstack-ansible | 18:59 | |
openstackgerrit | Merged openstack/openstack-ansible-os_monasca-agent master: Use a common python build/install role https://review.openstack.org/599199 | 19:01 |
openstackgerrit | Merged openstack/openstack-ansible-os_congress master: Use a common python build/install role https://review.openstack.org/557047 | 19:09 |
*** kencjohnston_ has quit IRC | 19:10 | |
*** mrhillsman has quit IRC | 19:10 | |
*** jamespo has quit IRC | 19:10 | |
*** boxrick has quit IRC | 19:10 | |
*** AndyWojo has quit IRC | 19:10 | |
*** chris_hultin has quit IRC | 19:10 | |
*** xar- has quit IRC | 19:10 | |
*** toan has quit IRC | 19:10 | |
*** ioni has quit IRC | 19:10 | |
*** Tyrantelf has quit IRC | 19:10 | |
*** kencjohnston has joined #openstack-ansible | 19:10 | |
*** xar- has joined #openstack-ansible | 19:11 | |
openstackgerrit | Merged openstack/openstack-ansible-os_ironic master: Use a common python build/install role https://review.openstack.org/557055 | 19:14 |
openstackgerrit | Merged openstack/openstack-ansible-os_gnocchi master: Use a common python build/install role https://review.openstack.org/557050 | 19:18 |
openstackgerrit | Merged openstack/openstack-ansible-os_masakari master: Use a common python build/install role https://review.openstack.org/599197 | 19:19 |
*** hamzaachi has quit IRC | 19:35 | |
*** aicilamd has quit IRC | 19:41 | |
*** cshen has quit IRC | 19:46 | |
*** cshen has joined #openstack-ansible | 19:50 | |
*** olivierb has joined #openstack-ansible | 20:02 | |
openstackgerrit | Merged openstack/openstack-ansible stable/pike: Use ZUUL_SRC_PATH to pass the path to Zuul git sources https://review.openstack.org/591160 | 20:06 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: Move bionic jobs to voting, add release note. https://review.openstack.org/594926 | 20:06 |
*** fghaas has joined #openstack-ansible | 20:19 | |
*** cshen has quit IRC | 20:20 | |
*** fghaas has quit IRC | 20:23 | |
*** cshen has joined #openstack-ansible | 20:27 | |
*** johnny101 has quit IRC | 20:29 | |
*** johnny101 has joined #openstack-ansible | 20:29 | |
*** cshen has quit IRC | 20:31 | |
*** hamzaachi has joined #openstack-ansible | 20:32 | |
*** hamzaachi has quit IRC | 20:33 | |
*** hamzaachi has joined #openstack-ansible | 20:33 | |
openstackgerrit | Merged openstack/openstack-ansible stable/rocky: Update ansible to 2.5.8 https://review.openstack.org/599253 | 20:58 |
*** DanyC has quit IRC | 21:16 | |
*** DanyC has joined #openstack-ansible | 21:17 | |
*** Emine has quit IRC | 21:20 | |
*** DanyC has quit IRC | 21:21 | |
*** hamzaachi has quit IRC | 21:35 | |
*** cshen has joined #openstack-ansible | 21:48 | |
*** cshen has quit IRC | 21:52 | |
*** luksky has quit IRC | 22:03 | |
*** DanyC has joined #openstack-ansible | 22:17 | |
*** DanyC has quit IRC | 22:23 | |
openstackgerrit | Merged openstack/openstack-ansible-os_magnum master: Use a common python build/install role https://review.openstack.org/557061 | 22:33 |
*** cshen has joined #openstack-ansible | 22:40 | |
*** cshen has quit IRC | 22:45 | |
*** DanyC has joined #openstack-ansible | 22:45 | |
*** DanyC has quit IRC | 22:50 | |
*** threestrands has joined #openstack-ansible | 23:03 | |
*** tosky has joined #openstack-ansible | 23:03 | |
*** threestrands has quit IRC | 23:03 | |
*** threestrands has joined #openstack-ansible | 23:04 | |
*** threestrands has quit IRC | 23:05 | |
*** threestrands has joined #openstack-ansible | 23:05 | |
*** threestrands has quit IRC | 23:06 | |
*** threestrands has joined #openstack-ansible | 23:07 | |
*** threestrands has quit IRC | 23:08 | |
*** threestrands has joined #openstack-ansible | 23:08 | |
mnaser | d34dh0r53: is that a stable/pike only cahnge? | 23:13 |
d34dh0r53 | mnaser: yes, that version of stestr doesn't like both white and blacklists in the same command. Queens+ doesn't seem to be affected | 23:15 |
*** dave-mccowan has joined #openstack-ansible | 23:16 | |
*** aicilamd has joined #openstack-ansible | 23:26 | |
*** aicilamd has quit IRC | 23:28 | |
*** DanyC has joined #openstack-ansible | 23:46 | |
openstackgerrit | Merged openstack/openstack-ansible-nspawn_hosts stable/rocky: Simplify inserting apt keys into nspawn image https://review.openstack.org/598965 | 23:50 |
*** DanyC has quit IRC | 23:50 | |
*** tosky has quit IRC | 23:52 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!