*** markvoelker has joined #openstack-ansible | 00:03 | |
*** markvoelker has quit IRC | 00:07 | |
*** masber has joined #openstack-ansible | 00:20 | |
*** markvoelker has joined #openstack-ansible | 00:21 | |
*** masuberu has quit IRC | 00:22 | |
*** dxiri has joined #openstack-ansible | 00:25 | |
*** thorst has quit IRC | 00:29 | |
*** smatzek has quit IRC | 00:31 | |
*** esberglu has quit IRC | 00:40 | |
*** yifei has joined #openstack-ansible | 00:44 | |
*** yifei has quit IRC | 00:48 | |
*** yifei has joined #openstack-ansible | 00:51 | |
*** dave-mccowan has quit IRC | 00:52 | |
*** dave-mccowan has joined #openstack-ansible | 00:55 | |
*** markvoelker has quit IRC | 00:55 | |
*** chyka has joined #openstack-ansible | 00:59 | |
*** hw_wutianwei has joined #openstack-ansible | 00:59 | |
*** mrtenio has quit IRC | 01:02 | |
*** vnogin has quit IRC | 01:02 | |
*** chyka has quit IRC | 01:03 | |
prometheanfire | so, can I get https://review.openstack.org/507189 this merged yet? | 01:03 |
---|---|---|
*** snowman4839 has quit IRC | 01:09 | |
*** markvoelker has joined #openstack-ansible | 01:09 | |
*** snowman4839 has joined #openstack-ansible | 01:09 | |
*** kiennt26 has joined #openstack-ansible | 01:11 | |
*** acormier has joined #openstack-ansible | 01:11 | |
*** acormier has quit IRC | 01:13 | |
*** kiennt26 has quit IRC | 01:13 | |
*** acormier has joined #openstack-ansible | 01:13 | |
*** markvoelker has quit IRC | 01:14 | |
*** woodard_ has joined #openstack-ansible | 01:15 | |
*** woodard has quit IRC | 01:15 | |
*** dave-mccowan has quit IRC | 01:15 | |
*** kiennt26 has joined #openstack-ansible | 01:16 | |
*** markvoelker has joined #openstack-ansible | 01:18 | |
*** markvoelker has quit IRC | 01:23 | |
*** dave-mccowan has joined #openstack-ansible | 01:23 | |
*** markvoelker has joined #openstack-ansible | 01:27 | |
*** markvoelker has quit IRC | 01:32 | |
*** markvoelker has joined #openstack-ansible | 01:37 | |
*** markvoelker has quit IRC | 01:41 | |
*** markvoelker has joined #openstack-ansible | 01:46 | |
*** markvoelker has quit IRC | 01:50 | |
spotz | prometheanfire: There's anote that keystone is waiting on pike on the etherpad evrardjp and odyssey4me were working off of for fixing gate stuff. Might check with them in the morning | 01:51 |
spotz | grr waiting on master prometheanfire | 01:51 |
*** acormier has quit IRC | 01:52 | |
prometheanfire | k | 01:52 |
prometheanfire | thanks | 01:52 |
*** acormier has joined #openstack-ansible | 01:52 | |
*** thorst has joined #openstack-ansible | 01:53 | |
*** thorst has quit IRC | 01:54 | |
*** markvoelker has joined #openstack-ansible | 01:55 | |
*** acormier has quit IRC | 01:57 | |
*** snowman4839 has quit IRC | 01:58 | |
*** snowman4839 has joined #openstack-ansible | 01:59 | |
*** dave-mccowan has quit IRC | 02:04 | |
cloudnull | evenings | 02:11 |
*** markvoelker has quit IRC | 02:16 | |
*** markvoelker has joined #openstack-ansible | 02:17 | |
openstackgerrit | Merged openstack/openstack-ansible stable/pike: Include the Ansible version variable when installing ara https://review.openstack.org/513434 | 02:18 |
openstackgerrit | Merged openstack/openstack-ansible master: Ensure container facts are set on container create https://review.openstack.org/507191 | 02:18 |
*** acormier has joined #openstack-ansible | 02:20 | |
*** acormier has quit IRC | 02:25 | |
*** qcrivera has joined #openstack-ansible | 02:25 | |
*** markvoelker has quit IRC | 02:29 | |
*** qcrivera has quit IRC | 02:32 | |
*** markvoelker has joined #openstack-ansible | 02:43 | |
*** markvoelker has quit IRC | 02:47 | |
*** acormier has joined #openstack-ansible | 02:54 | |
*** markvoelker has joined #openstack-ansible | 03:01 | |
*** markvoelker has quit IRC | 03:05 | |
*** markvoelker has joined #openstack-ansible | 03:10 | |
*** markvoelker has quit IRC | 03:15 | |
*** gkadam-away has joined #openstack-ansible | 03:15 | |
*** gkadam-away is now known as gkadam | 03:19 | |
*** acormier has quit IRC | 03:20 | |
*** acormier has joined #openstack-ansible | 03:20 | |
openstackgerrit | Antony Messerli proposed openstack/openstack-ansible-ops master: Clean out old mariadb apt sources https://review.openstack.org/513400 | 03:20 |
*** acormier has quit IRC | 03:25 | |
*** huxinhui_ has joined #openstack-ansible | 03:27 | |
*** markvoelker has joined #openstack-ansible | 03:28 | |
*** thorst has joined #openstack-ansible | 03:55 | |
*** thorst has quit IRC | 04:00 | |
*** markvoelker has quit IRC | 04:02 | |
*** poopcat has quit IRC | 04:05 | |
*** germs has quit IRC | 04:05 | |
*** markvoelker has joined #openstack-ansible | 04:16 | |
*** chyka has joined #openstack-ansible | 04:18 | |
*** markvoelker has quit IRC | 04:21 | |
*** chyka has quit IRC | 04:22 | |
*** markvoelker has joined #openstack-ansible | 04:26 | |
*** markvoelker has quit IRC | 04:30 | |
*** markvoelker has joined #openstack-ansible | 04:35 | |
*** markvoelker has quit IRC | 04:39 | |
*** markvoelker has joined #openstack-ansible | 04:44 | |
*** markvoelker has quit IRC | 04:48 | |
*** ianychoi_ has joined #openstack-ansible | 04:48 | |
*** ianychoi has quit IRC | 04:50 | |
*** markvoelker has joined #openstack-ansible | 04:53 | |
*** ianychoi__ has joined #openstack-ansible | 04:54 | |
*** ianychoi_ has quit IRC | 04:56 | |
*** markvoelker has quit IRC | 04:57 | |
*** markvoelker has joined #openstack-ansible | 05:02 | |
*** ianychoi has joined #openstack-ansible | 05:07 | |
*** ianychoi__ has quit IRC | 05:10 | |
*** jbadiapa has joined #openstack-ansible | 05:16 | |
*** newmember has joined #openstack-ansible | 05:16 | |
*** armaan has joined #openstack-ansible | 05:23 | |
*** markvoelker has quit IRC | 05:35 | |
*** markvoelker has joined #openstack-ansible | 05:41 | |
*** markvoelker has quit IRC | 05:45 | |
*** thorst has joined #openstack-ansible | 05:57 | |
*** ggillies has quit IRC | 05:57 | |
*** thorst has quit IRC | 06:02 | |
*** mma has joined #openstack-ansible | 06:02 | |
*** Oku_OS-away is now known as Oku_OS | 06:08 | |
mma | Hey, got a build failure from zuul that I suspect has nothing to do with my actual change. The dev docs says: "To re-run check or gate jobs, leave a comment on the review with the form “recheck”." Does that mean a reply to the zuul failure comment, or a general reply on the change? | 06:11 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_keystone stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513623 | 06:26 |
*** jvidal has joined #openstack-ansible | 06:26 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_keystone stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513623 | 06:26 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_ceilometer master: [DNM] Quick Attempt to fix ceilometer https://review.openstack.org/513051 | 06:29 |
*** markvoelker has joined #openstack-ansible | 06:32 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-galera_client stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513624 | 06:33 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-rabbitmq_server stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513625 | 06:33 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-haproxy_server stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513626 | 06:37 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-pip_install stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513627 | 06:38 |
*** kukacz has quit IRC | 06:41 | |
*** kukacz has joined #openstack-ansible | 06:43 | |
*** mma has quit IRC | 06:46 | |
*** mcarden has quit IRC | 06:47 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_neutron stable/pike: Add git and iptables to the list of packages https://review.openstack.org/512631 | 06:54 |
*** armaan has quit IRC | 06:54 | |
*** rich1iu has quit IRC | 06:54 | |
*** armaan has joined #openstack-ansible | 06:54 | |
evrardjp | odyssey4me: for this https://review.openstack.org/#/c/513406/ shouldn't the new jobs be included in the test of the patch too? | 06:55 |
evrardjp | mma reply on the change "recheck" | 06:56 |
*** vnogin has joined #openstack-ansible | 06:56 | |
evrardjp | mma: but right now we have trouble with our gates, and we are still not 100% ready in zuul v3 | 06:56 |
*** mma has joined #openstack-ansible | 06:57 | |
evrardjp | mma: I just answered your question | 06:57 |
evrardjp | let me copy and paste again :) | 06:57 |
evrardjp | mma reply on the change "recheck" | 06:57 |
evrardjp | mma: but right now we have trouble with our gates, and we are still not 100% ready in zuul v3 | 06:57 |
evrardjp | what is the change you are talking about? | 06:57 |
mma | evrardjp: I actually responded to my own question. Or well, I tried to but then realized I had gotten disconnected from IRC without noticing. I see the job running in zuul since 45 minutes ago. :) | 07:00 |
*** yolanda has joined #openstack-ansible | 07:00 | |
evrardjp | haha | 07:00 |
evrardjp | ok | 07:00 |
mma | I really need to install an IRC client, currently on the webchat which is.. well not that communicative on the status :) | 07:00 |
evrardjp | plenty of them available! | 07:01 |
mma | The change in question is: 504015, which you have already +2 a couple of days ago so it's all good. | 07:01 |
mma | Just noticed that zuul gave an error and that error seems to have been issues with fetching GPG key for rabbitmq. So my guess is it was some temporary failure in the infra, hence the recheck. | 07:02 |
*** armaan has quit IRC | 07:02 | |
mma | Still trying to get my head around the whole process here, but getting there. Slowly :) | 07:02 |
evrardjp | mma: oh yeah, we get that sometimes. But mhayden is working on it, and will change this gpg for centos story by being a little smarter | 07:04 |
evrardjp | sometimes infra has trouble with external data. So we try to improve there any way we can | 07:04 |
mma | evrardjp: Sounds good :) | 07:06 |
*** vnogin has quit IRC | 07:12 | |
*** mbuil has joined #openstack-ansible | 07:13 | |
*** mma has quit IRC | 07:17 | |
*** mma has joined #openstack-ansible | 07:17 | |
*** sep has joined #openstack-ansible | 07:22 | |
*** mma has quit IRC | 07:24 | |
*** muran has joined #openstack-ansible | 07:25 | |
*** muran has left #openstack-ansible | 07:25 | |
*** mma has joined #openstack-ansible | 07:28 | |
*** mma has quit IRC | 07:28 | |
*** mma has joined #openstack-ansible | 07:28 | |
hwoarang | good morning | 07:43 |
hwoarang | i would appreciate any reviews on https://review.openstack.org/#/c/511768/ | 07:44 |
*** mrch has joined #openstack-ansible | 07:50 | |
evrardjp | good morning hwoarang | 07:50 |
evrardjp | hwoarang: sounds good | 07:53 |
hwoarang | thank you | 07:53 |
hwoarang | i am looking at the zuulv3 patches now | 07:53 |
*** cshen_ has joined #openstack-ansible | 07:53 | |
hwoarang | but i can't find what the new 'queue:' is supposed to do. Although I found an explanation by odyssey4me i want to find where this is written on the docs | 07:53 |
evrardjp | right now I am not pushing too many patches in a row | 07:53 |
evrardjp | but we are in a good state | 07:53 |
*** chyka has joined #openstack-ansible | 07:54 | |
evrardjp | https://docs.openstack.org/infra/zuul/feature/zuulv3/user/config.html#project | 07:55 |
evrardjp | hwoarang: ^ | 07:55 |
evrardjp | "To group two or more related projects into a shared queue for a dependent pipeline, set the queue parameter to the same value for those projects." | 07:55 |
hwoarang | hmm | 07:55 |
evrardjp | Now it's a question of whether we inherit from a standard queue or not. I believe it's fine. | 07:56 |
*** andressanchez has joined #openstack-ansible | 07:56 | |
hwoarang | i see | 07:56 |
evrardjp | well that's my understanding, don't take my word for it. | 07:56 |
evrardjp | odyssey4me: is the infra liaison after all :) | 07:57 |
evrardjp | what I don't understand yet is this: | 07:57 |
evrardjp | https://review.openstack.org/#/c/513406/ | 07:57 |
*** thorst has joined #openstack-ansible | 07:58 | |
evrardjp | why aren't the new jobs on this test? | 07:58 |
*** chyka has quit IRC | 07:58 | |
hwoarang | good question | 08:00 |
evrardjp | I've asked on infra. Maybe the job definition is wrong, but then I'd expect a failure message | 08:02 |
*** thorst has quit IRC | 08:03 | |
hwoarang | i am also looking at the propose job | 08:03 |
hwoarang | i think it's not too hard to bring it back | 08:03 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible master: Initial integrated zuulv3 jobs https://review.openstack.org/513406 | 08:06 |
evrardjp | found out why | 08:07 |
evrardjp | hwoarang: oh that would be great | 08:07 |
evrardjp | I think the propose job already exists | 08:07 |
evrardjp | we just have to either create a new job based on it | 08:08 |
evrardjp | or we hack it together so it works for us, and is properly triggered | 08:08 |
evrardjp | I think the propose job should be already working for us, but I haven't got the chance to check at the logs yet. | 08:08 |
evrardjp | want me to help you there? | 08:09 |
hwoarang | evrardjp: yeah everything is ready i think. i just need to create a playbook in project-config | 08:10 |
hwoarang | but let me investigate a bit more | 08:10 |
hwoarang | and i dont think we need to sync any new files | 08:10 |
evrardjp | no | 08:10 |
evrardjp | but there is a playbook already for it | 08:10 |
evrardjp | https://github.com/openstack-infra/project-config/blob/master/playbooks/proposal/propose-updates.yaml | 08:11 |
hwoarang | ah yes true | 08:11 |
hwoarang | just a job i guess | 08:11 |
hwoarang | with the appropriate parameter | 08:11 |
evrardjp | https://github.com/openstack-infra/project-config/blob/abee45599018604610aaf9c7180df04e0b56182b/zuul.yaml#L190-L195 ? isn't enough? | 08:12 |
evrardjp | or maybe we create another one, next to it, with the appropriate vars | 08:12 |
hwoarang | you need one job to set 'update_target' to openstack-ansible-tests | 08:13 |
hwoarang | iirc | 08:14 |
evrardjp | yeah | 08:14 |
evrardjp | ok | 08:14 |
evrardjp | but I thought that was already in our jobs | 08:14 |
evrardjp | wait a sec | 08:14 |
hwoarang | hmm can this be in the in-repo config? because we use the special node with secrets i think we may not be allowed | 08:15 |
*** rodolof has joined #openstack-ansible | 08:15 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible master: Initial integrated zuulv3 jobs https://review.openstack.org/513406 | 08:18 |
evrardjp | hwoarang: https://raw.githubusercontent.com/openstack-infra/project-config/master/zuul.d/projects.yaml | 08:18 |
evrardjp | in this | 08:18 |
evrardjp | search for openstack-ansible-tests | 08:18 |
hwoarang | ah | 08:19 |
evrardjp | it looks like the job should already have triggered | 08:19 |
hwoarang | yeah sounds about right | 08:19 |
evrardjp | so I am not sure whether we should change this code, and just put it in tree, in the tests repo? | 08:19 |
hwoarang | did we change one of the 'common' files in the tests repo and that job didn't work? | 08:20 |
*** mcarden has joined #openstack-ansible | 08:20 | |
evrardjp | (https://github.com/openstack/openstack-ansible-tests/blob/master/zuul.d/project.yaml) | 08:20 |
evrardjp | yes | 08:20 |
evrardjp | during the same commit we introduced the changes for zuul in tests repo | 08:21 |
evrardjp | change in test-repo-clone which should have been mirrored | 08:21 |
evrardjp | that's why I think I should check the logs instead of creating a new job first | 08:21 |
hwoarang | i am not sure because the doc says that the in-repo project definitions are layered on top of the common ones. | 08:21 |
hwoarang | not overriden | 08:21 |
evrardjp | ok | 08:21 |
evrardjp | well that's my understanding too. | 08:21 |
evrardjp | but bugs happen | 08:21 |
evrardjp | that's why logs matter! | 08:21 |
evrardjp | :D | 08:21 |
hwoarang | :) | 08:22 |
evrardjp | let me dig with you on those | 08:22 |
openstackgerrit | Manuel Buil proposed openstack/openstack-ansible-os_neutron master: L3 Support for ODL deployments in flat networks https://review.openstack.org/497458 | 08:22 |
evrardjp | hwoarang: so for me, this one: https://review.openstack.org/#/c/512414/ hsould have triggered something | 08:23 |
*** newmember has quit IRC | 08:23 | |
*** newmember has joined #openstack-ansible | 08:23 | |
hwoarang | Andreas comment is interesting | 08:24 |
evrardjp | let's dig | 08:24 |
evrardjp | http://logs.openstack.org/ac/ac930796d66e7fdd952477df03d75c4024c8f5a2/post/ | 08:24 |
evrardjp | which one? | 08:24 |
hwoarang | "do you have changes up to remove these jobs from project-config and openstack-zuul-jobs?" | 08:24 |
evrardjp | where? | 08:24 |
evrardjp | I think we do | 08:25 |
evrardjp | some already merged | 08:25 |
evrardjp | https://review.openstack.org/#/c/512635/ | 08:25 |
hwoarang | let me look at the logs one moment | 08:25 |
evrardjp | I linked you to them, not checked yet | 08:25 |
evrardjp | I see the git clone happening | 08:27 |
evrardjp | but no file added | 08:27 |
evrardjp | or git review done | 08:27 |
hwoarang | + '[' -n '' ']' | 08:27 |
hwoarang | so because this is not true, no update happens | 08:27 |
evrardjp | ok we know what to fix! | 08:27 |
evrardjp | great! | 08:27 |
evrardjp | could you deal with that part? | 08:28 |
hwoarang | that's in generic code though | 08:28 |
evrardjp | I thought that was our code | 08:28 |
evrardjp | maybe we should directly do another job that triggers /usr/local/jenkins/slave_scripts/sync_openstack_ansible_common_files.sh ? | 08:29 |
evrardjp | sorry sync_openstack_ansible_common_files.sh | 08:30 |
evrardjp | or maybe better, directly do a playbook that does this sync? | 08:30 |
evrardjp | lookup of the files, copying them, done. | 08:30 |
hwoarang | let me ask infra | 08:30 |
hwoarang | something is not right | 08:30 |
evrardjp | yeah | 08:31 |
evrardjp | I agree | 08:31 |
evrardjp | but still, I think it would be better with not that layering of shell scripts | 08:31 |
evrardjp | just a playbook is good enough | 08:31 |
evrardjp | cacti is behaving fine, I'll throw more patches to it | 08:37 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-lxc_container_create stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513659 | 08:38 |
*** rodolof has quit IRC | 08:38 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-memcached_server stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513660 | 08:38 |
*** rodolof has joined #openstack-ansible | 08:39 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-openstack_hosts stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513661 | 08:39 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-openstack_openrc stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513662 | 08:39 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-rsyslog_server stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513663 | 08:39 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-rsyslog_client stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513664 | 08:40 |
*** electrofelix has joined #openstack-ansible | 08:42 | |
evrardjp | hwoarang: could you vote on https://review.openstack.org/#/c/512631/ ? | 08:42 |
evrardjp | this way we can reintroduce master upgrade testing | 08:42 |
*** newmember has quit IRC | 08:45 | |
*** newmember has joined #openstack-ansible | 08:45 | |
*** cshen_ has quit IRC | 08:49 | |
*** gus has quit IRC | 08:57 | |
*** jamielennox has quit IRC | 08:57 | |
Taseer | evrardjp: How do I debug the zuul CI logs, and figure out what is causing the error ? | 09:01 |
evrardjp | hwoarang: or anyone else for the vote :) | 09:01 |
*** vnogin has joined #openstack-ansible | 09:01 | |
evrardjp | Taseer: what's the patchset? | 09:01 |
evrardjp | clicking on the job name is what I do | 09:02 |
evrardjp | not sure that's the question. | 09:02 |
*** jamielennox has joined #openstack-ansible | 09:02 | |
Taseer | evrardjp: https://review.openstack.org/#/c/503971/ | 09:02 |
*** gus has joined #openstack-ansible | 09:03 | |
*** thorst has joined #openstack-ansible | 09:03 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible master: (WIP) Integrate Congress with OSA. https://review.openstack.org/503971 | 09:06 |
evrardjp | Taseer: did a few comments there. | 09:06 |
evrardjp | can congress be a standalone role that has its own tests? | 09:06 |
evrardjp | Taseer: you should pay attention to the indentation | 09:07 |
evrardjp | https://github.com/Taseer94/openstack-ansible-os_congress/blob/master/tests/test-intall-congress.yml | 09:07 |
evrardjp | this can break a lot of things | 09:07 |
*** thorst has quit IRC | 09:08 | |
evrardjp | Taseer: you can probably remove the with items on this task: https://github.com/Taseer94/openstack-ansible-os_congress/blob/master/tests/test-congress-functional.yml#L23 | 09:09 |
evrardjp | and directly use the port number | 09:09 |
evrardjp | if there is only one api | 09:09 |
Taseer | evrardjp: Okay, thanks for the comments. | 09:09 |
evrardjp | Also I think you should remove this: https://github.com/Taseer94/openstack-ansible-os_congress/blob/master/tox.ini#L106-L126 | 09:10 |
evrardjp | but you're definitely closer | 09:10 |
evrardjp | in general we accept the role for inclusion before doing a scenario, but I like the idea of having a scenario for it already :) | 09:11 |
evrardjp | so to answer your question, I've rebased your patch | 09:11 |
evrardjp | so you're gonna have the latest zuul jobs running | 09:11 |
evrardjp | when the results come back you can simply click on the job name | 09:12 |
evrardjp | then check the job output | 09:12 |
evrardjp | FYI one of the new jobs has already been tested for your patchset, the linters. | 09:12 |
evrardjp | See it here: http://logs.openstack.org/71/503971/6/check/openstack-tox-linters/ce187a8/ | 09:12 |
evrardjp | you'll see all the issues with the space alignments there I guess :p | 09:13 |
Taseer | evrardjp: Thank you so much ! | 09:14 |
evrardjp | yw! | 09:14 |
openstackgerrit | Merged openstack/openstack-ansible-os_neutron master: Add OpenStack-Ansible metadata https://review.openstack.org/510500 | 09:14 |
openstackgerrit | Merged openstack/openstack-ansible-lxc_hosts stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513427 | 09:14 |
openstackgerrit | Merged openstack/openstack-ansible-os_cinder master: Adds support for specifying GID AND UID for cinder system user https://review.openstack.org/504015 | 09:14 |
evrardjp | wowo things merging! | 09:14 |
mma | :) | 09:15 |
*** ArchiFleKs has quit IRC | 09:17 | |
*** ArchiFleKs has joined #openstack-ansible | 09:18 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_ceilometer master: Initial OSA zuul v3 role jobs https://review.openstack.org/512775 | 09:22 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_ceilometer master: Initial OSA zuul v3 role jobs https://review.openstack.org/512775 | 09:22 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_ceilometer master: Initial OSA zuul v3 role jobs https://review.openstack.org/512775 | 09:23 |
*** shardy has joined #openstack-ansible | 09:26 | |
openstackgerrit | Manuel Buil proposed openstack/openstack-ansible-os_neutron master: L3 Support for ODL deployments in flat networks https://review.openstack.org/497458 | 09:31 |
*** kiennt26 has quit IRC | 09:32 | |
*** kiennt26 has joined #openstack-ansible | 09:33 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible master: Initial integrated zuulv3 jobs https://review.openstack.org/513406 | 09:34 |
hwoarang | evrardjp: the propose thing is an infra issue | 09:35 |
evrardjp | ok | 09:35 |
evrardjp | how do you want to proceed | 09:36 |
*** yolanda has quit IRC | 09:36 | |
evrardjp | wait for them to fix/fix with them, or create a new job for us | 09:36 |
evrardjp | reducing the code we have in their repo, and keeping things in our repo | 09:36 |
evrardjp | if that works | 09:36 |
evrardjp | probably not possible due to credentials I guess? | 09:37 |
hwoarang | evrardjp: AJaeger: hwoarang: assuming that I located the correct version of the script, we seem to need to drop the "remote/" here https://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/scripts/propose_update.sh#n127 | 09:37 |
hwoarang | should have quoted that but anyway | 09:37 |
evrardjp | haha | 09:37 |
evrardjp | I got it ;) | 09:37 |
evrardjp | yeah makes sense | 09:38 |
hwoarang | yeah i think because of the credentials | 09:38 |
evrardjp | or we need to check references instead | 09:38 |
hwoarang | not sure why that is a problem now... | 09:38 |
evrardjp | but not both | 09:38 |
evrardjp | what do you mean? | 09:38 |
evrardjp | the remote/ ? | 09:38 |
hwoarang | the code in that script hasn't changed recently | 09:38 |
hwoarang | it's been like that and our job was working ok | 09:38 |
evrardjp | mmm remotes is fine | 09:39 |
evrardjp | what is $ZUUL_REFNAME$ | 09:39 |
hwoarang | Andreas said they are working on propose thing... so lets wail the next run | 09:39 |
evrardjp | If that's shell script, shouldn't it be $ZUUL_REFNAME ? | 09:39 |
evrardjp | not sure the impact of having $ | 09:40 |
evrardjp | me and my shell scripting skillz :/ | 09:40 |
*** newmember has quit IRC | 09:40 | |
*** newmember has joined #openstack-ansible | 09:41 | |
hwoarang | evrardjp: "<AJaeger> hwoarang: please keep an eye on it - we know the proposal jobs are broken and fix them step by step..." | 09:42 |
evrardjp | ok | 09:42 |
evrardjp | I'd love having proposal bot ready | 09:43 |
evrardjp | :) | 09:43 |
hwoarang | yeah that would be helpful indeed | 09:43 |
hwoarang | lets see what up for fixing today | 09:43 |
*** pcaruana has joined #openstack-ansible | 09:46 | |
*** hw_wutianwei has quit IRC | 09:48 | |
andressanchez | Good morning, I have a newbie doubt: if setup-infrastructure fails, specifically while setting up galera server. In order to try again the playbook with changes i am guessing it is necessary to delete the container, but i am not sure if only doing that will do the trick! if you could enlighten me i would appreciate it! thank you! | 09:49 |
hwoarang | odyssey4me: are you working on collecting the logs in a post-* phase? otherwise I want to have a look at it | 09:50 |
hwoarang | but i dont know if it conflicts with your work | 09:51 |
*** yifei has quit IRC | 09:51 | |
*** stuartgr has joined #openstack-ansible | 10:05 | |
openstackgerrit | Merged openstack/openstack-ansible-os_neutron stable/pike: Add git and iptables to the list of packages https://review.openstack.org/512631 | 10:12 |
*** kiennt26 has quit IRC | 10:12 | |
*** rodolof has quit IRC | 10:14 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_nova master: Update paste, policy and rootwrap configurations 2017-10-14 https://review.openstack.org/512044 | 10:19 |
odyssey4me | hwoarang evrardjp infra has been working with the release team to work out a bunch of issues relating to the proposal bot, so you might want to check the status with them before wasting too much time on the proposal bot stuff | 10:20 |
hwoarang | odyssey4me: ok | 10:20 |
hwoarang | i am also investingating the logs issue | 10:20 |
hwoarang | everything seems to be in place | 10:21 |
hwoarang | except that we probalby copy logs in the wrong place and that's why they are not uploaded anymore | 10:21 |
evrardjp | I am doing other stuff, ensuring jobs are passing, testing jobs | 10:21 |
*** vnogin has quit IRC | 10:21 | |
evrardjp | manually | 10:21 |
evrardjp | so I have things to keep me busy :) | 10:21 |
evrardjp | we are in a far better state now | 10:22 |
evrardjp | I will have to work on the previous branches though, and these will be completely different beasts without zuul cloner I guess | 10:22 |
odyssey4me | hwoarang I'm not working on the logs collection move to post just yet | 10:22 |
odyssey4me | so go ahead | 10:22 |
hwoarang | ok | 10:22 |
odyssey4me | from my initial look IIRC we only collect logs for roles in nodepool, so it should be an easy split out | 10:23 |
*** cshen_ has joined #openstack-ansible | 10:24 | |
evrardjp | odyssey4me: what does that mean odyssey4me? | 10:25 |
evrardjp | can't we just add a post job that copies the files around? | 10:25 |
evrardjp | or tars the file round | 10:26 |
odyssey4me | not a post job, a post playbook | 10:26 |
odyssey4me | let me show you what I mean | 10:26 |
odyssey4me | currently the role test executes the log collection here: https://github.com/openstack/openstack-ansible-tests/blob/98d4033c63f0118c5f4afb3535191706417fb04b/test-ansible-functional.sh#L99-L105 | 10:27 |
odyssey4me | we only do the log collection when we're in nodepool: https://github.com/openstack/openstack-ansible-tests/blob/master/test-log-collect.sh#L36 | 10:28 |
odyssey4me | so, we can take the conditional out now - remove the gate_job_exit_tasks bit - then just add the execution of the log collection as a post playbook in the common functional job | 10:29 |
odyssey4me | the only trick is going to be detecting pass/failure for the env var TEST_EXIT_CODE to be present (or some replacement of it) | 10:29 |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: zuul.d: run.yml: Set WORKING_DIR for log collection https://review.openstack.org/513706 | 10:33 |
hwoarang | odyssey4me: evrardjp this ^^^ or some variation of it should do the trick | 10:33 |
odyssey4me | where is work_root ? | 10:35 |
hwoarang | work_root: /var/lib/zuul/builds/628de8ca27b74975bbaf630b87577857/work | 10:35 |
odyssey4me | I would expect that we want to set that to the same directory that we're using to execute from? | 10:35 |
*** vnogin has joined #openstack-ansible | 10:35 | |
evrardjp | oh neat if that works | 10:35 |
odyssey4me | ie the same value as the chdir | 10:35 |
odyssey4me | lemme just revisit the script quickly - hang on | 10:36 |
odyssey4me | yeah, working_dir is meant to be the root of the repo being tested | 10:36 |
odyssey4me | for example: https://github.com/openstack/openstack-ansible-tests/blob/98d4033c63f0118c5f4afb3535191706417fb04b/test-ansible-functional.sh#L40 | 10:36 |
*** huxinhui_ has quit IRC | 10:37 | |
hwoarang | odyssey4me: it's not the same as chdir | 10:38 |
evrardjp | odyssey4me: hwoarang maybe we can wire the ANSIBLE_LOG_PATH to the WORKDIR or to the log thing? | 10:38 |
evrardjp | or you want the extra files and all? | 10:38 |
odyssey4me | hwoarang the script expects that WORKING_DIR is the root of the clone | 10:38 |
odyssey4me | so your patch should fail fast | 10:39 |
hwoarang | hmm | 10:39 |
odyssey4me | in fact, WORKING_DIR is being set as pwd already, so it's likely working | 10:39 |
hwoarang | odyssey4me: bu the zuul role 'upload-logs' expects all logs to be in zuul.executor.work_root/logs | 10:39 |
hwoarang | we can't put them anywhere else | 10:40 |
odyssey4me | the most likely reason why logs are not currently collected is because /etc/nodepool no longer exists | 10:40 |
hwoarang | it has to be in /var/lib/ | 10:40 |
hwoarang | hmm | 10:40 |
odyssey4me | evrardjp that may also be why our integrated and role tests are failing out so fast | 10:40 |
odyssey4me | *due to timeouts | 10:40 |
evrardjp | failing so fast? | 10:40 |
evrardjp | what what? | 10:41 |
odyssey4me | our roles and integrated build are failing due to timeouts a lot, right? | 10:41 |
evrardjp | not really more than jenkins | 10:41 |
evrardjp | to what I experienced recently | 10:41 |
odyssey4me | we should check whether /etc/nodepool exists as our detection method for being in nodepool - because that may be a cause | 10:41 |
odyssey4me | we may have to adjust, because that may not exist any more | 10:41 |
hwoarang | wait wait | 10:42 |
evrardjp | what would be the impact? I am not sure to understand you | 10:42 |
hwoarang | /etc/nodepool exists | 10:42 |
evrardjp | let me check the script | 10:42 |
odyssey4me | ok, if that is the case then we're good there | 10:42 |
hwoarang | because we see #### BEGIN LOG COLLECTION ### in te logs | 10:42 |
hwoarang | let me explain | 10:42 |
odyssey4me | because the use of local proxies/repositoreies all keys off that: https://github.com/openstack/openstack-ansible-tests/blob/master/common-tasks/test-set-nodepool-vars.yml#L18 | 10:43 |
hwoarang | $pwd is /home/zuul/src/git.openstack.org/openstack/openstack-ansible-os_keystone/ | 10:43 |
hwoarang | this is the wrong location for us | 10:43 |
hwoarang | we need to do stuff in /var/lib/zuul/builds/628de8ca27b74975bbaf630b87577857/work | 10:43 |
hwoarang | or at least the logs need to be put in /var/lib/zuul/builds/628de8ca27b74975bbaf630b87577857/work/logs | 10:43 |
hwoarang | not /home/zuul/src/git.openstack.org/openstack/openstack-ansible-os_keystone/logs | 10:43 |
evrardjp | odyssey4me: OH | 10:43 |
odyssey4me | ok, so this is a new change in how the logs are collected | 10:43 |
hwoarang | otherwise they are not going to be uploaded | 10:43 |
evrardjp | I see what you meant. You are expecting slowness because we fail to use the proper vars | 10:43 |
evrardjp | because not set because no detection. gotcha. | 10:44 |
evrardjp | hwoarang: I think we are good for a gate specific post play to do the copy of what we need | 10:44 |
odyssey4me | hwoarang ok, that means we're going to have to implement a new var for the log location and use it | 10:44 |
evrardjp | or that. | 10:45 |
hwoarang | that's what i tried to do in my patch | 10:45 |
evrardjp | but if we implement that, we then need wiring later | 10:45 |
hwoarang | set the variable we use properly | 10:45 |
evrardjp | to separate them | 10:45 |
odyssey4me | but yes, better is just to modify the script to no longer care about /etc/nodepool and just use it in the gate job config as a post playbook | 10:45 |
odyssey4me | hwoarang but that variable (WORKING_DIR) is used as the repo root, not the log root | 10:46 |
hwoarang | aha | 10:46 |
hwoarang | so it's used elsewhere too | 10:46 |
odyssey4me | yes | 10:46 |
hwoarang | true | 10:46 |
odyssey4me | it's shared across all the other test scripts | 10:47 |
hwoarang | lets try something else then | 10:47 |
odyssey4me | so better to add a new var in https://github.com/openstack/openstack-ansible-tests/blob/master/test-log-collect.sh that sets the place for logs | 10:47 |
odyssey4me | then change the use of WORKING_DIR/logs there to the new var | 10:47 |
hwoarang | ok | 10:48 |
odyssey4me | that var will also have to be implemented in this script to make sure it matches: https://github.com/openstack/openstack-ansible-tests/blob/ac930796d66e7fdd952477df03d75c4024c8f5a2/test-ansible-env-prep.sh#L38 | 10:48 |
odyssey4me | oh actually, never mind that | 10:48 |
odyssey4me | we should rather collect that info | 10:49 |
*** newmember has quit IRC | 10:50 | |
hwoarang | i wonder if we need to do stuff in /var/lib instead of the checkout repo | 10:51 |
*** newmember has joined #openstack-ansible | 10:51 | |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: zuul.d: run.yml: Use new BUILD_DIR variable for log collection https://review.openstack.org/513706 | 10:51 |
hwoarang | anyway | 10:51 |
mbuil | guys, when debugging, I thought that if I want to check all available variables in one host, I should check: hostvars[inventory_hostname], however, I am noticing that a template has a variable which is not in hostvars and that template is successfully created. So I am assuming that I was wrong and there are more variables apart from the ones in the hostvars dictionary. Is my assumption correct or am I getting confused somehow?? | 10:55 |
evrardjp | yes hostvars is just part of all ansible runtime vars | 10:56 |
evrardjp | vars is the full runtime | 10:56 |
evrardjp | well | 10:56 |
evrardjp | more or less | 10:56 |
evrardjp | what are you trying to find out? | 10:56 |
evrardjp | mbuil: ^ | 10:56 |
odyssey4me | evrardjp almost there - one more small tweak | 10:57 |
odyssey4me | sorry, I meant hwoarang | 10:58 |
mbuil | evrardjp: I wanted to know the values of neutron_provider_network ==> https://github.com/openstack/openstack-ansible-os_neutron/blob/master/templates/plugins/ml2/ml2_conf.ini.j2#L14 | 10:58 |
hwoarang | odyssey4me: tell me | 10:58 |
odyssey4me | hwoarang check the review | 10:58 |
hwoarang | ok | 10:58 |
hwoarang | odyssey4me: ok about ara | 10:58 |
hwoarang | upstream zuul jobs already generate ara output | 10:59 |
hwoarang | :) | 10:59 |
evrardjp | mbuil: template it into a file you'll get your value :) | 10:59 |
hwoarang | so we should drop our ara code | 10:59 |
odyssey4me | yes, but that's not the output from our tasks - that's the output of the initiation of our job | 10:59 |
hwoarang | aha | 10:59 |
odyssey4me | the two are different | 10:59 |
hwoarang | ok i didn't spend time on it to be honest | 10:59 |
hwoarang | let me fix | 10:59 |
evrardjp | ara would need two folders I guess. | 11:00 |
evrardjp | lunchtime | 11:01 |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: zuul.d: run.yml: Use new BUILD_DIR variable for log collection https://review.openstack.org/513706 | 11:01 |
odyssey4me | no worries about the folder, because we make our own | 11:01 |
odyssey4me | hwoarang yes, that should work | 11:01 |
odyssey4me | I'll do a follow on patch to move the log collection to the post section | 11:02 |
openstackgerrit | Taseer Ahmed proposed openstack/openstack-ansible master: (WIP) Integrate Congress with OSA. https://review.openstack.org/503971 | 11:03 |
*** thorst has joined #openstack-ansible | 11:04 | |
openstackgerrit | Taseer Ahmed proposed openstack/openstack-ansible master: Integrate Congress with OSA. https://review.openstack.org/503971 | 11:07 |
*** thorst has quit IRC | 11:09 | |
odyssey4me | hwoarang do you know if there's a built-in way of detecting the success/failure of the job? | 11:13 |
*** pbandark has joined #openstack-ansible | 11:13 | |
hwoarang | odyssey4me: nope i haven't seen such a thing (yet) | 11:14 |
hwoarang | but post jobs are always executed so we may as well always collect them? | 11:14 |
odyssey4me | hwoarang yes, but we have a conditional which only generates ARA reports when the job fails | 11:14 |
hwoarang | ah right | 11:14 |
evrardjp | odyssey4me: hwoarang that was implemented to avoid the inodes problem | 11:16 |
evrardjp | so if we smarter in the post job, maybe it will be better for infra and still provide good features for us | 11:16 |
hwoarang | maybe infra can provide a way to detect the 'run' playbook failure/success so it can be shared with other teams too | 11:17 |
*** sep_ has joined #openstack-ansible | 11:18 | |
evrardjp | oh that would be nice | 11:18 |
evrardjp | we can probably have irc notifications already | 11:18 |
evrardjp | but that's gonna become a spam fast | 11:19 |
hwoarang | ;p | 11:20 |
*** sep has quit IRC | 11:20 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Move log collection to 'post' stage https://review.openstack.org/513711 | 11:22 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible master: Initial integrated zuulv3 jobs https://review.openstack.org/513406 | 11:23 |
evrardjp | removed the release part | 11:23 |
evrardjp | odyssey4me: ^ . It was just that I saw that I forgot to migrate it. Now I realise why thanks to your comment | 11:24 |
odyssey4me | evrardjp hwoarang assuming https://review.openstack.org/513711 works, it moves the log collection to the post stage, so it will run regardless of how the job ends | 11:24 |
hwoarang | i like it | 11:24 |
odyssey4me | that will mean that even if a job times out, logs are collected | 11:24 |
evrardjp | well let's see | 11:25 |
evrardjp | it should appear directly :) | 11:25 |
evrardjp | "directly" | 11:25 |
odyssey4me | it also means that the log collection is not included in the job timing - so we don't end up with a fail due to timeout if the job is collecting logs | 11:25 |
evrardjp | yeah that happens a few times. | 11:25 |
evrardjp | happened* | 11:25 |
evrardjp | that's the most frustrating error :p | 11:26 |
evrardjp | next step converting those nasty shell script into ansible. | 11:26 |
evrardjp | :p | 11:26 |
evrardjp | odyssey4me: we should have more issues and less timeouts now btw, with the change hwoarang introduced, we kinda discard a terrible networking node earlier. | 11:29 |
odyssey4me | ja | 11:30 |
evrardjp | maybe we should move all this fetch tasks into the deploy node, at the beggining, in async. | 11:30 |
hwoarang | i am really curious to look at the logs | 11:30 |
evrardjp | haha me too : | 11:30 |
*** chyka has joined #openstack-ansible | 11:30 | |
*** chyka has quit IRC | 11:34 | |
*** pbandark1 has joined #openstack-ansible | 11:36 | |
hwoarang | error: RPC failed; curl 52 Empty reply from server | 11:36 |
hwoarang | pfffffff | 11:36 |
*** pbandark has quit IRC | 11:37 | |
*** pbandark1 is now known as pbandark | 11:37 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Add cross-repo base job https://review.openstack.org/513453 | 11:38 |
hwoarang | evrardjp: we also need to figure out why delay:10 has no effect in the 'gate' ansible | 11:38 |
hwoarang | i blame ansible | 11:38 |
*** sep_ is now known as sep | 11:38 | |
evrardjp | I blame the gates | 11:40 |
evrardjp | :p | 11:40 |
evrardjp | let's blame both it seems a nice sport. | 11:40 |
hwoarang | do we need to add jobs for ocata branch too? | 11:43 |
hwoarang | i am looking at this https://review.openstack.org/#/c/512630/ | 11:43 |
hwoarang | and te zuul voting it's pretty useless | 11:43 |
hwoarang | *the | 11:43 |
*** mma_ has joined #openstack-ansible | 11:51 | |
*** mma has quit IRC | 11:51 | |
odyssey4me | hwoarang yeah, evrardjp is working his way down slowly, so we don't break zuul again | 11:52 |
*** thorst has joined #openstack-ansible | 11:52 | |
hwoarang | ah i didnt know we broke it once already :) | 11:52 |
odyssey4me | twice ;) | 11:52 |
odyssey4me | see http://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=63979&rra_id=all | 11:53 |
odyssey4me | we maxed out the memory twice, just with our jobs | 11:53 |
*** jvidal has quit IRC | 11:53 | |
hwoarang | ah that's why evrardjp kept mentioning cacti... | 11:53 |
odyssey4me | yup | 11:53 |
hwoarang | lol | 11:54 |
hwoarang | ok | 11:54 |
*** cshen_ has quit IRC | 11:59 | |
*** gkadam has quit IRC | 12:00 | |
evrardjp | that's why I am super careful and super slow | 12:01 |
evrardjp | but today I didn't hesitate to put 6 patches in a row | 12:01 |
evrardjp | and it didn't do anything | 12:01 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Add cross-repo base job https://review.openstack.org/513453 | 12:02 |
odyssey4me | yeah, it does seem to be bahving better | 12:02 |
* odyssey4me is asking in infra whether something's changed | 12:04 | |
*** armaan has joined #openstack-ansible | 12:04 | |
odyssey4me | evrardjp ok, so based on my discussion with mordred now in infra, it seems that using required-projects for all our repositories is the best way for us to get the git clones done successfully | 12:05 |
odyssey4me | unfortunately required-projects cannot be dynamic - ie we can't use a map extract filter on the a-r-r file or something | 12:05 |
odyssey4me | we are going to have to define the list of all the repositories in the job | 12:05 |
odyssey4me | evrardjp yes, they implemented https://review.openstack.org/#/c/513441/ which is why the memory usage is better | 12:07 |
odyssey4me | I would suggest upping the frequency of changes now :) | 12:07 |
odyssey4me | perhaps stick to the core os_ and infra roles, get all those done back to newton so that we unblock work... then do the peripheral roles | 12:08 |
evrardjp | I still don't know I will do for O and N, because they are very different | 12:09 |
evrardjp | I will focus on that | 12:10 |
evrardjp | see if that can directly map | 12:10 |
evrardjp | odyssey4me: nice change | 12:10 |
evrardjp | odyssey4me: why doing this in the job, while we have a generic approach? | 12:11 |
*** jled has joined #openstack-ansible | 12:11 | |
odyssey4me | evrardjp maybe for O and N just implement the functional jobs for xenial/trusty and leave it at that? | 12:13 |
odyssey4me | evrardjp 'why doing this in job' - what do you mean by 'this' ? | 12:13 |
evrardjp | what do you mean? Just starting the tox, and not caring about the repo clone? | 12:13 |
evrardjp | why doing the required-project thing, while we have a get-ansible-role-requirements ? | 12:14 |
evrardjp | is there any advantage? | 12:14 |
evrardjp | because we'll have to keep the get-ansible-role-requirements for developers | 12:14 |
evrardjp | (for now at least) | 12:15 |
odyssey4me | evrardjp agreed, except we're getting a lot of git clone failures in tests right now | 12:16 |
odyssey4me | implementing required-projects will mean that zuul puts the repo there ahead of time for us using synchronise from a cache | 12:16 |
odyssey4me | get-ansible-role-requirements will still run, it will find those repositories there, and skip cloning them | 12:16 |
odyssey4me | so it doesn't change how our test executes - but it improves the reliability of the tests execution | 12:17 |
evrardjp | odyssey4me: is it fast to do it? I thought this was slow | 12:17 |
odyssey4me | what do you mean ? | 12:17 |
evrardjp | the required-project handling is something happening in the pre_job,right? | 12:18 |
evrardjp | I thought adding things into required-project took more time than us doing git cloning, because the required-projects did many extra things. | 12:19 |
evrardjp | if that's not the case, let's do it then. | 12:19 |
odyssey4me | not sure where you get that idea from | 12:19 |
odyssey4me | the only issue is that we have to maintain the list of required projects, which is a bit of a pain | 12:19 |
odyssey4me | but meh, better than failing tests due to git clone issues | 12:20 |
evrardjp | it's fine, it's just on a base template in tests/ | 12:20 |
evrardjp | and one in the integrated | 12:20 |
evrardjp | seems ok to me. | 12:20 |
evrardjp | they don't change much | 12:20 |
odyssey4me | we could even perhaps implement a common job parent in tests for the integrated build too | 12:20 |
evrardjp | but I am worried about silent issues | 12:20 |
odyssey4me | just to keep the required projects in one place | 12:20 |
odyssey4me | silent issues? | 12:20 |
odyssey4me | erm, hwoarang - no logs: http://logs.openstack.org/06/513706/3/check/openstack-ansible-functional-ubuntu-xenial/8feba59/ | 12:21 |
odyssey4me | https://review.openstack.org/513706 | 12:21 |
evrardjp | we add something in a repo, but we forgot to add it there, and then our get-a-r-r script will fetch it because not defined into required projects | 12:21 |
evrardjp | then issues will start to appear, and we forgot why it does for this one ... | 12:21 |
odyssey4me | evrardjp oh that won't happen - get-a-r-r will always fetch anything in a-r-r that's missing | 12:22 |
evrardjp | yeah | 12:22 |
evrardjp | that's exactly what I mean | 12:22 |
odyssey4me | it has to, because it has to work outside of infra | 12:22 |
evrardjp | yeah | 12:22 |
evrardjp | but then we're gonna get issues in infra on some repo we forgot to add | 12:22 |
evrardjp | that's what I meant | 12:22 |
odyssey4me | all that will mean is that it won't be pre-cached, so we might get a git clone failure | 12:23 |
evrardjp | but I know you have quite a brain for those things. | 12:23 |
evrardjp | yeah | 12:23 |
odyssey4me | if that happens, hopefully we remember to add it to the base job | 12:23 |
odyssey4me | or, we could make the proposal bot keep it up to date for us ;) | 12:23 |
evrardjp | I like the idea of having things in tests repo | 12:23 |
evrardjp | that's not a terrible idea. | 12:23 |
evrardjp | generate what to clone, based on a-r-r | 12:24 |
evrardjp | for N and O, what did you mean by just doing the functional tests? | 12:24 |
odyssey4me | no, not based on a-r-r - it already knows about all our repositories, because it has to propose patches to them... so just make sure the parent job for required projects has all our repositories in it | 12:24 |
evrardjp | you mean just running the tox for now by adding the zuul.d definitions? | 12:24 |
odyssey4me | evrardjp what's the hesitation for N and O ? | 12:25 |
odyssey4me | they also just need to run tox -e functional, just for less platforms | 12:25 |
evrardjp | ok | 12:25 |
evrardjp | let me do a quick thing | 12:25 |
odyssey4me | want me to propose the tests repo patches to define the jobs? | 12:26 |
evrardjp | nah I can handle it, as long as you review it. | 12:26 |
odyssey4me | sure - just do a small set first - not all of them | 12:26 |
*** acormier has joined #openstack-ansible | 12:27 | |
*** acormier has quit IRC | 12:27 | |
*** acormier has joined #openstack-ansible | 12:28 | |
*** pcaruana has quit IRC | 12:31 | |
*** vnogin has quit IRC | 12:31 | |
*** acormier has quit IRC | 12:33 | |
*** jled has left #openstack-ansible | 12:34 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-tests stable/ocata: Initial OSA zuul v3 role jobs https://review.openstack.org/513724 | 12:36 |
openstackgerrit | Merged openstack/openstack-ansible-tests master: Remove useless and confusing file https://review.openstack.org/511549 | 12:38 |
openstackgerrit | Merged openstack/openstack-ansible-os_designate master: Add OpenStack-Ansible metadata https://review.openstack.org/510485 | 12:38 |
openstackgerrit | Merged openstack/openstack-ansible-galera_server stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513426 | 12:38 |
openstackgerrit | Merged openstack/openstack-ansible-galera_server stable/pike: Suppress command/shell warnings https://review.openstack.org/511935 | 12:38 |
*** woodard_ has quit IRC | 12:39 | |
*** woodard has joined #openstack-ansible | 12:40 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-tests stable/newton: Initial OSA zuul v3 role jobs https://review.openstack.org/513726 | 12:41 |
*** vnogin has joined #openstack-ansible | 12:41 | |
evrardjp | odyssey4me: I will do only neutron at first | 12:41 |
evrardjp | this way we have a pattern | 12:42 |
evrardjp | i think it's quite simple jobs then | 12:46 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_neutron stable/ocata: Initial OSA zuul v3 role jobs https://review.openstack.org/513727 | 12:46 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_neutron stable/ocata: Initial OSA zuul v3 role jobs https://review.openstack.org/513727 | 12:47 |
odyssey4me | evrardjp made some suggestions to the tests repo patches | 12:47 |
evrardjp | thanks | 12:47 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-tests stable/ocata: Initial OSA zuul v3 role jobs https://review.openstack.org/513724 | 12:48 |
evrardjp | fine for me. | 12:49 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Move log collection to 'post' stage https://review.openstack.org/513711 | 12:49 |
evrardjp | let me change the neutron patch for ocata then | 12:49 |
evrardjp | already properly defined | 12:49 |
evrardjp | I am thinking we should have the openstack-ansible-upgrade-ubuntu-xenial-nv into the standard tests one | 12:49 |
evrardjp | and only implement specific tests into each repo | 12:50 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-tests stable/ocata: Initial OSA zuul v3 role jobs https://review.openstack.org/513724 | 12:50 |
evrardjp | odyssey4me: edited | 12:51 |
evrardjp | not all the jobs for upgrade have to be voting in ocata xenial | 12:51 |
evrardjp | am I correct? | 12:51 |
odyssey4me | as far as I know they're all voting | 12:51 |
evrardjp | I thought some of the repos didn't implement the upgrade and started with nv | 12:51 |
odyssey4me | not as far as I know | 12:52 |
evrardjp | ok | 12:52 |
evrardjp | let me roll back then | 12:52 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-tests stable/ocata: Initial OSA zuul v3 role jobs https://review.openstack.org/513724 | 12:52 |
*** mma has joined #openstack-ansible | 12:53 | |
openstackgerrit | Merged openstack/openstack-ansible-ops master: Exclude elasticsearch from container-destroy task https://review.openstack.org/512431 | 12:53 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_neutron stable/ocata: Initial OSA zuul v3 role jobs https://review.openstack.org/513727 | 12:54 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_neutron stable/ocata: Initial OSA zuul v3 role jobs https://review.openstack.org/513727 | 12:55 |
*** gouthamr has joined #openstack-ansible | 12:55 | |
*** mma_ has quit IRC | 12:56 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_neutron stable/newton: Initial OSA zuul v3 role jobs https://review.openstack.org/513733 | 12:58 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-tests stable/newton: Initial OSA zuul v3 role jobs https://review.openstack.org/513726 | 12:59 |
openstackgerrit | Merged openstack/openstack-ansible-memcached_server stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513660 | 13:00 |
evrardjp | I am worried why linters pass sometimes and fail at other times: https://review.openstack.org/#/c/513627/ | 13:01 |
evrardjp | It's valid but it should failed in check... | 13:02 |
odyssey4me | is it failing due to git clone failures? | 13:02 |
odyssey4me | why're you making the upgrade jobs non voting? | 13:03 |
*** markvoelker has quit IRC | 13:03 | |
odyssey4me | nvm, it's a job definition that's not used - responded in review | 13:04 |
mgariepy | can a core push this on please ?https://review.openstack.org/#/c/512386/ | 13:05 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-galera_client stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513624 | 13:06 |
*** vnogin has quit IRC | 13:09 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_keystone stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513623 | 13:11 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-pip_install stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513627 | 13:13 |
*** vnogin has joined #openstack-ansible | 13:13 | |
hwoarang | odyssey4me: i am looking at the logs. the BUILD_DIR variable had no effect | 13:13 |
*** SerenaFeng has joined #openstack-ansible | 13:13 | |
hwoarang | everything is still populated in '/home/zuul/src/git.openstack.org/openstack/openstack-ansible-tests' | 13:14 |
hwoarang | something is fishy | 13:14 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-openstack_openrc stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513662 | 13:14 |
*** esberglu has joined #openstack-ansible | 13:14 | |
hwoarang | i will try something else | 13:15 |
cloudnull | mornings | 13:16 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Add commonly used repositories to required projects https://review.openstack.org/513734 | 13:16 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-rsyslog_server stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513663 | 13:16 |
openstackgerrit | Merged openstack/openstack-ansible-os_ceilometer master: Initial OSA zuul v3 role jobs https://review.openstack.org/512775 | 13:18 |
evrardjp | odyssey4me: maybe I should have started with something not special, instead of neutron in O and N | 13:21 |
evrardjp | this was my most urgent | 13:21 |
odyssey4me | cloudnull there we go - cross-repo test working: https://review.openstack.org/513457 | 13:21 |
odyssey4me | evrardjp oh why? | 13:21 |
evrardjp | well because neutron is broken and has to have forward ports or non voting jobs to pass. | 13:22 |
evrardjp | for the upgrade jobs | 13:22 |
odyssey4me | oh I see | 13:22 |
evrardjp | well let me propose glance | 13:22 |
evrardjp | easier :p | 13:22 |
odyssey4me | oh in that case use nv jobs, but define them in that repo with a TODO to remove them once the upgrade issues are worked out | 13:23 |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: zuul.d: run.yml: Use new BUILD_DIR variable for log collection https://review.openstack.org/513706 | 13:26 |
*** armaan has quit IRC | 13:26 | |
hwoarang | i dont think my patch is being tested... | 13:26 |
*** john51 has quit IRC | 13:28 | |
*** john51 has joined #openstack-ansible | 13:28 | |
*** toan has quit IRC | 13:29 | |
cloudnull | odyssey4me: that's kinda awesome! | 13:29 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_glance stable/ocata: Initial OSA zuul v3 role jobs https://review.openstack.org/513739 | 13:30 |
evrardjp | ahah my music automatically switched at the end of my playlist to "War Pigs", I didn't even ask for it, awesome! | 13:32 |
*** toan has joined #openstack-ansible | 13:32 | |
odyssey4me | evrardjp well, it's definitely working: https://review.openstack.org/513734 | 13:34 |
odyssey4me | in the zuul status page the linters has passed, and the log shows all the right things were done | 13:34 |
evrardjp | oh yeah no doubt | 13:35 |
odyssey4me | I wanted to make a single parent job rather than maintain the list twice - but that can't be done because the linters and functional jobs don't have the same parent | 13:35 |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: zuul.d: run.yml: Use new BUILD_DIR variable for log collection https://review.openstack.org/513706 | 13:36 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_glance stable/newton: Initial OSA zuul v3 role jobs https://review.openstack.org/513743 | 13:36 |
odyssey4me | evrardjp see updated comment in https://review.openstack.org/#/c/513727/4 | 13:37 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_neutron stable/ocata: Initial OSA zuul v3 role jobs https://review.openstack.org/513727 | 13:37 |
evrardjp | odyssey4me: ahah good catch | 13:37 |
evrardjp | at the end these are just ALWAYS the same names and all. | 13:38 |
evrardjp | I am gonna be happy when this will be stable | 13:38 |
evrardjp | odyssey4me: it's alright to define it twice | 13:39 |
evrardjp | althought I'd be happy to use jinja for it | 13:39 |
evrardjp | because we are using jinja, we should probably load that from a macro | 13:39 |
odyssey4me | FYI missing the depends on in https://review.openstack.org/#/c/513743/ | 13:39 |
evrardjp | DARN | 13:39 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_glance stable/newton: Initial OSA zuul v3 role jobs https://review.openstack.org/513743 | 13:40 |
evrardjp | I am surprised I haven't written glace (ice) for glance yet. | 13:41 |
*** yolanda has joined #openstack-ansible | 13:42 | |
*** dave-mccowan has joined #openstack-ansible | 13:42 | |
*** rodolof has joined #openstack-ansible | 13:44 | |
evrardjp | what you added was in tests right? | 13:45 |
*** dave-mcc_ has joined #openstack-ansible | 13:45 | |
odyssey4me | evrardjp who me? what're you referncing? | 13:46 |
odyssey4me | and yes - in tests | 13:46 |
evrardjp | soryr | 13:46 |
evrardjp | sorry | 13:46 |
odyssey4me | if you're referring to the required repo list | 13:46 |
evrardjp | yes | 13:46 |
*** gunix has quit IRC | 13:47 | |
evrardjp | let me rephrase what I have in my head | 13:47 |
evrardjp | I thought it could be a good idea to add that too for the integrated. | 13:47 |
evrardjp | but then I realise that it could be a problem for the roles, because they aren't checked out at our version | 13:47 |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: zuul.d: run.yml: Use new BUILD_DIR variable for log collection https://review.openstack.org/513706 | 13:47 |
evrardjp | so I thought it was a problem for the IRR too. But then I remember that we do not freeze roles in irr. | 13:48 |
*** dave-mccowan has quit IRC | 13:48 | |
evrardjp | role's tests I mean. | 13:48 |
odyssey4me | it might be for the integrated - I'll look at that a bit later -need to go get a hair cut now | 13:50 |
*** gunix has joined #openstack-ansible | 13:59 | |
openstackgerrit | Merged openstack/openstack-ansible-lxc_container_create stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513659 | 14:01 |
openstackgerrit | Merged openstack/openstack-ansible-rsyslog_client stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513664 | 14:01 |
*** SerenaFeng has quit IRC | 14:05 | |
*** jwitko has joined #openstack-ansible | 14:09 | |
*** markvoelker has joined #openstack-ansible | 14:11 | |
*** vnogin has quit IRC | 14:14 | |
*** vnogin has joined #openstack-ansible | 14:18 | |
*** mma_ has joined #openstack-ansible | 14:27 | |
*** mma has quit IRC | 14:30 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_ceilometer master: Add OpenStack-Ansible metadata https://review.openstack.org/510482 | 14:36 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_ironic master: Add OpenStack-Ansible metadata https://review.openstack.org/510493 | 14:37 |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: zuul.d: run.yml: Use new BUILD_DIR variable for log collection https://review.openstack.org/513706 | 14:38 |
*** rodolof has quit IRC | 14:41 | |
*** rodolof has joined #openstack-ansible | 14:42 | |
*** weezS has joined #openstack-ansible | 14:48 | |
*** dave-mcc_ has quit IRC | 14:53 | |
*** dave-mccowan has joined #openstack-ansible | 14:54 | |
*** vnogin has quit IRC | 14:55 | |
*** vnogin has joined #openstack-ansible | 14:57 | |
*** cshen has quit IRC | 14:59 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_neutron stable/newton: Initial OSA zuul v3 role jobs https://review.openstack.org/513733 | 15:03 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_neutron stable/newton: Initial OSA zuul v3 role jobs https://review.openstack.org/513733 | 15:04 |
evrardjp | hwoarang: do you know if there is something to do in our gates to run as root, or is that supposed to be the default? | 15:05 |
*** dave-mcc_ has joined #openstack-ansible | 15:05 | |
logan- | not sure about IRR but the old integrated job def called gate-check-commit using sudo | 15:06 |
*** dave-mccowan has quit IRC | 15:06 | |
hwoarang | evrardjp: well the default user has pwdless sudo. but i am not sure if you can run as real root | 15:07 |
*** vnogin has quit IRC | 15:08 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-openstack_hosts stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513661 | 15:08 |
evrardjp | well I can do sudo -s then gate check commit | 15:09 |
evrardjp | http://logs.openstack.org/06/513406/14/check/openstack-ansible-deploy-aio-ubuntu-xenial/889e72e/job-output.txt.gz#_2017-10-20_11_31_30_790016 | 15:09 |
evrardjp | see also | 15:09 |
evrardjp | https://review.openstack.org/#/c/513406/14/zuul.d/playbooks/run.yml | 15:09 |
evrardjp | maybe we can also use normal ansible things | 15:10 |
evrardjp | become_user: root become_method: sudo | 15:10 |
*** Oku_OS is now known as Oku_OS-away | 15:10 | |
odyssey4me | could do with another review here to help stabilise the role tests: https://review.openstack.org/#/c/513734/ | 15:12 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_glance stable/newton: Initial OSA zuul v3 role jobs https://review.openstack.org/513743 | 15:13 |
*** vnogin has joined #openstack-ansible | 15:15 | |
Taseer | evrardjp: do all the scenarios need to pass for zuul to give +1 ? from the latest patch, tox linters and legacy-openstack-ceph-centos7 failed, docs and two passed, remaining timed out. | 15:16 |
*** vnogin has quit IRC | 15:16 | |
odyssey4me | Taseer if it's not a required (voting) job, the test will show 'non-voting' next to it | 15:16 |
evrardjp | Taseer: odyssey4me your job is in between. I think I rebased on the new jobs by mistake. You can rebase on current master if you prefer. | 15:17 |
evrardjp | that's why you have too many jobs. | 15:17 |
evrardjp | but what odyssey4me said is true. | 15:18 |
evrardjp | oh no you are on the correct thing. I didn't rebase by mistake | 15:18 |
*** vnogin has joined #openstack-ansible | 15:19 | |
evrardjp | well, still the old job definitions, but you don't have twice the amount of jobs. | 15:19 |
evrardjp | it means you have to pass linters tests yes. | 15:19 |
*** acormier has joined #openstack-ansible | 15:20 | |
*** vnogin has quit IRC | 15:20 | |
*** ansibler has quit IRC | 15:20 | |
evrardjp | Taseer: on your last run it's probably not your fault, but I have seen a few linting issues in your changes, did you try to run linting on your development machine? | 15:21 |
evrardjp | hwoarang: cloudnull mhayden anyone: could you check on what odyssey4me has written here: https://review.openstack.org/#/c/513734/ please ? | 15:21 |
odyssey4me | evrardjp what do you mean 'check' ? | 15:22 |
evrardjp | logan-: jmccrory mgariepy too ^ :) | 15:22 |
cloudnull | done | 15:22 |
evrardjp | review | 15:22 |
odyssey4me | hwoarang already reviewed | 15:22 |
evrardjp | oh lord | 15:22 |
odyssey4me | haha, MOAR WORKFLOW | 15:22 |
evrardjp | I clicked the link I didn't click on refresh darn chrome plugin. | 15:23 |
evrardjp | facepalm. | 15:23 |
odyssey4me | right, so between that one and https://review.openstack.org/513200 we should see the majority of common failures I've seen die | 15:23 |
odyssey4me | now to figure out why that other patch is failing | 15:23 |
evrardjp | which other? | 15:24 |
evrardjp | 513200? | 15:24 |
Taseer | evrardjp: so, if I pass the linter tests, I will start getting reviews on it ? | 15:24 |
*** acormier has quit IRC | 15:24 | |
evrardjp | yeah, we can think about it indeed, if it passes tests :) | 15:24 |
evrardjp | but because you're introducing a scenario, you'll probably have a few changes to include to make it work... the fight is not over yet :) | 15:25 |
evrardjp | but we'll get you there | 15:26 |
openstackgerrit | Merged openstack/openstack-ansible-ops master: Clean out old mariadb apt sources https://review.openstack.org/513400 | 15:26 |
Taseer | evrardjp: Thanks. I have come so far. I think I'll make it, with your help :) | 15:26 |
evrardjp | we are there for that! | 15:27 |
evrardjp | Taseer: do you see a good way to integrate congress with other components? | 15:27 |
*** vnogin has joined #openstack-ansible | 15:27 | |
evrardjp | to have a real use of this scenario? | 15:27 |
evrardjp | or do you want to just test congress? | 15:27 |
evrardjp | let me check your spec | 15:28 |
*** vnogin has quit IRC | 15:28 | |
Taseer | okay, more of testing. maybe OPNFV community will get help. It was in their XCI Jira Tickets | 15:29 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest master: Use the deployment host to download images for tempest https://review.openstack.org/513200 | 15:29 |
odyssey4me | heh, found it - please re-review cloudnull & evrardjp if you have a moment | 15:29 |
evrardjp | Taseer: because we can maybe avoid you the burden of doing another scenario, if you just want to test the role | 15:29 |
evrardjp | odyssey4me: yeah I will review | 15:29 |
*** vnogin has joined #openstack-ansible | 15:30 | |
evrardjp | odyssey4me: makes sense! | 15:30 |
odyssey4me | role test at least, an extra scenario only if there're special integrations | 15:30 |
evrardjp | not so sure if basename would work here, but let's try. | 15:30 |
*** vnogin has quit IRC | 15:31 | |
Taseer | evrardjp: I will discuss this with XCI PTL Fatih, and see how he wants it to be done | 15:31 |
*** mrch has quit IRC | 15:32 | |
*** vnogin has joined #openstack-ansible | 15:32 | |
*** vnogin has quit IRC | 15:33 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-tests stable/newton: Initial OSA zuul v3 role jobs https://review.openstack.org/513726 | 15:35 |
odyssey4me | evrardjp do we need to update run_tests_common too? | 15:36 |
odyssey4me | probably not - I think newton/ocata's versions are very simple | 15:37 |
*** cshen has joined #openstack-ansible | 15:39 | |
*** vnogin has joined #openstack-ansible | 15:40 | |
*** vnogin has quit IRC | 15:41 | |
*** vnogin has joined #openstack-ansible | 15:42 | |
*** vnogin has quit IRC | 15:43 | |
openstackgerrit | Manuel Buil proposed openstack/openstack-ansible-os_neutron master: L3 Support for ODL deployments https://review.openstack.org/497458 | 15:44 |
cloudnull | ohay spotz: https://thenewstack.io/folding-big-tent-whats-next-openstack/?utm_content=buffer7802a&utm_medium=social&utm_source=twitter.com&utm_campaign=buffer -- nice piece ! | 15:44 |
*** vnogin has joined #openstack-ansible | 15:46 | |
cloudnull | a better link to the pod cast https://youtu.be/qWQ07WYYhP8 | 15:46 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Remove unified queue from gate pipeline https://review.openstack.org/513775 | 15:46 |
openstackgerrit | Merged openstack/openstack-ansible-os_cinder stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513423 | 15:47 |
*** vnogin has quit IRC | 15:47 | |
odyssey4me | evrardjp I will replicate https://review.openstack.org/513775 to all the other repositories next week some time. | 15:48 |
odyssey4me | I just wanted to get an initial patch up to remind myself. | 15:48 |
evrardjp | I can sed :p | 15:49 |
evrardjp | but I don't want to prevent you from sedding yourself! | 15:49 |
evrardjp | that can sound weird | 15:50 |
odyssey4me | if you can't sed or you don't have sed then you aint no friend of mine :p | 15:50 |
evrardjp | awk is the deal. | 15:50 |
evrardjp | "FRIENDSHIP DENIED" | 15:51 |
evrardjp | :) | 15:51 |
*** cshen has quit IRC | 15:53 | |
*** vnogin has joined #openstack-ansible | 15:55 | |
*** newmember has quit IRC | 15:55 | |
*** vnogin has quit IRC | 15:56 | |
spotz | thanks cloudnull and evrardjp | 15:57 |
evrardjp | spotz: I am watching your interview right now! :p | 15:57 |
evrardjp | oh darn everyboday already talk about it! | 15:58 |
evrardjp | I just received the tweet. | 15:58 |
evrardjp | <-- slow | 15:58 |
*** cshen has joined #openstack-ansible | 15:58 | |
spotz | evrardjp: heheh | 15:58 |
evrardjp | darn he wanted to grill you on all topics. | 15:59 |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: zuul.d: run.yml: Use new BUILD_DIR variable for log collection https://review.openstack.org/513706 | 15:59 |
*** cshen has quit IRC | 15:59 | |
*** vnogin has joined #openstack-ansible | 16:01 | |
*** vnogin has quit IRC | 16:01 | |
spotz | evrardjp: Yeah and I had no clue what we were going to be talking about. Work set it up | 16:02 |
evrardjp | :) | 16:03 |
*** vnogin has joined #openstack-ansible | 16:04 | |
*** vnogin has quit IRC | 16:05 | |
evrardjp | odyssey4me: don't work too much on this, I know you :p | 16:06 |
evrardjp | I have to go myself | 16:07 |
openstackgerrit | Merged openstack/openstack-ansible-os_tempest stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513424 | 16:12 |
openstackgerrit | Merged openstack/openstack-ansible-haproxy_server stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513626 | 16:12 |
openstackgerrit | Merged openstack/openstack-ansible-rabbitmq_server stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513625 | 16:12 |
openstackgerrit | Merged openstack/openstack-ansible-os_horizon stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513425 | 16:12 |
openstackgerrit | Merged openstack/openstack-ansible-os_heat stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513422 | 16:12 |
openstackgerrit | Merged openstack/openstack-ansible-lxc_hosts master: Stop trying to set MTU on LXC bridge interface https://review.openstack.org/512789 | 16:12 |
openstackgerrit | Merged openstack/openstack-ansible-openstack_openrc stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513662 | 16:12 |
openstackgerrit | Merged openstack/ansible-hardening master: Fix logic error https://review.openstack.org/512289 | 16:12 |
openstackgerrit | Merged openstack/openstack-ansible master: playbooks: os-lxc-container-setup: Only stop the container if it's running https://review.openstack.org/511768 | 16:12 |
*** cshen has joined #openstack-ansible | 16:12 | |
*** vnogin has joined #openstack-ansible | 16:14 | |
*** chyka has joined #openstack-ansible | 16:15 | |
*** vnogin has quit IRC | 16:15 | |
*** vnogin has joined #openstack-ansible | 16:16 | |
openstackgerrit | Jimmy McCrory proposed openstack/openstack-ansible-haproxy_server stable/pike: Regenerate haproxy.cfg on base config changes https://review.openstack.org/513791 | 16:17 |
*** gkadam has joined #openstack-ansible | 16:17 | |
*** cshen has quit IRC | 16:18 | |
*** vnogin has quit IRC | 16:18 | |
*** vnogin has joined #openstack-ansible | 16:18 | |
*** vnogin has quit IRC | 16:20 | |
openstackgerrit | Merged openstack/openstack-ansible-tests master: Add commonly used repositories to required projects https://review.openstack.org/513734 | 16:22 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests stable/pike: Add openstack-ansible-linters job https://review.openstack.org/513794 | 16:22 |
mhayden | oh look at that merging | 16:22 |
mhayden | happiness on a friday | 16:22 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests stable/pike: Add commonly used repositories to required projects https://review.openstack.org/513796 | 16:24 |
*** mbuil has quit IRC | 16:25 | |
openstackgerrit | Merged openstack/openstack-ansible-os_ceilometer master: Add OpenStack-Ansible metadata https://review.openstack.org/510482 | 16:25 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests stable/ocata: Add openstack-ansible-linters job https://review.openstack.org/513797 | 16:26 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests stable/ocata: Add commonly used repositories to required projects https://review.openstack.org/513798 | 16:27 |
*** vnogin has joined #openstack-ansible | 16:29 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests stable/newton: Add openstack-ansible-linters job https://review.openstack.org/513800 | 16:30 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests stable/newton: Add commonly used repositories to required projects https://review.openstack.org/513801 | 16:30 |
*** vnogin has quit IRC | 16:31 | |
*** vnogin has joined #openstack-ansible | 16:34 | |
*** vnogin has quit IRC | 16:35 | |
*** idlemind has joined #openstack-ansible | 16:41 | |
*** vnogin has joined #openstack-ansible | 16:41 | |
*** vnogin has quit IRC | 16:43 | |
openstackgerrit | Merged openstack/openstack-ansible-os_ironic master: Add OpenStack-Ansible metadata https://review.openstack.org/510493 | 16:43 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest master: Use the deployment host to download images for tempest https://review.openstack.org/513200 | 16:45 |
odyssey4me | silly mistake again cloudnull evrardjp ^ dammit you guys trust me too much :p | 16:45 |
odyssey4me | I suppose one trusts the tests, not the human. ;) | 16:45 |
* cloudnull goes to nit-pick now that he knows he cant trust odyssey4me | 16:46 | |
cloudnull | :P | 16:46 |
* odyssey4me goes to pour a scotch. | 16:46 | |
*** pbandark has quit IRC | 16:46 | |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: zuul.d: run.yml: Use new BUILD_DIR variable for log collection https://review.openstack.org/513706 | 16:47 |
*** vnogin has joined #openstack-ansible | 16:51 | |
*** vnogin has quit IRC | 16:52 | |
*** vnogin has joined #openstack-ansible | 16:58 | |
*** vnogin has quit IRC | 16:59 | |
openstackgerrit | Major Hayden proposed openstack/openstack-ansible master: Remove jinja from failed_when https://review.openstack.org/510995 | 17:04 |
*** gkadam has quit IRC | 17:05 | |
*** vnogin has joined #openstack-ansible | 17:08 | |
*** electrofelix has quit IRC | 17:09 | |
*** vnogin has quit IRC | 17:09 | |
openstackgerrit | Major Hayden proposed openstack/openstack-ansible-os_swift stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513813 | 17:10 |
openstackgerrit | Major Hayden proposed openstack/openstack-ansible-os_swift stable/pike: Suppress rsync warnings w/command https://review.openstack.org/501263 | 17:10 |
*** vnogin has joined #openstack-ansible | 17:11 | |
*** vnogin has quit IRC | 17:12 | |
*** asb47 has quit IRC | 17:13 | |
*** gunix has quit IRC | 17:15 | |
Tahvok | odyssey4me: I've checked external dns without providing admin* credentials or url config | 17:17 |
Tahvok | And unfortunately it does not work at all without them | 17:17 |
Tahvok | Unfortunately I'm on ocata, and unable to confirm this currently for pike release. But, I've checked the code, and found almost no work since januarly this year, so I'm 99% positive that it will not work in pike release as well: | 17:18 |
Tahvok | https://github.com/openstack/neutron/commits/master/neutron/services/externaldns/drivers/designate/driver.py | 17:19 |
*** mma_ has quit IRC | 17:20 | |
Tahvok | I will try to confirm at #openstack-neutron channel, and will open a bug if needed, however my patches currently cannot work without setting the admin* and url configurations | 17:21 |
*** mma has joined #openstack-ansible | 17:21 | |
*** germs has joined #openstack-ansible | 17:21 | |
*** germs1 has joined #openstack-ansible | 17:23 | |
*** gouthamr has quit IRC | 17:24 | |
*** germs has quit IRC | 17:25 | |
*** vnogin has joined #openstack-ansible | 17:26 | |
openstackgerrit | Marc Gariépy (mgariepy) proposed openstack/openstack-ansible stable/newton: Update haproxy check script in keepalived. https://review.openstack.org/513814 | 17:27 |
*** vnogin has quit IRC | 17:27 | |
dmsimard | mgariepy: hey are you around Montreal ? I forget | 17:31 |
*** vnogin has joined #openstack-ansible | 17:31 | |
*** vnogin has quit IRC | 17:33 | |
*** germs1 is now known as germs | 17:34 | |
mgariepy | dmsimard, i'm in sherbrooke. | 17:34 |
mgariepy | so about 2h away from montreal | 17:34 |
dmsimard | mgariepy: oh, dang, a bit far. We're hosting an Ansible Meetup in Montreal in November if you happen to be around https://www.meetup.com/Ansible-Montreal/events/244352092/ | 17:35 |
openstackgerrit | Merged openstack/openstack-ansible-rsyslog_server stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513663 | 17:38 |
mgariepy | c'est effectivement pas a la porte :) | 17:38 |
*** dave-mcc_ is now known as dave-mccowan | 17:39 | |
openstackgerrit | Major Hayden proposed openstack/ansible-hardening master: Optionally search for world-writable files https://review.openstack.org/513817 | 17:42 |
mhayden | dmsimard: s'il vous plaît passer la poutine | 17:42 |
*** stuartgr has left #openstack-ansible | 17:44 | |
openstackgerrit | Merged openstack/openstack-ansible-pip_install stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513627 | 17:44 |
openstackgerrit | Merged openstack/openstack-ansible-os_glance stable/ocata: Initial OSA zuul v3 role jobs https://review.openstack.org/513739 | 17:55 |
mgariepy | mhayden, can you push this one please ? https://review.openstack.org/#/c/512386/ | 17:56 |
* mhayden ganders | 17:56 | |
mhayden | mgariepy: can i trade you for a looksee at https://review.openstack.org/#/c/513487/ ? | 17:59 |
mhayden | it's a big patch | 17:59 |
mhayden | pace yourself | 17:59 |
mgariepy | what is fedora ? | 18:00 |
*** cshen has joined #openstack-ansible | 18:02 | |
openstackgerrit | Major Hayden proposed openstack/ansible-hardening master: Optionally search for world-writable files https://review.openstack.org/513817 | 18:05 |
mhayden | mgariepy: oh, i see whatcha did there | 18:05 |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: zuul.d: run.yml: Use new BUILD_DIR variable for log collection https://review.openstack.org/513706 | 18:13 |
openstackgerrit | Merged openstack/openstack-ansible-tests stable/ocata: Initial OSA zuul v3 role jobs https://review.openstack.org/513724 | 18:13 |
*** vnogin has joined #openstack-ansible | 18:21 | |
*** acormier has joined #openstack-ansible | 18:21 | |
*** masber has quit IRC | 18:22 | |
*** acormier has quit IRC | 18:24 | |
*** acormier has joined #openstack-ansible | 18:24 | |
*** yolanda has quit IRC | 18:24 | |
*** vnogin has quit IRC | 18:26 | |
openstackgerrit | Merged openstack/openstack-ansible-os_molteniron master: Add OpenStack-Ansible metadata https://review.openstack.org/511449 | 18:31 |
openstackgerrit | Merged openstack/openstack-ansible-galera_client stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513624 | 18:39 |
*** cshen has quit IRC | 18:47 | |
mgariepy | when is the next tag for Pike due ? | 18:54 |
mgariepy | this one would also be useful if some core are still available: https://review.openstack.org/#/c/513814/ | 19:06 |
mhayden | mgariepy: gotcha | 19:06 |
mgariepy | thanks mhayden | 19:07 |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: [DNM]zuul.d: run.yml: Use new BUILD_DIR variable for log collection https://review.openstack.org/513706 | 19:07 |
openstackgerrit | Merged openstack/openstack-ansible-tests master: Remove unified queue from gate pipeline https://review.openstack.org/513775 | 19:13 |
openstackgerrit | Merged openstack/openstack-ansible-lxc_hosts master: Remove the continue option for aria2c https://review.openstack.org/512386 | 19:13 |
*** markvoelker has quit IRC | 19:14 | |
*** taseer1 has joined #openstack-ansible | 19:16 | |
*** Taseer has quit IRC | 19:16 | |
*** taseer2 has joined #openstack-ansible | 19:17 | |
snowman4839 | when defining the ceph mons in user_variables, do you use the managment bridge ip or the storage bridge ip? | 19:18 |
*** taseer1 has quit IRC | 19:21 | |
*** dxiri has quit IRC | 19:26 | |
*** acormier has quit IRC | 19:27 | |
*** dxiri has joined #openstack-ansible | 19:28 | |
openstackgerrit | Merged openstack/openstack-ansible-tests stable/pike: Add Fedora support to bindep install step https://review.openstack.org/513487 | 19:30 |
openstackgerrit | Merged openstack/openstack-ansible-tests stable/newton: Initial OSA zuul v3 role jobs https://review.openstack.org/513726 | 19:30 |
openstackgerrit | Merged openstack/openstack-ansible-os_glance stable/newton: Initial OSA zuul v3 role jobs https://review.openstack.org/513743 | 19:30 |
openstackgerrit | Merged openstack/openstack-ansible-os_neutron stable/newton: Initial OSA zuul v3 role jobs https://review.openstack.org/513733 | 19:30 |
openstackgerrit | Merged openstack/openstack-ansible-os_neutron stable/ocata: Initial OSA zuul v3 role jobs https://review.openstack.org/513727 | 19:30 |
openstackgerrit | Merged openstack/openstack-ansible-tests stable/newton: Add openstack-ansible-linters job https://review.openstack.org/513800 | 19:30 |
openstackgerrit | Merged openstack/openstack-ansible-tests stable/newton: Add commonly used repositories to required projects https://review.openstack.org/513801 | 19:30 |
*** dxiri has quit IRC | 19:32 | |
*** acormier has joined #openstack-ansible | 19:33 | |
*** markvoelker has joined #openstack-ansible | 19:45 | |
*** weezS has quit IRC | 19:47 | |
*** markvoelker has quit IRC | 19:49 | |
*** mma has quit IRC | 19:53 | |
*** markvoelker has joined #openstack-ansible | 19:54 | |
*** markvoelker has quit IRC | 19:59 | |
*** perniciouscaffei has joined #openstack-ansible | 20:01 | |
openstackgerrit | Monty Taylor proposed openstack/openstack-ansible-tests master: [DNM]zuul.d: run.yml: Use new BUILD_DIR variable for log collection https://review.openstack.org/513706 | 20:01 |
*** markvoelker has joined #openstack-ansible | 20:03 | |
*** markvoelker has quit IRC | 20:07 | |
*** yolanda has joined #openstack-ansible | 20:10 | |
*** dxiri has joined #openstack-ansible | 20:11 | |
*** markvoelker has joined #openstack-ansible | 20:12 | |
*** markvoelker has quit IRC | 20:13 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible master: Initial integrated zuulv3 jobs https://review.openstack.org/513406 | 20:13 |
*** markvoelker has joined #openstack-ansible | 20:14 | |
*** dxiri has quit IRC | 20:16 | |
*** acormier has quit IRC | 20:22 | |
*** acormier has joined #openstack-ansible | 20:22 | |
*** acormier has quit IRC | 20:28 | |
*** acormier has joined #openstack-ansible | 20:29 | |
*** markvoelker_ has joined #openstack-ansible | 20:29 | |
*** dxiri has joined #openstack-ansible | 20:32 | |
*** markvoelker has quit IRC | 20:32 | |
*** acormier has quit IRC | 20:33 | |
*** dave-mccowan has quit IRC | 20:35 | |
*** dave-mccowan has joined #openstack-ansible | 20:36 | |
*** dxiri has quit IRC | 20:36 | |
*** thorst has quit IRC | 20:39 | |
cloudnull | snowman4839: personally I'd use the storage address for all of that traffic however others may have a different opinion that | 20:44 |
cloudnull | mattt: logan-? | 20:44 |
*** dxiri has joined #openstack-ansible | 20:48 | |
*** dxiri has quit IRC | 20:52 | |
*** dxiri has joined #openstack-ansible | 21:00 | |
*** jwitko_ has joined #openstack-ansible | 21:02 | |
*** dxiri has quit IRC | 21:05 | |
*** jwitko has quit IRC | 21:05 | |
*** jwitko_ has quit IRC | 21:09 | |
*** dxiri has joined #openstack-ansible | 21:11 | |
*** dxiri has quit IRC | 21:15 | |
*** dxiri has joined #openstack-ansible | 21:15 | |
*** masber has joined #openstack-ansible | 21:18 | |
*** thorst has joined #openstack-ansible | 21:20 | |
*** thorst has quit IRC | 21:20 | |
*** masber has quit IRC | 21:22 | |
*** jwitko has joined #openstack-ansible | 21:35 | |
*** rodolof has quit IRC | 21:38 | |
*** rodolof has joined #openstack-ansible | 21:38 | |
openstackgerrit | Merged openstack/openstack-ansible-tests stable/ocata: Add openstack-ansible-linters job https://review.openstack.org/513797 | 21:40 |
*** jwitko has quit IRC | 21:41 | |
*** rodolof has quit IRC | 21:42 | |
*** rodolof has joined #openstack-ansible | 21:42 | |
*** shardy has quit IRC | 21:45 | |
*** chyka has quit IRC | 21:50 | |
*** vnogin has joined #openstack-ansible | 22:02 | |
openstackgerrit | Merged openstack/openstack-ansible-tests stable/ocata: Add commonly used repositories to required projects https://review.openstack.org/513798 | 22:05 |
*** vnogin has quit IRC | 22:07 | |
*** aaron has joined #openstack-ansible | 22:13 | |
*** aaron is now known as Guest54922 | 22:13 | |
*** dxiri has quit IRC | 22:24 | |
*** rodolof has quit IRC | 22:25 | |
*** perniciouscaffei has quit IRC | 22:29 | |
*** esberglu has quit IRC | 22:34 | |
snowman4839 | should setup-infrastructure fail if the haproxy service doesn't come up? | 22:35 |
openstackgerrit | Merged openstack/openstack-ansible-tests stable/pike: Add openstack-ansible-linters job https://review.openstack.org/513794 | 22:40 |
openstackgerrit | Merged openstack/openstack-ansible-tests stable/pike: Add commonly used repositories to required projects https://review.openstack.org/513796 | 22:40 |
openstackgerrit | Merged openstack/openstack-ansible-os_swift master: Add openstack-ansible-linters test https://review.openstack.org/513126 | 22:41 |
openstackgerrit | Merged openstack/openstack-ansible-openstack_hosts stable/pike: Initial OSA zuul v3 role jobs https://review.openstack.org/513661 | 22:41 |
*** thorst has joined #openstack-ansible | 23:21 | |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible master: Improve execution time by using dynamic groups https://review.openstack.org/513876 | 23:26 |
*** thorst has quit IRC | 23:26 | |
cloudnull | snowman4839: yes. setup-infra is a meta playbook which simply calls other full playbooks. | 23:28 |
cloudnull | you can run haproxy-install.yml alone if you wish to test just that part | 23:28 |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible master: Improve execution time by using dynamic groups https://review.openstack.org/513876 | 23:28 |
*** acormier has joined #openstack-ansible | 23:29 | |
*** acormier has quit IRC | 23:33 | |
*** Guest54922 has quit IRC | 23:45 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!