| openstackgerrit | RedHat RDO CI proposed openstack/tripleo-heat-templates stable/queens: GATE CHECK for TripleO https://review.openstack.org/567224 | 00:00 |
|---|---|---|
| openstackgerrit | RedHat RDO CI proposed openstack/tripleo-quickstart-extras master: GATE CHECK for quickstart-extras https://review.openstack.org/560445 | 00:00 |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/newton: Use str_replace for known_hosts https://review.openstack.org/569912 | 00:00 |
| *** zaneb has joined #tripleo | 00:02 | |
| *** ffiore has quit IRC | 00:04 | |
| *** fragatina has quit IRC | 00:07 | |
| *** bdodd is now known as bdodd|afk | 00:07 | |
| *** fragatina has joined #tripleo | 00:07 | |
| *** rlandy is now known as rlandy|bbl | 00:09 | |
| *** dmacpher has quit IRC | 00:09 | |
| *** fragatina has quit IRC | 00:12 | |
| openstackgerrit | Merged openstack/tripleo-common stable/queens: Fix redis healthcheck https://review.openstack.org/569853 | 00:18 |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/ocata: Fix known hosts config script value https://review.openstack.org/570030 | 00:18 |
| *** mvenesio has quit IRC | 00:21 | |
| *** slaweq has joined #tripleo | 00:25 | |
| *** mmethot has quit IRC | 00:26 | |
| *** mmethot_ has joined #tripleo | 00:26 | |
| *** zshi has quit IRC | 00:27 | |
| openstackgerrit | John Fulton proposed openstack/tripleo-heat-templates master: Set default application for Ceph Luminous openstack_pools https://review.openstack.org/570043 | 00:27 |
| *** mmethot_ has quit IRC | 00:28 | |
| *** zaneb has quit IRC | 00:30 | |
| *** jtomasek has joined #tripleo | 00:33 | |
| *** d0ugal_ has quit IRC | 00:33 | |
| *** d0ugal_ has joined #tripleo | 00:40 | |
| *** yamahata has joined #tripleo | 00:41 | |
| *** spsurya has quit IRC | 00:41 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/pike: Use str_replace for known_hosts https://review.openstack.org/569910 | 00:44 |
| openstackgerrit | Merged openstack/tripleo-upgrade stable/queens: Separate scripts creation tasks for under/overcloud. https://review.openstack.org/569974 | 00:44 |
| *** bnemec has joined #tripleo | 00:47 | |
| *** slaweq_ has joined #tripleo | 00:54 | |
| *** bdodd|afk is now known as bdodd | 00:56 | |
| openstackgerrit | John Trowbridge proposed openstack/tripleo-quickstart-extras master: Add ability to update containers https://review.openstack.org/566877 | 00:57 |
| *** pchavva has quit IRC | 00:57 | |
| *** slaweq has quit IRC | 00:58 | |
| openstackgerrit | John Trowbridge proposed openstack-infra/tripleo-ci master: Add dry run option to toci_quickstart https://review.openstack.org/567060 | 01:00 |
| *** mdnadeem has joined #tripleo | 01:03 | |
| *** yamahata has quit IRC | 01:06 | |
| *** dmacpher has joined #tripleo | 01:08 | |
| *** bdodd is now known as bdodd|afk | 01:13 | |
| *** pchavva has joined #tripleo | 01:15 | |
| *** zaneb has joined #tripleo | 01:15 | |
| *** psahoo has joined #tripleo | 01:21 | |
| *** gbarros has joined #tripleo | 01:23 | |
| *** bnemec has quit IRC | 01:25 | |
| *** gvrangan has joined #tripleo | 01:25 | |
| *** slaweq_ has quit IRC | 01:29 | |
| openstackgerrit | Steve Baker proposed openstack/ansible-role-container-registry master: Use relative paths for data_files https://review.openstack.org/570088 | 01:30 |
| stevebaker | EmilienM: hey, is anything invoking the container-registry role before the undercloud install? | 01:33 |
| openstackgerrit | Marius Cornea proposed openstack/tripleo-upgrade stable/queens: DNM: Run enable-ssh-admin.sh script before fast forward run https://review.openstack.org/570074 | 01:36 |
| EmilienM | stevebaker: no | 01:42 |
| EmilienM | stevebaker: I mean... "not yet" ;-) | 01:42 |
| stevebaker | EmilienM: I'm thinking that oooq should install and run a playbook which looks like the tasks in tht | 01:43 |
| *** gvrangan has quit IRC | 01:44 | |
| EmilienM | stevebaker: why oooq | 01:44 |
| EmilienM | ? | 01:44 |
| stevebaker | EmilienM: where else would the playbook come from when only packages are installed? | 01:45 |
| EmilienM | stevebaker: tripleoclient? | 01:46 |
| EmilienM | is the playbook static? | 01:47 |
| EmilienM | if default values work, we can even put the playbook in the role | 01:47 |
| stevebaker | EmilienM: static with a couple of variables | 01:47 |
| *** pchavva has quit IRC | 01:47 | |
| stevebaker | EmilienM: tripleo-common playbooks is another option | 01:47 |
| EmilienM | as well | 01:48 |
| EmilienM | but I really don't want to rely on oooq for that | 01:48 |
| EmilienM | because CI/dev isn't the only use case | 01:48 |
| *** mdnadeem has quit IRC | 01:48 | |
| EmilienM | our operators who need to change containers before a deployment don't want oooq I presume | 01:48 |
| stevebaker | EmilienM: on the other hand, the playbook is literally just "set variables, run role container-registry" | 01:49 |
| EmilienM | stevebaker: yeah we can pick the client or common | 01:49 |
| EmilienM | we can start with client | 01:49 |
| stevebaker | since there is playbooks in common, I'll put it there. | 01:49 |
| EmilienM | ok | 01:50 |
| EmilienM | stevebaker: hey can you review this one when you have some spare time this week? https://review.openstack.org/#/c/570076/ to fix the undercloud upgrade CI job | 01:51 |
| EmilienM | thanks | 01:51 |
| stevebaker | EmilienM: sure. Next question would be what oooq task runs this playbook? undercloud-prep-containers.sh? | 01:51 |
| stevebaker | even before that might be better | 01:52 |
| EmilienM | stevebaker: IMHO the playbook should be run by tripleoclient itself | 01:52 |
| EmilienM | maybe a new command like "openstack tripleo container init" ? | 01:52 |
| EmilienM | the init would be run by default in the deploy | 01:53 |
| stevebaker | EmilienM: oh, lets just make it an embedded playbook in tripleoclient then | 01:53 |
| EmilienM | but could be run beforehand the deploy | 01:53 |
| EmilienM | yes! | 01:53 |
| stevebaker | gotcha | 01:53 |
| EmilienM | the "openstack tripleo deploy" would run "openstack tripleo init" by default | 01:53 |
| EmilienM | but init could be run beforehand | 01:53 |
| stevebaker | yep, ok | 01:53 |
| EmilienM | and init would be a set of playbooks we want before the actual deployment | 01:53 |
| EmilienM | like, deploy docker | 01:54 |
| EmilienM | it's an idea worth exploring | 01:54 |
| stevebaker | EmilienM: if we had an init, it will inevitably be used for other things ;) | 01:55 |
| openstackgerrit | Marius Cornea proposed openstack/tripleo-quickstart-extras master: DNM: export OVERCLOUD_HOSTS when running enable-ssh-admin.sh https://review.openstack.org/570091 | 01:56 |
| EmilienM | stevebaker: we would have "init, deploy, upgrade" | 01:58 |
| EmilienM | it dosn't sound crazy | 01:58 |
| EmilienM | having a "pre-init" would be crazy :D | 01:59 |
| *** d0ugal_ has quit IRC | 01:59 | |
| openstackgerrit | Marius Cornea proposed openstack/tripleo-upgrade stable/queens: DNM: Run enable-ssh-admin.sh script before fast forward run https://review.openstack.org/570074 | 02:01 |
| stevebaker | EmilienM: pre-init is yum install python-tripleoclient. maybe an init can help it not install every package | 02:03 |
| stevebaker | EmilienM: maybe an init can help the rpm not have to install every package | 02:03 |
| *** shreshtha has joined #tripleo | 02:04 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Remove UpgradeRemoveUnusedPackages from undercloud-upgrade-prepare.yaml https://review.openstack.org/569323 | 02:06 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates stable/ocata: Don't use crudini to get rhn server https://review.openstack.org/569180 | 02:07 |
| *** pradk has joined #tripleo | 02:07 | |
| EmilienM | stevebaker: let's propose something basic with an init | 02:07 |
| EmilienM | stevebaker: btw, https://review.openstack.org/#/c/568715/ if you have time as well :D | 02:07 |
| stevebaker | EmilienM: +1 | 02:07 |
| EmilienM | I'll stop with asking for review today :D | 02:07 |
| EmilienM | and go finish my slides for tripleo project updates, tomorrow | 02:08 |
| *** bkopilov has quit IRC | 02:08 | |
| *** d0ugal_ has joined #tripleo | 02:08 | |
| stevebaker | good luck | 02:08 |
| *** gvrangan has joined #tripleo | 02:11 | |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: WIP: native zuulv3 tests https://review.openstack.org/566779 | 02:11 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Save and close stdout on exit https://review.openstack.org/570094 | 02:11 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Add log directory option to functional tests https://review.openstack.org/570095 | 02:11 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Rename output log files https://review.openstack.org/570096 | 02:11 |
| EmilienM | thanks! | 02:12 |
| *** tvignaud has quit IRC | 02:17 | |
| *** tvignaud has joined #tripleo | 02:18 | |
| *** rh-jelabarre has quit IRC | 02:20 | |
| *** gbarros has quit IRC | 02:30 | |
| openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-common master: DNM: test container-check patch https://review.openstack.org/570101 | 02:38 |
| *** nyechiel_ has quit IRC | 02:46 | |
| *** mdnadeem has joined #tripleo | 02:48 | |
| *** mdnadeem has quit IRC | 02:52 | |
| *** fragatina has joined #tripleo | 02:56 | |
| *** gvrangan has quit IRC | 02:56 | |
| *** gvrangan has joined #tripleo | 02:58 | |
| *** fragatina has quit IRC | 02:59 | |
| *** fragatina has joined #tripleo | 02:59 | |
| *** fragatina has quit IRC | 03:04 | |
| *** fragatina has joined #tripleo | 03:04 | |
| *** shreshtha has quit IRC | 03:08 | |
| *** mdnadeem has joined #tripleo | 03:09 | |
| *** bkopilov has joined #tripleo | 03:25 | |
| *** gvrangan has quit IRC | 03:29 | |
| *** bdodd|afk is now known as bdodd | 03:32 | |
| *** jtomasek has quit IRC | 03:39 | |
| *** ramishra has joined #tripleo | 03:41 | |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Rename output log files https://review.openstack.org/570096 | 03:54 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: WIP: native zuulv3 tests https://review.openstack.org/566779 | 03:54 |
| *** gvrangan has joined #tripleo | 03:55 | |
| *** gvrangan has quit IRC | 03:57 | |
| *** matbu has joined #tripleo | 04:03 | |
| *** tzumainn has quit IRC | 04:04 | |
| *** saneax-_-|AFK has quit IRC | 04:08 | |
| *** bdodd is now known as bdodd|afk | 04:12 | |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: WIP: native zuulv3 tests https://review.openstack.org/566779 | 04:14 |
| *** gvrangan has joined #tripleo | 04:14 | |
| *** shreshtha has joined #tripleo | 04:14 | |
| *** gvrangan has quit IRC | 04:19 | |
| *** yprokule has joined #tripleo | 04:19 | |
| *** dxiri has quit IRC | 04:20 | |
| *** cshastri has joined #tripleo | 04:28 | |
| *** dxiri has joined #tripleo | 04:31 | |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Save and close stdout on exit https://review.openstack.org/570094 | 04:33 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Add log directory option to functional tests https://review.openstack.org/570095 | 04:33 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Rename output log files https://review.openstack.org/570096 | 04:33 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: WIP: native zuulv3 tests https://review.openstack.org/566779 | 04:33 |
| *** udesale has joined #tripleo | 04:35 | |
| *** links has joined #tripleo | 04:38 | |
| *** gvrangan has joined #tripleo | 04:39 | |
| *** yprokule has quit IRC | 04:40 | |
| *** dxiri has quit IRC | 04:40 | |
| *** marios has joined #tripleo | 04:41 | |
| *** bdodd|afk is now known as bdodd | 04:42 | |
| *** jaganathan has quit IRC | 04:45 | |
| *** jaganathan has joined #tripleo | 04:45 | |
| *** ykarel has joined #tripleo | 04:47 | |
| *** gvrangan has quit IRC | 04:51 | |
| *** ratailor has joined #tripleo | 04:54 | |
| *** radeks has joined #tripleo | 04:58 | |
| *** tdasilva has quit IRC | 04:58 | |
| openstackgerrit | Merged openstack/python-tripleoclient master: Add -y option to "openstack undercloud upgrade" https://review.openstack.org/570076 | 04:59 |
| *** janki has joined #tripleo | 05:06 | |
| openstackgerrit | Cédric Jeanneret proposed openstack/tripleo-common master: WIP: Set FixedIps by default https://review.openstack.org/569818 | 05:06 |
| *** radeks has quit IRC | 05:08 | |
| *** gvrangan has joined #tripleo | 05:09 | |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: WIP: native zuulv3 tests https://review.openstack.org/566779 | 05:10 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: WIP: native zuulv3 tests https://review.openstack.org/566779 | 05:13 |
| *** gvrangan has quit IRC | 05:14 | |
| *** bdodd is now known as bdodd|afk | 05:15 | |
| *** pdeore has joined #tripleo | 05:16 | |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: WIP: native zuulv3 tests https://review.openstack.org/566779 | 05:17 |
| *** jfrancoa has joined #tripleo | 05:20 | |
| *** nyechiel_ has joined #tripleo | 05:23 | |
| *** radeks has joined #tripleo | 05:24 | |
| *** psachin has joined #tripleo | 05:26 | |
| *** tdasilva has joined #tripleo | 05:27 | |
| *** yamahata has joined #tripleo | 05:31 | |
| *** ratailor has quit IRC | 05:36 | |
| *** ratailor has joined #tripleo | 05:36 | |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: WIP: native zuulv3 tests https://review.openstack.org/566779 | 05:37 |
| *** dparkes has quit IRC | 05:39 | |
| *** gvrangan has joined #tripleo | 05:44 | |
| *** gbarros has joined #tripleo | 05:46 | |
| *** ratailor has quit IRC | 05:50 | |
| *** yolanda has joined #tripleo | 05:50 | |
| *** nyechiel_ has quit IRC | 05:50 | |
| ramishra | containerized undercloud install fails with http://paste.openstack.org/show/721688/, for local CA, seems the certmonger service could not be restarted | 05:51 |
| ramishra | it goes away after system restart, probably a bug, jaosorior ^^^ | 05:51 |
| jaosorior | ramishra: any logs on why certmonger couldn't be restarted? | 05:52 |
| ramishra | looks like https://bugzilla.redhat.com/show_bug.cgi?id=1504688 | 05:52 |
| openstack | bugzilla.redhat.com bug 1504688 in dbus "certmonger is not able to set well-known bus name "org.fedorahosted.certmonger"" [Urgent,Closed: notabug] - Assigned to dking | 05:52 |
| jaosorior | ramishra: oh, that :/ | 05:53 |
| *** ratailor has joined #tripleo | 05:53 | |
| jaosorior | ramishra: can we try to open that again? so we could get a workaround at least :/ | 05:53 |
| jaosorior | ramishra: I'm not a certmonger dev :/ but I can try to poke that team | 05:53 |
| ramishra | jaosorior: we need a workaround in tripleo probably | 05:53 |
| ramishra | sure will raise a bug in tripleo atleast;) | 05:54 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: upgrade: remove packages at step 4 https://review.openstack.org/570116 | 05:56 |
| EmilienM | marios, hey I would like a review on ^ to make sure it's ok for you | 05:56 |
| EmilienM | thanks! | 05:56 |
| marios | EmilienM: i'll check. first thought is we lose some of the advantage because removal at step2 means we don't have to yum update them (i.e. yum update happens on step3 ) | 05:57 |
| marios | EmilienM: will have a closer look | 05:57 |
| marios | EmilienM: though having said that i don't think this feature is very widely used if at all (off by default) | 05:58 |
| EmilienM | marios: oh right | 05:59 |
| EmilienM | marios: but I actually did this patch because it breaks the upgrade when upgrading the undercloud | 05:59 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: WIP: native zuulv3 tests https://review.openstack.org/566779 | 05:59 |
| EmilienM | marios: see https://bugs.launchpad.net/tripleo/+bug/1772804 (which is another bug, but see the trace) | 05:59 |
| openstack | Launchpad bug 1772804 in tripleo "Upgrade process doesn't fail when upgrade_tasks returns 1 in containerized undercloud" [High,Triaged] - Assigned to Emilien Macchi (emilienm) | 05:59 |
| EmilienM | marios: the package was first removed, and then ansible tries to see if service runs (from another upgrade_task on another service) | 05:59 |
| ramishra | jaosorior: FWIW https://bugs.launchpad.net/tripleo/+bug/1772807 | 06:00 |
| openstack | Launchpad bug 1772807 in tripleo "default containerized undercloud with local CA fails with "Error org.freedesktop.DBus.Error.TimedOut"" [Undecided,New] | 06:00 |
| *** gbarros has quit IRC | 06:00 | |
| jaosorior | ramishra: thanks | 06:02 |
| EmilienM | marios: happy to chat later, /me goes to bed, have a good day ;-) | 06:02 |
| *** links has quit IRC | 06:04 | |
| *** shreshtha has quit IRC | 06:04 | |
| *** quiquell|off is now known as quiquell | 06:05 | |
| *** shreshtha has joined #tripleo | 06:05 | |
| *** yolanda has quit IRC | 06:05 | |
| *** paramite_ has joined #tripleo | 06:08 | |
| marios | EmilienM: ack good night :) | 06:09 |
| *** fragatina has quit IRC | 06:09 | |
| *** cshastri has quit IRC | 06:09 | |
| *** fragatina has joined #tripleo | 06:09 | |
| *** udesale_ has joined #tripleo | 06:16 | |
| *** yprokule has joined #tripleo | 06:16 | |
| *** ykarel_ has joined #tripleo | 06:17 | |
| *** udesale has quit IRC | 06:18 | |
| *** rlandy|bbl has quit IRC | 06:18 | |
| *** yolanda has joined #tripleo | 06:19 | |
| *** ykarel has quit IRC | 06:19 | |
| *** links has joined #tripleo | 06:19 | |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: WIP: native zuulv3 tests https://review.openstack.org/566779 | 06:21 |
| *** nkinder has joined #tripleo | 06:26 | |
| *** kopecmartin has joined #tripleo | 06:28 | |
| *** cylopez has joined #tripleo | 06:28 | |
| *** ramishra has quit IRC | 06:28 | |
| *** ramishra has joined #tripleo | 06:31 | |
| *** dparkes has joined #tripleo | 06:31 | |
| *** jbadiapa_ has quit IRC | 06:32 | |
| *** _jbadiapa has joined #tripleo | 06:32 | |
| *** pcaruana has joined #tripleo | 06:33 | |
| *** wolverineav has joined #tripleo | 06:34 | |
| *** cshastri has joined #tripleo | 06:36 | |
| *** lvdombrkr has joined #tripleo | 06:36 | |
| *** gfidente has joined #tripleo | 06:39 | |
| *** gfidente has quit IRC | 06:39 | |
| *** gfidente has joined #tripleo | 06:39 | |
| *** radek__ has joined #tripleo | 06:42 | |
| *** cshastri has quit IRC | 06:43 | |
| *** holser__ has joined #tripleo | 06:44 | |
| *** udesale__ has joined #tripleo | 06:44 | |
| *** yamahata has quit IRC | 06:44 | |
| *** udesale_ has quit IRC | 06:46 | |
| *** jchhatbar has joined #tripleo | 06:50 | |
| *** janki has quit IRC | 06:51 | |
| *** jchhatba_ has joined #tripleo | 06:52 | |
| *** udesale__ has quit IRC | 06:52 | |
| *** udesale__ has joined #tripleo | 06:52 | |
| *** ykarel__ has joined #tripleo | 06:53 | |
| *** rajinir has quit IRC | 06:53 | |
| *** shardy has joined #tripleo | 06:54 | |
| *** jchhatbar has quit IRC | 06:55 | |
| *** ffiore has joined #tripleo | 06:55 | |
| *** ykarel_ has quit IRC | 06:56 | |
| *** pmannidi has quit IRC | 06:57 | |
| *** threestrands has quit IRC | 06:57 | |
| *** rcernin has quit IRC | 06:57 | |
| *** dmacpher has quit IRC | 06:58 | |
| *** dmacpher has joined #tripleo | 06:59 | |
| *** cshastri has joined #tripleo | 07:01 | |
| *** ykarel__ is now known as ykarel|mtg | 07:01 | |
| *** pdeore_ has joined #tripleo | 07:01 | |
| *** udesale_ has joined #tripleo | 07:01 | |
| *** pdeore has quit IRC | 07:04 | |
| openstackgerrit | Numan Siddique proposed openstack/tripleo-quickstart master: Enable OVN DB HA profile in fset30 https://review.openstack.org/523804 | 07:04 |
| *** udesale__ has quit IRC | 07:04 | |
| *** dparkes has quit IRC | 07:05 | |
| *** dparkes has joined #tripleo | 07:06 | |
| *** saneax has joined #tripleo | 07:06 | |
| *** saneax has quit IRC | 07:08 | |
| *** saneax has joined #tripleo | 07:09 | |
| *** dparkes has quit IRC | 07:10 | |
| *** udesale_ has quit IRC | 07:11 | |
| *** udesale has joined #tripleo | 07:12 | |
| *** udesale has quit IRC | 07:12 | |
| *** udesale has joined #tripleo | 07:12 | |
| *** jaosorior has quit IRC | 07:18 | |
| *** jaosorior has joined #tripleo | 07:19 | |
| *** tesseract has joined #tripleo | 07:20 | |
| openstackgerrit | Yurii Prokulevych proposed openstack/tripleo-upgrade stable/queens: Move SSL undercloud validation out of UC script create tasks. https://review.openstack.org/570130 | 07:20 |
| *** ukalifon has joined #tripleo | 07:22 | |
| *** _jbadiapa has quit IRC | 07:23 | |
| *** dmacpher has quit IRC | 07:24 | |
| openstackgerrit | pini proposed openstack/tripleo-ha-utils master: switch to use the upstream utils repo https://review.openstack.org/570132 | 07:26 |
| *** ssbarnea_ has joined #tripleo | 07:27 | |
| *** bogdando has joined #tripleo | 07:30 | |
| *** florianf has joined #tripleo | 07:36 | |
| *** _jbadiapa has joined #tripleo | 07:38 | |
| openstackgerrit | Cédric Jeanneret proposed openstack/tripleo-common master: WIP: Set FixedIps by default https://review.openstack.org/569818 | 07:41 |
| *** anilvenkata has joined #tripleo | 07:41 | |
| *** amoralej|off is now known as amoralej | 07:42 | |
| openstackgerrit | Bogdan Dobrelya proposed openstack/python-tripleoclient master: Store UC hosts/portmaps heat params in output_dir https://review.openstack.org/564135 | 07:42 |
| *** olap has joined #tripleo | 07:42 | |
| bogdando | o/ PTAL https://review.openstack.org/#/c/563988/ | 07:43 |
| openstackgerrit | Bogdan Dobrelya proposed openstack/python-tripleoclient master: Provide the default roles file for standalone https://review.openstack.org/569992 | 07:49 |
| *** udesale_ has joined #tripleo | 07:51 | |
| openstackgerrit | Bogdan Dobrelya proposed openstack/python-tripleoclient master: Fallback the default roles file for UC config https://review.openstack.org/569992 | 07:51 |
| openstackgerrit | Sergii Golovatiuk proposed openstack/tripleo-common stable/queens: Add special 'all' for deploy_on_servers server name to match all https://review.openstack.org/567538 | 07:52 |
| *** pdeore__ has joined #tripleo | 07:52 | |
| *** udesale has quit IRC | 07:54 | |
| *** jpena|off is now known as jpena | 07:54 | |
| *** pdeore_ has quit IRC | 07:55 | |
| cgoncalves | shardy, jaosorior: morning :) re: octavia image rpm install, the only way around I found was to use Ansible's raw module "raw: sudo yum -y install octavia-amphora-image" and add that command to the tripleo-common/sudoers file | 07:55 |
| cgoncalves | shardy, jaosorior: installing the rpm in one of the controller nodes proved not be that feasible because a) node need to be subscribed to OSP channel which in many cases it is not and b) would require a default route to outside which is also unlikely to have | 07:57 |
| openstackgerrit | Thomas Herve proposed openstack/tripleo-heat-templates master: Trigger scenario003 on designate changes https://review.openstack.org/570139 | 07:59 |
| openstackgerrit | Thomas Herve proposed openstack/tripleo-heat-templates master: Pass designate configuration to neutron https://review.openstack.org/569990 | 08:00 |
| openstackgerrit | RedHat RDO CI proposed openstack/tripleo-heat-templates stable/ocata: GATE CHECK for TripleO https://review.openstack.org/564291 | 08:00 |
| openstackgerrit | RedHat RDO CI proposed openstack/tripleo-heat-templates stable/pike: GATE CHECK for TripleO https://review.openstack.org/564285 | 08:00 |
| *** gfidente has quit IRC | 08:01 | |
| *** links has quit IRC | 08:01 | |
| *** d0ugal_ has quit IRC | 08:08 | |
| *** d0ugal has joined #tripleo | 08:08 | |
| *** b00tcat has joined #tripleo | 08:11 | |
| *** marios has quit IRC | 08:13 | |
| *** avivgt has joined #tripleo | 08:13 | |
| *** links has joined #tripleo | 08:14 | |
| *** marios has joined #tripleo | 08:14 | |
| *** gkadam has joined #tripleo | 08:17 | |
| openstackgerrit | Cédric Jeanneret proposed openstack/tripleo-common master: WIP: Set FixedIps by default https://review.openstack.org/569818 | 08:26 |
| jaosorior | shardy: can you take a look at this? ^^ | 08:27 |
| jaosorior | we're still missing IPv6 (gonna work on that next) | 08:27 |
| jaosorior | but we got IPv4 working in our tests | 08:28 |
| Tengu | \o/ | 08:28 |
| openstackgerrit | Jose Luis Franco proposed openstack/tripleo-upgrade stable/queens: Move SSL undercloud validation out of UC script create tasks. https://review.openstack.org/570130 | 08:28 |
| jaosorior | shardy: IPv6 will come as part of the same patch. | 08:28 |
| *** gfidente has joined #tripleo | 08:30 | |
| *** psahoo has quit IRC | 08:31 | |
| *** pblaho has joined #tripleo | 08:31 | |
| *** suuuper has joined #tripleo | 08:35 | |
| *** ramishra_ has joined #tripleo | 08:35 | |
| *** ramishra has quit IRC | 08:35 | |
| quiquell | Hi, some core's love here ? | 08:38 |
| quiquell | https://review.openstack.org/#/c/566565/ | 08:38 |
| quiquell | It's part of a CI effort to refactor upgrades/updates | 08:38 |
| *** shardy has quit IRC | 08:39 | |
| *** ykarel|mtg is now known as ykarel | 08:39 | |
| chandankumar | sshnaidm|off: panda|off can we get this merged https://review.rdoproject.org/r/13851 | 08:39 |
| chandankumar | ? | 08:39 |
| *** akrivoka has joined #tripleo | 08:41 | |
| *** psahoo has joined #tripleo | 08:45 | |
| *** derekh has joined #tripleo | 08:50 | |
| *** skramaja has joined #tripleo | 08:50 | |
| *** sshnaidm|off is now known as sshnaidm|rover | 08:52 | |
| sshnaidm|rover | chandankumar, can you please remove 021 in the same patch? | 08:52 |
| chandankumar | sshnaidm|rover: sure | 08:53 |
| *** shardy has joined #tripleo | 08:53 | |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates master: Add update/upgrade/ffwd tasks for OsloMessagingRpc https://review.openstack.org/570148 | 09:02 |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates master: Add update/upgrade/ffwd tasks for OsloMessagingRpc https://review.openstack.org/570148 | 09:03 |
| *** pdeore__ is now known as pdeore | 09:06 | |
| *** leanderthal has joined #tripleo | 09:07 | |
| *** salmankhan has joined #tripleo | 09:09 | |
| openstackgerrit | Jiri Stransky proposed openstack-infra/tripleo-ci master: Add keystone only upgrade jobs https://review.openstack.org/558425 | 09:09 |
| *** dparkes has joined #tripleo | 09:13 | |
| *** psachin has quit IRC | 09:23 | |
| *** masco has joined #tripleo | 09:23 | |
| *** dparkes has quit IRC | 09:24 | |
| *** dparkes has joined #tripleo | 09:26 | |
| *** zoli is now known as zoli|lunch | 09:30 | |
| *** udesale__ has joined #tripleo | 09:31 | |
| *** yolanda has quit IRC | 09:32 | |
| *** udesale has joined #tripleo | 09:32 | |
| *** udesale_ has quit IRC | 09:33 | |
| *** salmankhan has quit IRC | 09:35 | |
| *** salmankhan has joined #tripleo | 09:35 | |
| *** nkinder has quit IRC | 09:35 | |
| *** udesale__ has quit IRC | 09:36 | |
| jistr | jfrancoa: the mini revert didn't work, it's too tied into other services, e.g. keystone expected it. So i'm trying a full fix instead, so far just took the tasks from rabbitmq: https://review.openstack.org/#/c/570148 | 09:38 |
| *** jchhatba_ has quit IRC | 09:40 | |
| *** jchhatba_ has joined #tripleo | 09:40 | |
| *** CristinaPauna has joined #tripleo | 09:42 | |
| *** yolanda has joined #tripleo | 09:45 | |
| *** cylopez has left #tripleo | 09:46 | |
| jchhatba_ | Hi. Can I get some reviews for https://review.openstack.org/#/c/567469/ | 09:49 |
| jchhatba_ | mandre, bogdando skramaja ^ | 09:49 |
| openstackgerrit | binhong.hua proposed openstack/tripleo-quickstart-extras master: Move --scale to {{role}}Count https://review.openstack.org/557573 | 09:51 |
| openstackgerrit | Gael Chamoulaud proposed openstack/tripleo-validations master: Add new nova-event-callback validation https://review.openstack.org/513333 | 09:52 |
| *** jchhatba_ is now known as janki | 09:52 | |
| skramaja | done janki | 09:58 |
| janki | skramaja, thanks | 10:01 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-docs master: Add instructions for Ceph upgrade in Pike to Queens section https://review.openstack.org/570156 | 10:01 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-docs master: Add instructions for Ceph upgrade in Pike to Queens section https://review.openstack.org/570156 | 10:03 |
| jaosorior | shardy: will folks always give the ExternalNetCidr if IPv6 is enabled? | 10:09 |
| *** bkopilov has quit IRC | 10:16 | |
| *** psahoo has quit IRC | 10:16 | |
| ccamacho | jaosorior quick question im missing network-environment.yaml in my undercloud | 10:17 |
| ccamacho | do you know how is generated by default? | 10:18 |
| *** dparkes has quit IRC | 10:18 | |
| ccamacho | jistr^ maybe ? | 10:18 |
| jaosorior | ccamacho: it's generated by tripleo-quickstart usually | 10:19 |
| jaosorior | else you gotta write it yourself | 10:19 |
| jaosorior | if you use tags all it'll be auto generated | 10:20 |
| jaosorior | if you don't, then it'll be missing | 10:20 |
| jaosorior | I usually copy mine from a CI run | 10:20 |
| ccamacho | jaosorior yeahp but I cant use all tags, I used --tags untagged,provision,environment,libvirt,undercloud-scripts,undercloud-inventory,overcloud-scripts | 10:21 |
| ccamacho | And now im running | 10:21 |
| ccamacho | sudo ./overcloud-prep-containers.sh | 10:21 |
| ccamacho | ./overcloud-prep-flavors.sh | 10:21 |
| ccamacho | ./overcloud-prep-images.sh | 10:21 |
| openstackgerrit | Arx Cruz proposed openstack/tripleo-quickstart master: Remove tempest compute-feature-enabled.attach_encrypted_volume https://review.openstack.org/570158 | 10:21 |
| ccamacho | ./overcloud-prep-network.sh | 10:21 |
| ccamacho | ./overcloud-deploy.sh ---- (fails with the missing network-environment.yaml file ) | 10:21 |
| ccamacho | ./overcloud-deploy-post.sh | 10:21 |
| ccamacho | ./overcloud-validate.sh | 10:21 |
| jaosorior | ccamacho: OK, then grab it from a CI run | 10:21 |
| jaosorior | ccamacho: I don't recall what tag is needed :/ | 10:22 |
| jaosorior | sshnaidm|rover: do you remember? ^^ | 10:22 |
| shardy | jaosorior: there will always be a value, but it might be the default from network_data.yaml via the heat default vs a user provided value | 10:22 |
| *** dparkes has joined #tripleo | 10:22 | |
| sshnaidm|rover | jaosorior, overcloud-prep-config? | 10:22 |
| jaosorior | ccamacho: I had tried to add it here https://review.openstack.org/#/c/567202/ | 10:22 |
| jaosorior | yes! | 10:23 |
| jaosorior | shardy: right. I'm trying to figure out how to detect if we'll use IPv6 | 10:23 |
| ccamacho | jaosorior!! woooow so your bug is verified then :P | 10:23 |
| ccamacho | thanks man | 10:23 |
| *** anilvenkata has quit IRC | 10:24 | |
| *** wolverineav has quit IRC | 10:24 | |
| sshnaidm|rover | jaosorior, maybe the solution could be tagging generation of network-environments.yaml with tag overcloud-scripts | 10:24 |
| shardy | jaosorior: if you look at the flattened parameters schema we already generate for the UI, I think you can look at either the Value or Default key for the ExternalNetCidr parameter and see if it's an ipv6 address | 10:24 |
| shardy | with the Value taking precedence over the Default | 10:25 |
| *** wolverineav has joined #tripleo | 10:25 | |
| jaosorior | uhm... | 10:25 |
| jaosorior | I don't think I have access to the heat defaults at that point | 10:25 |
| Tengu | we will find a way :). | 10:25 |
| jaosorior | only to the user-provided things, like the parameter_defaults and the resource_registry | 10:25 |
| openstackgerrit | Arx Cruz proposed openstack/tripleo-quickstart-extras master: Removing options in configure-tempest https://review.openstack.org/570159 | 10:25 |
| shardy | jaosorior: the heat defaults are in the flattened parameters schema I just mentioned | 10:26 |
| shardy | that's how the UI populates the defaults in all the forms | 10:26 |
| jaosorior | sshnaidm|rover: I think that would be nicer | 10:26 |
| ccamacho | jaosorior im going to grab what's here: https://github.com/openstack/tripleo-quickstart-extras/tree/master/roles/overcloud-prep-config for not doing it all again | 10:26 |
| shardy | jaosorior: yeah there are two things, the merged user environment and the parameters schema from validation | 10:26 |
| jaosorior | shardy: are those flattened parameters saved somewhere in swift? | 10:26 |
| openstackgerrit | Arx Cruz proposed openstack/tripleo-quickstart master: Remove tempest compute-feature-enabled.attach_encrypted_volume https://review.openstack.org/570158 | 10:27 |
| shardy | I think you need the second one (or perhaps both) | 10:27 |
| shardy | jaosorior: not sure, let me take a look | 10:27 |
| pdeore | Hi everyone, could you please review https://review.openstack.org/#/c/569970/ ? | 10:27 |
| openstackgerrit | Merged openstack/tripleo-ha-utils master: switch to use the upstream utils repo https://review.openstack.org/570132 | 10:27 |
| openstackgerrit | Merged openstack/tripleo-ha-utils master: Fix the way iptables rules are managed https://review.openstack.org/569993 | 10:27 |
| *** wolverineav has quit IRC | 10:29 | |
| shardy | https://github.com/openstack/tripleo-common/blob/master/workbooks/plan_management.yaml#L549 | 10:29 |
| shardy | jaosorior: I don't think it's stored although IIRC there's some caching somewhere, I guess the derive parameters model has access to this too, so maybe that's a good place to wire in your changes? | 10:30 |
| shardy | since you're essentially deriving the public VIP IP from the plan and user data? | 10:30 |
| jaosorior | shardy: but, that's not called at all for the CLI workflow, is it? | 10:31 |
| shardy | jaosorior: yes we should use the same workflows for UI and CLI now | 10:31 |
| shardy | https://github.com/openstack/tripleo-common/blob/master/workbooks/derive_params.yaml#L20 | 10:32 |
| *** psachin has joined #tripleo | 10:32 | |
| shardy | looks like we call the action there, not looked closely at the caching but we will want to make sure its not calling heat every time as it'a fairly expensive operation | 10:32 |
| jaosorior | shardy: figuring out where that's called. Cause right now we're calling all this stuff from the deploy workflow | 10:36 |
| jaosorior | I actually don't see an instance of derive_parameters or derive_params in tripleoclient :/ | 10:37 |
| ccamacho | sshnaidm|rover please check my comment for https://review.openstack.org/#/c/567202 (Abandoned a few days ago) | 10:37 |
| openstackgerrit | Carlos Goncalves proposed openstack/tripleo-common master: Fix Octavia amphora image RPM install https://review.openstack.org/570163 | 10:38 |
| shardy | jaosorior: it'll be called from another workflow I expect | 10:38 |
| sshnaidm|rover | ccamacho, solution could be tagging generation of network-environments.yaml with tag overcloud-scripts | 10:39 |
| cgoncalves | shardy, jaosorior: https://review.openstack.org/#/c/570163/ | 10:39 |
| jaosorior | shardy: right, just trying to tie everything together to make sure it all works as we expect. | 10:39 |
| *** udesale has quit IRC | 10:40 | |
| *** zoli|lunch is now known as zoli | 10:40 | |
| shardy | jaosorior: ack - tripleoclient has a test (test_invoke_plan_env_workflows) which shows it driving tripleo.derive_params.v1.derive_parameters | 10:41 |
| ccamacho | sshnaidm|rover yeah, but just a heads up as it might be a valid bug if either the file isn't generated with the tag overcloud-prep-config or overcloud-scripts | 10:41 |
| shardy | ./tripleoclient/workflows/parameters.py seems to call the workflow based on the plan-environment I think | 10:42 |
| jaosorior | at what point in the CLI deployment is that called? | 10:42 |
| shardy | maybe skramaja can confirm how that's intended to work as I don't see tripleo.derive_params.v1.derive_parameters in the default plan-environment, possibly you have to add it for DPDK deployments and others that need it? | 10:43 |
| openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart-extras master: WIP: tag creating network environment file with overcloud-scripts https://review.openstack.org/570164 | 10:43 |
| sshnaidm|rover | ccamacho, can you try please if it works ^^ | 10:43 |
| jaosorior | that might be the case | 10:44 |
| shardy | jaosorior: workflow_params.invoke_plan_env_workflows is run in ./tripleoclient/v1/overcloud_deploy.py right before we do the deploy | 10:44 |
| ccamacho | sshnaidm|rover sure thing :) /me testing it right now | 10:44 |
| shardy | so sounds like it might fit your use-case | 10:44 |
| rasca | hey folks, can I ask you for the last +2 on this https://review.openstack.org/#/c/563104/ since we're all good with the gates? | 10:45 |
| * skramaja reading throu shardy jaosorior | 10:48 | |
| jaosorior | shardy: OK, we could give it a try | 10:49 |
| skramaja | inorder to enable derive params workflow, we use -p option on the deploycommand to override the default plan environment with this one https://github.com/openstack/tripleo-heat-templates/blob/master/plan-samples/plan-environment-derived-params.yaml | 10:51 |
| skramaja | this will have the parameters required for starting the workflow. | 10:51 |
| skramaja | https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/overcloud_deploy.py#L213 | 10:52 |
| skramaja | jaosorior: shardy based on the -p option cli will invoke the workflow.. ^ here. | 10:53 |
| shardy | skramaja: thanks for confirming, sounds like jaosorior can do the same for testing and we can discuss enabling it by default if needed for his requirements | 10:55 |
| *** shreshtha has quit IRC | 10:55 | |
| jaosorior | skramaja: oh, but we need to pass a plan file then? | 10:56 |
| *** mdnadeem has quit IRC | 11:00 | |
| *** yprokule has quit IRC | 11:02 | |
| skramaja | jaosorior: yeah. our is specific to nfv deployments only not common to all. if it is common, we could add it to the default plan environtment file itself.. | 11:04 |
| jaosorior | I see | 11:04 |
| jaosorior | skramaja: will derive_parameters work even if we don't override the default plan? | 11:05 |
| shardy | jaosorior: No, check the tripleoclient code I linked, it only runs the workflow if it's specified in the plan-environment, so we'd potentially need to update the default plan-environment or it won't run | 11:06 |
| skramaja | derive parameters when invoked, will check the list of roles and their features, if nfv features is present, it will run the formulas to drive the parameters. | 11:06 |
| skramaja | we dont want to invoke by default as it will add additional step for non-nfv deployments too.. | 11:06 |
| skramaja | https://github.com/openstack/tripleo-common/blob/master/workbooks/derive_params.yaml#L432 | 11:07 |
| shardy | yeah we may need to revisit that decision if jaosorior needs this to run by default but we'll have to make sure the overhead is fairly small | 11:07 |
| skramaja | ^ here where the feature identification is done based on services and parameters. .. | 11:08 |
| *** shardy is now known as shardy_lunch | 11:08 | |
| skramaja | yes., we could always enable it by default | 11:08 |
| jaosorior | so... we're hooking this stuff now qa | 11:09 |
| skramaja | and there is a requirement for the control of using this feature should be with user.. that we can achieve by adding workflow_parameters in the plan environment.. | 11:09 |
| skramaja | so it is doable.. | 11:10 |
| jaosorior | * to the deploy workflow | 11:10 |
| openstackgerrit | Quique Llorente proposed openstack/tripleo-quickstart-extras master: Export ZUUL_CHANGES in the reproducer https://review.openstack.org/570167 | 11:14 |
| *** dtantsur has joined #tripleo | 11:18 | |
| *** dtantsur has quit IRC | 11:18 | |
| skramaja | jaosorior: yes, it needs few additional changes on the workflow, we can add it. your requirement is for testing vip? | 11:18 |
| *** dtantsur has joined #tripleo | 11:19 | |
| jaosorior | skramaja: I'm trying to do predictable VIPs (either the external one or the ctlplane one... depends on the case) by default | 11:21 |
| jaosorior | and that, because I'm ultimately trying to do TLS by default | 11:21 |
| jaosorior | so we need a predictable IP for the cert | 11:21 |
| *** bfournie has quit IRC | 11:21 | |
| skramaja | ok. derive parameters workflow is more of getting the introspection data to derive the parameteres to get the best performace.. | 11:25 |
| *** pdeore has quit IRC | 11:27 | |
| skramaja | what i see as your requirement is kind of static. may be we need add few customizations to remove unnessary bits out for your requiremtent. it has all the role specific node and parameter checking and deriving. | 11:27 |
| *** b00tcat has quit IRC | 11:28 | |
| *** rh-jelabarre has joined #tripleo | 11:29 | |
| skramaja | static here i mean is that it is not dependent on the hardware.. | 11:29 |
| bogdando | mwhahaha, EmilienM: is ready to restore wf+1 back https://review.openstack.org/#/c/564135/ :) | 11:29 |
| *** pcaruana|afk| has joined #tripleo | 11:29 | |
| *** pcaruana has quit IRC | 11:29 | |
| *** wolverineav has joined #tripleo | 11:29 | |
| *** salmankhan has quit IRC | 11:35 | |
| jaosorior | skramaja: right, it isn't | 11:35 |
| openstackgerrit | John Trowbridge proposed openstack-infra/tripleo-ci master: Add dry run option to toci_quickstart https://review.openstack.org/567060 | 11:35 |
| *** quiquell is now known as quiquell|lunch | 11:37 | |
| skramaja | let me know if you need any info, happy to help jaosorior. though most of the work is done by jaganathan and abisop on the workflows. | 11:38 |
| *** marios has quit IRC | 11:39 | |
| *** raildo has joined #tripleo | 11:41 | |
| *** salmankhan has joined #tripleo | 11:41 | |
| *** marios has joined #tripleo | 11:41 | |
| openstackgerrit | Sergii Golovatiuk proposed openstack/python-tripleoclient master: Replace logging with oslo_log for upgrade classes https://review.openstack.org/567819 | 11:45 |
| openstackgerrit | Bogdan Dobrelya proposed openstack/puppet-tripleo master: Rework GDPR compliant logrotate config https://review.openstack.org/569976 | 11:46 |
| *** abishop has joined #tripleo | 11:46 | |
| *** shreshtha has joined #tripleo | 11:48 | |
| *** shreshtha has quit IRC | 11:48 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-docs master: Add developer docs for Pike to Queens upgrade https://review.openstack.org/569443 | 11:48 |
| *** shreshtha has joined #tripleo | 11:48 | |
| *** dmacpher has joined #tripleo | 11:48 | |
| *** pkovar has joined #tripleo | 11:49 | |
| *** mburned_out is now known as mburned | 11:50 | |
| jaosorior | shardy_lunch: oooh, actually, we had potentially IPv6 working since we're taking the allocation pool into account, which should be ipv6 already | 11:52 |
| *** amoralej is now known as amoralej|lunch | 11:52 | |
| jaosorior | trying out an ipv6 deployment with the patch | 11:53 |
| jaosorior | bogdando: I think there's a containerized undercloud bug | 11:57 |
| jaosorior | bogdando: if you update an action/workflow from tripleo-common, it doesn't get built in CI | 11:57 |
| jaosorior | well, the container won't have the new action | 11:57 |
| jaosorior | EmilienM: ^^ | 11:57 |
| EmilienM | hello | 11:57 |
| *** jpena is now known as jpena|lunch | 11:58 | |
| *** morazi has joined #tripleo | 11:59 | |
| *** bfournie has joined #tripleo | 11:59 | |
| bogdando | jaosorior: IIRC applying changes to workflows used to require some dances, already, is it only the containers issue? | 11:59 |
| jaosorior | bogdando: yes, at least in CI | 11:59 |
| bogdando | I think there are snippets around, remove workflows, recreate et al | 11:59 |
| *** tzumainn has joined #tripleo | 12:00 | |
| bogdando | prolly this is only CI case and we can fix that for quickstart only? | 12:00 |
| jaosorior | bogdando: no idea how that works :D | 12:00 |
| bogdando | shardy_lunch: ^^ perchance? | 12:00 |
| jaosorior | bogdando: at least not in containers | 12:00 |
| bogdando | once you are back :) | 12:00 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: docker: cleanup update tasks https://review.openstack.org/568715 | 12:00 |
| openstackgerrit | Merged openstack/ansible-role-container-registry master: Use relative paths for data_files https://review.openstack.org/570088 | 12:00 |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/newton: Modify vhost user socket directory's default value in environment file https://review.openstack.org/561602 | 12:00 |
| bogdando | jaosorior: ok, can you submit a bug? | 12:01 |
| jaosorior | bogdando: will do | 12:01 |
| jaosorior | bogdando: we just stumbled upon this :D | 12:01 |
| *** skramaja has quit IRC | 12:01 | |
| *** dbecker has joined #tripleo | 12:05 | |
| moguimar | jaosorior: no meeting today? | 12:05 |
| jaosorior | moguimar: no, sent a mail about it | 12:05 |
| moguimar | k | 12:06 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-common master: Set FixedIp by default for public-facing VIP https://review.openstack.org/569818 | 12:06 |
| jaosorior | Tengu: worked :D ^^ | 12:06 |
| jistr | ansmith, ykarel, bandini: hi guys, we're trying to get an upgrade job working for Q->R upgrade, and we seem to be broken by https://review.openstack.org/#/c/507963/ | 12:08 |
| jistr | we're probably missing upgrade logic from RabbitMQ to OsloMessagingRpc | 12:09 |
| jistr | can you help me understand what happened, what should be done, and perhaps help with the implementation? | 12:09 |
| ansmith | jistr: would look forward to helping | 12:10 |
| jistr | i sorta assumed that we still deploy RabbitMQ as we did via pacemaker bundle, and we just changed how the services are configured to connect to it | 12:10 |
| jistr | so i just copied the upgrade tasks from P->Q upgrade https://review.openstack.org/#/c/570148 | 12:11 |
| jistr | but hit a wall real quick | 12:11 |
| jistr | the immediate surprise/question i have now is -- are we not managing RabbitMQ via a pacemaker bundle anymore? | 12:11 |
| ansmith | the intent is that everything stays the same and that oslomsg maps to existing rabbitmq | 12:11 |
| ansmith | it is still the same rabbitmq via pacemaker | 12:12 |
| ansmith | let me look at the p->q upgrade patch | 12:12 |
| bogdando | EmilienM, jaosorior: can we merge please https://review.openstack.org/#/c/564135/ ? | 12:13 |
| *** shardy_lunch is now known as shardy | 12:13 | |
| bogdando | before I have to rebase it again :) | 12:13 |
| bogdando | mwhahaha: updated https://review.openstack.org/#/c/569992/ | 12:14 |
| jistr | ansmith, bandini: i can't find anything mentioning the rabbitmq_bundle puppet class in the new rpc-rabbitmq template | 12:14 |
| jistr | i'd expect this | 12:14 |
| jistr | https://github.com/openstack/tripleo-heat-templates/blob/master/docker/services/pacemaker/rabbitmq.yaml#L198 | 12:14 |
| EmilienM | bogdando: I think mwhahaha wanted to land https://review.openstack.org/#/c/567951/9 first iirc | 12:14 |
| jistr | but there's this | 12:15 |
| jistr | https://github.com/openstack/tripleo-heat-templates/blob/master/docker/services/messaging/rpc-rabbitmq.yaml#L206 | 12:15 |
| shardy | jaosorior, bogdando: Sorry catching up on backscroll what was the question about containerized undercloud? | 12:15 |
| jistr | similarly, there's none of the 'pcmklatest' image tagging logic in rpc-rabbitmq | 12:15 |
| jistr | https://github.com/openstack/tripleo-heat-templates/blob/master/docker/services/pacemaker/rabbitmq.yaml#L75-L81 | 12:15 |
| jaosorior | shardy: found an issue in CI, where, if you change an action/workbook from tripleo-common, the containerized undercloud won't have the reflected action in the container | 12:15 |
| bogdando | shardy: it was mainly about how-to test workflow changes in CI | 12:15 |
| bogdando | we prolly need to add an init container step with some dances | 12:16 |
| jistr | ansmith, bandini: which leads me to think it's not just missing update/upgrade/ffwd tasks, but there's more stuff that we regressed on (e.g. the various tagging/init containers etc.) when we switched from RabbitMQ to OsloMessagingRpc | 12:16 |
| bogdando | to populate workflows | 12:16 |
| shardy | jaosorior, bogdando: Ah I see, sorry not sure how that's wired up - for containers we'd build and overlay a tripleo-ci repo, so I guess we need the same for containers but I'm not sure on the details tbh | 12:16 |
| bogdando | or limit the fix for CI/quickstart only | 12:16 |
| shardy | doesn't the same requirement exist for any container, e.g also any Depends-On for containerized overcloud etc? | 12:17 |
| *** udesale has joined #tripleo | 12:17 | |
| ansmith | jistr: https://github.com/openstack/tripleo-heat-templates/blob/master/docker/services/pacemaker/rpc-rabbitmq.yaml | 12:17 |
| bogdando | shardy: I think it is handled by containers yum updates in CI | 12:18 |
| bogdando | but populating workflows can't be done via yum update AFAIK | 12:18 |
| jistr | ansmith: oh i'm looking at a completely wrong template, aren't i :) thanks, will look again :) | 12:18 |
| jistr | that's what i get for using the 'T' search feature on github :D | 12:19 |
| ansmith | jistr: there is a question on how to best consolidate the "redundancy", need to work with bandini and eck on it | 12:19 |
| ykarel | jistr, /me no idea about this | 12:20 |
| shardy | bogdando: maybe d0ugal can confirm but I thought the workflows were loaded into the DB, which I assume happens after the container starts, e.g the state isn't actually in the container, right? | 12:20 |
| *** leitan has joined #tripleo | 12:20 | |
| jaosorior | shardy: right, but the actions are python code, which should be in the container | 12:20 |
| ansmith | jistr: let me know if you run into anything next, would like to help out | 12:21 |
| jaosorior | so even if the workflow definitions are up to date, if the actions don't match they'll fail | 12:21 |
| openstackgerrit | Oliver Walsh proposed openstack/instack-undercloud stable/ocata: WIP: prevent duplicate default cells being created by ocata upgrade https://review.openstack.org/570081 | 12:21 |
| shardy | jaosorior: right, but the actions will be updated by whatever the yum update method is, right? | 12:21 |
| shardy | as you'll pull in the overlay tripleo-common | 12:21 |
| jaosorior | they aren't as far as I can tell | 12:21 |
| jaosorior | that's the error log we're seeing | 12:21 |
| jistr | ansmith: thanks a bunch, for starters i'll try to patch the correct file this time, and see where that gets us with CI :) | 12:21 |
| jistr | ansmith, ykarel, bandini: sorry for the noise | 12:21 |
| shardy | jaosorior: Ok sounds like a bug, I was confused due to the mention of workflows vs actions above | 12:22 |
| jaosorior | shardy: was a bit hard to explain | 12:22 |
| jaosorior | I don't think we do container builds on the fs001 for the undercloud container | 12:23 |
| *** avivgt has quit IRC | 12:24 | |
| *** trown|outtypewww is now known as trown | 12:27 | |
| beagles | shardy: skramaja: if you get a chance - https://review.openstack.org/#/c/568138/ ... for context related patch https://review.openstack.org/#/c/568089/ (workflowed but latter is dependent on former) | 12:28 |
| beagles | marios: I'm trying to sort out an issue with https://review.openstack.org/#/c/567655/ - do you know who could work me through the details of FFU tasks and why running this as a FFU task vs. running it as standalone ansible post-upgrade might differ? | 12:30 |
| shardy | beagles: this changes the defaults in something we already released, what are the upgrade implications? | 12:32 |
| marios | beagles: o/ sure, what do you need to know. i can help as can most folks in upgrades squad jistr matbu lbezdick | 12:33 |
| marios | beagles: in the review you have the tasks in both upgrade/post-upgrades tasks | 12:33 |
| marios | beagles: perhaps this will help? (but not sure what you're after exactly) https://docs.openstack.org/tripleo-docs/latest/install/developer/upgrades/fast_fw_upgrade.html | 12:34 |
| marios | beagles: in terms of the steps/what executed when? | 12:34 |
| beagles | shardy: good question, I've asked nmagnezi to hop in and chat with you on it | 12:34 |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates master: Bring update/upgrade tasks for OsloMessagingRpc up to date https://review.openstack.org/570148 | 12:34 |
| nmagnezi | o/ | 12:34 |
| nmagnezi | what is the questions | 12:34 |
| * nmagnezi scrolls | 12:34 | |
| shardy | nmagnezi: Hi! Looking at https://review.openstack.org/#/c/568138/7/docker/services/octavia/octavia-deployment-config.yaml | 12:34 |
| shardy | we already released queens with the defaults that are being changed | 12:35 |
| *** rlandy has joined #tripleo | 12:35 | |
| shardy | what happens if someone upgrades and those values change? | 12:35 |
| *** quiquell|lunch is now known as quiquell | 12:35 | |
| beagles | marios: that actually is one question - do you need both or is it sufficient to simply have it in upgrade tasks as these will be run anyway? | 12:35 |
| shardy | seems like the answer could be "something bad" but I wanted to check :) | 12:35 |
| shardy | https://github.com/openstack/tripleo-heat-templates/blob/stable/queens/docker/services/octavia/octavia-deployment-config.yaml#L80 | 12:36 |
| marios | beagles: right. depends what you need to happen when. if it is enough for it to happen on upgrade tasks, and they stay intact for the duration of ffwd-upgrade tasks (and this doesn't create any problem with other services etc ) then yes | 12:36 |
| beagles | marios: I'm guessing it is FFU tasks followed by upgrade tasks maybe? | 12:36 |
| nmagnezi | shardy, well, I am no tripleO expect but I think that once that subnet got created that remains the case, no? beagles can keep me honest here | 12:36 |
| marios | beagles: exactly see the steps in that doc i linked | 12:36 |
| jistr | beagles: yes the last step of FFU is "almost normal" upgrade of 12->13 | 12:36 |
| marios | beagles: step 3 is the ffu tasks and then step 4 is the upgrade tasks and 'normal' config | 12:36 |
| marios | jistr: s/almost/exactly/ | 12:37 |
| marios | jistr: :) i mean it is literally 'openstack overcloud upgrade run' we use there | 12:37 |
| beagles | okay so I can ditch the FFU acl settings I think because this is only relevant for upgrading from pike->queens | 12:37 |
| nmagnezi | shardy, so for one, we can backport this change to stable/queens. as for existing deployments we issues a release notes so I guess it's up to operators to change that subnet if they would like not to be limited by low amount of addresses in that regard | 12:37 |
| jistr | marios, beagles: right it's the same command, but the starting state of the cloud differs... we start with services being stopped. That might or might not make a difference. | 12:37 |
| shardy | nmagnezi: well I'm not sure we can backport it (upstream) to queens if we want to follow the stable/maint rules | 12:38 |
| marios | jistr: ack | 12:38 |
| shardy | hence my question | 12:38 |
| beagles | jistr: marios: yeah, the acl only matters when moving from a non-containerized to containerized deployment so actually it does matter if someone is going from ocata->queens I think | 12:38 |
| nmagnezi | shardy, you got a point there | 12:38 |
| beagles | jistr: marios: the key point is it needs to happen somewhere around where the neutron agent containers are launched for the first time because that is where the permissions change on /var/lib/neutron occurs | 12:40 |
| nmagnezi | shardy, during the upgrade process (I presume you mean queens first release to a newer tag release) do those templates being re-run? | 12:40 |
| marios | beagles: if that means it can be 'completely after the containers are launch' then consider post_upgrade_tasks | 12:41 |
| *** vpickard_ is now known as vpickard | 12:41 | |
| beagles | jistr: marios: which actually probably is in the FFU realm right? | 12:41 |
| marios | beagles: otherwise, you may need to add it to the docker config? | 12:41 |
| shardy | nmagnezi: yes all config is re-applied, but I'm not certain if there's any conditionals which prevent that in the octavia post-install ansible | 12:41 |
| marios | beagles: no the ffu tasks aren't containerizing anything they are mostly concerend with stopping systemd things and running dbsyncs where necessary when going through the intermediary versions | 12:42 |
| jistr | beagles: i think that would be the pike->queens upgrade i think, both for ffwd-upgrade and slow upgrade. Neutron in Pike is still uncontainerized, even when going with the slow upgrades. | 12:42 |
| beagles | marios: ah oikay | 12:42 |
| jistr | beagles: is "somewhere around" really ok, or do we need it *before* the neutron containers are launched? | 12:43 |
| nmagnezi | shardy, you think it would make sense to add a conditional that prevents re-creation in a case where that subnet already exists? | 12:43 |
| bogdando | jaosorior: we do not build containers in CI jobs AFAIK. Instead we use yum update to pull in depends-on changes from repos, where those supposed to sit in | 12:43 |
| bogdando | so changes gets into rpms, then yum updated in containers, CI only | 12:44 |
| *** cshastri has quit IRC | 12:44 | |
| bogdando | if something is missing in tripleo common code, we have a bug in the updater logic | 12:44 |
| marios | jistr: beagles right. i asked if 'after' is ok too so he could consider the post upgrade tasks then | 12:44 |
| bogdando | EmilienM, sshnaidm|rover: ^^ | 12:44 |
| bogdando | I know there were some changes to how we update packages? | 12:44 |
| bogdando | in containers | 12:45 |
| *** ratailor has quit IRC | 12:45 | |
| beagles | shardy: we check for the existence of the networks before we create them so heat parameter changes like this shouldn't affect anything on upgrade | 12:45 |
| beagles | shardy: if it does it is probably a bug | 12:45 |
| jaosorior | bogdando: where is that code? | 12:45 |
| shardy | beagles: Ok, so we just need a release note saying the network range changed and that it won't be applied on upgrade from queens I guess | 12:45 |
| shardy | https://github.com/openstack/tripleo-common/blob/master/playbooks/roles/octavia-overcloud-config/tasks/network.yml | 12:46 |
| shardy | I see the conditionals there now | 12:46 |
| bogdando | jaosorior: http://git.openstack.org/cgit/openstack/tripleo-common/tree/scripts/container-update.py | 12:46 |
| bogdando | let me check for oooq... | 12:46 |
| beagles | marios: jistr: yeah, that might work - depends how persnickety neutron agents can be in an interval between the time the agents are launched and the acls are modified | 12:46 |
| beagles | nmagnezi: ^ see shardy's comment on rel note | 12:47 |
| nmagnezi | beagles, ack I'll add something about upgrades | 12:47 |
| jistr | beagles, marios: so i'm looking at the patch, and if doing the ACLs before the container is started is fine, then i think the patch is already doing it well (but we can drop it from ffwd_tasks, upgrade_tasks should be enough) | 12:48 |
| beagles | marios: jistr: what the interesting problem is - when I run the setfacl commands or an acl ansible task, I end up with a different result if I run the tasks via ansible from an undercloud command line | 12:48 |
| beagles | marios: jistr: there is something happening with the effective permissions there that I can't quite sort out where is coming from | 12:49 |
| bogdando | jaosorior: http://codesearch.openstack.org/?q=update_containers&i=nope&files=&repos=tripleo-ci,tripleo-quickstart-extras | 12:49 |
| jaosorior | bogdando: awesome, thanks | 12:49 |
| marios | beagles: strange only guess is 'environment variables' maybe vs ansible defaults | 12:49 |
| jistr | beagles: at which point in the procedure are you running it, when you're running it manually from undercloud? | 12:49 |
| *** pradk has quit IRC | 12:49 | |
| beagles | marios: jistr: after the upgrade has completed | 12:49 |
| bogdando | jaosorior: note, http://git.openstack.org/cgit/openstack/tripleo-quickstart-extras/tree/roles/overcloud-prep-containers/tasks/overcloud-prep-containers.yml is obsolete now that we have the script moved into tripleo common, there is a patch to address that, not merged yet | 12:50 |
| beagles | marios: jistr: yeah, it is strange. I was doing some experimentation with permissions etc try to find an ordering that produced the same results but nada | 12:50 |
| bogdando | jaosorior: btw, PTAL https://review.openstack.org/#/c/569976/ reworked gdpr logrotate | 12:51 |
| bogdando | I ran out of better ideas | 12:51 |
| jistr | beagles: and running it after upgrade gives you better/correct results? Could it be that we need to wait for some files to appear (by starting the neutron containers) so that the acl settings work the way we need? | 12:51 |
| beagles | marios: jistr: in this case the files are expected to exist so the "default" acl entries should not be relevant. If you log into the machine and run the setfacl as root from the comand line you get what you want too - so it hints at either "order of operations" or enviroment as the difference | 12:51 |
| beagles | jistr: yeah it might be something like that ... even the mounting of the directory into the container might be messing things up | 12:52 |
| *** mcornea has joined #tripleo | 12:53 | |
| jaosorior | bogdando: uhm... I guess that makes sense | 12:53 |
| jaosorior | I can't think of anything better either | 12:53 |
| beagles | jistr: marios: it's probably worthwhile to try it as a post upgrade task to see if it is ordering. If if still doesn't get the proper result then it hints more strongly to an ansible runtime environment issue | 12:54 |
| jistr | beagles: if we'd be ok running both containerized and non-containerized alongside *without* the ACL rules in place for a while, then applying it in the post_upgrade_tasks could be an option. It should be closer to what you did manually -- essentially apply the ACLs after the upgrade. | 12:54 |
| jistr | beagles: yea sounds like a good plan to me | 12:54 |
| marios | beagles: ack if i recall post upgrade only has 2 sec let me check (steps i mean) just fyi we can make it what you need though | 12:54 |
| marios | beagles: ah 4 in fact https://github.com/openstack/tripleo-heat-templates/blob/6d10c0516b447d2b75dbe184c4097d931e0f0e6f/common/deploy-steps.j2#L21 | 12:55 |
| marios | beagles: so you can keep it as you already have it with "step2" | 12:55 |
| marios | beagles: i mean if you move to postupgradetasks | 12:55 |
| cgoncalves | bogdando, hi! could you please review https://review.openstack.org/#/c/570163/ ? | 12:56 |
| *** myoung|ruck is now known as myoung | 12:56 | |
| beagles | marios: ack | 12:56 |
| cgoncalves | jaosorior, ^ I'd appreciate your review too, being you a tripleo and security expert :) | 12:56 |
| beagles | marios: jistr: quick question - upgrade_tasks happen before the post upgrade tasks? | 12:57 |
| *** toure|gone is now known as toure | 12:57 | |
| openstackgerrit | melissaml proposed openstack/tripleo-quickstart-extras master: fix typos in documentation https://review.openstack.org/570178 | 12:58 |
| jistr | beagles: yes, it's 1) upgrade_tasks 2) "the normal deploy workflow" -- puppet steps, containers starting steps, ..., and 3) post_upgrade_tasks | 12:58 |
| *** rbrady has quit IRC | 12:58 | |
| beagles | jistr: okay cool thanks | 12:58 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: upgrade: remove packages at step 4 https://review.openstack.org/570116 | 12:58 |
| jaosorior | cgoncalves: LGTM | 12:58 |
| *** jpena|lunch is now known as jpena | 12:59 | |
| *** amoralej|lunch is now known as amoralej | 12:59 | |
| EmilienM | marios: I updated the commit message on https://review.openstack.org/570116 and answered your questions, and also changed the step as agreed on IRC | 12:59 |
| cgoncalves | jaosorior, thank you, really! | 12:59 |
| jaosorior | cgoncalves: thanks for coding it :) | 13:00 |
| marios | beagles: sorry was getting coffee jistr already answered | 13:03 |
| marios | EmilienM: ack will check thanks | 13:03 |
| beagles | marios :) | 13:03 |
| *** pchavva has joined #tripleo | 13:03 | |
| openstackgerrit | Emilien Macchi proposed openstack/python-tripleoclient master: Make standalone role name configurable https://review.openstack.org/568378 | 13:05 |
| openstackgerrit | Emilien Macchi proposed openstack/python-tripleoclient master: Start standalone.conf https://review.openstack.org/569474 | 13:05 |
| openstackgerrit | Emilien Macchi proposed openstack/python-tripleoclient master: Move undercloud.conf to the undercloud config namespace https://review.openstack.org/569477 | 13:05 |
| openstackgerrit | Emilien Macchi proposed openstack/python-tripleoclient master: WIP: Handle config loading in tripleo deploy https://review.openstack.org/569535 | 13:05 |
| EmilienM | chandankumar: could you help to debug something with containerized tempest? | 13:07 |
| chandankumar | EmilienM: yup log link | 13:07 |
| EmilienM | chandankumar: http://logs.openstack.org/64/560264/27/check/tripleo-ci-centos-7-containers-multinode/d574a5f/logs/undercloud/home/zuul/tempest.log.txt.gz#_2018-05-23_04_22_11 | 13:08 |
| EmilienM | chandankumar: you can use http://logs.openstack.org/64/560264/27/check/tripleo-ci-centos-7-containers-multinode/d574a5f/logs/reproducer-quickstart.sh to reproduce if you use this tool | 13:09 |
| *** vpickard has quit IRC | 13:12 | |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates master: Add acl to paths that are shared among related neutron processes https://review.openstack.org/567655 | 13:12 |
| *** vpickard has joined #tripleo | 13:12 | |
| EmilienM | chandankumar: the IP used to reach keystone is wrong | 13:12 |
| EmilienM | chandankumar: see http://logs.openstack.org/64/560264/27/check/tripleo-ci-centos-7-containers-multinode/d574a5f/logs/undercloud/home/zuul/overcloud_deploy_post.log.txt.gz#_2018-05-23_04_14_34 | 13:12 |
| EmilienM | chandankumar: it should be 24.1, not 24.10 | 13:12 |
| *** tcw has quit IRC | 13:13 | |
| EmilienM | chandankumar: actually nevermind, it works with 24.10 as well, http://logs.openstack.org/64/560264/27/check/tripleo-ci-centos-7-containers-multinode/d574a5f/logs/undercloud/home/zuul/tempest.log.txt.gz#_2018-05-23_04_16_57 | 13:13 |
| chandankumar | EmilienM: it takes the value from overcloudrc file | 13:13 |
| *** tcw has joined #tripleo | 13:13 | |
| EmilienM | chandankumar: it's weird that before in the script keystone is reachable | 13:13 |
| chandankumar | EmilienM: is it coming in all jobs or this one only? | 13:14 |
| EmilienM | chandankumar: just for 560264 but I did recheck and obtained the same results | 13:14 |
| EmilienM | chandankumar: the undercloud is containerized in this multinode job | 13:15 |
| EmilienM | chandankumar: note it works fine on OVB job which also containerize tempest AFIK https://logs.rdoproject.org/23/569323/4/openstack-check/gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-master/Z159930af42e248c294900dc158876b4d/tempest.html.gz | 13:15 |
| *** wolverineav has quit IRC | 13:16 | |
| *** yolanda has quit IRC | 13:17 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates stable/pike: Honor {{role.name}}ExtraConfig with deprecated params https://review.openstack.org/570191 | 13:19 |
| *** yolanda has joined #tripleo | 13:19 | |
| *** eck` is now known as eck`gone | 13:21 | |
| *** gvrangan has quit IRC | 13:21 | |
| *** eck`gone is now known as eck` | 13:22 | |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo master: Remove support for puppet-ceph https://review.openstack.org/570193 | 13:22 |
| *** lblanchard has joined #tripleo | 13:25 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates stable/pike: Remove ceph-osd from the list of packages updated by CephExternal https://review.openstack.org/570196 | 13:27 |
| chandankumar | EmilienM: which other multinode jobs have containerized UC? | 13:27 |
| jistr | beagles: FYI if you're testing, a small thing that needs fixing (in both files): https://review.openstack.org/#/c/567655/11/docker/services/neutron-l3.yaml | 13:29 |
| *** janki has quit IRC | 13:31 | |
| *** janki has joined #tripleo | 13:31 | |
| *** psachin has quit IRC | 13:32 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates stable/ocata: Remove ceph-osd from the list of packages updated by CephExternal https://review.openstack.org/570199 | 13:33 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates stable/newton: Remove ceph-osd from the list of packages updated by CephExternal https://review.openstack.org/570200 | 13:35 |
| *** lblanchard1 has joined #tripleo | 13:36 | |
| *** lblanchard has quit IRC | 13:36 | |
| *** ykarel is now known as ykarel|afk | 13:36 | |
| chandankumar | sshnaidm|rover: is this familiar with reproducer script http://paste.openstack.org/show/721731/ | 13:38 |
| chandankumar | ? | 13:38 |
| *** social has joined #tripleo | 13:38 | |
| *** lblanchard has joined #tripleo | 13:39 | |
| *** lblanchard1 has quit IRC | 13:39 | |
| *** lblanchard1 has joined #tripleo | 13:41 | |
| *** lblanchard has quit IRC | 13:41 | |
| sshnaidm|rover | chandankumar, nope | 13:42 |
| chandankumar | sshnaidm|rover: it is coming while creating reproducer env with http://logs.openstack.org/64/560264/27/check/tripleo-ci-centos-7-containers-multinode/d574a5f/logs/reproducer-quickstart.sh | 13:43 |
| chandankumar | on rdocloud | 13:43 |
| *** wolverineav has joined #tripleo | 13:43 | |
| sshnaidm|rover | chandankumar, try to remove you previous work folder | 13:44 |
| *** pdeore has joined #tripleo | 13:45 | |
| jistr | merge? :) -- multinode NIC config compatible with upgrade jobs https://review.openstack.org/#/c/569781 | 13:46 |
| bogdando | EmilienM, mwhahaha: PTAL https://review.openstack.org/#/c/564135/ | 13:47 |
| bogdando | also please https://review.openstack.org/#/c/569992/ | 13:48 |
| *** ramishra_ has quit IRC | 13:49 | |
| *** bkopilov has joined #tripleo | 13:49 | |
| mwhahaha | bogdando: 564135 is going to conflict with 567951 which is in the gate, you may need to rebase it | 13:50 |
| openstackgerrit | Marius Cornea proposed openstack/tripleo-upgrade stable/queens: DNM: Run enable-ssh-admin.sh script before fast forward run https://review.openstack.org/570074 | 13:50 |
| chandankumar | sshnaidm|rover: still same issue | 13:51 |
| *** ramishra has joined #tripleo | 13:51 | |
| EmilienM | chandankumar: none | 13:51 |
| openstackgerrit | Marius Cornea proposed openstack/tripleo-quickstart-extras master: export OVERCLOUD_HOSTS when running enable-ssh-admin.sh https://review.openstack.org/570091 | 13:52 |
| openstackgerrit | Nir Magnezi proposed openstack/tripleo-heat-templates master: Make lb-mgmt-subnet a class B subnet https://review.openstack.org/568138 | 13:52 |
| honza | florianf: would you mind pushing these 2 patches across the finish line? https://review.openstack.org/#/c/561211/ | 13:53 |
| nmagnezi | shardy, beagles updated the release note for https://review.openstack.org/#/c/568138/ | 13:53 |
| openstackgerrit | Nir Magnezi proposed openstack/tripleo-heat-templates master: Make lb-mgmt-subnet a class B subnet https://review.openstack.org/568138 | 13:54 |
| shardy | nmagnezi: thanks, +2 | 13:55 |
| nmagnezi | shardy, thank you for the feedback :-) | 13:56 |
| florianf | honza: yep, let me have a look | 14:01 |
| *** pradk has joined #tripleo | 14:01 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-common master: Remove the upgrade-non-controller.sh and disable_upgrade_deployment https://review.openstack.org/557694 | 14:01 |
| florianf | honza: done | 14:02 |
| honza | florianf: perfect, thank you | 14:02 |
| openstackgerrit | Thomas Herve proposed openstack/puppet-tripleo master: Automatically configure neutron with designate https://review.openstack.org/569987 | 14:03 |
| openstackgerrit | Thomas Herve proposed openstack/puppet-tripleo master: Automatically configure neutron with designate https://review.openstack.org/569987 | 14:04 |
| *** dbecker has quit IRC | 14:07 | |
| *** janki has quit IRC | 14:07 | |
| *** bdodd|afk is now known as bdodd | 14:10 | |
| *** ohochman has joined #tripleo | 14:13 | |
| openstackgerrit | Thomas Herve proposed openstack/tripleo-heat-templates master: Pass designate configuration to neutron https://review.openstack.org/569990 | 14:18 |
| *** matbu has quit IRC | 14:20 | |
| *** zoli is now known as zoli|brb | 14:22 | |
| *** rpioso|afk is now known as rpioso | 14:22 | |
| *** matbu has joined #tripleo | 14:22 | |
| *** saneax has quit IRC | 14:28 | |
| *** artom_ is now known as artom | 14:28 | |
| *** slaweq has joined #tripleo | 14:33 | |
| EmilienM | bogdando: like I said, rebase on top and https://review.openstack.org/#/c/567951/9 | 14:34 |
| bogdando | EmilienM: ack | 14:34 |
| openstackgerrit | Oliver Walsh proposed openstack/instack-undercloud stable/ocata: WIP: prevent duplicate default cells being created by ocata upgrade https://review.openstack.org/570081 | 14:34 |
| openstackgerrit | Jose Luis Franco proposed openstack/tripleo-quickstart master: Pass tripleo-admin as ssh-user in FFWD job. https://review.openstack.org/570208 | 14:35 |
| bogdando | what about https://review.openstack.org/#/c/569992/ ? can it make it w/o rebasing EmilienM? | 14:35 |
| EmilienM | bogdando: yes, gerrit doesn't show conflict on top right | 14:36 |
| bogdando | indeed, thanks | 14:36 |
| openstackgerrit | John Fulton proposed openstack/tripleo-heat-templates master: Set default application for Ceph Luminous openstack_pools https://review.openstack.org/570043 | 14:38 |
| *** homegrown has quit IRC | 14:39 | |
| openstackgerrit | John Fulton proposed openstack/tripleo-heat-templates master: Set default application for Ceph Luminous openstack_pools https://review.openstack.org/570043 | 14:39 |
| openstackgerrit | Cédric Jeanneret proposed openstack/tripleo-quickstart master: New config in order to get IPv6 with pacemaker on a single-nic isolated net https://review.openstack.org/570209 | 14:41 |
| *** janki has joined #tripleo | 14:43 | |
| *** bnemec has joined #tripleo | 14:44 | |
| pdeore | EmilienM, jaosorior, shardy , jistr could please have a look at https://review.openstack.org/#/c/569970 whenever you are free ... | 14:44 |
| openstackgerrit | Bogdan Dobrelya proposed openstack/python-tripleoclient master: Store UC hosts/portmaps heat params in output_dir https://review.openstack.org/564135 | 14:49 |
| *** nyechiel_ has joined #tripleo | 14:51 | |
| openstackgerrit | Bogdan Dobrelya proposed openstack/tripleo-quickstart-extras master: Fix path and wire-in UC deploy role data file https://review.openstack.org/563988 | 14:51 |
| rasca | mwhahaha, ok, I confirm that the last failure that I got in RDOPhase2 was not related to the ssh script | 14:51 |
| rasca | mwhahaha, about the other issue we were talking about, do we already have some sort of bug? | 14:52 |
| rasca | mwhahaha, I'm seeing messages like "Error: Evaluation Error: Error while evaluating a Function Call, Could not find data item oslo_messaging_rpc_password in any Hiera data file and no default supplied at /etc/puppet/modules/tripleo/manifests/profile/base/nova.pp:85:30 on node overcloud-novacompute-1.localdomain" | 14:53 |
| mwhahaha | rasca: yea that | 14:54 |
| mwhahaha | rasca: so that seems to be a missing environment config of some sport | 14:54 |
| mwhahaha | rasca: the jobs might need to be updated for master | 14:54 |
| mwhahaha | rasca: it sounds like it's using OS::Tripleo::Service::Rabbitmq instead of the new oslomessaging ones | 14:55 |
| mwhahaha | rasca: so i think there needs to be some review of the environment configs | 14:55 |
| rasca | mwhahaha, oh it should be the case | 14:55 |
| rasca | mwhahaha, I'm using the same roles definitions for both pike and master at the moment so maybe that's the problem | 14:56 |
| mwhahaha | rasca: yea that's likely the problem | 14:56 |
| *** mvenesio has joined #tripleo | 14:57 | |
| rasca | mwhahaha, ok, let me try to solve that | 14:57 |
| rasca | mwhahaha, too many failures today man | 14:57 |
| *** avivgt has joined #tripleo | 14:58 | |
| openstackgerrit | Merged openstack/tripleo-quickstart-extras master: Support upgrade_cleanup param in undercloud.conf https://review.openstack.org/569898 | 15:01 |
| *** pdeore has quit IRC | 15:01 | |
| *** rbrady has joined #tripleo | 15:02 | |
| *** zoli|brb is now known as zoli | 15:02 | |
| openstackgerrit | Jose Luis Franco proposed openstack/tripleo-upgrade stable/queens: Do not run nic template conversion on default tht location https://review.openstack.org/569272 | 15:02 |
| *** rbrady is now known as Guest75989 | 15:02 | |
| *** matbu has quit IRC | 15:07 | |
| *** zaneb has quit IRC | 15:07 | |
| mwhahaha | uh oh | 15:07 |
| mwhahaha | seeing packaging failures on tripleoclient | 15:08 |
| * mwhahaha sighs | 15:08 | |
| openstackgerrit | Raoul Scarazzini proposed openstack/tripleo-ha-utils master: Fix the last old repository references https://review.openstack.org/570210 | 15:08 |
| mwhahaha | hrm maybe it's my patch | 15:09 |
| * mwhahaha goes looking at the rebase | 15:09 | |
| openstackgerrit | Harald Jensås proposed openstack/puppet-tripleo master: Fix duplicate entries in /etc/sysconfig/iptables https://review.openstack.org/568351 | 15:10 |
| *** noslzzp_ has quit IRC | 15:11 | |
| openstackgerrit | Merged openstack/tripleo-quickstart master: fs050: enable upgrade cleanup https://review.openstack.org/569899 | 15:11 |
| openstackgerrit | Merged openstack/tripleo-upgrade stable/queens: Set mon_max_pg_per_osd to 300 during ceph upgrade https://review.openstack.org/569981 | 15:11 |
| openstackgerrit | Merged openstack/tripleo-common master: Update heat api healtchecks https://review.openstack.org/569852 | 15:11 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Fix known hosts config script value https://review.openstack.org/570029 | 15:11 |
| openstackgerrit | Merged openstack-infra/tripleo-ci master: Add ability to use a different release per playbook https://review.openstack.org/566565 | 15:11 |
| *** yamahata has joined #tripleo | 15:11 | |
| openstackgerrit | Raoul Scarazzini proposed openstack/tripleo-ha-utils master: Fix the last old repository references https://review.openstack.org/570210 | 15:12 |
| *** bnemec has quit IRC | 15:13 | |
| openstackgerrit | yatin proposed openstack/tripleo-heat-templates master: Use /usr/bin/env python to autodetect python https://review.openstack.org/570213 | 15:17 |
| *** pblaho has quit IRC | 15:18 | |
| *** lvdombrkr has quit IRC | 15:22 | |
| *** ukalifon has quit IRC | 15:22 | |
| openstackgerrit | Ryan Brady proposed openstack/tripleo-common master: [WIP]Fixes ordering of environment files in a deployment plan https://review.openstack.org/570215 | 15:23 |
| *** pblaho has joined #tripleo | 15:23 | |
| openstackgerrit | Ryan Brady proposed openstack/tripleo-common master: [WIP]Fixes ordering of environment files in a deployment plan https://review.openstack.org/570215 | 15:24 |
| *** slaweq has quit IRC | 15:24 | |
| *** dbecker has joined #tripleo | 15:26 | |
| *** bogdando has quit IRC | 15:30 | |
| *** mburned is now known as mburned_out | 15:32 | |
| *** dbecker has quit IRC | 15:32 | |
| *** pcaruana|afk| has quit IRC | 15:33 | |
| *** social has quit IRC | 15:33 | |
| *** quiquell is now known as quiquell|off | 15:34 | |
| *** ssbarnea_ has quit IRC | 15:36 | |
| openstackgerrit | Jose Luis Franco proposed openstack/tripleo-quickstart master: Unset test_ping in fs051 upgrades job. https://review.openstack.org/570216 | 15:37 |
| *** ramishra has quit IRC | 15:37 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: upgrade: remove packages at step 3 https://review.openstack.org/570116 | 15:39 |
| openstackgerrit | melissaml proposed openstack/tripleo-heat-templates master: fix a typo in collectd.yaml https://review.openstack.org/570218 | 15:39 |
| *** matbu has joined #tripleo | 15:40 | |
| *** arxcruz is now known as arxcruz|rover | 15:42 | |
| *** rlandy is now known as rlandy|rover | 15:42 | |
| *** dparkes has quit IRC | 15:43 | |
| *** dbecker has joined #tripleo | 15:43 | |
| *** salmankhan1 has joined #tripleo | 15:43 | |
| openstackgerrit | Jiri Stransky proposed openstack-infra/tripleo-ci master: Add keystone only upgrade jobs https://review.openstack.org/558425 | 15:44 |
| *** salmankhan has quit IRC | 15:45 | |
| *** salmankhan1 is now known as salmankhan | 15:45 | |
| *** etingof is now known as etingof|brb | 15:45 | |
| *** ssbarnea_ has joined #tripleo | 15:46 | |
| *** leitan has quit IRC | 15:46 | |
| *** leitan has joined #tripleo | 15:46 | |
| *** anilvenkata has joined #tripleo | 15:47 | |
| shardy | Anyone know the status of tripleo-ui when using a containerized undercloud, jrist perhaps? | 15:48 |
| *** ssbarnea_ has quit IRC | 15:48 | |
| mwhahaha | shardy: it should work | 15:48 |
| mwhahaha | shardy: at least it was working-ish when emilien last was messing with it | 15:49 |
| shardy | mwhahaha: Ok good news, I'm rebasing an older test env and wanted both UI and containers :) | 15:49 |
| shardy | mwhahaha: Ok thanks, I'll give it a go | 15:49 |
| *** slaweq has joined #tripleo | 15:49 | |
| shardy | as a side note setting containerized_undercloud: true is no longer enough to actually deploy a containerized undercloud ;) | 15:49 |
| mwhahaha | shardy: in oooq? | 15:50 |
| shardy | looks like I'm missing undercloud_install_cli_options: " --use-heat" and maybe some other stuff | 15:50 |
| jrist | yeah should work | 15:50 |
| shardy | https://github.com/openstack/tripleo-quickstart/blob/master/config/general_config/featureset027.yml#L16 | 15:50 |
| jrist | shadower: yeah --use-heat for now | 15:50 |
| jrist | shardy: ^ | 15:50 |
| shardy | mwhahaha: yeah, looks like that boolean doesn't set all the right vars, I'll push a patch when I figure out what's missing | 15:50 |
| jrist | shadower: disregard :) | 15:50 |
| mwhahaha | shardy: yea we haven't fully cut over | 15:50 |
| *** ssbarnea_ has joined #tripleo | 15:51 | |
| shardy | jrist: Ok thanks for confirming, I'll kick off another oooq run :) | 15:51 |
| *** suuuper has quit IRC | 15:51 | |
| *** leitan has quit IRC | 15:51 | |
| jrist | good luck! | 15:51 |
| jrist | I'm doing one right now :) | 15:51 |
| *** zshi has joined #tripleo | 15:53 | |
| *** bdodd is now known as bdodd|afk | 15:54 | |
| EmilienM | shardy: I tried last week and it worked | 15:55 |
| shardy | EmilienM: ack thanks for the info | 15:57 |
| *** mburned_out is now known as mburned | 15:57 | |
| *** ssbarnea_ has quit IRC | 15:58 | |
| *** avivgt has quit IRC | 15:59 | |
| *** ffiore has quit IRC | 15:59 | |
| openstackgerrit | RedHat RDO CI proposed openstack/tripleo-quickstart-extras master: GATE CHECK for quickstart-extras https://review.openstack.org/560445 | 16:00 |
| *** marios has quit IRC | 16:01 | |
| *** kopecmartin has quit IRC | 16:03 | |
| *** arxcruz|rover is now known as arxcruz|ruck | 16:03 | |
| *** yamahata has quit IRC | 16:04 | |
| *** dprince has joined #tripleo | 16:04 | |
| dpeacock | mwhahaha: Ping - can you take another look at https://review.openstack.org/#/c/564830/ when you get a chance please? I noticed just now it's lingering. | 16:04 |
| mwhahaha | dpeacock: it's merged? | 16:05 |
| *** bdodd|afk is now known as bdodd | 16:05 | |
| dpeacock | Not as far as I can tell. | 16:05 |
| dpeacock | The related patches merged already | 16:05 |
| *** pchavva has quit IRC | 16:06 | |
| mwhahaha | dpeacock: May 9 12:44 PM | 16:06 |
| mwhahaha | dpeacock: Change has been successfully merged by Zuul | 16:06 |
| mwhahaha | dpeacock: now, https://review.openstack.org/#/c/568727 is not merged | 16:06 |
| mwhahaha | but i can't do anything with that | 16:06 |
| *** udesale has quit IRC | 16:07 | |
| mwhahaha | that's something for the TC probably after the summit | 16:07 |
| dpeacock | Sorry - think I pasted wrong link - https://review.openstack.org/#/c/564823/ | 16:07 |
| mwhahaha | i can just +1 | 16:07 |
| mwhahaha | that's an infra thing, also probably needs to wait for teh summit | 16:07 |
| dpeacock | ok - taa | 16:08 |
| *** zoli is now known as zoli|gone | 16:11 | |
| *** zoli|gone is now known as zoli | 16:11 | |
| *** salmankhan has quit IRC | 16:12 | |
| *** jfrancoa has quit IRC | 16:15 | |
| *** jfrancoa has joined #tripleo | 16:16 | |
| *** links has quit IRC | 16:16 | |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates master: Add acl to paths that are shared among related neutron processes https://review.openstack.org/567655 | 16:17 |
| *** slaweq has quit IRC | 16:19 | |
| *** salmankhan has joined #tripleo | 16:19 | |
| *** janki has quit IRC | 16:19 | |
| *** bdodd is now known as bdodd|afk | 16:21 | |
| *** olap has quit IRC | 16:21 | |
| *** bdodd|afk is now known as bdodd | 16:22 | |
| *** morazi has quit IRC | 16:23 | |
| *** ykarel|afk has quit IRC | 16:24 | |
| *** dbecker has quit IRC | 16:24 | |
| *** pkovar has quit IRC | 16:24 | |
| *** slaweq has joined #tripleo | 16:24 | |
| gfidente | mwhahaha jaosorior do you guys know if from the output of the heat stack or in hiera we have a list of the IPs member of a given role? | 16:26 |
| *** gvrangan has joined #tripleo | 16:26 | |
| *** masco has quit IRC | 16:26 | |
| mwhahaha | gfidente: good question | 16:26 |
| mwhahaha | gfidente: i assume we do somewhere because we use it for the inventory building in newer versions | 16:27 |
| mwhahaha | cause ansible knows :D | 16:27 |
| *** dciabrin_ has quit IRC | 16:27 | |
| *** jfrancoa has quit IRC | 16:27 | |
| gfidente | mwhahaha ah let me check that code | 16:27 |
| *** dciabrin_ has joined #tripleo | 16:27 | |
| *** pcaruana|afk| has joined #tripleo | 16:29 | |
| *** nyechiel_ has quit IRC | 16:30 | |
| *** dxiri has joined #tripleo | 16:31 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/tripleo-heat-templates master: Add a way to override base path when file driver is used https://review.openstack.org/570226 | 16:31 |
| jaosorior | gfidente: you could get that info from ansible using groups | 16:31 |
| jaosorior | not sure if we have such a thing in hiera though | 16:32 |
| *** radek__ has quit IRC | 16:33 | |
| *** radeks has quit IRC | 16:34 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: upgrade: remove packages at step 3 https://review.openstack.org/570116 | 16:35 |
| *** gvrangan has quit IRC | 16:36 | |
| openstackgerrit | Alex Schultz proposed openstack/python-tripleoclient master: Make standalone role name configurable https://review.openstack.org/568378 | 16:37 |
| openstackgerrit | Alex Schultz proposed openstack/python-tripleoclient master: Start standalone.conf https://review.openstack.org/569474 | 16:37 |
| openstackgerrit | Alex Schultz proposed openstack/python-tripleoclient master: Move undercloud.conf to the undercloud config namespace https://review.openstack.org/569477 | 16:37 |
| openstackgerrit | Alex Schultz proposed openstack/python-tripleoclient master: WIP: Handle config loading in tripleo deploy https://review.openstack.org/569535 | 16:37 |
| gfidente | jaosorior mwhahaha https://github.com/openstack/tripleo-heat-templates/blob/master/overcloud.j2.yaml#L1009 | 16:38 |
| gfidente | that's it | 16:38 |
| mwhahaha | see i knew it was there somewhere :D | 16:39 |
| * mwhahaha keeps pretending like he knows what he's doing | 16:39 | |
| *** gvrangan has joined #tripleo | 16:40 | |
| *** etingof|brb is now known as etingof | 16:40 | |
| *** gbarros has joined #tripleo | 16:41 | |
| *** zshi has quit IRC | 16:45 | |
| *** trozet_ is now known as trozet | 16:45 | |
| openstackgerrit | Carlos Goncalves proposed openstack/tripleo-common master: Fix Octavia amphora image RPM install https://review.openstack.org/570163 | 16:45 |
| cgoncalves | shardy, jaosorior, beagles: I'm so sorry. can you revisit https://review.openstack.org/#/c/570163/ ? | 16:46 |
| *** slaweq has quit IRC | 16:46 | |
| *** zshi has joined #tripleo | 16:47 | |
| *** dprince has quit IRC | 16:47 | |
| *** janki has joined #tripleo | 16:47 | |
| trozet | can someone help me figure out why this job failed? I see the deploy has RC 1, but I cannot find any indication of what actually failed: http://logs.openstack.org/05/486905/36/experimental/tripleo-ci-centos-7-scenario008-multinode-oooq/ccf868a/logs/undercloud/home/zuul/overcloud_deploy.log.txt.gz | 16:47 |
| openstackgerrit | Alan Bishop proposed openstack/tripleo-heat-templates master: Rename Dell EMC XTREMIO iSCSI environment file https://review.openstack.org/570230 | 16:48 |
| *** jpena is now known as jpena|off | 16:49 | |
| mwhahaha | trozet: Timed out waiting for messages from Execution (ID: 0b0c1f9b-e476-41c1-820c-17c5069bcab4, State: RUNNING). The WebSocket timed out before the Workflow completed. | 16:51 |
| *** bnemec has joined #tripleo | 16:51 | |
| mwhahaha | dat stderr/stdout mixing | 16:51 |
| trozet | mwhahaha: yeah i saw that, what does that websocket do...i remember seeing some code around that in tripleo-common | 16:51 |
| *** slaweq has joined #tripleo | 16:51 | |
| *** pcaruana|afk| has quit IRC | 16:51 | |
| *** sshnaidm|rover is now known as sshnaidm|off | 16:52 | |
| *** gbarros has quit IRC | 16:52 | |
| mwhahaha | trozet: so the deploy command polls for results from the workfloows via websocket | 16:53 |
| *** shreshtha has quit IRC | 16:53 | |
| mwhahaha | trozet: so the deploy probably hung and then hit a timeout | 16:53 |
| mwhahaha | trozet: since the last output was host config step3, i'd assume some sort of pacemaker problems | 16:54 |
| mwhahaha | trozet: that's generally what goes wrong around step3 | 16:54 |
| trozet | mwhahaha: yeah i was just about to say we dont do anything in step 3 for odl | 16:54 |
| *** dparkes has joined #tripleo | 16:55 | |
| openstackgerrit | Merged openstack/tripleo-ha-utils master: Fix the last old repository references https://review.openstack.org/570210 | 16:55 |
| *** elgxl has joined #tripleo | 16:55 | |
| *** janki has quit IRC | 16:56 | |
| trozet | mwhahaha: ok thanks i'll try again and see what happens | 16:56 |
| trozet | btw lol @ Welcome to Rocky. CI status: GREEEEEEEEN (maybe) | 16:56 |
| openstackgerrit | Tim Rozet proposed openstack/tripleo-heat-templates master: Add OPNFV scenario environment https://review.openstack.org/486905 | 16:57 |
| *** holser__ has quit IRC | 16:57 | |
| *** slaweq has quit IRC | 16:58 | |
| *** dtantsur is now known as dtantsur|afk | 16:59 | |
| *** derekh has quit IRC | 17:00 | |
| *** shreshtha has joined #tripleo | 17:00 | |
| *** trown is now known as trown|lunch | 17:00 | |
| *** ssbarnea_ has joined #tripleo | 17:01 | |
| *** noslzzp has joined #tripleo | 17:01 | |
| *** pchavva has joined #tripleo | 17:04 | |
| *** florianf has quit IRC | 17:05 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Fix upgrade_tasks for TripleO UI https://review.openstack.org/570237 | 17:05 |
| *** shreshtha has quit IRC | 17:06 | |
| *** gbarros has joined #tripleo | 17:06 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: upgrade: remove packages at step 3 https://review.openstack.org/570116 | 17:07 |
| *** rpioso is now known as rpioso|eat | 17:09 | |
| *** cshastri has joined #tripleo | 17:09 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: nova-ironic: add support for post-upgrade cleanup https://review.openstack.org/570241 | 17:10 |
| *** slaweq has joined #tripleo | 17:11 | |
| *** pkovar has joined #tripleo | 17:11 | |
| *** amoralej is now known as amoralej|off | 17:12 | |
| *** gvrangan has quit IRC | 17:12 | |
| *** zaneb has joined #tripleo | 17:12 | |
| *** gvrangan has joined #tripleo | 17:13 | |
| *** salmankhan has quit IRC | 17:15 | |
| *** slaweq has quit IRC | 17:15 | |
| *** homegrown has joined #tripleo | 17:16 | |
| *** anilvenkata has quit IRC | 17:19 | |
| *** elgxl has quit IRC | 17:23 | |
| *** elgxl has joined #tripleo | 17:24 | |
| redrobot | Ohai, TripleO friends! I'm new here. Looking forward to working with y'all. | 17:24 |
| *** gfidente is now known as gfidente|afk | 17:26 | |
| *** bnemec has quit IRC | 17:28 | |
| *** elgxl has quit IRC | 17:28 | |
| *** leitan has joined #tripleo | 17:30 | |
| mwhahaha | redrobot: oh noes not you | 17:31 |
| mwhahaha | also, welcome | 17:32 |
| *** gvrangan has quit IRC | 17:32 | |
| *** gvrangan has joined #tripleo | 17:34 | |
| *** matbu has quit IRC | 17:35 | |
| *** noslzzp has quit IRC | 17:35 | |
| *** cshastri has quit IRC | 17:37 | |
| redrobot | hehe | 17:37 |
| *** tesseract has quit IRC | 17:38 | |
| *** bdodd is now known as bdodd|afk | 17:39 | |
| *** gbarros has quit IRC | 17:41 | |
| *** gvrangan has quit IRC | 17:41 | |
| *** mvenesio_ has joined #tripleo | 17:43 | |
| *** mvenesio has quit IRC | 17:43 | |
| *** gvrangan has joined #tripleo | 17:45 | |
| *** ssbarnea_ has quit IRC | 17:45 | |
| *** bdodd|afk is now known as bdodd | 17:45 | |
| *** zaneb has quit IRC | 17:46 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/queens: Mount ODL log file on host node https://review.openstack.org/567469 | 17:48 |
| *** mburned is now known as mburned_out | 17:48 | |
| *** salmankhan has joined #tripleo | 17:50 | |
| *** slaweq has joined #tripleo | 17:50 | |
| *** colonwq has quit IRC | 17:53 | |
| *** pcaruana has joined #tripleo | 17:55 | |
| *** gbarros has joined #tripleo | 17:56 | |
| *** hamzy has joined #tripleo | 17:56 | |
| *** anilvenkata has joined #tripleo | 17:57 | |
| *** elgxl has joined #tripleo | 17:58 | |
| *** matbu has joined #tripleo | 17:59 | |
| *** jtomasek has joined #tripleo | 18:00 | |
| *** ohochman has quit IRC | 18:02 | |
| *** gvrangan has quit IRC | 18:03 | |
| *** gbarros has quit IRC | 18:03 | |
| *** jtomasek has quit IRC | 18:04 | |
| *** dhill_ has joined #tripleo | 18:05 | |
| *** nyechiel_ has joined #tripleo | 18:08 | |
| *** mburned_out is now known as mburned | 18:08 | |
| *** rpioso|eat is now known as rpioso | 18:12 | |
| *** elgxl has quit IRC | 18:15 | |
| *** elgxl has joined #tripleo | 18:16 | |
| *** salmankhan has quit IRC | 18:17 | |
| *** trown|lunch is now known as trown | 18:17 | |
| openstackgerrit | Marius Cornea proposed openstack/tripleo-upgrade master: Pass --ssh-user option during fast forward upgrade run https://review.openstack.org/570245 | 18:17 |
| *** slaweq_ has joined #tripleo | 18:19 | |
| *** elgxl has quit IRC | 18:19 | |
| openstackgerrit | Merged openstack/tripleo-quickstart-extras master: Remove hardcoded cinder v1 option for tempestconf https://review.openstack.org/568776 | 18:20 |
| openstackgerrit | Merged openstack/python-tripleoclient stable/queens: Refactor, fix ansible inventory method parameters for ffwd-upgrade run https://review.openstack.org/567233 | 18:20 |
| openstackgerrit | Merged openstack/tripleo-common stable/queens: Add special 'all' for deploy_on_servers server name to match all https://review.openstack.org/567538 | 18:20 |
| openstackgerrit | Marius Cornea proposed openstack/tripleo-upgrade stable/queens: DNM: Run enable-ssh-admin.sh script before fast forward run https://review.openstack.org/570074 | 18:22 |
| *** slaweq has quit IRC | 18:22 | |
| *** jtomasek has joined #tripleo | 18:22 | |
| openstackgerrit | Marius Cornea proposed openstack/tripleo-upgrade stable/queens: DNM: Run enable-ssh-admin.sh script before fast forward run https://review.openstack.org/570074 | 18:24 |
| *** matbu has quit IRC | 18:25 | |
| *** jtomasek has quit IRC | 18:26 | |
| mcornea | EmilienM: hi could you please check https://review.openstack.org/#/c/570091/ when you get a minute? | 18:28 |
| *** ohochman has joined #tripleo | 18:28 | |
| *** alee has joined #tripleo | 18:30 | |
| *** pkovar has quit IRC | 18:31 | |
| *** slaweq_ has quit IRC | 18:34 | |
| *** jtomasek has joined #tripleo | 18:34 | |
| *** hamzy has quit IRC | 18:35 | |
| *** slaweq has joined #tripleo | 18:39 | |
| openstackgerrit | Matt Young proposed openstack/tripleo-quickstart master: DNM: testing rhos-13 thirdparty gate https://review.openstack.org/567281 | 18:41 |
| *** mburned is now known as mburned_out | 18:42 | |
| *** nyechiel_ has quit IRC | 18:42 | |
| *** hamzy has joined #tripleo | 18:44 | |
| *** leitan has quit IRC | 18:44 | |
| *** leitan has joined #tripleo | 18:44 | |
| openstackgerrit | Damien Ciabrini proposed openstack/puppet-tripleo master: Disallow SSLv2, SSLv3 and TLS1.0 in rabbitmq for FedRAMP compliance https://review.openstack.org/567443 | 18:45 |
| *** jtomasek has quit IRC | 18:46 | |
| *** zaneb has joined #tripleo | 18:47 | |
| *** leitan has quit IRC | 18:49 | |
| *** loacker has joined #tripleo | 18:52 | |
| *** slaweq_ has joined #tripleo | 18:53 | |
| *** Guest75989 has quit IRC | 18:54 | |
| *** slaweq has quit IRC | 18:56 | |
| *** zaneb has quit IRC | 18:58 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/ocata: Enhance completion message when upgrading non controller nodes https://review.openstack.org/569794 | 19:01 |
| *** fragatina has quit IRC | 19:01 | |
| *** matbu has joined #tripleo | 19:02 | |
| *** trwyth has joined #tripleo | 19:03 | |
| *** rlandy|rover is now known as rlandy|rover|bia | 19:04 | |
| *** edmondsw has joined #tripleo | 19:04 | |
| *** dpeacock has quit IRC | 19:04 | |
| *** rlandy|rover|bia is now known as rlandy|rvr|biab | 19:04 | |
| *** jaganathan has quit IRC | 19:06 | |
| *** salmankhan has joined #tripleo | 19:07 | |
| *** matbu has quit IRC | 19:08 | |
| *** anilvenkata has quit IRC | 19:09 | |
| *** dpeacock has joined #tripleo | 19:09 | |
| *** leanderthal has quit IRC | 19:13 | |
| *** gkadam has quit IRC | 19:13 | |
| *** edmondsw_ has joined #tripleo | 19:14 | |
| *** jaganathan has joined #tripleo | 19:14 | |
| openstackgerrit | Alex Schultz proposed openstack/tripleo-heat-templates master: Fix 3node deployment https://review.openstack.org/570252 | 19:15 |
| *** edmondsw has quit IRC | 19:16 | |
| *** trown is now known as trown|brb | 19:22 | |
| *** trwyth has quit IRC | 19:22 | |
| *** bdodd is now known as bdodd|afk | 19:23 | |
| *** bdodd|afk is now known as bdodd | 19:27 | |
| *** radeks has joined #tripleo | 19:27 | |
| *** nyechiel_ has joined #tripleo | 19:28 | |
| *** morazi has joined #tripleo | 19:29 | |
| *** radeks_ has joined #tripleo | 19:29 | |
| *** hamzy has quit IRC | 19:30 | |
| *** slaweq_ has quit IRC | 19:30 | |
| *** edmondsw_ has quit IRC | 19:31 | |
| *** edmondsw has joined #tripleo | 19:32 | |
| *** radeks has quit IRC | 19:33 | |
| *** loacker has quit IRC | 19:34 | |
| *** dhill_ has quit IRC | 19:35 | |
| *** edmondsw_ has joined #tripleo | 19:36 | |
| *** edmondsw has quit IRC | 19:36 | |
| *** alee has quit IRC | 19:38 | |
| *** edmondsw_ has quit IRC | 19:41 | |
| openstackgerrit | Merged openstack/python-tripleoclient master: Update HostnameMap generation https://review.openstack.org/567951 | 19:46 |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/queens: Use str_replace for known_hosts https://review.openstack.org/569909 | 19:46 |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/ocata: Don't use crudini to get rhn server https://review.openstack.org/569180 | 19:46 |
| openstackgerrit | Merged openstack/tripleo-common stable/queens: Add validation for empty file system backup on Undercloud https://review.openstack.org/568250 | 19:46 |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/queens: Fix cinder-backup image wrangling on update https://review.openstack.org/569146 | 19:46 |
| *** trown|brb is now known as trown | 19:48 | |
| *** mburned_out is now known as mburned | 19:55 | |
| *** zshi has quit IRC | 19:56 | |
| openstackgerrit | Merged openstack/tripleo-common stable/queens: Persist package update ansible logs https://review.openstack.org/565853 | 19:57 |
| openstackgerrit | Merged openstack/python-tripleoclient stable/queens: Update Heat stack on update/upgrade/ffwd converge https://review.openstack.org/567243 | 19:57 |
| openstackgerrit | Merged openstack/tripleo-validations stable/queens: Fail if the Overcloud inventory is empty https://review.openstack.org/566881 | 19:57 |
| openstackgerrit | Merged openstack/python-tripleoclient stable/queens: Add .deployment.v1.deploy_on_servers to ffwd-upgrade prepare https://review.openstack.org/568604 | 19:57 |
| openstackgerrit | Merged openstack/python-tripleoclient stable/queens: Add exclude-path to undercloud backups https://review.openstack.org/568210 | 19:57 |
| openstackgerrit | Merged openstack/tripleo-common stable/queens: Find deployment name in config download https://review.openstack.org/567249 | 19:57 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Don't delete br-ex on upgrade in multinode jobs https://review.openstack.org/569781 | 19:57 |
| openstackgerrit | Merged openstack/tripleo-ui master: Reinstate logout link id https://review.openstack.org/560992 | 19:57 |
| openstackgerrit | Merged openstack/tripleo-ui master: Add an id to the DropdownToggle in UserDropdown https://review.openstack.org/561211 | 19:57 |
| *** nyechiel_ has quit IRC | 20:01 | |
| *** pchavva has quit IRC | 20:06 | |
| *** salmankhan has quit IRC | 20:07 | |
| *** rlandy|rvr|biab is now known as rlandy|rover | 20:08 | |
| *** slaweq_ has joined #tripleo | 20:10 | |
| *** holser__ has joined #tripleo | 20:11 | |
| *** slaweq_ has quit IRC | 20:15 | |
| *** alee has joined #tripleo | 20:20 | |
| *** hamzy has joined #tripleo | 20:22 | |
| *** eck` is now known as eck`gone | 20:23 | |
| *** eck`gone is now known as eck` | 20:24 | |
| *** trown is now known as trown|outtypewww | 20:26 | |
| *** noslzzp has joined #tripleo | 20:27 | |
| *** mcornea has quit IRC | 20:28 | |
| *** mvenesio_ has quit IRC | 20:28 | |
| *** pcaruana has quit IRC | 20:28 | |
| *** mvenesio has joined #tripleo | 20:29 | |
| *** raildo has quit IRC | 20:29 | |
| *** noslzzp_ has joined #tripleo | 20:29 | |
| *** noslzzp has quit IRC | 20:30 | |
| *** lblanchard1 has quit IRC | 20:32 | |
| *** mvenesio has quit IRC | 20:33 | |
| *** slaweq has joined #tripleo | 20:34 | |
| *** noslzzp_ has quit IRC | 20:35 | |
| *** nyechiel_ has joined #tripleo | 20:39 | |
| *** mcornea has joined #tripleo | 20:42 | |
| *** mburned is now known as mburned_out | 20:46 | |
| *** morazi has quit IRC | 20:47 | |
| *** edmondsw has joined #tripleo | 20:49 | |
| *** bdodd is now known as bdodd|afk | 20:49 | |
| *** slaweq has quit IRC | 20:49 | |
| *** slaweq has joined #tripleo | 20:51 | |
| stevebaker | morning | 20:52 |
| *** edmondsw has quit IRC | 20:53 | |
| mwhahaha | ahoy | 20:53 |
| openstackgerrit | Ryan Brady proposed openstack/tripleo-common master: [WIP]Fixes ordering of environment files in a deployment plan https://review.openstack.org/570215 | 20:55 |
| *** bdodd|afk is now known as bdodd | 20:55 | |
| *** wolverineav has quit IRC | 20:56 | |
| *** wolverineav has joined #tripleo | 20:56 | |
| *** bdodd is now known as bdodd|afk | 20:56 | |
| *** bdodd|afk is now known as bdodd | 20:56 | |
| openstackgerrit | Alan Bishop proposed openstack/puppet-tripleo master: Add missing parameters for Cinder Netapp backend https://review.openstack.org/570256 | 20:57 |
| *** rbrady has joined #tripleo | 21:00 | |
| *** wolverineav has quit IRC | 21:00 | |
| *** radeks_ has quit IRC | 21:00 | |
| *** rbrady is now known as Guest19060 | 21:01 | |
| *** zshi has joined #tripleo | 21:01 | |
| *** alee has quit IRC | 21:03 | |
| *** fragatina has joined #tripleo | 21:03 | |
| *** slaweq_ has joined #tripleo | 21:05 | |
| *** abishop has quit IRC | 21:08 | |
| *** slaweq has quit IRC | 21:08 | |
| *** trwyth has joined #tripleo | 21:09 | |
| *** edmondsw has joined #tripleo | 21:09 | |
| *** nyechiel_ has quit IRC | 21:09 | |
| *** morazi has joined #tripleo | 21:10 | |
| *** vpickard is now known as vpickard_ | 21:10 | |
| *** gfidente|afk has quit IRC | 21:12 | |
| *** nyechiel_ has joined #tripleo | 21:20 | |
| *** nyechiel_ has quit IRC | 21:25 | |
| *** slaweq_ has quit IRC | 21:30 | |
| *** edmondsw has quit IRC | 21:33 | |
| *** wolverineav has joined #tripleo | 21:34 | |
| *** bdodd is now known as bdodd|afk | 21:36 | |
| *** bfournie has quit IRC | 21:38 | |
| *** bfournie has joined #tripleo | 21:38 | |
| *** wolverineav has quit IRC | 21:38 | |
| *** bfournie has quit IRC | 21:38 | |
| *** matbu has joined #tripleo | 21:38 | |
| *** holser__ has quit IRC | 21:39 | |
| EmilienM | mcornea: of course | 21:39 |
| *** zaneb has joined #tripleo | 21:41 | |
| *** slaweq has joined #tripleo | 21:41 | |
| *** pradk has quit IRC | 21:45 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Remove UpgradeRemoveUnusedPackages from undercloud-upgrade-prepare.yaml https://review.openstack.org/569323 | 21:45 |
| *** zaneb has quit IRC | 21:47 | |
| *** alee has joined #tripleo | 21:48 | |
| *** gunix has left #tripleo | 21:49 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Fix upgrade_tasks for TripleO UI https://review.openstack.org/570237 | 21:51 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Fix upgrade_tasks for TripleO UI https://review.openstack.org/570237 | 21:51 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: upgrade: remove packages at step 3 https://review.openstack.org/570116 | 21:51 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: nova-ironic: add support for post-upgrade cleanup https://review.openstack.org/570241 | 21:51 |
| *** zaneb has joined #tripleo | 21:53 | |
| *** morazi has quit IRC | 21:56 | |
| stevebaker | EmilienM: hey, do you know what creates br-ctlplane during containerised undercloud install? All I see is instack-undercloud running os-net-config | 21:58 |
| dsneddon | stevebaker, The OVS bridge still gets created outside of the container, because we use "host networking". The problem is that you still can't attach containers directly to VLANs inside the container, so the container host has to be attached to the VLAN through the bridge, and the container inherits that. | 22:01 |
| EmilienM | stevebaker: hey oh hi | 22:01 |
| EmilienM | https://github.com/openstack/tripleo-heat-templates/blob/e64c10b9c13188f37e6f122475fe02280eaa6686/net-config-undercloud.j2.yaml#L63 | 22:02 |
| EmilienM | the containerized undercloud uses this net-config | 22:02 |
| EmilienM | in fact in CI we use this one : https://github.com/openstack/tripleo-heat-templates/blob/e64c10b9c13188f37e6f122475fe02280eaa6686/ci/common/net-config-simple-bridge.yaml#L50 | 22:02 |
| *** matbu has quit IRC | 22:02 | |
| stevebaker | dsneddon: this is for the undercloud container registry, which binds to the ip on br-ctlplane | 22:02 |
| stevebaker | EmilienM: ok, that sounds like something else which needs to happen during undercloud init | 22:03 |
| dsneddon | stevebaker, I thought you were asking where br-ctlplane is created, and that happens in the os-net-config template that EmilienM pasted. | 22:03 |
| *** noslzzp has joined #tripleo | 22:06 | |
| EmilienM | stevebaker: lol nice one :P | 22:07 |
| EmilienM | stevebaker: network init :) | 22:07 |
| stevebaker | ugh, can't we just bind the container registry to *? | 22:08 |
| EmilienM | stevebaker: on 0.0.0.0 ? | 22:09 |
| stevebaker | yeah | 22:09 |
| EmilienM | according to doc, yes https://docs.docker.com/registry/deploying/#customize-the-published-port | 22:10 |
| EmilienM | but I haven't tried | 22:10 |
| EmilienM | I can try, give me one sec | 22:10 |
| stevebaker | having to run os-net-config earlier than undercloud install will be a real pita | 22:11 |
| *** noslzzp has quit IRC | 22:11 | |
| EmilienM | yeah it worked | 22:11 |
| EmilienM | /etc/docker-distribution/registry/config.yml | 22:11 |
| EmilienM | I changed to 0.0.0.0 and it worked | 22:11 |
| stevebaker | EmilienM: any objection to 0.0.0.0 being the default in the role instead of localhost? | 22:12 |
| EmilienM | tcp 0 0 undercloud.loca:msgsrvr 0.0.0.0:* LISTEN root 50219 14928/registry | 22:12 |
| EmilienM | so yeah it was well binded | 22:12 |
| EmilienM | as msgsrvr is 8787 port in /etc/services | 22:12 |
| EmilienM | stevebaker: I'm fine | 22:13 |
| stevebaker | cool | 22:13 |
| EmilienM | mwhahaha: did you have a chance to read our conversation with stevebaker about init steps we want to run before a deploy? | 22:13 |
| EmilienM | in summary, we said: | 22:13 |
| mwhahaha | no when was that? | 22:14 |
| EmilienM | we would like a step, openstack tripleo init, that would do idempotent things like deploy docker, do netconfig | 22:14 |
| EmilienM | and by default it would also run in the deploy in case it wasn't run before | 22:14 |
| EmilienM | mwhahaha: yesterday on IRC around that time or maybe later in the evening I guess | 22:14 |
| openstackgerrit | Steve Baker proposed openstack/ansible-role-container-registry master: Listen to all interfaces by default https://review.openstack.org/570269 | 22:15 |
| mwhahaha | EmilienM: do you mean for standalone? | 22:15 |
| * mwhahaha is unsure context | 22:15 | |
| stevebaker | mwhahaha: the problem I'm trying to solve is that docker and registry needs to be set up before undercloud install, so the undercloud images can be prepared. So we're proposing a "tripleo init" command be added which invokes the container-registry role | 22:16 |
| mwhahaha | i scrolled up and read | 22:17 |
| stevebaker | I'm hoping netconfig can still happen during undercloud install | 22:17 |
| stevebaker | but container images are becoming as foundational as yum, so it needs to be set up | 22:17 |
| mwhahaha | it sounds like we need to extract out the undercloud hostprep but have a hook for container prep things that could be done before the actuald eployment | 22:18 |
| mwhahaha | so i guess the question is what is the use case for this, would we actually customize containers prior to the undercloud install? | 22:19 |
| mwhahaha | or is that only for a standalone context | 22:19 |
| stevebaker | mwhahaha: the end goal is a container-check type thing which does *all* required images for undercloud and overcloud in one hit. The output of this needs to be an environment file which both deploys can use. Thats why doing it during undercloud install doesn't quite fit. Its not obvious why an "undercloud install" command would generate an environment file to be consumed by "overcloud install" | 22:21 |
| *** rcernin has joined #tripleo | 22:21 | |
| mwhahaha | i don't think you can do both in one hit | 22:22 |
| mwhahaha | because you don't know what the overcloud is actually going to deploy at undercloud install time | 22:22 |
| mwhahaha | but from a deployment framework standpoint, being able to execute the container bits at any specific point in the deployment could be leveraged for both cases | 22:22 |
| stevebaker | we could with a bit of oooq refactoring. But maybe it will no longer be practical to filter images by containerised service detection, so it would have to be driven by role data filtering only | 22:23 |
| mwhahaha | what's the use case? CI only? | 22:23 |
| stevebaker | mwhahaha: doing container-check twice for the same images would be quite slow. And yes this is likely a CI use case onely] | 22:24 |
| mwhahaha | if it's not CI only then oooq is not something that should be considered | 22:24 |
| mwhahaha | er that's wrong | 22:25 |
| *** hamzy has quit IRC | 22:25 | |
| *** slaweq has quit IRC | 22:25 | |
| stevebaker | I think its CI only | 22:25 |
| mwhahaha | if it's CI only then oooq is something that could be considered | 22:25 |
| mwhahaha | but if we ever want this to be used by a user (or other downstream process) then we should keep it in tripleo stuff | 22:25 |
| *** rlandy|rover is now known as rlandy|rover|bbl | 22:26 | |
| mwhahaha | you could consider this a standalone thing to standup a seperate repo somewhere, do work, populate it and then just pass it as the source for undercloud/overcloud installations | 22:26 |
| mwhahaha | that would be beneficial if we replaced either the oooq or tripleo containers registry to use something other than docker | 22:27 |
| * mwhahaha wished we used something like rkt which can pull from an http endpoint | 22:27 | |
| stevebaker | actually there might be a way for undercloud and overcloud to do their own prepare, but no duplicate rebuilding to happen. Let me have a think | 22:27 |
| stevebaker | mwhahaha, EmilienM: I need a job where both the undercloud and overcloud are containerised, and they both have a container-check job run on them. Is there such a thing yet? | 22:28 |
| mwhahaha | no | 22:29 |
| mwhahaha | we don't run container-check on containerized undercloud yet | 22:29 |
| mwhahaha | primarily because this problem | 22:29 |
| stevebaker | which job might that be added to? | 22:29 |
| mwhahaha | well if you wanted to tackle the containerized undercloud problem you could just do that first | 22:29 |
| *** alee has quit IRC | 22:29 | |
| mwhahaha | i think that's really the issue | 22:29 |
| mwhahaha | trying to figure out how to populate an outside docker registry with containers then pull it in to the undercloud install | 22:30 |
| mwhahaha | but we provide the container registry file in the undercloud install so it's figuring out how to do that | 22:30 |
| mwhahaha | but then you lose coverage on that part of the installation | 22:30 |
| * mwhahaha wishes for a whiteboard for this conversation | 22:31 | |
| stevebaker | mwhahaha: This sounds like it can all be done with the tripleo-modify-image role being invoked during install. It seems to be working on the overcloud tests https://review.openstack.org/#/c/569866/ https://review.openstack.org/#/c/569696/ | 22:31 |
| *** alee has joined #tripleo | 22:32 | |
| mwhahaha | stevebaker: yea we run that every install, but the problem is where do you stick the output | 22:33 |
| stevebaker | mwhahaha: the output images? in the undercloud registry | 22:33 |
| mwhahaha | stevebaker: right but that doesn't exist until the undercloud is installed | 22:33 |
| *** akrivoka has quit IRC | 22:34 | |
| mwhahaha | stevebaker: so you can't do that for the containerized undercloud install because it doesn't exist yet | 22:34 |
| stevebaker | mwhahaha: ok, let me walk you through the sequence | 22:34 |
| mwhahaha | yea that's why i wanted a whiteboard :D | 22:34 |
| stevebaker | 1. something (oooq) writes out the prepare parameter like this https://review.openstack.org/#/c/569696/3/roles/overcloud-prep-containers/templates/containers-prepare-parameter.yaml.j2 | 22:35 |
| stevebaker | 2. undercloud.conf refers to this file | 22:36 |
| *** edmondsw has joined #tripleo | 22:37 | |
| stevebaker | 3. undercloud install starts, some early thing runs the container-registry role (I think this is missing currently) | 22:37 |
| *** matbu has joined #tripleo | 22:37 | |
| stevebaker | 4. kolla_builder.container_images_prepare_multi is run during tripleo deploy | 22:38 |
| stevebaker | 5. this triggers pull/modify/push to undercloud registry, generates environment parameters pointing to modified images | 22:38 |
| stevebaker | 6. ... | 22:38 |
| stevebaker | 7. profit | 22:38 |
| mwhahaha | so why does 3 need to happen? | 22:39 |
| mwhahaha | hmm ok wait let me think | 22:39 |
| stevebaker | because 4. needs a registry to push to | 22:39 |
| openstackgerrit | Marius Cornea proposed openstack/tripleo-upgrade stable/queens: DNM: Run enable-ssh-admin.sh script before fast forward run https://review.openstack.org/570074 | 22:39 |
| mwhahaha | ok so i think that's my disconnect, wby can't the deploy have 4 injected into it? | 22:39 |
| mwhahaha | like an internal update containers step | 22:40 |
| stevebaker | so either we do 3 early in "undercloud install" or "tripleo deploy", or we create an earlier command like "tripleo init" | 22:40 |
| mwhahaha | stevebaker: i'd rather not because then we lose the customer workflow where the install creates the registry in the undercloud | 22:40 |
| *** mcornea has quit IRC | 22:40 | |
| mwhahaha | so my thought is injecting a step into the deployment that maybe calls the tripleo image prepare would work | 22:41 |
| stevebaker | mwhahaha: one reason is because the running heat which does the tripleo deploy is containerised, so its too late for heat | 22:41 |
| mwhahaha | no | 22:41 |
| *** alee has quit IRC | 22:41 | |
| mwhahaha | we stand up the standalone heat and ansible can call it | 22:41 |
| stevebaker | oh its a host process? | 22:41 |
| mwhahaha | let me describe my thought | 22:41 |
| mwhahaha | yea | 22:41 |
| *** edmondsw has quit IRC | 22:41 | |
| mwhahaha | unless you specify to use a heat container | 22:41 |
| mwhahaha | but we don't | 22:41 |
| stevebaker | I thought we were containerised by default | 22:42 |
| mwhahaha | so no | 22:42 |
| stevebaker | ok, that changes some | 22:42 |
| mwhahaha | the heat we use in the install is a standalone process | 22:42 |
| mwhahaha | there is an option to use a container but that is not currently enabled | 22:42 |
| mwhahaha | because it required docker be pre-setup | 22:42 |
| mwhahaha | which is this problem | 22:42 |
| openstackgerrit | Ryan Brady proposed openstack/python-tripleoclient master: Changes Creation Of overcloudrc File to Call Workflow https://review.openstack.org/545351 | 22:42 |
| mwhahaha | stevebaker: so let me describe what i was thinking | 22:42 |
| mwhahaha | 1) same as yours | 22:42 |
| mwhahaha | 2) same as yours | 22:42 |
| mwhahaha | 3) we update the deployment steps to identify that this configuration is set, inject ansible tasks to perform the prepare after we setup docker in the deployment | 22:43 |
| *** bfournie has joined #tripleo | 22:43 | |
| mwhahaha | 4) these commands push to the container registry we setup during the install | 22:43 |
| mwhahaha | 5) configure continues | 22:43 |
| mwhahaha | so really it makes docker a hostprep step | 22:43 |
| mwhahaha | those aren't containerized | 22:43 |
| stevebaker | well isn't it convenient that ContainerImagePrepare is a heat parameter. I never imagined that heat would consume it for real ;) | 22:44 |
| mwhahaha | and so we'd need a host prep task that would be undercloud specific but it runs the image prepare/mangling | 22:44 |
| mwhahaha | what's missing from the current bits is the ansible needed to run whatever code you have | 22:44 |
| mwhahaha | basically containerized undercloud it's heat generates ansible -> ansible does all the things | 22:45 |
| mwhahaha | the heat instance is a standalone process so we don't need docker up yet | 22:45 |
| stevebaker | mwhahaha: I think all it needs is something in t-h-t running "openstack tripleo container image prepare" straight after the registry is set up | 22:45 |
| mwhahaha | there is support in the framework to docker exec a heat container but that's likely something we would not use in ci | 22:45 |
| mwhahaha | stevebaker: right so we just need to come up with the host prep steps to do that in the framework | 22:45 |
| mwhahaha | stevebaker: for most deploys it would be off (like a customer) but for CI we can enable it and have the containers updated | 22:46 |
| stevebaker | mwhahaha: prepare would always run, but customers generally wouldn't have modify_role things in their ContainerImagePrepare, unless they're building a 3rd-party image on the fly | 22:48 |
| *** lebauce_ has quit IRC | 22:48 | |
| mwhahaha | stevebaker: yea that's what i mean, the modify bits | 22:48 |
| mwhahaha | stevebaker: because the prepare is what we have today where we generate the urls to use the upstream locations | 22:48 |
| mwhahaha | stevebaker: the modify is the pull containers from A, apply B changes, upload to C, update references container locations to C/latest | 22:49 |
| stevebaker | mwhahaha: yes, default prepare data will pull from upstream, but discover versioned tags | 22:49 |
| stevebaker | mwhahaha: ok I'll look into doing this now. Reviews on these would help https://review.openstack.org/#/c/569688/ https://review.openstack.org/#/c/567427/ | 22:50 |
| *** dtrainor has joined #tripleo | 22:51 | |
| stevebaker | oh you reviewed the second | 22:51 |
| stevebaker | mwhahaha: then the one remaining bit to deal with is some logic in oooq to detect images modified for the undercloud so they don't get modified again for the overcloud, but that is an optimisation | 22:53 |
| mwhahaha | stevebaker: so i think we should be able to handle that in the input convention if we like use the tag as a hash of the rpms we're applying or something | 22:54 |
| mwhahaha | stevebaker: but yes, we'd only want to do the adjustment once | 22:54 |
| *** slaweq has joined #tripleo | 22:54 | |
| *** bdodd|afk is now known as bdodd | 22:55 | |
| *** dtrainor has quit IRC | 22:55 | |
| stevebaker | mwhahaha: I'm thinking we should call prepare here instead of creating a whole new "service" file https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/services/docker-registry.yaml#L67 | 23:00 |
| mwhahaha | stevebaker: yes i think that's an OK spot | 23:00 |
| mwhahaha | stevebaker: additional services in heat also has an impact on deployment times | 23:01 |
| * mwhahaha needs to file a bug for that | 23:01 | |
| *** zaneb has quit IRC | 23:02 | |
| stevebaker | mwhahaha: I think that is a known issue for OS::Heat::ResourceChain which is hard to optimise | 23:05 |
| mwhahaha | stevebaker: yea but even if you OS::Heat::None all the services it takes forever | 23:05 |
| mwhahaha | so it's the creation of the heat bits around that | 23:05 |
| mwhahaha | which is why we've gotten progressively slower in deployments because our resource registry has a bunch of services | 23:06 |
| mwhahaha | so for every 3rdparty entry we add time for heat to process those | 23:06 |
| mwhahaha | i tested it on the standalone stuff and it's pretty awful | 23:06 |
| mwhahaha | that's the next thing we need to make dynamic so we don't always include all those services if we don't have to | 23:06 |
| *** noslzzp has joined #tripleo | 23:07 | |
| stevebaker | yep, sounds like a plan | 23:07 |
| openstackgerrit | Alex Schultz proposed openstack/tripleo-heat-templates master: Don't run swift on 3node https://review.openstack.org/570275 | 23:07 |
| EmilienM | mwhahaha: ^ how do you share images? | 23:08 |
| mwhahaha | EmilienM: ? | 23:08 |
| mwhahaha | EmilienM: so it's about populating the local repository during the deployment | 23:08 |
| mwhahaha | EmilienM: So in what I described, we'd actually switch out the heat references to use the undercloud registry, but populated it during the registry setup | 23:09 |
| mwhahaha | EmilienM: This is an optimization as well because it would preload the images into the local docker registry during installation and then we would fail at that point rather than later during say the nova-api container deployment | 23:09 |
| mwhahaha | EmilienM: so the overcloud gets the updated containers for free since they are already loaded in the undercloud | 23:10 |
| mwhahaha | the docker/docker-registry setup is done before any docker launches | 23:10 |
| mwhahaha | because it's a hostprep step | 23:10 |
| *** matbu has quit IRC | 23:10 | |
| mwhahaha | essentially being step1 | 23:10 |
| *** hamzy has joined #tripleo | 23:10 | |
| stevebaker | EmilienM: I think images can be shared, either with a change in tripleo-common or oooq | 23:11 |
| mwhahaha | so at step2 we'd have docker available and the containers loaded into the local repo | 23:11 |
| mwhahaha | it's likely an oooq where we populate the container images references ahead of time | 23:11 |
| mwhahaha | or you could add something into the image prepare to layer a query against the underclodu registry first, then fall back to upstream | 23:12 |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates master: Add acl to paths that are shared among related neutron processes https://review.openstack.org/567655 | 23:12 |
| mwhahaha | kinda like a yum-priorites but for containers | 23:12 |
| mwhahaha | anyway | 23:12 |
| * mwhahaha wanders off | 23:12 | |
| EmilienM | mwhahaha: I was talking about 3nodes | 23:16 |
| *** lebauce_ has joined #tripleo | 23:19 | |
| *** noslzzp has quit IRC | 23:20 | |
| *** slaweq has quit IRC | 23:20 | |
| *** toure is now known as toure|gone | 23:21 | |
| openstackgerrit | Marius Cornea proposed openstack/tripleo-quickstart master: DNM: Remove ceph from fast forward upgrade job https://review.openstack.org/570278 | 23:24 |
| openstackgerrit | Marius Cornea proposed openstack/tripleo-upgrade stable/queens: DNM: Run enable-ssh-admin.sh script before fast forward run https://review.openstack.org/570074 | 23:25 |
| openstackgerrit | Marius Cornea proposed openstack/tripleo-quickstart master: DNM: Remove ceph from fast forward upgrade job https://review.openstack.org/570278 | 23:33 |
| openstackgerrit | Marius Cornea proposed openstack/tripleo-upgrade stable/queens: DNM: Run enable-ssh-admin.sh script before fast forward run https://review.openstack.org/570074 | 23:33 |
| *** nyechiel_ has joined #tripleo | 23:33 | |
| *** slaweq has joined #tripleo | 23:34 | |
| openstackgerrit | Merged openstack/python-tripleoclient master: Fallback the default roles file for UC config https://review.openstack.org/569992 | 23:35 |
| *** slaweq_ has joined #tripleo | 23:37 | |
| *** edmondsw has joined #tripleo | 23:37 | |
| *** slaweq has quit IRC | 23:40 | |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Save and close stdout on exit https://review.openstack.org/570094 | 23:40 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Add log directory option to functional tests https://review.openstack.org/570095 | 23:40 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Rename output log files https://review.openstack.org/570096 | 23:40 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: WIP: native zuulv3 tests https://review.openstack.org/566779 | 23:41 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Reduce path length in PS4 for debug https://review.openstack.org/570282 | 23:41 |
| *** pmannidi has joined #tripleo | 23:41 | |
| *** rh-jelabarre has quit IRC | 23:45 | |
| *** myoung is now known as myoung|off | 23:46 | |
| *** nyechiel_ has quit IRC | 23:47 | |
| *** bdodd is now known as bdodd|afk | 23:47 | |
| *** slaweq_ has quit IRC | 23:49 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: WIP - add missing services for upgrade cleanup https://review.openstack.org/570283 | 23:54 |
| *** trwyth has quit IRC | 23:54 | |
| *** bdodd|afk is now known as bdodd | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!