| *** sdake has quit IRC | 00:01 | |
| *** karimb has quit IRC | 00:03 | |
| *** ooolpbot has joined #tripleo | 00:11 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 00:11 |
|---|---|---|
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1659384 | 00:11 |
| *** ooolpbot has quit IRC | 00:11 | |
| openstack | Launchpad bug 1659384 in tripleo "All ovb jobs failing to set up snmp" [Critical,In progress] - Assigned to Ben Nemec (bnemec) | 00:11 |
| *** ansmith has joined #tripleo | 00:15 | |
| openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: POC: Use tripleo-repos in tripleo.sh https://review.openstack.org/395813 | 00:18 |
| *** dsneddon has quit IRC | 00:23 | |
| openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart-extras: WIP: Support uploading files in modify image role https://review.openstack.org/425472 | 00:30 |
| *** bfournie has joined #tripleo | 00:31 | |
| *** limao has joined #tripleo | 00:41 | |
| *** rbrady-afk has quit IRC | 00:47 | |
| *** trown is now known as trown|outtypewww | 00:47 | |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart-extras: pass the topology of the overcloud w/ topology vs extra-args https://review.openstack.org/425453 | 00:47 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: WIP: Support builds in multinode https://review.openstack.org/425474 | 00:48 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: Collect only logs we need https://review.openstack.org/425459 | 00:57 |
| openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart-extras: Support exclude list for logs collecting https://review.openstack.org/425395 | 01:00 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add release notes for Ocata 6.0.0 https://review.openstack.org/424365 | 01:07 |
| *** ooolpbot has joined #tripleo | 01:11 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 01:11 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1659384 | 01:11 |
| *** ooolpbot has quit IRC | 01:11 | |
| openstack | Launchpad bug 1659384 in tripleo "All ovb jobs failing to set up snmp" [Critical,In progress] - Assigned to Ben Nemec (bnemec) | 01:11 |
| openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart-extras: Add host info collection to collect-logs https://review.openstack.org/413224 | 01:13 |
| openstackgerrit | Andy Smith proposed openstack/puppet-tripleo: Use rpc and notify transport_url for oslo_messaging backends https://review.openstack.org/425484 | 01:15 |
| *** sdake has joined #tripleo | 01:33 | |
| *** sdake_ has joined #tripleo | 01:35 | |
| *** sdake has quit IRC | 01:38 | |
| *** sdake_ has quit IRC | 01:43 | |
| openstackgerrit | Michael Chapman proposed openstack/tripleo-quickstart: Add the ability to run all tasks as root https://review.openstack.org/418198 | 01:45 |
| *** rajinir has quit IRC | 01:46 | |
| *** sdake has joined #tripleo | 01:49 | |
| *** sdake_ has joined #tripleo | 01:50 | |
| *** sdake has quit IRC | 01:54 | |
| *** sdake_ has quit IRC | 02:09 | |
| *** ooolpbot has joined #tripleo | 02:11 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 02:11 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1659384 | 02:11 |
| *** ooolpbot has quit IRC | 02:11 | |
| openstack | Launchpad bug 1659384 in tripleo "All ovb jobs failing to set up snmp" [Critical,In progress] - Assigned to Ben Nemec (bnemec) | 02:11 |
| openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart-extras: Add option to leave tar.gz logs files and flat log https://review.openstack.org/425497 | 02:11 |
| *** sdake has joined #tripleo | 02:11 | |
| *** rbrady-afk has joined #tripleo | 02:13 | |
| *** yamahata has quit IRC | 02:16 | |
| *** cwolferh has quit IRC | 02:17 | |
| EmilienM | jrist: back from hockey (and alive) | 02:20 |
| *** noama has quit IRC | 02:20 | |
| EmilienM | jrist: no there is not, I use to read git log | 02:20 |
| EmilienM | thrash: thanks | 02:20 |
| *** noama has joined #tripleo | 02:20 | |
| openstackgerrit | Merged openstack/python-tripleoclient: Migrate overcloud delete to full workflow https://review.openstack.org/422806 | 02:29 |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Move Multinode environments to THT https://review.openstack.org/424190 | 02:29 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: (newton) Import multinode + pingtest from tripleo-ci https://review.openstack.org/424811 | 02:29 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: ceph-rgw: add missing user parameter https://review.openstack.org/423526 | 02:30 |
| EmilienM | YEAH | 02:34 |
| EmilienM | sorry | 02:34 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Fix for AllNodesExtraConfig and fix environment files to create swap files/partitions https://review.openstack.org/424079 | 02:35 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: WIP: Support builds in multinode https://review.openstack.org/425474 | 02:36 |
| *** fzdarsky_ has joined #tripleo | 02:37 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Add scenario005 job (4nodes) https://review.openstack.org/418098 | 02:38 |
| openstackgerrit | Merged openstack/tripleo-puppet-elements: Add explicit install of net-snmp https://review.openstack.org/425346 | 02:39 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Add scenario005 scenario https://review.openstack.org/418099 | 02:39 |
| openstackgerrit | Andy Smith proposed openstack/puppet-tripleo: Use rpc and notify transport_url for oslo_messaging backends https://review.openstack.org/425484 | 02:41 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Add scenario005 scenario https://review.openstack.org/418099 | 02:41 |
| *** fzdarsky|afk has quit IRC | 02:41 | |
| *** owalsh is now known as owalsh-afk | 02:43 | |
| *** ansmith has quit IRC | 02:46 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: TEST: DONT RECHECK or REVIEW: periodic jobs https://review.openstack.org/359215 | 02:50 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Use os-net-config in multinode jobs https://review.openstack.org/425241 | 02:51 |
| *** sdake has quit IRC | 02:58 | |
| *** cwolferh has joined #tripleo | 03:00 | |
| *** bkopilov has quit IRC | 03:00 | |
| *** d0ugal has quit IRC | 03:00 | |
| *** yamahata has joined #tripleo | 03:10 | |
| *** ooolpbot has joined #tripleo | 03:11 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 03:11 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1659384 | 03:11 |
| *** ooolpbot has quit IRC | 03:11 | |
| openstack | Launchpad bug 1659384 in tripleo "All ovb jobs failing to set up snmp" [Critical,In progress] - Assigned to Ben Nemec (bnemec) | 03:11 |
| openstackgerrit | Ben Kero proposed openstack/tripleo-quickstart: Add optional undercloud-setup role to quickstart-extras playbook https://review.openstack.org/424947 | 03:16 |
| openstackgerrit | Ben Kero proposed openstack-infra/tripleo-ci: WIP: Replace multinode playbook with default quickstart playbook https://review.openstack.org/424683 | 03:17 |
| *** d0ugal has joined #tripleo | 03:18 | |
| *** newmember has quit IRC | 03:23 | |
| *** sdake has joined #tripleo | 03:46 | |
| openstackgerrit | Tim Rozet proposed openstack/puppet-tripleo: Enables OpenDaylight Clustering in HA deployments https://review.openstack.org/425383 | 03:54 |
| *** cwolferh has quit IRC | 03:56 | |
| openstackgerrit | Ben Kero proposed openstack/tripleo-quickstart-extras: overcloud-prep-images: Ensure only 1 overcloud-full.qcow2 in glance https://review.openstack.org/425516 | 04:03 |
| *** sdake_ has joined #tripleo | 04:07 | |
| *** sdake has quit IRC | 04:09 | |
| *** ooolpbot has joined #tripleo | 04:11 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 04:11 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1659384 | 04:11 |
| *** ooolpbot has quit IRC | 04:11 | |
| openstack | Launchpad bug 1659384 in tripleo "All ovb jobs failing to set up snmp" [Critical,In progress] - Assigned to Ben Nemec (bnemec) | 04:11 |
| *** thrash is now known as thrash|g0ne | 04:12 | |
| *** bkopilov has joined #tripleo | 04:28 | |
| *** sdake has joined #tripleo | 04:31 | |
| *** sdake_ has quit IRC | 04:32 | |
| *** ianw is now known as ianw_pto | 04:34 | |
| dtrainor | ping bandini, i'm still seeing failures with gate-tripleo-ci-centos-7-ovb-ha and gate-tripleo-ci-centos-7-ovb-nonha. are we still waiting for some part of https://review.openstack.org/#/c/362389/ to be incorporated? | 04:36 |
| openstackgerrit | Jiri Tomasek proposed openstack/tripleo-ui: [WIP] Nodes Table tagging https://review.openstack.org/424547 | 04:42 |
| larsks | where does the certificate with CN=overcloud live? | 04:44 |
| *** yamahata has quit IRC | 04:46 | |
| *** sdake has quit IRC | 04:49 | |
| openstackgerrit | Merged openstack/tripleo-puppet-elements: Add explicit install of net-snmp https://review.openstack.org/425339 | 04:51 |
| *** cwolferh has joined #tripleo | 04:54 | |
| *** Guest58531 has quit IRC | 05:04 | |
| openstackgerrit | Ben Kero proposed openstack/tripleo-quickstart: Add optional undercloud-setup role to quickstart-extras playbook https://review.openstack.org/424947 | 05:04 |
| *** mgagne has joined #tripleo | 05:07 | |
| *** mgagne is now known as Guest33539 | 05:07 | |
| *** ealcaniz has joined #tripleo | 05:12 | |
| *** jtomasek has quit IRC | 05:18 | |
| *** abregman has joined #tripleo | 05:22 | |
| *** sdake has joined #tripleo | 05:25 | |
| *** rlandy has quit IRC | 05:42 | |
| *** sdake has quit IRC | 05:45 | |
| *** ealcaniz has quit IRC | 05:49 | |
| *** ealcaniz has joined #tripleo | 05:54 | |
| *** sdake has joined #tripleo | 06:06 | |
| openstackgerrit | Ben Kero proposed openstack/tripleo-quickstart: Add optional undercloud-setup role to quickstart-extras playbook https://review.openstack.org/424947 | 06:14 |
| *** jprovazn has joined #tripleo | 06:15 | |
| *** sdake has quit IRC | 06:16 | |
| *** sdake has joined #tripleo | 06:17 | |
| *** ealcaniz has quit IRC | 06:17 | |
| *** crenguta has joined #tripleo | 06:17 | |
| openstackgerrit | Saravanan KR proposed openstack/puppet-tripleo: Add support to changing the Rabbitmq password on update https://review.openstack.org/402061 | 06:20 |
| *** sdake_ has joined #tripleo | 06:21 | |
| *** ealcaniz has joined #tripleo | 06:21 | |
| bandini | dtrainor: do you have link to failing jobs | 06:22 |
| bandini | dtrainor: http://tripleo.org/cistatus.html seems moderately green | 06:23 |
| *** sdake has quit IRC | 06:23 | |
| *** sdake_ has quit IRC | 06:27 | |
| *** saneax has joined #tripleo | 06:34 | |
| *** saneax is now known as saneax-_-|AFK | 06:46 | |
| *** rasca has joined #tripleo | 06:49 | |
| *** florianf has joined #tripleo | 06:51 | |
| *** ealcaniz has quit IRC | 06:56 | |
| openstackgerrit | Ben Kero proposed openstack/tripleo-quickstart: Add optional undercloud-setup role to quickstart-extras playbook https://review.openstack.org/424947 | 07:03 |
| *** tesseract has joined #tripleo | 07:11 | |
| *** tdasilva has quit IRC | 07:15 | |
| *** yprokule has joined #tripleo | 07:26 | |
| *** noama has quit IRC | 07:27 | |
| *** masco has joined #tripleo | 07:27 | |
| *** noama has joined #tripleo | 07:28 | |
| *** noama has quit IRC | 07:29 | |
| *** cylopez has joined #tripleo | 07:30 | |
| *** tzumainn has joined #tripleo | 07:35 | |
| openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Use the builtin Mistral engine command 'fail' when failing workflows https://review.openstack.org/425047 | 07:36 |
| openstackgerrit | Michele Baldessari proposed openstack/tripleo-image-elements: Set IRONIC_AGENT_MULTIPATH_DRIVERS to '' when undefined https://review.openstack.org/425565 | 07:38 |
| bandini | yolanda: could you take a look at ^, it might very well be that the problem in itself is that the env variable is unset from time to time, so the above would just be a workaround | 07:42 |
| openstackgerrit | Ben Kero proposed openstack/tripleo-quickstart: Add optional undercloud-setup role to quickstart-extras playbook https://review.openstack.org/424947 | 07:45 |
| *** tremble has joined #tripleo | 07:47 | |
| *** tremble has joined #tripleo | 07:47 | |
| *** tzumainn has quit IRC | 07:50 | |
| *** aufi has joined #tripleo | 07:52 | |
| openstackgerrit | Chris Jones proposed openstack/tripleo-common: Add action for generating fencing parameters. https://review.openstack.org/422321 | 07:53 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: Add SkipUpgradeConfigTags for upgrade config https://review.openstack.org/424072 | 07:54 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: Adds a pre-upgrade check that service is running (step0) https://review.openstack.org/414598 | 07:54 |
| *** jlinkes has joined #tripleo | 07:54 | |
| *** tdasilva has joined #tripleo | 07:55 | |
| *** jlinkes has quit IRC | 07:59 | |
| openstackgerrit | Merged openstack/tripleo-common: Adding release nodes for Congress and Tacker https://review.openstack.org/425436 | 08:02 |
| *** dsariel has joined #tripleo | 08:03 | |
| *** aufi has quit IRC | 08:03 | |
| *** ooolpbot has joined #tripleo | 08:11 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 08:11 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1659493 | 08:11 |
| *** ooolpbot has quit IRC | 08:11 | |
| openstack | Launchpad bug 1659493 in tripleo "IRONIC_AGENT_MULTIPATH_DRIVERS is undefined during IPA image build" [Critical,In progress] - Assigned to Michele Baldessari (michele) | 08:11 |
| *** jlinkes has joined #tripleo | 08:13 | |
| *** aufi has joined #tripleo | 08:18 | |
| *** tdasilva has quit IRC | 08:22 | |
| openstackgerrit | Martin André proposed openstack/tripleo-common: Install python-heat-agent-json-file in overcloud-full https://review.openstack.org/424552 | 08:23 |
| *** b00tcat has joined #tripleo | 08:23 | |
| *** zoli|gone is now known as zoli | 08:26 | |
| *** abregman is now known as abregman|mtg | 08:27 | |
| *** amoralej has joined #tripleo | 08:27 | |
| openstackgerrit | Carlos Camacho proposed openstack-infra/tripleo-ci: Add SNMP support to major CI upgrade https://review.openstack.org/424077 | 08:27 |
| *** ccamacho has joined #tripleo | 08:27 | |
| *** tdasilva has joined #tripleo | 08:29 | |
| *** panda|zZ is now known as panda | 08:31 | |
| panda | bandini: Why is IRONIC_AGENT_MULTIPATH_DRIVERS sometimes not set ? what happens when dracut --add-drivers is called without arguments ? | 08:33 |
| bandini | panda: I don't know why it is not set. the issue is that bash then complains it is undefined | 08:34 |
| panda | bandini: I think that change is going to fail because dracut will try to add $RAMDISK as a driver | 08:34 |
| bandini | panda: oh let me see | 08:34 |
| bandini | panda: ah right | 08:34 |
| bandini | panda: who is supposed to set IRONIC_AGENT_MULTIPATH_DRIVERS all the time? | 08:35 |
| openstackgerrit | Ben Kero proposed openstack/tripleo-quickstart: Add optional undercloud-setup role to quickstart-extras playbook https://review.openstack.org/424947 | 08:37 |
| *** aufi has quit IRC | 08:37 | |
| openstackgerrit | Michele Baldessari proposed openstack/tripleo-image-elements: Set IRONIC_AGENT_MULTIPATH_DRIVERS to '' when undefined https://review.openstack.org/425565 | 08:38 |
| *** ealcaniz has joined #tripleo | 08:39 | |
| *** rasca has quit IRC | 08:41 | |
| bandini | panda: thanks for spotting that! | 08:41 |
| bandini | panda: I'd love to understand who is in charge to set that variable though as the commit that introduced it says it defaults to a couple of drivers, yet Ng and I have seen a bunch of jobs fail due to it being undefined | 08:42 |
| *** rasca has joined #tripleo | 08:42 | |
| panda | bandini: unthank me for all the things I've not spotted | 08:42 |
| bandini | lol | 08:42 |
| panda | bandini: I think it's set before calling dib, but i'm not sure, I'm checking | 08:43 |
| bandini | panda: Merci! | 08:43 |
| panda | bandini: the documentation says it defaults to something | 08:43 |
| *** jpich has joined #tripleo | 08:44 | |
| bandini | panda: right, for example here http://logs.openstack.org/98/417898/28/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/f1bf016/console.html if you download undercloud.tar.xz and look in /home/jenkins/ipa*log we see it is undefined | 08:44 |
| bandini | and the dib building fails | 08:44 |
| panda | bandini: been there, done that :) | 08:46 |
| bandini | panda: yes but did you collect the 20$? :D | 08:46 |
| bandini | we should start the game tripleopoly | 08:46 |
| *** chem has quit IRC | 08:46 | |
| *** gfidente has joined #tripleo | 08:48 | |
| *** gfidente has quit IRC | 08:48 | |
| *** gfidente has joined #tripleo | 08:48 | |
| *** aufi has joined #tripleo | 08:49 | |
| *** nyechiel_ has joined #tripleo | 08:49 | |
| *** jtomasek has joined #tripleo | 08:50 | |
| *** arxcruz has quit IRC | 08:52 | |
| *** iranzo has joined #tripleo | 08:52 | |
| *** arxcruz has joined #tripleo | 08:53 | |
| b00tcat | morning! is deploying an overcloud (Newton release of tripleo) with RHEL 7.3 supported? | 08:57 |
| openstackgerrit | Julie Pichon proposed openstack-infra/tripleo-ci: Remove references to Liberty (now EOL'd) https://review.openstack.org/417814 | 08:58 |
| openstackgerrit | Julie Pichon proposed openstack-infra/tripleo-ci: Add newton branch to delorean-build branches https://review.openstack.org/397439 | 08:58 |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #tripleo | 09:00 | |
| *** apetrich has quit IRC | 09:01 | |
| *** shardy has joined #tripleo | 09:02 | |
| b00tcat | ah, nvm, I was being hit by https://bugs.launchpad.net/diskimage-builder/+bug/1641106 I just found out | 09:02 |
| openstack | Launchpad bug 1641106 in diskimage-builder "centos7 image missing lsb_release" [Undecided,In progress] - Assigned to Oliver Walsh (owalsh) | 09:02 |
| shardy | morning all | 09:03 |
| openstackgerrit | Raoul Scarazzini proposed openstack/tripleo-quickstart-extras: Fix node cycle on HA Validation https://review.openstack.org/425587 | 09:03 |
| *** abregman|mtg is now known as abregman | 09:04 | |
| gfidente | morning shardy :) | 09:04 |
| shardy | ugh, looks like stable/newton CI is still broken? | 09:05 |
| shardy | 4 days in a row :( | 09:05 |
| openstackgerrit | mathieu bultel proposed openstack/tripleo-quickstart-extras: Add no-cleanup arg for overcloud deployment https://review.openstack.org/407059 | 09:05 |
| shardy | https://review.openstack.org/#/c/425343/ hasn't landed yet, I guess that's why | 09:05 |
| *** crenguta has quit IRC | 09:06 | |
| matbu | shardy: o/ | 09:07 |
| *** crenguta has joined #tripleo | 09:08 | |
| *** ooolpbot has joined #tripleo | 09:11 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 09:11 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1659493 | 09:11 |
| *** ooolpbot has quit IRC | 09:11 | |
| openstack | Launchpad bug 1659493 in tripleo "IRONIC_AGENT_MULTIPATH_DRIVERS is undefined during IPA image build" [Critical,In progress] - Assigned to Michele Baldessari (michele) | 09:11 |
| shardy | marios, matbu: Hey given that we're blocked on CI again, should I perhaps consider rebasing all outstanding composable upgrades patches (including the ones you guys have posted) into one branch on gerrit? | 09:13 |
| shardy | Or I could push a temporary fork of t-h-t we could test from | 09:13 |
| marios | o/ shardy sure go ahead - whatever works/easiest | 09:13 |
| shardy | a long series in gerrit will be a pain for rebase (and more load on CI), but at least we can all be locally testing the same thing | 09:13 |
| matbu | shardy: yep +1 | 09:13 |
| shardy | I'm happy to just keep rebasing it until we get CI unblocked | 09:14 |
| marios | shardy: i'm applying them downstream anyway so my tht will still trail a bit | 09:14 |
| shardy | Ok, cool, will do that now, thanks | 09:14 |
| shardy | marios: Ok, well I guess it's still going to be easier for you as the series should apply with less conflicts | 09:14 |
| marios | shardy: for sure | 09:14 |
| matbu | marios: are you working on downstream in all your test/dev ? | 09:14 |
| shardy | I just want a way to check out one t-h-t branch and locally test everything | 09:14 |
| marios | matbu: yeah i start with vanila 10 | 09:15 |
| * shardy is only testing upstream builds atm | 09:15 | |
| matbu | me too :) | 09:15 |
| marios | matbu: i have a second box i should setup(it currently has vanila 9 on it for -> 10 testing) | 09:15 |
| shardy | I guess it's pretty much the same - I deploy stable/newton tht/overcloud-full, then upgrade it like we do in CI | 09:15 |
| marios | matbu: yeah it makes sense for me to have this setup because it means i have same as the qe guys | 09:15 |
| matbu | marios: yes i know you're right | 09:16 |
| *** fzdarsky_ is now known as fzdarsky | 09:16 | |
| shardy | Nice, I guess we've got better coverage with these two setups - my focus is on getting CI passes, thus I need to be as close to that env as possible :) | 09:16 |
| marios | matbu: well obviously we need both and it doesn't happen downstream if it didn't happen upstream yet anyway... | 09:16 |
| *** limao has quit IRC | 09:17 | |
| *** crenguta has quit IRC | 09:17 | |
| *** crenguta has joined #tripleo | 09:19 | |
| openstackgerrit | Martin André proposed openstack/puppet-tripleo: Rename controller_admin_vip to controller_admin_host https://review.openstack.org/425296 | 09:20 |
| *** mcornea has joined #tripleo | 09:21 | |
| *** dbecker has joined #tripleo | 09:21 | |
| *** athomas has joined #tripleo | 09:21 | |
| *** iranzo has quit IRC | 09:24 | |
| *** lucas-afk is now known as lucasagomes | 09:25 | |
| shardy | gfidente: Hey question re the OSD service upgrade | 09:25 |
| *** dsariel has quit IRC | 09:25 | |
| shardy | gfidente: is there any reason we can't just do a rolling upgrade of those, or does it have to be manual? | 09:25 |
| shardy | I know in previous releases we just used a script, but this upgrade seems simpler? | 09:26 |
| gfidente | shardy rolling you mean from ansible? | 09:29 |
| gfidente | in this release we're mainly not upgrading ceph | 09:29 |
| gfidente | so stop/update/start is fine | 09:29 |
| gfidente | as long as we do it on the nodes one by one | 09:29 |
| openstackgerrit | Michele Baldessari proposed openstack/tripleo-image-elements: Set IRONIC_AGENT_MULTIPATH_DRIVERS to '' when undefined https://review.openstack.org/425565 | 09:29 |
| gfidente | in pike I think we'll have to upgrade to a new ceph release | 09:30 |
| gfidente | so the script might need a little more logic, but should be able to do it with this same process, just a longer list of tasks | 09:30 |
| openstackgerrit | Michele Baldessari proposed openstack/tripleo-image-elements: Set IRONIC_AGENT_MULTIPATH_DRIVERS to '' when undefined https://review.openstack.org/425565 | 09:31 |
| shardy | gfidente: ack, so https://review.openstack.org/#/c/423350/ is OK, and we don't need to disable upgrades for the CephStorage role this release | 09:32 |
| gfidente | ah yeah we don't need to do it manually no | 09:32 |
| shardy | marios: ^^ FYI - I thought you had a patch disabling upgrades for cephstorage but can't find it atm | 09:33 |
| marios | shardy: it landed already it was just the disable_upgrades flag, gfidente reading back | 09:34 |
| shardy | marios: Ok, cool - we may be able to tweak that to not disable for CephStorage, modulo some testing | 09:35 |
| shardy | when my batched upgrades patch lands | 09:35 |
| gfidente | I think I can deprecate the upgrade scripts too | 09:35 |
| marios | shardy: i mean it added the disable_upgrade_deployment into the default roles_data.yaml for ceph storage. i probably need to tweak https://review.openstack.org/#/c/419886/ too then if we aren't going to manually run the script we shouldn't deliver it either to ceph n odes | 09:35 |
| openstackgerrit | Christian Schwede proposed openstack/tripleo-common: Use Mistral to create Swift temporary URLs https://review.openstack.org/413229 | 09:35 |
| * shardy will test it today with multiple OSD nodes | 09:35 | |
| gfidente | we don't need to ship those with ocata | 09:36 |
| gfidente | I was assuming we were doing it from heat as well | 09:36 |
| shardy | marios: I think as mentioned in https://review.openstack.org/#/c/424661/ we should have the option of dumping the script on the nodes (either when automated upgrades are disabled, or perhaps always) | 09:36 |
| gfidente | from the service templates | 09:36 |
| marios | gfidente: so you mean to say, we could use ansible for all the steps that we are currently defining in https://github.com/openstack/tripleo-heat-templates/blob/8ab1918f026aaa9d06dd4cadd1ad5e15f06b220f/extraconfig/tasks/major_upgrade_ceph_storage.sh ? or i misunderstood | 09:36 |
| gfidente | marios yes in this release yes | 09:36 |
| shardy | marios: but yeah, I'm trying to figure out if we can automate things a little better this release | 09:36 |
| shardy | without so many operator driven steps | 09:37 |
| gfidente | the key thing is the fact that from newton to oacata the ceph release is not changing | 09:37 |
| *** derekh has joined #tripleo | 09:37 | |
| openstackgerrit | Luke Hinds proposed openstack/tripleo-heat-templates: Add AuditD composable service https://review.openstack.org/421873 | 09:37 |
| shardy | obviously we'll still need that for Compute, but perhaps not so much for *Storage? | 09:37 |
| gfidente | marios so none of that is needed, we should not run it at all | 09:37 |
| shardy | we probably need to start purging t-h-t of scripts that don't apply to this release soon | 09:37 |
| shardy | as it's a little confusing :) | 09:38 |
| gfidente | yeah ^^ that | 09:38 |
| marios | 11:36 < shardy> marios: I think as mentioned in https://review.openstack.org/#/c/424661/ we <-- wrt this i am referring to the current ceph upgrade script i.e. the one at https://github.com/openstack/tripleo-heat-templates/blob/8ab1918f026aaa9d06dd4cadd1ad5e15f06b220f/extraconfig/tasks/major_upgrade_ceph_storage.sh i think you were referring to the ansible tasks | 09:38 |
| lhinds | shardy: I added the capability map in the end for auditd, as a new patch might not make it in considering timelines for f-freeze | 09:38 |
| gfidente | /me will do after lunch | 09:38 |
| lhinds | shardy: do went with your idea :) | 09:38 |
| lhinds | do/so | 09:38 |
| marios | gfidente: i see so that was all special handling for the version jump? | 09:39 |
| gfidente | marios exactly | 09:39 |
| marios | gfidente: so we can just do like other services? stop, update, db sync/w.e.... profit? | 09:40 |
| gfidente | special handling as it went from bds scripts to systemd | 09:40 |
| gfidente | they added selinux | 09:40 |
| *** morazi has joined #tripleo | 09:40 | |
| *** jpena|off is now known as jpena | 09:40 | |
| gfidente | marios yep in newton>ocata yes | 09:40 |
| gfidente | we don't need the 'special handling' | 09:40 |
| marios | gfidente: but we still need the batched though? the concern for the compute nodes was 'one at a time' because i need to migrate my stuff between nodes | 09:40 |
| marios | gfidente: does the same apply for ceph? | 09:40 |
| gfidente | still batched yes | 09:40 |
| gfidente | because we need to keep the cluster active at all times to keep vms up | 09:41 |
| marios | gfidente: ok, so the migrate/failover is automated (so not the same as compute migration needing operator input) so we can automate the ceph upgrade as long as it is batch 1 | 09:41 |
| marios | gfidente: do you alsready have a review out for the ceph upgrade_tasks? | 09:41 |
| gfidente | ack | 09:41 |
| gfidente | shardy did mon/osd I added rgw | 09:41 |
| marios | gfidente: cool add me on review please if you didn't already | 09:42 |
| gfidente | and that is all we had in newton , I think mcornea tested it too | 09:42 |
| *** shardy has quit IRC | 09:42 | |
| gfidente | marios ack, thanks | 09:42 |
| *** shardy has joined #tripleo | 09:42 | |
| * shardy has flaky network today | 09:42 | |
| *** karimb has joined #tripleo | 09:43 | |
| openstackgerrit | Luke Hinds proposed openstack/puppet-tripleo: Add AuditD Profile https://review.openstack.org/421872 | 09:43 |
| *** karimb has quit IRC | 09:43 | |
| *** salmankhan has joined #tripleo | 09:46 | |
| *** tosky has joined #tripleo | 09:47 | |
| sshnaidm | panda, did you look at periodic jobs today? | 09:49 |
| panda | sshnaidm: they're all failing | 09:51 |
| sshnaidm | panda, nevermind, found a bug of bulding images | 09:51 |
| panda | sshnaidm: working with bandini on the error shown by ooolpbot | 09:51 |
| panda | sshnaidm: then I'll check for newton and mitaka, which are failing for different errors | 09:52 |
| sshnaidm | panda, this ironic unbound variable, right? | 09:52 |
| sshnaidm | https://bugs.launchpad.net/tripleo/+bug/1659493 | 09:52 |
| openstack | Launchpad bug 1659493 in tripleo "IRONIC_AGENT_MULTIPATH_DRIVERS is undefined during IPA image build" [Critical,In progress] - Assigned to Michele Baldessari (michele) | 09:52 |
| *** karimb has joined #tripleo | 09:52 | |
| *** jpena is now known as jpena|off | 09:53 | |
| panda | sshnaidm: yes | 09:55 |
| openstackgerrit | Carlos Camacho proposed openstack/puppet-tripleo: Use transport_url for rabbitmq connection parameters in heat https://review.openstack.org/410333 | 09:58 |
| *** noama has joined #tripleo | 09:59 | |
| *** jpena|off is now known as jpena | 10:00 | |
| *** paramite has joined #tripleo | 10:02 | |
| *** paramite_ has joined #tripleo | 10:02 | |
| openstackgerrit | yolanda.robla proposed openstack/tripleo-image-elements: Set IRONIC_AGENT_MULTIPATH_DRIVERS to '' when undefined https://review.openstack.org/425565 | 10:05 |
| yolanda | bandini, i amended the patch, it actually was my fault | 10:05 |
| yolanda | i forgot to add the environment.d directory on my initial commit | 10:05 |
| yolanda | i had it on my change but didn't add to git :( | 10:05 |
| *** sshnaidm is now known as sshnaidm|afk | 10:06 | |
| panda | I imagined something like this ... :) | 10:06 |
| panda | the lack of default was too evident not have been considered | 10:06 |
| bandini | yolanda: ah that explains it ;) | 10:07 |
| yolanda | panda, what's worse, to don't consider that, or to forget to add to the commit? | 10:07 |
| yolanda | sorry to break things :( | 10:07 |
| panda | yolanda: git is a procedural mistake, and those happen, I think the other is worse | 10:08 |
| panda | yolanda: we were reasoning about the usefulness of the empty dracut command | 10:08 |
| bandini | yolanda: no worries and thanks for the quick fix ;) | 10:08 |
| panda | yolanda: that just builds a ramdisk without doing anything at all | 10:08 |
| panda | yolanda: is that needed ? | 10:08 |
| yolanda | i didn't want to touch that bit but no, it seems not needed | 10:09 |
| yolanda | this will be noop | 10:09 |
| panda | yolanda: weel it creates and replace any existing ramdisk | 10:09 |
| panda | so resulting in a noop if a vanilla ramdisk already exists | 10:09 |
| yolanda | well yes, but no need to recreate if we don't need to add anything extra... | 10:09 |
| *** ooolpbot has joined #tripleo | 10:11 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 10:11 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1659493 | 10:11 |
| *** ooolpbot has quit IRC | 10:11 | |
| openstack | Launchpad bug 1659493 in tripleo "IRONIC_AGENT_MULTIPATH_DRIVERS is undefined during IPA image build" [Critical,In progress] - Assigned to Yolanda Robla (yolanda.robla) | 10:11 |
| shardy | marios, ccamacho, matbu: Hey can we just land https://review.openstack.org/#/c/424072 and https://review.openstack.org/#/c/414598 ? | 10:11 |
| shardy | I think both have been tested locally (at least by me and marios), and CI is green for both | 10:11 |
| matbu | shardy: yes agree +1 | 10:12 |
| matbu | this all green :) | 10:12 |
| ccamacho | I tested https://review.openstack.org/#/c/424072 also yesterday | 10:14 |
| marios | shardy: looking | 10:14 |
| ccamacho | marios voted now | 10:15 |
| marios | shardy: ack yeah fine by me | 10:15 |
| shardy | Ok cool, two less to have in the mega-branch I'm preparing ;) | 10:16 |
| b00tcat | you guys mentioned before that the CI for Newton is currently broken no? | 10:16 |
| shardy | b00tcat: yeah unfortunately, until https://review.openstack.org/#/c/425343/ lands | 10:16 |
| *** owalsh-afk is now known as owalsh | 10:17 | |
| b00tcat | shardy: ok np, I was wondering because of a weird -1 that I got from Jenkins, thanks | 10:17 |
| *** abregman is now known as abregman|afk | 10:19 | |
| * matbu brb | 10:23 | |
| *** yolanda has quit IRC | 10:23 | |
| shardy | marios, ccamacho, gfidente: How about also landing https://review.openstack.org/#/c/423160/ https://review.openstack.org/#/c/423182/ https://review.openstack.org/#/c/423350 and https://review.openstack.org/#/c/423384 https://review.openstack.org/#/c/405248 ? | 10:24 |
| *** abregman|afk is now known as abregman | 10:25 | |
| shardy | gfidente: you have a -1 on the ceph-mon patch, but I'd be happy to fix that as a followup if it means we can reduce the size of this branch | 10:25 |
| shardy | everything else is green and has been well tested locally | 10:25 |
| *** abregman is now known as abregman|mtg | 10:25 | |
| *** yolanda has joined #tripleo | 10:26 | |
| *** paramite has quit IRC | 10:26 | |
| *** paramite_ has quit IRC | 10:26 | |
| shardy | I've been waiting 18 hours for an experimental job to run on https://review.openstack.org/#/c/423383/ so I think we just have to move forward with the CI passes on those | 10:27 |
| marios | shardy: so i haven't tested or really looked at the ceph-mon/batched stuff yet | 10:27 |
| marios | shardy: but either way, there *is* no currrent newton->ocata tripleo upgrade, i.e we aren't breaking something | 10:27 |
| marios | shardy: so sure i thought we already agreed we should just land and fix | 10:27 |
| marios | shardy: if we need whatever | 10:28 |
| shardy | marios: Yeah - my feeling is it's functional enough to land with some sanity reviews, then if needed we'll polish it with follow-up patches | 10:28 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/tripleo-common: Updated from global requirements https://review.openstack.org/424627 | 10:28 |
| shardy | I just want to make it easier to test the current state, and ideally without a branch containing 20+ patches | 10:28 |
| shardy | marios: yeah, I had been trying to do the "right" thing by getting experimental passes for each service in tripleo-ci patches | 10:29 |
| shardy | but that is taking too long if we want to make the FF deadline | 10:29 |
| marios | shardy: do you have to combine/merge the batched stuff with the role fragments /me looks at the review a bit | 10:29 |
| shardy | marios: yes, one way or another I have to rebase to reconcile those conflicts | 10:29 |
| shardy | I'd just rather land what's green, then I'll rebase the fragments stuff on master | 10:30 |
| marios | shardy: so i mean do you want to land this first? | 10:30 |
| shardy | vs rebasing the whole lot onto a new branch | 10:30 |
| marios | https://review.openstack.org/#/c/423160/ (this) | 10:30 |
| shardy | marios: yeah, it makes no difference to me what order we land them | 10:30 |
| * shardy has a bunch of conflicts to deal with either way ;) | 10:30 | |
| marios | shardy: vs https://review.openstack.org/#/c/424905/ I mean | 10:30 |
| marios | shardy: ack ok, just checking | 10:30 |
| shardy | marios: yeah my preference is to land https://review.openstack.org/#/c/423160/ first | 10:31 |
| *** karimb has quit IRC | 10:31 | |
| shardy | https://review.openstack.org/#/c/424905 is more complex, less well tested by me locally, and I'd like to re-test it with owalsh's tripleo-common patch at https://review.openstack.org/#/c/425220/ which I think is better than my patch linked in the Depends-On | 10:32 |
| shardy | so basically I'd be fine to rebase that one, fix the conflicts and re-test | 10:32 |
| shardy | then hopefully we can land it tomorrow | 10:32 |
| *** apetrich has joined #tripleo | 10:33 | |
| ccamacho | shardy I tested https://review.openstack.org/#/c/405248(with the snmp patch) and https://review.openstack.org/#/c/423384 (when checking the skipupgrade tag) Ill vote there, now checking https://review.openstack.org/#/c/423160/ | 10:34 |
| shardy | ccamacho: Oh yeah, we can probably land the snmp patch too | 10:36 |
| shardy | I rebased that on master | 10:36 |
| shardy | https://review.openstack.org/#/c/405250/ | 10:36 |
| shardy | it's not tested by the OVB jobs, so I'd say we approve it anyway? | 10:37 |
| ccamacho | shardy do you think is worth to rebase and run the checks again? (for 405250) | 10:39 |
| ccamacho | it should pass | 10:39 |
| shardy | ccamacho: personally I don't think it's worth using the OVB bandwidth, because nothing in the OVB jobs touches upgrade_tasks | 10:40 |
| shardy | but it's your call :) | 10:40 |
| *** abregman|mtg is now known as abregman | 10:40 | |
| ccamacho | It's locally tested so I think will be fine | 10:40 |
| shardy | ccamacho: yup, and when CI gets unblocked we can rebase your tripleo-ci patch and prove it in CI | 10:41 |
| ccamacho | cool | 10:41 |
| ccamacho | done | 10:41 |
| shardy | thanks! | 10:41 |
| ccamacho | ***BTW takes hell of a lot of time to test all those patches :) | 10:42 |
| openstackgerrit | Florian Fuchs proposed openstack/tripleo-ui: Fix blank overcloud credentials https://review.openstack.org/408138 | 10:46 |
| *** sshnaidm|afk is now known as sshnaidm | 10:47 | |
| openstackgerrit | Luke Hinds proposed openstack/tripleo-heat-templates: Add AuditD composable service https://review.openstack.org/421873 | 10:48 |
| openstackgerrit | Gabriele Cerami proposed openstack-infra/tripleo-ci: Do not use cache ipa image if we're gating tripleo-image-elements https://review.openstack.org/425626 | 10:49 |
| *** zoli is now known as zoli|lunch | 10:49 | |
| *** karimb has joined #tripleo | 10:50 | |
| *** shardy has quit IRC | 10:51 | |
| *** shardy has joined #tripleo | 10:52 | |
| openstackgerrit | Julie Pichon proposed openstack/tripleo-ui: Create .pot message file under ./i18n https://review.openstack.org/425627 | 10:55 |
| sshnaidm | Tomorrow is final feature freeze, right? | 10:56 |
| *** dtantsur|afk is now known as dtantsur | 10:57 | |
| *** apetrich has quit IRC | 10:57 | |
| *** yprokule has quit IRC | 10:59 | |
| shardy | sshnaidm: yes the ocata-3 release will be cut today or tomorrow, then we'll be in feature freeze | 11:00 |
| shardy | sshnaidm: I expect there will be a few exceptions given how many features aren't landed, but hopefully less than last cycle | 11:00 |
| openstackgerrit | Luke Hinds proposed openstack/tripleo-heat-templates: Adds SSH Banner text into sshd_config https://review.openstack.org/408638 | 11:04 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: WIP: Support builds in multinode https://review.openstack.org/425474 | 11:04 |
| openstackgerrit | Oliver Walsh proposed openstack/tripleo-common: Add support for jinja2 includes https://review.openstack.org/425220 | 11:09 |
| *** ooolpbot has joined #tripleo | 11:11 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 11:11 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1659493 | 11:11 |
| *** ooolpbot has quit IRC | 11:11 | |
| openstack | Launchpad bug 1659493 in tripleo "IRONIC_AGENT_MULTIPATH_DRIVERS is undefined during IPA image build" [Critical,In progress] - Assigned to Yolanda Robla (yolanda.robla) | 11:11 |
| *** karimb has quit IRC | 11:12 | |
| *** apetrich has joined #tripleo | 11:13 | |
| *** suuuper has joined #tripleo | 11:14 | |
| *** openstackgerrit has quit IRC | 11:17 | |
| *** akrivoka has joined #tripleo | 11:20 | |
| *** openstackgerrit has joined #tripleo | 11:23 | |
| openstackgerrit | Alejandro Andreu proposed openstack/tripleo-puppet-elements: Install Curator from RPM in MidoNet element https://review.openstack.org/425205 | 11:23 |
| *** apetrich has quit IRC | 11:25 | |
| *** karimb has joined #tripleo | 11:27 | |
| *** michapma_alt has quit IRC | 11:28 | |
| *** karimb has quit IRC | 11:28 | |
| *** michapma_alt has joined #tripleo | 11:29 | |
| *** karimb has joined #tripleo | 11:30 | |
| openstackgerrit | Merged openstack/tripleo-quickstart-extras: Fix node cycle on HA Validation https://review.openstack.org/425587 | 11:30 |
| openstackgerrit | Oliver Walsh proposed openstack/tripleo-heat-templates: Add support for Jinja2 includes https://review.openstack.org/425638 | 11:32 |
| *** shardy has quit IRC | 11:32 | |
| *** shardy has joined #tripleo | 11:34 | |
| *** cmyster has quit IRC | 11:34 | |
| *** zoli|lunch is now known as zoli | 11:34 | |
| b00tcat | hey, is this a bug? https://paste.fedoraproject.org/536864/48543042/ if I pass the --tripleo-puppet-elements to "openstack overcloud image build" I expect it to look for the t-p-e in the folder that I specified | 11:34 |
| b00tcat | not in /usr/bin/../share/diskimage-builder/elements which is the default path :/ | 11:35 |
| shardy | b00tcat: yes looking at the code it seems that is broken | 11:38 |
| shardy | try --elements-path | 11:38 |
| shardy | or, here's where it's broken: | 11:38 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add SkipUpgradeConfigTags for upgrade config https://review.openstack.org/424072 | 11:38 |
| openstackgerrit | Merged openstack/tripleo-puppet-elements: Add explicit install of net-snmp https://review.openstack.org/425343 | 11:39 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add snmp service support for composable upgrades https://review.openstack.org/405250 | 11:39 |
| shardy | https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/overcloud_image.py#L322 | 11:40 |
| *** jkilpatr has quit IRC | 11:40 | |
| shardy | it only uses the hard-coded path and ignores the variable from the --tripleo-puppet-elements option | 11:40 |
| b00tcat | damn | 11:40 |
| shardy | b00tcat: all those options are deprecated, so I think you're supposed to specify them in the image yaml file | 11:40 |
| shardy | but it should be easy to either fix that CLI, or use --elements-path instead | 11:40 |
| b00tcat | shardy: ah right, in Ocata there was the YAML | 11:41 |
| shardy | they basically do the same thing AFAICS? | 11:41 |
| b00tcat | except for elements-path being a full path with several elems | 11:41 |
| b00tcat | ok I'll submit a patch whenever I get a change, thanks! | 11:41 |
| b00tcat | chance* | 11:41 |
| shardy | b00tcat: Ok sounds good, thanks! | 11:41 |
| *** salmankhan has quit IRC | 11:42 | |
| *** salmankhan has joined #tripleo | 11:42 | |
| *** thrash|g0ne is now known as thrash | 11:43 | |
| shardy | ccamacho: Hey can you check https://review.openstack.org/#/c/423160/ when you get a moment please? | 11:44 |
| *** ansmith has joined #tripleo | 11:44 | |
| sshnaidm | cores, please approve oooq patch in ci: https://review.openstack.org/#/c/423870/ , thanks | 11:51 |
| *** jprovazn has quit IRC | 11:52 | |
| *** jprovazn has joined #tripleo | 11:52 | |
| *** yprokule has joined #tripleo | 11:54 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Ignore systemctl return code in yum_update.sh https://review.openstack.org/425209 | 11:55 |
| *** jkilpatr has joined #tripleo | 11:58 | |
| EmilienM | Hello folks | 11:59 |
| shardy | Hey EmilienM | 11:59 |
| sshnaidm | EmilienM, hi | 12:00 |
| *** ansmith has quit IRC | 12:01 | |
| shardy | EmilienM: hey when you get time, can you please check this series https://review.openstack.org/#/c/423160/ | 12:01 |
| *** jpena is now known as jpena|lunch | 12:01 | |
| shardy | we had a discussion earlier and agreed to try and land some of these to enable easier local testing | 12:01 |
| *** fultonj has joined #tripleo | 12:02 | |
| shardy | I had been waiting on experimental job passes on tripleo-ci, but the CI is so busy I don't think we can wait | 12:02 |
| shardy | the experimental jobs just never run | 12:02 |
| shardy | ci on those patches is all green until the telemetry one, so I'd say we land the first 5 patches | 12:02 |
| shardy | and I'll re-test the telemetry patch locally | 12:02 |
| shardy | and rebase the various other upgrades patches we have outstanding (some of which conflict with this series) | 12:03 |
| *** liverpooler has quit IRC | 12:03 | |
| *** bkopilov has quit IRC | 12:05 | |
| panda | the experimental-tripleo queue may take up to 14 hours to launch a job | 12:06 |
| shardy | panda: yeah, I can't keep waiting that long for every patch, when we have like 10 services still to land | 12:06 |
| ccamacho | shardy finished checking https://review.openstack.org/#/c/423160/ | 12:07 |
| *** jkilpatr has quit IRC | 12:07 | |
| ccamacho | That one I haven't tested it locally but LGTM | 12:07 |
| shardy | ccamacho: thanks! | 12:08 |
| shardy | ccamacho: yeah, I've tested it locally and we've got basic coverage because the upgrades-nv passed | 12:08 |
| shardy | we'll need folks to test the rolling part with multiple nodes, but I guess that can happen later | 12:08 |
| *** shardy is now known as shardy_lunch | 12:10 | |
| ccamacho | shardy as this have higher prio, im deploying a newton OC, Ill start checking https://review.openstack.org/#/c/419886 but ping me please if needed other local checks/reviews | 12:11 |
| *** ooolpbot has joined #tripleo | 12:11 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 12:11 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1659384 | 12:11 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1659493 | 12:11 |
| *** ooolpbot has quit IRC | 12:11 | |
| openstack | Launchpad bug 1659384 in tripleo "All ovb jobs failing to set up snmp" [Critical,In progress] - Assigned to Ben Nemec (bnemec) | 12:11 |
| openstack | Launchpad bug 1659493 in tripleo "IRONIC_AGENT_MULTIPATH_DRIVERS is undefined during IPA image build" [Critical,In progress] - Assigned to Yolanda Robla (yolanda.robla) | 12:11 |
| EmilienM | shardy_lunch: sure thing | 12:11 |
| EmilienM | panda, sshnaidm: do we have clue on why promotion didn't happen? | 12:12 |
| panda | EmilienM: it's on the alert | 12:12 |
| panda | removing alert from, I'm still seeing on newton an mitaka periodic, but not eslewhere https://bugs.launchpad.net/tripleo/+bug/1659384 | 12:12 |
| openstack | Launchpad bug 1659384 in tripleo "All ovb jobs failing to set up snmp" [Critical,In progress] - Assigned to Ben Nemec (bnemec) | 12:12 |
| EmilienM | this one? https://bugs.launchpad.net/tripleo/+bug/1659384 | 12:13 |
| panda | maybe just a package delay | 12:13 |
| EmilienM | yeah | 12:13 |
| panda | EmilienM: no, the other | 12:13 |
| EmilienM | bnemec fixed it :/ | 12:13 |
| EmilienM | ah | 12:13 |
| EmilienM | https://bugs.launchpad.net/tripleo/+bug/1659493 | 12:13 |
| openstack | Launchpad bug 1659493 in tripleo "IRONIC_AGENT_MULTIPATH_DRIVERS is undefined during IPA image build" [Critical,In progress] - Assigned to Yolanda Robla (yolanda.robla) | 12:13 |
| panda | EmilienM: yes | 12:13 |
| panda | EmilienM: this is the fix https://review.openstack.org/425565 and this should activate ipa image building on the gate for tripleo-image-elements https://review.openstack.org/425626 | 12:14 |
| panda | so it will not slip through the gates again | 12:14 |
| EmilienM | panda: ok +A on the fix | 12:14 |
| EmilienM | panda: +2 on the ooo-ci | 12:14 |
| EmilienM | I'll see how ovb works again with this patch ^ | 12:15 |
| *** karimb has quit IRC | 12:16 | |
| EmilienM | shardy_lunch: patch approved, reviewing telemetry now | 12:16 |
| *** anshul has joined #tripleo | 12:17 | |
| *** karimb has joined #tripleo | 12:19 | |
| *** tremble has left #tripleo | 12:19 | |
| *** jkilpatr has joined #tripleo | 12:20 | |
| *** cmyster has joined #tripleo | 12:23 | |
| *** ramishra_ has quit IRC | 12:24 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Add Ceph RBD mirror Pacemaker profile https://review.openstack.org/407039 | 12:24 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Add Ceph RBD mirror Pacemaker profile https://review.openstack.org/407039 | 12:24 |
| thrash | EmilienM: for tripleo-common... release notes since... 5.7.0? Newton? | 12:25 |
| EmilienM | thrash: since last tag, in december would be enough | 12:26 |
| thrash | EmilienM: ack. | 12:26 |
| EmilienM | thx | 12:26 |
| EmilienM | thrash, d0ugal, jpich, Ng: beside https://review.openstack.org/#/c/417898/ - do you know any other thing we want in triploeclient before I push on the release button? | 12:27 |
| thrash | EmilienM: https://review.openstack.org/#/c/419506/ on down | 12:28 |
| thrash | https://review.openstack.org/#/c/418507/5 | 12:28 |
| thrash | but they depend on: https://review.openstack.org/#/c/416284/ | 12:28 |
| Ng | EmilienM: nothing else from me beside the one you linked | 12:28 |
| thrash | EmilienM: I'll add release note to that last one real quick. | 12:29 |
| jpich | EmilienM: I think https://review.openstack.org/#/c/414447/ was just about ready, I can add the release note in a separate patch today | 12:29 |
| EmilienM | thrash: doing recheck on https://review.openstack.org/#/c/416284/ to see OVB | 12:29 |
| thrash | EmilienM: let me add the relno. :) | 12:29 |
| EmilienM | thrash: ack on both | 12:30 |
| EmilienM | jpich: ack, thanks | 12:31 |
| EmilienM | jpich: removing -1 then, and approving | 12:31 |
| d0ugal | EmilienM: I don't have anything other than ^ | 12:31 |
| EmilienM | thanks folks | 12:31 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Do not try to update the 'ceph' metapackage from CephMon role https://review.openstack.org/425659 | 12:32 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: WIP: Support builds in multinode https://review.openstack.org/425474 | 12:33 |
| *** pkovar has joined #tripleo | 12:33 | |
| *** ccamacho is now known as ccamacho|mcdnlds | 12:34 | |
| jpich | EmilienM: Thanks! | 12:34 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: WIP: Conntainers with multi-nic and net-iso https://review.openstack.org/422448 | 12:34 |
| openstackgerrit | Brad P. Crochet proposed openstack/tripleo-common: Add support for calling validations in deploy and introspect workflows https://review.openstack.org/416284 | 12:34 |
| thrash | EmilienM: I assume we can add some kind of bot that will -1 a patch if it doesn't include a release note from now on? | 12:35 |
| thrash | If one doesn't already exist. :) | 12:35 |
| EmilienM | thrash: ah! I'm not aware of any bot like this, though I would rather rely on humain reviews for that :-) | 12:36 |
| EmilienM | not every patch deserves a release note (eg: quick fix within a cycle) | 12:36 |
| thrash | EmilienM: Hmmm... Could include a way to signify that in the commit? Release-Note-Required: False | 12:37 |
| thrash | :D | 12:37 |
| thrash | just spitballing. | 12:38 |
| d0ugal | lol, that would drive people mad :) | 12:39 |
| EmilienM | thrash: it's already hard to teach folks how to make release notes | 12:39 |
| d0ugal | thrash: we should also require a bug or blueprint :) | 12:39 |
| thrash | d0ugal: I hate you. | 12:39 |
| *** karimb has quit IRC | 12:39 | |
| d0ugal | lol | 12:39 |
| thrash | >_< | 12:39 |
| d0ugal | but if the review references a bug it probably needs a release note... | 12:40 |
| d0ugal | but that might just stop people referencing bugs :-D | 12:40 |
| thrash | d0ugal: ooh... That's a good one... | 12:40 |
| thrash | Look, I am *king* of hating to do this kind of thing... But it's not *that* bad. | 12:41 |
| d0ugal | lol | 12:41 |
| Ng | d0ugal: thrash: https://review.openstack.org/#/c/422321/ is full CI green now, just needs one of its earlier +2s restored :) | 12:43 |
| d0ugal | Done! | 12:44 |
| gfidente | marios shardy_lunch I remember you were talking about a global check of the status before starting upgrade? | 12:44 |
| Ng | d0ugal: thanks! | 12:44 |
| *** apetrich has joined #tripleo | 12:45 | |
| EmilienM | thrash: right, doing a note takes 1 min or 2 max | 12:45 |
| marios | gfidente: hey man | 12:47 |
| marios | gfidente: i just left a comment on your review https://review.openstack.org/#/c/423360/3 | 12:47 |
| d0ugal | It'd be nice if we could include the note in the commit message or something, seems like you end up duplicating it. | 12:47 |
| marios | gfidente: it points at the step0,validation i think you are referring to that | 12:48 |
| *** ramishra has joined #tripleo | 12:48 | |
| gfidente | marios ah cool I can add that yes | 12:48 |
| gfidente | so I wonder | 12:48 |
| *** chem has joined #tripleo | 12:48 | |
| gfidente | I could add the same for the monitors too right? | 12:48 |
| marios | gfidente: yes please | 12:49 |
| gfidente | cool | 12:49 |
| *** apetrich has quit IRC | 12:50 | |
| *** apetrich has joined #tripleo | 12:50 | |
| *** dprince has joined #tripleo | 12:51 | |
| *** lucasagomes is now known as lucas-hungry | 12:52 | |
| *** egafford has joined #tripleo | 12:52 | |
| *** hjensas has joined #tripleo | 12:54 | |
| *** hjensas has joined #tripleo | 12:54 | |
| *** tosky has quit IRC | 12:54 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/instack-undercloud: Ensure Aodh uses its own mysql db https://review.openstack.org/424950 | 12:56 |
| *** pradk has joined #tripleo | 12:56 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Do not try to update the 'ceph' metapackage from CephMon role https://review.openstack.org/425659 | 12:57 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-refresh-config: Updated from global requirements https://review.openstack.org/420924 | 12:57 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Add upgrade support for CephRGW service https://review.openstack.org/423360 | 12:59 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Add upgrade support for CephRGW service https://review.openstack.org/423360 | 13:00 |
| *** jpena|lunch is now known as jpena | 13:00 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add support for batched upgrades to composable upgrades https://review.openstack.org/423160 | 13:02 |
| *** shardy_lunch is now known as shardy | 13:04 | |
| *** ealcaniz has quit IRC | 13:05 | |
| *** adarazs is now known as adarazs_lunch | 13:07 | |
| openstackgerrit | Brad P. Crochet proposed openstack/tripleo-common: Add release notes since 5.7.0 https://review.openstack.org/425677 | 13:08 |
| *** bfournie has quit IRC | 13:08 | |
| EmilienM | thrash: ♥ | 13:10 |
| EmilienM | I'll send ♥ to everyone doing release notes | 13:10 |
| *** jayg|g0n3 is now known as jayg | 13:10 | |
| *** ooolpbot has joined #tripleo | 13:11 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 13:11 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1659493 | 13:11 |
| *** ooolpbot has quit IRC | 13:11 | |
| openstack | Launchpad bug 1659493 in tripleo "IRONIC_AGENT_MULTIPATH_DRIVERS is undefined during IPA image build" [Critical,In progress] - Assigned to Yolanda Robla (yolanda.robla) | 13:11 |
| dtantsur | EmilienM, where is my ♥ then? :( | 13:15 |
| EmilienM | dtantsur: dude, I sent ♥ to you every day on private | 13:16 |
| *** lucas-hungry is now known as lucasagomes | 13:16 | |
| dtantsur | thanks <3 | 13:16 |
| dtantsur | :) | 13:16 |
| bandini | EmilienM: do you prefer me piggy-backing existing reviews for tht/puppet-tripleo release notes or simply a separate one using reno? | 13:17 |
| *** lblanchard has joined #tripleo | 13:19 | |
| *** trown|outtypewww is now known as trown | 13:19 | |
| *** karimb has joined #tripleo | 13:20 | |
| openstackgerrit | Julie Pichon proposed openstack/python-tripleoclient: Release note for disabling password generation https://review.openstack.org/425685 | 13:21 |
| *** owalsh is now known as owalsh-brb | 13:21 | |
| EmilienM | bandini: in general, I just ask in the review if we can add the note. If it has +2 already I don't block on it and ask for a top patch. If CI doesn't pass or already has negative review, I ask for a release note | 13:21 |
| EmilienM | bandini: in a few months, when folks are used to reno, I would -1 even if +2 already | 13:22 |
| EmilienM | bandini: smooth transition :) | 13:22 |
| *** fultonj has quit IRC | 13:23 | |
| bandini | EmilienM: ack. and for stuff that already landed without a release note? Just create a separate release note or do we have some "mega-release-note" reviews in flight already? | 13:23 |
| EmilienM | bandini: +1 for separate | 13:24 |
| *** fultonj has joined #tripleo | 13:24 | |
| bandini | EmilienM: ack. will get to it today for my stuff | 13:24 |
| EmilienM | thx | 13:24 |
| *** owalsh-brb has quit IRC | 13:26 | |
| openstackgerrit | Dan Radez proposed openstack/puppet-tripleo: Adding congress service https://review.openstack.org/413881 | 13:26 |
| EmilienM | matbu, marios, shardy: so prior to start moving upgrade job to scenarios, like I said I would do, I would need one first thing: move test-environments/multinode_major_upgrade.yaml into THT so we can version it. It takes 2 simple patches to do it, and should be very helpful for us so we can version the files between cycles. Can I do it now? | 13:27 |
| *** ansmith has joined #tripleo | 13:28 | |
| matbu | EmilienM: yes sure | 13:28 |
| matbu | EmilienM: do you want to do it ? | 13:28 |
| EmilienM | and I can do it in a way that doesn't break you | 13:28 |
| EmilienM | matbu: yes, I already have the patches ready | 13:28 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Add release note for composable upgrades https://review.openstack.org/425689 | 13:29 |
| matbu | EmilienM: ack nice | 13:29 |
| marios | EmilienM: ack | 13:29 |
| *** zoli is now known as zoli|brb | 13:29 | |
| marios | EmilienM: where should i vote (review?) | 13:29 |
| *** owalsh-brb has joined #tripleo | 13:29 | |
| *** owalsh-brb is now known as owalsh | 13:29 | |
| trown | matbu: for the quickstart upgrades work, can we create some job that actually tests it? That has been the barrier for reviewing it for me | 13:30 |
| EmilienM | marios: i'll let you know in a few | 13:30 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: ci: import multinode_major_upgrade.yaml from tripleo-ci https://review.openstack.org/425690 | 13:31 |
| trown | matbu: I know I could manually test it myself, but then if we merge it there is nothing to stop it from being immediately broken by something | 13:31 |
| shardy | we could add it to the tht experimental pipeline and prove we get the same results as the current upgrades job | 13:31 |
| trown | ya, experimental would be a good start | 13:31 |
| trown | matbu: also, is it requiring OVB, or can we make it work via the multinode job we have? | 13:32 |
| openstackgerrit | Merged openstack/tripleo-image-elements: Set IRONIC_AGENT_MULTIPATH_DRIVERS to '' when undefined https://review.openstack.org/425565 | 13:32 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add upgrade support for ceph-mon service https://review.openstack.org/423182 | 13:33 |
| *** dsariel has joined #tripleo | 13:33 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add upgrade support for ceph OSD service https://review.openstack.org/423350 | 13:33 |
| trown | if it is multinode, we could also put it in the check queue for quickstart/extras right away | 13:33 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Skip upgrade steps where no tasks are defined https://review.openstack.org/423384 | 13:33 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add ironic service support for composable upgrades https://review.openstack.org/405248 | 13:33 |
| openstackgerrit | Merged openstack/tripleo-ui: Add pattern option to po2json call https://review.openstack.org/424219 | 13:33 |
| openstackgerrit | Merged openstack/tripleo-ui: Create .pot message file under ./i18n https://review.openstack.org/425627 | 13:33 |
| openstackgerrit | Merged openstack/puppet-tripleo: [keepalived] fix netmask for vip https://review.openstack.org/418577 | 13:33 |
| *** mhenkel has joined #tripleo | 13:34 | |
| matbu | trown: hmm idk if it could work on multinode | 13:34 |
| trown | matbu: why not? seems like upgrades are entirely post deploy operation, which does not require nova/ironic | 13:35 |
| matbu | trown: the fastest thing i can do, is to put a job on ci.centos | 13:36 |
| openstackgerrit | Sven Anderson proposed openstack/instack-undercloud: Add a pre-upgrade online_data_migration step for undercloud https://review.openstack.org/425323 | 13:36 |
| matbu | trown: i have a job that run internally | 13:36 |
| matbu | trown: where i tested the reviews | 13:36 |
| shardy | The current upgrades job works on multinode, so this has to work on multinode if we're ever going to replace the current job with this one? | 13:37 |
| trown | matbu: ci.centos is a non-starter unfortunately... we already have jobs failing there because we exhaust the inventory... maybe rlandy could help you setup an internal job that reports on quickstart/extras patches via third-party ... we already have one such job running on quickstart/extras patches | 13:37 |
| shardy | IMO we don't want any new jobs running on OVB | 13:37 |
| *** tosky has joined #tripleo | 13:37 | |
| trown | +1 | 13:37 |
| matbu | shardy: right | 13:37 |
| trown | I would say new OVB jobs would be ok if they actually required testing nova/ironic... like scale or the like | 13:38 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Move multinode_major_upgrade.yaml to THT https://review.openstack.org/425694 | 13:38 |
| trown | but for things that do not actually require any nova/ironic interaction we should be using deployed server | 13:38 |
| EmilienM | matbu, marios, shardy: https://review.openstack.org/#/q/topic:upgrade/move | 13:38 |
| matbu | trown: shardy but tbh, i don't know how should I set a job on mutlinode right now | 13:38 |
| shardy | trown: Yeah, I guess I meant we already have OK coverage of the deployment pieces, so we should be building out breadth of coverage via multinode jobs | 13:38 |
| *** sdake has joined #tripleo | 13:38 | |
| shardy | which are much faster and more reliable | 13:38 |
| *** bfournie has joined #tripleo | 13:39 | |
| shardy | I thought multinode support was getting added to quickstart? | 13:39 |
| trown | it is already there | 13:39 |
| shardy | If we can't be building on that then I don't get why we're spending time on this now | 13:39 |
| matbu | EmilienM: thanks (i'll review in a minute) | 13:39 |
| trown | we have a simple job running on all tripleo-ci/quickstart/extras patches | 13:40 |
| EmilienM | matbu: not urgent :D | 13:40 |
| shardy | we have an upgrades job, eventually we'll want to switch it to quickstart | 13:40 |
| shardy | but now we've got a massive amount of work to get upgrades finished for the release | 13:40 |
| shardy | I don't get why this is happening now at all tbh | 13:40 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Do not try to update the 'ceph' metapackage from CephMon role https://review.openstack.org/425659 | 13:40 |
| trown | same... I am mostly responding to being called out on the ML for being slow to review | 13:41 |
| *** rlandy has joined #tripleo | 13:41 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/diskimage-builder: Try configuring iBFT devices in dhcp-all-interfaces https://review.openstack.org/391787 | 13:41 |
| *** eck` is now known as jeckersb_gone | 13:41 | |
| matbu | shardy: yep right, I raise this on ML because it's pending for a while and also because i think all the DFGs is starting to be interesting on being able to work on upgrade on their component | 13:41 |
| trown | reviewing a new major CI feature with no CI on the patches though is always going to be a tough sell though... same happened with containers | 13:42 |
| openstackgerrit | Merged openstack/python-tripleoclient: Provide an option to disable password generation on deployment https://review.openstack.org/414447 | 13:42 |
| openstackgerrit | Merged openstack/puppet-tripleo: Ensure basic Ceph configuration is performed by RBD mirror https://review.openstack.org/425236 | 13:42 |
| matbu | trown: CI on CI :) we have CI on the patch | 13:42 |
| trown | matbu: CI to say your patch doesnt break anything... not that it works | 13:42 |
| matbu | shardy: so yes, it's may not be the real good time now, but i didn't required extra time on that | 13:43 |
| *** rbrady-afk is now known as rbrady | 13:43 | |
| matbu | s/i/it/ | 13:43 |
| shardy | matbu: sure, I'm not saying it won't be helpful, but seriously - look at the backlog of patches to make upgrades work | 13:44 |
| matbu | shardy: so it's more for end user experience with upgrade than CI releated stuff | 13:44 |
| shardy | feature freeze is tomorrow | 13:44 |
| matbu | shardy: ye right | 13:44 |
| openstackgerrit | John Trowbridge proposed openstack/tripleo-quickstart-extras: WIP Add build-images role https://review.openstack.org/414336 | 13:45 |
| *** bkopilov has joined #tripleo | 13:45 | |
| marios | shardy: can we test https://review.openstack.org/#/c/424905/ with https://review.openstack.org/#/c/425220/ as-is or should we use https://review.openstack.org/#/c/425086/ for now? | 13:46 |
| trown | matbu: ya, helping out with quickstart upgrades worked was brought up on our CI meeting too, and I had the same question there about priority... personally I think tripleo would benefit more from making quickstart based basic jobs rock solid so we have confidence to switch in early pike, then work on switching upgrade job over to quickstart based at that point should be much easier | 13:47 |
| *** hewbrocca_afk is now known as hewbrocca | 13:48 | |
| EmilienM | shardy: I commented on your reno for upgrades | 13:48 |
| trown | matbu: which has the added benefit of not distracting from existing working upgrades job | 13:48 |
| trown | matbu: in any case, I am not merging a giant feature to quickstart without CI running for it on all quickstart/extras patches... it will be broken very quickly without that | 13:50 |
| *** hewbrocca is now known as hewbrocca_afk | 13:50 | |
| *** hewbrocca_afk is now known as hewbrocca | 13:51 | |
| openstackgerrit | Merged openstack/tripleo-common: Add action for generating fencing parameters. https://review.openstack.org/422321 | 13:51 |
| matbu | EmilienM: im wondering here: https://review.openstack.org/#/c/425694 does the tht package is already installed at this stage ? | 13:52 |
| matbu | trown: k so can you answer to the email on ML plz? | 13:52 |
| trown | matbu: sure | 13:53 |
| *** ccamacho|mcdnlds is now known as ccamacho | 13:53 | |
| trown | matbu: you responded to another thread so it shows up a bit weird in my email client by the way... but no bigs | 13:53 |
| EmilienM | matbu: so when the THT will land, RDO needs ~30 min to package it. That's why I have this conditional in tripleo-ci, to rely on old file is package not built yet | 13:54 |
| EmilienM | matbu: make sense? | 13:54 |
| matbu | EmilienM: hmm, this is not what i'm asking :D i just wonder if the tht package is already installed on the file system at this stage ( i thought it would be install during the UC installation) | 13:56 |
| openstackgerrit | Honza Pokorny proposed openstack/tripleo-ui: Add developer docs https://review.openstack.org/421000 | 13:56 |
| *** zoli|brb is now known as zoli | 13:56 | |
| *** amoralej is now known as amoralej|lunch | 13:57 | |
| EmilienM | matbu: ahhh good question | 13:58 |
| EmilienM | matbu: yes I think so | 13:59 |
| *** adarazs_lunch is now known as adarazs | 14:00 | |
| trown | lol @ CIFE | 14:00 |
| trown | we need more acronyms :P | 14:00 |
| *** ansmith has quit IRC | 14:01 | |
| openstackgerrit | mathieu bultel proposed openstack/tripleo-heat-templates: Add pacemaker composable upgrade steps https://review.openstack.org/403397 | 14:02 |
| *** crenguta has quit IRC | 14:02 | |
| *** rhallisey has joined #tripleo | 14:02 | |
| *** crenguta has joined #tripleo | 14:04 | |
| openstackgerrit | Oliver Walsh proposed openstack/tripleo-common: Run nova manage cell_v2 discover_host after ironic nodes are registered https://review.openstack.org/425273 | 14:04 |
| openstackgerrit | Michele Baldessari proposed openstack/puppet-tripleo: WIP Qpid dispatch router puppet profile https://review.openstack.org/425710 | 14:04 |
| shardy | marios: either works - I've tested with both, but I need to rebase the t-h-t patch | 14:05 |
| shardy | doing that now | 14:05 |
| openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates: WIP Qpid dispatch router composable role https://review.openstack.org/423326 | 14:05 |
| *** Goneri has joined #tripleo | 14:06 | |
| EmilienM | trown: lol sorry | 14:06 |
| trown | hehe I am only teasing... CIFE is fine :) | 14:07 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: WIP: Support builds in multinode https://review.openstack.org/425474 | 14:08 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Set the correct default for gnocchi workers https://review.openstack.org/425347 | 14:09 |
| marios | shardy: k, i went with your original one but i get a Exception updating plan: no loader for this environment specified | 14:09 |
| marios | shardy: am re-upgrading my undercloud to latest and will patch again with the rebases (lots of stuff happened today, or at least we _+2 lots of stuff not sure it landed all yet) | 14:10 |
| *** saneax-_-|AFK is now known as saneax | 14:11 | |
| matbu | marios: got the same yesterday, but i think we're missing patches | 14:11 |
| marios | matbu: ack | 14:12 |
| *** shardy has quit IRC | 14:14 | |
| matbu | marios: i'm fighting against another bug right now | 14:14 |
| *** ansmith has joined #tripleo | 14:14 | |
| *** shardy has joined #tripleo | 14:15 | |
| marios | shardy: should i rebase https://review.openstack.org/#/c/424905/? | 14:16 |
| marios | shardy: i want to rebase my one that has that ^ as parent so can do | 14:16 |
| *** tzumainn has joined #tripleo | 14:16 | |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Add release note for composable upgrades https://review.openstack.org/425689 | 14:16 |
| openstackgerrit | Brad P. Crochet proposed openstack/tripleo-common: Add release notes since 5.7.0 https://review.openstack.org/425677 | 14:17 |
| openstackgerrit | Dan Radez proposed openstack/tripleo-heat-templates: Adding Tacker Support https://review.openstack.org/424135 | 14:17 |
| shardy | marios: it's not a clean rebase, I'm doing it now | 14:18 |
| marios | shardy: ack thanks, no it is not. meld is flashing all sorts of colors | 14:18 |
| shardy | Yeah I expected it to be tricky - I'll do a quick rework by eye, then we can test it | 14:19 |
| shardy | ideally I wanted to rebased, test, push | 14:19 |
| Ng | EmilienM: https://review.openstack.org/#/c/417898/ just finished CI, so it just needs a +A and the gate job to run :) | 14:19 |
| shardy | but I don't want to hold you up :) | 14:19 |
| *** jeckersb_gone is now known as eck` | 14:19 | |
| marios | heh you're definitely not holding anyone up shardy srsly | 14:20 |
| EmilienM | Ng: done, good work here | 14:21 |
| *** absubram has joined #tripleo | 14:22 | |
| Ng | EmilienM: much appreciated! | 14:22 |
| *** amoralej|lunch is now known as amoralej | 14:22 | |
| matbu | marios: perhaps his own machine which is crying for having time to rest :) | 14:22 |
| EmilienM | d0ugal, if you can review https://review.openstack.org/#/c/416284/10 please it passed well on https://review.openstack.org/#/c/418507/ | 14:23 |
| EmilienM | thrash: should we recheck on https://review.openstack.org/#/c/419506 or CI failure is legit? | 14:24 |
| *** jcoufal has joined #tripleo | 14:24 | |
| thrash | EmilienM: looking | 14:24 |
| thrash | EmilienM: just did a recheck | 14:25 |
| EmilienM | thrash: your 2 client patches are the remaining ones before releasing | 14:25 |
| EmilienM | thrash: can you review https://review.openstack.org/#/c/425685/ please? | 14:26 |
| *** absubram_ has joined #tripleo | 14:26 | |
| *** absubram has quit IRC | 14:27 | |
| *** absubram_ is now known as absubram | 14:27 | |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Disable puppet on upgrade for roles not upgrading https://review.openstack.org/424905 | 14:27 |
| openstackgerrit | Michael Henkel proposed openstack/tripleo-heat-templates: Re-organizes Contrail services to the correct roles https://review.openstack.org/425717 | 14:27 |
| shardy | marios: ^^ wasn't too bad actually but that's untested | 14:27 |
| shardy | marios: I think I can simplify it a little bit by assigning the list of enabled roles to a variable | 14:28 |
| shardy | I made it depend on owalsh's patch as I tested that earlier and it works well | 14:28 |
| *** jmelvin has joined #tripleo | 14:29 | |
| *** liverpooler has joined #tripleo | 14:30 | |
| EmilienM | shardy: would I need to backport https://review.openstack.org/#/c/425690/ into newton? I'm not sure | 14:30 |
| *** salmankhan has quit IRC | 14:32 | |
| shardy | EmilienM: No that only works for CI on patches landing to ocata | 14:35 |
| shardy | we install newton, then upgrade to ocata | 14:35 |
| shardy | no older releases are currently supported by that job | 14:35 |
| *** shardy has quit IRC | 14:35 | |
| *** suuuper has quit IRC | 14:39 | |
| cmyster | quick question, is panko package thing resolved ? | 14:39 |
| openstackgerrit | Ana Krivokapic proposed openstack/tripleo-common: Add utility functions for deleting/emptying swift containers https://review.openstack.org/409315 | 14:41 |
| EmilienM | matbu, marios, bkero: FYI https://review.openstack.org/425727 | 14:41 |
| EmilienM | cmyster: which issue? | 14:41 |
| *** suuuper has joined #tripleo | 14:42 | |
| cmyster | EmilienM: there was a packaging issue with panko, can't install some package because panko was outdated | 14:45 |
| cmyster | 1-2 days ago ? | 14:45 |
| cmyster | I can't seem to find it | 14:46 |
| EmilienM | cmyster: afik all looks ok now | 14:48 |
| cmyster | EmilienM: no big deal, I just found something similar downstream and I think its the same issue as that issue | 14:48 |
| cmyster | but again, no big deal | 14:48 |
| EmilienM | cmyster: pank::mysql thing? | 14:48 |
| openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates: WIP Qpid dispatch router composable role https://review.openstack.org/423326 | 14:48 |
| cmyster | could be :) | 14:48 |
| cmyster | sounds familiar | 14:48 |
| EmilienM | cmyster: yeah it's a bad downstream build | 14:48 |
| cmyster | right ok | 14:49 |
| EmilienM | but not related to upstream | 14:49 |
| cmyster | ok, just wanted to make sure | 14:49 |
| *** cdearborn has joined #tripleo | 14:50 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/instack-undercloud: Fix ironic-related deprecations https://review.openstack.org/425736 | 14:51 |
| EmilienM | panda: have you checked why mitaka/newton job failed on https://review.openstack.org/#/c/425626/ ? | 14:51 |
| panda | EmilienM: yes, it's the snmp bug | 14:54 |
| EmilienM | panda: ack, cool | 14:54 |
| bkero | EmilienM: hey, scenario upgrades :) | 14:55 |
| EmilienM | bkero: yes. I'm preparing it and hopefully transition to oooq will be smooth | 14:55 |
| EmilienM | bkero: since we'll have the templates & jobs in place | 14:55 |
| EmilienM | bkero: it will be "easy" to make the transition i think | 14:55 |
| bkero | EmilienM: oooq should be unobtrusive | 14:55 |
| EmilienM | cool | 14:55 |
| openstackgerrit | Tim Rozet proposed openstack/puppet-tripleo: Enables OpenDaylight Clustering in HA deployments https://review.openstack.org/425383 | 14:56 |
| *** jrist has quit IRC | 14:56 | |
| openstackgerrit | Merged openstack/python-tripleoclient: Use the overcloudrc generated in a Mistral action https://review.openstack.org/410272 | 14:56 |
| *** owalsh is now known as owalsh-afk | 14:58 | |
| adarazs | for anyone interested, TripleO CI squad meeting in 30 minutes, add agenda items at https://etherpad.openstack.org/p/tripleo-ci-squad-meeting (ping for EmilienM, rbrady, trown, sshnaidm, weshay, hrybacki, panda, rlandy, bkero) | 15:00 |
| hrybacki | ack adarazs | 15:00 |
| rbrady | thanks adarazs | 15:00 |
| *** hjensas has quit IRC | 15:01 | |
| openstackgerrit | Merged openstack/python-tripleoclient: Replace yaml.load() with yaml.safe_load() https://review.openstack.org/420589 | 15:01 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Add docker nova services https://review.openstack.org/420921 | 15:05 |
| bkero | EmilienM: I'll get to it after I get the scenarios implemented :) | 15:05 |
| bkero | the job for that just got +2W'd | 15:05 |
| bkero | so I can get onto implementing it now | 15:06 |
| *** absubram has quit IRC | 15:06 | |
| *** crenguta has quit IRC | 15:06 | |
| thrash | EmilienM: https://review.openstack.org/#/c/416284/ is hitting the ironic agent image problem I believe. | 15:07 |
| thrash | EmilienM: definitely unrelated to my patch | 15:07 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Add docker mistral services https://review.openstack.org/425744 | 15:08 |
| *** adarazs is now known as bug | 15:09 | |
| *** bug is now known as adarazs | 15:09 | |
| *** jrist has joined #tripleo | 15:09 | |
| *** jrist has quit IRC | 15:09 | |
| *** jrist has joined #tripleo | 15:09 | |
| EmilienM | bkero: which patch? | 15:10 |
| EmilienM | thrash: good to know | 15:10 |
| bkero | EmilienM: the one you +1'd. https://review.openstack.org/#/c/424034/ | 15:10 |
| openstackgerrit | Ana Krivokapic proposed openstack/python-tripleoclient: Don't create swift container if plan creation fails https://review.openstack.org/409350 | 15:11 |
| EmilienM | thrash: since https://review.openstack.org/#/c/425565/ merged, should we recheck? | 15:11 |
| EmilienM | to make sure we don't break | 15:11 |
| EmilienM | thrash: I saw ovb passing 2 days ago on this patch but since you change the code a little | 15:12 |
| EmilienM | I would recheck, in case | 15:12 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Enable docker services in the registry https://review.openstack.org/421567 | 15:13 |
| *** bala has joined #tripleo | 15:13 | |
| *** salmankhan has joined #tripleo | 15:18 | |
| thrash | EmilienM: it's got a recheck pending it looks like. | 15:19 |
| EmilienM | ok | 15:20 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Add release notes for Manila/CephFS with managed Ceph https://review.openstack.org/425747 | 15:20 |
| thrash | EmilienM: ovb jobs are currently queued | 15:20 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: Adds a AllNodesPreUpgradeConfig before the upgrade steps https://review.openstack.org/419886 | 15:22 |
| EmilienM | thrash: ok | 15:22 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/diskimage-builder: Ubuntu: Do not fail the GRUB2 package is already configured https://review.openstack.org/425750 | 15:26 |
| *** cylopez has quit IRC | 15:27 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/diskimage-builder: Ubuntu: Do not fail if the GRUB2 package is already configured https://review.openstack.org/425750 | 15:28 |
| *** eck` is now known as jeckersb_gone | 15:28 | |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: WIP: Support builds in multinode https://review.openstack.org/425474 | 15:28 |
| dtrainor | hi bandini, here's a ci failure i'm looking at http://logs.openstack.org/03/424903/4/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/bb0b56b/console.html | 15:29 |
| *** owalsh-afk is now known as owalsh | 15:30 | |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: Contrail services assigned to wrong roles https://review.openstack.org/425752 | 15:30 |
| *** apetrich has quit IRC | 15:30 | |
| bandini | dtrainor: looking, sec | 15:30 |
| *** jeckersb_gone is now known as eck` | 15:31 | |
| bandini | dtrainor: that is https://bugs.launchpad.net/tripleo/+bug/1659384 | 15:31 |
| openstack | Launchpad bug 1659384 in tripleo "All ovb jobs failing to set up snmp" [Critical,In progress] - Assigned to Ben Nemec (bnemec) | 15:31 |
| *** jaosorior has joined #tripleo | 15:31 | |
| openstackgerrit | Merged openstack/tripleo-quickstart-extras: Fixing setuptools version https://review.openstack.org/425097 | 15:31 |
| openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Remove rdo-qemu-ev workaround https://review.openstack.org/425753 | 15:32 |
| openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Drop ceph workaround for mitaka https://review.openstack.org/425754 | 15:32 |
| bnemec | EmilienM: Do we actually expect the ocata ovb job to pass at this point? | 15:32 |
| jaosorior | beagles, EmilienM can you re-check this https://review.openstack.org/#/c/407898/ ? I had to rebase cause of a merge conflict | 15:32 |
| EmilienM | bnemec: I don't know | 15:36 |
| EmilienM | bnemec: we'll figure out later today | 15:36 |
| EmilienM | jaosorior: done, approved. | 15:37 |
| bnemec | EmilienM: Okay, it's just that we're up to 10 jobs per tripleo-ci patch now. | 15:37 |
| EmilienM | bnemec: right, this time window until we have stable/ocata was expected, since infra merged my patch this morning | 15:38 |
| openstackgerrit | Dmitry Tantsur proposed openstack/instack-undercloud: Allow enabling node auto-discovery on the undercloud https://review.openstack.org/423265 | 15:40 |
| *** yamahata has joined #tripleo | 15:44 | |
| jaosorior | mwhahaha, EmilienM: Is there a way I can set a default for "define" resource in puppet via hiera? | 15:44 |
| pradk | can someone +A this https://review.openstack.org/#/c/424950/ | 15:44 |
| *** karimb has quit IRC | 15:44 | |
| *** tosky has quit IRC | 15:45 | |
| pradk | ty jaosorior | 15:45 |
| mwhahaha | jaosorior: yea I think it'll look up from heira | 15:45 |
| jaosorior | mwhahaha: thought that was for classes and includes, but not defines | 15:45 |
| EmilienM | yeah, you can't use define from hiera out of the box | 15:46 |
| EmilienM | it require a hiera plugin that we don't have in tripleo AFIK | 15:46 |
| mwhahaha | I thought I saw it in my debug logs, would have to double check | 15:46 |
| mwhahaha | Gimme a few and I'll look into it | 15:46 |
| *** shardy has joined #tripleo | 15:46 | |
| EmilienM | you could use something like https://github.com/puppetlabs/puppetlabs-create_resources but it's old and tbh I never tried it | 15:47 |
| openstackgerrit | Merged openstack/python-tripleoclient: Add overcloud_parameters.GenerateFencingParameters. https://review.openstack.org/417898 | 15:47 |
| openstackgerrit | Merged openstack/python-tripleoclient: Release note for disabling password generation https://review.openstack.org/425685 | 15:47 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Ignore systemctl return code in yum_update.sh https://review.openstack.org/425209 | 15:47 |
| *** openstackgerrit has quit IRC | 15:48 | |
| jaosorior | mwhahaha, EmilienM: well, for all users we create for MySQL (which is pretty much for all services) I want to set this parameter https://github.com/openstack/puppet-openstacklib/blob/master/manifests/db/mysql.pp#L64 | 15:49 |
| jaosorior | mwhahaha, EmilienM which I tried here https://review.openstack.org/#/c/425292/1/puppet/services/database/mysql-internal-tls-certmonger.yaml but I don't think that actually worked | 15:50 |
| EmilienM | jaosorior: you'll need to patch https://github.com/openstack/puppet-nova/blob/master/manifests/db/mysql.pp#L40 I think | 15:50 |
| EmilienM | and all modules | 15:50 |
| EmilienM | no it won't work | 15:50 |
| EmilienM | I mean: https://review.openstack.org/#/c/425292/1/puppet/services/database/mysql-internal-tls-certmonger.yaml | 15:50 |
| owalsh | shardy: are we still running the final converge for upgrades? | 15:51 |
| jaosorior | dammit | 15:52 |
| jaosorior | EmilienM: isn't there an easier way than patching all modules? | 15:52 |
| jaosorior | that doesn't scale very well :/ | 15:52 |
| *** karimb has joined #tripleo | 15:52 | |
| jaosorior | EmilienM: what if I set the default for the resource on the step_config: | 15:53 |
| shardy | owalsh: Right now no, but we'll need some way to avoid the upgrade tasks running on any subsequent update, which may involve a final stack update (unless we pass a flag in which differentiates an upgrade from any other update) | 15:53 |
| shardy | owalsh: we won't rely on the converge to apply puppet anymore tho, as we'll apply it as part of the upgrade, after the ansible steps | 15:53 |
| *** tosky has joined #tripleo | 15:54 | |
| *** openstackgerrit has joined #tripleo | 15:55 | |
| openstackgerrit | Merged openstack/tripleo-puppet-elements: Change upstream URL for puppet-contrail https://review.openstack.org/424872 | 15:55 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Fix for AllNodesExtraConfig and fix environment files to create swap files/partitions https://review.openstack.org/424079 | 15:55 |
| owalsh | shardy: I just realised nova needs a SIGHUP after the upgrade when the version is pinned to auto | 15:55 |
| shardy | owalsh: Ok, could that happen every time puppet runs, or only after the upgrade is completed? | 15:56 |
| EmilienM | jaosorior: I know :-/ let me think a bit more (maybe mwhahaha also has a better idea) | 15:56 |
| openstackgerrit | Merged openstack/tripleo-common: Remove remaining vendor plugins from default image YAML https://review.openstack.org/424152 | 15:56 |
| shardy | owalsh: we'll run puppet once after the upgrade on the controllers, then again everywhere during the "converge" | 15:56 |
| shardy | although the converge will just re-assert the existing state, and not change anything (in theory) | 15:56 |
| owalsh | shardy: we could remove the auto version pin | 15:57 |
| shardy | and rewrite the environemt to disable the upgrade steps | 15:57 |
| openstackgerrit | Alejandro Andreu proposed openstack/tripleo-puppet-elements: Install Curator from RPM in MidoNet element https://review.openstack.org/425205 | 15:57 |
| shardy | owalsh: Ok, so it's not really auto, it does a one-time detection on service reload? | 15:57 |
| owalsh | shardy: yes | 15:57 |
| shardy | owalsh: Ok we can revert to the explicit pin approach then I guess | 15:58 |
| shardy | and just do the same as previous releases | 15:58 |
| jaosorior | EmilienM: what if I write this in the step_config (which ends up at the top of the generated manifest) http://paste.openstack.org/show/596611/ wouldn't that set the default for all the resources of that type created? | 15:58 |
| owalsh | shardy: we can stick with auto for upgrade, but default to '' | 15:59 |
| mwhahaha | jaosorior: EmilienM: i guess you're right, you can't use hiera to set a default for a param. the other solution would be to switch all the invocations of that resource to an ensure_resource which would take a $defaults that could use a hiera lookup or something | 15:59 |
| *** iranzo has joined #tripleo | 15:59 | |
| *** yamahata has quit IRC | 15:59 | |
| jaosorior | mwhahaha: that would also mean changing most openstack modules :/ | 15:59 |
| fpan | shardy: could you take a look at https://review.openstack.org/#/c/422898/ and do a +workflow | 16:00 |
| mwhahaha | jaosorior: don't we expose all those via puppet classes? | 16:00 |
| mwhahaha | jaosorior: you just don't want to define them all i guess | 16:00 |
| shardy | fpan: sure, done! | 16:01 |
| fpan | shardy: thanks! | 16:01 |
| jaosorior | mwhahaha: I don't want to define them all, and that parameter is exposed only in openstacklib (which gets used by all openstack modules) | 16:02 |
| jaosorior | mwhahaha: I just wanted to set the default there, so all mysql users (created via that resource) use it | 16:02 |
| mwhahaha | jaosorior: yea there's no clean way to do that, | 16:03 |
| jaosorior | mwhahaha, EmilienM would it be acceptable to do a hieracall from openstacklib? https://github.com/openstack/puppet-openstacklib/blob/master/manifests/db/mysql.pp#L64 | 16:03 |
| mwhahaha | jaosorior: no | 16:03 |
| mwhahaha | we keep hiera out of openstack lib | 16:03 |
| mwhahaha | and all the other modules for that matter | 16:03 |
| jaosorior | mwhahaha: why? | 16:03 |
| EmilienM | s/openstack lib/Puppet OpenStack/ | 16:03 |
| EmilienM | because Hiera is not a requirement to deploy Puppet OpenStack modules | 16:04 |
| jaosorior | EmilienM: I was referring to puppet-openstacklib | 16:04 |
| jaosorior | oh, ok | 16:04 |
| EmilienM | jaosorior: a lot of our users don't use hiera | 16:04 |
| jaosorior | EmilienM: I see (I legitimately didn't know that) | 16:04 |
| *** yamahata has joined #tripleo | 16:05 | |
| jaosorior | EmilienM, mwhahaha sooo... what about what I said earlier? passing this http://paste.openstack.org/show/596611/ via t-h-t on the step_config | 16:06 |
| mwhahaha | jaosorior: I think you want the spaceship operator | 16:06 |
| jaosorior | ah | 16:07 |
| jaosorior | you're right | 16:07 |
| EmilienM | mwhahaha: you h@cker | 16:07 |
| jaosorior | what I mentioned would also work only with classes and includes | 16:07 |
| EmilienM | mwhahaha: can you do it on a Define? I think I never tried | 16:08 |
| mwhahaha | yes | 16:08 |
| EmilienM | cool, yeah do it then, even if it's not the cleanest way ;-) | 16:08 |
| mwhahaha | Openstacklib::Db::Mysql <||> { tls_options => ['SSL'] } | 16:08 |
| mwhahaha | that's all you need | 16:08 |
| mwhahaha | so if $tls_enabled -^ | 16:08 |
| EmilienM | sounds neat & quick for now | 16:08 |
| mwhahaha | then any catalog that has that resource would get updated | 16:09 |
| jaosorior | mwhahaha: that manifest where I would call it would only be defined if tls everywhere is enabled (it's OS::Heat::None by default) | 16:09 |
| *** salmankhan has quit IRC | 16:10 | |
| *** jrist has quit IRC | 16:10 | |
| jaosorior | mwhahaha, EmilienM I'll give it a try | 16:11 |
| jaosorior | thanks guys | 16:11 |
| *** ansmith has quit IRC | 16:12 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Add upgrade support for CephRGW service https://review.openstack.org/423360 | 16:12 |
| * beagles wonders if it is worthwhile combing IRC logs and mine for tidbits like the above | 16:13 | |
| mwhahaha | you don't know what you're looking for until you need it :D | 16:13 |
| *** karimb has quit IRC | 16:15 | |
| beagles | :) | 16:15 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Force MySQL users to use SSL if internal TLS is enabled https://review.openstack.org/425292 | 16:16 |
| jpich | d0ugal: I saw an email about the "requirements" freeze on the dev list, do we need to bump tripleo-common over there to whatever will be our Ocata version? (I'm lost once again in version numbers) | 16:16 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci: DO NOT MERGE - testing TLS everywhere job with MySQL https://review.openstack.org/425295 | 16:16 |
| *** ckyriakidou has joined #tripleo | 16:16 | |
| *** salmankhan has joined #tripleo | 16:16 | |
| d0ugal | d0ugal: hrm, good question. I'm not really sure. EmilienM ? | 16:17 |
| *** ckyriakidou has quit IRC | 16:17 | |
| jpich | d0ugal, EmilienM: Actually it looks like the version at https://github.com/openstack/requirements/blob/master/global-requirements.txt is pretty old already, 5.4 is referenced under Newton? https://releases.openstack.org/newton/index.html#tripleo-common | 16:18 |
| jpich | https://github.com/openstack/requirements/blob/master/global-requirements.txt#L285 | 16:18 |
| openstackgerrit | Marios Andreou proposed openstack/instack-undercloud: Ammend the completion and error message to signal an upgrade https://review.openstack.org/425770 | 16:18 |
| d0ugal | jpich: I guess updating it to the latest makes sense | 16:18 |
| * d0ugal shrugs | 16:18 | |
| *** abregman has quit IRC | 16:19 | |
| jpich | d0ugal: Off the top of your mind, do you know what will be the first version number for Ocata? It feels like that's what we should reference even if it doesn't exist yet (...if that's possible) | 16:20 |
| *** aufi has quit IRC | 16:20 | |
| jaosorior | mwhahaha: ah, so that spaceship operator doesn't need to be in the top level? | 16:20 |
| mwhahaha | no it just has to be in the catalog | 16:20 |
| mwhahaha | that's the awesomeness/danger of it | 16:20 |
| *** jcoufal has quit IRC | 16:20 | |
| *** jrist has joined #tripleo | 16:21 | |
| *** jrist has quit IRC | 16:21 | |
| *** jrist has joined #tripleo | 16:21 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci: DO NOT MERGE - testing TLS everywhere job (neutron) https://review.openstack.org/419529 | 16:22 |
| *** ansmith has joined #tripleo | 16:24 | |
| jpich | d0ugal: I think ocata is 6.0 but we can't require installing >= something that doesn't exist yet, so... I'll send a requirements update with the latest number I can find and hope for the best I guess | 16:25 |
| yolanda | ah trown, thx for review | 16:27 |
| yolanda | just saw it | 16:27 |
| *** karimb has joined #tripleo | 16:27 | |
| *** jtomasek has quit IRC | 16:27 | |
| d0ugal | jpich: latests is 5.7 I think! | 16:27 |
| d0ugal | jpich: and yeah, I think 6.0 is Ocata | 16:28 |
| jpich | d0ugal: Thanks! | 16:28 |
| EmilienM | jpich, d0ugal: good question, tbh since we use RDO packages, we don't really care about this thing. Maybe we should do a better job at updating it? | 16:29 |
| openstackgerrit | Dan Radez proposed openstack/tripleo-heat-templates: Adding Tacker Support https://review.openstack.org/424135 | 16:29 |
| marios | bnemec: thanks :) | 16:29 |
| jaosorior | beagles: ping | 16:31 |
| beagles | jaosorior: yo! | 16:31 |
| jaosorior | beagles: hey dude; how acquainted are you with the neutron CLI client? | 16:31 |
| d0ugal | EmilienM: our unit tests care about it :) | 16:31 |
| jpich | EmilienM: I suppose. Doing a poor job of it can create all sorts of other issues (e.g. the mess with the Mitaka unit tests atm https://review.openstack.org/#/c/421543/ ). I'll try to pay more attention to it, but I think because we're going to release tripleo-common on the same day/after requirements are frozen on master we'll always be a little bit off | 16:31 |
| beagles | jaosorior: mmmm... so-so | 16:32 |
| beagles | jaosorior: what's on the go? | 16:32 |
| jpich | d0ugal, EmilienM: Looks like upper-constraints was automagically updated to the last version at least. I'll need to read more about how that one works :) | 16:32 |
| *** thrash is now known as thrash|biab | 16:33 | |
| EmilienM | jpich, d0ugal: thanks for looking at it :-) | 16:33 |
| jaosorior | beagles: so I'm thinking about switching the neutron commands for the openstack unified CLI in the pingtest https://github.com/openstack-infra/tripleo-ci/blob/master/scripts/tripleo.sh#L965 | 16:34 |
| EmilienM | thrash|biab: since 416284 is still queued, I propose to land 416284 once we see 419506 passing OVB (419506 is behind the other in zuul) | 16:34 |
| jaosorior | beagles: but, for instance, here https://github.com/openstack-infra/tripleo-ci/blob/master/scripts/tripleo.sh#L996 I'm a bit thrown off by the --router:external ... I don't even find that in the regular neutron CLi documentation | 16:35 |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: fixup https://review.openstack.org/425776 | 16:35 |
| *** dprince has quit IRC | 16:37 | |
| *** b00tcat has quit IRC | 16:37 | |
| *** jcoufal has joined #tripleo | 16:37 | |
| *** leanderthal has joined #tripleo | 16:38 | |
| beagles | jaosorior: when you see --router:external, it basically means that this network will be routed through the L3 agent to an network external to it.. | 16:42 |
| beagles | jaosorior: I've not used the osc for this, but the --external command line option looks promising | 16:42 |
| *** yamahata has quit IRC | 16:42 | |
| * beagles needs to take a few minutes and scan the code | 16:43 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci: TESTING openstack CLI instead of neutron https://review.openstack.org/425782 | 16:46 |
| jaosorior | trying it out | 16:46 |
| *** leanderthal has quit IRC | 16:47 | |
| pradk | this can be merged now https://review.openstack.org/#/c/424746/ | 16:48 |
| *** apetrich has joined #tripleo | 16:48 | |
| *** saneax is now known as saneax-_-|AFK | 16:52 | |
| *** iranzo has quit IRC | 16:53 | |
| bnemec | jaosorior: I would really rather not do that right now. AFAIK the neutron client is not deprecated yet, and every ci patch eats up 10 ovb jobs. :-/ | 16:54 |
| openstackgerrit | Merged openstack/tripleo-common: Add element to regenerate dracut https://review.openstack.org/420698 | 16:54 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Use versionless keystone endpoint for barbican-related configurations https://review.openstack.org/407898 | 16:55 |
| *** tesseract has quit IRC | 16:55 | |
| openstackgerrit | Martin André proposed openstack/tripleo-common: Add missing linebreak in delorean-current.repo https://review.openstack.org/425789 | 16:55 |
| shardy | marios, matbu: can you guys talk and reconcile your two patches delivering upgrade scripts? | 16:56 |
| shardy | https://review.openstack.org/#/c/419886 | 16:56 |
| openstackgerrit | Merged openstack/os-refresh-config: Updated from global requirements https://review.openstack.org/420924 | 16:56 |
| *** athomas has quit IRC | 16:56 | |
| shardy | I commented there, but I think your patches overlap | 16:56 |
| openstackgerrit | Ryan Brady proposed openstack/tripleo-common: Adds support to create deployment plan from git repository https://review.openstack.org/417280 | 16:56 |
| jaosorior | bnemec: so, the good thing is that the patch that tests it is on top of my TLS-everywhere testing patches. Those do exit 0 on everything except one job... where I run the actual TLS-everywhere job | 16:57 |
| EmilienM | mwhahaha: can you give a super-quick status on the undercloud-upgrade CI job? (blockers, etc) | 16:57 |
| jaosorior | fuck | 16:57 |
| jaosorior | it's not a neutron CLI bug, just realised what I missed | 16:58 |
| EmilienM | mwhahaha: or is everything in https://review.openstack.org/#/c/419121/ should make it pass again | 16:58 |
| jaosorior | bnemec: anyway... I started thinking about it cause I tried getting neutron with TLS in the internal network | 16:58 |
| jaosorior | BUT, I think I just realised where I messed up | 16:58 |
| mwhahaha | EmilienM: the job was failing because we don't actually have a compute instance | 16:58 |
| matbu | shardy: /me looks | 16:58 |
| mwhahaha | EmilienM: but the patch in nova that removes the requirement when that lands should clear that up | 16:58 |
| EmilienM | mwhahaha: it's likely a bug in nova, then | 16:58 |
| openstackgerrit | Ryan Brady proposed openstack/tripleo-common: Adds support to create deployment plan from git repository https://review.openstack.org/417280 | 16:58 |
| EmilienM | mwhahaha: https://review.openstack.org/#/c/424243/ ? | 16:59 |
| *** morazi has quit IRC | 16:59 | |
| mwhahaha | EmilienM: well from what I hear they argue that upgrading without any compute nodes doesn't make sense | 16:59 |
| mwhahaha | EmilienM: yea that | 16:59 |
| matbu | shardy: btw, is there a reviews (i didn't find one but maybe i missed it) that automaticaly skip the services that are not deployed ? | 16:59 |
| matbu | shardy: if not i'll make one in a short | 16:59 |
| shardy | matbu: the service templates aren't included if the services aren't deployed | 16:59 |
| *** leanderthal has joined #tripleo | 17:00 | |
| EmilienM | mwhahaha: ok, let's see when we have a promotion | 17:00 |
| shardy | matbu: basically the composable services interface takes care of it, we only run upgrade_steps for services that are enabled | 17:00 |
| *** dbecker has quit IRC | 17:00 | |
| *** saneax-_-|AFK is now known as saneax | 17:00 | |
| EmilienM | panda, sshnaidm: what are current blockers for promotion in your knowledge? | 17:00 |
| matbu | shardy: hm yep but on my local deployment that's not the cases | 17:00 |
| shardy | matbu: can you provide more details? Not sure I understand exactly | 17:00 |
| *** rajinir has joined #tripleo | 17:01 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Clean TLS proxy-related setup for neutron-server profile https://review.openstack.org/425320 | 17:01 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Use TLS proxy for neutron server's internal TLS https://review.openstack.org/425267 | 17:01 |
| matbu | shardy: the role_data is full of upgrade_tasks with services that is not deployed on my env | 17:01 |
| EmilienM | sshnaidm, panda: oh I see 359215 in zuul, running OVB, that could gives us a promotion (jobs are running now) | 17:01 |
| shardy | matbu: have you deployed with a customized roles_data.yaml, then redeployed? | 17:01 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Pass parameters for TLS proxy in front of neutron server https://review.openstack.org/425317 | 17:01 |
| sshnaidm | EmilienM, yeah, panda ran them today | 17:01 |
| EmilienM | cool | 17:01 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci: DO NOT MERGE - testing TLS everywhere job (neutron) https://review.openstack.org/419529 | 17:02 |
| shardy | matbu: there is an open bug where the plan doesn't automatically clean the roles_data | 17:02 |
| EmilienM | sshnaidm: blockers should be 0 at this time, we promoted Puppet CI this morning | 17:02 |
| shardy | matbu: try doing an openstack plan delete overcloud | 17:02 |
| shardy | before the initial deploy | 17:02 |
| matbu | shardy: yep, i provied a custom role_data | 17:02 |
| *** [1]cdearborn has joined #tripleo | 17:02 | |
| shardy | or do openstack overcloud delete overcloud | 17:02 |
| shardy | to delete it | 17:02 |
| *** liverpooler has quit IRC | 17:03 | |
| matbu | shardy: ack i will try that | 17:03 |
| shardy | openstack stack output show overcloud EnabledServices | 17:03 |
| bnemec | EmilienM: That's like a whole 4 hours ago. Plenty of time for things to have gone wrong in the interim. :-P | 17:03 |
| shardy | matbu: ^^ that tells you what is enabled | 17:03 |
| shardy | anything in that list will be upgraded | 17:03 |
| shardy | if anything else gets upgraded, it's a bug and I'll need more details or a reproducer | 17:03 |
| EmilienM | bnemec: yeah - though I stay confident this time | 17:04 |
| *** liverpooler has joined #tripleo | 17:04 | |
| bnemec | EmilienM: Well, someone's got to balance my pessimism. | 17:04 |
| EmilienM | thrash|biab: I'm about to approve https://review.openstack.org/#/c/416284/ in a very few. I'm watching OVB jobs and they are deploying step1 already. I think it's good to go | 17:04 |
| matbu | shardy: ok probably an issue with my dployment, because the services is in the list (for example, i didn't deploy glance, but glance is in the list) | 17:05 |
| EmilienM | bnemec: someone's got to balance my unrealistic way to think sometimes ;-) | 17:05 |
| matbu | shardy: and i have no glance configured | 17:05 |
| matbu | shardy: i'll clean my env, before warn for a bug | 17:05 |
| gfidente | bnemec++ | 17:06 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: Support builds in multinode https://review.openstack.org/425474 | 17:07 |
| gfidente | EmilienM don't hate me but I need bnemec spirit so much on my submissions | 17:08 |
| gfidente | I always pull him | 17:08 |
| gfidente | now thinking about it I might add him to a couple more | 17:08 |
| EmilienM | :-) | 17:09 |
| bnemec | gfidente: You need someone to leave a "This will probably fail on an unrelated bug" comment on your patches? | 17:09 |
| bnemec | I can probably do that. :-) | 17:09 |
| gfidente | no also "This will probably fail in this scenario" | 17:10 |
| shardy | matbu: https://bugs.launchpad.net/tripleo/+bug/1644875 is the bug | 17:12 |
| openstack | Launchpad bug 1644875 in tripleo "Plan isn't purged of roles_data on a new create" [Medium,Triaged] | 17:12 |
| shardy | I'm sad to see it's not even assigned | 17:12 |
| shardy | d0ugal: any way we can get some eyes on that one? ^^ | 17:12 |
| shardy | it'd be very good to fix before the release IMO | 17:13 |
| *** hewbrocca is now known as hewbrocca_afk | 17:13 | |
| matbu | shardy: ack this what i'm hitting | 17:14 |
| *** yprokule has quit IRC | 17:14 | |
| *** tobias-fiberdata has quit IRC | 17:14 | |
| EmilienM | shardy: it's in tripleo-common or client too? | 17:14 |
| matbu | shardy: and the workaround works | 17:14 |
| shardy | EmilienM: I think it's actually a bug in triple-common | 17:14 |
| matbu | it moved to pike | 17:14 |
| EmilienM | ok | 17:14 |
| shardy | the plan delete workflow doesn't clean the roles_data | 17:14 |
| EmilienM | we can move it to rc1 if it's in tripleo-common | 17:15 |
| shardy | or rather, we don't clean it on a new create, when the stack has been removed | 17:15 |
| EmilienM | I updated tags | 17:15 |
| shardy | EmilienM: currently there's some hacky workarounds for this in tripleoclient, which is why I added the tripleoclient tag | 17:15 |
| openstackgerrit | Merged openstack/instack-undercloud: Ensure Aodh uses its own mysql db https://review.openstack.org/424950 | 17:15 |
| shardy | the correct fix is to actually purge the whole plan when you do a new deploy with a stale plan and a deleted stack | 17:15 |
| shardy | which can be done in tripleo-common | 17:15 |
| shardy | atm all we do is empty the swift container from tripleoclient IIRC | 17:16 |
| EmilienM | moved to rc1 and appropriated tags | 17:16 |
| *** cdearborn has quit IRC | 17:16 | |
| EmilienM | d0ugal, shardy: can you quickly review https://review.openstack.org/#/c/419506/ (and +2 if good)? it's our last blocker to release tripleoclient at this time | 17:17 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: DONT REVIEW: testing build multi https://review.openstack.org/425793 | 17:17 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Allow dnsmasq_dns_servers to be configured for DHCP Agent https://review.openstack.org/422898 | 17:19 |
| *** lucasagomes is now known as lucas-afk | 17:19 | |
| *** dtantsur is now known as dtantsur|afk | 17:21 | |
| EmilienM | shardy: can you review this CI upgrade thing please? https://review.openstack.org/#/c/425694 | 17:22 |
| pradk | shardy, are you comfortable if i merge this https://review.openstack.org/#/c/405249/ | 17:22 |
| pradk | shardy, i think it should be in good shape .. we still have to figure out redis, but that can be separate | 17:23 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: ci: import multinode_major_upgrade.yaml from tripleo-ci https://review.openstack.org/425690 | 17:23 |
| *** jpich has quit IRC | 17:23 | |
| openstackgerrit | Michael Henkel proposed openstack/tripleo-heat-templates: fixup https://review.openstack.org/425796 | 17:23 |
| *** zoli is now known as zoli|gone | 17:24 | |
| *** derekh has quit IRC | 17:24 | |
| *** bala has quit IRC | 17:24 | |
| shardy | pradk: Yeah I think so - I was going to test locally again, but I think lets land it and deal with any issues in a followup | 17:25 |
| shardy | pradk: thanks for your help with that one :) | 17:25 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/tripleo-common: Updated from global requirements https://review.openstack.org/424627 | 17:32 |
| *** karimb has quit IRC | 17:33 | |
| *** iranzo has joined #tripleo | 17:33 | |
| *** rasca has quit IRC | 17:34 | |
| *** karimb has joined #tripleo | 17:36 | |
| gfidente | thanks jaosorior :) | 17:40 |
| *** jmelvin has quit IRC | 17:42 | |
| jaosorior | gfidente: no biggie | 17:42 |
| *** liverpooler has quit IRC | 17:43 | |
| *** liverpoo1er has joined #tripleo | 17:43 | |
| *** karimb has quit IRC | 17:46 | |
| *** jlinkes has quit IRC | 17:48 | |
| owalsh | shardy: https://review.openstack.org/425220 did the last recheck fail? Didn't think it had finished yet | 17:49 |
| shardy | owalsh: doh, yeah sorry you're right, misread the timestamp from the jenkins check queue | 17:50 |
| *** trown is now known as trown|lunch | 17:51 | |
| shardy | I'll check back on it later, hopefully we can land it tomorrow - tested it locally and it works well, thanks! | 17:51 |
| openstackgerrit | Merged openstack/instack-undercloud: Add release notes for Ocata 6.0.0 https://review.openstack.org/424746 | 17:52 |
| *** shardy is now known as shardy_afk | 17:53 | |
| owalsh | shardy: np. What is the situation with FF and landing these patches tomorrow? | 17:57 |
| jaosorior | EmilienM: where did the multinode-upgrades job rub? for https://review.openstack.org/#/c/425694/1 | 17:58 |
| *** david-lyle has quit IRC | 17:59 | |
| jaosorior | EmilienM: the change looks fine, I was just wondering if you had a chance to run it | 18:00 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: Add multinode scenario job for repo promotion https://review.openstack.org/424282 | 18:05 |
| *** pkovar has quit IRC | 18:05 | |
| EmilienM | jaosorior: gate-tripleo-ci-centos-7-multinode-upgrades-nv | 18:07 |
| *** owalsh is now known as owalsh-afk | 18:08 | |
| EmilienM | jaosorior: ah, I see your question, it didn't run | 18:08 |
| EmilienM | jaosorior: brb mtg | 18:08 |
| jaosorior | EmilienM: ok | 18:08 |
| jaosorior | EmilienM: has it been tried out? Do you think it's fine to merge? | 18:10 |
| EmilienM | jaosorior: yes, it's fine | 18:11 |
| * bnemec apologizes in advance | 18:11 | |
| openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Test with scheduler hints https://review.openstack.org/378040 | 18:11 |
| openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Add feature testing matrix to readme https://review.openstack.org/399269 | 18:11 |
| openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Add support for testing predictable placement https://review.openstack.org/378014 | 18:11 |
| openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Test hostname map https://review.openstack.org/378017 | 18:11 |
| *** salmankhan1 has joined #tripleo | 18:12 | |
| *** salmankhan has quit IRC | 18:12 | |
| *** salmankhan1 is now known as salmankhan | 18:12 | |
| *** owalsh-afk has quit IRC | 18:14 | |
| *** leanderthal has quit IRC | 18:15 | |
| EmilienM | bnemec: bouuu | 18:17 |
| *** funzo_ has joined #tripleo | 18:17 | |
| *** funzo_ has quit IRC | 18:18 | |
| *** owalsh-afk has joined #tripleo | 18:18 | |
| *** owalsh-afk is now known as owalsh | 18:18 | |
| *** gfidente is now known as gfidente|afk | 18:18 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/instack-undercloud: Update release notes to include aodh db changes https://review.openstack.org/425817 | 18:19 |
| openstackgerrit | Merged openstack/tripleo-common: Add support for calling validations in deploy and introspect workflows https://review.openstack.org/416284 | 18:19 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/tripleo-common: Updated from global requirements https://review.openstack.org/424627 | 18:20 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: Fix multinode env for periodic scenario jobs https://review.openstack.org/424282 | 18:22 |
| EmilienM | thrash|biab: https://review.openstack.org/#/c/419506/ in gate now | 18:23 |
| EmilienM | if there is anything else we need in tripleoclient, it's the right time to ask, I'm about to release once https://review.openstack.org/#/c/419506/ is merged. | 18:23 |
| openstackgerrit | Merged openstack/python-tripleoclient: Add support for calling validations from deployment https://review.openstack.org/418507 | 18:23 |
| openstackgerrit | Merged openstack/python-tripleoclient: Remove vendor plugins from image build https://review.openstack.org/424331 | 18:24 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: Add periodic scenario jobs to promotion https://review.openstack.org/425819 | 18:24 |
| sshnaidm | EmilienM, bnemec could you please review? https://review.openstack.org/#/c/424282/ it's required for periodic scenario jobs to run smoothly | 18:25 |
| EmilienM | sshnaidm: looking | 18:25 |
| *** yamahata has joined #tripleo | 18:26 | |
| *** jtomasek has joined #tripleo | 18:26 | |
| openstackgerrit | Honza Pokorny proposed openstack/tripleo-ui: Fix typo in List Plan HTML https://review.openstack.org/425820 | 18:26 |
| EmilienM | sshnaidm: this syntax removes "periodic-" from JOBTYPE, right? | 18:26 |
| sshnaidm | EmilienM, yeah | 18:27 |
| EmilienM | sshnaidm: nice :D | 18:27 |
| *** dprince has joined #tripleo | 18:28 | |
| jaosorior | sshnaidm: but, where does the JOBTYPE come from? wouldn't it have the gate- prefix for regular jobs? | 18:28 |
| sshnaidm | jaosorior, it's from jjb definition of the job: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/projects.yaml#L14207 | 18:30 |
| sshnaidm | jaosorior, for regular it's only: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/projects.yaml#L14191 | 18:30 |
| jaosorior | sshnaidm: aaah right | 18:30 |
| EmilienM | sshnaidm: https://review.openstack.org/#/c/425184/ BOOM | 18:31 |
| jaosorior | great | 18:31 |
| sshnaidm | EmilienM, but it's check, not recheck! | 18:32 |
| *** rcernin has joined #tripleo | 18:32 | |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: [WIP] break out general_config into config, topology and provisioner https://review.openstack.org/425451 | 18:32 |
| bnemec | Promotion job failed :-( | 18:34 |
| bnemec | Man, those ocata stable jobs are going to be brutal. | 18:35 |
| bnemec | We get all the slowness of master and none of the cached images. | 18:35 |
| *** tosky has quit IRC | 18:35 | |
| *** dsneddon_afk is now known as dsneddon | 18:35 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add Ceph RBD mirror Pacemaker profile https://review.openstack.org/407039 | 18:37 |
| EmilienM | bnemec: Stack overcloud CREATE_COMPLETE though | 18:38 |
| bnemec | EmilienM: Yeah, something looks fishy with the deployed nodes check. | 18:39 |
| * bnemec blames cells v2 | 18:39 | |
| EmilienM | echo 'Wrong number of nodes deployed' | 18:39 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Conform CephExternal template to the new hiera hook https://review.openstack.org/425288 | 18:39 |
| EmilienM | let's read the logs | 18:39 |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: [WIP] break out general_config into config, topology and provisioner https://review.openstack.org/425451 | 18:39 |
| EmilienM | bnemec: you weren't in the TripleO CI squad meeting this morning but we have talked about using RDO Trello to track CI failures more easily than using etherpad | 18:39 |
| bnemec | EmilienM: I only track ci problems with alert bugs. | 18:41 |
| EmilienM | http://logs.openstack.org/15/359215/43/check-tripleo/gate-tripleo-ci-centos-7-ovb-nonha/ad8aab8/logs/ | 18:41 |
| EmilienM | where are overcloud logs? | 18:41 |
| jaosorior | EmilienM: what patch was that? | 18:41 |
| EmilienM | jaosorior: promotion patch | 18:41 |
| EmilienM | bnemec: yes but alerts are not enough | 18:41 |
| EmilienM | bnemec: alerts are not used when a promotion is blocked | 18:41 |
| EmilienM | http://logs.openstack.org/15/359215/43/check-tripleo/gate-tripleo-ci-centos-7-ovb-nonha/ad8aab8/logs/undercloud/var/log/nova/nova-api.txt.gz#_2017-01-26_18_22_51_334 | 18:41 |
| EmilienM | touché (it's cells) | 18:42 |
| EmilienM | mwhahaha: ^ | 18:42 |
| bnemec | EmilienM: I'm saying it's kind of irrelevant to me since I don't use either today. :-) | 18:42 |
| EmilienM | bnemec: it would be great to find a process that works for the team | 18:43 |
| EmilienM | bnemec: and that's what we discussed this morning | 18:43 |
| EmilienM | mwhahaha: do we setup cells on the undercloud? I don't think so | 18:43 |
| EmilienM | mwhahaha: https://review.openstack.org/#/c/419121/ is only for upgrades | 18:44 |
| *** jpena is now known as jpena|off | 18:44 | |
| EmilienM | mwhahaha: do we need to just include include ::nova::cell_v2::simple_setup in the undercloud? | 18:45 |
| EmilienM | (same for overcloud) | 18:45 |
| EmilienM | owalsh: fyi | 18:47 |
| *** jprovazn has quit IRC | 18:48 | |
| *** Goneri has quit IRC | 18:48 | |
| *** nyechiel_ has quit IRC | 18:48 | |
| *** jtomasek has quit IRC | 18:49 | |
| *** nyechiel_ has joined #tripleo | 18:49 | |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: docker: eliminate copy-json.py in favor of json-file https://review.openstack.org/416420 | 18:49 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: docker: new hybrid deployment architecture and configuration https://review.openstack.org/416421 | 18:49 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Add docker keystone service https://review.openstack.org/416605 | 18:49 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-cloud-config: Updated from global requirements https://review.openstack.org/423276 | 18:50 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Add docker mysql service https://review.openstack.org/414601 | 18:50 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Add docker mistral services https://review.openstack.org/425744 | 18:51 |
| *** fzdarsky is now known as fzdarsky|afk | 18:51 | |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Add docker nova services https://review.openstack.org/420921 | 18:52 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-tripleoclient: Updated from global requirements https://review.openstack.org/425837 | 18:52 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Add docker neutron services https://review.openstack.org/422999 | 18:52 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Add docker Glance API service https://review.openstack.org/400870 | 18:53 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Add docker heat services https://review.openstack.org/417639 | 18:53 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/tripleo-common: Updated from global requirements https://review.openstack.org/424627 | 18:53 |
| *** masco has quit IRC | 18:53 | |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Add docker ironic services https://review.openstack.org/421517 | 18:54 |
| *** trown|lunch is now known as trown | 18:54 | |
| *** mcornea has quit IRC | 18:55 | |
| *** nyechiel_ has quit IRC | 18:58 | |
| bnemec | Bleh, 5.5 hour queue | 18:58 |
| * bnemec is losing his fight against the check-tripleo queue | 18:58 | |
| *** nyechiel_ has joined #tripleo | 18:58 | |
| EmilienM | yeah docker killed us :P | 18:59 |
| trown | pinche docker | 18:59 |
| jaosorior | bnemec: aren't the updates ovb jobs the ones that take the longest? | 18:59 |
| jaosorior | trown: lol | 18:59 |
| bnemec | jaosorior: No, actually they aren't, at least not always. | 18:59 |
| bnemec | The worst jobs are newton and ocata anything because they have to build images every time. | 19:00 |
| jaosorior | bnemec: why? | 19:00 |
| bnemec | jaosorior: We don't cache images for stable branches | 19:00 |
| jaosorior | bnemec: maybe we should :O | 19:00 |
| bnemec | jaosorior: I said as much on the mailing list a few days ago when we were discussion how to handle the ocata release. | 19:01 |
| EmilienM | ok lunch break at this point | 19:01 |
| bnemec | *discussing | 19:01 |
| *** jtomasek has joined #tripleo | 19:02 | |
| bnemec | We can probably go a little higher on the testenvs too, although above about 75 we will start to run out of testenv workers. | 19:02 |
| bnemec | And adding testenv workers requires a broker restart that will kill all of the running jobs. | 19:02 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Add docker nova services https://review.openstack.org/420921 | 19:03 |
| jaosorior | bnemec: currently we're at 65, no? | 19:03 |
| bnemec | jaosorior: 70 | 19:04 |
| EmilienM | I would avoid changing this number now, please | 19:04 |
| bnemec | I bumped it again yesterday. | 19:04 |
| jaosorior | oh, ok | 19:04 |
| mwhahaha | EmilienM: yes for undercloud install we just need to include it | 19:04 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Enable docker services in the registry https://review.openstack.org/421567 | 19:04 |
| EmilienM | mwhahaha: overcloud too, I guess? | 19:04 |
| *** crenguta has joined #tripleo | 19:05 | |
| *** derekh has joined #tripleo | 19:05 | |
| EmilienM | mwhahaha: do we need to give some parameters, or just include? | 19:05 |
| *** nyechiel_ has quit IRC | 19:06 | |
| mwhahaha | Just include | 19:06 |
| mwhahaha | Overcloud same thing probably | 19:07 |
| *** nyechiel_ has joined #tripleo | 19:07 | |
| EmilienM | mwhahaha: done ^ | 19:07 |
| openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: nova: create basic setup for cells https://review.openstack.org/425844 | 19:07 |
| *** jmelvin has joined #tripleo | 19:07 | |
| *** pkovar has joined #tripleo | 19:08 | |
| EmilienM | mwhahaha: from where should we do it on the overcloud? on each node where nova compute runs? | 19:08 |
| mwhahaha | No where the API runs | 19:08 |
| EmilienM | ok | 19:08 |
| mwhahaha | It's a nova manage command so needs DB connection | 19:09 |
| *** crenguta has quit IRC | 19:09 | |
| EmilienM | mwhahaha: last question, should I remove https://github.com/openstack/puppet-tripleo/blob/master/manifests/profile/base/nova/api.pp#L95-L107 in the same time? | 19:09 |
| mwhahaha | Yes not needed anymore | 19:10 |
| EmilienM | ok | 19:10 |
| mwhahaha | I'll review after lunch | 19:10 |
| EmilienM | thx | 19:10 |
| *** crenguta has joined #tripleo | 19:11 | |
| jaosorior | beagles: could you check this out https://review.openstack.org/#/c/421575/ ? | 19:12 |
| beagles | jaosorior: yup | 19:12 |
| openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: nova: create basic setup for cells https://review.openstack.org/425844 | 19:12 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: nova: deploy basic setup for cells https://review.openstack.org/425848 | 19:13 |
| EmilienM | ok these 2 things should give us a promotion once it lands, hopefully | 19:14 |
| * EmilienM lunch_real now | 19:15 | |
| *** jmelvin has quit IRC | 19:15 | |
| *** nyechiel_ has quit IRC | 19:15 | |
| *** jmelvin has joined #tripleo | 19:15 | |
| *** nyechiel_ has joined #tripleo | 19:16 | |
| *** akrivoka has quit IRC | 19:17 | |
| *** yolanda has quit IRC | 19:17 | |
| *** absubram has joined #tripleo | 19:18 | |
| *** salmankhan has quit IRC | 19:18 | |
| *** akrivoka has joined #tripleo | 19:18 | |
| *** nyechiel_ has quit IRC | 19:20 | |
| *** jtomasek has quit IRC | 19:20 | |
| *** nyechiel_ has joined #tripleo | 19:20 | |
| *** thrash|biab is now known as thrash | 19:21 | |
| thrash | EmilienM: looks like we're all set? | 19:22 |
| EmilienM | thrash: we are. | 19:22 |
| EmilienM | Good job here! | 19:22 |
| thrash | EmilienM: excellent! | 19:22 |
| EmilienM | thrash: I'll proceed to the release once it lands | 19:22 |
| EmilienM | it's in gate now. | 19:22 |
| thrash | EmilienM: for tripleo-common... https://review.openstack.org/#/c/425677/ | 19:22 |
| thrash | Release notes | 19:22 |
| *** jtomasek has joined #tripleo | 19:23 | |
| EmilienM | thrash: awesome! | 19:23 |
| EmilienM | ok let me cook my french dinner now | 19:23 |
| jaosorior | thrash: hey dude, could you check this out https://review.openstack.org/#/c/421575/ ? :D | 19:24 |
| thrash | jaosorior: ack | 19:25 |
| thrash | EmilienM: :) | 19:25 |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: fixup https://review.openstack.org/425855 | 19:26 |
| *** karimb has joined #tripleo | 19:26 | |
| *** derekh has quit IRC | 19:27 | |
| *** saneax is now known as saneax-_-|AFK | 19:31 | |
| openstackgerrit | Ana Krivokapic proposed openstack/python-tripleoclient: Add plan export command https://review.openstack.org/425858 | 19:31 |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Default some needed variables https://review.openstack.org/425859 | 19:31 |
| openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart-extras: Support uploading files in modify image role https://review.openstack.org/425472 | 19:31 |
| openstackgerrit | Dan Radez proposed openstack/tripleo-heat-templates: Adding Tacker Support https://review.openstack.org/424135 | 19:32 |
| openstackgerrit | Rajini Ram proposed openstack/tripleo-heat-templates: Composable services support for Cinder Eqlx https://review.openstack.org/425861 | 19:32 |
| *** yolanda has joined #tripleo | 19:34 | |
| *** jkilpatr has quit IRC | 19:35 | |
| *** florianf has quit IRC | 19:38 | |
| *** sshnaidm is now known as sshnaidm|off | 19:41 | |
| openstackgerrit | Michael Henkel proposed openstack/tripleo-heat-templates: Re-organizes Contrail services to the correct roles https://review.openstack.org/425717 | 19:41 |
| *** suuuper has quit IRC | 19:43 | |
| *** david-lyle has joined #tripleo | 19:45 | |
| openstackgerrit | Alex Schultz proposed openstack/instack-undercloud: Run cell v2 setup differently for upgrades https://review.openstack.org/419121 | 19:45 |
| *** karimb has quit IRC | 19:45 | |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: Contrail services assigned to wrong roles https://review.openstack.org/425752 | 19:46 |
| openstackgerrit | Alex Schultz proposed openstack/instack-undercloud: Run cell v2 setup differently for upgrades https://review.openstack.org/419121 | 19:47 |
| *** nyechiel_ has quit IRC | 19:47 | |
| *** nyechiel_ has joined #tripleo | 19:47 | |
| openstackgerrit | Rajini Ram proposed openstack/tripleo-heat-templates: Composable service support for Cinder Dell Storage Center https://review.openstack.org/425866 | 19:48 |
| openstackgerrit | Rajini Ram proposed openstack/tripleo-heat-templates: Composable service support for Cinder Dell Storage Center https://review.openstack.org/425866 | 19:49 |
| openstackgerrit | Harry Rybacki proposed openstack/tripleo-quickstart-extras: Add optional Sphinx build verification https://review.openstack.org/425868 | 19:50 |
| *** akrivoka has quit IRC | 19:51 | |
| *** jkilpatr has joined #tripleo | 19:51 | |
| openstackgerrit | Ryan Brady proposed openstack/tripleo-common: Adds support to create deployment plan from git repository https://review.openstack.org/417280 | 19:53 |
| *** weshay is now known as weshay_brb | 19:55 | |
| *** Guest33539 is now known as mgagne | 20:01 | |
| *** mgagne has quit IRC | 20:01 | |
| *** mgagne has joined #tripleo | 20:01 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/tripleo-heat-templates: Remove create-legacy-resource-types opts https://review.openstack.org/424178 | 20:03 |
| openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart-extras: Support uploading files in modify image role https://review.openstack.org/425472 | 20:04 |
| openstackgerrit | Merged openstack/instack-undercloud: Deprecate instack-virt-setup https://review.openstack.org/425338 | 20:05 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Do not try to update the 'ceph' metapackage from CephMon role https://review.openstack.org/425659 | 20:05 |
| EmilienM | thrash: 419506 is not passing multinode (transient error) I'm rechecking | 20:06 |
| openstackgerrit | Emilien Macchi proposed openstack/python-tripleoclient: Add support for calling validations from introspection https://review.openstack.org/419506 | 20:06 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add telemetry service support for composable upgrades https://review.openstack.org/405249 | 20:06 |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Move multinode_major_upgrade.yaml to THT https://review.openstack.org/425694 | 20:07 |
| *** jlinkes has joined #tripleo | 20:07 | |
| *** amoralej is now known as amoralej|off | 20:07 | |
| openstackgerrit | James Slagle proposed openstack/tripleo-heat-templates: Add deployed server bootstrap for RHEL https://review.openstack.org/425879 | 20:07 |
| *** Goneri has joined #tripleo | 20:11 | |
| *** jtomasek has quit IRC | 20:16 | |
| dtrainor | bandini, thanks for following up re: failed gate-tripleo-ci-centos-7-ovb-* gates. i've taken a sick day, sorry for the delayed response. | 20:16 |
| *** ramishra has quit IRC | 20:19 | |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Use bootstrap subnodes environment in CI https://review.openstack.org/425888 | 20:20 |
| *** weshay_brb is now known as weshay | 20:20 | |
| openstackgerrit | Michele Baldessari proposed openstack/tripleo-common: WIP Add a Qdr Password for the Qpid dispatch router https://review.openstack.org/425890 | 20:21 |
| openstackgerrit | Michele Baldessari proposed openstack/puppet-tripleo: WIP Qpid dispatch router puppet profile https://review.openstack.org/425710 | 20:22 |
| slagle | jaosorior: can you also review https://review.openstack.org/#/c/425344/? since you reviewed the other deprecation part :) | 20:22 |
| openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates: WIP Qpid dispatch router composable role https://review.openstack.org/423326 | 20:22 |
| jaosorior | slagle: thanks, hadn't seen that one | 20:22 |
| slagle | jaosorior: thanks! | 20:23 |
| openstackgerrit | Merged openstack/tripleo-docs: Document instack-virt-setup deprecation https://review.openstack.org/425344 | 20:25 |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Use bootstrap subnodes environment in CI https://review.openstack.org/425888 | 20:26 |
| radez | anyone have a sec to lock at some logs with me? Im getting Resource CREATE failed: The Referenced Attribute (ControllerServiceChain role_data) is incorrect. in my tht patch | 20:26 |
| *** ramishra has joined #tripleo | 20:27 | |
| radez | https://review.openstack.org/#/c/424135/ | 20:27 |
| *** dsariel has quit IRC | 20:30 | |
| jaosorior | radez: damn dude, that's a hard one | 20:31 |
| radez | jaosorior: heh, I'm not sure that's what I wanted to hear :) | 20:32 |
| jaosorior | radez: well, I did find some details in your patch, but that's not what's gonna make it work :/ | 20:33 |
| jaosorior | haven't figured out what's the issue for that one | 20:33 |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Support composable HA for the Ceph rbdmirror daemon https://review.openstack.org/424573 | 20:33 |
| radez | jaosorior: comment away :) happy to make corrections | 20:33 |
| jaosorior | I did | 20:33 |
| jaosorior | ah | 20:34 |
| jaosorior | I think I found it | 20:34 |
| openstackgerrit | Rajini Ram proposed openstack/tripleo-heat-templates: Composable service support for Cinder Dell Storage Center https://review.openstack.org/425866 | 20:34 |
| jaosorior | radez: ^^ | 20:35 |
| radez | oh yea? | 20:35 |
| jaosorior | commented | 20:35 |
| radez | awesome, thx jaosorior! | 20:35 |
| jaosorior | radez: hopefully that makes it work | 20:35 |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: Contrail services assigned to wrong roles https://review.openstack.org/425752 | 20:36 |
| *** gfidente|afk has quit IRC | 20:36 | |
| openstackgerrit | Merged openstack/tripleo-docs: Update manila deployoment steps https://review.openstack.org/424577 | 20:38 |
| *** ansmith has quit IRC | 20:38 | |
| *** milan has joined #tripleo | 20:38 | |
| openstackgerrit | Dan Radez proposed openstack/tripleo-heat-templates: Adding Tacker Support https://review.openstack.org/424135 | 20:40 |
| *** fultonj has quit IRC | 20:40 | |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: [WIP] break out general_config into config, topology and provisioner https://review.openstack.org/425451 | 20:40 |
| radez | jaosorior: added a comment about haproxy, is there a different way I should be handling invoking the haproxy stuff? | 20:42 |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: [WIP] break out general_config into config, topology and provisioner https://review.openstack.org/425451 | 20:43 |
| jaosorior | radez not really, that should have been handled by your puppet code | 20:44 |
| *** dhill__ has joined #tripleo | 20:44 | |
| jaosorior | radez: so, for instance, here https://github.com/openstack/puppet-tripleo/blob/master/manifests/haproxy.pp#L550 | 20:45 |
| jaosorior | that's how it will be enabled by haproxy automatically | 20:45 |
| jaosorior | radez: basically what happens is that heat gets all the services, gets them in a list | 20:45 |
| jaosorior | and then automagically generates that hieradata | 20:45 |
| jaosorior | SERVICE_enabled | 20:46 |
| *** dhill_ has quit IRC | 20:46 | |
| jaosorior | radez: in your case, the service is named tacker (which is fine) | 20:46 |
| jaosorior | so there should be automatically an entry called tacker_enabled | 20:46 |
| jaosorior | which I saw you have already in haproxy.pp | 20:46 |
| *** dhill__ has quit IRC | 20:46 | |
| *** dhill_ has joined #tripleo | 20:46 | |
| radez | oh so I don't need to explicitly define that, I can take that line out and the tacker_enabled will get defined for me? | 20:47 |
| radez | amy Iunderstanding that correctly? | 20:47 |
| jaosorior | radez: yep | 20:47 |
| openstackgerrit | Rajini Ram proposed openstack/tripleo-heat-templates: Composable services support for Cinder Eqlx https://review.openstack.org/425861 | 20:47 |
| radez | *ack, ok that makes sense then, I'll pull out that line | 20:47 |
| jaosorior | radez: we should at some point create a "create a new service guide" | 20:48 |
| openstackgerrit | Dan Radez proposed openstack/tripleo-heat-templates: adding Congress Support https://review.openstack.org/330050 | 20:48 |
| jaosorior | cause honestly, it's not obvious | 20:49 |
| openstackgerrit | Rajini Ram proposed openstack/tripleo-heat-templates: Composable services support for Cinder Eqlx https://review.openstack.org/425861 | 20:49 |
| openstackgerrit | Dan Radez proposed openstack/tripleo-heat-templates: Adding Tacker Support https://review.openstack.org/424135 | 20:49 |
| EmilienM | thrash: http://logs.openstack.org/06/419506/7/check/gate-tripleo-ci-centos-7-nonha-multinode/6622dc3/console.html#_2017-01-26_20_48_20_480648 | 20:50 |
| openstackgerrit | Rajini Ram proposed openstack/tripleo-heat-templates: Composable service support for Cinder Dell Storage Center https://review.openstack.org/425866 | 20:51 |
| EmilienM | thrash: let me check what version of tripleo-common & client we had | 20:51 |
| EmilienM | thrash: ok a recheck will work | 20:53 |
| EmilienM | thrash: tripleo-common wasn't built with your patch yet | 20:53 |
| *** dougbtv_ has joined #tripleo | 20:53 | |
| *** dougbtv has quit IRC | 20:54 | |
| *** dougbtv_ is now known as dougbtv | 20:54 | |
| thrash | EmilienM: yeah.. That would make a diff. :) | 20:54 |
| openstackgerrit | John Trowbridge proposed openstack/tripleo-quickstart-extras: WIP Add build-images role https://review.openstack.org/414336 | 20:54 |
| EmilienM | thrash: yeah, kicking off ci jobs in a sex | 20:55 |
| EmilienM | in a sec* | 20:55 |
| thrash | EmilienM: Uhhhhh | 20:55 |
| thrash | lol | 20:55 |
| EmilienM | yeah in a sex too if you want | 20:55 |
| openstackgerrit | Emilien Macchi proposed openstack/python-tripleoclient: Add support for calling validations from introspection https://review.openstack.org/419506 | 20:55 |
| EmilienM | this lag is quite annoying but I don't know what to do | 20:56 |
| *** salmankhan has joined #tripleo | 20:57 | |
| *** ansmith has joined #tripleo | 20:57 | |
| openstackgerrit | Merged openstack/tripleo-quickstart-extras: [build-test-packages] combine all methods of providing changes https://review.openstack.org/424914 | 20:58 |
| trown | EmilienM: that is a sure sign you are working too hard :P | 20:58 |
| trown | (kicking off CI jobs in a sex) | 20:59 |
| EmilienM | trown: lol, what I did is terrible I know but trust my, x is near to c on keyboard | 21:00 |
| *** salmankhan has quit IRC | 21:01 | |
| *** eck` is now known as jeckersb_gone | 21:03 | |
| *** jcoufal has quit IRC | 21:03 | |
| *** jcoufal has joined #tripleo | 21:04 | |
| *** jcoufal has quit IRC | 21:14 | |
| *** pkovar has quit IRC | 21:17 | |
| *** jcoufal has joined #tripleo | 21:18 | |
| *** nyechiel_ has quit IRC | 21:19 | |
| openstackgerrit | Rajini Ram proposed openstack/tripleo-heat-templates: Composable service support for Cinder Dell EMC Storage Center https://review.openstack.org/425866 | 21:19 |
| *** liverpoo1er has quit IRC | 21:21 | |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Use bootstrap subnodes environment in CI https://review.openstack.org/425888 | 21:25 |
| openstackgerrit | Michael Henkel proposed openstack/tripleo-heat-templates: Re-organizes Contrail services to the correct roles https://review.openstack.org/425717 | 21:27 |
| *** dsariel has joined #tripleo | 21:29 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: nova: deploy basic setup for cells https://review.openstack.org/425848 | 21:30 |
| *** jcoufal has quit IRC | 21:30 | |
| EmilienM | bnemec, jaosorior: any review on https://review.openstack.org/#/c/425844/ is welcome, it will help to get promotion | 21:30 |
| EmilienM | same for https://review.openstack.org/#/c/425848/ | 21:31 |
| jaosorior | EmilienM: db-manage used in the underlying manifest idempotent? | 21:31 |
| EmilienM | jaosorior: yup sir, we test it in p-o-i and puppet-nova gates | 21:31 |
| jaosorior | nice! | 21:32 |
| jaosorior | EmilienM: and, does it matter if it's ran on every node? or does it only need to be the bootstrap node? | 21:32 |
| EmilienM | jaosorior: it needs to be run on one nova-api node at bootstrap | 21:33 |
| EmilienM | to avoid races | 21:33 |
| jaosorior | EmilienM: so, wouldn't this be an issue then https://review.openstack.org/#/c/425848/2/manifests/profile/base/nova/api.pp ? cause that will run fine in the bootstrap node, on step 3, but then on step 4 it will be ran again by the other nodes | 21:34 |
| EmilienM | jaosorior: not a big deal, since cells would already been setup | 21:35 |
| jaosorior | alrighty, if that's no biggie, then it's fine | 21:35 |
| *** jkilpatr has quit IRC | 21:35 | |
| *** crenguta has quit IRC | 21:36 | |
| openstackgerrit | Tim Rozet proposed openstack/tripleo-heat-templates: Enables OpenDaylight clustering in HA deployments https://review.openstack.org/425934 | 21:36 |
| jaosorior | EmilienM: could you check this out https://review.openstack.org/#/c/425267/ ? just finished the test run on the tls-everywhere job and it worked :D | 21:37 |
| openstackgerrit | Tim Rozet proposed openstack/puppet-tripleo: Enables OpenDaylight Clustering in HA deployments https://review.openstack.org/425383 | 21:37 |
| openstackgerrit | Harry Rybacki proposed openstack/tripleo-quickstart: Ensure gate jobs verify sphinx output https://review.openstack.org/425936 | 21:37 |
| radez | jaosorior: my new error says that it cannot find my puppet class, puppet-tacker has been packaged, is there something I've missed that isn't not included? | 21:37 |
| jaosorior | EmilienM: the flow is: 1. initial puppet change. 2. add needed bits to t-h-t. 3. clean up puppet. | 21:38 |
| radez | jaosorior: I have a puppet-tacker-distgit and I've taken under-review off it afaik | 21:38 |
| openstackgerrit | Rajini Ram proposed openstack/tripleo-heat-templates: Composable service support for Cinder Dell EMC Storage Center https://review.openstack.org/425866 | 21:38 |
| jaosorior | radez: is it part of OPM? | 21:38 |
| radez | jaosorior: does this make it part of OPM? https://review.rdoproject.org/r/#/c/4236/5/rdo.yml | 21:39 |
| radez | wasn't aware there was another step beyond that if there are any | 21:39 |
| jaosorior | radez: this https://github.com/rdo-packages/openstack-puppet-modules-distgit | 21:39 |
| EmilienM | jaosorior: ok i'll check in a few, let me clean my current pipe | 21:40 |
| radez | jaosorior: ah, k I've not done that, will put it in now. thx | 21:40 |
| *** thrash is now known as thrash|brb | 21:40 | |
| *** ansmith has quit IRC | 21:42 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: horizon: be more flexible in hiera neutron https://review.openstack.org/425941 | 21:47 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Allow to separate Horizon from Neutron https://review.openstack.org/425942 | 21:51 |
| *** milan has quit IRC | 21:51 | |
| *** jayg is now known as jayg|g0n3 | 21:53 | |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: Contrail services assigned to wrong roles https://review.openstack.org/425752 | 21:53 |
| *** rhallisey has quit IRC | 21:56 | |
| *** milan has joined #tripleo | 21:56 | |
| *** rhallisey has joined #tripleo | 21:56 | |
| *** rhallisey has quit IRC | 21:57 | |
| *** jeckersb_gone is now known as eck` | 21:57 | |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: Support builds in multinode https://review.openstack.org/425474 | 21:57 |
| *** jmelvin has quit IRC | 21:57 | |
| EmilienM | slagle: I think https://review.openstack.org/#/c/425222 can be merged safely, isn't? | 22:00 |
| EmilienM | slagle: the ovb-ha error is unrelated and it passed other ovb jobs, I think it's ok | 22:00 |
| bnemec | EmilienM: Yeah, I was debating the same thing. | 22:01 |
| EmilienM | bnemec: thank you for https://review.openstack.org/#/c/424857/1/goals/pike/python35.rst | 22:01 |
| bnemec | Clearly the undercloud network configured correctly since the nodes got deployed. | 22:02 |
| EmilienM | bnemec: really helpful | 22:02 |
| *** trown is now known as trown|outtypewww | 22:02 | |
| EmilienM | bnemec: I'm more worried by the fact we don't have functional tests against python3.5 yet, or do we? | 22:02 |
| bnemec | EmilienM: I don't know how we would. Does centos 7 even have python 3.5? | 22:03 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: DONT REVIEW: testing build multi https://review.openstack.org/425793 | 22:03 |
| EmilienM | that's part of the Community Goal: having functional tests | 22:03 |
| EmilienM | the unit tests part is actually optional :/ | 22:03 |
| bnemec | EmilienM: No package python3 available. | 22:05 |
| bnemec | I hope there's a python SIG or something publishing py 3.5 for centos. | 22:05 |
| bnemec | It isn't available with our current repo setup. | 22:05 |
| *** milan has quit IRC | 22:06 | |
| *** jlinkes has quit IRC | 22:08 | |
| *** jlinkes has joined #tripleo | 22:12 | |
| *** jkilpatr has joined #tripleo | 22:13 | |
| *** dprince has quit IRC | 22:15 | |
| *** milan has joined #tripleo | 22:16 | |
| *** cdearborn has joined #tripleo | 22:20 | |
| *** mhenkel has quit IRC | 22:21 | |
| *** mhenkel has joined #tripleo | 22:21 | |
| *** milan has quit IRC | 22:21 | |
| openstackgerrit | Dan Prince proposed openstack/tripleo-puppet-elements: Fail fast and warn if old hiera data is detected https://review.openstack.org/425955 | 22:25 |
| openstackgerrit | Merged openstack/tripleo-common: Add utility functions for deleting/emptying swift containers https://review.openstack.org/409315 | 22:25 |
| openstackgerrit | Merged openstack/puppet-tripleo: Adding congress service https://review.openstack.org/413881 | 22:25 |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Do not use cache ipa image if we're gating tripleo-image-elements https://review.openstack.org/425626 | 22:26 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add novajoin entries to the TLS-everywhere environment file https://review.openstack.org/421575 | 22:27 |
| *** thrash|brb is now known as thrahs | 22:29 | |
| *** thrahs is now known as thrash | 22:29 | |
| thrash | haha | 22:29 |
| thrash | at least I didn't "trash" myself. | 22:29 |
| *** lblanchard has quit IRC | 22:29 | |
| *** jlinkes has quit IRC | 22:32 | |
| *** rbrady is now known as rbrady-afk | 22:34 | |
| *** milan has joined #tripleo | 22:35 | |
| *** [1]cdearborn has quit IRC | 22:35 | |
| *** mhenkel has quit IRC | 22:35 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci: DO NOT MERGE - testing TLS everywhere job (neutron) https://review.openstack.org/419529 | 22:38 |
| *** mhenkel has joined #tripleo | 22:38 | |
| EmilienM | jaosorior: +2 on the tls/neutron, lgtm | 22:41 |
| jaosorior | EmilienM: thanks dude | 22:42 |
| jaosorior | EmilienM: if you have time, this is the t-h-t patch https://review.openstack.org/#/c/425317/2 and this cleans up the puppet bits https://review.openstack.org/#/c/425320/ | 22:43 |
| jaosorior | EmilienM: I currently -W the clean up cause I'm running the TLS-everywhere test on it | 22:44 |
| EmilienM | dude you're the TLS master | 22:45 |
| jaosorior | EmilienM: yay, thanks for checking it out dude :D | 22:45 |
| *** owalsh_ has joined #tripleo | 22:46 | |
| *** owalsh has quit IRC | 22:46 | |
| jaosorior | beagles: hey dude, if you have time could you check this out https://review.openstack.org/#/c/425267/ ? sorry to have poked you today a lot O_O | 22:47 |
| *** jlinkes has joined #tripleo | 22:48 | |
| beagles | jaosorior: np | 22:49 |
| *** pradk has quit IRC | 22:50 | |
| jaosorior | beagles: is octavia gonna be terminated by haproxy? | 22:57 |
| beagles | jaosorior: I don't think much testing (if any) has happened with Octavia in HA environments | 23:00 |
| beagles | jaosorior: I'll have to poke and prod about the state of affairs there. | 23:00 |
| *** jlinkes has quit IRC | 23:01 | |
| fpan | dsneddon: hey, could you take a look at https://review.openstack.org/#/c/424426/ | 23:02 |
| openstackgerrit | Merged openstack/diskimage-builder: Use %i instead of %I in dhcp-interface@.service https://review.openstack.org/409947 | 23:02 |
| *** saneax-_-|AFK is now known as saneax | 23:02 | |
| beagles | jaosorior: if I'm reading question correctly that is. | 23:04 |
| jaosorior | beagles: ok | 23:05 |
| jaosorior | beagles: if you have time today, these are the two remaining patches for TLS in the internal network for neutron https://review.openstack.org/#/c/425317/2 (t-h-t) and https://review.openstack.org/#/c/425320/ (puppet-tripleo) | 23:07 |
| jaosorior | beagles: the puppet patch is with -workflow because I'm running the TLS-everywhere test currently on it, just to make sure (I already ran it on the patch you reviewed and it worked) | 23:07 |
| openstackgerrit | Merged openstack/puppet-tripleo: Support composable HA for the Ceph rbdmirror daemon https://review.openstack.org/424573 | 23:09 |
| beagles | jaosorior: ack! | 23:12 |
| jaosorior | beagles: dude, thanks! | 23:12 |
| beagles | jaosorior: np! | 23:12 |
| *** absubram has quit IRC | 23:13 | |
| *** idlemind has quit IRC | 23:13 | |
| *** pmannidi has joined #tripleo | 23:14 | |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: [WIP] break out general_config into config, topology and provisioner https://review.openstack.org/425451 | 23:16 |
| beagles | was just thinking of those recent issues with neutron bigswitch plugin and cisco (?). The issues were db-sync things. I'm not sure I understand what is firing off the db-sync commands. Those services wouldn't have been enabled in tripleo, so I'm not sure why they would get triggered | 23:17 |
| jaosorior | beagles: undercloud or overcloud? | 23:19 |
| beagles | jaosorior: good question. overcloud I think | 23:19 |
| jaosorior | beagles: ok, in the overcloud, as far as I've seen, we do db-sync always; around step 3 | 23:19 |
| jaosorior | on the bootstrap node | 23:19 |
| jaosorior | beagles: ok, for neutron it seems to be step 4 | 23:20 |
| jaosorior | beagles: so this gets triggered https://github.com/openstack/puppet-tripleo/blob/master/manifests/profile/base/neutron/server.pp#L81 | 23:21 |
| jaosorior | which ends up here https://github.com/openstack/puppet-neutron/blob/master/manifests/server.pp#L332 | 23:21 |
| *** panda is now known as panda|Zz | 23:21 | |
| beagles | jaosorior: sure. it's that these third party things ... must be an alembic thing | 23:21 |
| jaosorior | oh | 23:21 |
| jaosorior | right, that would be an issue with the neutron-db-sync call | 23:21 |
| jaosorior | well | 23:21 |
| jaosorior | neutron-db-manage | 23:22 |
| jaosorior | which, as far as I've seen, runs for pretty much everything | 23:22 |
| jaosorior | (even the lbaas tables) | 23:22 |
| beagles | jaosorior: I've never really gave it much though but it looks like it reaches out and does more than you might expect | 23:22 |
| jaosorior | beagles: I would have thought that it runs depending on what's enabled on the config, but yeah, seems it just takes everything | 23:22 |
| jaosorior | beagles: haven't dug into that code much though | 23:22 |
| beagles | jaosorior: me neither .. might take a look. I noticed the lbaas thing earlier this week... I was going through my list of octavia stuff and was like "hey wait, I needed to sync the lbaas db in, how is this even starting" | 23:23 |
| beagles | jaosorior: and there it was | 23:24 |
| *** ipsecguy has quit IRC | 23:24 | |
| jaosorior | beagles: seems that it works with python endpoints | 23:24 |
| jaosorior | beagles: so if the package that ends up setting those endpoints is installed, it will run the database migration for it | 23:25 |
| *** idlemind has joined #tripleo | 23:25 | |
| jaosorior | beagles: https://github.com/openstack/neutron/blob/master/neutron/db/migration/cli.py#L53 | 23:26 |
| jaosorior | beagles: we could try to limit that list by explicitly setting what we want in the config https://github.com/openstack/neutron/blob/master/neutron/db/migration/cli.py#L66 | 23:26 |
| beagles | jaosorior: yup was already wit ya | 23:27 |
| jaosorior | beagles: cause I guess we'll need that for octavia anyway | 23:27 |
| * beagles nods | 23:28 | |
| jaosorior | beagles: it can get a bit tricky to figure out what we need though :/ | 23:28 |
| beagles | indeed | 23:30 |
| jaosorior | beagles: well... you could bake the logic in the puppet manifest; and if the octavia service is enabled (which you could know since the SERVICE_enabled entries are in all the nodes) you could delete lbaas entry from the list of subprojects to do the migration for https://forge.puppet.com/puppetlabs/stdlib#delete | 23:34 |
| jaosorior | and I guess we would need that for every case where we need to enable/disable something from there | 23:34 |
| beagles | jaosorior: oh I need the lbaas db :) it works in my favor for now | 23:34 |
| jaosorior | aaah, ok, thought octavia would need to handle that by itself | 23:35 |
| beagles | jaosorior: it's just neutron plugin stuff really | 23:35 |
| beagles | jaosorior: mmm... I better double check anyways | 23:35 |
| *** ipsecguy has joined #tripleo | 23:35 | |
| jaosorior | my knowledge of octavia is quite poor | 23:36 |
| beagles | jaosorior: the install docs some of the guys gave me have it in there with an explicit direction to populate those tables so I think I'm good | 23:36 |
| beagles | jaosorior: same here really it's kind of new on me. I'm just at the "this is how it is configured" and "this is how it is supposed to work"... by and by I'll no doubt end up digging through more of the code | 23:37 |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: fixup https://review.openstack.org/425975 | 23:38 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Add docker zaqar service https://review.openstack.org/425976 | 23:38 |
| *** noama has quit IRC | 23:42 | |
| *** bfournie has quit IRC | 23:42 | |
| *** noama has joined #tripleo | 23:43 | |
| openstackgerrit | Michael Henkel proposed openstack/tripleo-heat-templates: Re-organizes Contrail services to the correct roles https://review.openstack.org/425717 | 23:45 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: Support builds in multinode https://review.openstack.org/425474 | 23:47 |
| *** iranzo has quit IRC | 23:48 | |
| *** noama has quit IRC | 23:48 | |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: DONT REVIEW: testing build multi https://review.openstack.org/425793 | 23:48 |
| openstackgerrit | Jason E. Rist proposed openstack/tripleo-ui: Add release notes for Ocata 3.0.0 https://review.openstack.org/425979 | 23:49 |
| jrist | EmilienM: ^ | 23:49 |
| *** noama has joined #tripleo | 23:49 | |
| openstackgerrit | Jason E. Rist proposed openstack/tripleo-ui: Add release notes for Ocata 3.0.0 https://review.openstack.org/425979 | 23:50 |
| *** david-lyle has quit IRC | 23:51 | |
| *** jaosorior has quit IRC | 23:53 | |
| *** david-lyle has joined #tripleo | 23:54 | |
| *** rcernin has quit IRC | 23:56 | |
| *** rcernin has joined #tripleo | 23:57 | |
| openstackgerrit | Dan Sneddon proposed openstack/tripleo-specs: Spec: tripleo-routed-networks-deployment - Routed Spine and Leaf https://review.openstack.org/421009 | 23:58 |
| openstackgerrit | Dan Sneddon proposed openstack/tripleo-specs: Spec: TripleO Ironic Inspector changes to support routed networks https://review.openstack.org/421011 | 23:59 |
| *** rcernin has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!