| openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart-extras master: GATE CHECK https://review.openstack.org/430277 | 00:00 |
|---|---|---|
| *** karimb has quit IRC | 00:08 | |
| *** tzumainn has quit IRC | 00:08 | |
| *** japestinho has quit IRC | 00:20 | |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart master: Revert "Generate/inject instack.json if overcloud nodes" https://review.openstack.org/452998 | 00:30 |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/ocata: Setting keystone region for tacker https://review.openstack.org/452731 | 00:40 |
| *** artom__ has joined #tripleo | 00:44 | |
| *** artom__ has quit IRC | 00:46 | |
| *** artom__ has joined #tripleo | 00:46 | |
| *** artom_ has quit IRC | 00:48 | |
| *** ansmith has quit IRC | 00:55 | |
| *** ramishra has joined #tripleo | 01:20 | |
| *** jpstnh has joined #tripleo | 01:28 | |
| *** limao has joined #tripleo | 01:36 | |
| *** fzdarsky_ has joined #tripleo | 01:39 | |
| *** limao_ has joined #tripleo | 01:39 | |
| *** limao has quit IRC | 01:41 | |
| *** fzdarsky has quit IRC | 01:42 | |
| *** thrash is now known as thrash|g0ne | 01:44 | |
| *** limao_ has quit IRC | 01:47 | |
| *** limao has joined #tripleo | 01:48 | |
| *** ramishra has quit IRC | 01:50 | |
| *** limao has quit IRC | 01:58 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Add ceilometer ipmi agent https://review.openstack.org/430447 | 01:59 |
| *** limao has joined #tripleo | 01:59 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci master: TEST: DONT RECHECK or REVIEW: periodic jobs https://review.openstack.org/359215 | 02:07 |
| *** bkopilov has quit IRC | 02:09 | |
| *** limao has quit IRC | 02:10 | |
| *** limao has joined #tripleo | 02:10 | |
| EmilienM | inc0: just in case you get no reply, please send email to ML | 02:16 |
| EmilienM | inc0: but yeah, puppet community could help if it helps infra | 02:16 |
| EmilienM | inc0: why don't use use https://github.com/cristifalcas/puppet-docker_distribution ? | 02:17 |
| openstackgerrit | James Slagle proposed openstack/tripleo-common master: Add skip_deploy_identifier https://review.openstack.org/452314 | 02:21 |
| *** limao has quit IRC | 02:21 | |
| *** limao has joined #tripleo | 02:22 | |
| *** trozet has joined #tripleo | 02:24 | |
| *** tvignaud has quit IRC | 02:26 | |
| *** jaganathan has joined #tripleo | 02:26 | |
| *** maeca1 has joined #tripleo | 02:32 | |
| *** limao has quit IRC | 02:32 | |
| *** limao has joined #tripleo | 02:33 | |
| *** tvignaud has joined #tripleo | 02:39 | |
| *** limao_ has joined #tripleo | 02:47 | |
| *** limao__ has joined #tripleo | 02:49 | |
| *** limao__ has quit IRC | 02:50 | |
| *** limao__ has joined #tripleo | 02:50 | |
| *** limao has quit IRC | 02:50 | |
| *** limao_ has quit IRC | 02:52 | |
| *** trozet has quit IRC | 02:53 | |
| *** maeca1 has quit IRC | 02:57 | |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Allow installing proliant tools from an OS package https://review.openstack.org/344214 | 02:59 |
| openstackgerrit | Pradeep Kilambi proposed openstack/tripleo-heat-templates stable/ocata: Set auth flag so ceilometer auth is enabled https://review.openstack.org/452540 | 03:02 |
| *** links has joined #tripleo | 03:08 | |
| *** noslzzp has quit IRC | 03:13 | |
| *** ramishra has joined #tripleo | 03:14 | |
| *** atheurer has quit IRC | 03:25 | |
| *** rlandy has quit IRC | 03:28 | |
| *** bkopilov has joined #tripleo | 03:39 | |
| *** gkadam has joined #tripleo | 03:55 | |
| *** limao__ has quit IRC | 04:00 | |
| *** radeks has quit IRC | 04:04 | |
| *** limao has joined #tripleo | 04:07 | |
| *** limao_ has joined #tripleo | 04:09 | |
| *** limao has quit IRC | 04:11 | |
| *** nmathew has joined #tripleo | 04:18 | |
| *** limao_ has quit IRC | 04:19 | |
| *** limao has joined #tripleo | 04:20 | |
| *** limao has quit IRC | 04:31 | |
| *** limao has joined #tripleo | 04:31 | |
| *** yamahata has joined #tripleo | 04:34 | |
| *** janki has joined #tripleo | 04:37 | |
| *** limao has quit IRC | 04:42 | |
| *** limao has joined #tripleo | 04:43 | |
| *** rcernin has joined #tripleo | 04:46 | |
| *** itzikb has joined #tripleo | 04:51 | |
| *** gkadam has quit IRC | 04:51 | |
| *** skramaja has joined #tripleo | 04:53 | |
| *** limao has quit IRC | 04:54 | |
| *** limao has joined #tripleo | 04:54 | |
| *** limao has quit IRC | 04:55 | |
| *** limao has joined #tripleo | 04:56 | |
| itzikb | Hi, How do I add release notes to a patch? | 04:56 |
| *** dparkes has quit IRC | 04:58 | |
| openstackgerrit | Itzik Brown proposed openstack/tripleo-heat-templates master: Exposing the ability to enable/disable the repository https://review.openstack.org/452521 | 05:00 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Introduce manual setting of DIB_INIT_SYSTEM https://review.openstack.org/414179 | 05:02 |
| *** anshul has joined #tripleo | 05:03 | |
| *** nmathew- has joined #tripleo | 05:04 | |
| *** jchhatbar has joined #tripleo | 05:05 | |
| *** limao has quit IRC | 05:06 | |
| *** gkadam has joined #tripleo | 05:07 | |
| *** CaptTofu__ has joined #tripleo | 05:08 | |
| *** limao has joined #tripleo | 05:08 | |
| *** cmyster_ has joined #tripleo | 05:09 | |
| *** ratailor has joined #tripleo | 05:09 | |
| *** rcernin has quit IRC | 05:09 | |
| *** skramaja_ has joined #tripleo | 05:10 | |
| *** rcernin has joined #tripleo | 05:11 | |
| *** rodrigod` has joined #tripleo | 05:12 | |
| *** hewbrocc` has joined #tripleo | 05:12 | |
| *** owalsh_ has joined #tripleo | 05:12 | |
| *** numans_ has joined #tripleo | 05:13 | |
| *** skramaja has quit IRC | 05:13 | |
| *** skramaja_ is now known as skramaja | 05:14 | |
| *** janki has quit IRC | 05:14 | |
| *** nmathew has quit IRC | 05:14 | |
| *** ccamacho has quit IRC | 05:14 | |
| *** cmyster has quit IRC | 05:14 | |
| *** owalsh has quit IRC | 05:14 | |
| *** dtantsur|afk has quit IRC | 05:14 | |
| *** hewbrocca_afk has quit IRC | 05:14 | |
| *** akrzos has quit IRC | 05:14 | |
| *** jbadiapa has quit IRC | 05:14 | |
| *** rodrigods has quit IRC | 05:14 | |
| *** numans has quit IRC | 05:14 | |
| *** panda|bbl has quit IRC | 05:14 | |
| *** chandankumar has quit IRC | 05:14 | |
| *** CaptTofu_ has quit IRC | 05:14 | |
| *** bkero has quit IRC | 05:14 | |
| *** dtantsur has joined #tripleo | 05:15 | |
| *** akrzos has joined #tripleo | 05:15 | |
| *** udesale has joined #tripleo | 05:17 | |
| *** cmyster_ is now known as cmyster | 05:19 | |
| *** cmyster has joined #tripleo | 05:19 | |
| *** limao has quit IRC | 05:19 | |
| *** limao has joined #tripleo | 05:20 | |
| *** bkero has joined #tripleo | 05:20 | |
| *** jaganathan has quit IRC | 05:20 | |
| *** jbadiapa has joined #tripleo | 05:21 | |
| *** panda|bbl has joined #tripleo | 05:21 | |
| *** chandankumar has joined #tripleo | 05:22 | |
| *** ccamacho has joined #tripleo | 05:22 | |
| *** ramishra has quit IRC | 05:24 | |
| *** prateek has joined #tripleo | 05:26 | |
| *** limao has quit IRC | 05:30 | |
| *** limao has joined #tripleo | 05:31 | |
| *** pgadiya has joined #tripleo | 05:31 | |
| *** mdnadeem has joined #tripleo | 05:31 | |
| *** michapma_dsk has joined #tripleo | 05:32 | |
| *** limao has quit IRC | 05:41 | |
| *** limao has joined #tripleo | 05:42 | |
| *** udesale__ has joined #tripleo | 05:42 | |
| *** madgoat has joined #tripleo | 05:42 | |
| *** madgoat has left #tripleo | 05:43 | |
| *** udesale has quit IRC | 05:43 | |
| *** iranzo has joined #tripleo | 05:44 | |
| jaosorior_away | dtrainor: no worries, asking directly is the way to go sometimes | 05:46 |
| dtrainor | sometimes it is, i still don't like it much :) | 05:46 |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-docs master: WIP: Post deployment installation instructions for Octavia https://review.openstack.org/447496 | 05:47 |
| *** yprokule has joined #tripleo | 05:52 | |
| *** limao has quit IRC | 05:53 | |
| *** limao has joined #tripleo | 05:54 | |
| *** dparkes has joined #tripleo | 05:55 | |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-docs master: WIP: Post deployment installation instructions for Octavia https://review.openstack.org/447496 | 05:57 |
| *** jaosorior_away is now known as jaosorior | 06:01 | |
| *** limao has quit IRC | 06:04 | |
| *** leanderthal|afk is now known as leanderthal | 06:04 | |
| *** limao has joined #tripleo | 06:05 | |
| *** chem_gone is now known as chem | 06:06 | |
| *** udesale__ has quit IRC | 06:08 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/instack-undercloud master: Set Zaqar roles https://review.openstack.org/452098 | 06:09 |
| *** Vijayendra has joined #tripleo | 06:12 | |
| *** agurenko has joined #tripleo | 06:13 | |
| *** ealcaniz has joined #tripleo | 06:14 | |
| *** limao has quit IRC | 06:15 | |
| *** limao has joined #tripleo | 06:16 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates master: TLS-everywhere: Enable for TLS libvirt live migration https://review.openstack.org/450726 | 06:19 |
| *** jaganathan has joined #tripleo | 06:19 | |
| *** tesseract has joined #tripleo | 06:26 | |
| *** limao has quit IRC | 06:27 | |
| *** limao has joined #tripleo | 06:27 | |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/tripleo-heat-templates master: Ensure upgrade step orchestration accross roles. https://review.openstack.org/452828 | 06:27 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates master: TLS-everywhere: Enable for TLS libvirt live migration https://review.openstack.org/450726 | 06:27 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: DO NOT MERGE: Testing TLS everywhere for libvirt https://review.openstack.org/450729 | 06:28 |
| *** lmiccini has joined #tripleo | 06:30 | |
| openstackgerrit | Merged openstack/python-tripleoclient master: Support multiple messages from package_update https://review.openstack.org/446435 | 06:31 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates master: glance: deploy services with Keystone v3 endpoints https://review.openstack.org/442798 | 06:35 |
| *** cylopez has joined #tripleo | 06:35 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates master: glance: deploy services with Keystone v3 endpoints https://review.openstack.org/442798 | 06:37 |
| chem | marios: hi, I think I've got a winner here https://review.openstack.org/#/c/452828/, I know it's kinda unrelated to batch upgrade, but looks like a major orchestration issue | 06:38 |
| *** limao has quit IRC | 06:38 | |
| *** limao has joined #tripleo | 06:39 | |
| *** jpena|away is now known as jpena | 06:39 | |
| chem | marios: the CI is all green the backport https://review.openstack.org/#/c/452830/ | 06:39 |
| *** pgadiya has quit IRC | 06:44 | |
| *** limao has quit IRC | 06:50 | |
| *** limao has joined #tripleo | 06:50 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo master: Add TLS in the internal network for Swift Proxy https://review.openstack.org/453028 | 06:54 |
| *** jlinkes has joined #tripleo | 06:55 | |
| *** jcoufal has joined #tripleo | 07:00 | |
| *** limao has quit IRC | 07:01 | |
| marios | chem: hey man, going through the reviews now (saw your comments on the batch upgrade order one too) | 07:01 |
| *** limao has joined #tripleo | 07:01 | |
| chem | marios: I have this feeling that this could explain it all | 07:02 |
| *** athomas has joined #tripleo | 07:05 | |
| *** ukalifon has joined #tripleo | 07:05 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo master: Add TLS in the internal network for Swift Proxy https://review.openstack.org/453028 | 07:05 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates master: TLS-everywhere: Enable for TLS libvirt live migration https://review.openstack.org/450726 | 07:09 |
| *** shardy has joined #tripleo | 07:09 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates master: Add parameters for internal TLS for swift proxy https://review.openstack.org/453034 | 07:09 |
| *** limao has quit IRC | 07:11 | |
| *** ffiore has joined #tripleo | 07:11 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates master: Add parameters for internal TLS for swift proxy https://review.openstack.org/453034 | 07:12 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo master: Clean up TLS-related bits from swift-proxy https://review.openstack.org/453038 | 07:12 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo master: Clean up TLS-related bits from swift-proxy https://review.openstack.org/453038 | 07:12 |
| *** aufi has joined #tripleo | 07:13 | |
| openstackgerrit | Andreas Florath proposed openstack/diskimage-builder master: Introduce manual setting of DIB_INIT_SYSTEM https://review.openstack.org/414179 | 07:13 |
| *** florianf has joined #tripleo | 07:13 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: DO NOT MERGE: Testing TLS everywhere for swift proxy https://review.openstack.org/453039 | 07:16 |
| jaosorior | mandre: was there already an internal mirror for the docker images set up in CI? Would like to use it. | 07:17 |
| *** limao has joined #tripleo | 07:19 | |
| *** pcaruana has joined #tripleo | 07:20 | |
| mandre | jaosorior: give it a try at 192.168.101.150:5000 | 07:20 |
| mandre | jaosorior: I'm not sure if it's already functional, though... you may want to check with dprince when he's up | 07:21 |
| mandre | jaosorior: related patch https://review.openstack.org/#/c/452276/ | 07:22 |
| jaosorior | oh | 07:23 |
| *** ealcaniz has quit IRC | 07:23 | |
| jaosorior | maybe I should wait then | 07:23 |
| jaosorior | mandre: thanks dude | 07:23 |
| *** limao_ has joined #tripleo | 07:23 | |
| jaosorior | mandre: so meanwhile. What's the containers job doing? is it pulling all the images? | 07:23 |
| *** limao has quit IRC | 07:24 | |
| mandre | jaosorior: right now it's pulling from docker.io | 07:24 |
| jaosorior | mandre: ah right, so it works cause it's not using network isolation | 07:24 |
| *** ealcaniz has joined #tripleo | 07:25 | |
| *** shardy has quit IRC | 07:25 | |
| mandre | jaosorior: nah it also work for net-iso, the images are pulled on the undercloud and served to the overcloud through a local registry | 07:26 |
| jaosorior | right | 07:26 |
| jaosorior | ah | 07:26 |
| jaosorior | right | 07:26 |
| jaosorior | yeah, I had a failure cause I tried doing net-iso without pulling the images to the undercloud | 07:26 |
| *** bogdando has joined #tripleo | 07:26 | |
| openstackgerrit | Brent Eagles proposed openstack/puppet-tripleo master: Add missing octavia auth include to keystone manifest https://review.openstack.org/453044 | 07:27 |
| *** udesale has joined #tripleo | 07:27 | |
| *** jcoufal has quit IRC | 07:29 | |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Introduce manual setting of DIB_INIT_SYSTEM https://review.openstack.org/414179 | 07:29 |
| *** prateek_ has joined #tripleo | 07:31 | |
| *** limao has joined #tripleo | 07:31 | |
| *** prateek_ has quit IRC | 07:32 | |
| *** prateek_ has joined #tripleo | 07:32 | |
| *** limao__ has joined #tripleo | 07:33 | |
| *** prateek has quit IRC | 07:33 | |
| *** limao_ has quit IRC | 07:34 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: DO NOT MERGE: Testing TLS-everywhere with keystone container https://review.openstack.org/449570 | 07:35 |
| *** limao has quit IRC | 07:35 | |
| *** karimb has joined #tripleo | 07:36 | |
| *** limao has joined #tripleo | 07:42 | |
| *** limao__ has quit IRC | 07:45 | |
| *** limao_ has joined #tripleo | 07:45 | |
| *** limao has quit IRC | 07:46 | |
| *** skramaja is now known as skramaja_afk | 07:46 | |
| *** udesale__ has joined #tripleo | 07:48 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates master: TLS-everywhere: Enable for TLS libvirt live migration https://review.openstack.org/450726 | 07:48 |
| *** prateek has joined #tripleo | 07:49 | |
| *** prateek_ has quit IRC | 07:49 | |
| *** udesale has quit IRC | 07:50 | |
| *** limao_ has quit IRC | 07:51 | |
| chem | marios: matbu I think this heat conditional cannot exist | 07:51 |
| chem | marios: matbu like, at all. See my comment https://review.openstack.org/#/c/452828/5/puppet/major_upgrade_steps.j2.yaml@207 | 07:51 |
| chem | marios: I think both of review are linked. I would remove the heat conditional related to upgrade from mine and you would handle the batch_upgrade conditional removal | 07:52 |
| *** amoralej|off is now known as amoralej | 07:53 | |
| *** jpich has joined #tripleo | 07:54 | |
| chem | marios: with that you could rebase on top without problem. | 07:54 |
| marios | chem: well, you don't have to convince me :) | 07:55 |
| chem | marios: this mix of source of truth (jinja/head db) is just preventing us for doing to clever stuff | 07:55 |
| marios | chem: yes my v13 at https://review.openstack.org/#/c/448602/ still removes the conditional until someone tells me there is a better way (matbu exploring something in roles_data and gfidente posted https://review.openstack.org/#/c/452789/ yesterday) | 07:56 |
| chem | marios: oki, I'm removing my part in my review | 07:56 |
| *** yamahata has quit IRC | 07:58 | |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-docs master: WIP: Post deployment installation instructions for Octavia https://review.openstack.org/447496 | 07:58 |
| *** lucas-afk is now known as lucasagomes | 07:59 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #tripleo | 08:00 | |
| marios | beagles: y u no sleep | 08:01 |
| marios | beagles: i'll have your sleep if you're not gonna use it, just sayin | 08:02 |
| *** zoli|gone is now known as zoli | 08:02 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/tripleo-heat-templates master: DO NOT MERGE: testing ironic https://review.openstack.org/428053 | 08:02 |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/tripleo-heat-templates master: Ensure upgrade step orchestration accross roles. https://review.openstack.org/452828 | 08:05 |
| openstackgerrit | Juan Badia Payno proposed openstack/tripleo-heat-templates master: Increase documentation about parameters https://review.openstack.org/440759 | 08:06 |
| matbu | marios: hey man, so finally +1 your review (batch_upgrade) | 08:06 |
| chem | marios: matbu so I've adjusted mine to remove conditional on Upgrade tasks | 08:08 |
| chem | marios: I don't think there is any other way :) | 08:09 |
| marios | ack thanks matbu - i think unless we can test a better way before thursday then we should land this and iterate when we work it out later (the conditional) - it _is_ a shame about the overhead though I mean with keeping all the batch upgrade tasks even when they are []. We *could* have a 'batch_upgrade_tasks_enabled' in roles_data but that would get messy, quickly. the operator would have to update that | 08:09 |
| marios | according to any of their services using upgrade batch tasks | 08:09 |
| openstackgerrit | Bogdan Dobrelya proposed openstack/tripleo-quickstart master: Instack.json for overcloud or BM udercloud https://review.openstack.org/453056 | 08:10 |
| matbu | marios: yep , but im not sure its the good way (adding it in role_data) | 08:10 |
| matbu | marios: it sounds confusing for operator | 08:10 |
| chem | marios: matbu but I think https://review.openstack.org/#/c/448602/13/puppet/major_upgrade_steps.j2.yaml@203 still hold. You have to make sure you're depending on the previous for all roles | 08:11 |
| *** nmathew has joined #tripleo | 08:11 | |
| matbu | marios: another way would be in tripleoclient i think, but it would be bad, because tht will be definitely unusable outside of the tripleoclient | 08:12 |
| chem | marios: speaking of overhead, do we know what we're talking about ? | 08:12 |
| *** nmathew- has quit IRC | 08:12 | |
| *** jprovazn has joined #tripleo | 08:12 | |
| matbu | chem: no :D | 08:12 |
| *** snecklifter has quit IRC | 08:13 | |
| openstackgerrit | Merged openstack/python-tripleoclient master: Support multiple messages from create/update_deployment_plan https://review.openstack.org/446434 | 08:13 |
| *** gfidente has joined #tripleo | 08:13 | |
| chem | marios: matbu in the timeline I've seen small upgrade task take less than one minute, so when there is no upgrade task at all we're talking what, ... seconds and some Kb of memory ? | 08:13 |
| openstackgerrit | Merged openstack/python-tripleoclient master: Support multiple messages from deploy_plan https://review.openstack.org/446437 | 08:13 |
| openstackgerrit | Merged openstack/diskimage-builder master: Use stevedore for plugin config of block device https://review.openstack.org/447090 | 08:13 |
| matbu | chem: hard to say, i guess it depends on the number of nodes | 08:14 |
| marios | chem: ack i can update my review to also add that loop in the batch tasks | 08:14 |
| chem | marios: ack, cool :) | 08:14 |
| matbu | chem: also on the undercloud performance, but i don't think it would save so much time, thats true | 08:15 |
| chem | matbu: marios "premature optimization is the root of all evil (or at least most of it) in programming." | 08:15 |
| chem | matbu: marios https://en.wikiquote.org/wiki/Donald_Knuth ... | 08:16 |
| * marios always thought gfidente was the root of all evil | 08:16 | |
| * marios shrugs | 08:16 | |
| chem | haha | 08:16 |
| matbu | :) | 08:16 |
| gfidente | you guys | 08:17 |
| chem | marios: will you rebase your patch on top of mine, as we're touching the same file it would kinda make sense (or the other way round, I don't care) ? | 08:18 |
| gfidente | have good arguments for that? | 08:18 |
| gfidente | because I spent yesterday night trying to get the shit working | 08:18 |
| gfidente | and it didn't | 08:18 |
| gfidente | so I can offer you an argument! | 08:18 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient master: Remove the single-message interface to websockets https://review.openstack.org/446439 | 08:20 |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/tripleo-heat-templates stable/ocata: Ensure upgrade step orchestration accross roles. https://review.openstack.org/452830 | 08:22 |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/tripleo-heat-templates stable/ocata: Add manual ovs upgrade script for workaround ovs upgrade issue https://review.openstack.org/451231 | 08:23 |
| *** anshul has quit IRC | 08:23 | |
| *** dsariel has joined #tripleo | 08:23 | |
| gfidente | matbu I thought I was on a good bath | 08:24 |
| gfidente | by removing the jinja conditionals which check for enabled_roles | 08:24 |
| gfidente | and let the condition: decide if the resource was to be created or not | 08:24 |
| gfidente | as I thought jinja was effectively removing from the templated the resources related to non-enabled roles | 08:25 |
| openstackgerrit | Arx Cruz proposed openstack-infra/tripleo-ci master: DO NOT REVIEW: testing tempest for oooq https://review.openstack.org/453059 | 08:25 |
| gfidente | so in fact now I *do* see in the rendered templates all the resources | 08:25 |
| gfidente | I am seeing if that fixes it | 08:25 |
| matbu | gfidente: the problem here, is that we mixed heat condition and jinja code | 08:26 |
| matbu | gfidente: and since jinja don't knowanything about heat db | 08:26 |
| d0ugal | jpich: last one! https://review.openstack.org/#/c/446439/ | 08:26 |
| matbu | gfidente: and the templating comes first, then it's hard to solve that, without adding a variable in role_data that indicated if the role has some upgrade_batch_tasks | 08:27 |
| jpich | d0ugal: Woohoo!! Maybe we should keep it there for posterity ;) | 08:27 |
| *** prateek_ has joined #tripleo | 08:27 | |
| d0ugal | :(( | 08:28 |
| gfidente | matbu so indeed that is what I explored yesterday | 08:28 |
| gfidente | check https://review.openstack.org/#/c/452789/ | 08:28 |
| chem | gfidente: all in all we had a major orchestration issue all across the board because https://bugs.launchpad.net/tripleo/+bug/1679486 | 08:28 |
| openstack | Launchpad bug 1679486 in tripleo "N->O Upgrade, ochestration is broken." [Critical,In progress] - Assigned to Sofer Athlan-Guyot (sofer-athlan-guyot) | 08:28 |
| matbu | gfidente: maybe we can make a pluging somewhere, for example, i don't know where this is used for: https://github.com/openstack/tripleo-heat-templates/blob/master/tools/process-templates.py | 08:28 |
| jpich | d0ugal: I'm rebuilding my environments atm, will poke after if it is still around | 08:28 |
| chem | gfidente: see https://review.openstack.org/#/c/452828/ when you have time | 08:28 |
| d0ugal | jpich: okay, thanks. it only deletes code :) | 08:29 |
| jpich | d0ugal: Oooh it's that one is it, the one that depended on everything else | 08:29 |
| d0ugal | jpich: yup | 08:30 |
| gfidente | chem but I don't want to remove the conditions in the templates | 08:30 |
| gfidente | I want to remove the use of enabled_roles in jinja and let the conditions do the work | 08:30 |
| *** prateek has quit IRC | 08:30 | |
| jpich | d0ugal: Yay delete all the code. If CI's good I'm good too | 08:30 |
| d0ugal | jpich: thanks! | 08:30 |
| chem | gfidente: well, that will be for upgrade v100.1 | 08:31 |
| gfidente | chem no it's easier than it seems, check https://review.openstack.org/#/c/452789/ | 08:31 |
| *** prateek_ has quit IRC | 08:31 | |
| matbu | chem: on which upgrade version we are now ? :) | 08:32 |
| jaosorior | any reviews for this https://review.openstack.org/#/c/452098/3 ? | 08:32 |
| chem | matbu: ... | 08:32 |
| chem | gfidente: in your review the depend on only depends on the role's previous step. So if you have controller role and novacontrol role for instance, then novacontrol will only wait for novacontrol step-1 to be finish, without giving a damn about at which step the controller is. This completly break orchestration as novacontrol will reach step5, do its upgrade db stuff while controller will still be at step 3 for instance. | 08:35 |
| matbu | gfidente: in your review line 197, for the controller role for example, if the upgrade_batch condition is false, so the ControllerUpgradeBatch is not created, the UpgradeStep will depend on non existing resource | 08:35 |
| chem | gfidente: you have to depend on all other roles | 08:35 |
| chem | gfidente: just for correctness | 08:36 |
| *** dsariel has quit IRC | 08:36 | |
| *** anshul has joined #tripleo | 08:36 | |
| matbu | i put the comment in the review, easiest to interpret and read | 08:37 |
| *** limao has joined #tripleo | 08:37 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates master: Enforce upgrade_batch_tasks before upgrade_tasks order https://review.openstack.org/448602 | 08:40 |
| *** ckyriakidou has joined #tripleo | 08:40 | |
| *** derekh has joined #tripleo | 08:40 | |
| marios | chem: i didn't rebase yet, i can do that now or are you going to update your review? | 08:40 |
| marios | chem: i mean for another revision ? /me checks | 08:40 |
| sshnaidm|off | apetrich, ping | 08:41 |
| *** sshnaidm|off is now known as sshnaidm | 08:41 | |
| apetrich | sshnaidm, heya | 08:41 |
| sshnaidm | apetrich, hi, I have a question about tls, I see you created tls library in oooq, right? | 08:42 |
| apetrich | sshnaidm, aye for khaleesi I think but it was generic enough to go to oooq | 08:43 |
| sshnaidm | apetrich, I'd like to know which IP range is in /home/jenkins/enable-tls.yaml generated | 08:43 |
| chem | marios: I think I'm done | 08:43 |
| *** dsariel has joined #tripleo | 08:43 | |
| sshnaidm | apetrich, yeah, I think it's just copied | 08:43 |
| chem | gfidente: I detailed my point https://review.openstack.org/#/c/452828/ and used my super -1 power :) | 08:46 |
| *** prateek has joined #tripleo | 08:47 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates master: Add parameters for internal TLS for swift proxy https://review.openstack.org/453034 | 08:48 |
| apetrich | sshnaidm, the cert is supposed to use the same ip as the PublicVirtualFixedIPs from network-environment.yaml | 08:48 |
| apetrich | sshnaidm, is that what you are asking? | 08:50 |
| sshnaidm | apetrich, yeah, thanks | 08:51 |
| marios | chem: ack rebasing | 08:52 |
| sshnaidm | apetrich, and if there is no network-environment.yaml ? | 08:53 |
| dtantsur | d0ugal, hi! mind re-reviewing https://review.openstack.org/#/c/408581/ please? got 1x +2 and CI passing | 08:53 |
| d0ugal | dtantsur: sure | 08:53 |
| sshnaidm | apetrich, my problem is that certificate is generated for 10.0.0.5, but I have 192.168.24.0 range | 08:54 |
| sshnaidm | apetrich, trying to understand where is the mistake | 08:54 |
| *** panda|bbl is now known as panda | 08:54 | |
| apetrich | sshnaidm, I think it comes from roles/common/defaults/main.yml L148 | 08:55 |
| apetrich | sshnaidm, it is a default so you can override if needed | 08:55 |
| *** panda is now known as Guest45726 | 08:55 | |
| apetrich | sshnaidm, -e overcloud_public_vip=192.168.24.FOO | 08:55 |
| apetrich | sshnaidm, but you have to know the endpoint vip beforehand | 08:56 |
| *** salmankhan has joined #tripleo | 08:56 | |
| apetrich | sshnaidm, the cert doesn't work with range | 08:56 |
| *** limao has quit IRC | 08:57 | |
| sshnaidm | apetrich, yeah, right | 08:58 |
| *** limao has joined #tripleo | 08:58 | |
| *** Guest45726 is now known as panda | 09:01 | |
| jaosorior | sshnaidm: I guess it asumes network isolation | 09:03 |
| jaosorior | where 10.0.0.0/24 is the default range | 09:03 |
| *** tosky has joined #tripleo | 09:03 | |
| jaosorior | if you're deploying with 192.168.24.0 range being the default ctlplane range | 09:03 |
| *** michapma_dsk has quit IRC | 09:03 | |
| sshnaidm | jaosorior, yeah, something broke in last days, non iso jobs got net-iso ranges | 09:04 |
| *** akrivoka has joined #tripleo | 09:04 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo master: Add TLS in the internal network for Swift Proxy https://review.openstack.org/453028 | 09:05 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo master: Clean up TLS-related bits from swift-proxy https://review.openstack.org/453038 | 09:05 |
| *** stendulker has joined #tripleo | 09:05 | |
| sshnaidm | and I don't know where is comes from.. | 09:06 |
| *** akrivoka has quit IRC | 09:07 | |
| *** nmathew has quit IRC | 09:08 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/ocata: Set auth flag so ceilometer auth is enabled https://review.openstack.org/452540 | 09:08 |
| *** akrivoka has joined #tripleo | 09:08 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates master: Enforce upgrade_batch_tasks before upgrade_tasks order https://review.openstack.org/448602 | 09:09 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates master: Ensure upgrade step orchestration accross roles. https://review.openstack.org/452828 | 09:09 |
| marios | chem: ^^ | 09:09 |
| chem | marios: was about to let some comment ... | 09:10 |
| * chem looking at the last version now | 09:10 | |
| gfidente | chem agreed and I can add the dependencies for the other roles | 09:12 |
| gfidente | chem for this one https://bugs.launchpad.net/tripleo/+bug/1678101 | 09:14 |
| openstack | Launchpad bug 1678101 in tripleo "batch_upgrade_tasks not executed before upgrade_tasks" [High,In progress] - Assigned to Marios Andreou (marios-b) | 09:14 |
| chem | gfidente: don't see how you can achieve that given the current state of affair, but I can't say I know everything about THT :) | 09:14 |
| gfidente | chem I think a for loop would do? | 09:14 |
| gfidente | matbu yes indeed but the purpose of the heat fix https://review.openstack.org/#/c/419588/ is precisely to allow that | 09:15 |
| gfidente | to depend on resources which a condition might not create | 09:15 |
| *** limao has quit IRC | 09:15 | |
| *** limao has joined #tripleo | 09:16 | |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-quickstart-extras master: Upgrade to containerized overcloud https://review.openstack.org/448576 | 09:16 |
| openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart master: Revert "Generate/inject instack.json if overcloud nodes" https://review.openstack.org/452998 | 09:19 |
| matbu | gfidente: yep, which is weird | 09:19 |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-quickstart-extras master: Upgrade to containerized overcloud https://review.openstack.org/448576 | 09:20 |
| chem | gfidente: can well if that heat patch is working then yes I think we can include back the conditional. | 09:21 |
| chem | matbu: marios ^ | 09:21 |
| chem | gfidente: but it seems it's not | 09:21 |
| gfidente | I think we want to merge marios' patch | 09:22 |
| adarazs | jistr: ping me when that change is ready for a review. looks like you're actively working on it, so I didn't do a proper review yet. | 09:22 |
| chem | gfidente: and mine :) | 09:22 |
| gfidente | I am just making experiments to see if I get anything working | 09:22 |
| adarazs | jistr: meaning the upgrade to containerized overcloud. | 09:22 |
| chem | gfidente: sounds good to me :) | 09:22 |
| gfidente | and your yes | 09:22 |
| openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart-extras master: DONT REVIEW: test non-iso settings https://review.openstack.org/453075 | 09:22 |
| *** athomas has quit IRC | 09:23 | |
| jistr | adarazs: will do, thanks! | 09:23 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: DO NOT MERGE: Testing TLS-everywhere with keystone container https://review.openstack.org/449570 | 09:23 |
| marios | chem: i don't think it is a nit https://review.openstack.org/#/c/448602/15/puppet/major_upgrade_steps.j2.yaml | 09:25 |
| *** ratailor_ has joined #tripleo | 09:25 | |
| matbu | chem: the heat patch is working, since it allow dependcies on resource which is conditioanlly defined, but the patch doesn't care about the fact when the condition is false and it doesnt create it | 09:25 |
| gfidente | problem is that I think we're mixing up stuff for different bugs | 09:25 |
| *** ratailor is now known as ratailor_ | 09:26 | |
| gfidente | I think marios is meant to ensure Upgrade happens after UpgradeBatch | 09:26 |
| chem | matbu: ack, understood. | 09:26 |
| gfidente | and chem is meant to add the for loop to depend on all roles | 09:26 |
| chem | matbu: make sense | 09:26 |
| gfidente | can we isolate the code changes needed by the specific bugs? | 09:26 |
| gfidente | I think it'd be easier to merge them | 09:26 |
| marios | gfidente: no | 09:26 |
| marios | gfidente: not yet | 09:26 |
| matbu | chem: this patch needs something like if res_name in stack.t.get(stack.t.RESOURCES) and " resource condition" is True: | 09:27 |
| gfidente | not isolate the code changes or not merge them? | 09:27 |
| marios | gfidente: too much rebase/discussion/hands. lets all settle agree and test before landing | 09:27 |
| *** zoli is now known as zoli|lunch | 09:27 | |
| *** ratailor_ is now known as Guest6565 | 09:27 | |
| matbu | i can try to add a fix into heat | 09:27 |
| marios | chem: so will you revisit your -1? https://review.openstack.org/#/c/448602/15/puppet/major_upgrade_steps.j2.yaml | 09:27 |
| marios | chem: or will we update | 09:27 |
| matbu | but it sounds late... | 09:27 |
| gfidente | matbu is you add resource condition true | 09:27 |
| gfidente | you're back at the initial problem | 09:27 |
| openstackgerrit | Dmitry Tantsur proposed openstack/puppet-tripleo master: Stop including ironic::drivers::ssh in the ironic-conductor profile https://review.openstack.org/453078 | 09:27 |
| gfidente | where you can't set dependencies on resources which aren't created because of a condition | 09:28 |
| gfidente | *if | 09:28 |
| chem | marios: I'm not sure, let me re-reread the stuff. I kinda doesn't make sense to me | 09:28 |
| Guest6565 | I'm new to tripleo, could anybody please let me know, why we have different project like puppet-nova when we have openstack/nova ? | 09:28 |
| chem | marios: how come the config is depending on the task ? | 09:28 |
| matbu | gfidente: hmm not sure, i'll try something | 09:29 |
| Guest6565 | I tried to search, but didn't got any useful info. | 09:29 |
| chem | marios: from the shardy's comment above it looks like we did that only for nice display of config resource | 09:29 |
| *** pgadiya has joined #tripleo | 09:29 | |
| matbu | i just need a working env now :) | 09:29 |
| *** pgadiya has quit IRC | 09:29 | |
| karimb | hello folks, how am i supposed to access the undercloud UI ? | 09:29 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates master: [WIP] Ensure non-batched tasks start after batched tasks https://review.openstack.org/452789 | 09:29 |
| karimb | it seems port 3000 should be used, but it seems to use private provisioning ip specifically .... | 09:30 |
| chem | marios: I cannot understand how it's not a nit | 09:30 |
| *** athomas has joined #tripleo | 09:31 | |
| *** pmannidi has quit IRC | 09:31 | |
| *** skramaja_afk is now known as skramaja | 09:33 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/instack-undercloud master: Remove support for the deprecated pxe_ssh driver https://review.openstack.org/453081 | 09:33 |
| chem | marios: I want to -1 myself https://review.openstack.org/#/c/452828/7/puppet/major_upgrade_steps.j2.yaml@188 | 09:34 |
| marios | chem: shardy comment i think is about having the config resources serialized at all... i mean, even before it was depending on UpgradeBatch_Step i..e the deploy resource not the config resource | 09:35 |
| *** dtantsur is now known as dtantsur|relocat | 09:35 | |
| *** dtantsur|relocat is now known as dtantsur|bbl | 09:35 | |
| *** karimb_ has joined #tripleo | 09:35 | |
| marios | chem: we do the same at https://review.openstack.org/#/c/448602/15/puppet/major_upgrade_steps.j2.yaml@178 where UpgradeConfig_Step1 depends on Upgrade_Step0 | 09:36 |
| *** limao has quit IRC | 09:37 | |
| *** limao has joined #tripleo | 09:37 | |
| *** nmathew has joined #tripleo | 09:38 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo master: Add TLS in the internal network for Swift Proxy https://review.openstack.org/453028 | 09:39 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo master: Clean up TLS-related bits from swift-proxy https://review.openstack.org/453038 | 09:39 |
| *** karimb has quit IRC | 09:39 | |
| *** shardy has joined #tripleo | 09:39 | |
| chem | marios: yep but why do we do that, how come the config resource depends on the resource that get its config from it ? | 09:39 |
| chem | marios: the sentence is hard to read, but it's just a transliteration of the code I believe | 09:40 |
| chem | matbu: WDYT ? ^ | 09:40 |
| chem | marios: matbu so here we depends on the resource ROLEupgrade_stepX https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/major_upgrade_steps.j2.yaml#L109 | 09:42 |
| chem | marios: matbu and there we use it https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/major_upgrade_steps.j2.yaml#L147 | 09:42 |
| chem | marios: matbu is that a heat thing I'm missing ? | 09:42 |
| *** japestinho has joined #tripleo | 09:42 | |
| matbu | chem: nop | 09:44 |
| matbu | chem: the heat thing happen in UpgradeStepX | 09:44 |
| karimb_ | shouldnt the public urls for the undercloud be accessible from the outside? | 09:44 |
| marios | chem: replied at https://review.openstack.org/#/c/448602/15/puppet/major_upgrade_steps.j2.yaml | 09:45 |
| chem | marios: thanks .... I was confused, now I'm no longer | 09:47 |
| *** adarazs is now known as adarazs_lunch | 09:48 | |
| marios | chem: so in the example you point at UpgradeBatchConfig_Step{{step}}: depends_on UpgradeBatch_Step{{step -1}} i.e. the config depending on completion of the previous deployment resource (step-1) | 09:48 |
| chem | marios: yep, got it | 09:48 |
| marios | chem: i mean it is the same as I'm doing | 09:48 |
| *** thrash|g0ne is now known as thrash | 09:48 | |
| chem | marios: I +1 | 09:48 |
| chem | marios: hum ... wait ... so do we need the loop here ? | 09:49 |
| chem | marios: I mean the config resource doesn't depend on the other role resource, stricly speaking | 09:50 |
| chem | marios: it's only for the resource that we need to make sure that it's enforced | 09:50 |
| chem | marios: so I believe this would do https://review.openstack.org/#/c/452828/7/puppet/major_upgrade_steps.j2.yaml@188 | 09:53 |
| chem | marios: without the loop for config resource | 09:53 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates master: TLS-everywhere: Enable for TLS libvirt live migration https://review.openstack.org/450726 | 09:58 |
| *** limao has quit IRC | 09:58 | |
| *** limao has joined #tripleo | 09:58 | |
| openstackgerrit | Saravanan KR proposed openstack/tripleo-common master: Add an action to fetch and flatten the heat resource tree and parameters https://review.openstack.org/450021 | 10:01 |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-quickstart master: Add featureset011 - upgrade from BM to containerized https://review.openstack.org/450295 | 10:02 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates master: [WIP] Ensure non-batched tasks start after batched tasks https://review.openstack.org/452789 | 10:05 |
| openstackgerrit | Christopher Brown proposed openstack/tripleo-heat-templates master: Enable Neutron LBaaS Integration https://review.openstack.org/313933 | 10:06 |
| rook | gfidente any clue what is holding up ci for https://review.openstack.org/#/c/451826/ ? | 10:07 |
| *** snecklifter has joined #tripleo | 10:07 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo master: TLS-everywhere: Add resources for libvirt's cert for live migration https://review.openstack.org/450723 | 10:08 |
| gfidente | rook yeah you need to fix lint and tests | 10:08 |
| gfidente | rook give me a sec, I think I can update it | 10:09 |
| rook | gfidente: ok ? I don't mind doing it | 10:09 |
| rook | just tell me where to look gfidente | 10:09 |
| rook | looks like all the CI blewup on it :) | 10:09 |
| *** anshul has quit IRC | 10:10 | |
| gfidente | rook ah ok yes so there is a syntax issue http://logs.openstack.org/26/451826/8/check/gate-puppet-ceph-puppet-syntax-4-centos-7/c27dbe7/console.html#_2017-03-31_00_54_32_744960 | 10:11 |
| gfidente | and a bunch of lint issues http://logs.openstack.org/26/451826/8/check/gate-puppet-ceph-puppet-lint/56064e8/console.html#_2017-03-31_00_54_05_236356 | 10:11 |
| gfidente | best would be if you could run the tests locally before updating to make sure they pass | 10:12 |
| gfidente | but it takes a bit | 10:12 |
| rook | yeah, i don't have the env built to do local testing, is there a doc/gist/pastebin explaining what I need to do? | 10:12 |
| gfidente | not really, but you can install the ruby gems in the user dir | 10:13 |
| gfidente | rook sec I'll try to put up a pastebin | 10:13 |
| rook | gfidente: no worries -- typically when i see all that red i think CI is broke | 10:14 |
| rook | sorry for the bad assumption! | 10:14 |
| *** itzikb_ has joined #tripleo | 10:14 | |
| *** rook is now known as rook|mobile | 10:15 | |
| *** itzikb has quit IRC | 10:15 | |
| marios | chem: k, so is your current -1 at https://review.openstack.org/#/c/448602/15/puppet/major_upgrade_steps.j2.yaml@94 ? if so yeah i will update to remove the loop on the config definition | 10:16 |
| gfidente | rook|mobile try http://paste.openstack.org/show/605375/ | 10:17 |
| marios | chem: will you update https://review.openstack.org/#/c/452828/7/puppet/major_upgrade_steps.j2.yaml@188 | 10:17 |
| chem | marios: oups, you lost me. So you removing the loop for Config is ok for you ? | 10:18 |
| *** limao has quit IRC | 10:19 | |
| sshnaidm | adarazs, do you know if we have /home/jenkins/network-environment.yaml for non net-iso jobs? I don't see it generated: https://github.com/openstack/tripleo-quickstart-extras/blob/master/roles/overcloud-prep-config/tasks/main.yml#L20-L21 | 10:19 |
| *** limao has joined #tripleo | 10:20 | |
| marios | chem: well specifically the loop on the config part yeah otherwise why are you -1 on that review | 10:20 |
| chem | marios: only that | 10:20 |
| marios | chem: maybe lets start with that :) why are you -1 @ https://review.openstack.org/#/c/448602/ | 10:20 |
| marios | chem: ok will update for that then. agree we have the loop for the deployment resource so don't need controller batchconfig step1 to depend on all other role batchconfig step0. the controller batchstep1 does that already and is enough) | 10:21 |
| openstackgerrit | Christopher Brown proposed openstack/tripleo-heat-templates master: Enable Neutron LBaaS Integration https://review.openstack.org/313933 | 10:22 |
| chem | marios: oki, cool, will remove my W-1 on mine as well and we can land this thing :) | 10:22 |
| *** anshul has joined #tripleo | 10:23 | |
| marios | chem: no we need to run it | 10:23 |
| chem | marios: yes, of course :) | 10:23 |
| marios | chem: been waiting for this to settle before i haven't even run the latest versions yet last one i ran was v 13 @ https://review.openstack.org/#/c/448602 | 10:24 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates master: Enforce upgrade_batch_tasks before upgrade_tasks order https://review.openstack.org/448602 | 10:24 |
| *** jaganathan has quit IRC | 10:25 | |
| *** dsariel has quit IRC | 10:27 | |
| *** udesale__ has quit IRC | 10:30 | |
| skramaja | shardy: do you agree with the comment - https://review.openstack.org/#/c/441054/6/workbooks/baremetal.yaml@702 | 10:32 |
| *** jaganathan has joined #tripleo | 10:32 | |
| *** jkilpatr has quit IRC | 10:38 | |
| *** shardy has quit IRC | 10:38 | |
| *** owalsh_ is now known as owalsh | 10:39 | |
| openstackgerrit | Merged openstack/tripleo-common master: Validate nodes_json before trying to register or updates any nodes https://review.openstack.org/408581 | 10:39 |
| *** shardy has joined #tripleo | 10:40 | |
| *** limao has quit IRC | 10:40 | |
| *** limao has joined #tripleo | 10:41 | |
| *** shardy_ has joined #tripleo | 10:43 | |
| *** shardy has quit IRC | 10:44 | |
| shardy_ | skramaja: Hi, so yes I think it's a valid comment - I think what dtansur wants is to consume the list of matching nodes via the same API as "openstack overcloud profiles list" instead of directly matching the profile string in the ironic node capabilities | 10:44 |
| skramaja | thanks shardy_, i just wanted to cross check before doing it. thanks. i will do. | 10:45 |
| *** zoli|lunch is now known as zoli | 10:46 | |
| *** dbecker has joined #tripleo | 10:46 | |
| *** adarazs_lunch is now known as adarazs | 10:46 | |
| shardy_ | sorry dtantsur|bbl ^^ typoed your nick | 10:47 |
| marios | chem: will you post an update to https://review.openstack.org/#/c/452828/7/puppet/major_upgrade_steps.j2.yaml ? | 10:47 |
| shardy_ | skramaja: ack, thanks for picking it up! :) | 10:48 |
| *** arxcruz has quit IRC | 10:48 | |
| marios | chem: or is the -1 just from earlier and now waitin on the testing | 10:48 |
| adarazs | sshnaidm: I don't think we use that file or that it's needed when we don't have a network isolation in place. | 10:50 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: DO NOT MERGE: Testing TLS-everywhere with keystone container https://review.openstack.org/449570 | 10:50 |
| sshnaidm | panda, adarazs, do you know if we support overcloud_ssl without net-iso in oooq? | 10:51 |
| imran_work | uefi introspection just freezes my machines | 10:53 |
| panda | gchamoul: ^ | 10:53 |
| chem | marios: "Waiting for a successful run." | 10:54 |
| marios | chem: ack | 10:54 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Use mailx in yum-minimal https://review.openstack.org/453115 | 10:54 |
| gchamoul | sshnaidm: don't think so | 10:54 |
| *** lucasagomes is now known as lucas-hungry | 10:56 | |
| *** dtantsur|bbl is now known as dtantsur | 10:57 | |
| sshnaidm | panda, yet another point to complete | 10:57 |
| *** jaganathan has quit IRC | 10:57 | |
| bandini | EmilienM: will try to look at the bz you pointed out last night this afternoon (haven't forgotten ;) | 10:57 |
| ccamacho | hey shardy_ morning, I have some WIP working locally related to the pre|post puppet merge with pre|post config resources (https://review.openstack.org/#/c/452734), now, I have an issue with the upgrades job failing with this error "StackValidationFailed: resources.AllNodesDeploySteps.resources.ControllerPostConfig.resources.ControllerPostPuppetMaintenanceModeDeployment.resources.Controller: Property error: Controller.Properties | 10:57 |
| panda | sshnaidm: are we keeping track | 10:58 |
| panda | ? | 10:58 |
| sshnaidm | panda, what do you mean? we have this doc with features, right? | 10:58 |
| *** flepied has quit IRC | 11:01 | |
| florianf | mandre, shadower: Hi! Could you have a look at this patch: https://review.openstack.org/#/c/451711/ (It's supposed to fix validations CI in ocata). | 11:01 |
| *** limao has quit IRC | 11:02 | |
| *** garyk has joined #tripleo | 11:02 | |
| *** limao has joined #tripleo | 11:02 | |
| garyk | question: is it possible to copy files from the undercloud the the overcloud in post installation scripts? | 11:03 |
| garyk | if so how? | 11:03 |
| *** skramaja_ has joined #tripleo | 11:04 | |
| panda | sshnaidm: we have, the OVERCLOUD_SSL feature is marked as partially supported, the action item was to check certificate presence in deployment. The card to work on it is created. I'm not sure it's prioritized, and anyway, we have to add the implementation for SSL and non-netiso scenarios. So it's partially tracked, but infos have to be added, and at this point a prioiritization is required. | 11:06 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci master: Set SSL args for non-iso OVB job https://review.openstack.org/453126 | 11:06 |
| openstackgerrit | Jiri Tomasek proposed openstack/puppet-tripleo stable/ocata: Add tunnel timeout for ui proxy container https://review.openstack.org/453127 | 11:06 |
| *** skramaja has quit IRC | 11:06 | |
| *** dbecker has quit IRC | 11:06 | |
| sshnaidm | panda, I think certificate presence is different problem | 11:06 |
| sshnaidm | panda, actually I think certificate presence is not a problem now.. I'll check it today | 11:07 |
| gfidente | matbu marios so this works for me https://review.openstack.org/#/c/452789/ | 11:08 |
| gfidente | have you seen the rendered templates? http://sprunge.us/faKY | 11:09 |
| gfidente | it solves depends_on for all roles, for loops hiding needed resources and dependencies for batched vs non-batches | 11:10 |
| *** mgould has joined #tripleo | 11:10 | |
| jaosorior | sshnaidm: why is non-ha non-iso? | 11:11 |
| marios | chem: ERROR: Failed to validate: : resources.AllNodesDeploySteps: : The specified reference "ControllerUpgradeBatch_Step2}" (in CephStorageUpgrade_Step0) is incorrect.', | 11:11 |
| marios | chem: not sure where yet | 11:11 |
| sshnaidm | jaosorior, that's the featureset of non-ha job | 11:11 |
| jaosorior | funky | 11:11 |
| jaosorior | wasn't there a plan to converge the non-ha and ha jobs? so only net-iso would be tested? | 11:11 |
| sshnaidm | jaosorior, yes, hopefully new job will be such one | 11:12 |
| *** bkopilov has quit IRC | 11:12 | |
| jaosorior | ah, so it'll be a new job | 11:12 |
| jaosorior | alrighty | 11:12 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Introduce manual setting of DIB_INIT_SYSTEM https://review.openstack.org/414179 | 11:12 |
| sshnaidm | panda, do we have enabling ceph anywhere in featuresets/config files? | 11:13 |
| *** arxcruz has joined #tripleo | 11:13 | |
| * sshnaidm thinks if it worth to add ceph to ha job too.. | 11:14 | |
| gfidente | sshnaidm there was a conversation on the ml a while ago about making ceph the default for ha scenario | 11:15 |
| panda | sshnaidm: I think I remember a minimal_ceph configuration at a point in time | 11:15 |
| gfidente | because it doesn't make sense to have lvm in cinder when there are multiple cinder instances | 11:15 |
| gfidente | so I'd +1 that | 11:15 |
| *** flepied has joined #tripleo | 11:15 | |
| sshnaidm | gfidente, ok, will add this then | 11:15 |
| jaosorior | I would dig that | 11:15 |
| *** artom_ has joined #tripleo | 11:16 | |
| gfidente | sshnaidm you could try hci so it doesn't need additional nodes | 11:17 |
| gfidente | but then either we modify to deploy storagemgmt network on the compute nodes | 11:17 |
| gfidente | or we deploye without network isolation | 11:17 |
| gfidente | alternatively, we can use network isolation and ceph without modifying the standard nic templates deploying a dedicated cephstorage node | 11:17 |
| chem | marios: that would be L199 of https://review.openstack.org/#/c/448602/16/puppet/major_upgrade_steps.j2.yaml | 11:18 |
| jaosorior | A dedicated ceph storage node doesn't sound unreasonable if more services are added to the deployment. | 11:18 |
| *** itzikb_ has quit IRC | 11:18 | |
| *** jkilpatr has joined #tripleo | 11:18 | |
| *** artom_ has quit IRC | 11:18 | |
| *** stendulker has quit IRC | 11:18 | |
| marios | chem: yeah i think so the question is why | 11:19 |
| chem | marios: well the outer loop is the step number | 11:19 |
| sshnaidm | actually we deploy ceph on net-iso ovb-updates job: http://logs.openstack.org/60/447860/1/check-tripleo/gate-tripleo-ci-centos-7-ovb-updates/e63bc9e/logs/ | 11:19 |
| *** skramaja_ has quit IRC | 11:19 | |
| *** artom has joined #tripleo | 11:19 | |
| sshnaidm | but it has only one controller | 11:19 |
| mandre | florianf: I'm confused, is it still required after https://review.openstack.org/#/c/451815/? | 11:19 |
| chem | marios: so by the time we are at 0 the max -1 is 2 but 2 hasn't been defined yet | 11:19 |
| gfidente | marios can you post the rendered templates somewhere so I compare with the one I have? | 11:19 |
| chem | marios: this is sad | 11:19 |
| *** flepied has quit IRC | 11:19 | |
| gfidente | sshnaidm yeah we deploy it in nonha job as well | 11:19 |
| *** skramaja_ has joined #tripleo | 11:20 | |
| *** artom has quit IRC | 11:20 | |
| marios | gfidente: tox -e templates | 11:20 |
| chem | marios: for this to work we have to make two step loops | 11:20 |
| gfidente | marios yeah I mean the version resulting from your changes | 11:20 |
| *** artom has joined #tripleo | 11:20 | |
| sshnaidm | gfidente, yes, so I want to combine ha and non-ha jobs, and think what to do with ceph node | 11:20 |
| chem | marios: one for batch ond one for non batch | 11:20 |
| gfidente | sshnaidm +1 :) thanks! | 11:20 |
| *** artom__ has quit IRC | 11:20 | |
| chem | marios: so that when we reach non batch all batch have been defined | 11:20 |
| marios | chem: comment on the review? easier i think? | 11:20 |
| chem | marios: ack | 11:20 |
| *** pkovar has joined #tripleo | 11:21 | |
| *** rhallisey has joined #tripleo | 11:22 | |
| *** artom_ has joined #tripleo | 11:22 | |
| *** artom has quit IRC | 11:22 | |
| *** limao has quit IRC | 11:23 | |
| chem | marios: what I said was wrong ... | 11:23 |
| dtantsur | folks, are oooq jobs down? | 11:24 |
| *** limao has joined #tripleo | 11:24 | |
| dtantsur | it fails on something weird for me.. http://logs.openstack.org/78/453078/1/check/gate-tripleo-ci-centos-7-nonha-multinode-oooq/15ca666/console.html#_2017-04-04_09_47_55_491303 | 11:24 |
| dtantsur | trown|outtypewww, ^^^ | 11:24 |
| sshnaidm | panda, do we have "undercloud_telemetry" in our features file? | 11:26 |
| marios | gfidente: sure i could post them but it is just https://review.openstack.org/#/c/448602/16 and https://review.openstack.org/#/c/452828/7 so if you grab th efirst one it has the 2nd as parent so tox -e templates gives you same | 11:26 |
| gfidente | marios ok doing that | 11:26 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates master: Pass hieradata for TLS in the internal network for Ceph RGW https://review.openstack.org/409728 | 11:27 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates master: Pass hieradata for TLS in the internal network for Ceph RGW https://review.openstack.org/409728 | 11:27 |
| florianf | mandre: don't we? IIUC the global requirements patch *allows* us to exclude the bad versions. but we still have to *do* it, don't we? | 11:29 |
| sshnaidm | panda, nm, found it | 11:29 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci master: WIP: combined OVB ha and non-ha job https://review.openstack.org/453135 | 11:30 |
| chem | marios: I reviewed, don't know if that could be the root cause ... | 11:32 |
| *** eck` is now known as eck`gone | 11:32 | |
| garyk | shardy_: i am looking at http://hardysteven.blogspot.co.il/2016/08/tripleo-deploy-artifacts-and-puppet.html - question - if i use a file called x.tar.gz, will this be unpacket at /? | 11:34 |
| panda | sshnaidm: it's not implemented yet, it's installed by default AFAIK | 11:34 |
| *** mdnadeem has quit IRC | 11:36 | |
| openstackgerrit | Merged openstack/tripleo-quickstart master: Revert "Generate/inject instack.json if overcloud nodes" https://review.openstack.org/452998 | 11:36 |
| openstackgerrit | Saravanan KR proposed openstack/tripleo-specs master: Deriving TripleO Parameters https://review.openstack.org/423304 | 11:37 |
| openstackgerrit | Bogdan Dobrelya proposed openstack/tripleo-quickstart-extras master: Use openstack deploy to install the undercloud https://review.openstack.org/419040 | 11:37 |
| *** flepied has joined #tripleo | 11:38 | |
| *** artom_ has quit IRC | 11:40 | |
| *** artom_ has joined #tripleo | 11:40 | |
| mandre | florianf: you're right, I got a bit mixed up between global requirements and upper constraints | 11:42 |
| *** morazi has joined #tripleo | 11:43 | |
| florianf | mandre: there are two patches for that change, on in master and one in stable/ocata. I guess the cleanest way would be to abandon the one for ocata then approve the one in master and backport it to ocata. | 11:43 |
| florianf | mandre: that's the one in master: https://review.openstack.org/#/c/452898/ | 11:43 |
| *** artom__ has joined #tripleo | 11:44 | |
| *** limao has quit IRC | 11:44 | |
| marios | + | 11:44 |
| marios | 32-0+985 | 11:44 |
| *** artom_ has quit IRC | 11:44 | |
| *** limao has joined #tripleo | 11:45 | |
| mandre | florianf: exact, thanks for pointing it out, I missed the change in master | 11:45 |
| *** ccamacho is now known as ccamacho|lunch | 11:46 | |
| *** abishop_ has joined #tripleo | 11:47 | |
| *** artom_ has joined #tripleo | 11:48 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/tripleo-common master: Updated from global requirements https://review.openstack.org/451075 | 11:48 |
| *** jaganathan has joined #tripleo | 11:49 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates master: [WIP] Ensure non-batched tasks start after batched tasks https://review.openstack.org/452789 | 11:50 |
| bogdando | jistr, flaper87, mandre: hostpath patch seems passing ! https://review.openstack.org/#/c/442603/ | 11:50 |
| gfidente | marios chem I am telling you I have a version which fixes it all | 11:51 |
| gfidente | https://review.openstack.org/452789 | 11:51 |
| bogdando | could we merge it fast before broken due to conflicts? | 11:51 |
| gfidente | I've got upgrade passing, updating twice | 11:51 |
| *** mdnadeem has joined #tripleo | 11:51 | |
| gfidente | it condenses both fixes | 11:51 |
| *** fzdarsky_ is now known as fzdarsky|lunch | 11:51 | |
| jistr | bogdando: it's all jobs that don't deploy containers though | 11:52 |
| gfidente | can we reproduce that behavior in your patches and merge them? | 11:52 |
| jaosorior | bogdando: the regular gate doesn't test containers | 11:52 |
| jaosorior | bogdando: I did check experimental so the containers job runs | 11:52 |
| bogdando | jistr: oh, I didn't nore that | 11:52 |
| bogdando | note | 11:52 |
| *** artom__ has quit IRC | 11:52 | |
| d0ugal | thrash: it is safe to rebase this now :) https://review.openstack.org/#/c/451624/ | 11:53 |
| thrash | d0ugal: excellent. :) | 11:53 |
| bogdando | then let's wait for CI, it will likely fail :) | 11:53 |
| *** deadnull has joined #tripleo | 11:53 | |
| mandre | bogdando: great! I'll have a look | 11:53 |
| *** jpena is now known as jpena|lunch | 11:54 | |
| *** artom_ has quit IRC | 11:54 | |
| marios | gfidente: checking | 11:54 |
| gfidente | marios check the rendered templates http://sprunge.us/faKY | 11:54 |
| gfidente | chem ^^ | 11:54 |
| mandre | anyone else seeing zaqar errors throwing a 401 (Unauthorized)? | 11:54 |
| mandre | http://paste.openstack.org/show/605387/ | 11:54 |
| gfidente | I tried to describe in the commit message the various changes it includes | 11:55 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates master: Pass hieradata for TLS in the internal network for Ceph RGW https://review.openstack.org/409728 | 11:55 |
| *** artom has joined #tripleo | 11:56 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo master: Enable TLS in the internal network for Ceph RGW https://review.openstack.org/453141 | 11:56 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo master: Enable TLS in the internal network for Ceph RGW https://review.openstack.org/453141 | 11:56 |
| gfidente | marios chem we don't need to remove the conditions, Heat will do the right thing | 11:56 |
| openstackgerrit | James Slagle proposed openstack/instack-undercloud master: Remove instack-virt-setup https://review.openstack.org/451914 | 11:56 |
| *** Guest20391 is now known as assassin | 11:57 | |
| gfidente | we need to make jinja create the resources for all roles instead | 11:57 |
| marios | gfidente: so you remove the dependencies from configs alltogether | 11:57 |
| gfidente | plus there is other stuff described in the commit msg | 11:57 |
| gfidente | marios right they can be created at all times | 11:57 |
| gfidente | the deployments have to be orchestrated | 11:57 |
| gfidente | if you try an upgrade run it works as intended | 11:57 |
| *** artom has quit IRC | 11:58 | |
| *** dbecker has joined #tripleo | 11:58 | |
| gfidente | marios chem the rendered templates are actually easier to read as well | 11:58 |
| *** artom has joined #tripleo | 11:58 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo master: TLS-everywhere: Add resources for libvirt's cert for live migration https://review.openstack.org/450723 | 11:59 |
| dtantsur | slagle, good catch about dependencies on tripleo-docs >_< | 11:59 |
| *** artom has quit IRC | 12:00 | |
| *** artom has joined #tripleo | 12:00 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/puppet-tripleo master: Stop including ironic::drivers::ssh in the ironic-conductor profile https://review.openstack.org/453078 | 12:00 |
| openstackgerrit | Dmitry Tantsur proposed openstack/instack-undercloud master: Remove support for the deprecated pxe_ssh driver https://review.openstack.org/453081 | 12:01 |
| *** rodrigod` is now known as rodrigods | 12:01 | |
| *** rodrigods has joined #tripleo | 12:01 | |
| *** rodrigods has quit IRC | 12:01 | |
| *** rodrigods has joined #tripleo | 12:02 | |
| *** lucas-hungry is now known as lucasagomes | 12:02 | |
| adarazs | guys, I'm trying to set up auto-abandon for old changes for quickstart project. does somebody know how to do that? | 12:02 |
| *** dtantsur is now known as dtantsur|mtg | 12:03 | |
| *** rodrigods has quit IRC | 12:03 | |
| *** rodrigods has joined #tripleo | 12:03 | |
| slagle | dtantsur|mtg: oh you do the depends-on docs changes too? :) | 12:03 |
| *** trown|outtypewww is now known as trown | 12:03 | |
| dtantsur|mtg | slagle, yep :( | 12:04 |
| marios | chem thanks for pointing out extra } in https://review.openstack.org/#/c/448602/16/puppet/major_upgrade_steps.j2.yaml@199 | 12:04 |
| *** dsariel has joined #tripleo | 12:04 | |
| trown | dtantsur|mtg: slagle is there a quickstart bug for that? we should handle that better | 12:05 |
| * dtantsur|mtg just noticed, no bug so far | 12:05 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates master: Enforce upgrade_batch_tasks before upgrade_tasks order https://review.openstack.org/448602 | 12:05 |
| *** limao has quit IRC | 12:06 | |
| openstackgerrit | Bogdan Dobrelya proposed openstack/tripleo-quickstart-extras master: Custom containerized undercloud deployments https://review.openstack.org/450792 | 12:06 |
| *** limao has joined #tripleo | 12:06 | |
| slagle | trown: yes i just filed it | 12:06 |
| slagle | let me get the link | 12:06 |
| openstackgerrit | Pradeep Kilambi proposed openstack/puppet-tripleo stable/ocata: Decouple ceilometer user create from API https://review.openstack.org/452541 | 12:06 |
| slagle | trown: https://bugs.launchpad.net/tripleo-quickstart/+bug/1679650 | 12:06 |
| openstack | Launchpad bug 1679650 in tripleo-quickstart "quickstart fails trying to map project name tripleo-docs for dlrn" [High,New] | 12:06 |
| trown | slagle: awesome thanks! | 12:07 |
| *** amoralej is now known as amoralej|lunch | 12:08 | |
| *** shardy_ is now known as shardy_lunch | 12:08 | |
| *** nyechiel has joined #tripleo | 12:09 | |
| *** karimb_ has quit IRC | 12:09 | |
| *** chlong has joined #tripleo | 12:10 | |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-docs master: WIP: Post deployment installation instructions for Octavia https://review.openstack.org/447496 | 12:10 |
| *** Guest6565 has quit IRC | 12:11 | |
| *** artom has quit IRC | 12:12 | |
| EmilienM | bandini: thanks | 12:12 |
| *** artom has joined #tripleo | 12:12 | |
| flaper87 | bogdando: thanks | 12:14 |
| bandini | slagle: did you solve your cluster thing last night in the end? | 12:15 |
| *** maeca1 has joined #tripleo | 12:15 | |
| openstackgerrit | Merged openstack/python-tripleoclient master: Remove the single-message interface to websockets https://review.openstack.org/446439 | 12:16 |
| *** artom has quit IRC | 12:16 | |
| *** artom has joined #tripleo | 12:16 | |
| ccamacho|lunch | Hey slagle! copy/paste the help message :) I have some WIP working locally related to the pre|post puppet merge with pre|post config resources (https://review.openstack.org/#/c/452734), now, I have an issue with the upgrades job failing with this error "StackValidationFailed: resources.AllNodesDeploySteps.resources.ControllerPostConfig.resources.ControllerPostPuppetMaintenanceModeDeployment.resources.Controller: Property | 12:17 |
| *** ansmith has joined #tripleo | 12:17 | |
| bogdando | mandre: the typical error I have on my undercloud container envs as well http://logs.openstack.org/03/442603/5/experimental/gate-tripleo-ci-centos-7-undercloud-containers/cb7c85d/console.html#_2017-04-04_12_07_11_557343 | 12:17 |
| bogdando | mandre: that stands for the failed CI https://review.openstack.org/#/c/442603/ | 12:18 |
| *** artom has quit IRC | 12:18 | |
| *** artom has joined #tripleo | 12:18 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: DO NOT MERGE: Testing TLS-everywhere with keystone container https://review.openstack.org/449570 | 12:20 |
| *** morazi has quit IRC | 12:20 | |
| *** karimb has joined #tripleo | 12:20 | |
| *** dprince has joined #tripleo | 12:22 | |
| *** dbecker has quit IRC | 12:22 | |
| *** itzikb has joined #tripleo | 12:23 | |
| itzikb | Hi, How do I add release notes for my patch? | 12:23 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo master: Add TLS in the internal network for Swift Proxy https://review.openstack.org/453028 | 12:24 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo master: Clean up TLS-related bits from swift-proxy https://review.openstack.org/453038 | 12:24 |
| jaosorior | itzikb: reno new "Some title goes here" | 12:24 |
| *** jkilpatr has quit IRC | 12:24 | |
| EmilienM | sshnaidm: how are things today? | 12:24 |
| *** mich_chap has joined #tripleo | 12:24 | |
| EmilienM | sshnaidm: just an fyi, please don't review https://review.openstack.org/#/c/451938/ yet | 12:25 |
| mandre | bogdando: I think your error is with mysql boostrapping, http://paste.openstack.org/show/605392/ | 12:25 |
| itzikb | jaosorior: sorry. reno? | 12:25 |
| *** bfournie has joined #tripleo | 12:25 | |
| jaosorior | itzikb: yep https://docs.openstack.org/developer/reno/ | 12:25 |
| sshnaidm | EmilienM, looks good in general, periodic ovb-updates job failed because timeout | 12:26 |
| EmilienM | sshnaidm: oh damn again? | 12:27 |
| *** limao has quit IRC | 12:27 | |
| sshnaidm | EmilienM, yeah, a few minutes.. | 12:27 |
| *** morazi has joined #tripleo | 12:27 | |
| openstackgerrit | Saravanan KR proposed openstack/tripleo-common master: Add an action to fetch and flatten the heat resource tree and parameters https://review.openstack.org/450021 | 12:27 |
| EmilienM | sshnaidm: it's the image building I guess | 12:27 |
| openstackgerrit | Pradeep Kilambi proposed openstack/instack-undercloud master: Dont include wsgi profile when legacy flag is disabled https://review.openstack.org/453150 | 12:27 |
| EmilienM | sshnaidm: which will probably be improved with the local mirrors that we'll use | 12:27 |
| *** mdnadeem has quit IRC | 12:27 | |
| EmilienM | sshnaidm: instead of centos.org repos | 12:27 |
| *** limao has joined #tripleo | 12:27 | |
| *** jayg|g0n3 is now known as jayg | 12:28 | |
| sshnaidm | EmilienM, hope so | 12:28 |
| EmilienM | sshnaidm: let's try to get a promotion and if by Wednesday end of day we don't have one, let's remove it again from the pipeline | 12:28 |
| openstackgerrit | Saravanan KR proposed openstack/tripleo-common master: Add an action to fetch and flatten the heat resource tree and parameters https://review.openstack.org/450021 | 12:28 |
| sshnaidm | EmilienM, I can always run it manually without ovb-updates | 12:29 |
| *** ramishra has joined #tripleo | 12:29 | |
| *** jchhatbar has quit IRC | 12:30 | |
| *** remix_tj has quit IRC | 12:30 | |
| *** pradk has joined #tripleo | 12:32 | |
| EmilienM | sshnaidm: yeah, go for it otherwise | 12:33 |
| openstackgerrit | Jiri Tomasek proposed openstack/tripleo-ui master: Add Toolbar component and filters state https://review.openstack.org/452175 | 12:34 |
| *** rlandy has joined #tripleo | 12:34 | |
| *** noslzzp has joined #tripleo | 12:34 | |
| openstackgerrit | Jiri Stransky proposed openstack-infra/tripleo-ci master: Support container upgrades in multinode OOOQ CI https://review.openstack.org/450784 | 12:37 |
| openstackgerrit | Itzik Brown proposed openstack/tripleo-heat-templates master: Exposing the ability to enable/disable the repository https://review.openstack.org/452521 | 12:37 |
| weshay | adarazs, trown fyi... created a mtg today to chat w/ david simard re: container image builds | 12:37 |
| *** skramaja has joined #tripleo | 12:38 | |
| *** jkilpatr has joined #tripleo | 12:39 | |
| *** shardy_lunch is now known as shardy | 12:40 | |
| openstackgerrit | Florian Fuchs proposed openstack/tripleo-validations stable/ocata: prevent validations from failing when undercloud.conf is missing: https://review.openstack.org/453154 | 12:40 |
| *** skramaja_ has quit IRC | 12:41 | |
| *** eck`gone is now known as eck` | 12:41 | |
| rook|mobile | ok gfidente mwhahaha - the linting of puppet w/ bash it is complaining about enclosing things in {} however the line it starts on is set -ex... | 12:41 |
| rook|mobile | https://review.openstack.org/#/c/451826 gfidente mwhahaha commit in question | 12:42 |
| adarazs | weshay: okay. | 12:42 |
| rook|mobile | I have the s/DISK/disk fixed. | 12:42 |
| mwhahaha | rook|mobile: I think you need to escape the $ for DISK. \$DISK also I saw one where you have {} for disk | 12:44 |
| *** skramaja_ has joined #tripleo | 12:44 | |
| mwhahaha | rook|mobile: I'll look more later when I get to my desk | 12:44 |
| trown | weshay: k | 12:45 |
| *** mdnadeem has joined #tripleo | 12:45 | |
| *** rook|mobile is now known as rook-tower | 12:45 | |
| *** janki has joined #tripleo | 12:45 | |
| *** liverpooler has joined #tripleo | 12:46 | |
| *** skramaja has quit IRC | 12:47 | |
| *** limao has quit IRC | 12:48 | |
| openstackgerrit | Luke Hinds proposed openstack/puppet-tripleo master: SSHD Service extensions https://review.openstack.org/443113 | 12:48 |
| *** limao has joined #tripleo | 12:49 | |
| *** jpena|lunch is now known as jpena | 12:50 | |
| chem | gfidente: he, I really think that correctness should come before speed. So maybe there is room for improvement (there always will be) be for now it would be nice just to have it working. Then we can make improvment. WDYT ? | 12:51 |
| chem | gfidente: furthermore do we know at what kind of optimization we are looking at when we don't create the resource in terms of time and memory size ? | 12:52 |
| gfidente | chem note that I am not trying to optimize | 12:54 |
| gfidente | I am avoiding removal of existing optimizations | 12:55 |
| *** jcoufal has joined #tripleo | 12:55 | |
| gfidente | we have a working submission which fixes the bug without removing the conditions https://review.openstack.org/#/c/452789 | 12:55 |
| chem | gfidente: well it make the whole thing simpler to debug for the current state of affair (ie non working orchestration) | 12:55 |
| gfidente | what makes debug simpler? | 12:56 |
| EmilienM | dprince: you might want to see https://review.openstack.org/#/c/401003/ | 12:56 |
| EmilienM | pabelanger: hey can you attend the tripleo meeting on IRC in 1h ? | 12:56 |
| shardy | chem: the reason for the conditionals isn't to reduce memory usage, it's to avoid multiple round-trips to all the nodes to have a heat hook do nothing | 12:57 |
| shardy | because it's serialized over each node, that will take a noticeable amount of time for upgrade_batch_tasks | 12:57 |
| gfidente | shardy so this finally worked https://review.openstack.org/#/c/452789 | 12:57 |
| shardy | chem: that said, I agree, if it's causing problems I'd be OK with removing it and re-optimizing later | 12:57 |
| *** skramaja_ has quit IRC | 12:57 | |
| *** ccamacho|lunch is now known as ccamacho | 12:58 | |
| gfidente | shardy the issue was on UPDATE_FAILED | 12:58 |
| *** fzdarsky|lunch is now known as fzdarsky | 12:58 | |
| gfidente | shardy deploying the initial overcloud with the correct templates made update and deps on conditionals to work | 12:58 |
| shardy | gfidente: aha, can you reproduce that now, e.g you could use OS::Heat::TestResource in a minimal repoducer? | 12:59 |
| gfidente | shardy no I only figured I should have deployed the initial overcloud with those templates | 12:59 |
| chem | shardy: we faced this other issue https://bugs.launchpad.net/tripleo/+bug/1679486 and it seems that gfidente fix is able to handle this one as well | 12:59 |
| openstack | Launchpad bug 1679486 in tripleo "N->O Upgrade, ochestration is broken." [Critical,In progress] - Assigned to Marios Andreou (marios-b) | 12:59 |
| shardy | chem: ack | 12:59 |
| gfidente | chem ack, check the commit message and especially the rendered templats | 12:59 |
| shardy | gfidente: about to go on a call now, but I'll review later | 12:59 |
| gfidente | chem the rendered templates are much easier to parse | 13:00 |
| chem | gfidente: oki, thanks | 13:00 |
| gfidente | I mean easier for a human | 13:00 |
| gfidente | you just get all resources for all roles | 13:00 |
| gfidente | with depends_on and conditions | 13:00 |
| gfidente | and let heat do the work | 13:00 |
| chem | marios: you're following this, gfidente just want to get rid of our patches :) | 13:00 |
| chem | gfidente: that sound good :) | 13:01 |
| gfidente | marios chem you will hate me forever | 13:01 |
| gfidente | I know | 13:01 |
| *** dtantsur|mtg is now known as dtantsur | 13:01 | |
| gfidente | but we can make that approach in your submissions | 13:01 |
| gfidente | the reason why I opened a new one is that I didn't want to overwrite the existing | 13:01 |
| gfidente | because of the fundamental difference in relying more on jinja conditions vs heat conditions | 13:02 |
| gfidente | if we remove the heat conditions upgrade takes longer, as shardy pointed | 13:02 |
| chem | gfidente: for me the fundamental is to have orchestration working :) | 13:02 |
| gfidente | and it does | 13:02 |
| gfidente | ! | 13:03 |
| *** lhinds|afk is now known as lhinds | 13:03 | |
| chem | gfidente: oki, np, I don't care which code solve it :) | 13:03 |
| weshay | bogdando, your update fixed the undercloud install.. +1 | 13:03 |
| chem | gfidente: yours seems more elegent though | 13:03 |
| *** links has quit IRC | 13:04 | |
| gfidente | shardy break the flag in roles_data to disable upgrades ? | 13:04 |
| *** jchhatbar has joined #tripleo | 13:05 | |
| snecklifter | shardy: https://review.openstack.org/#/c/313933/ <- thanks for review, I dived in, any better? | 13:07 |
| *** rbowen has joined #tripleo | 13:07 | |
| *** janki has quit IRC | 13:07 | |
| shardy | gfidente: there's a flag used to calculate enabled_roles, see the top of the file | 13:07 |
| shardy | gfidente: won't we ignore that now and just create SoftwareDeploymentGroups for all roles? | 13:08 |
| *** tzumainn has joined #tripleo | 13:08 | |
| shardy | that was the point of looping on enabled_roles | 13:08 |
| shardy | gfidente: the depends_on lists should all be created based on enabled_roles, so nothing referenced is ever missing | 13:08 |
| gfidente | shardy yes I see what you mean now, if there are upgrade_tasks in a role they are executed | 13:08 |
| gfidente | because the conditional is not false | 13:09 |
| shardy | gfidente: yes but there are two conditionals | 13:09 |
| shardy | one in jinja and one in heat | 13:09 |
| shardy | the first one is broken now AFAICT | 13:09 |
| gfidente | right that removes the one in jinja | 13:09 |
| *** Goneri has joined #tripleo | 13:09 | |
| shardy | Ok, cool, yeah we can't do that | 13:09 |
| *** limao has quit IRC | 13:09 | |
| *** bkopilov has joined #tripleo | 13:09 | |
| gfidente | shardy can we just look the depends over enabled_roles too then? | 13:09 |
| gfidente | *loop | 13:10 |
| *** artom has quit IRC | 13:10 | |
| *** limao has joined #tripleo | 13:10 | |
| *** shardy is now known as shardy_mtg | 13:10 | |
| *** artom has joined #tripleo | 13:10 | |
| gfidente | shardy_mtg lines 183 and 105 | 13:10 |
| gfidente | I'll update the submission | 13:10 |
| openstackgerrit | Martin André proposed openstack/tripleo-common master: Remove Percona-Server-shared-56 from openstack-base image https://review.openstack.org/453164 | 13:12 |
| openstackgerrit | mathieu bultel proposed openstack/tripleo-quickstart-extras master: Allow complex upgrade deployment for N to O https://review.openstack.org/439598 | 13:13 |
| openstackgerrit | mathieu bultel proposed openstack/tripleo-quickstart-extras master: Download rpm tht package for mixed upgrade https://review.openstack.org/449350 | 13:13 |
| *** itzikb has quit IRC | 13:13 | |
| *** amoralej|lunch is now known as amoralej | 13:14 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates master: [WIP] Ensure non-batched tasks start after batched tasks https://review.openstack.org/452789 | 13:15 |
| gfidente | shardy_mtg ^^ should do it | 13:17 |
| *** trozet has joined #tripleo | 13:18 | |
| slagle | can someone review https://review.openstack.org/#/c/452836/ plz? | 13:20 |
| slagle | we need to eventualy backport it to ocata so that pacemaker works with split stack | 13:21 |
| *** prateek_ has joined #tripleo | 13:22 | |
| openstackgerrit | Carlos Camacho proposed openstack/tripleo-heat-templates master: Merge pre|post puppet resources into pre|post config. https://review.openstack.org/452734 | 13:23 |
| *** liverpooler has quit IRC | 13:24 | |
| *** toure|biab is now known as toure | 13:24 | |
| *** prateek has quit IRC | 13:24 | |
| *** liverpooler has joined #tripleo | 13:24 | |
| slagle | thanks :) | 13:25 |
| *** jaganathan has quit IRC | 13:25 | |
| gfidente | slagle ty :) | 13:25 |
| *** lblanchard has joined #tripleo | 13:26 | |
| *** prateek_ has quit IRC | 13:27 | |
| *** jchhatbar has quit IRC | 13:32 | |
| *** suuuper has joined #tripleo | 13:34 | |
| *** liverpooler has quit IRC | 13:34 | |
| *** limao has quit IRC | 13:34 | |
| *** liverpooler has joined #tripleo | 13:34 | |
| *** limao has joined #tripleo | 13:35 | |
| *** jaosorior is now known as jaosorior_away | 13:36 | |
| *** pblaho has quit IRC | 13:36 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-quickstart master: Use AFS mirrors for optstool, delorean and qemu https://review.openstack.org/451938 | 13:37 |
| EmilienM | tripleo meeting in 20 min, on #openstack-meeting-alt - feel free to add items for today's agenda https://etherpad.openstack.org/p/tripleo-meeting-items | 13:42 |
| *** eck` is now known as eck`gone | 13:43 | |
| openstackgerrit | Bogdan Dobrelya proposed openstack/tripleo-heat-templates master: Mount individual hostpath logs on /var/log https://review.openstack.org/442603 | 13:44 |
| *** maeca1 has quit IRC | 13:44 | |
| *** dsariel has quit IRC | 13:45 | |
| *** garyk has quit IRC | 13:47 | |
| openstackgerrit | Adriano Petrich proposed openstack/tripleo-common master: add caching the GetParametersAction https://review.openstack.org/444220 | 13:47 |
| openstackgerrit | Florian Fuchs proposed openstack/tripleo-validations stable/ocata: Migrate from keystoneclient to keystoneauth https://review.openstack.org/453180 | 13:50 |
| jrist | jtomasek: quick question - on 11/ocata we don't have favicon built into webpack yet - can I just shove it into the index.html? | 13:50 |
| jtomasek | jrist: I think so, if there is index.html | 13:52 |
| *** limao has quit IRC | 13:52 | |
| jrist | jtomasek: there is for now :) | 13:52 |
| jtomasek | ok | 13:52 |
| jrist | with webpack + favicon or html plugin it'd be something like index.ejs | 13:52 |
| *** eck`gone is now known as eck` | 13:52 | |
| *** limao has joined #tripleo | 13:52 | |
| *** nmathew has quit IRC | 13:53 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/instack-undercloud stable/ocata: TEST: Set a dependency on collector https://review.openstack.org/453181 | 13:56 |
| beagles | ls | 13:56 |
| beagles | argh | 13:57 |
| *** shardy_mtg is now known as shardy | 13:57 | |
| openstackgerrit | Florian Fuchs proposed openstack/tripleo-validations stable/ocata: Use generic keystone plugins instead of version-specific https://review.openstack.org/453182 | 13:58 |
| openstackgerrit | Jiri Tomasek proposed openstack/tripleo-ui master: Add Toolbar component and filters state https://review.openstack.org/452175 | 13:59 |
| *** cdearborn has joined #tripleo | 13:59 | |
| EmilienM | tripleo meeting | 14:00 |
| openstackgerrit | Florian Fuchs proposed openstack/tripleo-validations stable/ocata: Fix token auth in ansible inventory script https://review.openstack.org/453183 | 14:00 |
| *** akuznetsov has joined #tripleo | 14:01 | |
| *** janki has joined #tripleo | 14:03 | |
| openstackgerrit | Florian Fuchs proposed openstack/tripleo-validations stable/ocata: Add missing mistralclient requirement https://review.openstack.org/453188 | 14:05 |
| openstackgerrit | James Slagle proposed openstack/tripleo-heat-templates master: Use the local collector to bootstrap deployed servers https://review.openstack.org/405032 | 14:05 |
| openstackgerrit | Martin André proposed openstack/tripleo-heat-templates master: Use httpd in Heat docker services https://review.openstack.org/447504 | 14:05 |
| *** mdnadeem has quit IRC | 14:07 | |
| openstackgerrit | Florian Fuchs proposed openstack/tripleo-validations stable/ocata: Update reno for stable/ocata https://review.openstack.org/453191 | 14:08 |
| *** akuznetsov has quit IRC | 14:08 | |
| *** dmarlin has joined #tripleo | 14:08 | |
| openstackgerrit | Florian Fuchs proposed openstack/tripleo-validations stable/ocata: Typo fix: choses => chooses https://review.openstack.org/453192 | 14:08 |
| openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart-extras master: Use web repo with installed packages for image build https://review.openstack.org/451936 | 14:13 |
| *** limao has quit IRC | 14:13 | |
| *** limao has joined #tripleo | 14:14 | |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci master: WIP: Support built packages to be installed in image https://review.openstack.org/452755 | 14:15 |
| *** jaganathan has joined #tripleo | 14:17 | |
| openstackgerrit | Jiri Tomasek proposed openstack/tripleo-ui master: Correctly mark number values in translation https://review.openstack.org/453194 | 14:17 |
| *** aufi has quit IRC | 14:18 | |
| *** gkadam has quit IRC | 14:19 | |
| *** jpstnh has quit IRC | 14:20 | |
| *** itzikb has joined #tripleo | 14:21 | |
| *** ealcaniz has quit IRC | 14:23 | |
| *** janki has quit IRC | 14:23 | |
| *** jaganathan has quit IRC | 14:24 | |
| *** itzikb has quit IRC | 14:24 | |
| *** dsariel has joined #tripleo | 14:24 | |
| *** yprokule has quit IRC | 14:25 | |
| *** janki has joined #tripleo | 14:29 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Purge initial firewall for deployed-server's https://review.openstack.org/452836 | 14:33 |
| *** limao has quit IRC | 14:34 | |
| *** limao has joined #tripleo | 14:35 | |
| *** zoli is now known as zoli|brb | 14:37 | |
| *** matbu is now known as matbu|bbl | 14:40 | |
| openstackgerrit | Cyril Lopez proposed openstack/puppet-tripleo master: Add a trigger to call ldap_backend define https://review.openstack.org/451829 | 14:41 |
| chem | matbu|bbl: marios gfidente and anybody intersted on upgrade/orchestration and heat conditional are welcome in ~15min (at next X:00) https://redhats.bluejeans.com/5369692477 | 14:44 |
| gfidente | chem++ thanks | 14:45 |
| openstackgerrit | James Slagle proposed openstack/tripleo-heat-templates stable/ocata: Purge initial firewall for deployed-server's https://review.openstack.org/453207 | 14:46 |
| *** janki has quit IRC | 14:48 | |
| *** garyk has joined #tripleo | 14:49 | |
| gfidente | rook-tower looks like syntax passed | 14:50 |
| garyk | question - is there a hook to do something before database migration in controller deployment? | 14:50 |
| *** dbecker has joined #tripleo | 14:50 | |
| gfidente | rook-tower need fixes for lint and then probably the tests | 14:50 |
| gfidente | rook-tower lint too :) | 14:52 |
| *** janki has joined #tripleo | 14:52 | |
| openstackgerrit | Carlos Camacho proposed openstack/tripleo-heat-templates master: Merge pre|post puppet resources into pre|post config. https://review.openstack.org/452734 | 14:53 |
| *** itzikb has joined #tripleo | 14:55 | |
| *** limao has quit IRC | 14:56 | |
| *** limao has joined #tripleo | 14:56 | |
| *** skramaja has joined #tripleo | 14:58 | |
| *** jpena is now known as jpena|away | 15:00 | |
| *** ratailor has joined #tripleo | 15:00 | |
| EmilienM | arxcruz: please /join #openstack-dev | 15:01 |
| *** fpan has joined #tripleo | 15:01 | |
| chem | matbu|bbl: you wanna come ? | 15:03 |
| pradk | EmilienM, can you remove your -1 and merge this https://review.openstack.org/#/c/452541/ .. update job passes as expected | 15:04 |
| pradk | since the tht patch is merged | 15:04 |
| sshnaidm | trown, how can I ensure that overcloud vip gets specific IP? | 15:05 |
| *** yamahata has joined #tripleo | 15:05 | |
| EmilienM | arxcruz: I wanted to discuss with Matt again on #openstack-dev. Ping me when you're on this channel. | 15:05 |
| EmilienM | pradk: approved | 15:05 |
| pradk | ty sir | 15:05 |
| *** agurenko has quit IRC | 15:05 | |
| EmilienM | arxcruz: I -1 your spec, I think we don't need such a tool since OpenStack Infra already provides the mechanism for RSS, we could probably re-use it instead of writting our own tool | 15:06 |
| openstackgerrit | Cyril Lopez proposed openstack/tripleo-heat-templates master: Add trigger to setup a LDAP backend as keystone domaine https://review.openstack.org/451830 | 15:06 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Remove not-working all-in-one upgrade environment https://review.openstack.org/452800 | 15:06 |
| EmilienM | arxcruz: and if your tool bring more features that openstack-health, let's discuss it here | 15:06 |
| openstackgerrit | Cyril Lopez proposed openstack/tripleo-heat-templates master: Add trigger to setup a LDAP backend as keystone domaine https://review.openstack.org/451830 | 15:08 |
| rook-tower | gfidente: oh i thought i fixed it | 15:12 |
| *** yamahata has quit IRC | 15:12 | |
| rook-tower | lint is fixed gfidente | 15:13 |
| *** jaganathan has joined #tripleo | 15:14 | |
| rook-tower | gfidente lol I am at a loss on this other stuff | 15:16 |
| *** [1]cdearborn has joined #tripleo | 15:17 | |
| *** limao has quit IRC | 15:17 | |
| *** limao has joined #tripleo | 15:18 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo stable/ocata: Deploy WSGI apps at the same step (3) https://review.openstack.org/453219 | 15:18 |
| EmilienM | mwhahaha: (not clean but working) backport ^ | 15:18 |
| *** leanderthal is now known as leanderthal|afk | 15:18 | |
| openstackgerrit | Luke Hinds proposed openstack/puppet-tripleo master: SSHD Service extensions https://review.openstack.org/443113 | 15:18 |
| mwhahaha | EmilienM: k | 15:18 |
| *** ukalifon has quit IRC | 15:19 | |
| openstackgerrit | Alex Schultz proposed openstack/tripleo-heat-templates master: Run update after RHEL registration https://review.openstack.org/449724 | 15:21 |
| *** jaganathan has quit IRC | 15:21 | |
| chandankumar | trown: please have a look on this review https://review.openstack.org/451747 | 15:23 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo stable/newton: Deploy WSGI apps at the same step (3) https://review.openstack.org/453221 | 15:25 |
| EmilienM | mwhahaha: and to newton ^ :D let's see if all of this pass CI | 15:25 |
| openstackgerrit | Merged openstack/puppet-tripleo stable/newton: Move horizon to step 3 https://review.openstack.org/452977 | 15:25 |
| openstackgerrit | Harry Rybacki proposed openstack/tripleo-quickstart master: [WIP] Add supplemental node provisioner https://review.openstack.org/451523 | 15:28 |
| openstackgerrit | Alex Schultz proposed openstack/tripleo-heat-templates master: Allow for update after RHEL registration https://review.openstack.org/449724 | 15:29 |
| openstackgerrit | Harry Rybacki proposed openstack/tripleo-quickstart master: Update quickstart-extras to deploy FreeIPA https://review.openstack.org/453223 | 15:30 |
| * adarazs off | 15:30 | |
| *** dparkes has quit IRC | 15:31 | |
| shardy | sshnaidm: Have you looked at the *FixedIPs parameters (see overcloud.j2.yaml) ? | 15:32 |
| openstackgerrit | Harry Rybacki proposed openstack/tripleo-quickstart-extras master: [WIP] Add role to deploy FreeIPA https://review.openstack.org/436198 | 15:32 |
| *** cdearborn has quit IRC | 15:32 | |
| *** nyechiel has quit IRC | 15:32 | |
| *** nyechiel has joined #tripleo | 15:34 | |
| *** anshul has quit IRC | 15:34 | |
| sshnaidm | shardy, looking.. is it PublicVirtualFixedIPs ? | 15:35 |
| *** ckyriakidou has quit IRC | 15:35 | |
| openstackgerrit | Saravanan KR proposed openstack/tripleo-common master: Workflow to select nodes matching a profile/role https://review.openstack.org/441054 | 15:35 |
| *** panda is now known as panda|bbl | 15:36 | |
| shardy | sshnaidm: if you want to control the public vip then yes, there's a parameter for the vip on each network | 15:36 |
| *** zoli|brb is now known as zoli | 15:36 | |
| *** jokke_ has joined #tripleo | 15:37 | |
| sshnaidm | shardy, yeah, I want to ensure that everybody will connect to specific IP | 15:37 |
| *** limao has quit IRC | 15:39 | |
| *** limao has joined #tripleo | 15:39 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-tripleo master: Add httpchk for http services https://review.openstack.org/439052 | 15:41 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-tripleoclient master: Avoid race condition when setting up websocket https://review.openstack.org/451624 | 15:41 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-tripleoclient master: Remove update abort https://review.openstack.org/449241 | 15:42 |
| sshnaidm | trown, panda|bbl, weshay maybe you know - how does oooq ensure that overcloud gets this ip? https://github.com/openstack/tripleo-quickstart/blob/57443434af78565cb7850886383f04b5e3ab4a08/roles/common/defaults/main.yml#L156-L160 | 15:42 |
| sshnaidm | I don't see it's used anywhere | 15:42 |
| openstackgerrit | Saravanan KR proposed openstack/tripleo-common master: Workflow to select nodes matching a profile/role https://review.openstack.org/441054 | 15:44 |
| chem | gfidente: he, don't forget to remove -1 on https://review.openstack.org/#/c/452828/ to move on with our plan :) | 15:46 |
| gfidente | chem ack | 15:46 |
| chem | gfidente: thanks | 15:46 |
| gfidente | I want to push the 'add back conditions' change | 15:46 |
| gfidente | and point to it when remove the -1 | 15:46 |
| chem | gfidente: sounds good | 15:47 |
| *** aufi has joined #tripleo | 15:49 | |
| EmilienM | sshnaidm: FYI we have go for https://review.openstack.org/#/c/451938/ | 15:49 |
| sshnaidm | EmilienM, ok, will review | 15:50 |
| EmilienM | sshnaidm: I haven't looked yet at the RDO CI failures | 15:50 |
| *** artom has quit IRC | 15:51 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates master: Add back Heat conditions in upgrade workflow https://review.openstack.org/453238 | 15:53 |
| EmilienM | adarazs: what is the magic command to retrigger rdo ci jobs again please? | 15:54 |
| *** artom has joined #tripleo | 15:57 | |
| EmilienM | weshay: ^ | 15:57 |
| *** atheurer has joined #tripleo | 15:58 | |
| openstackgerrit | Steven Hardy proposed openstack/python-tripleoclient master: Write user parameters environment to swift https://review.openstack.org/450264 | 15:58 |
| openstackgerrit | Steven Hardy proposed openstack/python-tripleoclient master: Write rhel registration parameters env to swift https://review.openstack.org/450708 | 15:58 |
| openstackgerrit | Steven Hardy proposed openstack/python-tripleoclient master: Write breakpoint cleanup env to swift https://review.openstack.org/450709 | 15:58 |
| openstackgerrit | Steven Hardy proposed openstack/python-tripleoclient master: Don't merge environments during plan creation https://review.openstack.org/448209 | 15:58 |
| weshay | EmilienM, there is recheck, full-ci-check | 15:59 |
| *** katkapilatova has left #tripleo | 15:59 | |
| *** lmiccini has quit IRC | 15:59 | |
| *** pblaho has joined #tripleo | 16:00 | |
| *** lucasagomes is now known as lucas-afk | 16:00 | |
| *** trown is now known as trown|lunch | 16:00 | |
| *** limao_ has joined #tripleo | 16:01 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates master: Ensure non-batched tasks start after batched tasks https://review.openstack.org/452789 | 16:01 |
| *** pcaruana has quit IRC | 16:01 | |
| *** limao has quit IRC | 16:04 | |
| *** ckyriakidou has joined #tripleo | 16:07 | |
| *** Goneri has quit IRC | 16:08 | |
| *** Goneri has joined #tripleo | 16:09 | |
| *** zoli is now known as zoli|gone | 16:12 | |
| *** karimb has quit IRC | 16:12 | |
| *** cylopez has quit IRC | 16:12 | |
| *** limao_ has quit IRC | 16:13 | |
| *** derekh has quit IRC | 16:13 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/tripleo-puppet-elements master: Install socat package for Ironic serial console support https://review.openstack.org/453247 | 16:19 |
| openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci master: DNM Other test for 451445 https://review.openstack.org/451978 | 16:21 |
| openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci master: Explicitly set REVIEW_RELEASE on master https://review.openstack.org/451445 | 16:21 |
| bogdando | folks, PTAL http://lists.openstack.org/pipermail/openstack-dev/2017-April/114956.html | 16:22 |
| *** jpich has quit IRC | 16:23 | |
| *** ipsecguy_ has joined #tripleo | 16:24 | |
| *** ipsecguy has quit IRC | 16:26 | |
| *** janki has quit IRC | 16:26 | |
| bnemec | Hmm, undercloud ugprade job is broken? http://logs.openstack.org/78/453078/2/check/gate-tripleo-ci-centos-7-undercloud-upgrades-nv/ac0887c/logs/var/log/undercloud_upgrade.txt.gz#_2017-04-04_12_38_04_000 | 16:27 |
| openstackgerrit | Michele Baldessari proposed openstack/puppet-tripleo master: WIP Make the cluster-check property configurable https://review.openstack.org/453250 | 16:30 |
| bnemec | Bug opened: https://bugs.launchpad.net/ironic/+bug/1679759 | 16:32 |
| openstack | Launchpad bug 1679759 in tripleo "Undercloud upgrade failing on ironic dbsync" [Critical,Triaged] | 16:32 |
| *** atheurer has quit IRC | 16:33 | |
| *** dmacpher is now known as dmacpher-afk | 16:33 | |
| *** morazi has quit IRC | 16:35 | |
| *** itzikb has quit IRC | 16:36 | |
| *** cwolferh has quit IRC | 16:36 | |
| mwhahaha | bnemec: i've seen that before but i thought it's been fixed as it was a problem in ironic or s omething | 16:45 |
| bnemec | mwhahaha: There was https://bugs.launchpad.net/tripleo/+bug/1678218 but that was fixed in inspector. Maybe ironic has the same issue? | 16:47 |
| openstack | Launchpad bug 1678218 in tripleo "Undercloud install failing with ironic-inspector dbsync error" [Critical,Fix released] - Assigned to John Trowbridge (trown) | 16:47 |
| mwhahaha | bnemec: yea that one | 16:47 |
| mwhahaha | bnemec: though seems odd that the repo configuration is the problem heh | 16:48 |
| mwhahaha | bnemec: seems like that was just hiding it until some promotion occured | 16:49 |
| *** ratailor has quit IRC | 16:50 | |
| EmilienM | sshnaidm: re: AFS mirror thing, seems like the patch I have is failing on ovb-ha-oooq pingtest http://logs.openstack.org/38/451938/8/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha-oooq-nv/8401f7d/logs/oooq/undercloud/home/jenkins/overcloud_validate.log.txt.gz | 16:51 |
| *** ffiore has quit IRC | 16:51 | |
| *** suuuper has quit IRC | 16:52 | |
| EmilienM | sshnaidm: I don't see the failure in the stack show :/ | 16:52 |
| *** gfidente is now known as gfidente|afk | 16:53 | |
| sshnaidm | EmilienM, it seems not related to patch.. | 16:57 |
| sshnaidm | EmilienM, I see some errors in nova and neutron there, but not sure it's the root cause | 16:57 |
| sshnaidm | http://logs.openstack.org/38/451938/8/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha-oooq-nv/8401f7d/logs/oooq/overcloud-novacompute-0/var/log/nova/nova-compute.log.txt.gz#_2017-04-04_16_17_11_475 | 16:57 |
| sshnaidm | http://logs.openstack.org/38/451938/8/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha-oooq-nv/8401f7d/logs/oooq/overcloud-novacompute-0/var/log/neutron/openvswitch-agent.log.txt.gz#_2017-04-04_16_21_34_451 | 16:57 |
| *** athomas has quit IRC | 16:59 | |
| openstackgerrit | Merged openstack/puppet-tripleo stable/ocata: Decouple ceilometer user create from API https://review.openstack.org/452541 | 16:59 |
| openstackgerrit | Merged openstack/tripleo-common master: Updated from global requirements https://review.openstack.org/451075 | 16:59 |
| openstackgerrit | Merged openstack/tripleo-quickstart-extras master: Added command to list tempest plugins. https://review.openstack.org/451747 | 16:59 |
| *** aufi has quit IRC | 17:02 | |
| *** dciabrin has quit IRC | 17:03 | |
| *** tesseract has quit IRC | 17:04 | |
| *** trown|lunch is now known as trown | 17:05 | |
| *** dtantsur is now known as dtantsur|afk | 17:07 | |
| openstackgerrit | Ana Krivokapic proposed openstack/tripleo-common master: Use Swift to store Plan environment https://review.openstack.org/452291 | 17:09 |
| openstackgerrit | Alexander Chuzhoy proposed openstack/tripleo-docs master: Accessing the ramdisk - selinux note https://review.openstack.org/452974 | 17:10 |
| *** paramite has quit IRC | 17:12 | |
| *** b00tcat has joined #tripleo | 17:14 | |
| b00tcat | Hi, I have some nodes in a tripleo deployment that I could delete - however in ironic the UUID is still showing up and the provisioning is still active | 17:14 |
| b00tcat | (although they're powered off) | 17:14 |
| *** akrivoka has quit IRC | 17:14 | |
| b00tcat | how can I clean the nodes to make them "deployable" again? | 17:15 |
| openstackgerrit | Alexander Chuzhoy proposed openstack/tripleo-docs master: Accessing the ramdisk - selinux note https://review.openstack.org/452974 | 17:16 |
| *** salmankhan has quit IRC | 17:16 | |
| *** atheurer has joined #tripleo | 17:18 | |
| *** bogdando has quit IRC | 17:19 | |
| openstackgerrit | Merged openstack/instack-undercloud master: Dont include wsgi profile when legacy flag is disabled https://review.openstack.org/453150 | 17:19 |
| openstackgerrit | Pradeep Kilambi proposed openstack/instack-undercloud stable/ocata: Dont include wsgi profile when legacy flag is disabled https://review.openstack.org/453270 | 17:20 |
| radez | thrash: I'm trying to deploy an ocata delpoyment and I keep getting this error when I first deploy | 17:20 |
| *** amoralej is now known as amoralej|off | 17:20 | |
| radez | Started Mistral Workflow tripleo.baremetal.v1.configure. Execution ID: f05de9fb-5fd9-4141-a281-853a7c858895 | 17:21 |
| radez | Failed to configure nodes: [u'Node boot could not be found. (HTTP 404)'] | 17:21 |
| radez | is that somethign you've seen before? | 17:21 |
| radez | it's a virtual baremetal with vbmc pm | 17:21 |
| thrash | radez: nope... is that on undercloud install? | 17:21 |
| radez | using profiles on the nodes and boot_option:local is set | 17:21 |
| radez | thrash: already done the undercloud install | 17:22 |
| radez | trying to execute an overcloud deploy | 17:22 |
| thrash | hmm | 17:22 |
| radez | I found this: http://git.openstack.org/cgit/openstack/python-tripleoclient/tree/tripleoclient/workflows/baremetal.py#n236 | 17:22 |
| radez | but I don't know what that means | 17:23 |
| radez | :) | 17:23 |
| radez | I did notice that the boot option was none | 17:23 |
| radez | https://paste.fedoraproject.org/paste/nuj8Vco8ivPxWQuWIybMDV5M1UNdIGYhyRLivL9gydE= | 17:23 |
| radez | on that mintral workflow but I don't know if that's the issue or not? | 17:23 |
| thrash | radez: that should work | 17:23 |
| radez | kk | 17:23 |
| thrash | radez: can you dig down into the actual action executions and see which failed? | 17:24 |
| thrash | radez: 'openstack action execution list' | 17:24 |
| radez | sure | 17:24 |
| * trown is learning stuff | 17:24 | |
| *** ckyriakidou has quit IRC | 17:24 | |
| radez | https://paste.fedoraproject.org/paste/gz63QcSmttO3vk0icrWrhV5M1UNdIGYhyRLivL9gydE= | 17:25 |
| radez | thrash: ^ | 17:25 |
| thrash | radez: hmmm | 17:26 |
| thrash | radez: is there even a plan? | 17:26 |
| thrash | radez: swift list | 17:26 |
| radez | (undercloud) [stack@undercloud ~]$ swift list | 17:26 |
| radez | overcloud | 17:26 |
| *** mgould is now known as mgould|afk | 17:26 | |
| radez | hm, I wonder if it's related to the command before the deploy... | 17:27 |
| radez | I'm running a script | 17:27 |
| radez | this guy: openstack overcloud node configure boot | 17:27 |
| radez | lemme try that again and see if it throws this | 17:27 |
| thrash | radez: that would be more likely | 17:28 |
| thrash | :D | 17:28 |
| radez | well, that worked... | 17:28 |
| radez | heh | 17:28 |
| thrash | hmm | 17:28 |
| * radez looks at the script | 17:28 | |
| radez | thrash: yea that's the huckleberry | 17:30 |
| radez | (undercloud) [stack@undercloud ~]$ openstack overcloud node configure boot | 17:30 |
| radez | Started Mistral Workflow tripleo.baremetal.v1.configure. Execution ID: 4966cd2c-8fc4-4a99-9506-e33fdf3c7a03 | 17:30 |
| radez | Waiting for messages on queue 'f47482f8-1687-4f69-a9ce-8a2a1f3e528e' with no timeout. | 17:30 |
| radez | Failed to configure nodes: [u'Node boot could not be found. (HTTP 404)'] | 17:30 |
| radez | I'm updateing a script from newton to ocata, do I not need to do this anymore? | 17:31 |
| radez | or do am I missing something? | 17:31 |
| thrash | radez: likely not, but let me double check | 17:31 |
| radez | ack, thx thrash | 17:31 |
| trown | radez: you dont need that on ocata | 17:31 |
| radez | YEA! mystery solved | 17:31 |
| thrash | there, see, I checked | 17:31 |
| thrash | :D | 17:31 |
| radez | lol,nicly done | 17:32 |
| skramaja | shardy: i was trying to check the output of heat stack-preview after applying the heat patch https://review.openstack.org/#/c/441047 and restarting all heat services | 17:32 |
| radez | thx much for your help guys thrash trown | 17:32 |
| skramaja | shardy: this is the output - http://chunk.io/f/14998fad81144974ac777efb478f44c6 | 17:32 |
| skramaja | skramaja: i dont find an evaluted services list. | 17:32 |
| skramaja | shardy: i dont find an evaluted services list. | 17:32 |
| bnemec | b00tcat: Try ironic node-set-provision-state [uuid] deleted | 17:33 |
| shardy | skramaja: you mean the roles_data outputs? | 17:33 |
| shardy | skramaja: atm stack-preview is a little limited, it can't evaluate stack outputs, because it doesn't differentiate static resolution of data from that which can only be evaluated at runtime | 17:34 |
| skramaja | shardy: yes, i did replate the parameters.get action with to preview instead of validate | 17:34 |
| dprince | vkmc: hi, of couple of suggestions here https://review.openstack.org/#/c/449292/2 | 17:34 |
| shardy | skramaja: I discussed this with zaneb recently and he has some patches which may be a step towards enabling a preview of the roles_data, but currently it's not possible unfortunately | 17:34 |
| *** morazi has joined #tripleo | 17:34 | |
| openstackgerrit | Merged openstack/tripleo-incubator master: H803 hacking have been deprecated https://review.openstack.org/413938 | 17:34 |
| dprince | vkmc: also, what do you think about creating an environments/docker-services/manilla* file to enable the 'docker' version of Manilla for these patches too? | 17:35 |
| skramaja | ok shardy. i will take a hidden parameter on the service to enabled the derived workflow per service for now. may be latter we can improvide it. | 17:35 |
| skramaja | *improvise | 17:36 |
| *** dsneddon is now known as dsneddon_afk | 17:36 | |
| openstackgerrit | James Slagle proposed openstack/tripleo-heat-templates master: Use the local collector to bootstrap deployed servers https://review.openstack.org/405032 | 17:37 |
| shardy | skramaja: Ok that sounds like a reasonable interim solution | 17:37 |
| *** deadnull has quit IRC | 17:38 | |
| skramaja | thanks shardy. as of now all open items has been closed for derived workflow. once the spec is updated with these details, i will ping you. thanks. | 17:39 |
| vkmc | dprince, thanks for the feedback on that patch set! | 17:40 |
| vkmc | dprince, with environment you mean a ci job? | 17:40 |
| dprince | vkmc: doesn't have to be in a CI job. But I would use the environments/docker-services/manilla file to enable the Manilla services... which are disable by default I think | 17:41 |
| vkmc | dprince, all right, I'll take a look | 17:41 |
| sshnaidm | EmilienM, commented on https://review.openstack.org/#/c/451938 | 17:42 |
| dprince | vkmc: like we do for these ones https://github.com/dprince/undercloud_containers/blob/master/doit.sh#L142 | 17:42 |
| vkmc | dprince, cool | 17:42 |
| EmilienM | sshnaidm: ack, thanks. I'll update it | 17:43 |
| *** dsariel has quit IRC | 17:44 | |
| *** chem is now known as chem_gone | 17:46 | |
| pabelanger | EmilienM: let me know once you land the proxy cache change from tripleo | 17:46 |
| openstackgerrit | Carlos Camacho proposed openstack/tripleo-heat-templates master: Merge pre|post puppet resources into pre|post config. https://review.openstack.org/452734 | 17:47 |
| dprince | flaper87: any thoughts on this https://review.openstack.org/#/c/451916/ | 17:47 |
| *** d0ugal has quit IRC | 17:48 | |
| dprince | flaper87: that is working well for me locally and I think perhaps should be our default | 17:48 |
| EmilienM | pabelanger: ok | 17:48 |
| *** pkovar has quit IRC | 17:49 | |
| *** d0ugal has joined #tripleo | 17:49 | |
| *** tosky has quit IRC | 17:52 | |
| *** skramaja has quit IRC | 17:53 | |
| flaper87 | dprince: I looked at it earlier today, I think it makes sense. Do we have any feedback from more knowledgeable folks on this area? Perhaps someone from kolla/docker/k8s/openshift ? larsks ? | 17:55 |
| *** prateek has joined #tripleo | 17:56 | |
| dprince | flaper87: would appreciate this. Given how things are I don't think we have a lot of options besides this. | 17:56 |
| dprince | flaper87: For upgrades, we can't just make a storage partition out of 'thin air'. And even on green field deployments we can't really do anything custom because Ironic doesn't allow you to custom partition the root disk yet | 17:57 |
| dprince | flaper87: so short of wasting a spare disk drive (normally used for Swift, Ceph, Cinder, etc.) I think we'd have to use some sort of overlay | 17:57 |
| dprince | flaper87: *but* I did here someone say there were different version of the overlay driver. So perhaps there is something I'm missing. | 17:58 |
| larsks | flaper87: docker since version 1.12 has the "overlay2" driver, which "is much more efficient than overlay in terms of inode utilization", but requires kernel > 4.0. | 18:02 |
| larsks | flaper87: I think given your constraints the overlay driver is probably your only real choice. | 18:02 |
| b00tcat | thanks bnemec, that did the trick | 18:03 |
| larsks | The only alternative would be to allow the operator to provide as input information about a block device to use for device mapper. | 18:03 |
| larsks | ...but there might not be one available. Although: | 18:03 |
| larsks | Using the overlay driver, you still really want a dedicated filesystem for /var/lib/docker. | 18:03 |
| dprince | larsks: Ironic doesn't let us create any partitions on the root drive though | 18:04 |
| *** dciabrin has joined #tripleo | 18:04 | |
| larsks | dprince: Fair enough. | 18:04 |
| dprince | larsks: ack on the overlay2 driver. I think Martin and Ian mentioned that one to me as well. | 18:05 |
| dprince | larsks: just so I'm clear, we can't make use of that for current Centos/RHEL version though right? | 18:06 |
| dprince | larsks: without updating the kernel... | 18:06 |
| larsks | dprince: that would be my best guess, although I would check to make sure the necessary support hasn't been backported or something... | 18:07 |
| dprince | larsks: ack, I will try to dig this up and ask as well | 18:07 |
| larsks | dprince: to your earlier comment, I still think you will need to make some sort of storage out of "thin air", because you really want /var/lib/docker separate. | 18:08 |
| larsks | (So that you don't consume all the disk space and/or inodes on your parent filesystem) | 18:09 |
| dprince | larsks: nothing to say we wouldn't do that today with RPMs on the root disks | 18:09 |
| dprince | larsks: space is space | 18:09 |
| dprince | larsks: Kolla upstream seems to be doing the 'out of thin air' trick here http://git.openstack.org/cgit/openstack/kolla/tree/tools/setup_RedHat.sh#n28 | 18:10 |
| dprince | larsks: thoughts on this? | 18:10 |
| dprince | larsks: they are creating a loopback device on an existing partition | 18:10 |
| dprince | larsks: then creating a custom filesystem on that! | 18:10 |
| openstackgerrit | Bob Fournier proposed openstack/os-net-config master: os_net_config should map nics that are down if nic is in mapping file https://review.openstack.org/453284 | 18:10 |
| larsks | dprince: I would step away quickly from anything involving loopback devices. | 18:10 |
| dprince | larsks: seems like adding the loopback device back into the mix is a layer we don't want | 18:11 |
| dprince | larsks: right, I agree | 18:11 |
| dprince | larsks: I would be happier if Ironic supported custom partitioning. And then we could optionally put it on a separate partition for new deployments | 18:12 |
| dprince | larsks: but without that I think it just needs to be all on one disk for now | 18:12 |
| larsks | dprince: this makes me think of the lemony snicket theme song. "Look away! Look away!" | 18:12 |
| openstackgerrit | Bob Fournier proposed openstack/os-net-config master: os_net_config should map nics that are down if nic is in mapping file https://review.openstack.org/453284 | 18:13 |
| dprince | larsks: yeah, well definatly look away now... because without this we default to loopback :/ | 18:13 |
| openstackgerrit | Carlos Camacho proposed openstack/tripleo-heat-templates master: Merge pre|post puppet resources into pre|post config. https://review.openstack.org/452734 | 18:13 |
| *** dtrainor has quit IRC | 18:18 | |
| *** dtrainor has joined #tripleo | 18:22 | |
| *** florianf has quit IRC | 18:23 | |
| *** dparkes has joined #tripleo | 18:24 | |
| *** yatinkarel has quit IRC | 18:24 | |
| *** salmankhan has joined #tripleo | 18:27 | |
| *** dprince has quit IRC | 18:27 | |
| *** salmankhan has quit IRC | 18:31 | |
| *** snecklifter has quit IRC | 18:32 | |
| *** shardy has quit IRC | 18:32 | |
| *** dparkes has quit IRC | 18:35 | |
| *** yatinkarel has joined #tripleo | 18:37 | |
| *** artom has quit IRC | 18:42 | |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart master: include the openstack release in the image cache path https://review.openstack.org/453295 | 18:42 |
| *** artom has joined #tripleo | 18:42 | |
| *** dsneddon_afk has quit IRC | 18:44 | |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart master: add note to not run quickstart from the ssh_user (stack) https://review.openstack.org/453297 | 18:45 |
| *** snecklifter has joined #tripleo | 18:46 | |
| openstackgerrit | James Slagle proposed openstack/python-tripleoclient master: Add --skip-deploy-identifier https://review.openstack.org/452223 | 18:47 |
| *** salmankhan has joined #tripleo | 18:49 | |
| *** artom has quit IRC | 18:50 | |
| *** artom has joined #tripleo | 18:50 | |
| openstackgerrit | Merged openstack/instack-undercloud master: Set Zaqar roles https://review.openstack.org/452098 | 18:55 |
| *** dprince has joined #tripleo | 18:57 | |
| *** mich_chap has quit IRC | 18:58 | |
| *** dtrainor has quit IRC | 19:00 | |
| slagle | undercloud upgrades job appears broken | 19:00 |
| slagle | blood red trail on the ci status page | 19:01 |
| openstackgerrit | Merged openstack/puppet-tripleo master: Remove cluster_enabled setting for etcd https://review.openstack.org/446564 | 19:02 |
| *** dtrainor has joined #tripleo | 19:02 | |
| EmilienM | slagle: yeah, bnemec said it before here, https://bugs.launchpad.net/ironic/ | 19:03 |
| slagle | 2017-04-04 16:58:16.000 | 2017-04-04 16:58:16,685 INFO: [1;31mError: /Stage[main]/Ironic::Db::Sync/Exec[ironic-dbsync]: Failed to call refresh: ironic-dbsync --config-file /etc/ironic/ironic.conf returned 1 instead of one of [0][0m | 19:03 |
| EmilienM | https://bugs.launchpad.net/ironic/+bug/1679759 | 19:03 |
| openstack | Launchpad bug 1679759 in tripleo "Undercloud upgrade failing on ironic dbsync" [Critical,Triaged] | 19:03 |
| slagle | Oh | 19:03 |
| EmilienM | oh but I thought it was fixed | 19:05 |
| *** salmankhan has quit IRC | 19:05 | |
| slagle | the bug is unassigned | 19:05 |
| EmilienM | slagle: the trace, I saw it last week | 19:05 |
| EmilienM | and someone from ironic told me it was fixed and now it appears again | 19:05 |
| slagle | EmilienM: that was for ironic-inspector! | 19:06 |
| EmilienM | bnemec: did you show it to ironic folks? | 19:06 |
| bnemec | EmilienM: I just added ironic to the bug | 19:06 |
| bnemec | It does look similar to the inspector bug we had recently too. | 19:06 |
| bnemec | mwhahaha suggested that maybe we promoted a broken Ironic since we don't promote based on undercloud upgrades. | 19:07 |
| EmilienM | this is not an ironic bug | 19:12 |
| *** owalsh_ has joined #tripleo | 19:12 | |
| EmilienM | the ironic bug has been fixed in https://github.com/openstack/ironic/commit/d31d984871a03c3e7cc4be85a0f2fb12bda5dc7e | 19:12 |
| EmilienM | and our undercloud deployed an old version of ironic | 19:12 |
| EmilienM | see http://logs.openstack.org/78/453078/2/check/gate-tripleo-ci-centos-7-undercloud-upgrades-nv/ac0887c/logs/rpm-qa.txt.gz | 19:13 |
| *** salmankhan has joined #tripleo | 19:13 | |
| EmilienM | let's look at yum logs, I'm pretty sure it's related to our tooling | 19:13 |
| slagle | we upgrade to the promoted repo | 19:13 |
| slagle | (i thought) | 19:13 |
| slagle | so it may not be latest ironic | 19:13 |
| EmilienM | well, it should, we have promotions every day | 19:14 |
| EmilienM | let's look https://trunk.rdoproject.org/centos7-master/current-tripleo/ | 19:14 |
| EmilienM | https://trunk.rdoproject.org/centos7-master/current-tripleo/openstack-ironic-api-7.0.1-0.20170330223755.fe47326.el7.centos.noarch.rpm | 19:15 |
| *** owalsh has quit IRC | 19:15 | |
| EmilienM | https://github.com/openstack/ironic/commit/fe47326 | 19:15 |
| *** dsneddon has joined #tripleo | 19:15 | |
| EmilienM | its a commit from 5 days ago | 19:15 |
| EmilienM | and our CI deploys: | 19:16 |
| EmilienM | Apr 4 12:13:07 centos-7-osic-cloud1-s3500-8260585 yum[21404]: Installed: 1:openstack-ironic-api-7.0.1-0.20170404112528.7f1639e.el7.centos.noarch | 19:16 |
| EmilienM | which is older | 19:16 |
| EmilienM | that's the repo used to upgrade the undercloud: https://trunk.rdoproject.org/centos7/07/5a/075aa4737dd47772f66afec62449d746043c9fe5_fcd9dba6/ | 19:17 |
| EmilienM | and upgrade job passed this morning: https://review.openstack.org/#/c/453126/ | 19:18 |
| EmilienM | with openstack-ironic-api-7.0.1-0.20170330223755.fe47326.el7.centos.noarch :-/ | 19:19 |
| *** paramite has joined #tripleo | 19:19 | |
| *** eck` is now known as eck`gone | 19:19 | |
| *** morazi has quit IRC | 19:20 | |
| *** ckyriakidou has joined #tripleo | 19:23 | |
| EmilienM | this patch is interesting: https://review.openstack.org/#/c/453028/ | 19:23 |
| EmilienM | it passed at 7.39 am and failed at 11am | 19:24 |
| EmilienM | here's a packaging diff: https://www.diffchecker.com/JV5yZt12 | 19:25 |
| *** fzdarsky is now known as fzdarsky|afk | 19:25 | |
| EmilienM | ironic was upgraded from fe47326 to 7f1639e | 19:25 |
| slagle | the job did the right thing afaict, it installed ocata and upgraded to master | 19:26 |
| *** owalsh_ is now known as owalsh | 19:26 | |
| *** dprince has quit IRC | 19:26 | |
| slagle | BZZZT | 19:27 |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart master: fix virthost yum repo setup https://review.openstack.org/452008 | 19:27 |
| EmilienM | slagle: we need to take in consideration that packages upgrades are broken until pike-1 is out | 19:28 |
| EmilienM | because nvr thing | 19:28 |
| slagle | yea | 19:28 |
| slagle | ok, forget it then | 19:28 |
| EmilienM | so i think even if the repo is deployed | 19:28 |
| EmilienM | the package won't get upgrades | 19:28 |
| EmilienM | and if it, please explain me how :D | 19:28 |
| slagle | versions that don't go backwards would be great | 19:29 |
| *** dprince has joined #tripleo | 19:29 | |
| EmilienM | slagle: it would have been great if upstream could tag an alpha or something | 19:30 |
| EmilienM | it would have been sooooo helpful | 19:30 |
| EmilienM | but I'm done with that | 19:30 |
| EmilienM | bnemec: can you update the bug report please? i think it's invalid | 19:30 |
| slagle | seems this could be easily solved by using a release tag prefix | 19:30 |
| *** bfournie has quit IRC | 19:30 | |
| openstackgerrit | Dan Prince proposed openstack/tripleo-common master: Remove Percona-Server-shared-56 from openstack-base image https://review.openstack.org/453164 | 19:30 |
| EmilienM | slagle: easy if all projects would have done it | 19:31 |
| slagle | 0-newton, 1-ocata, 2-pike, etc | 19:31 |
| EmilienM | slagle: except puppet modules, nobody did it | 19:31 |
| slagle | no, in dlrn packaging | 19:31 |
| EmilienM | ah | 19:31 |
| slagle | instead of just using the date as the release | 19:31 |
| EmilienM | let me poke apevec | 19:31 |
| EmilienM | he was working on a workaround like this in rdo | 19:31 |
| EmilienM | let's switch on #rdo | 19:31 |
| *** prateek has quit IRC | 19:36 | |
| *** matbu|bbl is now known as matbu | 19:38 | |
| EmilienM | sshnaidm: hey | 19:39 |
| EmilienM | sshnaidm: looking at the second comment on https://review.openstack.org/#/c/451938/8/config/release/tripleo-ci/consistent-master.yml | 19:39 |
| EmilienM | what url do you prefer? | 19:39 |
| EmilienM | which one from https://buildlogs.centos.org/centos/7/cloud/x86_64/ ? | 19:39 |
| *** cdearborn has joined #tripleo | 19:40 | |
| EmilienM | sshnaidm: because I took the same one as before: https://trunk.rdoproject.org/centos7-master/delorean-deps.repo | 19:40 |
| paramite | EmilienM, hey Emilien, so sensu-client is still failing on scenario001 because wrong opstools repo, where can I change the repo? | 19:42 |
| EmilienM | paramite: what is the wrong repo that we use? and what is the correct one? | 19:43 |
| paramite | EmilienM, http://logs.openstack.org/31/433931/5/check/gate-tripleo-ci-centos-7-scenario001-multinode-oooq/7473e10/logs/subnode-2/etc/yum.repos.d/centos-opstools.repo.txt.gz | 19:43 |
| paramite | ^ this is wrong | 19:43 |
| paramite | EmilienM, 'yum install -y centos-release-opstools' is correct one | 19:44 |
| *** kbyrne has quit IRC | 19:46 | |
| EmilienM | paramite: we're switching it to : https://review.openstack.org/#/c/451938/8/config/release/tripleo-ci/master.yml | 19:47 |
| EmilienM | paramite: http://mirror.regionone.osic-cloud1.openstack.org/centos/7/opstools/x86_64/ | 19:48 |
| EmilienM | is it good ^ ? | 19:48 |
| paramite | EmilienM, that is perfectly fine | 19:48 |
| paramite | EmilienM, thanks! | 19:49 |
| EmilienM | good | 19:49 |
| EmilienM | paramite: give me some time and we fix it this week | 19:49 |
| EmilienM | paramite: this patch will fix it | 19:49 |
| paramite | EmilienM, great, thanks | 19:49 |
| EmilienM | yw | 19:50 |
| *** kbyrne has joined #tripleo | 19:50 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-quickstart master: Use AFS mirrors for optstool, delorean and qemu https://review.openstack.org/451938 | 19:51 |
| EmilienM | sshnaidm: when you back, please see my comment ^ thanks | 19:52 |
| *** jprovazn has quit IRC | 19:53 | |
| *** [1]cdearborn has quit IRC | 19:53 | |
| *** eck`gone is now known as eck` | 19:53 | |
| openstackgerrit | Martin Mágr proposed openstack/tripleo-heat-templates master: Add monitoring agents deployment to CI https://review.openstack.org/433931 | 19:54 |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci master: yum distro-sync in undercloud_upgrade https://review.openstack.org/453317 | 19:54 |
| EmilienM | paramite: ^ depends-on should pass. Let me know the results please | 19:55 |
| paramite | EmilienM, will do | 19:55 |
| openstackgerrit | Merged openstack/instack-undercloud master: Remove instack-virt-setup https://review.openstack.org/451914 | 19:59 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Add flag to disable EPEL https://review.openstack.org/439926 | 20:00 |
| *** mich_chap has joined #tripleo | 20:04 | |
| b00tcat | hi | 20:05 |
| b00tcat | I need the NeutronPassword parameter into a custom service YAML file of mine | 20:05 |
| b00tcat | should I merge my YAML with the neutrno-base one? | 20:05 |
| *** panda|bbl is now known as panda|off | 20:25 | |
| pradk | can we merge this backport please https://review.openstack.org/#/c/453270/ | 20:27 |
| bnemec | pradk: No ovb results yet. | 20:32 |
| *** chem_gone has quit IRC | 20:36 | |
| openstackgerrit | John Trowbridge proposed openstack/tripleo-quickstart-extras master: Allow depending on changes we dont actually build https://review.openstack.org/453326 | 20:42 |
| openstackgerrit | Steve Baker proposed openstack/instack-undercloud master: Allow install user to run docker commands https://review.openstack.org/390549 | 20:42 |
| *** jkilpatr has quit IRC | 20:45 | |
| *** stevebaker has joined #tripleo | 20:47 | |
| stevebaker | morning | 20:47 |
| *** stevebaker is now known as stevebake | 20:47 | |
| slagle | stevebake: morning, i updated https://review.openstack.org/#/c/405032/ hope you don't mind | 20:48 |
| *** asalkeld has joined #tripleo | 20:48 | |
| stevebake | slagle: cool, I never got that working in the gate anyway | 20:49 |
| slagle | heh, yea i just noticed it's still failing ci | 20:50 |
| *** ansmith has quit IRC | 20:50 | |
| asalkeld | o/ | 20:51 |
| stevebake | asalkeld: ohai! | 20:51 |
| asalkeld | :-) | 20:51 |
| *** gfidente|afk is now known as gfidente | 20:55 | |
| *** trown is now known as trown|outtypewww | 20:55 | |
| *** oneswig has joined #tripleo | 20:55 | |
| *** rcernin has quit IRC | 20:55 | |
| slagle | stevebake: i think it's b/c the occ restart was removed, and there's nothign to do that on multinode jobs | 20:59 |
| slagle | i can add it to the bootstrap scripts | 20:59 |
| *** dprince has quit IRC | 21:01 | |
| *** salmankhan has quit IRC | 21:02 | |
| openstackgerrit | James Slagle proposed openstack/tripleo-heat-templates master: Use the local collector to bootstrap deployed servers https://review.openstack.org/405032 | 21:02 |
| *** jlinkes has quit IRC | 21:03 | |
| *** asbishop has joined #tripleo | 21:06 | |
| *** salmankhan has joined #tripleo | 21:08 | |
| *** abishop_ has quit IRC | 21:08 | |
| *** lblanchard has quit IRC | 21:09 | |
| *** Goneri has quit IRC | 21:10 | |
| *** asbishop has quit IRC | 21:11 | |
| *** artom has quit IRC | 21:13 | |
| *** artom has joined #tripleo | 21:14 | |
| *** snecklifter has quit IRC | 21:20 | |
| *** gfidente has quit IRC | 21:22 | |
| *** jkilpatr has joined #tripleo | 21:23 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/instack-undercloud stable/ocata: TEST: Set a dependency on collector https://review.openstack.org/453181 | 21:24 |
| *** dparkes has joined #tripleo | 21:25 | |
| *** atheurer has quit IRC | 21:31 | |
| *** abishop has joined #tripleo | 21:32 | |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart-extras master: WIP rhn repo support https://review.openstack.org/453345 | 21:35 |
| *** salmankhan has quit IRC | 21:35 | |
| *** cwolferh has joined #tripleo | 21:43 | |
| *** dbecker has quit IRC | 21:44 | |
| openstackgerrit | Merged openstack/instack master: Updated from global requirements https://review.openstack.org/431951 | 21:47 |
| *** morazi has joined #tripleo | 21:47 | |
| openstackgerrit | Merged openstack/puppet-tripleo master: Fixes missing neutron base in sriov https://review.openstack.org/452786 | 21:50 |
| *** artom has quit IRC | 21:53 | |
| *** artom has joined #tripleo | 21:54 | |
| openstackgerrit | John Fulton proposed openstack/tripleo-specs master: Enable TripleO to Deploy Ceph via Ceph Ansible https://review.openstack.org/387631 | 21:54 |
| *** dtrainor has quit IRC | 21:59 | |
| *** dtrainor has joined #tripleo | 21:59 | |
| openstackgerrit | Merged openstack/puppet-tripleo master: Use correct manage_firewall hieradata https://review.openstack.org/452777 | 22:02 |
| *** oneswig has quit IRC | 22:03 | |
| openstackgerrit | Merged openstack/puppet-tripleo master: Refactor enabled languages from an array to a hash https://review.openstack.org/442683 | 22:09 |
| *** noslzzp has quit IRC | 22:14 | |
| *** morazi has quit IRC | 22:18 | |
| *** flepied has quit IRC | 22:18 | |
| *** jcoufal has quit IRC | 22:20 | |
| *** ffiore has joined #tripleo | 22:22 | |
| *** asalkeld has quit IRC | 22:24 | |
| *** Goneri has joined #tripleo | 22:25 | |
| *** nyechiel has quit IRC | 22:26 | |
| *** thrash is now known as thrash|g0ne | 22:27 | |
| *** dbecker has joined #tripleo | 22:27 | |
| *** b00tcat has quit IRC | 22:33 | |
| *** dbecker has quit IRC | 22:41 | |
| *** ckyriakidou has quit IRC | 22:46 | |
| stevebake | EmilienM: hey you about? I have tripleo-common packaging questions | 22:55 |
| *** dparkes has quit IRC | 22:55 | |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart-extras master: WIP rhn repo support https://review.openstack.org/453345 | 22:55 |
| *** dmarlin has quit IRC | 22:56 | |
| *** dougbtv|laptop has joined #tripleo | 22:57 | |
| *** Goneri has quit IRC | 22:59 | |
| *** ffiore_ has joined #tripleo | 23:11 | |
| *** ffiore has quit IRC | 23:12 | |
| *** dhill_ has quit IRC | 23:16 | |
| *** ffiore_ has quit IRC | 23:17 | |
| *** dhill_ has joined #tripleo | 23:17 | |
| openstackgerrit | Merged openstack/diskimage-builder master: Add flag to disable EPEL https://review.openstack.org/439926 | 23:26 |
| openstackgerrit | Merged openstack/puppet-tripleo master: Configure migration SSH tunnel https://review.openstack.org/450826 | 23:29 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Remove kolla_config copy from services https://review.openstack.org/451366 | 23:29 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci master: WIP: Conntainers with multi-nic and net-iso https://review.openstack.org/422448 | 23:31 |
| *** bfournie has joined #tripleo | 23:36 | |
| *** pmannidi has joined #tripleo | 23:38 | |
| *** pmannidi is now known as pmannidi|brb | 23:38 | |
| *** pmannidi|brb has quit IRC | 23:38 | |
| *** pmannidi has joined #tripleo | 23:39 | |
| openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart-extras master: Use web repo with installed packages for image build https://review.openstack.org/451936 | 23:44 |
| openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-common master: DONT REVIEW: used for testing only https://review.openstack.org/447276 | 23:46 |
| *** dhill_ has quit IRC | 23:47 | |
| *** dhill_ has joined #tripleo | 23:48 | |
| *** masterjcool has quit IRC | 23:49 | |
| *** rook-tower is now known as rook | 23:49 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!