| *** dave-mcc_ has joined #openstack-ansible | 01:02 | |
| *** yifei has quit IRC | 01:16 | |
| *** yifei has joined #openstack-ansible | 01:48 | |
| *** yifei has left #openstack-ansible | 01:48 | |
| *** yifei has joined #openstack-ansible | 01:48 | |
| *** harryhuang_ has joined #openstack-ansible | 02:05 | |
| *** masuberu has quit IRC | 02:06 | |
| *** masuberu has joined #openstack-ansible | 02:06 | |
| *** deng_ has joined #openstack-ansible | 02:13 | |
| *** derekjhyang has joined #openstack-ansible | 02:16 | |
| *** deng_ has quit IRC | 02:24 | |
| *** deng_ has joined #openstack-ansible | 02:25 | |
| *** armaan has quit IRC | 02:29 | |
| *** armaan has joined #openstack-ansible | 02:29 | |
| *** mwarad has joined #openstack-ansible | 02:41 | |
| *** mwarad has quit IRC | 02:49 | |
| *** mwarad has joined #openstack-ansible | 02:56 | |
| *** masuberu has quit IRC | 03:06 | |
| *** masuberu has joined #openstack-ansible | 03:08 | |
| *** masber has joined #openstack-ansible | 03:15 | |
| *** masuberu has quit IRC | 03:18 | |
| *** mwarad has quit IRC | 03:22 | |
| *** adrianreza has joined #openstack-ansible | 03:40 | |
| *** dxiri has joined #openstack-ansible | 03:48 | |
| *** dave-mcc_ has quit IRC | 04:00 | |
| *** nattanon has quit IRC | 04:04 | |
| openstackgerrit | melissaml proposed openstack/openstack-ansible master: fix typos in documentation https://review.openstack.org/563493 | 04:48 |
|---|---|---|
| *** armaan has quit IRC | 04:54 | |
| *** armaan has joined #openstack-ansible | 04:54 | |
| *** niraj_singh has quit IRC | 05:11 | |
| *** dxiri has quit IRC | 05:12 | |
| *** dxiri has joined #openstack-ansible | 05:13 | |
| *** dxiri has quit IRC | 05:18 | |
| *** pmannidi has quit IRC | 05:37 | |
| *** mwarad has joined #openstack-ansible | 06:07 | |
| *** sar has joined #openstack-ansible | 06:12 | |
| *** mma has joined #openstack-ansible | 06:22 | |
| *** cshen has joined #openstack-ansible | 06:27 | |
| *** mwarad has quit IRC | 06:35 | |
| *** jbadiapa has joined #openstack-ansible | 06:51 | |
| *** peri has joined #openstack-ansible | 06:58 | |
| *** peri has quit IRC | 06:59 | |
| *** peri has joined #openstack-ansible | 06:59 | |
| *** peri has quit IRC | 06:59 | |
| *** epalper has joined #openstack-ansible | 07:00 | |
| Tahvok | Somehow we encontered this bug: https://bugs.launchpad.net/neutron/+bug/1648823 | 07:22 |
| openstack | Launchpad bug 1648823 in neutron "l3 agent HA communication failure" [Undecided,New] | 07:22 |
| Tahvok | Or similar one... | 07:22 |
| Tahvok | I'm ran ansible minor version upgrade, and got this after running infrastracture playbook | 07:23 |
| Tahvok | The network started jumping from one neutron to the other, and communication to the instances was lost every few seconds | 07:24 |
| Tahvok | I've just restarted our controller node, and it fixed the problem. Trying to investigate now of what happened exactly, but so far didn't find anything | 07:25 |
| *** pcaruana has joined #openstack-ansible | 07:25 | |
| *** cshen has quit IRC | 07:26 | |
| *** tosky has joined #openstack-ansible | 07:28 | |
| *** mwarad has joined #openstack-ansible | 07:33 | |
| *** admin0 has quit IRC | 07:45 | |
| *** DanyC has joined #openstack-ansible | 07:54 | |
| *** shardy has joined #openstack-ansible | 08:05 | |
| *** threestrands has quit IRC | 08:17 | |
| *** armaan has quit IRC | 08:24 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_watcher stable/queens: Do not log passwords https://review.openstack.org/563513 | 08:24 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_watcher stable/pike: Do not log passwords https://review.openstack.org/563514 | 08:24 |
| *** armaan has joined #openstack-ansible | 08:25 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_watcher stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/563515 | 08:25 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build stable/queens: Override the pip index used for the wheel build https://review.openstack.org/563516 | 08:25 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/queens: Add shade library to openstack-ansible venv https://review.openstack.org/563517 | 08:26 |
| *** mwe1 has quit IRC | 08:26 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/queens: bootstrap-host: Set repo build pip default mirror when in openstack-ci https://review.openstack.org/563518 | 08:26 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_server stable/queens: Allow pypiserver to be upgraded before repo build https://review.openstack.org/563519 | 08:27 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_install stable/queens: Disable the pip version check https://review.openstack.org/563521 | 08:29 |
| *** mwe1 has joined #openstack-ansible | 08:31 | |
| jrosser | not managing to enable the horizon fwaas panel - i have horizon_enable_neutron_fwaas: true, but rerunning the os-horizon.yml fails like this http://paste.openstack.org/show/719716/ think i'm missing a repo build step? | 08:35 |
| odyssey4me | jrosser yeah, what did you do to add the package for horizon? do you hav a review up for it? | 08:36 |
| evrardjp | jrosser: probably something to update | 08:36 |
| jrosser | odyssey4me: it appears like it is all there already | 08:36 |
| evrardjp | you have to check if the file there is part of the package | 08:36 |
| evrardjp | maybe they changed the file | 08:36 |
| evrardjp | and it didn't get updated | 08:36 |
| evrardjp | probably we should replace that mechanism with auto-discovery | 08:37 |
| evrardjp | it would be simpler | 08:37 |
| jrosser | ah right i see, let me dig around see whats actually there | 08:37 |
| odyssey4me | ah, I see it comes in through https://github.com/openstack/openstack-ansible-os_horizon/blob/master/defaults/main.yml#L307 | 08:37 |
| odyssey4me | it comes from pip directly, we don't build the package from source | 08:38 |
| Tahvok | Am I right to assume that there is no reason to restart rabbitmq on minor upgrade of osa if there was no change in rabbit version? | 08:38 |
| odyssey4me | Tahvok you have to set a var to make it upgrade rabbitmq, so yes - no reason | 08:38 |
| Tahvok | odyssey4me: according to minor version upgrade guide you should always set this var to true | 08:39 |
| Tahvok | But if the condition before that (see here: https://github.com/openstack/openstack-ansible-rabbitmq_server/blob/stable/ocata/tasks/rabbitmq_upgrade_check.yml#L56) | 08:39 |
| odyssey4me | you can, but it's wlways up to you - the upgrades guides are just guides which try to explain how one can do it - if you choose todo it other ways, that's up to you | 08:40 |
| Tahvok | Finds that no version upgrade is needed - no need to do it then | 08:40 |
| Tahvok | The thing is, that setting rabbitmq_upgrade var to true, will always stop and start rabbitmq service | 08:40 |
| Tahvok | Which will create failures accross all other openstack services | 08:41 |
| odyssey4me | ah, good point - it shouldn't do the stop/start unless there actually is a version change | 08:41 |
| Tahvok | But if there is no version to upgrade to - no need to run rabbitmq_upgrade_prep.yml at all, even if the var rabbitmq_upgrade is set to true | 08:41 |
| evrardjp | odyssey4me: odyssey4me https://github.com/openstack/openstack-ansible/blob/master/playbooks/defaults/repo_packages/openstack_services.yml#L116-L118 | 08:42 |
| evrardjp | woops double ping | 08:42 |
| odyssey4me | evrardjp oh, interesting - especially since the horizon role itself doesn't have those vars | 08:42 |
| Tahvok | I'm trying to investigate the issue I had with vrrp in neutron yesterday, and I found that all services had rabbitmq failures during the upgrade | 08:42 |
| evrardjp | odyssey4me: yeah that needs clean up | 08:43 |
| odyssey4me | evrardjp is there perhaps some magic way to drop any facts present in memory from set_facts ? | 08:44 |
| evrardjp | you can check the host vars | 08:45 |
| evrardjp | or vars but that's even bigger | 08:45 |
| evrardjp | ansible 2.4 I guess? | 08:45 |
| odyssey4me | well, I effectively want to set it back to the group_vars | 08:45 |
| odyssey4me | yes, this matters for queens/master | 08:45 |
| jrosser | ah yes names are different for the fwaas horizon files http://paste.openstack.org/show/719717/ | 08:47 |
| odyssey4me | evrardjp hmm, never mind - it looks like my theory of what's going wrong might be bogus | 08:48 |
| odyssey4me | it would appear that set_fact only lasts for the play - not across multiple plays in the same playbook | 08:48 |
| evrardjp | ansible_facts is a namespace available in 2.4 but I doubt you want to use it with set_fact without looking for trouble :p | 08:48 |
| evrardjp | odyssey4me: nope that shouldn't be the case | 08:49 |
| evrardjp | the set_fact stays for the playbook | 08:49 |
| Tahvok | Btw, had my first out of memory ansible run yesterday | 08:49 |
| Tahvok | Had to increase the deployment machine to 32 gb | 08:49 |
| evrardjp | maybe it doesn't pass the import of playbooks but it should be inside the same playbook at least | 08:49 |
| evrardjp | let me double check if that's not a regression | 08:49 |
| Taseer | evrardjp: good morning | 08:50 |
| Taseer | evrardjp: What is it not detecting your patchset ? => http://logs.openstack.org/39/560839/24/experimental/openstack-ansible-deploy-congress-ubuntu-xenial/7c1d94a/job-output.txt.gz#_2018-04-21_10_18_25_921889 | 08:51 |
| *** eumel8 has joined #openstack-ansible | 08:51 | |
| evrardjp | yup quickly checked odyssey4me | 08:51 |
| evrardjp | it works | 08:51 |
| evrardjp | double checking wiht ansible 2.4.1 | 08:51 |
| evrardjp | Taseer: because you don't have the depends-on anymore: https://review.openstack.org/#/c/560839/ | 08:52 |
| evrardjp | we discussed this | 08:52 |
| evrardjp | you can remove the depends on with the latest patchset of congress | 08:53 |
| evrardjp | ofc you need to bump congress for that to work. | 08:53 |
| evrardjp | ok you seem to have done it | 08:53 |
| Taseer | yeah... | 08:54 |
| odyssey4me | evrardjp ok, so within a playbook it stays across multiple plays? | 08:55 |
| evrardjp | you have to check in your logs and the repo build | 08:55 |
| evrardjp | if that hacking requirements is well built | 08:55 |
| evrardjp | odyssey4me: yes | 08:55 |
| evrardjp | confirmed | 08:55 |
| odyssey4me | ok, thanks | 08:55 |
| evrardjp | let me just do the include just in case | 08:55 |
| Taseer | evrardjp: do I not need to bump tempest, since the fix went in that repo ? | 08:56 |
| evrardjp | odyssey4me: works too. | 08:56 |
| evrardjp | Taseer: yes. | 08:56 |
| evrardjp | wait | 08:56 |
| evrardjp | where? | 08:56 |
| evrardjp | tempest? | 08:56 |
| evrardjp | could you show me the fix you're talking about? | 08:57 |
| evrardjp | The problem was with congress requirements | 08:57 |
| evrardjp | or congress tempest plugin | 08:57 |
| evrardjp | you have to bump those | 08:57 |
| evrardjp | if these are bumped, you have another problem | 08:57 |
| Taseer | ok... i see | 08:57 |
| Taseer | evrardjp: what is the problem after that ? | 08:58 |
| evrardjp | the patch I did was only for skipping test-requirements for some cases. It's not necessary if they fixed stuff. | 08:58 |
| evrardjp | Taseer: I don't know, you have to check yourself | 08:58 |
| *** niraj_singh has joined #openstack-ansible | 08:58 | |
| evrardjp | is there someone here with a good knowledge of memcached logging? | 09:00 |
| openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_horizon master: Correct the fwaas horizon panel filenames https://review.openstack.org/563536 | 09:03 |
| openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: Enable horizon fwaas panels when firewall_v2 driver is enabled https://review.openstack.org/563538 | 09:07 |
| odyssey4me | evrardjp ok, so this is curious | 09:08 |
| odyssey4me | when https://github.com/openstack/openstack-ansible/blob/master/playbooks/setup-infrastructure.yml runs, the repo-install playbook includes running repo-server, which runs these tasks: https://github.com/openstack/openstack-ansible/blob/master/playbooks/repo-server.yml#L22-L23 | 09:09 |
| evrardjp | yup? | 09:10 |
| evrardjp | and the facts aren't set? | 09:10 |
| odyssey4me | those tasks override the upper constraints, get-pip.py url and the pip default index to use upstream values because those in group_vars are not usable yet | 09:10 |
| odyssey4me | this is all as it should be | 09:10 |
| odyssey4me | once repo-install is over, the repo is there, so the repo is now usable | 09:11 |
| evrardjp | yup | 09:11 |
| evrardjp | isn't repo build that gets those? | 09:11 |
| evrardjp | isn't it repo_build that fills the constraints I mean | 09:11 |
| odyssey4me | when repo-use runs, it re-runs pip_install using those same facts - so what happens is that the pip conf isn't changed to use the repo because of those set facs | 09:11 |
| odyssey4me | *facts | 09:11 |
| odyssey4me | what's curious though is that later in that playbook the repo's upper constraints file is used - not the upstream constraints | 09:12 |
| odyssey4me | as an example, look at the utility playbook's pip install: http://logs.openstack.org/12/549012/6/check/openstack-ansible-deploy-aio_metal-ubuntu-xenial/33256d4/logs/ara-report/result/fb4ecf91-112b-4651-9005-e681a739523b/ | 09:13 |
| evrardjp | and the pip_install is an include role | 09:13 |
| odyssey4me | oh hang on - that's using the upstream ones | 09:13 |
| *** armaan has quit IRC | 09:13 | |
| evrardjp | yup | 09:14 |
| openstackgerrit | fupingxie proposed openstack/openstack-ansible-os_monasca master: Do not log passwords https://review.openstack.org/563540 | 09:14 |
| *** armaan has joined #openstack-ansible | 09:14 | |
| evrardjp | so we are working without a repo build for a while it looks like it :p | 09:14 |
| evrardjp | that might be the reason why ppl have issues :p | 09:14 |
| evrardjp | the repo use should include those common tasks | 09:14 |
| odyssey4me | yeah, and because we never run pip_install in the later plays when setup-openstack runs (new playbook) we never configure pip to use the repo | 09:15 |
| odyssey4me | well, if we include those tasks in the repo-use then they will re-test the upstream URL's instead of the group_vars because the facts are still in memory | 09:15 |
| evrardjp | yes but that's cool | 09:16 |
| evrardjp | we have no choice, the idempotency is already broken | 09:16 |
| odyssey4me | not really, what it means is that until setup-openstack runs we're never using the repo | 09:16 |
| evrardjp | because we switch something in the middle of the game | 09:17 |
| evrardjp | repo-use isn't just after repo build? | 09:17 |
| odyssey4me | so we should either run repo-use in setup-openstack... or we need to use something like include_vars/vars_files before we run pip install | 09:17 |
| evrardjp | I am confused there | 09:17 |
| evrardjp | just a sec | 09:18 |
| evrardjp | checking the code | 09:18 |
| odyssey4me | ok, so repo-build runs with the default pip index set to pypi due to the set_fact | 09:18 |
| evrardjp | https://github.com/openstack/openstack-ansible/blob/master/playbooks/repo-use.yml#L25 | 09:18 |
| evrardjp | so if we include the tasks to check the thing, it will override the set facts, isn't it? | 09:18 |
| evrardjp | let me double check | 09:18 |
| evrardjp | (sorry) | 09:18 |
| odyssey4me | so, let me walk you through it | 09:19 |
| odyssey4me | https://github.com/openstack/openstack-ansible/blob/master/inventory/group_vars/all/pip.yml#L33 is the default for pip_default_index | 09:19 |
| evrardjp | haha I see | 09:19 |
| evrardjp | the set_fact is ONLY the fallback | 09:19 |
| evrardjp | not the set_fact to the proper address, because we want to use group vars | 09:19 |
| odyssey4me | in setup-infrastructure before we build the repo, we check for whether the repo is present (it is not), so we override that default to the upstream value: https://github.com/openstack/openstack-ansible/blob/master/playbooks/common-tasks/set-upper-constraints.yml#L76 | 09:20 |
| evrardjp | yup | 09:20 |
| evrardjp | and then it sticks | 09:20 |
| evrardjp | because nothing re-sets it | 09:20 |
| evrardjp | so we have two choices | 09:20 |
| odyssey4me | once the repo is there, if we re-run the check https://github.com/openstack/openstack-ansible/blob/master/playbooks/common-tasks/set-upper-constraints.yml#L33 it will check the value we set as a set_fact, not the default in group_vars | 09:20 |
| evrardjp | either we make the set_fact use a variable that's pointing to an address, and we use another variable for the rest of the play, and we set_facts for all the cases (whether first url is 200 or not) | 09:21 |
| evrardjp | or we set_fact when we have done the process | 09:22 |
| evrardjp | yeah that's what I understood :) | 09:22 |
| evrardjp | thanks for the run through, it's better to make sure we are on the same page before discussing a fix :) | 09:23 |
| odyssey4me | I'm toying with the idea to just do vars_files: "{{ playbook_dir }}../inventory/all/pip.yml" in the repo-use playbook... or using include_vars... | 09:23 |
| odyssey4me | oh bother, set_fact is higher precedence than both | 09:24 |
| evrardjp | yes | 09:24 |
| evrardjp | I think it's cleaner if we rename the var to avoid cycle | 09:24 |
| evrardjp | and always set_fact | 09:24 |
| odyssey4me | we could move the repo-use to the beginning of the setup-openstack playbook, but that does leave it quite late | 09:24 |
| evrardjp | like url_to_test: blablabla, and then set_fact all the time | 09:25 |
| evrardjp | odyssey4me: I'd rather avoid that | 09:25 |
| evrardjp | the problem is if ppl run it separately | 09:25 |
| odyssey4me | so you're thinking we store the default in another set_fact var (or have a duplicate group_var)? | 09:25 |
| evrardjp | here, in repo use, we don't test it | 09:25 |
| evrardjp | let me use proper variable names here | 09:26 |
| evrardjp | just a sec | 09:26 |
| evrardjp | basically the problem comes with our process and the collision of pip_default_index | 09:26 |
| evrardjp | so we can say pip_default_index is x | 09:27 |
| evrardjp | (taken from group vars for now) | 09:27 |
| evrardjp | (but can be overriden by the user) | 09:27 |
| evrardjp | but in our processes, we are using a different variable | 09:27 |
| evrardjp | pip_index | 09:27 |
| evrardjp | or whatever the name | 09:27 |
| evrardjp | the advantage is that the set_fact process becomes a test of a generic pip_default_index, and setting the fact of the index we'll use: pip_index | 09:28 |
| openstackgerrit | fupingxie proposed openstack/openstack-ansible-os_zaqar master: Do not log passwords https://review.openstack.org/563544 | 09:29 |
| evrardjp | pip_index will be the pip_default_index from group var on http 200. Else it would be the other one | 09:29 |
| evrardjp | we can maybe remove that var from group_vars at the same time, let me check where it's used | 09:30 |
| openstackgerrit | Jakub Jursa proposed openstack/openstack-ansible-os_neutron stable/queens: added new plugin type - asap2 https://review.openstack.org/563546 | 09:30 |
| evrardjp | yes we can remove it from group_vars too | 09:31 |
| evrardjp | ok | 09:31 |
| openstackgerrit | Jakub Jursa proposed openstack/openstack-ansible-os_neutron stable/queens: added new plugin type - asap2 https://review.openstack.org/563546 | 09:31 |
| evrardjp | so | 09:31 |
| evrardjp | let's think even further | 09:31 |
| evrardjp | how can we improve the process to make it idempotent | 09:31 |
| evrardjp | if we make it idempotent accross the board, with or without the change, it makes our life even easier | 09:32 |
| odyssey4me | evrardjp ok, let me propose something and let's see how it goes | 09:32 |
| openstackgerrit | Jakub Jursa proposed openstack/openstack-ansible-os_neutron master: added new plugin type - asap2 https://review.openstack.org/563547 | 09:33 |
| openstackgerrit | fupingxie proposed openstack/openstack-ansible-os_tacker master: Do not log passwords https://review.openstack.org/563548 | 09:33 |
| evrardjp | odyssey4me: if we run the repo_build and the repo_server as isolated, we should be good for idempotency, shouldn't we? | 09:33 |
| odyssey4me | realistically this juggle only needs to happen on the greenfield deploy, before the repo is present - but it also needs to cater for when the repo disappears (when you're trying to replace your repo container) | 09:33 |
| evrardjp | odyssey4me: what do we have that requires pip before repo? | 09:34 |
| evrardjp | sorry to put everything on the table | 09:34 |
| odyssey4me | I'm trying not to overthink it because this will all change with the new way the repo is built... so I'm really only concerned with putting on a band aid. | 09:34 |
| *** yifei has quit IRC | 09:34 | |
| evrardjp | odyssey4me: the band aid would be the set_fact at all times | 09:34 |
| evrardjp | I think | 09:34 |
| *** hamza21 has joined #openstack-ansible | 09:35 | |
| odyssey4me | at this time we install things on the host before the repo is present - although I think some of hwoarang's patches might reduce that footprint | 09:35 |
| evrardjp | it would require a change in pip_install role to use a different var name, or the other way around an interface change and user change to have a different variable name in group vars. | 09:35 |
| evrardjp | I'd say the easier is to change pip_install :p | 09:36 |
| odyssey4me | yeah, I know - I'm trying to minimise the changes - lemme take a crack at it | 09:36 |
| odyssey4me | ah, no - pip install has too many consequences involved | 09:36 |
| openstackgerrit | fupingxie proposed openstack/openstack-ansible-os_congress master: Do not log passwords https://review.openstack.org/563549 | 09:37 |
| evrardjp | why not? if we change the pip_default_index from pip install defaults/main.yml to pip_install_default_index which takes the same value, and we set fact appropriately? | 09:37 |
| evrardjp | yeah maybe | 09:37 |
| evrardjp | I think pip_default_index is used in many places, and probably shouldn't be modified as interface | 09:38 |
| evrardjp | that's why i wanted to edit pip_install default/main.yml value to be scoped | 09:38 |
| odyssey4me | evrardjp I'm not really sure what you mean | 09:39 |
| odyssey4me | changing something in pip_install doesn't actually solve this problem to me - we're overriding pip_install's values | 09:39 |
| odyssey4me | it'll only help if we move this test mechanism into the role, which I don't think we should do | 09:40 |
| evrardjp | nope nope I meant... 1) keep pip_default_index as user level override 2) use set_facts to set pip_install_default_index (and other indexes for other uses cases like repo_server/repo_build) in the common task 3) call the common tasks inside repo use | 09:44 |
| evrardjp | the 2) would happen at all times, not conditionally, but the value set would depend on the http check | 09:44 |
| evrardjp | good thing is with that set in place, we can remove the group vars, by taking safe defaults that can be overriden in the common series of tasks, later. | 09:46 |
| odyssey4me | evrardjp let me push up a patch and you see what you think | 09:46 |
| odyssey4me | I think my approach is simple and will do the trick without creating any more tech debt. | 09:47 |
| evrardjp | odyssey4me: I can also do a patch on the side, so we compare the two if you want | 09:47 |
| evrardjp | ok | 09:47 |
| odyssey4me | sure, if you like | 09:47 |
| evrardjp | I will wait for yours first, and then we can discuss in the review | 09:47 |
| *** bertys has joined #openstack-ansible | 09:51 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Ensure that repo checks use the right values https://review.openstack.org/563552 | 09:53 |
| *** rmart04 has joined #openstack-ansible | 09:53 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Ensure that repo checks use the right values https://review.openstack.org/563552 | 09:54 |
| odyssey4me | evrardjp ^ that should do it | 09:54 |
| evrardjp | ahah reverting my reasoning | 09:55 |
| evrardjp | with using the backup instead of the main var | 09:55 |
| odyssey4me | yes, changing the main var will likely result in other side effects - this is simple | 09:56 |
| evrardjp | odyssey4me: reviewed | 09:58 |
| evrardjp | but that's exactly what I meant :) | 09:59 |
| evrardjp | I was just heading a little further | 09:59 |
| evrardjp | removing the group_vars | 09:59 |
| evrardjp | but that's fine | 09:59 |
| evrardjp | that's the minimum patch :) | 10:00 |
| odyssey4me | ok, that could work as an idea - but I think w'll find all sort of side effects | 10:00 |
| evrardjp | yes let's just stick with this | 10:00 |
| odyssey4me | I'd like to do that once we have the new builds happening right - the plan was then to move constraints setting into pip.conf, which will mean we can ditch the group_vars for all this | 10:00 |
| evrardjp | mmm | 10:01 |
| evrardjp | isn't it a problem? | 10:01 |
| evrardjp | for things that don't use it? | 10:01 |
| evrardjp | oh you mean conditionally use constraints into pip.conf depending on the project? | 10:01 |
| evrardjp | I don't think it's a simple idea as that, if ppl put everything into one machine, or consolidate multiple projects that makes it harder to work with | 10:06 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Ensure that repo checks use the right values https://review.openstack.org/563552 | 10:06 |
| openstackgerrit | Merged openstack/openstack-ansible-lxc_hosts master: Avoid mixing pip and distribution packages. https://review.openstack.org/562280 | 10:06 |
| niraj_singh | evrardjp: Sorry to interrupt you. But while installing cinder playbook. it is waiting for long time for the task [Perform online data migration]. is there any issue? | 10:07 |
| odyssey4me | evrardjp yeah, I don't think that's necessarily a set path - but it's an idea which I'll explore later | 10:07 |
| evrardjp | that set fact at L104 is .... | 10:07 |
| odyssey4me | niraj_singh that would depend on the size of your DB | 10:08 |
| odyssey4me | evrardjp yeah, it's not great | 10:08 |
| evrardjp | odyssey4me: http://nameofscience.org/wp-content/uploads/2017/04/sam-l-jackson-hold-on-to-your-butts-hold-onto-your-butts.jpg | 10:08 |
| odyssey4me | lemme see if I can make it easier to read | 10:08 |
| *** nicolasbock has joined #openstack-ansible | 10:09 | |
| evrardjp | odyssey4me: it's fine for me | 10:09 |
| evrardjp | the easiest is doing the cleanup of the url to prevent the whole replace inside the ternary | 10:09 |
| evrardjp | ternary this or that and have "this" and "that" already defined in advance in a previous set_fact | 10:10 |
| evrardjp | set_fact is fast anyway | 10:10 |
| evrardjp | odyssey4me: niraj_singh I am afraid you're hitting another issue that makes your cpu go to 11. | 10:11 |
| evrardjp | odyssey4me: niraj_singh is trying a new role IIRC | 10:11 |
| evrardjp | niraj_singh: check your cinder-backup isn't taking 100% of cpu maybe? | 10:11 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Ensure that repo checks use the right values https://review.openstack.org/563552 | 10:11 |
| evrardjp | odyssey4me: that's far more readable | 10:12 |
| evrardjp | using a good variable name helps :) | 10:12 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Ensure that repo checks use the right values https://review.openstack.org/563552 | 10:12 |
| evrardjp | let's see how it goes | 10:12 |
| odyssey4me | ok, so that should hopefully work - yeah, let's see how it goes | 10:13 |
| evrardjp | I am always scared with the >- | 10:13 |
| odyssey4me | why? | 10:13 |
| evrardjp | I am always confused with that | 10:14 |
| evrardjp | I can't remember what >- is removing | 10:14 |
| evrardjp | vs | | 10:14 |
| odyssey4me | | makes every line have a newline at the end | 10:14 |
| odyssey4me | +- removes the last newline | 10:14 |
| evrardjp | - is removing the nl somewheres | 10:14 |
| evrardjp | yes last line | 10:14 |
| odyssey4me | }- | 10:14 |
| odyssey4me | dammit | 10:14 |
| odyssey4me | |- | 10:14 |
| odyssey4me | > makes everything a single line | 10:14 |
| odyssey4me | but it still has a \n at the end | 10:15 |
| evrardjp | yes that's it | 10:15 |
| odyssey4me | >- makes it all a single line and removes the \n at the end | 10:15 |
| evrardjp | I can't remember | 10:15 |
| evrardjp | > or | | 10:15 |
| evrardjp | I remember the {%- -%} | 10:15 |
| odyssey4me | - is always the whitespace/return character remover | 10:15 |
| evrardjp | yeah | 10:15 |
| evrardjp | that's the easy part to remember | 10:15 |
| evrardjp | the > or | is always confusing me | 10:15 |
| evrardjp | | is multi line? | 10:16 |
| evrardjp | so not removing intermediary new lines | 10:16 |
| odyssey4me | yes, it makes each line have \n at the end of it | 10:16 |
| evrardjp | ok | 10:16 |
| evrardjp | that's super confusing for me :p > or | is not very explicit language term | 10:16 |
| odyssey4me | it's good for shell tasks where you're doing a little script in it | 10:16 |
| evrardjp | I am using it for like generating the network interfaces and things like that | 10:16 |
| evrardjp | I should remember | 10:17 |
| evrardjp | I never remember. | 10:17 |
| evrardjp | maybe now that we have talked about it for the xth time, I will remember. | 10:17 |
| *** armaan has quit IRC | 10:17 | |
| odyssey4me | :) I find it useful, so using it has caused me to remember | 10:17 |
| evrardjp | ? & * goes fine, but > | nope. Brain doesn't want it. | 10:17 |
| *** armaan has joined #openstack-ansible | 10:18 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Ensure that the repo servers also use the pypi cache https://review.openstack.org/563114 | 10:18 |
| evrardjp | reviewed ^ | 10:19 |
| evrardjp | odyssey4me: oh I didn't know that: https://pip.pypa.io/en/stable/user_guide/#config-file | 10:21 |
| evrardjp | you can have per venv pip.conf | 10:21 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Ensure that the repo servers also use the pypi cache https://review.openstack.org/563114 | 10:21 |
| odyssey4me | evrardjp yep | 10:21 |
| evrardjp | that's what we should do in the openstack bits | 10:22 |
| odyssey4me | yes, that is something I'd like to try out | 10:22 |
| evrardjp | that makes more sense | 10:22 |
| evrardjp | cool | 10:22 |
| *** armaan has quit IRC | 10:22 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Remove default pip_links value https://review.openstack.org/549012 | 10:24 |
| evrardjp | that or we do "user" requirements, becoming the user of the service (keystone, gnocchi, whatever) | 10:24 |
| odyssey4me | yeah, not a bad idea too | 10:25 |
| odyssey4me | it will all depend on how pip merges the configs - I'll do more experimenting once we're further along | 10:25 |
| odyssey4me | I'd prefer to keep the way it works as obvious as possible, rather than do obscure config merges. | 10:26 |
| evrardjp | jmccrory: could you ping me when you're available to discuss user configuration of inventory, please? | 10:26 |
| *** pcaruana has quit IRC | 10:26 | |
| evrardjp | odyssey4me: yes, that's why I think having a convention of having the pip.conf in the venv is simple. | 10:26 |
| odyssey4me | evrardjp mind reviewing this one? https://review.openstack.org/563095 | 10:28 |
| odyssey4me | evrardjp yeah, if we actually totally eradicate the use of pip on the host and only use the venv, then that does seem like the most obvious thing | 10:29 |
| evrardjp | yes that's what I want to do, eradicate the host configurations, woot! | 10:32 |
| odyssey4me | ah, awesome - the openstack-ansible-upgrade-aio-ubuntu-xenial job for stable/ocata (ie the n->o upgrade) is working again | 10:32 |
| evrardjp | thanks for the work odyssey4me | 10:33 |
| evrardjp | massive work there | 10:33 |
| evrardjp | d34dh0r53: should learn from that :p | 10:33 |
| *** pcaruana has joined #openstack-ansible | 10:33 | |
| odyssey4me | odd, looks like the deploy of stable/ocata failed due to bad ssl for percona: http://logs.openstack.org/periodic/git.openstack.org/openstack/openstack-ansible/stable/ocata/openstack-ansible-deploy-aio-ubuntu-xenial/2c43ad0/job-output.txt.gz#_2018-04-23_06_46_44_744439 | 10:34 |
| odyssey4me | the ceph deploy passed though, so perhaps a transient issue | 10:35 |
| *** armaan has joined #openstack-ansible | 10:35 | |
| evrardjp | let me check real quick | 10:36 |
| odyssey4me | pike, queens, master deploys for ubuntu all working | 10:36 |
| odyssey4me | queens->master upgrades failing because https://review.openstack.org/563111 had not yet merged | 10:38 |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: [RFC] Add support for using distribution packages for OpenStack services https://review.openstack.org/560308 | 10:39 |
| odyssey4me | note that I've backported that patch | 10:39 |
| *** nicolasbock has quit IRC | 10:39 | |
| odyssey4me | also, https://review.openstack.org/563518 should speed up the repo builds (also a backport) | 10:40 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/pike: bootstrap-host: Set repo build pip default mirror when in openstack-ci https://review.openstack.org/563565 | 10:40 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/ocata: bootstrap-host: Set repo build pip default mirror when in openstack-ci https://review.openstack.org/563566 | 10:41 |
| evrardjp | omg I curled a gzip file instead of checking its headers. | 10:43 |
| evrardjp | my whole session is busted. | 10:43 |
| openstackgerrit | Merged openstack/openstack-ansible-pip_install master: Ensure that pip.conf templates variables properly https://review.openstack.org/563095 | 10:44 |
| odyssey4me | haha, that happens | 10:45 |
| odyssey4me | evrardjp we're failing the idempotence test for pip_install on stable queens... and to fix it will change how the role executes quite a bit which I'm not sure we should backport | 10:45 |
| odyssey4me | I'm thinking that we should disable the idempotence test for queens and backwards, but do it moving forward. | 10:46 |
| evrardjp | I am not fond of changing stable branches | 10:46 |
| evrardjp | that sounds reasonable if there is too much work | 10:47 |
| odyssey4me | yeah, that's why I think the test should be removed given that it never really worked | 10:47 |
| odyssey4me | I don't think we should port https://review.openstack.org/562243 and https://review.openstack.org/562188 back | 10:47 |
| evrardjp | I checked on empty ubuntu for the ca, it should be alright (so I'd say the ocata is failure is temporarily happening, like you said ) | 10:47 |
| odyssey4me | yeah, I suspect that it's an SNI failure | 10:48 |
| evrardjp | this: https://review.openstack.org/#/c/562243/7 is okay to backport to me, but it means we have to backport the source/package install too I guess | 10:49 |
| evrardjp | which I'd prefer to avoid myself | 10:49 |
| evrardjp | but we should maybe discuss this in the community | 10:49 |
| odyssey4me | if we port https://review.openstack.org/562243 back, then the idempotence test should work. | 10:49 |
| evrardjp | yeah but it won't cleanly backport, that's my problem :p | 10:50 |
| *** nicolasbock has joined #openstack-ansible | 10:50 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_install stable/queens: Disable idempotence test https://review.openstack.org/563567 | 10:51 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_install stable/pike: Disable idempotence test https://review.openstack.org/563568 | 10:52 |
| evrardjp | odyssey4me: I am adding https://review.openstack.org/#/c/563567/1 into the community meeting agenda | 10:55 |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-lxc_hosts master: Revert "Install SELinux packages asynchronously" https://review.openstack.org/563569 | 10:56 |
| odyssey4me | hmm, those pip installs are failing - I think I know why, but will build a host to verify | 11:13 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build master: Allow repo host packages to be upgraded before repo build https://review.openstack.org/563578 | 11:26 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build master: Add pip_install_options to pip install tasks https://review.openstack.org/563579 | 11:28 |
| *** ansmith has quit IRC | 11:28 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Ensure that repo checks use the right values https://review.openstack.org/563552 | 11:28 |
| evrardjp | one more can of worms :) | 11:30 |
| odyssey4me | yeah, but we are slowly figuring out how to use the pypiserver correctly | 11:30 |
| odyssey4me | up until now, it's not been used correctly - so as we lock it down better, we see failures and have to clean them up | 11:31 |
| evrardjp | I am travelling now, so I will be offline. Have a good day everyone! | 11:32 |
| openstackgerrit | Merged openstack/openstack-ansible-repo_build stable/queens: Constrain the pip/setuptools/wheel versions in the venvs https://review.openstack.org/562346 | 11:41 |
| *** deng_ has quit IRC | 11:43 | |
| openstackgerrit | Andy McCrae proposed openstack/ansible-config_template master: [WIP] Try to make py3 test fail https://review.openstack.org/563585 | 11:46 |
| *** jcosmao has left #openstack-ansible | 11:50 | |
| *** yifei has joined #openstack-ansible | 11:53 | |
| *** pcaruana has quit IRC | 11:54 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Ensure that the repo servers also use the pypi cache https://review.openstack.org/563114 | 11:56 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Remove default pip_links value https://review.openstack.org/549012 | 11:56 |
| openstackgerrit | Andy McCrae proposed openstack/ansible-config_template master: [WIP] Try to make py3 test fail https://review.openstack.org/563585 | 11:57 |
| *** yifei has quit IRC | 11:58 | |
| hwoarang | hmm seems like there are some issues with galera repos | 12:01 |
| hwoarang | again... | 12:01 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build master: Allow repo host packages to be upgraded before repo build https://review.openstack.org/563578 | 12:02 |
| *** sep has quit IRC | 12:06 | |
| *** sep has joined #openstack-ansible | 12:06 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_install stable/pike: Disable the pip version check https://review.openstack.org/563593 | 12:07 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_install stable/ocata: Disable the pip version check https://review.openstack.org/563594 | 12:07 |
| *** pcaruana has joined #openstack-ansible | 12:10 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_install stable/queens: Ensure that pip.conf templates variables properly https://review.openstack.org/563597 | 12:11 |
| odyssey4me | hwoarang yeah, I noticed that too | 12:11 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_install stable/pike: Ensure that pip.conf templates variables properly https://review.openstack.org/563599 | 12:13 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_install stable/ocata: Ensure that pip.conf templates variables properly https://review.openstack.org/563600 | 12:13 |
| *** dave-mccowan has joined #openstack-ansible | 12:49 | |
| *** kstev has joined #openstack-ansible | 12:49 | |
| *** dmsimard|off is now known as dmsimard | 12:51 | |
| *** kstev1 has joined #openstack-ansible | 12:54 | |
| *** kstev has quit IRC | 12:56 | |
| *** aludwar has joined #openstack-ansible | 12:56 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_install stable/pike: Ensure that pip.conf templates variables properly https://review.openstack.org/563599 | 12:57 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_install stable/pike: Disable the pip version check https://review.openstack.org/563593 | 12:57 |
| *** ansmith has joined #openstack-ansible | 12:57 | |
| mgariepy | morning everyone | 12:59 |
| *** kstev1 has quit IRC | 12:59 | |
| openstackgerrit | Merged openstack/openstack-ansible-pip_install stable/pike: Disable idempotence test https://review.openstack.org/563568 | 13:04 |
| d34dh0r53 | lol | 13:05 |
| *** sar has quit IRC | 13:10 | |
| *** vnogin has joined #openstack-ansible | 13:13 | |
| *** esberglu has joined #openstack-ansible | 13:15 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_congress master: Do not log passwords https://review.openstack.org/563549 | 13:21 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_congress stable/queens: Do not log passwords https://review.openstack.org/563618 | 13:22 |
| *** cmart has joined #openstack-ansible | 13:23 | |
| *** vnogin has quit IRC | 13:24 | |
| *** lvdombrkr has joined #openstack-ansible | 13:30 | |
| *** vnogin has joined #openstack-ansible | 13:38 | |
| openstackgerrit | Andy McCrae proposed openstack/ansible-config_template master: Fix py3 unicode issue https://review.openstack.org/563585 | 13:44 |
| *** kstev has joined #openstack-ansible | 13:47 | |
| *** vnogin has quit IRC | 13:51 | |
| *** dxiri has joined #openstack-ansible | 13:51 | |
| *** rmart04 has quit IRC | 13:51 | |
| *** rmart04 has joined #openstack-ansible | 13:52 | |
| *** vnogin has joined #openstack-ansible | 13:52 | |
| *** hamza21 has quit IRC | 13:52 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_congress stable/queens: Do not log passwords https://review.openstack.org/563618 | 13:55 |
| *** vnogin has quit IRC | 13:57 | |
| *** rmart04 has quit IRC | 13:57 | |
| *** rmart04 has joined #openstack-ansible | 13:57 | |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-os_tempest master: Use distribution packages for Tempest dependencies https://review.openstack.org/562606 | 14:03 |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: [RFC] Add support for using distribution packages for OpenStack services https://review.openstack.org/560308 | 14:09 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build master: Allow repo host packages to be upgraded before repo build https://review.openstack.org/563578 | 14:09 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Ensure new container facts are updated https://review.openstack.org/563641 | 14:15 |
| *** admin0 has joined #openstack-ansible | 14:16 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Ensure facts are updated for new containers https://review.openstack.org/563641 | 14:17 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Ensure facts are updated for new containers https://review.openstack.org/563641 | 14:18 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Ensure that the repo servers also use the pypi cache https://review.openstack.org/563114 | 14:25 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Remove default pip_links value https://review.openstack.org/549012 | 14:25 |
| *** lvdombrkr89 has joined #openstack-ansible | 14:37 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_congress stable/queens: Do not log passwords https://review.openstack.org/563618 | 14:38 |
| *** lvdombrkr has quit IRC | 14:40 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_congress stable/queens: Remove unused test-repo-clone script https://review.openstack.org/563649 | 14:40 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_congress stable/queens: Do not log passwords https://review.openstack.org/563618 | 14:40 |
| *** lvdombrkr89 has quit IRC | 14:47 | |
| *** mma has quit IRC | 14:48 | |
| *** cjloader has joined #openstack-ansible | 15:01 | |
| *** cjloader has quit IRC | 15:02 | |
| *** cjloader has joined #openstack-ansible | 15:02 | |
| *** DanyC has quit IRC | 15:02 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_congress master: Only replace python when re-initializing the venv https://review.openstack.org/562167 | 15:04 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_congress stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/563656 | 15:04 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_congress stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/563656 | 15:05 |
| *** mwarad has quit IRC | 15:05 | |
| *** rmart04 has quit IRC | 15:06 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Update Ansible to 2.4.4.0 https://review.openstack.org/563660 | 15:10 |
| openstackgerrit | Andy Smith proposed openstack/openstack-ansible-os_nova master: (WIP) Update to use oslo.messaging services for RPC and Notify https://review.openstack.org/558620 | 15:13 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Update Ansible to 2.4.4.0 https://review.openstack.org/563663 | 15:14 |
| openstackgerrit | German Eichberger proposed openstack/openstack-ansible-os_octavia master: Adds certificate generation https://review.openstack.org/553630 | 15:14 |
| hwoarang | odyssey4me: evrardjp one more look on https://review.openstack.org/#/c/560308/ please | 15:16 |
| hwoarang | now opensuse passes too | 15:16 |
| hwoarang | centos is a mystery | 15:16 |
| mgariepy | http://logs.openstack.org/08/560308/15/check/openstack-ansible-keystone-distro_install-centos-7/d851a06/logs/openstack/keystone1/keystone/keystone.log.txt.gz | 15:18 |
| hwoarang | aha | 15:18 |
| *** pcaruana has quit IRC | 15:19 | |
| hwoarang | thank you mgariepy | 15:19 |
| mgariepy | you are welcome | 15:20 |
| *** pylot has quit IRC | 15:21 | |
| odyssey4me | hwoarang that looks pretty good, I think | 15:22 |
| openstackgerrit | Andy McCrae proposed openstack/ansible-config_template master: Fix py3 unicode issue https://review.openstack.org/563585 | 15:22 |
| cloudnull | mornings | 15:26 |
| odyssey4me | o/ cloudnull - thanks for all those reviews on the w/end... it really helps things keep moving | 15:26 |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: [RFC] Add support for using distribution packages for OpenStack services https://review.openstack.org/560308 | 15:28 |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: tasks: Ensure /etc/keystone/credential-keys is created on time https://review.openstack.org/563667 | 15:28 |
| cloudnull | odyssey4me: no problem. Sorry I've largely been AFK this last week & a half. | 15:30 |
| cloudnull | anything pressing we need in / reviewed ? | 15:31 |
| cloudnull | or needs working on? | 15:31 |
| spotz | cloudnull: oooh review mine!:) https://review.openstack.org/#/c/551316/ :) | 15:32 |
| *** sar has joined #openstack-ansible | 15:32 | |
| cloudnull | spotz: done. | 15:32 |
| spotz | cloudnull: danke | 15:32 |
| odyssey4me | for the moment I've just been trying to figure out the missing bits to get the pypiserver working as it was supposed to - so I have a string of patches in for that and reviews of those are welcome... otherwise, depleting the bug queue would be nice :) | 15:32 |
| spotz | I'll try to hit reviews tomorrow night. Was doing my git and gerrit slide deck for summit this weekend. Was out of date:( | 15:33 |
| cloudnull | anyone want to review https://review.openstack.org/#/c/558227/ | 15:33 |
| spotz | looking cloudnull | 15:33 |
| spotz | done, I like easy ones:) | 15:34 |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-pip_install master: defaults: Add virtualenv to default pip packages https://review.openstack.org/563670 | 15:34 |
| *** chyka has joined #openstack-ansible | 15:38 | |
| *** eumel8 has quit IRC | 15:41 | |
| openstackgerrit | Paul Belanger proposed openstack/ansible-hardening stable/queens: Replace Fedora 26 with 27 https://review.openstack.org/563675 | 15:46 |
| pabelanger | Hmm, we need to backport changes to fedora-27 and debian-stable to ansible-hardening branches. How do you want to handle that? | 15:47 |
| pabelanger | I think it might just be easier to stop running testing on those branches, over backporting | 15:48 |
| *** mma has joined #openstack-ansible | 15:49 | |
| *** bertys has quit IRC | 15:49 | |
| *** mma has quit IRC | 15:53 | |
| openstackgerrit | Paul Belanger proposed openstack/ansible-hardening stable/queens: Remove fedora-26 / debian-jessie from testing https://review.openstack.org/563682 | 15:54 |
| pabelanger | mhayden: evrardjp: thoughts?^ | 15:54 |
| pabelanger | we need to clean up stable branches now that these nodes will be removed | 15:54 |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-openstack_hosts master: tasks: Use PIP distribution packages for openSUSE and Ubuntu https://review.openstack.org/563684 | 15:55 |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: tasks: Ensure /etc/keystone/credential-keys is created on time https://review.openstack.org/563667 | 15:56 |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: [RFC] Add support for using distribution packages for OpenStack services https://review.openstack.org/560308 | 15:56 |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible master: Use production like interfaces in the gate https://review.openstack.org/553636 | 15:58 |
| *** throwsb1 has joined #openstack-ansible | 15:59 | |
| cloudnull | pabelanger: on https://review.openstack.org/#/c/563682 - do we have a different debian / fedora image to test against? | 15:59 |
| *** admin0 has quit IRC | 16:01 | |
| *** mma has joined #openstack-ansible | 16:03 | |
| *** gyee has joined #openstack-ansible | 16:03 | |
| *** armaan has quit IRC | 16:03 | |
| *** armaan has joined #openstack-ansible | 16:04 | |
| *** armaan has quit IRC | 16:09 | |
| *** armaan has joined #openstack-ansible | 16:09 | |
| *** mwarad has joined #openstack-ansible | 16:12 | |
| *** pcaruana has joined #openstack-ansible | 16:13 | |
| *** vnogin has joined #openstack-ansible | 16:13 | |
| *** tnogisto has quit IRC | 16:13 | |
| pabelanger | cloudnull: fedora-27 and debian-stable are the new ones | 16:15 |
| pabelanger | cloudnull: eventually fedora-stable / fedora-latest once fedora-28 comes out | 16:15 |
| pabelanger | which will make managing them on stable branches a little more easier | 16:15 |
| *** DanyC has joined #openstack-ansible | 16:19 | |
| *** DanyC has quit IRC | 16:24 | |
| openstackgerrit | Merged openstack/openstack-ansible-tests master: Resolve common roles required for test https://review.openstack.org/558227 | 16:24 |
| *** vnogin has quit IRC | 16:29 | |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-openstack_hosts master: tasks: Use PIP distribution packages for openSUSE and Ubuntu https://review.openstack.org/563684 | 16:31 |
| *** DanyC has joined #openstack-ansible | 16:34 | |
| *** DanyC_ has joined #openstack-ansible | 16:35 | |
| *** sxc731 has joined #openstack-ansible | 16:37 | |
| *** epalper has quit IRC | 16:38 | |
| *** DanyC has quit IRC | 16:39 | |
| *** exodusftw has quit IRC | 16:43 | |
| *** mwarad_ has joined #openstack-ansible | 16:48 | |
| *** mwarad has quit IRC | 16:49 | |
| *** exodusftw has joined #openstack-ansible | 16:50 | |
| *** mwarad_ has quit IRC | 16:53 | |
| *** mwarad has joined #openstack-ansible | 16:56 | |
| openstackgerrit | German Eichberger proposed openstack/openstack-ansible-os_octavia master: Adds certificate generation https://review.openstack.org/553630 | 17:02 |
| *** DanyC has joined #openstack-ansible | 17:04 | |
| *** sar has quit IRC | 17:05 | |
| *** DanyC_ has quit IRC | 17:07 | |
| openstackgerrit | Andy Smith proposed openstack/openstack-ansible-os_cinder master: (WIP) Update to use oslo.messaging service for RPC and Notify https://review.openstack.org/560574 | 17:10 |
| *** mwarad has quit IRC | 17:10 | |
| *** sxc731 has quit IRC | 17:24 | |
| openstackgerrit | Merged openstack/openstack-ansible-galera_server master: Avoid mixing pip and distribution packages. https://review.openstack.org/562262 | 17:27 |
| *** DanyC has quit IRC | 17:30 | |
| *** DanyC has joined #openstack-ansible | 17:30 | |
| *** chyka_ has joined #openstack-ansible | 17:36 | |
| openstackgerrit | Andy Smith proposed openstack/openstack-ansible-os_keystone master: (WIP) Update to use oslo.messaging service for RPC and Notify https://review.openstack.org/562026 | 17:37 |
| *** chyka has quit IRC | 17:39 | |
| *** cmart has quit IRC | 17:40 | |
| *** DanyC has quit IRC | 17:40 | |
| *** chyka_ has quit IRC | 17:41 | |
| openstackgerrit | Antony Messerli proposed openstack/openstack-ansible master: Only run zfs command if command is present https://review.openstack.org/562010 | 17:45 |
| *** chyka has joined #openstack-ansible | 17:55 | |
| *** chyka has quit IRC | 18:00 | |
| *** poopcat has joined #openstack-ansible | 18:01 | |
| *** cmart has joined #openstack-ansible | 18:19 | |
| *** hamza21 has joined #openstack-ansible | 18:23 | |
| *** sar has joined #openstack-ansible | 18:53 | |
| *** DanyC has joined #openstack-ansible | 18:55 | |
| *** poopcat has quit IRC | 19:04 | |
| *** poopcat has joined #openstack-ansible | 19:06 | |
| *** chyka has joined #openstack-ansible | 19:10 | |
| openstackgerrit | Taseer Ahmed proposed openstack/openstack-ansible master: Improve Congress scenario testing https://review.openstack.org/560839 | 19:17 |
| *** DanyC has quit IRC | 19:17 | |
| *** cmart has quit IRC | 19:27 | |
| pabelanger | any more thoughts on https://review.openstack.org/563682/ ? | 19:29 |
| pabelanger | removal of fedora-26 / debian-jessie on stable branches | 19:29 |
| odyssey4me | pabelanger evrardjp is travelling today, he should be able to respond tomorrow | 19:32 |
| pabelanger | wfm | 19:37 |
| *** flaviosr has quit IRC | 19:39 | |
| *** flaviosr has joined #openstack-ansible | 19:40 | |
| openstackgerrit | Andy Smith proposed openstack/openstack-ansible-tests master: (WIP) Update to use oslo.messaging services for RPC and Notify https://review.openstack.org/559127 | 19:44 |
| *** cmart has joined #openstack-ansible | 19:54 | |
| *** dave-mccowan has quit IRC | 20:14 | |
| *** armaan has quit IRC | 20:14 | |
| *** sar has quit IRC | 20:26 | |
| openstackgerrit | Andy Smith proposed openstack/openstack-ansible-os_neutron master: (WIP) Update to use oslo.messaging service for RPC and Notify https://review.openstack.org/563758 | 20:35 |
| *** ansmith has quit IRC | 20:43 | |
| *** mma has quit IRC | 20:50 | |
| *** idlemind has quit IRC | 20:59 | |
| openstackgerrit | German Eichberger proposed openstack/openstack-ansible-os_octavia master: Adds certificate generation https://review.openstack.org/553630 | 21:18 |
| *** DanyC has joined #openstack-ansible | 21:22 | |
| *** vnogin has joined #openstack-ansible | 21:30 | |
| *** hamza21 has quit IRC | 21:32 | |
| *** vnogin has quit IRC | 21:35 | |
| *** ansmith has joined #openstack-ansible | 21:35 | |
| openstackgerrit | Merged openstack/openstack-ansible master: Only run zfs command if command is present https://review.openstack.org/562010 | 21:38 |
| *** DanyC has quit IRC | 21:41 | |
| *** exodusftw has quit IRC | 21:53 | |
| *** throwsb1 has quit IRC | 21:57 | |
| *** exodusftw has joined #openstack-ansible | 21:59 | |
| *** sawblade_ has joined #openstack-ansible | 22:10 | |
| *** sawblade6 has quit IRC | 22:14 | |
| *** lbragstad has quit IRC | 22:18 | |
| *** esberglu has quit IRC | 22:18 | |
| *** esberglu has joined #openstack-ansible | 22:19 | |
| *** esberglu has quit IRC | 22:21 | |
| *** threestrands has joined #openstack-ansible | 22:41 | |
| *** threestrands has quit IRC | 22:41 | |
| *** threestrands has joined #openstack-ansible | 22:41 | |
| *** cjloader has quit IRC | 22:47 | |
| *** kstev has quit IRC | 22:58 | |
| *** tosky has quit IRC | 23:54 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
