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 | RedHat RDO CI proposed openstack/tripleo-heat-templates stable/ocata: GATE CHECK for TripleO https://review.openstack.org/564291 | 00:00 |
*** bdodd is now known as bdodd|afk | 00:06 | |
*** slaweq has joined #tripleo | 00:11 | |
*** zaneb has joined #tripleo | 00:15 | |
*** slaweq has quit IRC | 00:15 | |
*** rpioso is now known as rpioso|afk | 00:18 | |
*** zaneb has quit IRC | 00:25 | |
openstackgerrit | Harald Jensås proposed openstack/python-tripleoclient master: Deployed Server - portmap should match neutron API https://review.openstack.org/569789 | 00:27 |
openstackgerrit | Harald Jensås proposed openstack/tripleo-heat-templates master: Get ctlplane interface config from the subnet https://review.openstack.org/569547 | 00:27 |
*** edmondsw has quit IRC | 00:28 | |
*** pblaho has quit IRC | 00:29 | |
*** pblaho has joined #tripleo | 00:33 | |
*** edmondsw has joined #tripleo | 00:35 | |
*** edmondsw has quit IRC | 00:39 | |
*** hamzy has quit IRC | 00:51 | |
*** mdnadeem has joined #tripleo | 01:08 | |
*** noslzzp has joined #tripleo | 01:08 | |
*** bdodd|afk is now known as bdodd | 01:09 | |
*** edmondsw has joined #tripleo | 01:11 | |
*** slaweq has joined #tripleo | 01:11 | |
*** mdnadeem has quit IRC | 01:13 | |
*** slaweq has quit IRC | 01:15 | |
*** edmondsw has quit IRC | 01:15 | |
*** psahoo has joined #tripleo | 01:19 | |
*** colonwq has joined #tripleo | 01:24 | |
*** matbu has joined #tripleo | 01:28 | |
*** mdnadeem has joined #tripleo | 01:30 | |
*** matbu has quit IRC | 01:42 | |
*** gvrangan has joined #tripleo | 01:43 | |
*** gvrangan has quit IRC | 01:44 | |
*** trwyth has joined #tripleo | 01:53 | |
*** zshi has quit IRC | 01:57 | |
*** trwyth has quit IRC | 01:57 | |
*** trwyth has joined #tripleo | 01:58 | |
*** skramaja has joined #tripleo | 02:09 | |
openstackgerrit | Saravanan KR proposed openstack/python-tripleoclient master: DeployIdentifier should be provided for any parameter actions https://review.openstack.org/569765 | 02:20 |
*** trwyth1 has joined #tripleo | 02:21 | |
*** dxiri has quit IRC | 02:21 | |
*** trwyth has quit IRC | 02:23 | |
openstackgerrit | Marius Cornea proposed openstack/tripleo-quickstart master: Remove ceph from fast forward upgrade job https://review.openstack.org/570278 | 02:23 |
openstackgerrit | Marius Cornea proposed openstack/tripleo-quickstart master: Remove ceph from fast forward upgrade job https://review.openstack.org/570278 | 02:24 |
*** rlandy|rover|bbl is now known as rlandy|rover | 02:26 | |
*** trwyth1 has left #tripleo | 02:27 | |
openstackgerrit | Steve Baker proposed openstack/tripleo-common master: Add a dry-run behaviour to image image_uploader https://review.openstack.org/570300 | 02:34 |
*** gvrangan has joined #tripleo | 02:35 | |
*** gvrangan has quit IRC | 02:39 | |
*** ramishra has joined #tripleo | 02:45 | |
*** psachin has joined #tripleo | 02:45 | |
*** hasanq has joined #tripleo | 02:46 | |
*** hasanq has quit IRC | 02:48 | |
*** rlandy|rover has quit IRC | 03:27 | |
*** skramaja has quit IRC | 03:43 | |
*** skramaja has joined #tripleo | 03:43 | |
openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Save and close stdout on exit https://review.openstack.org/570094 | 03:46 |
openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Add log directory option to functional tests https://review.openstack.org/570095 | 03:46 |
openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Rename output log files https://review.openstack.org/570096 | 03:46 |
openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: WIP: native zuulv3 tests https://review.openstack.org/566779 | 03:46 |
*** fragatina has quit IRC | 03:52 | |
*** fragatina has joined #tripleo | 03:52 | |
*** hamzy has joined #tripleo | 03:54 | |
*** fragatina has quit IRC | 03:55 | |
*** fragatina has joined #tripleo | 03:56 | |
*** fragatina has quit IRC | 03:58 | |
Tengu | hello there | 03:58 |
*** fragatina has joined #tripleo | 03:58 | |
*** rcernin has quit IRC | 03:58 | |
*** hamzy has quit IRC | 03:59 | |
*** pdeore has joined #tripleo | 04:00 | |
*** gvrangan has joined #tripleo | 04:00 | |
*** hamzy has joined #tripleo | 04:01 | |
*** fragatina has quit IRC | 04:02 | |
*** gvrangan has quit IRC | 04:03 | |
*** shreshtha has joined #tripleo | 04:07 | |
*** udesale has joined #tripleo | 04:08 | |
*** gbarros has joined #tripleo | 04:15 | |
*** rcernin has joined #tripleo | 04:17 | |
*** cshastri has joined #tripleo | 04:18 | |
*** noslzzp has quit IRC | 04:19 | |
*** rcernin has quit IRC | 04:23 | |
*** rcernin has joined #tripleo | 04:23 | |
*** fragatin_ has joined #tripleo | 04:24 | |
*** pcaruana has joined #tripleo | 04:24 | |
*** pcaruana has quit IRC | 04:29 | |
*** tzumainn has quit IRC | 04:30 | |
jaosorior | Good morning! | 04:31 |
*** pdeore has quit IRC | 04:34 | |
*** openstackgerrit has quit IRC | 04:34 | |
Tengu | how are you to day, jaosorior ? | 04:35 |
*** d0ugal has quit IRC | 04:35 | |
jaosorior | pretty good! | 04:35 |
jaosorior | you? | 04:35 |
jaosorior | so, this time I actually had breakfast :D | 04:35 |
*** ykarel|afk has joined #tripleo | 04:35 | |
Tengu | :D | 04:36 |
Tengu | I'm building a docker container in order to run tmate on a more.. hmm. closed/secured/foo env :) | 04:36 |
jaosorior | you should write a blog post about that | 04:39 |
jaosorior | I would like something similar but haven't dedicated the time | 04:39 |
Tengu | :) | 04:40 |
*** shreshtha_ has joined #tripleo | 04:40 | |
Tengu | let's say I decide to take the time for the sake of my laptop security :) | 04:40 |
*** radeks_ has joined #tripleo | 04:40 | |
*** shreshtha has quit IRC | 04:42 | |
*** janki has joined #tripleo | 04:42 | |
jaosorior | Tengu: let me know when you have time so we can continue the work :D | 04:43 |
Tengu | jaosorior: I'm currently (re⁴)deploying the quickstart with a non-ipv6 profiles, as the ipv6, although allowing to validate your patch yesterday, can't be properly used on my ipv4-only network :). but more than that, I can work ;). | 04:45 |
*** shreshtha has joined #tripleo | 04:46 | |
*** shreshtha is now known as shreshtha-away | 04:46 | |
*** gbarros has quit IRC | 04:47 | |
jaosorior | lets get back to business when you're done deploying | 04:47 |
Tengu | jaosorior: ok :). That should let me create my container :). | 04:48 |
Tengu | have some issues with tmate losing its server. | 04:48 |
*** shreshtha_ has quit IRC | 04:49 | |
*** pdeore has joined #tripleo | 04:50 | |
*** shreshtha_ has joined #tripleo | 04:51 | |
*** openstackgerrit has joined #tripleo | 04:53 | |
openstackgerrit | yatin proposed openstack/tripleo-heat-templates master: Use /usr/bin/env python to autodetect python https://review.openstack.org/570213 | 04:53 |
*** shreshtha-away has quit IRC | 04:54 | |
*** rcernin has quit IRC | 04:56 | |
openstackgerrit | Merged openstack-infra/tripleo-ci master: Pass dlrn_hash from the releases file https://review.openstack.org/569958 | 04:58 |
EmilienM | hello folks | 04:59 |
EmilienM | jaosorior: hey I have some reviews if you have time, they aren't big | 04:59 |
jaosorior | EmilienM: sure | 04:59 |
*** shreshtha_ has quit IRC | 04:59 | |
EmilienM | jaosorior: https://review.openstack.org/#/c/570237/ and the ones on top | 04:59 |
EmilienM | jaosorior: except the WIP | 05:00 |
EmilienM | thanks | 05:00 |
EmilienM | let me know if you need any review | 05:00 |
*** mrunge has quit IRC | 05:00 | |
jaosorior | EmilienM: will do, thanks | 05:01 |
Tengu | jaosorior: found out the issue: https://hub.docker.com/r/sigma/tmate/ - apparently needs "privilege" mode. | 05:01 |
*** mrunge has joined #tripleo | 05:02 | |
*** d0ugal has joined #tripleo | 05:03 | |
*** radeks_ has quit IRC | 05:04 | |
openstackgerrit | Merged openstack/tripleo-upgrade stable/queens: Move SSL undercloud validation out of UC script create tasks. https://review.openstack.org/570130 | 05:07 |
*** d0ugal has quit IRC | 05:10 | |
*** ccamacho has quit IRC | 05:10 | |
*** ratailor has joined #tripleo | 05:10 | |
*** anilvenkata has joined #tripleo | 05:11 | |
*** links has joined #tripleo | 05:13 | |
*** d0ugal_ has joined #tripleo | 05:14 | |
*** d0ugal__ has joined #tripleo | 05:23 | |
*** links has quit IRC | 05:24 | |
*** d0ugal_ has quit IRC | 05:24 | |
*** links has joined #tripleo | 05:25 | |
*** nyechiel_ has joined #tripleo | 05:33 | |
Tengu | \o/ | 05:35 |
Tengu | jaosorior: I have a secured tmate in a container «o/ | 05:35 |
*** links has quit IRC | 05:35 | |
Tengu | hit a couple of issues with docker, especially since the "-ti" option for exec doesn't actually provide a real TTY :] | 05:35 |
*** links has joined #tripleo | 05:36 | |
*** links has joined #tripleo | 05:36 | |
*** gbarros has joined #tripleo | 05:37 | |
Tengu | wow. it's actually working. | 05:38 |
*** pdeore_ has joined #tripleo | 05:39 | |
EmilienM | we should change the /topic to it | 05:39 |
*** udesale_ has joined #tripleo | 05:39 | |
*** EmilienM changes topic to "Welcome to Rocky. CI status: GREEEEEEEEN "wow it's actually working" | http://tripleo.org/ | https://docs.openstack.org/tripleo-docs/latest" | 05:40 | |
Tengu | EmilienM: XD I wasn't speaking about CI | 05:40 |
Tengu | but actually, if it's working, it's indeed a "wow-effect" :) | 05:40 |
EmilienM | :D | 05:41 |
EmilienM | I have the wow effect every day here | 05:41 |
*** pdeore has quit IRC | 05:41 | |
Tengu | :) can't wait to get your summary email | 05:42 |
*** udesale has quit IRC | 05:42 | |
*** dparkes has quit IRC | 05:42 | |
*** links has quit IRC | 05:42 | |
EmilienM | Tengu: it'll have to wait, i'm on PTO after the summit, taking a bit of time off | 05:43 |
Tengu | EmilienM: whaaat? PTO? nah, you don't have the right to PTO | 05:43 |
Tengu | :) | 05:43 |
EmilienM | :-O | 05:45 |
EmilienM | my dad says "if you don't have the right, take the left" | 05:45 |
EmilienM | I'll follow his advice on this one ;-) | 05:45 |
*** links has joined #tripleo | 05:46 | |
Tengu | ^^ | 05:48 |
jaosorior | I'll get my PTO in July :D 3 weeks off. It'll be great! | 05:48 |
*** jaganathan has quit IRC | 05:50 | |
*** jaganathan has joined #tripleo | 05:50 | |
*** marios has joined #tripleo | 05:56 | |
*** pgadiya has joined #tripleo | 05:58 | |
*** saneax has joined #tripleo | 05:59 | |
*** yamahata has joined #tripleo | 06:03 | |
*** links has quit IRC | 06:03 | |
*** links has joined #tripleo | 06:04 | |
*** jfrancoa has joined #tripleo | 06:05 | |
*** gbarros has quit IRC | 06:06 | |
*** saneax has quit IRC | 06:09 | |
*** elgxl has joined #tripleo | 06:12 | |
*** radek__ has joined #tripleo | 06:14 | |
*** radeks_ has joined #tripleo | 06:14 | |
*** kopecmartin has joined #tripleo | 06:15 | |
*** quiquell|off is now known as quiquell | 06:15 | |
*** links has quit IRC | 06:17 | |
*** olap has joined #tripleo | 06:18 | |
*** olap has quit IRC | 06:18 | |
*** olap has joined #tripleo | 06:18 | |
*** holser__ has joined #tripleo | 06:21 | |
*** links has joined #tripleo | 06:22 | |
openstackgerrit | Emilien Macchi proposed openstack/python-tripleoclient master: tripleo_deploy/upgrade: fail of upgrade_tasks don't return 0 https://review.openstack.org/570321 | 06:28 |
*** dparkes has joined #tripleo | 06:29 | |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: upgrade: remove packages at step 3 https://review.openstack.org/570116 | 06:30 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: nova-ironic: add support for post-upgrade cleanup https://review.openstack.org/570241 | 06:31 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: WIP - add missing services for upgrade cleanup https://review.openstack.org/570283 | 06:31 |
*** dciabrin has joined #tripleo | 06:32 | |
*** dciabrin_ has quit IRC | 06:33 | |
EmilienM | skramaja: thx for your review :) | 06:34 |
skramaja | welcome. i had a small nit in one of your review.. | 06:35 |
EmilienM | skramaja: I think this is good now? | 06:35 |
*** udesale__ has joined #tripleo | 06:36 | |
skramaja | checking.. | 06:36 |
Tengu | if anyone is using tmate in order to share session with others: https://github.com/cjeanner/docker-tmate-client | 06:36 |
skramaja | yes EmilienM, done. | 06:37 |
EmilienM | skramaja: ++ thanks | 06:37 |
EmilienM | alright I'm off to bed :) | 06:37 |
skramaja | sure. good night. | 06:37 |
*** links has quit IRC | 06:38 | |
*** links has joined #tripleo | 06:38 | |
*** udesale_ has quit IRC | 06:38 | |
EmilienM | thanks! | 06:39 |
*** pdeore__ has joined #tripleo | 06:42 | |
*** links has quit IRC | 06:42 | |
*** rcernin has joined #tripleo | 06:43 | |
*** alee has joined #tripleo | 06:43 | |
*** pdeore_ has quit IRC | 06:45 | |
*** lvdombrkr has joined #tripleo | 06:47 | |
*** matbu has joined #tripleo | 06:47 | |
*** links has joined #tripleo | 06:52 | |
*** _jbadiapa is now known as jbadiapa | 06:53 | |
*** pcaruana has joined #tripleo | 06:54 | |
*** ukalifon has joined #tripleo | 06:59 | |
*** ccamacho has joined #tripleo | 07:03 | |
*** avivgt has joined #tripleo | 07:04 | |
*** cylopez has joined #tripleo | 07:07 | |
*** janki has quit IRC | 07:07 | |
*** ccamacho has quit IRC | 07:07 | |
*** ccamacho has joined #tripleo | 07:08 | |
*** avivgt has quit IRC | 07:08 | |
*** aufi has joined #tripleo | 07:10 | |
*** ffiore has joined #tripleo | 07:16 | |
*** masco has joined #tripleo | 07:20 | |
*** bogdando has joined #tripleo | 07:20 | |
*** suuuper has joined #tripleo | 07:21 | |
*** tesseract has joined #tripleo | 07:23 | |
openstackgerrit | Lukas Bezdicka proposed openstack/tripleo-heat-templates master: Upgrades: Refactor httpd_enabled variable https://review.openstack.org/555781 | 07:25 |
openstackgerrit | Lukas Bezdicka proposed openstack/tripleo-heat-templates master: Upgrades: Refactor playbooks to set facts https://review.openstack.org/562294 | 07:25 |
*** cylopez has left #tripleo | 07:28 | |
*** ssbarnea_ has joined #tripleo | 07:28 | |
sileht | hi, https://review.openstack.org/#/c/569956/ have two +2, but no +A, any reason ? | 07:29 |
*** ykarel|afk is now known as ykarel|lunch | 07:30 | |
openstackgerrit | Jose Luis Franco proposed openstack/tripleo-upgrade stable/queens: DNM: Test pike->queens upgrades. https://review.openstack.org/563616 | 07:32 |
*** waleedm has joined #tripleo | 07:36 | |
*** jpena|off is now known as jpena | 07:37 | |
*** kbyrne has quit IRC | 07:45 | |
*** nyechiel_ has quit IRC | 07:46 | |
openstackgerrit | Bogdan Dobrelya proposed openstack/puppet-tripleo stable/queens: Rework GDPR compliant logrotate config https://review.openstack.org/570328 | 07:48 |
*** kbyrne has joined #tripleo | 07:49 | |
*** udesale__ is now known as udesale | 07:53 | |
*** aufi_ has joined #tripleo | 07:53 | |
openstackgerrit | Bogdan Dobrelya proposed openstack/tripleo-quickstart-extras master: Fix path and wire-in UC deploy role data file https://review.openstack.org/563988 | 07:55 |
*** aufi has quit IRC | 07:55 | |
openstackgerrit | Bogdan Dobrelya proposed openstack/python-tripleoclient master: Store UC hosts/portmaps heat params in output_dir https://review.openstack.org/564135 | 07:56 |
bogdando | o/ folks. I need help with debugging https://logs.rdoproject.org/16/566916/3/openstack-check/gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset035-master/Z219aad2e271a40d3ac6ad2d039999d31/undercloud/home/jenkins/overcloud_deploy.log.txt.gz#_2018-05-23_09_22_06 . The reproduce script does not reproduce that error, it passes on RDO cloud | 07:57 |
*** jistr is now known as jistr|iview | 07:57 | |
bogdando | that's https://review.openstack.org/#/c/566916/ | 07:58 |
*** sshnaidm|off is now known as sshnaidm | 07:59 | |
*** janki has joined #tripleo | 08:09 | |
*** ykarel|lunch is now known as ykarel | 08:12 | |
openstackgerrit | Dmitry Tantsur proposed openstack/tripleo-common master: Fix handling hardware types and drivers when generating fencing parameters https://review.openstack.org/567896 | 08:14 |
openstackgerrit | Cédric Jeanneret proposed openstack/tripleo-common master: Set FixedIp by default for public-facing VIP https://review.openstack.org/569818 | 08:14 |
*** gkadam has joined #tripleo | 08:17 | |
openstackgerrit | Dmitry Tantsur proposed openstack/python-tripleoclient master: Stop supporting the removed pxe_ssh driver in `generate fencing parameters` https://review.openstack.org/570349 | 08:21 |
Tengu | hey there, PTAL? :) https://review.openstack.org/#/c/570209/ | 08:22 |
*** jaganathan has quit IRC | 08:22 | |
*** anilvenkata has quit IRC | 08:28 | |
*** anilvenkata has joined #tripleo | 08:29 | |
*** jaganathan has joined #tripleo | 08:29 | |
*** avivgt has joined #tripleo | 08:30 | |
openstackgerrit | Lukas Bezdicka proposed openstack/tripleo-heat-templates master: Upgrades: Refactor playbooks to set facts https://review.openstack.org/562294 | 08:35 |
*** rcernin has quit IRC | 08:36 | |
*** b00tcat has joined #tripleo | 08:41 | |
*** derekh has joined #tripleo | 08:42 | |
openstackgerrit | Cédric Jeanneret proposed openstack/tripleo-common master: TLS by default for the overcloud for VIPs and FQDNs https://review.openstack.org/569364 | 08:50 |
*** amoralej|off is now known as amoralej | 08:55 | |
*** shreshtha has joined #tripleo | 08:58 | |
*** jistr|iview is now known as jistr | 09:07 | |
jaosorior | shardy: where is user-environment created? | 09:07 |
*** Tengu has quit IRC | 09:07 | |
*** tdasilva has quit IRC | 09:08 | |
jaosorior | jistr: setting EnablePublicTLS: true in t-h-t didn't help :/ cause we don't have access to the variables set in the resource registry from the environment nor the user environment :/ | 09:08 |
*** Hazelesque has quit IRC | 09:08 | |
jaosorior | jistr: talking about https://review.openstack.org/#/c/568736/6/overcloud-resource-registry-puppet.j2.yaml | 09:09 |
jaosorior | jistr: figuring out how to get it, or how to include it in a relevant environment by default. | 09:09 |
openstackgerrit | Ana Krivokapic proposed openstack/instack-undercloud master: Upload validations to swift on undercloud install https://review.openstack.org/564233 | 09:10 |
*** akrivoka has joined #tripleo | 09:10 | |
jaosorior | jistr: any ideas? :D | 09:13 |
marios | o/ dev docs for Q upgrade if you have time please @ https://review.openstack.org/#/c/569443/ && http://logs.openstack.org/43/569443/4/check/build-openstack-sphinx-docs/7ef0bbf/html/install/developer/upgrades/major_upgrade.html | 09:14 |
*** morazi has joined #tripleo | 09:14 | |
shardy | jaosorior: it's created in tripleoclient, which is actually a bug as we should instead add each environment to plan-environment and let the environments get merged by heat on deploy | 09:15 |
shardy | jaosorior: ref https://review.openstack.org/#/c/448209/ - turns out fixing that was quite hard as we need heat to handle retrieving the environments from swift to avoid the POST request limit | 09:16 |
shardy | IIRC ramishra was looking into that | 09:16 |
*** salmankhan has joined #tripleo | 09:16 | |
*** Hazelesque has joined #tripleo | 09:18 | |
jistr | jaosorior: i think it should be possible to get that info, but we may need to shuffle things around slightly. BTW we just got a green experimental upgrade job yesterday for scenario000 Q->master upgrade, so we should make sure the TLS by default will not break upgrades before we re-merge the patches (sorry :) ) | 09:18 |
jistr | jaosorior: i'm quite swamped today but i should be able to help on this tomorrow | 09:19 |
ramishra | shardy: yeah, in my plan for this cycle. But I'm waiting for some reviews to https://review.openstack.org/#/q/status:open+branch:master+topic:bug/1755453 to move foward:) | 09:19 |
shardy | ramishra: ack thanks, will check out the patches | 09:19 |
jaosorior | shardy: so, you wanna merge everything into plan-environment.yaml instead of having plan-environment.yaml and user-environment.yaml? | 09:21 |
jaosorior | jistr: sure man, no worries. Trying my best not to break you guys :D . But it sure isn't easy to get this working X_x third week working on just this. Help is appreciated. | 09:22 |
shardy | jaosorior: No we should pass a list of enabled environments then let heat merge them on the server side | 09:22 |
shardy | jaosorior: currently we merge everything in tripleoclient (and also in the process templates mistral action) | 09:22 |
jaosorior | shardy: and that list of enabled environment would be in plan-environment.yaml, right? | 09:22 |
shardy | jaosorior: yes, that's how the UI works now | 09:23 |
jaosorior | OK | 09:23 |
jaosorior | shardy: so, the thing is that I need to check a parameter that's in the resource registry https://review.openstack.org/#/c/568736/6/overcloud-resource-registry-puppet.j2.yaml | 09:23 |
jaosorior | that one specifically | 09:23 |
shardy | jaosorior: I'm not sure if we'll fix that this cycle though so probably best to assume the client side merging for now | 09:23 |
jaosorior | I wanted to get that variable from user-environment or plan-environment. But I'm starting to think I should just call swift to get the resource registry and check it there | 09:24 |
jaosorior | i'm doing this because this needs to work with upgrades. So it's the case where an older overcloud is being managed by a newer tripleo-common. So we wouldn't enable TLS for older releases that don't have this parameter. | 09:24 |
shardy | jaosorior: which resource_registry would you read from swift though? | 09:24 |
shardy | You need the merged output as ordering could change the value | 09:25 |
jaosorior | shardy: overcloud-resource-registry-puppet.yaml | 09:25 |
shardy | jaosorior: but that could be overridden by a user environment? | 09:25 |
shardy | jaosorior: would it be safer to look at the environment for the existing stack on upgrade? | 09:25 |
shardy | e.g openstack stack environment show overcloud or the mistral equivalent? | 09:25 |
shardy | that's the only definitive answer about what values are in use | 09:26 |
jaosorior | shardy: right, this is what I'm doing right now https://review.openstack.org/#/c/569364/5/tripleo_common/actions/deployment.py (check line 348) | 09:26 |
jaosorior | shardy: I'm checking the environment (what the user passes), and it he EnablePublicTLS flag is set we deploy TLS. In older releases this flag shouldn't exist, so it'll default to False, and won't try to deploy TLS | 09:27 |
*** elgxl has quit IRC | 09:27 | |
jaosorior | for newer releases, the default is True, and you can disable it by setting it to false (that's in the environment that disables TLS) | 09:27 |
jaosorior | however, we need a way to get the default value for that; right now we only can see that variable if the user sets it | 09:28 |
jaosorior | shardy: cause I can't look at the environment on new deployments | 09:28 |
shardy | jaosorior: isn't the default value for all parameters that exist in the flattened parameters we get from validation? | 09:28 |
jaosorior | validations is not always enabled | 09:28 |
shardy | jaosorior: different type of validation | 09:29 |
shardy | we always validate the plan AFAIK | 09:29 |
jaosorior | I don't know :D | 09:29 |
*** marios has quit IRC | 09:29 | |
shardy | jaosorior: the flattened parameters stuff I pointed to for derive parameters is always calculated AFAIK | 09:29 |
shardy | but you can optionally disable some other validations e.g pre-deploy checks via ansible and the like | 09:29 |
jaosorior | shardy: wasn't it the case that we don't call that by default? That's the answer I got from yesterday | 09:30 |
shardy | jaosorior: yes but if you move this to be done in a derive parameters workflow, we'd have to change that and do it by default | 09:30 |
*** wolverineav has joined #tripleo | 09:30 | |
shardy | e.g add one line to the plan-environment | 09:30 |
openstackgerrit | Jose Luis Franco proposed openstack/tripleo-upgrade stable/pike: Separate scripts creation tasks for under/overcloud. https://review.openstack.org/570357 | 09:30 |
*** marios has joined #tripleo | 09:31 | |
jaosorior | so... honestly I don't really understand what that means. As far as I understood from skramaja yesterday, it wouldn't be a good idea to enable it by default because it makes it hardware dependent (that's how they get hardware parameters for NFV) | 09:31 |
*** zoli is now known as zoli|lunch | 09:31 | |
shardy | jaosorior: enabling a workflow for your stuff wouldn't be hardware dependent | 09:31 |
jaosorior | so, yeah, I haven't enabled that by default because I don't know how, nor what it means to do that. | 09:31 |
shardy | the point is we don't run all the DPDK stuff by default | 09:32 |
*** Tengu has joined #tripleo | 09:32 | |
jaosorior | shardy: but that's what the derive parameter workflow does | 09:32 |
shardy | so, atm, we don't run any derive parameters workflows | 09:32 |
jaosorior | unless I'm getting very confused here | 09:32 |
shardy | jaosorior: it's pluggable | 09:32 |
jaosorior | which I think I am | 09:32 |
shardy | jaosorior: we can run one or more derive parameters workflows | 09:32 |
shardy | atm we run zero | 09:32 |
skramaja | we need to modify the existing workflows so that we avoid most of the instrospection data collection and other stuffs before applying formulas.. | 09:32 |
shardy | and for DPDK you can enable one | 09:32 |
shardy | and there's another one for Ceph IIRC | 09:32 |
*** loacker has joined #tripleo | 09:32 | |
jaosorior | lets check out the ceph one | 09:33 |
shardy | skramaja: ack, but in theory we can make this pluggable so "derive parameters" is a general purpose way of including workflows that, uh, derive parameters? | 09:33 |
skramaja | also when it is enabled by default, there should be control wit the user to use it or not. for which we could add additional checks along with service presese.. | 09:33 |
*** elgxl has joined #tripleo | 09:33 | |
shardy | like, for DPDK and ceph we derive parameters based on the introspection data | 09:33 |
shardy | but in this case we're deriving it from different data | 09:33 |
skramaja | if we add the vip requirement and no dpdk, getting instrospection data is a waste of effort. | 09:33 |
shardy | but it's basically the same sequence? | 09:34 |
shardy | skramaja: sure, we can make that optional | 09:34 |
skramaja | what i am saying is that we could tweak the workflow.. | 09:34 |
shardy | skramaja: yeah I think we're agreeing here :) | 09:34 |
shardy | does getting the introspection data actually take much time? | 09:34 |
shardy | probably depends on the number of nodes I guess | 09:34 |
skramaja | yes.. i agree to invoke by default with few additional changes to the workflow related to nfv and hci invocation.. | 09:34 |
skramaja | not much time. you dont need to force envery one to do introspection because of this.. | 09:35 |
openstackgerrit | Jose Luis Franco proposed openstack/tripleo-heat-templates stable/pike: DNM: Test undercloud upgrades in stable/pike. https://review.openstack.org/566264 | 09:35 |
shardy | we probably just need a way to tell if each derive parameters workflow needs introspection data, and calculate a boolean based on which workflows are enabled | 09:36 |
jaosorior | Tengu: just to catch you up, we're talking with shardy and skramaja about the issue we have, and the idea is that it would be better to do it as part of the "derive_parameters" workflow instead of deploy | 09:36 |
Tengu | jaosorior: hmm ok. I don't know that one, will dig in the doc. | 09:36 |
jaosorior | Tengu: now, that workflow is not used by default at all, and currently doesn't fit our needs, but it would need to be refactored (as far as I've understood) | 09:37 |
Tengu | ah :) | 09:37 |
*** mdnadeem has quit IRC | 09:37 | |
skramaja | shardy: yes. something like should be worked out. we could have feature mapping with hw dependecy.. | 09:37 |
jistr | skramaja, shardy: would the derive params workflow take priority over env files? | 09:37 |
shardy | jaosorior: yeah my assumption was that this was already completely pluggable, but sounds like we may need a few tweaks | 09:37 |
skramaja | no.. user provided env has always the prirority jistr | 09:38 |
Tengu | so this one right? https://specs.openstack.org/openstack/tripleo-specs/specs/pike/tripleo-derive-parameters.html | 09:38 |
jistr | ++ great | 09:38 |
shardy | Tengu: yes, that was implemented with Ceph and DPDK use-cases in mind, but it's a more general pattern where you need to calculate some parameters on deployment | 09:38 |
Tengu | shardy: hmm ok. | 09:39 |
*** cgoncalves is now known as cgoncalves|pto | 09:40 | |
skramaja | shardy: https://github.com/openstack/tripleo-common/blob/master/workbooks/derive_params.yaml#L432 currently the features association with role is done here. | 09:40 |
skramaja | for each role it will check the features associated with it and then get the introspection data.. we need to add a check if we need introspection data or not.. | 09:41 |
skramaja | jaosorior: could you provide what are the paraemters that you are looking for to derive? | 09:41 |
jaosorior | skramaja: so... this is what we're doing https://review.openstack.org/#/c/569818/ | 09:42 |
*** pmannidi has quit IRC | 09:42 | |
jaosorior | skramaja: in a nutshell, if the user didn't provide anything, we get the allocation range for network and set the *FixedIPs parameter. | 09:43 |
jaosorior | it could be using network isolation (in which case, we use the external network), or it could be using the ctlplane network (in which case we use ControlFixedIPs) | 09:43 |
jaosorior | if the user provided the FixedIPs parameter, then we don't do anything | 09:43 |
skramaja | ok. | 09:44 |
jaosorior | for this we need to get the allocation range for the ctlplane-subnet, and the ports that were already created in that subnet | 09:44 |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-common master: Set FixedIp by default for public-facing VIP https://review.openstack.org/569818 | 09:47 |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-common master: TLS by default for the overcloud for VIPs and FQDNs https://review.openstack.org/569364 | 09:47 |
jaosorior | Tengu: the capitalized If was actually in the "Fixed IPs" patch ^^ | 09:47 |
Tengu | jaosorior: damn. | 09:47 |
*** tdasilva has joined #tripleo | 09:47 | |
jaosorior | skramaja: so... would you have time to make this derive_parameters workfloe default? We could then hook up our stuff there. | 09:48 |
*** mdnadeem has joined #tripleo | 09:50 | |
jistr | skramaja: i'm looking at the workflow code, just one question, how do we ensure that the parameter priorities are correct? default < derived < user-provided | 09:50 |
skramaja | yes jaosorior, i will involve the jaganathan to make the required chages as he is the one who is actively working on it. i will drop a mail explaing the expectations so that that is clear.. | 09:51 |
jistr | the workflow just seems to edit the plan-env parameters directly? | 09:51 |
shardy | Thanks skramaja! | 09:51 |
jaosorior | skramaja: that would be awseme, thanks a lot! | 09:51 |
skramaja | jistr: https://github.com/openstack/tripleo-common/blob/2b6efe479eb87999431842b0ac203f9ad03c1092/tripleo_common/utils/plan.py#L142 | 09:51 |
jaosorior | Tengu: lets pospone the TLS by default work for now until we can hook up the TLS and VIP bits to the derive_parameters workflow | 09:52 |
Tengu | jaosorior: fine for me. I'll read a bit more about that. | 09:52 |
skramaja | here is where the merging is done. the "derived_params" will be stored in the plan_environemnt.yaml and then merged as part of preocess templates.. | 09:52 |
jaosorior | Tengu: that's a good idea. | 09:52 |
Tengu | and play a bit more with my tmate thingy. | 09:52 |
*** avivgt has quit IRC | 09:54 | |
jistr | skramaja: thanks wouldn't the priorities break when we merge shardy's patch? https://review.openstack.org/#/c/448209 | 09:54 |
jistr | shardy, skramaja: IIUC with that patch all user-provided env files from cmdline will end up in `environments` and thus taking lower prio than derived? | 09:55 |
jistr | sorry for too many Qs i'm trying to build up understanding, as i'm a bit worried about how this all plays out together :) | 09:55 |
shardy | jistr: doesn't user-environment.yaml already end up in the environments list, just as one big merged environment? | 09:56 |
jistr | shardy: yes but user env has only resource_registry, no parameter_defaults AFAIK | 09:57 |
jistr | at least that's what i saw when i poked into the plan... i didn't implement this so my knowledge isn't 100% | 09:58 |
jaosorior | jistr: that's correct. It only has the resource_registry | 09:58 |
jaosorior | user-provided parameters are in plan-environment.yaml | 09:58 |
shardy | Ok so that's another thing we'd need to fix to enable heat parameter_merge_strategies | 09:58 |
shardy | in addition to my tripleoclient patch we'd need to rework tripleo-common so we don't do all the merged_params.update stuff | 09:59 |
shardy | tbh one reason I've delayed doing this is because it seems quite risky | 09:59 |
shardy | so it's good to discuss it | 09:59 |
skramaja | intially i had plans to write the derive params as separate file directly on the overcloud container.. | 09:59 |
shardy | The way this should work, IMO, is we should have a list of environments where the ordering defines the precedence in the plan_environment | 10:00 |
shardy | we shouldn't have any environment data e.g parameters etc merged and stored inside plan_environment | 10:00 |
jistr | skramaja: i think that could fix it. Having everything in `environments`. `resource-registry-ouvercloud` as first, then `derived parameters` 2nd, then the rest of the files (user provided) | 10:00 |
shardy | that's beyond the scope of this TLS discussion though I think | 10:00 |
shardy | jistr: yeah that's what I had in mind | 10:01 |
skramaja | ok. when shardy revivies the patch, we have some work to do :) | 10:01 |
*** panda|off is now known as panda | 10:03 | |
jaosorior | skramaja: in the meantime, the plan is to keep the current approach, right? | 10:03 |
jistr | shardy: re beyond the scope of discussion, yes but it affects where can we put the TLS code right now to unblock jaosorior. If we know the `envrionments` patch and won't work well (yet) with how we handle derived parameters, i'm not sure if derived parameters is the best place. Or maybe better yet, yes let's use derived_parameters but that means we can't land the `environments` patch as it's currently. | 10:03 |
skramaja | yes jaosorior | 10:03 |
shardy | jistr: ack, understood - I'm just trying to discuss it so we end up with the logic in a sane place, vs wedging a ton of python code into the deploy action which may be fragile and hard to maintain? | 10:04 |
shardy | not trying to block but I expected the derive parameters interface to fit this requirement, if it doesn't then that's OK | 10:04 |
shardy | also I don't think impact on the environments patch I pushed should be a concern - that's been blocked for months for other reasons so I guess it's fairly low priority | 10:05 |
shardy | I see it as technical debt which would be good to fix but probably not super critical from a user perspective | 10:05 |
jistr | shardy: yes totally +1 re trying to bring some order into where we set what, and hopefully derived_parameters being the right place for this type of features | 10:06 |
*** wolverineav has quit IRC | 10:07 | |
*** wolverineav has joined #tripleo | 10:07 | |
jaosorior | shardy, jistr: It does sound like the best way to go forward, from both a logical and a maintainability point of view | 10:08 |
*** wolverin_ has joined #tripleo | 10:09 | |
*** wolverineav has quit IRC | 10:09 | |
*** anilvenkata has quit IRC | 10:09 | |
jistr | now i wonder if we have all the info we need at the point when we call (or will call) the derive_parameters workflow | 10:10 |
*** anilvenkata has joined #tripleo | 10:10 | |
*** wolverin_ has quit IRC | 10:10 | |
shardy | jistr: it's called right before the deploy starts so I assumed we'd have basically all data available | 10:10 |
jaosorior | jistr: well, the plan is to always call it (have it as a default). I'm not sure at what point of the deployment that happens though | 10:10 |
jaosorior | I see | 10:10 |
jistr | shardy: that sounds good then | 10:10 |
*** wolverineav has joined #tripleo | 10:10 | |
jistr | shardy, jaosorior: we should be able to look into `parameter_defaults` and see the final state of `EnablePublicTLS` in that case, and decide based on that | 10:11 |
jaosorior | Should a call to the derive workflow be added to the deploy workflow? that way we can make sure that it gets called in both the CLI and the UI | 10:11 |
jaosorior | jistr, skramaja, shardy, Tengu ^^ | 10:11 |
jistr | +1 | 10:12 |
*** elgxl has quit IRC | 10:12 | |
skramaja | https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/overcloud_deploy.py#L214 | 10:13 |
skramaja | its currently invoked from cli.. the idea of not integrating with deploy woflow is -- | 10:14 |
*** elgxl has joined #tripleo | 10:14 | |
skramaja | for ui, we wanted give an option for the user to see the parameters and then start the deployment/.. | 10:14 |
jistr | skramaja, shardy: also would be really nice to refactor it into being in the `environments` list, so that we know what came from where when debugging. I can imagine effectively amending the plan's global parameter_defaults directly without any way to see what came from derive_parameters (by inspecting the plan container later) could lead to some confusing situation + hard debugging. | 10:14 |
skramaja | in case if user want to alter it, he could do that in the UI.. though UI is not integrated it yet.. | 10:15 |
shardy | skramaja: Ok, but right now the derive params stuff just doesn't work via the UI? | 10:15 |
*** gfidente has joined #tripleo | 10:15 | |
*** gfidente has quit IRC | 10:15 | |
*** gfidente has joined #tripleo | 10:15 | |
*** wolverineav has quit IRC | 10:15 | |
shardy | jistr: agreed, and that fits well with what I was trying to do for the user environments | 10:15 |
jaosorior | skramaja: sure, that's fine. So, for the UI case... how do we make sure that the derive parameters get run by default? | 10:15 |
shardy | any help appreciated ;) | 10:15 |
jaosorior | skramaja: else we would break the UI, cause the default endpodint map would have TLS endpoints. | 10:15 |
skramaja | as of now plan-environment.yaml in the container will have the "derived_parameters" section seprately.. | 10:15 |
jistr | skramaja: ah right so that's good from debugging perspective | 10:16 |
shardy | Yeah but it still doesn't help if you have multiple derive parameters workflows | 10:16 |
Tengu | jaosorior: +42 | 10:16 |
jistr | shardy: yea i'd like to :D but we have so much stuff to do :D | 10:16 |
shardy | it'd be better to have environments have derive-parameters-foo.yaml and derive-parameters-xyz.yaml | 10:16 |
skramaja | shardy: agreed, but what i am saying is that we have the difference today also.. all the derived_params are merged as one.. | 10:17 |
* jistr currently prioritizing CI for updates/upgrades as lack of that has been a significant pain point for a long time | 10:17 | |
shardy | jistr: sure, just looking for opportunities to make incremental progress vs making more work to do later :) | 10:17 |
jistr | yea i understand :) | 10:17 |
skramaja | this is needed because if you remember the DPDK and HCI usecases, both will update same prameters.. | 10:17 |
skramaja | so the order of executing the formulas play an importnat role.. | 10:18 |
*** pkovar has joined #tripleo | 10:18 | |
openstackgerrit | Roman Gorshunov proposed openstack/diskimage-builder master: Allow to rebuild arbitrary images https://review.openstack.org/568697 | 10:20 |
*** psahoo has quit IRC | 10:21 | |
*** akrivoka has quit IRC | 10:25 | |
*** akrivoka has joined #tripleo | 10:26 | |
*** leanderthal has joined #tripleo | 10:27 | |
jistr | skramaja: what would be the ideal strategy for dealing with that case? merging a list / one of them overriding the other / sth else? (i'm not familiar with the details) | 10:27 |
openstackgerrit | Merged openstack/instack-undercloud master: Fail in undercloud pre_upgrade if overcloud not in *_COMPLETE state https://review.openstack.org/569345 | 10:28 |
openstackgerrit | Merged openstack/python-tripleoclient master: Drop default inclusion of undercloud.yaml https://review.openstack.org/568386 | 10:28 |
skramaja | jistr: currently we do that. parameters are derived on per role basis and updated using <RoleName>Parameteres in the "derived_parameters" so it will be merged.. | 10:29 |
skramaja | for example NovaHostReservedMemory is done by both DPDK and HCI. | 10:29 |
*** dtantsur|afk is now known as dtantsur | 10:30 | |
jistr | ack and i guess whatever runs later, wins | 10:30 |
jistr | ? | 10:30 |
shardy | skramaja: do the values get combined, or is it just appending to lists or updating dicts for the same parameter? | 10:30 |
openstackgerrit | Rabi Mishra proposed openstack/instack-undercloud stable/queens: Fail in undercloud pre_upgrade if overcloud not in *_COMPLETE state https://review.openstack.org/570367 | 10:30 |
shardy | because one of the motivations of moving to heat server side merging is it supports parameter_merge_strategy, which means you can control merging lists and maps from multiple environments | 10:31 |
shardy | doesn't help if it's e.g a number or something which needs to be somehow combined tho | 10:31 |
skramaja | shardy: jistr what ever run last will win. in which case the order is important. | 10:32 |
shardy | skramaja: yes, that's the same as environment merging though | 10:32 |
Tengu | the order is always important regarding the env -.- | 10:33 |
*** abishop has joined #tripleo | 10:33 | |
* Tengu remembers being stung a lot of time with ceph, docker and HA env order inclusion -.-.' | 10:33 | |
Tengu | it was nasty bites. | 10:33 |
*** zoli|lunch is now known as zoli | 10:34 | |
skramaja | the order is for derived parameters is static and embeded as part of the workflow based on agreement between feature owners.. so it does matter.. | 10:34 |
shardy | skramaja: nobody disagrees with that, we're asking what sort of data is merged | 10:35 |
shardy | is it e.g 3 merged with 4, where the second workflow knows they have to be added, or a data structure like [3] and [4] where you can reasonably know that lists are always appended? | 10:36 |
shardy | heat handles the second case, but not the first | 10:36 |
*** olap has quit IRC | 10:38 | |
jistr | shardy, skramaja: also i wonder if we have the derive workflows smart enough to say "this is not a HCI env, don't emit any derived params for HCI". If so, i think a lot of the conflicts just wouldn't happen in practice. | 10:40 |
skramaja | shardy: https://github.com/openstack/tripleo-common/blob/master/workbooks/derive_params_formulas.yaml#L536 | 10:40 |
*** ramishra has quit IRC | 10:40 | |
jistr | shardy, skramaja: and then handling shardy's example of 3, 4 => 7 vs. [3], [4] => [3, 4] personally i'd consider that a stretch goal | 10:41 |
*** maufart__ has joined #tripleo | 10:41 | |
shardy | jistr: yeah, my point was really that [3], [4] => [3, 4] (and similar with dicts) can be handled by heat parameter merging | 10:41 |
shardy | so if we ever fix bug #1635409 we could use that in the list of environments from plan-environment | 10:42 |
openstack | bug 1635409 in tripleo "parameter_merge_strategies: merge behaves like overwrite" [High,In progress] https://launchpad.net/bugs/1635409 - Assigned to Steven Hardy (shardy) | 10:42 |
*** ssbarnea_ has quit IRC | 10:42 | |
*** aufi_ has quit IRC | 10:44 | |
skramaja | i think we will have the goal to write it as environemt file as the next step.. | 10:45 |
*** mdnadeem has quit IRC | 10:46 | |
* jistr feels like having somewhat better understanding of the derived param context now | 10:49 | |
jistr | thanks skramaja | 10:49 |
* jistr gotta grab lunch before a chain of mtgs :/ tomorrow should be a better day :D | 10:49 | |
Tengu | lunch. hmmm. | 10:51 |
Tengu | fancy idea. | 10:51 |
*** ramishra has joined #tripleo | 10:57 | |
*** jpena is now known as jpena|lunch | 10:58 | |
openstackgerrit | Nir Magnezi proposed openstack/tripleo-heat-templates master: Make lb-mgmt-subnet a class B subnet https://review.openstack.org/568138 | 11:05 |
openstackgerrit | Nir Magnezi proposed openstack/tripleo-heat-templates master: Make lb-mgmt-subnet a class B subnet https://review.openstack.org/568138 | 11:06 |
*** amoralej is now known as amoralej|off | 11:07 | |
*** jpena|lunch is now known as jpena|off | 11:10 | |
*** rh-jelabarre has joined #tripleo | 11:11 | |
*** ansmith has quit IRC | 11:17 | |
openstackgerrit | Sylvain Afchain proposed openstack/tripleo-heat-templates master: skydive external_deploy_task migration https://review.openstack.org/561274 | 11:26 |
*** bfournie has quit IRC | 11:27 | |
*** pradk has joined #tripleo | 11:28 | |
*** ohamada has joined #tripleo | 11:28 | |
*** udesale_ has joined #tripleo | 11:33 | |
*** udesale has quit IRC | 11:35 | |
*** olap has joined #tripleo | 11:42 | |
*** pdeore__ has quit IRC | 11:42 | |
*** udesale_ has quit IRC | 11:42 | |
*** radeks__ has joined #tripleo | 11:43 | |
*** rasca has quit IRC | 11:44 | |
*** radeks_ has quit IRC | 11:45 | |
*** loacker has quit IRC | 11:51 | |
*** gfidente has quit IRC | 11:51 | |
*** d0ugal has joined #tripleo | 11:55 | |
*** gfidente has joined #tripleo | 11:55 | |
*** gfidente has joined #tripleo | 11:55 | |
*** d0ugal__ has quit IRC | 11:57 | |
*** jpena|off is now known as jpena | 11:58 | |
*** mvenesio has joined #tripleo | 12:00 | |
*** mburned_out is now known as mburned | 12:01 | |
*** dbecker has joined #tripleo | 12:04 | |
*** bfournie has joined #tripleo | 12:04 | |
* dtantsur LOL at "wow it's actually working" | 12:05 | |
Tengu | :] | 12:06 |
Tengu | dtantsur: my fault. although the context wasn't the CI -.-'. | 12:07 |
*** skramaja has quit IRC | 12:07 | |
Tengu | (although.. yes, wow, it IS working :D) | 12:08 |
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:08 |
*** Guest19060 is now known as rbrady | 12:10 | |
*** rbrady has joined #tripleo | 12:10 | |
*** quiquell is now known as quiquell|lunch | 12:12 | |
*** b00tcat has quit IRC | 12:13 | |
*** ratailor has quit IRC | 12:14 | |
*** tzumainn has joined #tripleo | 12:15 | |
numans | EmilienM, Hi, can you please take a look at this when you get some time - https://review.openstack.org/#/c/523804/ | 12:15 |
*** waleedm has quit IRC | 12:15 | |
*** elgxl has quit IRC | 12:18 | |
sshnaidm | jfrancoa, hey, are you aware of http://logs.openstack.org/91/564291/12/check/tripleo-ci-centos-7-undercloud-upgrades/7c33e27/logs/undercloud/home/zuul/undercloud_upgrade.log.txt.gz#_2018-05-24_09_09_32_718 ? | 12:20 |
mwhahaha | jaosorior: hi2u, re: https://review.openstack.org/#/c/565151/ | 12:23 |
jaosorior | mwhahaha: sup | 12:23 |
mwhahaha | jaosorior: i commented but we need to use something else for tls because that adds a requirement that haproxy needs to be on the keystone node | 12:23 |
*** lifeless_ has joined #tripleo | 12:24 | |
*** lifeless has quit IRC | 12:24 | |
jaosorior | mwhahaha: understood | 12:24 |
jaosorior | mwhahaha: originally that TLS bit was written on the Controller role (before we had composable roles). | 12:25 |
jaosorior | when composable roles came along, those tags were added to identify where to put the certs | 12:25 |
mwhahaha | jaosorior: the tags were before that but ok :D | 12:26 |
jaosorior | not sure what's the reasoning for needing both the controller and primary tags (I didn't add that). but if we can do without the primary tag, I'm fine with that | 12:26 |
jaosorior | it would just need to be fixed. | 12:26 |
mwhahaha | so the problem here is that its' also used for the determination of the bootstrap node which is the keystone node | 12:27 |
mwhahaha | not the haproxy node | 12:27 |
jaosorior | mwhahaha: another thing is that I actually have no idea how it used to work before. Because MySQL also needs to be in the primary role in order to bootstrap the mysql users and databases | 12:27 |
jaosorior | mwhahaha: right, primary tag == role that contains the bootstrap node | 12:27 |
mwhahaha | jaosorior: i have a feeling it broke because of config download and how e determine boostrap node | 12:28 |
mwhahaha | because i thought each role had their own bootstrap node | 12:28 |
jaosorior | mwhahaha: they don't | 12:28 |
mwhahaha | but that seems to no longer be the case | 12:28 |
*** rlandy has joined #tripleo | 12:28 | |
jaosorior | mwhahaha: I talked to shardy about this issue at some point. And he thought it would be better to get bootstrap nodes per-service | 12:28 |
jaosorior | I agree with his idea | 12:28 |
mwhahaha | i'll need to look into the queens version to see what was the case previously | 12:28 |
mwhahaha | because i think we need a bootstrap determination for each role | 12:29 |
jaosorior | mwhahaha: it can't be per-role, because a service might be present in several roles | 12:29 |
shardy | the problem is services can span more than one role, so in general I think a bootstrap node per service is a better pattern? | 12:29 |
jaosorior | I think a bootstrap node per service is the way to go | 12:29 |
shardy | and we already have a bootstrap calculation per service | 12:30 |
mwhahaha | i guess, but we use bootstrap node in the deployment to determine if we run stuff | 12:30 |
mwhahaha | which is a problem | 12:30 |
mwhahaha | https://bugs.launchpad.net/tripleo/+bug/1768142/comments/4 | 12:30 |
openstack | Launchpad bug 1768142 in tripleo "tripleo-ci-centos-7-3nodes-multinode timing out frequently in the gate" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz) | 12:30 |
*** rlandy is now known as rlandy|rover | 12:30 | |
shardy | mwhahaha: yeah, we just need to ensure we check if it's a bootstrap node for each service, vs the historical one-and-only controller bootstrap node | 12:30 |
jaosorior | mwhahaha: yes it is. It ties MySQL, Keystone, and HAProxy to always run together | 12:30 |
jaosorior | so, we need to fix it | 12:30 |
mwhahaha | k we should probably document that because that's not good | 12:30 |
mwhahaha | why can't we add a mysql tag and a haproxy tag | 12:31 |
mwhahaha | to solve for those | 12:31 |
shardy | with containers and knowledge of per-service bootstrap nodes, are those services still coupled? | 12:31 |
shardy | modulo bootstrap node bugs in puppet-tripleo | 12:31 |
*** udesale has joined #tripleo | 12:31 | |
mwhahaha | yea because of where we run deployment bootstrap bits | 12:31 |
jaosorior | mwhahaha: mysql also does database/user creation based on the manifest running on a bootstrap node | 12:31 |
mwhahaha | check that comment from the bug -^ | 12:31 |
shardy | mwhahaha: Ok I assumed that bootstrapping could happen from any node provided it had the DB connection details | 12:32 |
mwhahaha | shardy: yea i think it can, which is why i think switching the tags back to the api role (keystone node) should be ok | 12:32 |
mwhahaha | but i'm not sure about mysql users, it's working | 12:32 |
mwhahaha | but it seems that tls adds a new wrinkle | 12:33 |
jaosorior | shardy: any node could do the mysql bootstrapping, but we only run the puppet manifest that does the actual bootstrapping from the role containing MySQL | 12:33 |
shardy | jaosorior: Ok, and that's because the puppet-tripleo stuff looks at the old notion of bootstrap_server_id I guess? | 12:33 |
shardy | or bootstrap_nodeid rather | 12:34 |
arxcruz|ruck | mwhahaha: hi, fyi, although everything seems to be running okay, the queue is a little bit big :) | 12:34 |
mwhahaha | arxcruz|ruck: cool thanks | 12:34 |
jaosorior | shardy: yep https://github.com/openstack/puppet-tripleo/blob/master/manifests/profile/base/database/mysql.pp#L172 | 12:34 |
shardy | https://github.com/openstack/tripleo-heat-templates/blob/master/overcloud.j2.yaml#L724 | 12:34 |
shardy | jaosorior: Ok, well IIRC short_service_bootstrap_node was supposed to be used instead, but there's probably a lot of places which haven't been updated to do that | 12:35 |
jaosorior | right | 12:36 |
jaosorior | shardy: then mwhahaha said, TLS is still an issue. Although for that we don't really need the bootstrap part. It would suffice if we just use a haproxy tag | 12:36 |
jaosorior | at some point I wanna get rid of the current TLS setup and do it via ansible though. So maybe this is a good excuse to do just that. | 12:38 |
*** holser__ has quit IRC | 12:38 | |
*** holser__ has joined #tripleo | 12:39 | |
jaosorior | mwhahaha: we can do this, we can change the tags back, and I'll work on removing the heat resources dedicated for TLS, and use ansible instead. | 12:39 |
*** saneax has joined #tripleo | 12:39 | |
*** ansmith has joined #tripleo | 12:39 | |
beagles | shardy: can you have a look at https://review.openstack.org/#/c/567655/ to see if the logic makes sense to you (using the neutron user as a "sentinel value") | 12:40 |
jaosorior | mwhahaha: of course, if you want to do the ansible bits yourself, that would also be nice :D I'm quite swamped with stuff to do as well :/ | 12:40 |
mwhahaha | pfffft | 12:40 |
mwhahaha | i'll just move keystone back to the controller role | 12:40 |
mwhahaha | :D | 12:40 |
beagles | shardy: still need mcornea's okay before merging this but feedback is welcome :) | 12:41 |
*** quiquell|lunch is now known as quiquell | 12:41 | |
jaosorior | mwhahaha: :( | 12:41 |
*** lifeless_ has quit IRC | 12:41 | |
mwhahaha | i don't know why the haproxy stuff needs to be on the primary for tls, is it for determingin the hostname or something? | 12:42 |
openstackgerrit | Alex Schultz proposed openstack/tripleo-heat-templates master: Fix 3node deployment https://review.openstack.org/570252 | 12:42 |
openstackgerrit | Marios Andreou proposed openstack/tripleo-docs master: Minor fixes for Pike to Queens upgrade operator docs https://review.openstack.org/570380 | 12:43 |
bogdando | shardy: do you mean short_service_bootstrap_node should have replaced all hiera(bootstrap*) in puppets? | 12:43 |
bogdando | like https://github.com/openstack/puppet-tripleo/blob/master/manifests/profile/base/database/mysql.pp#L84 ? | 12:43 |
jaosorior | mwhahaha: it doesn't, I actually don't know why the primary tag was added there (it's been there for a while). All we need is for this https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/role.role.j2.yaml#L565 to know in what role to render the NodeTLSData | 12:43 |
*** lifeless has joined #tripleo | 12:44 | |
mwhahaha | jaosorior: because you could have multiple 'controllers' | 12:44 |
*** trown|outtypewww is now known as trown | 12:44 | |
mwhahaha | jaosorior: but we pick the 'primary' | 12:44 |
mwhahaha | for bootstrap | 12:44 |
jaosorior | mwhahaha: a whole role has the primary tag | 12:44 |
mwhahaha | so i guess maybe the fix would be mark both as 'controller' | 12:44 |
mwhahaha | right | 12:44 |
mwhahaha | that's the bootstrap one | 12:44 |
mwhahaha | so i think it was working previously because that's primarily used for keystone and the rest just works | 12:45 |
jaosorior | mwhahaha: so every node with that role will have that tag and will get the certificate | 12:45 |
mwhahaha | w/o any tag designations, the first role from the file becomes the primary node | 12:45 |
jaosorior | right | 12:45 |
mwhahaha | jaosorior: so it seems like we need to use a tag for tls | 12:45 |
mwhahaha | so all nodes taged with tls gets the certs | 12:45 |
mwhahaha | or something | 12:45 |
Tengu | why not deploy the cert everywhere? | 12:46 |
jaosorior | mwhahaha: or stop using NodeTLSData (which is what I want to do this release), and get ansible to provision the certs only in the HAProxy service profile | 12:46 |
mwhahaha | or that | 12:46 |
mwhahaha | anyway i'm working on figuring out how to fix 3node so we can figure that out later :D | 12:46 |
jaosorior | Tengu: that cert is the public cert for HAProxy, we don't want the computes to have that cert. | 12:46 |
mwhahaha | unraveling stuff is always run | 12:46 |
Tengu | jaosorior: mostly because of possible private key leak I guess - I don't see other reason in fact.. ? | 12:47 |
mwhahaha | s/run/fun | 12:47 |
shardy | bogdando: yes that is what I'm saying, but IIRC we only changed a few which were needed to enable composable HA | 12:47 |
jaosorior | Tengu: correct. It's a bundle actually. Both the cert AND the key. | 12:47 |
Tengu | and chain | 12:48 |
Tengu | but ok. | 12:48 |
jaosorior | well yeah, there are the intermediates there. | 12:48 |
bogdando | shardy: oh, that coould be a proper fix then! mwhahaha^^ | 12:48 |
Tengu | oh by the way.... any chance to get an integration with let's encrypt? that old topic :D | 12:48 |
*** rasca has joined #tripleo | 12:49 | |
jaosorior | Tengu: an alternative to this mess with tags and stuff is to remove this https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/role.role.j2.yaml#L565-L573 entirely. And do the cert provisioning via ansible. | 12:49 |
jaosorior | Tengu: so, if you want to dig into that, that's definitely very appreciated work :D | 12:49 |
Tengu | jaosorior: :D | 12:49 |
jaosorior | Tengu: and yes! that would be awesome! Problem is always getting the time :D | 12:49 |
Tengu | jaosorior: well, dropping the TLSNodeData will be... well. complicated with backward compatibility, I guess. | 12:50 |
*** pkovar has quit IRC | 12:50 | |
Tengu | *NodeTLSData - anyway. | 12:50 |
*** pkovar1 has joined #tripleo | 12:50 | |
*** pkovar1 is now known as pkovar | 12:50 | |
jaosorior | Tengu: maybe? we would be using the same parameters for the certs/keys | 12:50 |
Tengu | 2s | 12:50 |
Tengu | ah, ok, this isn't a file touched by the user. | 12:50 |
*** cylopez has joined #tripleo | 12:50 | |
*** tvignaud has quit IRC | 12:51 | |
Tengu | so basically we'd drop that one from puppet, and push it somewhere else, with some other name, so that ansible can use the user-provided params. | 12:51 |
bogdando | shardy: hm, I see no changes for tripleo puppet at all http://codesearch.openstack.org/?q=short_service_bootstrap_node&i=nope&files=&repos= | 12:51 |
*** cshastri has quit IRC | 12:51 | |
jaosorior | Tengu: we would use deploy_step_tasks | 12:52 |
jaosorior | we could even do it via host_prep_tasks | 12:53 |
Tengu | jaosorior: that would also impact the NodeTLSCAData I guess right? | 12:54 |
*** tcw has quit IRC | 12:54 | |
*** pdeore__ has joined #tripleo | 12:54 | |
jaosorior | Tengu: it wouldn't inpact it, there's no dependency there. And NodeTLSCAData runs on every node (gotta persist the CA everywhere) | 12:54 |
*** pdeore__ has quit IRC | 12:54 | |
shardy | bogdando: the string is e.g "mysql_short_bootstrap_node_name" | 12:55 |
Tengu | hmm. | 12:55 |
jaosorior | Tengu: yeah, I think we could do it via host_prep_tasks. | 12:56 |
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 | 12:56 |
Tengu | shardy: -^ cleaned my config as you commented :). | 12:56 |
jfrancoa | sshnaidm: hey, is that happening in master? | 12:57 |
Tengu | jaosorior: any link to some doc regarding those steps? | 12:57 |
sshnaidm | jfrancoa, ocata | 12:57 |
jaosorior | Tengu: not sure. But this is how it looks like https://github.com/openstack/tripleo-heat-templates/blob/master/docker/services/swift-proxy.yaml#L243 | 12:57 |
*** gfidente has quit IRC | 12:57 | |
jfrancoa | sshnaidm: oh, I'll have a look at it. thanks for the notice | 12:57 |
sshnaidm | jfrancoa, thanks | 12:58 |
jfrancoa | sshnaidm: is there any LP bug for it? | 12:58 |
sshnaidm | jfrancoa, mm.. not sure | 12:58 |
jaosorior | shardy: at what point of the deployment does host_prep_Tasks run? | 12:58 |
shardy | bogdando: see https://review.openstack.org/#/c/421405/ - arguably at that point we should have converted all manifests, not only the pacemaker bits needed to enable composable ha | 12:58 |
shardy | but if it helps we could do that now and remove the generic bootstrap_node stuff from t-h-t/hiera | 12:59 |
sshnaidm | jfrancoa, I'll submit | 12:59 |
shardy | jaosorior: early, before all the deploy steps | 12:59 |
shardy | https://github.com/openstack/tripleo-heat-templates/blob/master/common/deploy-steps.j2#L291 | 13:00 |
jaosorior | shardy: the ansible tasks don't get stored in the nodes nowadays, right? | 13:01 |
jaosorior | they only get persisted on the undercloud | 13:01 |
shardy | jaosorior: yup | 13:02 |
*** mcornea has joined #tripleo | 13:02 | |
jaosorior | Tengu: so yeah, effectively, both host_prep_tasks and deploy_step_tasks are ansible tasks that get ran on the overcloud hosts if a certain service is enabled. | 13:02 |
shardy | that template is for the pre-config-download implementation | 13:02 |
Tengu | jaosorior: so in the TLS case, that service would be haproxy in its container I guess? | 13:02 |
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:02 |
jaosorior | Tengu: right, so the catch is that it gets run on the host, not the container. | 13:03 |
jaosorior | Tengu: but so is the TLS setup currently. So that's not a big deal. | 13:03 |
jfrancoa | sshnaidm: thanks a lot, I think I didn't see it before because we were blocked by this other LP https://bugs.launchpad.net/tripleo/+bug/1769107 | 13:03 |
openstack | Launchpad bug 1769107 in tripleo "Undercloud upgrades job failing in stable/ocata" [Medium,Fix released] - Assigned to Jose Luis Franco (jfrancoa) | 13:03 |
shardy | jaosorior: yeah, we could in theory make host_prep_tasks into step-0 deploy-steps tasks or something, but previously we couldn't because deploy_steps_tasks don't work when not using config-download | 13:03 |
*** myoung|off is now known as myoung | 13:03 | |
Tengu | jaosorior: not a big issue, as the certificates are pushed on the host, and directory is mounted read-only in the containers | 13:03 |
jaosorior | exactly | 13:03 |
beagles | marios: I've updated ^ as per you recommendations - I pushed for review before test deploying though so could you eyeball it for obvious syntax errors | 13:03 |
jaosorior | shardy: are we supporting folks turning off config-download? | 13:04 |
Tengu | jaosorior: only thing: ensure haproxy is properly reloaded after we push the certificate. | 13:04 |
Tengu | but that's already managed I guess. | 13:04 |
Tengu | and as it's a pre_task, it should be OK | 13:04 |
shardy | jaosorior: no idea, I assumed it was at least deprecated | 13:04 |
*** tcw has joined #tripleo | 13:04 | |
*** saneax has quit IRC | 13:05 | |
jaosorior | Tengu: there are two options. Either copy the script that's currently run. Or dissect the script and do each task via ansible. But yeah, the script right now reloads HAProxy if the cert is updated. | 13:05 |
Tengu | jaosorior: disecting the script would be the right way I think, for the sake of code reusability (is it even English?) | 13:06 |
jaosorior | it is | 13:06 |
Tengu | :) | 13:06 |
Tengu | jaosorior: so. hm. basically, the change would be to move the TLS thingy in the docker/services/haproxy.yaml? | 13:09 |
*** tvignaud has joined #tripleo | 13:10 | |
jaosorior | Tengu: rather to puppet/services/haproxy.yaml, and then be called from the base up to puppet/services/pacemaker/haproxy.yaml docker/services/haproxy.yaml docker/services/pacemaker/haproxy.yaml | 13:11 |
*** tvignaud has quit IRC | 13:12 | |
* Tengu pulling t-h-t master and preparing git grep | 13:13 | |
*** vpickard_ is now known as vpickard | 13:14 | |
*** jistr is now known as jistr|mtg | 13:14 | |
*** toure|gone is now known as toure | 13:16 | |
*** ykarel is now known as ykarel|away | 13:17 | |
*** cylopez has left #tripleo | 13:18 | |
openstackgerrit | Merged openstack/python-tripleoclient master: Make standalone role name configurable https://review.openstack.org/568378 | 13:18 |
openstackgerrit | Marios Andreou proposed openstack/tripleo-docs master: Add developer docs for Pike to Queens upgrade https://review.openstack.org/569443 | 13:19 |
*** gfidente has joined #tripleo | 13:20 | |
*** zoli is now known as zoli|afk | 13:20 | |
*** tcw has quit IRC | 13:21 | |
bogdando | mwhahaha, jaosorior, shardy: can we merge https://review.openstack.org/#/c/564135/ please? | 13:21 |
bogdando | too much of rebasing after it gets wf+1 | 13:22 |
bogdando | and please do for https://review.openstack.org/#/c/563988/ | 13:22 |
bogdando | I think we should not block master merges cuz of stable/foo hickups? | 13:22 |
bogdando | sshnaidm: btw, that's another price we are now paying all :) | 13:23 |
sshnaidm | bogdando, all this is *zero* price relative to what we'll pay with branches.. | 13:24 |
*** links has quit IRC | 13:24 | |
bogdando | just brought an example, it's really weird we can't merge into rocky when ocata fails | 13:24 |
Tengu | jaosorior: (still trying to understand, sorry) so in fact, we should rewrite this snippet to something in ansible, in the pre_host_tasks? puppet/extraconfig/tls/tls-cert-inject.yaml | 13:25 |
*** wolverineav has joined #tripleo | 13:26 | |
jaosorior | Tengu: yep | 13:26 |
*** tvignaud has joined #tripleo | 13:26 | |
Tengu | damned | 13:26 |
*** radeks_ has joined #tripleo | 13:26 | |
Tengu | I can try to convert it as an ansible snippet. That would be a first step I guess. | 13:27 |
Tengu | and a way for me to train a bit more my ansible. | 13:27 |
*** radeks__ has quit IRC | 13:29 | |
*** radek__ has quit IRC | 13:30 | |
bogdando | sshnaidm: My point is we have a failure-triggering-noise, like intermittent/timeout things for each of jobs, say a probability of N, and the real faulure related to the subject patch, 1-N. Then, adding a two more OVB branched jobs and a two more zuul jobs makes each patch submitted into master highly likely blocked cuz of unrelated noise amplification | 13:30 |
bogdando | Let me try to calculate it ... :) | 13:31 |
sshnaidm | bogdando, what is failure-triggering-noise?? | 13:31 |
bogdando | timeouts, connectivity errors, all that makes us posting recheck | 13:31 |
bogdando | and is not related to real patches being tested | 13:31 |
bogdando | and now this noise gets amplified by a 4 times I think | 13:32 |
bogdando | so folks will post rechecs now like x4 time more often | 13:32 |
bogdando | which will make zuul queues x4 larger, and time to wait for jobs x4 as well | 13:33 |
bogdando | that is what I call 'recheck storms' | 13:33 |
bogdando | and is the primary reason I created the jobs dependencies & pipelines topic | 13:33 |
*** avivgt has joined #tripleo | 13:34 | |
bogdando | couldn't fine a better word for that noise sshnaidm | 13:34 |
bogdando | find* | 13:35 |
bogdando | we can call it distributed systems fee :) | 13:35 |
*** amoralej|off is now known as amoralej | 13:35 | |
*** artom has quit IRC | 13:37 | |
*** nyechiel_ has joined #tripleo | 13:39 | |
*** tcw has joined #tripleo | 13:41 | |
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:44 |
sshnaidm | bogdando, all you described is either infra failure and should be communicated to openstack-infra or product failures | 13:44 |
Tengu | jaosorior: doing the thing in ansible will be fun :). anyway. I call it a day, see you tomorrow! | 13:44 |
sshnaidm | bogdando, "noise" doesn't exist in CI :) | 13:44 |
jaosorior | Tengu: have a good one! | 13:45 |
sshnaidm | bogdando, if it's product failure we need to submit a bug and fix it, no matter related it to patch or not | 13:45 |
bogdando | sshnaidm: each time elastic recheck posts a comment it is highly likely cuz it saw that noise :) | 13:45 |
openstackgerrit | Merged openstack/python-tripleoclient master: Start standalone.conf https://review.openstack.org/569474 | 13:45 |
openstackgerrit | Merged openstack/python-tripleoclient master: Move undercloud.conf to the undercloud config namespace https://review.openstack.org/569477 | 13:45 |
bogdando | I'm mostly concerned of that noise amplification, not the product defects part of the issue | 13:45 |
sshnaidm | bogdando, elastic recheck is not related here, it posts completely non relevant stuff, I was always against it | 13:46 |
bogdando | you're right for the latter | 13:46 |
*** anilvenkata has quit IRC | 13:46 | |
bogdando | tho, the former takes a significant part of time we spend on moving patches forward | 13:46 |
bogdando | as well | 13:46 |
*** ykarel|away has quit IRC | 13:46 | |
sshnaidm | bogdando, product failures should be resolved, that's purpose of CI - to detect them. CI doesn't invent them.. | 13:47 |
sshnaidm | bogdando, if it's not fixed - it's not CI guilt, it's just a "messenger" :D | 13:48 |
openstackgerrit | Merged openstack/tripleo-common master: Add an "includes" prepare filter to match "excludes" https://review.openstack.org/569688 | 13:48 |
sshnaidm | bogdando, if you can't move patch forward, it's a time to solve all remaining problems that exist now.. | 13:49 |
sshnaidm | bogdando, and to cover them in additional tests so we can catch it later | 13:49 |
sshnaidm | bogdando, this is a normal development process as I always thought.. | 13:50 |
*** udesale has quit IRC | 13:51 | |
*** udesale has joined #tripleo | 13:51 | |
sshnaidm | jfrancoa, rlandy|rover arxcruz|ruck fyi: https://bugs.launchpad.net/tripleo/+bug/1773179 | 13:54 |
openstack | Launchpad bug 1773179 in tripleo "Undercloud upgrades fails in ocata because nova service failure" [High,Triaged] - Assigned to Ronelle Landy (rlandy) | 13:54 |
*** moguimar has quit IRC | 13:54 | |
*** tesseract-RH has joined #tripleo | 13:54 | |
jfrancoa | sshnaidm: I was rising the issue right now in the upgrades scrum call. It looks like the cause is that we don't have stable/ocata branch for tripleo-upgrade. So zuul is downloading master branch | 13:55 |
EmilienM | numans: sure | 13:55 |
*** tesseract-RH has quit IRC | 13:55 | |
jfrancoa | sshnaidm: and the undercloud upgrade process is not the same | 13:55 |
*** tesseract-RH has joined #tripleo | 13:55 | |
sshnaidm | jfrancoa, hmm.. so we need to remove this job from ocata? | 13:55 |
jfrancoa | sshnaidm: exactly, that's what I was going to ask you | 13:55 |
openstackgerrit | Merged openstack/tripleo-heat-templates master: Fix upgrade_tasks for TripleO UI https://review.openstack.org/570237 | 13:55 |
*** tesseract has quit IRC | 13:56 | |
sshnaidm | jfrancoa, I'll submit a patch and let's discuss it, I want to be sure we don't delete it for mistake.. | 13:56 |
trozet | mwhahaha: can you help this tripleo noob understand what failed in this log http://logs.openstack.org/05/486905/37/experimental/tripleo-ci-centos-7-scenario008-multinode-oooq/cc41774/logs/undercloud/home/zuul/overcloud_deploy.log.txt.gz | 13:57 |
jfrancoa | sshnaidm: great, thanks | 13:57 |
rlandy|rover | sshnaidm: you're addressing the bug? | 13:58 |
sshnaidm | rlandy|rover, yes | 13:58 |
mwhahaha | trozet: it seems to have timed out during the deploy? | 13:58 |
rlandy|rover | k - thanks | 13:58 |
*** pchavva has joined #tripleo | 13:58 | |
trozet | mwhahaha: what do you see that tells you that, cause i went over this log like 4x now | 13:58 |
mwhahaha | trozet: because it didn't finish | 13:58 |
mwhahaha | trozet: the log just stopped mid deploy | 13:59 |
*** derekh has quit IRC | 13:59 | |
trozet | mwhahaha: :/ | 13:59 |
mwhahaha | trozet: it was deploying Overcloud common deploy step tasks 3 which is the middle | 13:59 |
*** olivierb- has quit IRC | 13:59 | |
trozet | mwhahaha: so i guess i'll just try again? weird thing is the regular multinode job seems to pass everytime...i wonder if it could be a RAM issue? | 14:00 |
trozet | mwhahaha: ODL takes up 2.5 GB | 14:00 |
marios | beagles: was in a call ack thanks will check | 14:00 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-quickstart master: DNM - Revert "fs050: enable upgrade cleanup" https://review.openstack.org/570395 | 14:00 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-quickstart master: DNM - Revert "fs050: enable upgrade cleanup" https://review.openstack.org/570395 | 14:00 |
beagles | marios: thanks -just as well, I added a line since ;) | 14:01 |
*** ohochman has quit IRC | 14:01 | |
mwhahaha | trozet: lol 2.5g? | 14:01 |
trozet | mwhahaha: yeah java so efficient with garbage collection ya know? | 14:01 |
trozet | heap for days...or minutes... | 14:02 |
jistr|mtg | green sc000 upgrade job from Queens to master! Any takers? please merge https://review.openstack.org/#/c/558425/ and the 2 depends-on patches | 14:02 |
*** jistr|mtg is now known as jistr | 14:02 | |
*** leanderthal has quit IRC | 14:02 | |
jistr | GREEN UPGRADE JOBS FOR EVERYONE | 14:02 |
* jistr sellin it | 14:02 | |
openstackgerrit | Lukas Bezdicka proposed openstack/tripleo-heat-templates master: Upgrades: Refactor httpd_enabled variable https://review.openstack.org/555781 | 14:03 |
openstackgerrit | Lukas Bezdicka proposed openstack/tripleo-heat-templates master: Upgrades: Refactor playbooks to set facts https://review.openstack.org/562294 | 14:03 |
openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci master: Don't run undercloud-upgrades on ocata https://review.openstack.org/570398 | 14:03 |
sshnaidm | jfrancoa, rlandy|rover ^^ | 14:04 |
*** leanderthal has joined #tripleo | 14:04 | |
jfrancoa | sshnaidm: ack, thanks | 14:05 |
trozet | mwhahaha: you know how much ram these nodes have? or how to increase it...im looking for examples now | 14:05 |
*** lblanchard has joined #tripleo | 14:05 | |
mwhahaha | trozet: probably 8g and no | 14:06 |
trozet | trown:^^ | 14:06 |
trozet | probably here im thinking https://review.openstack.org/#/c/500872/11/config/general_config/featureset031.yml | 14:07 |
*** slaweq has joined #tripleo | 14:08 | |
*** marrusl has joined #tripleo | 14:12 | |
*** olap has quit IRC | 14:13 | |
*** sai_ has quit IRC | 14:15 | |
*** rook has quit IRC | 14:15 | |
*** sai_ has joined #tripleo | 14:17 | |
*** rook has joined #tripleo | 14:17 | |
*** rook is now known as Guest11079 | 14:18 | |
*** rpioso|afk is now known as rpioso | 14:18 | |
*** dbecker has quit IRC | 14:18 | |
*** edmondsw has joined #tripleo | 14:20 | |
*** olivierbourdon38 has joined #tripleo | 14:20 | |
openstackgerrit | Tim Rozet proposed openstack/tripleo-heat-templates master: Add OPNFV scenario environment https://review.openstack.org/486905 | 14:21 |
*** derekh has joined #tripleo | 14:21 | |
*** Guest11079 has quit IRC | 14:26 | |
*** sri_ has quit IRC | 14:28 | |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-quickstart master: Revert "fs050: enable upgrade cleanup" https://review.openstack.org/570395 | 14:29 |
*** rook_ has joined #tripleo | 14:29 | |
*** sri_ has joined #tripleo | 14:30 | |
*** pblaho has quit IRC | 14:30 | |
shardy | trozet: the multinode jobs use deployed_server: true e.g the nodes already exist and are provided by nodepool | 14:30 |
shardy | trozet: I don't think you can increase the memory without increasing it for all jobs | 14:31 |
trozet | shardy: happen to know how much ram? | 14:31 |
shardy | 8G IIRC, sec checking | 14:31 |
*** pblaho has joined #tripleo | 14:32 | |
*** dparkes has quit IRC | 14:32 | |
shardy | https://github.com/openstack-infra/project-config/blob/master/nodepool/nl01.openstack.org.yaml#L62 | 14:33 |
shardy | trozet: I think that nodepool stuff is used but I'm not super familiar with it | 14:34 |
shardy | If you need more it may have to be via 3rd party coverage I guess | 14:34 |
shardy | and/or disable services until you have enough spare | 14:34 |
trozet | shardy: ok we could trigger 3rd party opnfv if this is a blocker. I just rechecked again lets see if it passes | 14:35 |
trozet | shardy: thanks for your help | 14:35 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: nova-ironic: add support for post-upgrade cleanup https://review.openstack.org/570241 | 14:36 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: WIP - add missing services for upgrade cleanup https://review.openstack.org/570283 | 14:36 |
shardy | trozet: np - I'm not sure how much memory is available for the RDO 3rd party jobs | 14:36 |
trozet | shardy: i mean not RDO 3rd party, but create a new 3rd party which triggers OPNFV jenkins jobs | 14:36 |
shardy | trozet: ack yes I guess that would also work, I was looking for an existing job you could make use of | 14:37 |
trozet | shardy: yeah | 14:37 |
*** lvdombrkr has quit IRC | 14:38 | |
openstackgerrit | Emilien Macchi proposed openstack/python-tripleoclient master: tripleo_deploy/upgrade: fail of upgrade_tasks don't return 0 https://review.openstack.org/570321 | 14:40 |
*** dparkes has joined #tripleo | 14:40 | |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Check container health as part of the deploy https://review.openstack.org/569153 | 14:41 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Check container health as part of the deploy https://review.openstack.org/569153 | 14:41 |
*** masco has quit IRC | 14:42 | |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Parameterized deployment hosts https://review.openstack.org/566752 | 14:42 |
*** etingof has quit IRC | 14:43 | |
*** rleander_ has joined #tripleo | 14:44 | |
*** mvenesio has quit IRC | 14:45 | |
*** leitan has joined #tripleo | 14:45 | |
openstackgerrit | Emilien Macchi proposed openstack/python-tripleoclient master: WIP: Handle config loading in tripleo deploy https://review.openstack.org/569535 | 14:45 |
*** leanderthal has quit IRC | 14:46 | |
*** tesseract-RH has quit IRC | 14:47 | |
*** zoli|afk is now known as zoli | 14:48 | |
*** zoli is now known as zoli|wfh | 14:48 | |
*** zoli|wfh is now known as zoli | 14:48 | |
trozet | shardy, dsneddon: what is the name of the script in tht to generate the os-net-config script from the nic template? | 14:49 |
*** rleander_ has quit IRC | 14:49 | |
*** jtomasek has joined #tripleo | 14:55 | |
shardy | trozet: https://github.com/openstack/tripleo-heat-templates/blob/master/network/scripts/run-os-net-config.sh | 14:56 |
*** artom has joined #tripleo | 14:56 | |
*** abishop has quit IRC | 14:56 | |
*** dbecker has joined #tripleo | 14:57 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-tripleo master: Update netapp cinder configuration https://review.openstack.org/570406 | 14:57 |
*** maufart__ has quit IRC | 15:00 | |
jfrancoa | jaosorior: hey, do you have a moment for a doubt regarding Barbican docker images in tripleo-common? | 15:01 |
*** slaweq has quit IRC | 15:01 | |
*** matbu has quit IRC | 15:01 | |
*** tesseract-RH has joined #tripleo | 15:02 | |
bogdando | shardy: could you please comment or correct me for https://bugzilla.redhat.com/show_bug.cgi?id=1570938#c13 ? | 15:03 |
openstack | bugzilla.redhat.com bug 1570938 in python-tripleoclient "[OSP13] Overcloud deployment fail when attempting to call heat-templates not from their default path (/usr/share/openstack-tripleo-heat-templates)." [High,Assigned] - Assigned to dpeacock | 15:03 |
*** tesseract-RH has quit IRC | 15:03 | |
*** dbecker has quit IRC | 15:03 | |
*** tesseract has joined #tripleo | 15:03 | |
openstackgerrit | Alex Schultz proposed openstack/tripleo-heat-templates master: Don't run swift on 3node https://review.openstack.org/570275 | 15:05 |
openstackgerrit | Oliver Walsh proposed openstack/instack-undercloud stable/ocata: WIP: prevent duplicate default cells being created by ocata upgrade https://review.openstack.org/570081 | 15:05 |
shardy | bogdando: I'll comment, I think the simpler answer is any environment files which are j2 rendered must be in the --templates path | 15:09 |
*** ooolpbot has joined #tripleo | 15:10 | |
ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 15:10 |
ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1773188 | 15:10 |
*** ooolpbot has quit IRC | 15:10 | |
openstack | Launchpad bug 1773188 in tripleo "puppet-tripleo unit tests broken due to recent upstream puppet module cleanup" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz) | 15:10 |
openstackgerrit | Alex Schultz proposed openstack/puppet-tripleo master: Update netapp cinder configuration https://review.openstack.org/570406 | 15:10 |
*** matbu has joined #tripleo | 15:10 | |
*** dxiri has joined #tripleo | 15:10 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-tripleo master: Rework GDPR compliant logrotate config https://review.openstack.org/569976 | 15:11 |
trozet | shardy: yeah i thought there was some script that would render that script wiht the params from the template | 15:14 |
*** yamahata has quit IRC | 15:15 | |
bogdando | shardy: thanks! I also linked related LP | 15:17 |
bogdando | that's so tricky... | 15:17 |
bogdando | totally mind-blowing :D | 15:17 |
openstackgerrit | Jose Luis Franco proposed openstack/tripleo-quickstart master: Remove ceph from major upgrade jobs. https://review.openstack.org/568229 | 15:19 |
shardy | bogdando: yeah, it's a historical mistake that we allowed unconstrained -e paths, which causes a lot of complexity in tripleoclient | 15:19 |
*** dbecker has joined #tripleo | 15:19 | |
*** pcaruana has quit IRC | 15:19 | |
*** rook_ is now known as rook | 15:20 | |
*** slaweq has joined #tripleo | 15:25 | |
*** dparkes has quit IRC | 15:26 | |
*** kopecmartin has quit IRC | 15:26 | |
*** leanderthal has joined #tripleo | 15:27 | |
*** CristinaPauna has quit IRC | 15:29 | |
*** bogdando has quit IRC | 15:30 | |
*** ccamacho has quit IRC | 15:31 | |
*** CristinaPauna has joined #tripleo | 15:32 | |
*** gkadam has quit IRC | 15:32 | |
*** edmondsw has quit IRC | 15:33 | |
*** tcw has quit IRC | 15:34 | |
*** jtomasek has quit IRC | 15:35 | |
*** ukalifon has quit IRC | 15:36 | |
*** gvrangan has joined #tripleo | 15:37 | |
*** gvrangan has quit IRC | 15:38 | |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-quickstart master: Revert "Revert "Deploy container-multinode (fs010) with a containerized undercloud"" https://review.openstack.org/560264 | 15:38 |
EmilienM | chandankumar: could you look at my tempest error? | 15:38 |
*** janki has quit IRC | 15:40 | |
*** slaweq has quit IRC | 15:40 | |
*** gvrangan has joined #tripleo | 15:41 | |
*** quiquell is now known as quiquell|off | 15:42 | |
*** rbrady has quit IRC | 15:42 | |
*** gvrangan has quit IRC | 15:45 | |
*** gvrangan_ has joined #tripleo | 15:45 | |
*** mburned is now known as mburned_out | 15:45 | |
*** matbu has quit IRC | 15:48 | |
*** ykarel|away has joined #tripleo | 15:51 | |
*** gkadam has joined #tripleo | 15:53 | |
panda | jistr: do you have something in review for the fs000 full upgrade ? | 15:53 |
panda | job* | 15:53 |
*** gkadam has quit IRC | 15:54 | |
jistr | panda: yes, this and the 2 depends-on https://review.openstack.org/#/c/558425/ | 15:54 |
*** slaweq has joined #tripleo | 15:54 | |
panda | jistr: thanks | 15:55 |
*** marios has quit IRC | 15:55 | |
*** tcw has joined #tripleo | 15:56 | |
*** sshnaidm has quit IRC | 15:56 | |
panda | jistr: and this is for the job that upgrades both undercloud and overcloud ? | 15:57 |
*** edmondsw_ has joined #tripleo | 15:58 | |
jistr | panda: just overcloud, just keystone + dependencies | 15:58 |
jistr | even that has a run time between 2:30 and 2:45 | 15:58 |
*** abishop has joined #tripleo | 15:58 | |
panda | jistr: ok so, did you discuss with was a full upgrade of undercloud and overcloud at all ? | 15:59 |
*** edmondsw_ has quit IRC | 15:59 | |
panda | Wes* | 15:59 |
*** edmondsw_ has joined #tripleo | 15:59 | |
*** pkovar has quit IRC | 15:59 | |
panda | jistr: you ever discussed if a similar job was worth and added any benefit ? | 16:00 |
*** ykarel|away has quit IRC | 16:00 | |
*** tcw has quit IRC | 16:01 | |
*** tcw has joined #tripleo | 16:01 | |
*** dbecker has quit IRC | 16:02 | |
jistr | panda: yes we did. It fully validates the upgrade procedure as far as the framework itself is concerned. We already have the same job but for updates, it's gating now. | 16:02 |
panda | jistr: ok, so you don't have a review for the full undercloud + overcloud job. At this point will need to be put in as third party, there's no way we can fit the current timeout | 16:03 |
*** gbarros has joined #tripleo | 16:04 | |
*** pblaho has quit IRC | 16:04 | |
*** pblaho has joined #tripleo | 16:04 | |
*** ramishra has quit IRC | 16:06 | |
jistr | panda: yes that's fine, not much we can do better there unless we manage to increase the gating time limit. But even if we can just get the sc000 upgrade job gating, it would be good because e.g. we were broken by tls by default, and config-download by default. Both of those breakages would be prevented | 16:07 |
*** matbu has joined #tripleo | 16:08 | |
*** slaweq_ has joined #tripleo | 16:08 | |
*** raildo has joined #tripleo | 16:09 | |
*** dmellado has quit IRC | 16:09 | |
*** avivgt has quit IRC | 16:09 | |
*** ccamacho has joined #tripleo | 16:09 | |
*** gouthamr has quit IRC | 16:09 | |
*** ooolpbot has joined #tripleo | 16:10 | |
ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 16:10 |
ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1773188 | 16:10 |
*** ooolpbot has quit IRC | 16:10 | |
openstack | Launchpad bug 1773188 in tripleo "puppet-tripleo unit tests broken due to recent upstream puppet module cleanup" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz) | 16:10 |
*** ohamada has quit IRC | 16:11 | |
*** slaweq has quit IRC | 16:11 | |
*** zshi has joined #tripleo | 16:12 | |
*** stevebaker has quit IRC | 16:14 | |
*** jaganathan has quit IRC | 16:19 | |
*** leitan has quit IRC | 16:21 | |
*** gvrangan_ has quit IRC | 16:21 | |
*** leitan has joined #tripleo | 16:22 | |
*** lucasagomes is now known as lucas-afk | 16:23 | |
*** lucas-afk is now known as lucasagomes | 16:23 | |
*** pcaruana has joined #tripleo | 16:23 | |
*** alee has quit IRC | 16:25 | |
*** gvrangan has joined #tripleo | 16:26 | |
*** leitan has quit IRC | 16:26 | |
*** edmondsw has joined #tripleo | 16:29 | |
*** trown is now known as trown|lunch | 16:29 | |
openstackgerrit | Thomas Herve proposed openstack/puppet-tripleo master: Automatically configure neutron with designate https://review.openstack.org/569987 | 16:29 |
*** edmondsw has quit IRC | 16:30 | |
*** edmondsw has joined #tripleo | 16:30 | |
*** ffiore has quit IRC | 16:31 | |
openstackgerrit | John Fulton proposed openstack/tripleo-heat-templates master: Set default application for Ceph Luminous openstack_pools https://review.openstack.org/570043 | 16:32 |
*** edmondsw_ has quit IRC | 16:32 | |
*** pcaruana has quit IRC | 16:32 | |
*** edmondsw has quit IRC | 16:34 | |
*** edmondsw has joined #tripleo | 16:34 | |
*** leanderthal has quit IRC | 16:35 | |
*** holser__ has quit IRC | 16:37 | |
*** slaweq has joined #tripleo | 16:38 | |
*** slaweq_ has quit IRC | 16:41 | |
*** zaneb has joined #tripleo | 16:42 | |
*** lifeless has quit IRC | 16:43 | |
*** udesale has quit IRC | 16:44 | |
bandini | meh getting LP timeouts | 16:45 |
therve | Well I get rdo cloud 500 | 16:47 |
therve | The cloud is down, let's go home | 16:47 |
*** vpickard is now known as vpickard_ | 16:47 | |
*** matbu has quit IRC | 16:48 | |
*** slaweq has quit IRC | 16:48 | |
*** gvrangan has quit IRC | 16:48 | |
*** rbrady has joined #tripleo | 16:48 | |
*** slaweq has joined #tripleo | 16:49 | |
openstackgerrit | Michele Baldessari proposed openstack/puppet-tripleo master: WIP Fix up property names in case of mixed case hostnames https://review.openstack.org/570413 | 16:49 |
bandini | :) | 16:49 |
*** rbrady is now known as Guest3295 | 16:49 | |
*** sshnaidm has joined #tripleo | 16:49 | |
*** sshnaidm is now known as sshnaidm|off | 16:50 | |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Upgrades: Refactor httpd_enabled variable https://review.openstack.org/555781 | 16:50 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Upgrades: Refactor playbooks to set facts https://review.openstack.org/562294 | 16:51 |
Tengu | hey guys, PTAL? :) https://review.openstack.org/#/c/570209/ you want to get a proper single-nic IPv6 in quickstart, don't you? :) | 16:53 |
openstackgerrit | Alan Bishop proposed openstack/puppet-tripleo master: Add missing nfs_mount_options for Cinder Netapp backend https://review.openstack.org/570256 | 16:54 |
*** zaneb has quit IRC | 16:55 | |
*** alee has joined #tripleo | 16:56 | |
*** olivierbourdon38 has quit IRC | 16:56 | |
*** olivierbourdon38 has joined #tripleo | 17:00 | |
*** derekh has quit IRC | 17:00 | |
openstackgerrit | Michele Baldessari proposed openstack/puppet-tripleo master: WIP Fix up property names in case of mixed case hostnames https://review.openstack.org/570413 | 17:02 |
*** edmondsw_ has joined #tripleo | 17:02 | |
*** amoralej is now known as amoralej|off | 17:03 | |
*** gouthamr has joined #tripleo | 17:04 | |
*** matbu has joined #tripleo | 17:04 | |
*** edmondsw has quit IRC | 17:06 | |
*** gbarros has quit IRC | 17:07 | |
*** gvrangan has joined #tripleo | 17:08 | |
*** slaweq_ has joined #tripleo | 17:08 | |
*** anilvenkata has joined #tripleo | 17:10 | |
*** ooolpbot has joined #tripleo | 17:10 | |
ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 17:10 |
ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1773188 | 17:10 |
openstack | Launchpad bug 1773188 in tripleo "puppet-tripleo unit tests broken due to recent upstream puppet module cleanup" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz) | 17:10 |
*** ooolpbot has quit IRC | 17:10 | |
cmyster | .8 | 17:10 |
*** jfrancoa has quit IRC | 17:11 | |
*** dtantsur is now known as dtantsur|afk | 17:11 | |
*** slaweq has quit IRC | 17:12 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo master: Update netapp cinder configuration https://review.openstack.org/570406 | 17:13 |
EmilienM | mwhahaha: fixed lint ^ | 17:13 |
*** fragatin_ has quit IRC | 17:14 | |
mwhahaha | K | 17:14 |
*** stevebaker has joined #tripleo | 17:14 | |
*** zaneb has joined #tripleo | 17:15 | |
*** slaweq_ has quit IRC | 17:15 | |
pabelanger | EmilienM: where about's are you at the summit? Wanted to drop by and discuss something | 17:15 |
EmilienM | pabelanger: 220 | 17:16 |
EmilienM | pabelanger: I'm free in 5 min | 17:17 |
openstackgerrit | Alan Bishop proposed openstack/puppet-tripleo master: Add missing nfs_mount_options for Cinder Netapp backend https://review.openstack.org/570256 | 17:18 |
openstackgerrit | Merged openstack/tripleo-heat-templates stable/queens: FFU Set NetworkDeploymentActions CREATE,UPDATE for ffwd-upgrade prepare https://review.openstack.org/568236 | 17:19 |
pabelanger | EmilienM: okay, just outside | 17:20 |
*** zaneb has quit IRC | 17:22 | |
EmilienM | pabelanger: here in 2 min | 17:22 |
openstackgerrit | James Slagle proposed openstack/tripleo-heat-templates stable/queens: Convert resource_name to number https://review.openstack.org/570418 | 17:22 |
*** panda is now known as panda|off | 17:23 | |
*** tesseract has quit IRC | 17:25 | |
*** jpena is now known as jpena|off | 17:28 | |
openstackgerrit | James Slagle proposed openstack/tripleo-heat-templates stable/pike: Convert resource_name to number https://review.openstack.org/570419 | 17:28 |
*** gvrangan has quit IRC | 17:28 | |
openstackgerrit | James Slagle proposed openstack/tripleo-heat-templates stable/ocata: Convert resource_name to number https://review.openstack.org/570420 | 17:29 |
*** edmondsw_ has quit IRC | 17:31 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-tripleo master: Rework GDPR compliant logrotate config https://review.openstack.org/569976 | 17:31 |
*** edmondsw has joined #tripleo | 17:31 | |
*** fragatina has joined #tripleo | 17:35 | |
*** edmondsw has quit IRC | 17:36 | |
*** fragatin_ has joined #tripleo | 17:36 | |
*** fragatina has quit IRC | 17:36 | |
*** dparkes has joined #tripleo | 17:38 | |
*** gfidente has quit IRC | 17:39 | |
trozet | mwhahaha: just to clarify...multinode-oooq means HA right? | 17:40 |
mwhahaha | trozet: not necessarily | 17:40 |
trozet | mwhahaha: well what decides that? | 17:41 |
mwhahaha | trozet: multinode means 2 nodes in CI (one undercloud, one overcloud) | 17:41 |
mwhahaha | trozet: so depends on what you mean 'ha' | 17:41 |
trozet | mwhahaha: HA i mean 3 controllers | 17:41 |
mwhahaha | trozet: if you mean >1 controller, that's only ovb | 17:41 |
trozet | mwhahaha: oh, so then there is no reason to even include pacemaker | 17:41 |
mwhahaha | well not necessarily | 17:41 |
mwhahaha | we should be able to run 1 controller w/ pacemaker | 17:42 |
*** dmacpher has quit IRC | 17:42 | |
trozet | mwhahaha: but theres no reason to | 17:42 |
trozet | mwhahaha: so i can just remove that from the scenario | 17:42 |
*** dmacpher has joined #tripleo | 17:42 | |
mwhahaha | if it's your scenario, sure | 17:42 |
*** raildo has quit IRC | 17:42 | |
trozet | mwhahaha: cause ODL isnt managed by pacemaker anyway | 17:42 |
mwhahaha | k | 17:42 |
*** gbarros has joined #tripleo | 17:42 | |
trozet | mwhahaha: whats the valeu of using pacemaker in a non-ha deployment for other scenarios? | 17:43 |
mwhahaha | trozet: we enable it by default downstream | 17:43 |
trozet | mwhahaha: yeah, i never knew why that was | 17:43 |
mwhahaha | trozet: so it's a type of configuration that could be use | 17:43 |
mwhahaha | because it shouldn't matter | 17:44 |
mwhahaha | why do you want to disable it? | 17:44 |
mwhahaha | perf problems or something? | 17:44 |
*** trown|lunch is now known as trown | 17:44 | |
trozet | mwhahaha: well if its not doing anything, why is it running :) | 17:44 |
mwhahaha | because it's a different code path | 17:44 |
mwhahaha | and pulls in different services | 17:44 |
trozet | mwhahaha: oh so for code path consistencty between deployments | 17:45 |
trozet | i could see htat | 17:45 |
mwhahaha | it's ok if you want to turn it off in the odl scenario for upstream ci | 17:45 |
trozet | mwhahaha: is it possible after we get thsi scenario working to make an ovb thing as well so we can have HA and non-ha coverage for ODL? | 17:46 |
mwhahaha | just as long as you test it somewhere :D | 17:46 |
*** alee has quit IRC | 17:46 | |
mwhahaha | not sure, we'd have to check with the ci folks if we can do that | 17:46 |
*** beagles is now known as beagles_brb | 17:46 | |
openstackgerrit | Tim Rozet proposed openstack/tripleo-quickstart master: Updates OpenDaylight feature set 31 https://review.openstack.org/500872 | 17:47 |
*** gbarros has quit IRC | 17:47 | |
*** alee has joined #tripleo | 17:53 | |
*** edmondsw has joined #tripleo | 17:53 | |
*** slaweq has joined #tripleo | 17:53 | |
*** dhill has joined #tripleo | 17:53 | |
*** dhill is now known as Guest96125 | 17:54 | |
*** matbu has quit IRC | 17:54 | |
*** wolverineav has quit IRC | 17:55 | |
*** edmondsw has quit IRC | 17:58 | |
openstackgerrit | Tim Rozet proposed openstack/tripleo-heat-templates master: Add OPNFV scenario environment https://review.openstack.org/486905 | 18:00 |
*** loacker has joined #tripleo | 18:00 | |
*** wolverineav has joined #tripleo | 18:01 | |
*** psachin has quit IRC | 18:02 | |
*** wolverineav has quit IRC | 18:04 | |
*** wolverineav has joined #tripleo | 18:04 | |
*** itlinux has joined #tripleo | 18:06 | |
*** zoli is now known as zoli|gone | 18:08 | |
*** zoli|gone is now known as zoli | 18:08 | |
*** wolverineav has quit IRC | 18:09 | |
*** ooolpbot has joined #tripleo | 18:10 | |
ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 18:10 |
ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1773188 | 18:10 |
*** ooolpbot has quit IRC | 18:10 | |
openstack | Launchpad bug 1773188 in tripleo "puppet-tripleo unit tests broken due to recent upstream puppet module cleanup" [Critical,In progress] - Assigned to Emilien Macchi (emilienm) | 18:10 |
*** salmankhan has quit IRC | 18:14 | |
*** loacker has quit IRC | 18:14 | |
*** slaweq_ has joined #tripleo | 18:15 | |
openstackgerrit | John Fulton proposed openstack/tripleo-heat-templates master: Set default application for Ceph Luminous openstack_pools https://review.openstack.org/570043 | 18:16 |
openstackgerrit | John Fulton proposed openstack/tripleo-heat-templates master: Set default application for Ceph Luminous openstack_pools https://review.openstack.org/570043 | 18:17 |
*** slaweq has quit IRC | 18:18 | |
*** slaweq_ has quit IRC | 18:20 | |
*** shreshtha has quit IRC | 18:21 | |
*** leitan has joined #tripleo | 18:22 | |
*** itlinux has quit IRC | 18:22 | |
*** matbu has joined #tripleo | 18:26 | |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Upgrades: Refactor playbooks to set facts https://review.openstack.org/562294 | 18:26 |
*** leitan has quit IRC | 18:26 | |
*** abishop has quit IRC | 18:29 | |
*** morazi has quit IRC | 18:31 | |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Upgrades: Refactor playbooks to set facts https://review.openstack.org/562294 | 18:34 |
*** anilvenkata has quit IRC | 18:36 | |
*** rajinir has joined #tripleo | 18:37 | |
*** beagles_brb is now known as beagles | 18:40 | |
*** alee has quit IRC | 18:40 | |
*** leitan has joined #tripleo | 18:44 | |
*** leitan has joined #tripleo | 18:45 | |
*** leitan has quit IRC | 18:45 | |
sshnaidm|off | rlandy|rover, trown https://review.openstack.org/#/c/570398/ | 18:45 |
*** leitan has joined #tripleo | 18:45 | |
*** leitan has joined #tripleo | 18:46 | |
*** ansmith has quit IRC | 18:48 | |
*** slaweq has joined #tripleo | 18:50 | |
*** slaweq has quit IRC | 18:50 | |
*** slaweq has joined #tripleo | 18:50 | |
*** alee has joined #tripleo | 18:52 | |
*** edmondsw has joined #tripleo | 18:53 | |
*** wolverineav has joined #tripleo | 18:57 | |
*** edmondsw has quit IRC | 18:58 | |
*** slaweq has quit IRC | 19:02 | |
*** matbu has quit IRC | 19:06 | |
*** ChanServ has quit IRC | 19:07 | |
*** ooolpbot has joined #tripleo | 19:10 | |
ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 19:10 |
ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1773188 | 19:10 |
*** ooolpbot has quit IRC | 19:10 | |
openstack | Launchpad bug 1773188 in tripleo "puppet-tripleo unit tests broken due to recent upstream puppet module cleanup" [Critical,In progress] - Assigned to Emilien Macchi (emilienm) | 19:10 |
*** ChanServ has joined #tripleo | 19:12 | |
*** barjavel.freenode.net sets mode: +o ChanServ | 19:12 | |
*** nyechiel_ has quit IRC | 19:14 | |
*** gbarros has joined #tripleo | 19:15 | |
*** dbecker has joined #tripleo | 19:15 | |
*** Guest96125 has quit IRC | 19:16 | |
*** suuuper has quit IRC | 19:18 | |
*** ChanServ has quit IRC | 19:21 | |
*** zaneb has joined #tripleo | 19:22 | |
*** bcafarel has quit IRC | 19:24 | |
*** bcafarel has joined #tripleo | 19:25 | |
*** gvrangan has joined #tripleo | 19:25 | |
*** ChanServ has joined #tripleo | 19:30 | |
*** barjavel.freenode.net sets mode: +o ChanServ | 19:30 | |
*** melwitt is now known as Guest32198 | 19:33 | |
*** beagles is now known as Guest87851 | 19:34 | |
*** fungi is now known as Guest77695 | 19:34 | |
*** sdake is now known as Guest14735 | 19:34 | |
*** v1k0d3n is now known as Guest45438 | 19:34 | |
*** dmsimard is now known as Guest94448 | 19:34 | |
*** flaper87 is now known as Guest62139 | 19:34 | |
*** honza is now known as Guest60997 | 19:34 | |
*** tristanC is now known as Guest15248 | 19:34 | |
*** redrobot is now known as Guest5327 | 19:34 | |
*** vkmc has quit IRC | 19:34 | |
*** vkmc has joined #tripleo | 19:34 | |
*** vkmc has quit IRC | 19:34 | |
*** vkmc has joined #tripleo | 19:34 | |
*** salmankhan has joined #tripleo | 19:34 | |
*** slaweq has joined #tripleo | 19:35 | |
*** alee has quit IRC | 19:35 | |
*** edmondsw has joined #tripleo | 19:35 | |
*** Guest77695 is now known as fungi | 19:35 | |
*** zaneb has quit IRC | 19:36 | |
*** bandini has quit IRC | 19:36 | |
*** zshi has quit IRC | 19:37 | |
*** bandini has joined #tripleo | 19:38 | |
*** edmondsw has quit IRC | 19:39 | |
*** slaweq has quit IRC | 19:39 | |
*** rlandy|rover is now known as rlandy | 19:42 | |
*** rlandy is now known as rlandy|rover | 19:43 | |
*** gvrangan has quit IRC | 19:48 | |
*** raildo has joined #tripleo | 19:49 | |
*** Guest94448 has quit IRC | 19:51 | |
*** salmankhan has quit IRC | 19:51 | |
*** dmsimard has joined #tripleo | 19:52 | |
*** gvrangan has joined #tripleo | 19:56 | |
*** salmankhan has joined #tripleo | 19:56 | |
openstackgerrit | Merged openstack/tripleo-upgrade master: Pass --ssh-user option during fast forward upgrade run https://review.openstack.org/570245 | 19:57 |
*** Guest5327 is now known as redrobot | 19:57 | |
*** gvrangan has quit IRC | 19:58 | |
*** gvrangan has joined #tripleo | 20:00 | |
*** zaneb has joined #tripleo | 20:00 | |
openstackgerrit | Marius Cornea proposed openstack/tripleo-upgrade stable/queens: Pass --ssh-user option during fast forward upgrade run https://review.openstack.org/570435 | 20:01 |
openstackgerrit | Bob Fournier proposed openstack/tripleo-docs master: Document how to use a composable network with Ironic in Overcloud https://review.openstack.org/570437 | 20:07 |
*** zaneb has quit IRC | 20:07 | |
*** gbarros has quit IRC | 20:07 | |
*** matbu has joined #tripleo | 20:09 | |
*** ooolpbot has joined #tripleo | 20:10 | |
ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 20:10 |
ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1773188 | 20:10 |
*** ooolpbot has quit IRC | 20:10 | |
openstack | Launchpad bug 1773188 in tripleo "puppet-tripleo unit tests broken due to recent upstream puppet module cleanup" [Critical,In progress] - Assigned to Emilien Macchi (emilienm) | 20:10 |
*** salmankhan has quit IRC | 20:11 | |
*** dhill has joined #tripleo | 20:12 | |
*** dhill is now known as Guest64332 | 20:12 | |
*** etingof has joined #tripleo | 20:13 | |
openstackgerrit | Bob Fournier proposed openstack/tripleo-docs master: Document how to use a composable network with Ironic in Overcloud https://review.openstack.org/570437 | 20:19 |
*** slaweq has joined #tripleo | 20:22 | |
*** dbecker has quit IRC | 20:24 | |
*** lblanchard has quit IRC | 20:25 | |
*** trown is now known as trown|outtypewww | 20:27 | |
openstackgerrit | Michele Baldessari proposed openstack/puppet-tripleo master: Fix up property names in case of mixed case hostnames https://review.openstack.org/570413 | 20:32 |
bandini | mandre: ^ | 20:33 |
*** artom has quit IRC | 20:34 | |
openstackgerrit | Steve Baker proposed openstack/ansible-role-tripleo-modify-image master: Test new repo https://review.openstack.org/570444 | 20:34 |
openstackgerrit | Bob Fournier proposed openstack/tripleo-docs master: Document how to use a composable network with Ironic in Overcloud https://review.openstack.org/570437 | 20:35 |
openstackgerrit | Steve Baker proposed openstack/ansible-role-tripleo-modify-image master: Add .gitreview file https://review.openstack.org/570444 | 20:35 |
*** salmankhan has joined #tripleo | 20:38 | |
EmilienM | bandini: can you review https://review.openstack.org/#/c/570395/ please? | 20:39 |
stevebaker | EmilienM: hey, do you have the rights to add tripleo groups/users to this repo? https://review.openstack.org/#/admin/groups/1916,members | 20:43 |
*** radeks_ has quit IRC | 20:44 | |
EmilienM | stevebaker: no you need to ask infra to add a first contributor | 20:44 |
EmilienM | stevebaker: let me do it | 20:44 |
*** lifeless has joined #tripleo | 20:44 | |
*** bfournie has quit IRC | 20:45 | |
bandini | EmilienM: done | 20:46 |
stevebaker | EmilienM: thanks | 20:46 |
*** Guest3295 is now known as rbrady-afk | 20:46 | |
EmilienM | thx | 20:46 |
*** Guest64332 has quit IRC | 20:47 | |
*** raildo has quit IRC | 20:49 | |
*** slaweq_ has joined #tripleo | 20:51 | |
*** edmondsw has joined #tripleo | 20:52 | |
*** zaneb has joined #tripleo | 20:52 | |
*** zshi has joined #tripleo | 20:55 | |
*** slaweq has quit IRC | 20:55 | |
*** Guest87851 is now known as beagles | 20:57 | |
*** bcafarel has quit IRC | 20:58 | |
*** bcafarel has joined #tripleo | 20:59 | |
rlandy|rover | mwhahaha: hello - ever come across an error like this (ocata): https://ci.centos.org/artifacts/rdo/jenkins-tripleo-quickstart-promote-ocata-rdo_trunk-minimal-346/undercloud/home/stack/overcloud_deploy.log.gz | 21:02 |
EmilienM | bandini, stevebaker: can one of you +2 https://review.openstack.org/#/c/570406/ please | 21:02 |
EmilienM | to unblock gate on this repo | 21:02 |
EmilienM | thanks | 21:02 |
mwhahaha | rlandy|rover: nope but that looks like neutron errored | 21:02 |
stevebaker | EmilienM: yep | 21:03 |
rlandy|rover | hmmm ... happy times | 21:03 |
mwhahaha | rlandy|rover: is it consistent or is the first time we've seen that? | 21:05 |
rlandy|rover | mwhahaha: I've been debugging the ocata promotion all day - each run gives a different error. This is the first time I have seen that one though | 21:05 |
rlandy|rover | the last couple runs all died on the image md5 | 21:06 |
rlandy|rover | so we made some progress | 21:06 |
mwhahaha | rlandy|rover: ok seems odd, let me know if you need help | 21:06 |
rlandy|rover | the last time we actually got to a deployment we had a failure on 2018-05-24 08:55:03 | 2018-05-24 08:54:59Z [overcloud.Controller.0.SshHostPubKey]: CREATE_FAILED CREATE aborted | 21:07 |
rlandy|rover | so I guess I'll run again to see which error we can actually reproduce | 21:07 |
rlandy|rover | mwhahaha: thanks - will ping you if/when I have anything reproducible | 21:07 |
mwhahaha | rlandy|rover: create aborted is usually a timeout, wonder if it's a perf problem | 21:08 |
rlandy|rover | we won't know until we can get that far again | 21:08 |
mwhahaha | :D | 21:08 |
*** shardy has quit IRC | 21:08 | |
*** shardy has joined #tripleo | 21:09 | |
*** ooolpbot has joined #tripleo | 21:10 | |
ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 21:10 |
ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1773188 | 21:10 |
*** ooolpbot has quit IRC | 21:10 | |
openstack | Launchpad bug 1773188 in tripleo "puppet-tripleo unit tests broken due to recent upstream puppet module cleanup" [Critical,In progress] - Assigned to Emilien Macchi (emilienm) | 21:10 |
*** beagles is now known as beagles_biab | 21:14 | |
*** salmankhan has quit IRC | 21:15 | |
*** salmankhan has joined #tripleo | 21:20 | |
*** slaweq has joined #tripleo | 21:21 | |
*** edmondsw_ has joined #tripleo | 21:22 | |
*** matbu has quit IRC | 21:23 | |
*** slaweq_ has quit IRC | 21:24 | |
*** edmondsw has quit IRC | 21:26 | |
*** edmondsw_ is now known as edmondsw | 21:26 | |
*** leitan has quit IRC | 21:27 | |
openstackgerrit | Tim Rozet proposed openstack/tripleo-heat-templates master: Add OPNFV scenario environment https://review.openstack.org/486905 | 21:28 |
*** marrusl has quit IRC | 21:33 | |
*** lifeless_ has joined #tripleo | 21:34 | |
*** lifeless has quit IRC | 21:34 | |
mwhahaha | hmm seeing errors with fs001 on master, wonder if related to the rdo maintenance earlier ResourceInError: resources.Controller: Went to status ERROR due to "Message: No valid host was found. , Code: 500" | 21:37 |
mwhahaha | https://logs.rdoproject.org/19/566419/8/openstack-check/gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-master/Z3cca71b2091045b1b46f3d4f38a9e729/undercloud/home/jenkins/overcloud_deploy.log.txt.gz#_2018-05-24_21_11_32 | 21:37 |
*** akrivoka has quit IRC | 21:38 | |
*** alee has joined #tripleo | 21:38 | |
*** nyechiel_ has joined #tripleo | 21:39 | |
*** salmankhan has quit IRC | 21:40 | |
*** zaneb has quit IRC | 21:41 | |
*** zaneb has joined #tripleo | 21:41 | |
*** beagles_biab is now known as beagles | 21:45 | |
openstackgerrit | Alex Schultz proposed openstack/tripleo-puppet-elements master: Additional OpenStack cleanup https://review.openstack.org/567240 | 21:47 |
*** wolverineav has quit IRC | 21:48 | |
*** wolverineav has joined #tripleo | 21:49 | |
*** salmankhan has joined #tripleo | 21:50 | |
*** slaweq_ has joined #tripleo | 21:51 | |
*** edmondsw_ has joined #tripleo | 21:52 | |
*** wolverineav has quit IRC | 21:53 | |
*** slaweq has quit IRC | 21:54 | |
*** edmondsw has quit IRC | 21:56 | |
*** salmankhan has quit IRC | 21:56 | |
openstackgerrit | Alex Schultz proposed openstack/tripleo-puppet-elements master: Remove setup-puppet element https://review.openstack.org/570462 | 21:57 |
*** slaweq_ has quit IRC | 21:57 | |
openstackgerrit | Alex Schultz proposed openstack/tripleo-puppet-elements master: Remove sensu, fluentd, collectd from the images https://review.openstack.org/570464 | 22:01 |
*** bfournie has joined #tripleo | 22:01 | |
mwhahaha | pradk: please review -^ when you get a chance | 22:01 |
pradk | will do | 22:02 |
*** gvrangan has quit IRC | 22:07 | |
*** rajinir has quit IRC | 22:07 | |
*** edmondsw_ has quit IRC | 22:08 | |
*** edmondsw has joined #tripleo | 22:09 | |
openstackgerrit | Alex Schultz proposed openstack/tripleo-puppet-elements master: Remove opvercloud-cinder-volume element https://review.openstack.org/570465 | 22:10 |
*** ooolpbot has joined #tripleo | 22:10 | |
ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 22:10 |
ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1773188 | 22:10 |
*** ooolpbot has quit IRC | 22:10 | |
openstack | Launchpad bug 1773188 in tripleo "puppet-tripleo unit tests broken due to recent upstream puppet module cleanup" [Critical,In progress] - Assigned to Emilien Macchi (emilienm) | 22:10 |
*** edmondsw has quit IRC | 22:10 | |
*** edmondsw has joined #tripleo | 22:10 | |
*** slaweq has joined #tripleo | 22:10 | |
*** rcernin has joined #tripleo | 22:11 | |
*** artom has joined #tripleo | 22:13 | |
*** slaweq has quit IRC | 22:15 | |
*** salmankhan has joined #tripleo | 22:18 | |
*** paramite_ has quit IRC | 22:21 | |
*** edmondsw_ has joined #tripleo | 22:23 | |
*** edmondsw_ has quit IRC | 22:24 | |
*** salmankhan has quit IRC | 22:24 | |
*** edmondsw_ has joined #tripleo | 22:26 | |
*** edmondsw has quit IRC | 22:27 | |
*** zaneb has quit IRC | 22:27 | |
*** wolverineav has joined #tripleo | 22:28 | |
*** alee has quit IRC | 22:28 | |
*** edmondsw_ has quit IRC | 22:30 | |
*** loacker has joined #tripleo | 22:31 | |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Remove UpgradeRemoveUnusedPackages from undercloud-upgrade-prepare.yaml https://review.openstack.org/569323 | 22:31 |
*** zaneb has joined #tripleo | 22:31 | |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Remove UpgradeRemoveUnusedPackages from undercloud-upgrade-prepare.yaml https://review.openstack.org/569323 | 22:31 |
*** lifeless_ has quit IRC | 22:34 | |
openstackgerrit | John Fulton proposed openstack/tripleo-quickstart master: Update featureset024 to use ceph-ansible for Pike https://review.openstack.org/570469 | 22:42 |
*** matbu has joined #tripleo | 22:42 | |
openstackgerrit | John Fulton proposed openstack/tripleo-heat-templates stable/pike: WIP: Add CI/development block devices environment files https://review.openstack.org/570473 | 22:54 |
*** nyechiel_ has quit IRC | 22:54 | |
*** pchavva has quit IRC | 22:54 | |
*** edmondsw has joined #tripleo | 22:56 | |
openstackgerrit | Merged openstack/tripleo-common master: Fix Octavia amphora image RPM install https://review.openstack.org/570163 | 23:00 |
openstackgerrit | Merged openstack/tripleo-common stable/queens: Add a 'trash_output' flag to ansible playbook action https://review.openstack.org/566884 | 23:00 |
openstackgerrit | Merged openstack/python-tripleoclient master: Store UC hosts/portmaps heat params in output_dir https://review.openstack.org/564135 | 23:00 |
openstackgerrit | Merged openstack/tripleo-upgrade stable/pike: Separate scripts creation tasks for under/overcloud. https://review.openstack.org/570357 | 23:00 |
*** edmondsw has quit IRC | 23:00 | |
*** nyechiel_ has joined #tripleo | 23:09 | |
*** pmannidi has joined #tripleo | 23:09 | |
*** ooolpbot has joined #tripleo | 23:10 | |
ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 23:10 |
ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1773188 | 23:10 |
*** ooolpbot has quit IRC | 23:10 | |
openstack | Launchpad bug 1773188 in tripleo "puppet-tripleo unit tests broken due to recent upstream puppet module cleanup" [Critical,In progress] - Assigned to Emilien Macchi (emilienm) | 23:10 |
*** matbu has quit IRC | 23:11 | |
EmilienM | rlandy|rover: can we land https://review.openstack.org/#/c/568946/ ? | 23:13 |
*** nyechiel_ has quit IRC | 23:13 | |
*** dhill_ has joined #tripleo | 23:15 | |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-upgrade master: DNM, test https://review.openstack.org/568732 | 23:15 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-quickstart-extras master: execute the build / install of zuul changes in upgrade https://review.openstack.org/568946 | 23:16 |
openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci master: Add dry run option to toci_quickstart https://review.openstack.org/567060 | 23:16 |
openstackgerrit | Emilien Macchi proposed openstack/python-tripleoclient master: DNM Testing releases script upgrades with zuul change https://review.openstack.org/569229 | 23:16 |
EmilienM | trown|outtypewww, rlandy|rover: I rebased and retested ^ | 23:16 |
EmilienM | to see how it works, last test was May 18th! | 23:17 |
*** mcornea has quit IRC | 23:17 | |
*** gbarros has joined #tripleo | 23:18 | |
rlandy|rover | EmilienM: checking dependent patches | 23:19 |
rlandy|rover | oh ok - https://review.openstack.org/#/c/567060/ is the painful patch we can't get through | 23:20 |
rlandy|rover | I'm ok with landing https://review.openstack.org/#/c/568946/ | 23:20 |
rlandy|rover | the other CI people may complain - we'll see | 23:20 |
*** matbu has joined #tripleo | 23:21 | |
*** gbarros has quit IRC | 23:22 | |
rlandy|rover | EmilienM: got to run out for a bit - will check that testing when I get back | 23:24 |
*** rlandy|rover is now known as rlandy|rvr|bbl | 23:25 | |
*** zaneb has quit IRC | 23:27 | |
*** matbu has quit IRC | 23:30 | |
*** jtomasek has joined #tripleo | 23:32 | |
*** lifeless has joined #tripleo | 23:35 | |
*** matbu has joined #tripleo | 23:36 | |
*** alee has joined #tripleo | 23:40 | |
*** gbarros has joined #tripleo | 23:41 | |
EmilienM | rlandy|rvr|bbl: I don't want CI people to complain and hate me :D | 23:44 |
* EmilienM wants peace | 23:44 | |
*** gbarros has quit IRC | 23:44 | |
*** pradk has quit IRC | 23:50 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!