Friday, 2017-02-17

openstackgerritAttila Darazs proposed openstack/tripleo-quickstart-extras master: GATE CHECK  https://review.openstack.org/43027700:00
EmilienMbnemec: can you review https://review.openstack.org/#/c/434520 when you have time?00:03
pradkEmilienM, is there a better way to disable services via puppet? see comments in https://review.openstack.org/#/c/433824/00:04
openstackgerritEmilien Macchi proposed openstack/python-tripleoclient stable/ocata: stable/ocata CI run - do not merge  https://review.openstack.org/42623100:04
EmilienMpradk: let me look00:04
openstackgerritFlavio Percoco proposed openstack/tripleo-docs master: WIP: Containerized TripleO Development tips/tricks  https://review.openstack.org/43020100:05
EmilienMpradk: ok let me think00:05
pradki thought setting enabled=false in service classes will do it.. but it comes with a caveat of including the classes always00:06
EmilienMpradk: yeah00:07
EmilienMslagle is right00:07
EmilienMthough you want a way to stop services if they used to run, right?00:07
pradkright00:08
EmilienMlet me think00:08
pradknot just stop00:08
pradkdisable so they dont comeback on reboot00:08
pradkwhich is the original issue i'm trying to fix00:08
EmilienMpradk: can I push?00:09
pradksure00:09
EmilienMgimme 5 min00:09
EmilienMI need to look on stackoverflow00:10
*** pmannidi has joined #tripleo00:10
*** mhenkel__ has joined #tripleo00:14
*** mhenkel_ has quit IRC00:14
EmilienMpradk: give me 10 min :D00:16
*** lblanchard has quit IRC00:17
pradknp, i'll go grab some dinner and back in a bit00:17
EmilienMpradk: do you want to purge DB?00:21
EmilienMwhen removing00:21
EmilienMI guess yes00:21
*** dhill_ has quit IRC00:22
*** dhill_ has joined #tripleo00:22
pradkEmilienM, hmm we probably dont want to purge db is if has data and gather some metrics .. that would be up to the operator i would think00:26
EmilienMpradk: ok00:27
openstackgerritEmilien Macchi proposed openstack/instack-undercloud master: Allow to teardown Telemetry services  https://review.openstack.org/43515900:27
EmilienMpradk: this is a first shot ^00:27
openstackgerritEmilien Macchi proposed openstack/instack-undercloud master: Allow to teardown Telemetry services  https://review.openstack.org/43515900:28
rookis there work to support software raid w/ OOO ?00:32
*** jbryce has joined #tripleo00:38
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo master: Add virtual_packages support to norpm provider  https://review.openstack.org/43501100:41
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo master: Add virtual_packages support to norpm provider  https://review.openstack.org/43501100:49
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo master: Add virtual_packages support to norpm provider  https://review.openstack.org/43501100:49
rooknm, found what I was looking for :/00:50
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo stable/ocata: Add virtual_packages support to norpm provider  https://review.openstack.org/43518100:50
*** limao has joined #tripleo01:00
*** tbonds has quit IRC01:03
*** gatuus_ has quit IRC01:04
*** gatuus has quit IRC01:04
*** pmannidi has quit IRC01:08
*** rajinir_ has quit IRC01:10
*** lblanchard has joined #tripleo01:15
*** eck`gone is now known as eck`01:15
openstackgerritMerged openstack/tripleo-common master: Use the builtin Mistral engine command 'fail' when failing workflows  https://review.openstack.org/42504701:21
openstackgerritMerged openstack/tripleo-heat-templates master: Automatically backup and restore Swift rings from the undercloud  https://review.openstack.org/41446001:21
*** salmankhan has quit IRC01:21
openstackgerritMerged openstack-infra/tripleo-ci master: Deploy novajoin in the TLS-everywhere job  https://review.openstack.org/42748101:22
openstackgerritMerged openstack/tripleo-heat-templates master: Adds 'trunk' to ODL env service plugins  https://review.openstack.org/43329501:22
openstackgerritMerged openstack/tripleo-heat-templates master: Add Newton to Ocata UpgradeInitCommonCommand  https://review.openstack.org/42471501:22
openstackgerritMerged openstack-infra/tripleo-ci master: Fix quickstart issues  https://review.openstack.org/42545901:22
openstackgerritMerged openstack/tripleo-heat-templates stable/ocata: Update .gitreview for stable/ocata  https://review.openstack.org/43487801:22
openstackgerritMerged openstack/tripleo-heat-templates stable/ocata: CI fix broken references to multinode-core.yaml  https://review.openstack.org/43490801:23
openstackgerritMerged openstack/tripleo-heat-templates master: Apply post-upgrade step to not run puppet in post upgrade  https://review.openstack.org/43139801:23
*** sdake_ has joined #tripleo01:28
*** dsneddon_ has joined #tripleo01:30
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates stable/ocata: Apply post-upgrade step to not run puppet in post upgrade  https://review.openstack.org/43518901:30
*** sdake has quit IRC01:30
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates stable/ocata: Add Newton to Ocata UpgradeInitCommonCommand  https://review.openstack.org/43519001:30
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates stable/ocata: Automatically backup and restore Swift rings from the undercloud  https://review.openstack.org/43519101:30
*** dsneddon_ has quit IRC01:35
*** jcoufal has quit IRC01:39
*** gbarros_ has quit IRC01:40
*** mhenkel__ has quit IRC01:43
*** mhenkel_ has joined #tripleo01:44
*** mhenkel_ has quit IRC01:49
*** thrash is now known as thrash|g0ne01:49
openstackgerritJuan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: DO NOT MERGE - testing TLS everywhere job  https://review.openstack.org/42686301:50
*** Goneri has quit IRC02:02
*** lblanchard has quit IRC02:09
*** fzdarsky_ has joined #tripleo02:12
*** fzdarsky|afk has quit IRC02:16
*** mhenkel_ has joined #tripleo02:19
*** abehl has quit IRC02:20
*** tzumainn has quit IRC02:20
*** mhenkel_ has quit IRC02:23
*** michapma_alt has quit IRC02:28
*** chem has quit IRC02:58
openstackgerritPradeep Kilambi proposed openstack/instack-undercloud master: Allow to teardown Telemetry services  https://review.openstack.org/43515903:00
openstackgerritMerged openstack/tripleo-heat-templates master: docker: new hybrid deployment architecture and configuration  https://review.openstack.org/41642103:04
openstackgerritMerged openstack/tripleo-ui stable/ocata: Update .gitreview for stable/ocata  https://review.openstack.org/43498903:04
*** leanderthal|afk has quit IRC03:06
*** mhenkel_ has joined #tripleo03:07
*** mhenkel_ has quit IRC03:12
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: multinode/upgrade: add nova services to environment  https://review.openstack.org/43434803:18
*** leanderthal|afk has joined #tripleo03:19
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci master: TEST: DONT RECHECK or REVIEW: periodic jobs  https://review.openstack.org/35921503:20
*** dmacpher has joined #tripleo03:22
*** pradk has quit IRC03:32
*** deepbook5broo has joined #tripleo03:34
*** deepbook5broo has left #tripleo03:34
*** sdake_ has quit IRC03:36
*** sdake has joined #tripleo03:39
*** dsneddon_ has joined #tripleo03:39
*** nmathew has joined #tripleo03:49
*** david-lyle has quit IRC03:49
nmathewgm03:50
*** david-lyle has joined #tripleo03:50
*** sdake_ has joined #tripleo03:53
*** psahoo has joined #tripleo03:54
*** sdake has quit IRC03:55
*** sudipto_ has joined #tripleo03:57
*** sudipto has joined #tripleo03:57
*** dsneddon_ has quit IRC04:00
*** links has joined #tripleo04:03
*** mhenkel_ has joined #tripleo04:04
openstackgerritMichael Chapman proposed openstack/tripleo-quickstart master: Add support for virtual port types  https://review.openstack.org/40734704:06
*** pmannidi has joined #tripleo04:08
*** mhenkel_ has quit IRC04:09
*** sudipto has quit IRC04:12
*** sudipto_ has quit IRC04:12
*** karthiks has joined #tripleo04:31
*** xuao has joined #tripleo04:35
*** sudipto has joined #tripleo04:39
*** sudipto_ has joined #tripleo04:39
*** mdnadeem has joined #tripleo04:40
*** sudipto has quit IRC04:43
*** sudipto_ has quit IRC04:43
openstackgerritMichael Chapman proposed openstack/tripleo-quickstart master: Allow permissive access to non_root_user files  https://review.openstack.org/41819804:45
*** sudipto_ has joined #tripleo04:51
*** sudipto has joined #tripleo04:51
*** diana_clarke has quit IRC04:54
*** sdake_ has quit IRC04:54
*** sasha2 has quit IRC04:57
*** skramaja has joined #tripleo04:58
*** mhenkel_ has joined #tripleo05:02
*** mhenkel_ has quit IRC05:06
*** rcernin has joined #tripleo05:08
*** gkadam has joined #tripleo05:10
*** gkadam has quit IRC05:10
*** prateek has joined #tripleo05:13
*** rlandy|afk is now known as rlandy05:19
*** rlandy has quit IRC05:19
*** xuao has quit IRC05:36
*** udesale has joined #tripleo05:39
*** mhenkel_ has joined #tripleo05:40
*** abehl has joined #tripleo05:43
*** mhenkel_ has quit IRC05:45
*** dsneddon_ has joined #tripleo05:48
*** dsneddon_ has quit IRC05:53
*** padkrish has joined #tripleo05:57
*** yamahata has joined #tripleo05:59
nmathewhi all06:05
*** mhenkel_ has joined #tripleo06:08
nmathewhow can I do something like "ip route add 172.16.0.0/24 dev br-prv" using heat templates06:08
*** mhenkel_ has quit IRC06:10
*** mhenkel_ has joined #tripleo06:10
skramajanmathew: you can add routes to the network config templates.06:11
*** stendulker has joined #tripleo06:12
openstackgerritMerged openstack/tripleo-common stable/ocata: Update .gitreview for stable/ocata  https://review.openstack.org/43487606:14
openstackgerritMerged openstack/tripleo-image-elements stable/ocata: Update .gitreview for stable/ocata  https://review.openstack.org/43488006:14
*** mhenkel_ has quit IRC06:15
*** jcoufal has joined #tripleo06:15
nmathewthanks, skramaja I will look into os-net-config06:15
*** abehl has quit IRC06:18
*** rcernin has quit IRC06:21
*** saneax-_-|AFK is now known as saneax06:22
*** jaganathan has joined #tripleo06:29
*** padkrish has quit IRC06:33
*** nyechiel has joined #tripleo06:35
*** ealcaniz has joined #tripleo06:38
*** jerrygb has joined #tripleo06:41
*** gbarros has joined #tripleo06:43
*** jcoufal has quit IRC06:45
*** jerrygb has quit IRC06:46
*** flepied has joined #tripleo06:48
*** jbadiapa has quit IRC06:51
openstackgerritMerged openstack/instack stable/ocata: Update .gitreview for stable/ocata  https://review.openstack.org/43487306:53
*** jbadiapa has joined #tripleo06:54
*** mhenkel_ has joined #tripleo06:55
*** dmacpher has quit IRC06:55
*** Vijayendra has quit IRC07:00
*** jerrygb has joined #tripleo07:00
*** mhenkel_ has quit IRC07:01
*** rcernin has joined #tripleo07:01
*** iranzo has joined #tripleo07:02
*** iranzo has joined #tripleo07:02
*** jerrygb has quit IRC07:05
*** mhenkel_ has joined #tripleo07:07
*** tesseract has joined #tripleo07:11
*** abehl has joined #tripleo07:13
*** Vijayendra has joined #tripleo07:13
*** leanderthal|afk is now known as leanderthal07:14
*** mhenkel_ has quit IRC07:14
openstackgerritMerged openstack/tripleo-puppet-elements stable/ocata: Update .gitreview for stable/ocata  https://review.openstack.org/43488207:17
*** jprovazn has joined #tripleo07:17
*** fragatina has quit IRC07:19
*** akrivoka has joined #tripleo07:22
*** akrivoka has joined #tripleo07:23
*** ealcaniz has quit IRC07:25
*** ealcaniz has joined #tripleo07:25
*** yprokule has joined #tripleo07:26
*** dsariel has quit IRC07:26
*** abehl has quit IRC07:31
*** aufi has joined #tripleo07:32
*** fragatina has joined #tripleo07:35
*** fragatina has quit IRC07:37
*** fragatina has joined #tripleo07:38
*** fragatina has quit IRC07:38
*** dsariel has joined #tripleo07:39
*** fragatina has joined #tripleo07:39
*** amoralej|off is now known as amoralej07:46
openstackgerritChristian Schwede proposed openstack/tripleo-common stable/newton: Use Mistral to create Swift temporary URLs  https://review.openstack.org/43528407:47
openstackgerritRico Lin proposed openstack/os-collect-config master: Allow setting minimum polling interval for collect  https://review.openstack.org/42082307:47
openstackgerritChristian Schwede proposed openstack/tripleo-heat-templates stable/newton: Automatically backup and restore Swift rings from the undercloud  https://review.openstack.org/43528507:47
*** jd__ has quit IRC07:47
*** gbarros has quit IRC07:48
*** saibarspeis has joined #tripleo07:57
*** shardy_afk is now known as shardy07:57
*** migi_ is now known as migi07:58
openstackgerritMerged openstack/instack-undercloud stable/ocata: Set project name explicitly to service for panko  https://review.openstack.org/43514008:01
*** rasca has joined #tripleo08:05
*** nyechiel has quit IRC08:05
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates master: Add manual ceph-osd upgrade if operator prefers this  https://review.openstack.org/43436708:07
*** xuao has joined #tripleo08:16
*** tremble has joined #tripleo08:16
*** jlinkes has joined #tripleo08:20
shardyHi all, can I please get another reviewer to look at https://review.openstack.org/#/c/426375/ (tripleo-ci patch)08:22
shardyit's passing the upgrade job so should be good to merge I think08:22
*** pcaruana has joined #tripleo08:22
mariosshardy: done08:27
mariosshardy: I checked the env file really is in tht/ci/environments, in case you were lying  but you weren't :)08:28
mariosoh is EmilienM's patch /me coffee refill08:28
matbushardy: this review is weird08:30
* matbu tests something08:30
*** jpena|away is now known as jpena08:32
*** jbadiapa has quit IRC08:34
matbushardy: i comment the review08:34
mariosmatbu: o/ should i -A ?08:35
mariosmatbu: i just did, not sure it is fast enough :) lets see08:36
*** gfidente has joined #tripleo08:37
*** florianf has joined #tripleo08:38
*** b00tcat has joined #tripleo08:40
*** b00tcat has joined #tripleo08:40
*** florianf has quit IRC08:42
*** jbadiapa has joined #tripleo08:44
*** ccamacho has joined #tripleo08:44
matbumarios: it's not a big pb :) better to set the variable twice than never but .. :)08:45
*** jpena is now known as jpena|away08:45
*** b00tcat has quit IRC08:46
*** b00tcat has joined #tripleo08:47
*** bkopilov_ has quit IRC08:48
*** bkopilov has quit IRC08:48
*** florianf has joined #tripleo08:49
*** florianf has quit IRC08:51
*** jpich has joined #tripleo08:52
*** cylopez has joined #tripleo08:53
*** jpena|away is now known as jpena08:53
*** paramite has joined #tripleo08:58
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates stable/ocata: Apply puppet in non-controller script in step.  https://review.openstack.org/43531508:59
*** zzzeek has quit IRC09:00
shardymatbu: Yeah well spotted, those can probably be removed, let me try09:00
*** zzzeek has joined #tripleo09:00
shardyalso we need to modify that logic now ocata branched, but I'll do that in a different patch09:00
openstackgerritSteven Hardy proposed openstack-infra/tripleo-ci master: remove test-environments/multinode_major_upgrade.yaml  https://review.openstack.org/42637509:01
shardymatbu: ^^09:02
shardyI think we can perhaps remove the second conditional too09:03
shardyOr, actually perhaps that's needed so we do an exact match and don't include the env file for scenarios09:04
*** suuuper has joined #tripleo09:09
*** abehl has joined #tripleo09:09
*** paramite has quit IRC09:10
*** jpich has quit IRC09:14
*** dtantsur|afk is now known as dtantsur09:16
*** chem has joined #tripleo09:17
matbushardy: i think the exact match won't never works09:18
shardyhrm, you may be right, but the previous revision passed CI which is confusing..09:18
matbushardy: because the $TOCI_JOBTYPE is always something like : gate-tripleo-ci-centos-7-multinode-upgrades-nv09:18
*** zoli|gone is now known as zoli09:18
*** karimb has joined #tripleo09:18
* shardy looks at logs09:19
*** lucas-afk is now known as lucasagomes09:20
*** athomas has joined #tripleo09:20
shardyhttp://logs.openstack.org/75/426375/12/check/gate-tripleo-ci-centos-7-multinode-upgrades-nv/9b714b1/console.html#_2017-02-17_00_54_23_31156209:21
shardymatbu: ^^09:21
matbuhttp://logs.openstack.org/75/426375/12/check/gate-tripleo-ci-centos-7-multinode-upgrades-nv/9b714b1/console.html#_2017-02-17_01_06_28_16294609:22
* matbu looks09:22
matbuyep it looks like the exact match works09:22
matbuand $UPGRADE_ENV is set so yes09:23
matbubut then i think the if statement is useless, cause i don't see what would be the else case then09:23
mariosshardy: matbu filed https://bugs.launchpad.net/tripleo/+bug/1665585 on EmilienM request at https://review.openstack.org/#/c/434305/ fyi09:24
openstackLaunchpad bug 1665585 in tripleo "Ocata to Pike operator driven ansible upgrade" [Undecided,New] - Assigned to Marios Andreou (marios-b)09:24
matbuho wait there is no else09:24
matbushardy: so in all case the exact is useless09:25
matbuif im not sleeping :) (friday and coffeeless)09:25
*** paramite has joined #tripleo09:26
matbumarios: cool09:26
matbumarios: i think we should start of thinking of a tripleo-lifecycle or tripleo-upgrade upstream repo like we talk about during the scrum yesterday09:27
shardymatbu: what would we put in the repo?09:27
*** jaosorior has joined #tripleo09:27
d0ugalso, just to be clear, master is now open for Pike?09:28
matbushardy: a set of tools required for upgrade, like the ansible libraries09:28
*** jpich has joined #tripleo09:28
openstackgerritMerged openstack/tripleo-heat-templates master: Apply puppet in non-controller script in step.  https://review.openstack.org/43364109:29
mariosmatbu: i missed that bit sry09:29
shardymatbu: I'm still not clear which libraries - the pacemaker one?09:29
mariosmatbu: or i don't understand09:29
mariosmatbu: i mean about the repo thing09:29
shardyideally we shouldn't need a bunch of tripleo specific ansible plugins, just a workflow that uses the standard ansible interfaces09:29
mariosd0ugal: well there is stable/ocata in tripleo-heat-templates now fwiw but not sure if you meant that or if we were ok to propose pike features (either case i think yes)09:29
shardyalthough I can imagine some uses for custom modules in future09:29
shardyd0ugal: yes ocata branched so master is now pike09:30
shardyanything for ocata now needs to be backported09:30
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates master: Switch keystone default provider to fernet  https://review.openstack.org/43242009:31
matbushardy: yep pacemaker and some of python pieces that we could use with ansible and upgrade09:31
mariosmatbu: ah you mean ansible-pacemaker ?09:32
mariosgfidente: can you please check https://review.openstack.org/#/c/434367/5 i updated on your comments09:32
*** suuuper has quit IRC09:32
gfidentemarios yes so I was trying to make the shell script skip the CEPH_OSD part if disable_upgrade was set to true09:33
gfidentethough I think for compute nodes that will be set to true09:33
gfidenteeven though it's only disabled for nova-compute09:33
gfidenteso it won't fix the issue right?09:33
shardymatbu: I guess I'd prefer to see that maintained somewhere outside of TripleO09:33
shardysince it's specific to pacemaker, but not really TripleO?09:34
mariosgfidente i commented on the review... well the script being there in the first place means that yes disable_upgrade was true09:34
shardycertainly we can discuss it tho09:34
mariosgfidente: otherwise it doesn't get delivered09:34
gfidenteso I was thinking09:35
gfidentethis is potentially an issue for any other service deployed on compute node09:35
jaosoriorshardy: in CI is there a way to know if the image build was triggered or if we used cached images?09:35
*** paramite has quit IRC09:36
*** panda|zZ is now known as panda09:38
pandajaosorior: you have to look at some parts of the script output09:38
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates stable/ocata: Apply puppet in non-controller script in step.  https://review.openstack.org/43531509:39
marioschem: matbu ^^^ cherrypick of chem patch merged master09:39
mariosshardy: gfidente: can you please check https://review.openstack.org/#/c/435315/  (cherrypick to ocata) when you next have time09:40
chemmarios: thanks09:40
jaosoriorpanda: in a job (that was supposed to use cached images) I saw this http://logs.openstack.org/63/426863/9/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/149fd35/console.html#_2017-02-17_02_22_42_77824409:41
pandajaosorior: damn, probably still space problems09:42
pandaand I still don't have access to the mirror host09:42
jaosoriorpanda: so it goes ahead and build the images. which for the TLS everywhere job is not good since I was always assuming we would use cached images. So I was thinking about making it a bit more ressilient by checking if cached images were used or not09:42
jaosoriorpanda: who has access?09:43
shardyhttps://github.com/openstack-infra/tripleo-ci/blob/master/toci_instack_ovb.sh#L11209:43
shardyjaosorior: I think you'd look for the canusecache in the log, or the subsequent wget09:44
shardyjaosorior: also if there's images built, there will be image build logs09:44
marioschem: np, is for selfish reasons fwiw :) your patch has to land so mine can09:44
chemmarios: some says that selfishness is good for the world, maybe it's true ?09:44
pandajaosorior: sagi, derek, bnemec, maybe emilien and maybe you, my key was added too, but it doesn't seem to work09:45
mariosshardy: matbu chem gfidente ccamacho any objections to me +A on https://review.openstack.org/#/c/434305/2 it has 3x +2 already09:45
gfidentegooo09:46
jaosoriorpanda: I have the same issue09:46
jaosoriorpanda: my key was added but I still can't log in09:46
chemmarios: nope09:46
ccamachomario not really I was just not passing CI but now it is :)09:46
ccamacho**It09:46
mariosthanks ccamacho09:46
ccamachonp09:46
shardymarios: yup go for it :)09:47
shardyWho has been testing nova newton->ocata upgrades locally?09:47
shardyI hit quite a few problems yesterday, would be good to confirm other folks have it working09:47
mariosshardy: i have09:48
mariosshardy:  though downsttream setup i ended up grabbing puppet-tripleo and puppet-pacemaker from delorean repo on my overcloud nodes09:49
shardymarios: Ok, I'm testing with nova but no pacemaker09:49
mariosshardy: latest was like... http://paste.openstack.org/raw/599243/09:49
*** paramite has joined #tripleo09:49
mariosshardy: gets thorough ansible stasks and starts puppet09:49
shardyand I'm using tripleo.sh --repo-setup so should be getting trunk puppet*09:49
mariosshardy: cluster goes maintenance mode as expected then pupupet fails, for varitey of missing things until i started using the upstream package, then i hit this09:50
mariosshardy: in faact mcornea filed bugs sec09:50
shardymarios: Ok, my upgrade kept getting stuck at different nova tasks so I'll dig into it09:50
shardytrying to get it added to CI ref https://review.openstack.org/#/c/434348/09:50
*** yamahata has quit IRC09:51
shardymarios: do we have upstream bugs tagged so I can see what fixes are pending?09:51
shardyand/or what I can fix :)09:51
marioshttps://bugs.launchpad.net/tripleo/+bug/1665328 and https://bugs.launchpad.net/tripleo/+bug/1664265 are both things you can see in that paste of my run yesterday so i intend to dig into them a bit more after lunch09:51
openstackLaunchpad bug 1665328 in tripleo "Newton->Ocata upgrade fails during overcloud.AllNodesDeploySteps.AllNodesPostUpgradeSteps.ControllerDeployment_Step3.0 because httpd is stopped" [High,Triaged]09:51
openstackLaunchpad bug 1664265 in tripleo "nova-placement-api is not installed during Newton->Ocata upgrade" [High,Triaged]09:51
shardymarios: ack, thanks09:52
mariosshardy: for the httpd one, it could be we need to start that in ansible before we puppet-apply09:52
mariosshardy: but i haven't had a close look at all yet09:53
shardymarios: we can't because some services aren't configured yet09:53
shardyso we need puppet to configure them, then start httpd09:53
mariosshardy: hmm ok hope we can work around it in puppet then i mean whatever it is that fails at this point09:53
shardymarios: yeah I think it may be a puppet ordering thing perhaps09:53
shardyas some services are moved to run under httpd during the upgrade09:54
shardyIIRC we backed out the nova-api move, but I think there are others09:54
mariosshardy: right yeah i was aware of the nova-api revert also didn't know/thinkthere were others09:54
shardyis the placement API one because it's not in the newton image and the package provider is disabled?09:55
shardyperhaps we need to explicitly install it via the ansible tasks09:55
shardyI can take a look at that one if you like09:55
mariosshardy: yeah or add it to the upgradecommoninit09:55
mariosshardy: sure09:56
*** salmankhan has joined #tripleo09:56
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates stable/ocata: Add explicit swift check to tripleo_upgrade_node.sh  https://review.openstack.org/43533809:58
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates master: Install nova-placement package on upgrade  https://review.openstack.org/43533910:01
shardyI think that'll do it, locally testing now10:01
mariosshardy: nice10:02
shardyHmm OS::TripleO::Services::NovaPlacement doesn't exist in newton t-h-t, so we need some way to add it on upgrade10:03
*** percevalbot has quit IRC10:03
shardyWe're going to have to use the stable branch for the test environments10:03
mariosshardy: but dont you use use ocata tht for the upgrade10:04
mariosshardy: oh you mean for the ci job?10:04
shardymarios: yeah, but we currently also use ocata tht for the CI job deploy of newton10:04
shardyyeah10:04
shardyI'll make a coffee then try to fix it ;)10:04
shardywe're going to end up hacking on EmilienM's patch again ;)10:04
*** jd__ has joined #tripleo10:08
*** derekh has joined #tripleo10:08
jaosoriorderekh: hey dude10:08
*** percevalbot has joined #tripleo10:09
jaosoriorderekh: seems that the server where we're hosting the cached images might be having space issues http://logs.openstack.org/63/426863/9/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/149fd35/console.html#_2017-02-17_02_22_42_778244 do you hace access to it?10:09
derekhjaosorior: hey ya10:09
derekhjaosorior: yup, one sec and I'll take a look10:10
*** mcornea has joined #tripleo10:10
pandaderekh: I can take care of it, but first some else must understand why I can't access despite having the key in the admin list10:10
*** stendulker has quit IRC10:10
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates master: Change the relase name for the templates to pike.  https://review.openstack.org/43534510:14
openstackgerritMerged openstack/tripleo-heat-templates master: Deploy Manila with CephMDS in scenario004  https://review.openstack.org/42338610:15
derekhpanda: your key is added, it wasn't there because the server picks it up when it was first booted, your key wasn't on the list when it was created10:15
pandaderekh: oh, I thought there was a puppet script that added the key. works now, thanks.10:17
jaosoriornice, works for me now as well10:18
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo master: Create /etc/my.cnf.d/tripleo.cnf with proper bind-address  https://review.openstack.org/43142410:19
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates master: Make the DB URIs host-independent for all services  https://review.openstack.org/43142510:19
derekhjaosorior: ya, I synced the whole list10:19
derekhpanda: jaosorior so in /var/www/html the files are stored10:20
derekhthese builds-* directories are new?10:20
pandaderekh: yes, for stable branch promotion, but I'm not sure how are they used10:21
*** chem has quit IRC10:22
jaosoriorhaving 40G for a server where we store the images seems like too little10:22
*** chem has joined #tripleo10:23
pandaI thins the initial ideas was to store only some of the images from master10:23
pandanot all the images from all branches10:24
pandaalso, this is a side effect of having CI working well, lots of promoted images10:24
pandanow if only I could understand what  was going to be done with the stable branches images ...10:25
derekhpanda: jaosorior so looks like the stable image arn't currently used10:26
derekhpanda: jaosorior apache logs shows no downloads in the last few days10:26
derekhpanda: jaosorior http://paste.openstack.org/show/599370/10:26
pandaindeed, but I think sagi was starting to look on how to use them10:27
pandaanyway, I'll remove all images older than 2 days10:27
pandastarting from stable branches10:27
derekhpanda: ok10:27
*** limao has quit IRC10:28
derekhpanda: also there is a clean up script that is supposed to be removing old images, I wonder if it is searching the new stable directories for images to remove10:28
pandaderekh: I was searching that, where is it ?10:28
*** limao has joined #tripleo10:29
pandano entries in crontab -l /etc/crontab /etc/cron.d*10:29
*** jaosorior has left #tripleo10:29
*** jaosorior has quit IRC10:29
derekhjaosorior: ya, its small, if things are being cleaned up correctly then it should be ok10:30
derekhpanda: its at the bottom of the promote script /opt/stack/tripleo-ci/scripts/mirror-server/promote.sh10:30
derekhpanda: crontab -l as root10:30
*** jaosorior has joined #tripleo10:30
pandavim: command not found. /me ragequits. :)10:31
derekhjaosorior: ya, its small, if things are being cleaned up correctly then it should be ok10:32
*** limao has quit IRC10:33
derekhjaosorior: the reason its small is because of the size of the HD on the host machine, if we allowed it to take 100G that would be half of the storage on the compute node10:33
*** owalsh-afk is now known as owalsh10:34
openstackgerritMerged openstack/tripleo-ui master: Include j2 files in plan creation  https://review.openstack.org/43416110:35
jaosoriorderekh: oh, that makes sense then10:35
*** karthiks has quit IRC10:36
openstackgerritSteven Hardy proposed openstack-infra/tripleo-ci master: remove test-environments/multinode_major_upgrade.yaml  https://review.openstack.org/42637510:39
openstackgerritSteven Hardy proposed openstack-infra/tripleo-ci master: upgrades: switch 'newton' to $UPGRADE_RELEASE  https://review.openstack.org/43320910:39
shardyderekh: Hey, sorry another question for you when you get a sec10:40
derekhshardy: fire ahead10:41
shardyderekh: any ideas how we could make Depends-On work accross branches, e.g with the tripleo-ci repo?10:41
shardyhttps://github.com/openstack-infra/tripleo-ci/blob/master/scripts/deploy.sh#L30410:41
shardyderekh: currently we download the newton t-h-t manually, which works fine, but we've got no way to test under-review patches to newton to ensure the newton->ocata upgrade still works10:41
*** tosky has joined #tripleo10:41
shardyonly that patches to ocata work from a packaged newton10:41
shardyI can't think of a way to make it work without overwriting both t-h-t version10:42
pandaok, freed 19G of space, removed non-wotking builds dir, now image dir points to an image from yesterday evening10:43
shardyThe issue is in https://review.openstack.org/#/c/426375/ we're moving the environment used for the upgrades job from triple-ci to t-h-t10:43
shardyand we'll have no way to test backports to newton without landing them and seeing if the job breaks10:43
jaosoriorbandini: I wonder if there's a way to add a validation that will check if the MySQLClient was added10:44
jaosoriorthat would be pretty cool10:44
bandinijaosorior: right, I need to explore the whole validation stuff. I need to RTFM I guess10:45
bandiniI need it for a few other things as well10:45
derekhshardy: a newton package get build if you testing a newton patch, couldn't that be used instead of the one your downloading? although I guess I'm missing something10:45
derekh*gets built10:45
shardyderekh: Yeah, I guess the part I can't figure out is how to decide when to use a t-h-t package from the tripleo-ci repo10:46
shardye.g do we use it for the initial deploy (newton) or the upgrade (ocata)10:46
shardyI suppose we could look at the package versions, but that seems a little icky10:46
*** jerrygb has joined #tripleo10:46
bandinijaosorior: have you already done anything like that with validations? or have examples to look at?10:47
shardyderekh: actually, maybe looking at STABLE_RELEASE would be enough10:48
derekhshardy: either the package version or the STABLE_RELEASE env variable is all I can think of10:48
pandaugh cloud.centos.org images take 6G, why do we mirror all the images ?10:48
jaosoriorbandini: I know it should be possible but I haven't10:48
shardyderekh: heh, snap - yeah thanks for talking it through, that may well work :)10:48
bandinijaosorior: ack10:48
derekhshardy: ok, no prob10:48
pandaderekh: ^ ?10:48
openstackgerritJuan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: DO NOT MERGE - testing TLS everywhere job with RabbitMQ  https://review.openstack.org/42630710:48
*** michapma_alt has joined #tripleo10:49
*** pblaho has joined #tripleo10:49
*** xuao has quit IRC10:50
derekhpanda: we don't need them all, it wasn't 6G back when we started mirroring them, I guess I didn't expect it to increase so much10:50
*** jerrygb has quit IRC10:50
*** jerrygb has joined #tripleo10:51
*** jaosorior has quit IRC10:52
*** udesale has quit IRC10:52
*** jaosorior has joined #tripleo10:53
pandaderekh: ok, guest we'll have to update some of the script after the transition., no time now, and not sure what we really need. thanks for the assistance10:53
pandaguess*10:53
*** skramaja has quit IRC10:53
derekhpanda: no prob10:54
*** jerrygb_ has joined #tripleo10:54
*** jerrygb has quit IRC10:55
openstackgerritmathieu bultel proposed openstack/tripleo-heat-templates master: Add manual ovs upgrade script for workaround ovs upgrade issue  https://review.openstack.org/43434610:57
*** athomas has quit IRC10:58
*** jaganathan has quit IRC10:58
*** athomas has joined #tripleo10:58
*** jlinkes_ has joined #tripleo10:58
openstackgerritChristian Schwede proposed openstack/tripleo-common stable/newton: Use Mistral to create Swift temporary URLs  https://review.openstack.org/43528410:59
*** Vijayendra has quit IRC10:59
openstackgerritAttila Darazs proposed openstack/tripleo-quickstart-extras master: build-test-packages: use http urls  https://review.openstack.org/43536511:00
*** jlinkes has quit IRC11:00
openstackgerritMerged openstack/tripleo-ui master: Add i18n to nodes page and buttons  https://review.openstack.org/43317211:02
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: remove duplicate network_environment from undercloud-deploy role  https://review.openstack.org/43492411:02
*** jlinkes__ has joined #tripleo11:03
*** karimb has quit IRC11:04
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Allow direct override of the overcloud's DNS servers  https://review.openstack.org/43496311:06
*** jlinkes_ has quit IRC11:07
*** sudipto_ has quit IRC11:08
*** sudipto has quit IRC11:08
*** ealcaniz has quit IRC11:10
*** zoli is now known as zoli|lunch11:15
openstackgerritJulie Pichon proposed openstack/tripleo-ui stable/ocata: Add i18n to nodes page and buttons  https://review.openstack.org/43538111:15
openstackgerritJulie Pichon proposed openstack/tripleo-ui stable/ocata: Include j2 files in plan creation  https://review.openstack.org/43538211:16
openstackgerritMerged openstack/tripleo-heat-templates master: Add explicit swift check to tripleo_upgrade_node.sh  https://review.openstack.org/43430511:16
*** sdake has joined #tripleo11:18
openstackgerritChristian Schwede proposed openstack/tripleo-common stable/newton: Use Mistral to create Swift temporary URLs  https://review.openstack.org/43528411:18
jaosoriorpanda: oh, now this is happening http://logs.openstack.org/24/431424/3/experimental-tripleo/gate-tripleo-ci-centos-7-ovb-nonha/1d980f2/console.html#_2017-02-17_11_00_24_68947211:20
pandajaosorior: of course, the only image that was present for master, wasn't good.11:22
pandaremoving everything , starting manual promotion. At tleast the gate will work11:23
jaosoriorok11:24
*** cdearborn has joined #tripleo11:30
*** salmankhan has quit IRC11:33
*** salmankhan has joined #tripleo11:34
*** karimb has joined #tripleo11:34
*** yolanda has quit IRC11:36
*** noslzzp has quit IRC11:37
*** noslzzp has joined #tripleo11:39
*** jkilpatr has quit IRC11:39
openstackgerritmathieu bultel proposed openstack/tripleo-heat-templates master: Add manual ovs upgrade script for workaround ovs upgrade issue  https://review.openstack.org/43434611:39
*** yolanda has joined #tripleo11:39
*** _milan_ has joined #tripleo11:40
openstackgerritmathieu bultel proposed openstack/tripleo-heat-templates master: Add manual ovs upgrade script for workaround ovs upgrade issue  https://review.openstack.org/43434611:41
*** matbu is now known as matbu|pto11:42
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates master: multinode/upgrade: add nova services to environment  https://review.openstack.org/43434811:46
*** rhallisey has joined #tripleo11:47
*** nmathew has quit IRC11:48
*** ansmith has joined #tripleo11:48
*** sdake has quit IRC11:49
*** sdake has joined #tripleo11:50
lhindsfolks, specs for Pike..we don't have a /tripleo-specs/specs/pike directory yet, should I create this or wait until release tags are put in place?11:51
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates stable/newton: backport multinode_major_upgrade.yaml environment  https://review.openstack.org/43540211:53
openstackgerritSteven Hardy proposed openstack-infra/tripleo-ci master: remove test-environments/multinode_major_upgrade.yaml  https://review.openstack.org/42637511:55
openstackgerritSteven Hardy proposed openstack-infra/tripleo-ci master: upgrades: switch 'newton' to $UPGRADE_RELEASE  https://review.openstack.org/43320911:55
EmilienMhello11:56
d0ugalHey!11:56
EmilienMshardy: what's up?11:56
EmilienMd0ugal: hey you11:56
shardyEmilienM: Hey, good morning11:56
shardyEmilienM: Hey, I'm just trying to rework https://review.openstack.org/#/c/426375/ because https://review.openstack.org/#/c/434348/ won't work until we have branched versions of multinode_major_upgrade.yaml consumed in CI11:57
lhindshi EmilienM11:57
*** adarazs is now known as adarazs_lunch11:57
jaosoriorlhinds: yep; you can create it11:57
shardywe need to deploy newton without the nova placement service, then add it on upgrade11:57
lhindsthx jaosorior11:57
lhindsactually on the topic of specs, do we have a deadline for Pike in ooo?11:58
EmilienMshardy: I see11:58
EmilienMshardy: thanks!11:58
shardyEmilienM: we may need to just land https://review.openstack.org/#/c/435402/ when it passes CI, as I don't think the Depends-On to the stable/newton t-h-t will work11:59
shardyI'm trying to figure out how to make it work with the rpm2cpio install of $STABLE_RELEASE tht11:59
EmilienMshardy: I was about to propose a squash12:00
shardyprobably won't get that working today tho12:00
EmilienMshardy: that's an excellent idea12:00
shardyyeah, done :)12:00
*** bogdando has joined #tripleo12:00
EmilienMshardy: what to do with https://review.openstack.org/#/c/435191/ and https://review.openstack.org/#/c/435284/ ?12:01
*** jkilpatr has joined #tripleo12:01
shardyEmilienM: we need to either land or abandon https://review.openstack.org/#/c/435284/ as the Depends-On will block https://review.openstack.org/#/c/43519112:02
shardybackporting new mistral actions to a stable branch seems questionable to me though12:03
shardyif we consider the actions/workflows to be our "api"12:03
EmilienMshardy: I'm confused, one is for ocata and another for newton12:03
shardyEmilienM: Depends-On isn't branch aware12:03
EmilienMIMHO https://review.openstack.org/#/c/435191/ should be merged12:03
shardyhttps://review.openstack.org/#/q/Ic3da38cffdd993c768bdb137c17d625dff1aa372,n,z12:04
EmilienMah right12:04
EmilienMI didn't see the depends-on this time12:04
shardyDepends-On just looks at that, and will block because one of them isn't merged12:04
openstackgerritMerged openstack/puppet-tripleo master: xinetd: bind only on mysql network  https://review.openstack.org/35179512:04
shardyyeah12:04
EmilienMcschwede_: ^ thoughts?12:04
shardyIf you're OK to merge the new action then I guess we're good :)12:04
* shardy was feeling unsure from a stable-maint perspective, but it is related to a bugfix12:05
EmilienM"Default Swift ring deployment by TripleO are error-prone"12:05
EmilienMit's High12:05
EmilienMI think it's backward compatible, we don't touch an existing workflow12:06
shardyOk, that's fair12:06
EmilienMwe chance the "add_validation_ssh_key" task though12:06
EmilienMwe'll wait for CI12:06
*** jkilpatr has quit IRC12:09
cschwede_shardy, EmilienM: indeed, there is a change to the workflows - however, it's a new action and not changing an existing one. i'm biased on this too, and wouldn't have proposed if it was just a new feature12:09
*** jerrygb_ has quit IRC12:10
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Switch to net-config-multinode  https://review.openstack.org/42637212:10
cschwede_but then, it fixes the workflow, and this should help operators a lot especially because swift is now used in default deployments in newton12:10
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates stable/ocata: Deploy Manila with CephMDS in scenario004  https://review.openstack.org/43540812:10
*** ansiwen_ is now known as ansiwen12:10
EmilienMgfidente: please look this backport https://review.openstack.org/#/c/435408/12:11
EmilienMcschwede_: lgtm12:11
*** bogdando has quit IRC12:11
shardycschwede_: thanks for clarifying - is there any impact on update, e.g for update of an existing newton deployment?12:11
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates stable/ocata: Add explicit swift check to tripleo_upgrade_node.sh  https://review.openstack.org/43533812:11
*** dbecker has joined #tripleo12:12
cschwede_shardy: no, i don't think so. however, an operator will only benefit from this after a second update - the first run (when the rings are not yet stored on the undercloud) will just work as before - ie break if controller/storage nodes are added or replaced12:13
shardycschwede_: Ok, but after the update things like node replacement and scale-out will work better12:14
cschwede_shardy: exactly12:14
shardyOk great, sounds like good justification for backport to me :)12:14
shardythanks for clarifying12:15
cschwede_shardy: yw - thx for looking into it!12:15
*** jaosorior has quit IRC12:16
gfidenteEmilienM seems clean?12:16
EmilienMgfidente: I just added cherry-picked from12:18
EmilienMdsavineau: do we want https://review.openstack.org/#/c/351795/2 in stable/ocata?12:19
chemhi nice core people.  Before stable/mitaka goes to oblivion, could you please merge this one https://review.openstack.org/#/c/42809312:20
chemEmilienM: bandini ^ : this is a fully tested by myself commit.12:21
EmilienMchem: I was reviewing it12:21
chemEmilienM: thanks man12:22
shardycschwede_: Hey I added some comments to https://review.openstack.org/#/c/43519112:22
EmilienMchem: https://review.openstack.org/#/c/428093/21/extraconfig/tasks/major_upgrade_compute.sh12:22
shardycschwede_: I know it's a backport but I wish I'd seen that sooner12:22
EmilienMchem: is it going to cause network downtime on compute nodes for VMs?12:22
*** jkilpatr has joined #tripleo12:22
EmilienM(I think yes)12:22
shardynot blocking but perhaps we can iterate on it to remove the coupling with hard-coded roles12:22
shardyreally, we should not ever do that anymore12:22
EmilienMchem: otherwise, lgtm. I'll let bandini press on the magic button :D12:23
chemEmilienM: I'm checking before doing anything, it's just that I've notice that sometime it goes down after upgrade so I put it back up12:23
shardycschwede_: going to grab lunch but happy to chat more after12:23
*** shardy is now known as shardy_lunch12:23
chemEmilienM: if enable -> if down -> then restart12:23
cschwede_shardy: enjoy lunch! i share your concerns, happy to improve this - let's chat later (going to lunch soon as well)12:24
*** nmathew has joined #tripleo12:24
bandinichem: sec12:24
EmilienMchem: cool12:24
*** nmathew has quit IRC12:25
*** psahoo has quit IRC12:26
chembandini: np, take your time, if you need more info about how I tested it or if things are unclear let me know.  I will be off during 1 hour to enjoy fresh air.12:27
bandinichem: fresh air? what's that?12:27
chembandini: A new brand of coffee :)12:27
bandinichem: review seems sane and if you tested it it's fine12:27
bandinichem: ahahaha12:27
*** jerrygb has joined #tripleo12:29
openstackgerritJames Slagle proposed openstack/tripleo-heat-templates master: Add environment for deployed-server with pacemaker  https://review.openstack.org/43445212:30
openstackgerritJames Slagle proposed openstack/tripleo-heat-templates master: Generate Pre/Post Puppet Tasks for all roles  https://review.openstack.org/43445112:30
mariosbandini: man you live in the alps! open a window! (or not probably a bit cold today?)12:30
bandinimarios: ahaha it's a little chilly indeed :)12:30
*** jaosorior has joined #tripleo12:30
*** ccamacho is now known as ccamacho|lunch12:33
*** dprince has joined #tripleo12:35
gfidentebandini while you're around12:36
gfidenteI remember you had to deal with the firewall12:37
gfidenteI don't seem to get the default LOG/DROP rules on the controllers12:37
gfidenteis that the case for you too?12:37
gfidenteI am not sure how can that be given it's managed by the role which should be shared across all nodes12:38
gfidenteso was wondering if it's only me?12:38
mariosgfidente: not clear on https://review.openstack.org/#/c/434367/5 are you still exploring whether it is indeed a good idea all together (I mean enabling the manual ceph upgrade?)12:39
chemEmilienM: bandini thanks for the leap of faith :)12:39
mariosgfidente: otherwise shall we land and iterate?12:39
gfidentemarios I am worried that12:39
gfidenteif we land it as-is12:39
*** _milan_ is now known as milan_12:39
gfidenteit's going to stay as is12:39
*** sdake has quit IRC12:39
mariosgfidente: i guess you/ceph team still needs to talk to stakeholders about this whole 'which is the default' thing12:39
bandinigfidente: hohum. lemme see12:39
*** karimb has quit IRC12:39
gfidentewell I'd like the ansible tasks to stay the default12:39
gfidentebandini tks!12:40
gfidentemarios you know I think that disable flag should be per-service too12:40
bandinigfidente: so you're saying that you don't get the 998 and 999 log/drop all rules on controllers only, correct?12:40
gfidentenot per role12:40
gfidentebandini yeah12:40
gfidentebut I have them on non-controller12:40
mariosgfidente: well we could check if both nova_compute and ceph_osd are true12:40
mariosgfidente: then we have a hci role12:41
bandinihohum what's weird those are created by simply including the firewall module iirc12:41
bandinilemme see12:41
honzajtomasek: hey, i know i'm behind on reviews, hope to catch up today (been dealing with release issues)12:41
gfidentebandini yes and the profile is shared across all roles12:41
jtomasekhonza: no worries12:41
gfidentemarios yeah but what do we do then? we still don't know if "manual" was intended for ceph12:42
bandinigfidente: are those the only rules missing on your controllers?12:42
mariosgfidente: but manual is for the ROLE, not the service12:42
gfidentebandini actually no12:43
gfidentemarios yeah so our problem is that the param is per role but we need it per service12:43
bandinigfidente: ah. do you get a sense of how many/which are missing?12:43
bandinior do you have access to this system and I will take a peek?12:43
*** pkovar has joined #tripleo12:44
gfidentebandini ack12:45
*** jerrygb_ has joined #tripleo12:45
mariosgfidente: ok, gona try focus on something for a bit now, and will revisit it in a couple hours lets see if we come up with something12:46
mariosgfidente: hopefully we get a happy ending ;)12:46
*** sdake has joined #tripleo12:46
*** jerrygb has quit IRC12:46
gfidentemarios yes and in 10mins I have the standup so we can discuss the default thing knowing of the hci issue12:47
mariosgfidente: great12:47
mariosgfidente: thanks12:47
gfidentety12:48
*** limao has joined #tripleo12:54
*** adarazs_lunch is now known as adarazs12:54
*** athomas has quit IRC12:57
*** zoli|lunch is now known as zoli12:57
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates master: Switch keystone default provider to fernet  https://review.openstack.org/43242012:58
*** limao_ has joined #tripleo12:58
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates master: Switch keystone default provider to fernet  https://review.openstack.org/43242012:58
jaosoriorEmilienM: added release note ^^12:59
*** ealcaniz has joined #tripleo12:59
EmilienMjaosorior: thanks12:59
*** limao has quit IRC12:59
*** sdake_ has joined #tripleo12:59
*** lucasagomes is now known as lucas-hungry13:01
*** ealcaniz is now known as ealcaniz|lunch13:01
*** jayg|g0n3 is now known as jayg13:02
*** jerrygb_ has quit IRC13:02
*** jerrygb has joined #tripleo13:02
*** athomas has joined #tripleo13:02
*** sdake has quit IRC13:02
*** limao_ has quit IRC13:03
*** liverpooler has joined #tripleo13:03
*** limao has joined #tripleo13:05
*** psahoo has joined #tripleo13:07
*** trown|outtypewww is now known as trown13:08
*** michapma_alt has quit IRC13:10
*** fultonj has joined #tripleo13:11
*** owalsh is now known as owalsh-afk13:12
socialanyone hitting issues with iptables during overcloud-AllNodesDeploySteps Step1 - iptables seem to be started by puppet and set to block all traffic before setting up pacemaker13:13
*** ealcaniz|lunch has quit IRC13:13
*** ealcaniz has joined #tripleo13:14
openstackgerritJulie Pichon proposed openstack/puppet-tripleo master: Enable languages in UI config  https://review.openstack.org/43232813:14
EmilienMsocial: can you tell which traffic is blocked? I've seen that before with bandini13:14
*** sdake has joined #tripleo13:14
EmilienMsocial: do you see a trace in puppet catalog?13:14
bandinisocial: make sure the image you start from has an empty /etc/sysconfig/ip[6]tables13:15
socialEmilienM: https://paste.fedoraproject.org/paste/BmicRIiGH-ZOKfiIbYbWR15M1UNdIGYhyRLivL9gydE=13:15
socialbandini: I should have the image element to clean iptables13:15
EmilienMsocial: look http://logs.openstack.org/99/418099/24/experimental/gate-tripleo-ci-centos-7-scenario005-multinode/3b81278/logs/postci.txt.gz#_2017-02-10_17_12_49_00013:15
*** sdake_ has quit IRC13:15
socialbandini: could check if it actually ran13:15
bandinisocial: it probably did not run because you would not have those rules if that were the case13:16
*** michapma_alt has joined #tripleo13:16
EmilienMACCEPT     all  --  anywhere             anywhere13:16
EmilienMweird13:16
*** sudipto has joined #tripleo13:16
*** sudipto_ has joined #tripleo13:16
bandinithere us a REJECT at the end which used to mess us up13:17
bandinis/us/is/13:17
socialbandini: hmm /usr/share/tripleo-image-elements/iptables/install.d/99-empty-iptables13:17
socialchecking image build logs13:17
bandinisocial: the inclusion of the iptables element happens in tripleo-common13:18
bandinilemme find the exact file13:18
bandiniI have the memory of a goldfish13:18
jaosoriorpanda: has the promotion happened yet?13:18
jaosoriorpanda: trying to set up an oooq envirnoment and I get an error getting the image's md513:18
EmilienMjaosorior: promotion to what? trunk? we've got one last night13:18
socialbandini: yes it does not seem to be present13:19
bandinisocial: in /usr/share/tirpleo-common/image-yaml/overcloud-images.yaml you should have the iptables element13:19
bandinisocial: ack, so that should fully explain it13:19
jaosoriorEmilienM: we had issues with space in the server where we store the cached images. The latest promotion had a wrong image because of this.13:20
jaosoriorEmilienM: so we need another promotion to happen in order to get another good latest promoted image13:20
pandajaosorior: almost finished13:20
*** Vijayendra has joined #tripleo13:20
socialbandini: yes but the element is there :)13:21
socialbandini: I mean it does not seem to be present in image build13:21
jaosoriorEmilienM: I actually noticed this because the TLS-everywhere service was failing. And it was failing because it depended no a modification to the overcloud image that happens when we used the cached one. But we were not using the cached one, so it didn't get the newer cloud-init and it failed. Then we figured out why it wasn't using the cached image.13:21
*** thrash|g0ne is now known as thrash13:21
*** Vijayendra has quit IRC13:21
bandinisocial: maybe the image was built when tripleo-common or t-i-e did not have the iptables pruning stuff?13:21
*** ealcaniz has quit IRC13:22
socialbandini: rebuilding images, I'll update you13:22
bandinisocial: ack. I'll be around ;)13:22
*** salmankhan has quit IRC13:24
EmilienMpanda, jaosorior: ok, I didn't know sorry :)13:27
*** tbonds has joined #tripleo13:28
jaosoriorEmilienM: no biggie13:29
*** jpena is now known as jpena|lunch13:30
*** gbarros has joined #tripleo13:30
*** shardy_lunch is now known as shardy13:32
*** eck` is now known as eck`gone13:32
*** lblanchard has joined #tripleo13:33
shardyEmilienM: hey I think we can probably merge https://review.openstack.org/#/c/434250/ not it passed CI, what do you think?13:35
shardyit will be required for the nova integration to upgrade CI13:35
*** Goneri has joined #tripleo13:35
*** links has quit IRC13:36
*** ccamacho|lunch is now known as ccamacho13:37
*** artom has quit IRC13:37
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates master: Don't assume default network names in net_ip*map  https://review.openstack.org/41362613:38
EmilienMshardy: done13:39
shardythanks! :)13:39
*** fzdarsky_ has quit IRC13:40
*** suuuper has joined #tripleo13:43
cschwede_shardy: i left a few comments in https://review.openstack.org/#/c/435191/ - let me know what you think about it13:43
*** jmelvin has joined #tripleo13:43
*** noslzzp has quit IRC13:45
*** noslzzp has joined #tripleo13:45
*** diana_clarke has joined #tripleo13:45
slagleshardy: cschwede_ : i think we're going to need a way for the service profile templates to expose Task resources somehow13:45
shardyslagle: I was thinking can't we just do this via puppet in the existing ringbuilder profile?13:46
slagleb/c i already had to fix the hardcoding for Pacemaker Tasks as well: https://review.openstack.org/#/c/434451/13:46
shardyslagle: If it has to be outside puppet, I'd say we wire in ansible tasks like we do for update13:46
slagleshardy: i guess that might be possible, was just thinking of something that could be generic13:47
shardybut then we're mixing ansible and puppet on deploy which will be fun if folks abuse it13:47
*** owalsh-afk has quit IRC13:47
shardyslagle: yeah, I mean we could expose something exactly like upgrade_tasks13:47
shardybut it's not a great fit for the puppet profile abstraction we have atm13:47
slagleshardy: heh, yea. i was thinking maybe about wiring in something via the roles data13:47
shardymaybe it'll fit better in future with containers etc tho13:47
slaglee.g., extraposttasks: /some/nested/stack.yaml13:48
slaglethen have that rendered as a resource in the post steps13:48
*** rlandy has joined #tripleo13:48
shardyslagle: yeah we could do that, but I'd almost prefer to just have a list of ansible tasks13:48
shardysame as upgrade, could be a bit clearer?13:48
slagleyea, could work as well13:49
*** psahoo has quit IRC13:49
shardycschwede_: thanks, I'll reply on the review, and should we then raise a bug to track refining this in future?13:49
shardyI'm not trying to block the backport given it's already landed on master13:49
shardyI'd just like to see it improved in future13:50
jtomasekd0ugal: Hi, any idea on this error? https://bugs.launchpad.net/tripleo/+bug/166563713:50
openstackLaunchpad bug 1665637 in tripleo "Nodes assignment workflow fails" [High,Triaged]13:50
cschwede_shardy: as you wish - i would start working on it without a bug on it's own13:50
mcorneamarios: shardy I'm trying to skip the preupgrade validations but it looks that the validations are still running. I used this parameter:   SkipUpgradeConfigTags: [validation]13:50
openstackgerritJohn Trowbridge proposed openstack/tripleo-ui master: DO NOT MERGE, used in CI ONLY  https://review.openstack.org/42202513:50
slaglewe already need something like a "stop_tasks" for scaling down services. could also have a extra_tasks13:50
slagleshardy: it could all work the same as upgrade_tasks13:50
*** jaosorior has quit IRC13:50
shardyslagle: yeah, we could use the exact same approach13:50
EmilienMlike upgrade_tasks would call stop_tasks and start_tasks?13:51
EmilienMso we don't duplicate tasks13:51
shardywhich has the advantage that it's easy to dump out an ansible playbook for folks to drive directly, e.g for advanced users and development13:51
EmilienMand we would support scale-up/down & upgrades with the same snippets?13:51
*** jaosorior has joined #tripleo13:52
shardyEmilienM: yeah, the first step of any non-rolling upgrade would be to run the stop_tasks13:52
shardyonly atm those tasks are listed in upgrade_tasks13:52
shardyso we'd have to move them13:52
rbradyjtomasek: is this on master?13:53
openstackgerritPradeep Kilambi proposed openstack/instack-undercloud master: Set instance audit settings so nova sends notifications  https://review.openstack.org/43545213:53
*** prateek has quit IRC13:53
mariosmcornea: yeah i've used it successfully before fwiw (you will see the tasks in the logs, but it says 'skipped' or should do13:53
openstackgerritJohn Trowbridge proposed openstack/tripleo-quickstart master: Fix gate-check test patch  https://review.openstack.org/43545313:53
*** pradk has joined #tripleo13:53
rbradyjtomasek: is this when using python-tripleoclient or the UI?13:53
jtomasekrbrady: I've installed latest undercloud 4 days ago, I can compare the code of that action13:53
jtomasekrbrady: UI13:54
openstackgerritMartin Mágr proposed openstack/tripleo-heat-templates master: Add logging agents deployment to CI  https://review.openstack.org/41021013:54
openstackgerritMartin Mágr proposed openstack/tripleo-heat-templates master: Add monitoring agents deployment to CI  https://review.openstack.org/43393113:54
jtomasekrbrady: cli does not use that action afaik13:54
mariosmcornea: are you sure they are still running and that the param is set corectly in the env file etc?13:54
*** Vijayendra has joined #tripleo13:54
*** owalsh-afk has joined #tripleo13:54
mcorneamarios: well, the services are stopped so it fails. this is the env that I used: http://paste.openstack.org/show/599407/13:55
mariosmcornea: which check is it failing with is it pacemaker?13:56
rbradyjtomasek: can I see the js code that calls this?13:56
*** jaosorior has quit IRC13:56
*** jaosorior has joined #tripleo13:56
jtomasekrbrady: yes, one sec13:56
*** dprince has quit IRC13:57
mcorneamarios: no, it failed on cinder-api13:57
jtomasekrbrady: https://github.com/openstack/tripleo-ui/blob/master/src/js/actions/NodesActions.js#L30313:58
mariosmcornea: ok thats strange i ran it multiple times with all the services down. the only problem i had was with the default pacemaker services still being checked so added PacemakerResources: [] also to the env (is why i was asking)13:58
d0ugaljtomasek: looking13:58
*** sasha2 has joined #tripleo13:59
d0ugaljtomasek: oh, rbrady may have beaten me. /me reads the conversation13:59
*** abregman has joined #tripleo13:59
rbradyd0ugal: I'm still just looking, no solution yet13:59
mariosmcornea: has it previously been working (is this a regression i didn't get yet on my d/stream env)?14:00
rbradyd0ugal, jtomasek: what stuck out to me as odd from the error paste is " __init__() got an unexpected keyword argument 'url'"14:00
*** dprince has joined #tripleo14:00
*** karimb has joined #tripleo14:01
mcorneamarios: no, it didn't work for me, this is the first time I actually test it14:01
mariosmcornea: sec lemme find the review i seem to recall there was a depends-on maybe you're missing something in the env (but it is all landed by now so not sure)14:03
d0ugalrbrady, jtomasek - any idea where url comes from?14:03
openstackgerritJames Slagle proposed openstack/tripleo-heat-templates master: Add environment for deployed-server with pacemaker  https://review.openstack.org/43445214:03
openstackgerritJames Slagle proposed openstack/tripleo-heat-templates master: Generate Pre/Post Puppet Tasks for all roles  https://review.openstack.org/43445114:03
jtomasekd0ugal: I estimate actions/base.py14:04
d0ugaljtomasek, rbrady - oh, so it does. Sorry, I thought this was a Mistral action14:04
*** salmankhan has joined #tripleo14:05
d0ugaljtomasek, rbrady: why isn't the CLI broken?14:05
jtomasekd0ugal: CLI does not use this workflow14:05
*** abregman has quit IRC14:05
d0ugaljtomasek: but I hope the CLI at least uses the compute client somewhere?14:05
*** axisys_ has quit IRC14:05
d0ugalmaybe not?14:05
*** gbarros has quit IRC14:05
shardycschwede_: comments added to https://review.openstack.org/#/c/435191/ and I raised https://bugs.launchpad.net/tripleo/+bug/1665641 to keep track of this14:05
openstackLaunchpad bug 1665641 in tripleo "Swift ringbuilding logic coupled to Controller/ObjectStorage roles" [Medium,Triaged] - Assigned to Christian Schwede (cschwede)14:05
mariosmcornea: so this is the thing that added the skip upgrade config param https://review.openstack.org/#/c/424072/ and it depends on a ansible heat-agent change at https://review.openstack.org/#/c/424221/ in my upgrade init i have : http://paste.openstack.org/raw/599409/ (i pull that patch onto my env, the second one for ansible agent)14:06
*** axisys has joined #tripleo14:06
jtomasekd0ugal: hm, good question14:06
mariosmcornea: now that change is 3 weeks old so you may already have it? but is worth sanity checking14:06
mcorneamarios: yep, checking. thanks14:06
shardymarios: yeah I had to pull it in directly because there was a problem with the packaging building from the (old) heat-templates repo14:06
d0ugaljtomasek: no, it seems not14:07
shardynot yet checked if that's fixed14:07
d0ugaljtomasek: the compute client is only used for role parameters.14:07
d0ugaljtomasek: okay, I've got a fix incoming for you. /cc rbrady14:07
jtomasekd0ugal: ok, great, thanks!14:07
mcorneamarios: hm, I'm missing https://review.openstack.org/#/c/424221/14:08
mariosack thanks shardy14:08
mariosmcornea: cool that should be it then14:08
mariosmcornea: you can just put in the upgrade init14:08
mcorneamarios: but why didn't it land as part of the upgrade?14:08
d0ugaljtomasek: I am going by the assumption that this code works, but help testing it shortly would be good.14:09
mariosapparently shardy said there was an issue with package build i don't know for sure14:09
*** mdnadeem has quit IRC14:09
shardymcornea: the heat agents moved to a different repo fairly recently14:09
cschwede_shardy: thx! so it basically comes down to two things if i understood correctly: 1) getting rid of the embedded shell script, and if possible integrating this into puppet-tripleo (which sounds good to me) and 2) removing the hardcoded rolenames, maybe using a similar approach to slagle's patch https://review.openstack.org/#/c/434451/ ?14:09
shardymcornea: and the packaging was building from the old repo last time I looked14:10
shardynumber80 was looking into it IIRC but I've not checked the latest status14:10
shardycschwede_: I think if you can do (1) you don't need the task additions to t-h-t at all14:10
shardyyou just do the logic via puppet at the appropriate step of the deployment14:11
shardyhttps://github.com/openstack/puppet-tripleo/blob/master/manifests/profile/base/swift/ringbuilder.pp#L8314:11
cschwede_shardy: ah yes, that makes sense. ok, thx for clarification, i'll work on that14:11
shardycschwede_: like, you rework that logic there to work with your tarball based approach14:11
shardythen it should "just work" with composable roles (in theory ;)14:12
mariosshardy: am doing a run now with https://review.openstack.org/#/c/435339/ did it work ok for you? I restarted httpd and assuming the placement error goes away hoping to isolate that one https://bugs.launchpad.net/tripleo/+bug/166532814:12
openstackLaunchpad bug 1665328 in tripleo "Newton->Ocata upgrade fails during overcloud.AllNodesDeploySteps.AllNodesPostUpgradeSteps.ControllerDeployment_Step3.0 because httpd is stopped" [High,Triaged]14:12
cschwede_shardy: yes, sounds like an much cleaner approach to me14:12
shardymarios: sorry I spent all morning on CI patches, about to try it now14:12
mariosshardy: ack cool just checking it started about ~18  mins ago here so14:12
marioswe'll see in a bit hopefully :)14:13
shardymarios: hehe, sounds good - let me know as my next step is to wire nova upgrades in via CI :)14:13
pandajaosorior: we have promoted images again14:13
*** chlong has joined #tripleo14:15
mcorneashardy: marios thanks, it looks that that's the issue. I have outdated heat agents14:15
jaosorioryay14:15
mariosshardy: well it installs ok but we have to wait for the puppet http://paste.openstack.org/raw/599412/14:15
jtomasekd0ugal: ok, will do14:16
shardymarios: Ok, well that proves I didn't typo the yum task at least :)14:16
mariosshardy: its still a novelty and i like seeing these playbook runs in the logs :D14:16
mariosmcornea: great thanks for update and testing that14:16
shardyyeah it's working pretty nicely I think14:16
d0ugaljtomasek: this must have been broken for a while?14:16
shardyexcept when a task hangs14:17
shardywe may need to add a timeout somewhere14:17
jtomasekd0ugal: may be, I had relatively old undercloud untill recently14:17
jtomasek*until14:17
mariosshardy: don't think i've had that yet i mean they've failed plenty times (especially before i was skipping the validation for example)14:17
openstackgerritDougal Matthews proposed openstack/tripleo-common master: Update the Nova client creation  https://review.openstack.org/43546714:17
mariosshardy: but not hanging14:17
d0ugaljtomasek: ^14:17
d0ugaljtomasek: currently untested :)14:18
*** lucas-hungry is now known as lucasagomes14:18
*** jaosorior has quit IRC14:22
jtomasekd0ugal: tested now, works fine, thanks14:24
d0ugaljtomasek: woo :)14:25
d0ugalrbrady: can I haz a review?14:25
*** owalsh-afk is now known as owalsh14:25
*** dmacpher has joined #tripleo14:26
*** jpena|lunch is now known as jpena14:28
*** gatuus has joined #tripleo14:29
*** jaosorior has joined #tripleo14:30
*** tzumainn has joined #tripleo14:31
*** rmcadams has quit IRC14:34
*** pradk has quit IRC14:34
*** fultonj has quit IRC14:34
*** rmcadams has joined #tripleo14:34
*** fultonj has joined #tripleo14:35
*** artom has joined #tripleo14:35
*** gatuus has quit IRC14:37
*** dprince has quit IRC14:37
*** eck`gone is now known as eck`14:37
*** dprince has joined #tripleo14:38
*** amoralej is now known as amoralej|lunch14:41
*** trown is now known as trown|brb14:43
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: remove duplicate network_environment from undercloud-deploy role  https://review.openstack.org/43492414:46
openstackgerritAna Krivokapic proposed openstack/python-tripleoclient master: Add plan export command  https://review.openstack.org/42585814:46
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Allow direct override of the overcloud's DNS servers  https://review.openstack.org/43496314:46
*** paramite has quit IRC14:46
trozetEmilienM, shardy: can you please review https://review.openstack.org/#/c/427436/ ?14:46
*** trown|brb is now known as trown14:46
*** ealcaniz has joined #tripleo14:46
openstackgerritJohn Eckersberg proposed openstack/puppet-tripleo master: WIP Qpid dispatch router puppet profile  https://review.openstack.org/42571014:47
*** ealcaniz has quit IRC14:50
*** akrivoka has quit IRC14:51
*** dtantsur is now known as dtantsur|afk14:53
bandinimwhahaha: were the neutron restarts also caused by the norpm virtual thing, or those are still to be understood/looked at?14:55
EmilienMtrozet: one14:56
EmilienMdone14:56
mwhahahabandini: I didn't see those when. I reran step514:56
trozetEmilienM: ty14:56
bandinimwhahaha: ack will rerun the whole test with all your patches so far14:57
openstackgerritFeng Pan proposed openstack/puppet-tripleo master: Add VPP service  https://review.openstack.org/42743614:57
fpanEmilienM: thanks14:58
*** abehl has quit IRC14:59
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Add "install" option to modify-image role  https://review.openstack.org/43480314:59
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Enable novajoin/TLS-everywhere deployment  https://review.openstack.org/42231014:59
*** jerrygb has quit IRC14:59
*** jerrygb has joined #tripleo15:00
*** adarazs is now known as adarazs_afk15:00
*** pradk has joined #tripleo15:03
*** jerrygb has quit IRC15:04
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Enable novajoin/TLS-everywhere deployment  https://review.openstack.org/42231015:05
openstackgerritJiri Tomasek proposed openstack/tripleo-ui master: Cleanup Nodes selectors  https://review.openstack.org/43151515:07
openstackgerritJiri Tomasek proposed openstack/tripleo-ui master: Update DeploymentPlan page node counts  https://review.openstack.org/43344915:07
openstackgerritAndy Smith proposed openstack/puppet-tripleo master: Use rpc and notify transport_url for oslo_messaging backends  https://review.openstack.org/42548415:08
*** gbarros has joined #tripleo15:11
*** morazi has joined #tripleo15:14
openstackgerritMerged openstack/tripleo-quickstart master: Fix gate-check test patch  https://review.openstack.org/43545315:16
*** saibarspeis has quit IRC15:16
openstackgerritJiri Tomasek proposed openstack/tripleo-ui master: Update DeploymentPlan page node counts  https://review.openstack.org/43344915:16
*** jlinkes_ has joined #tripleo15:16
openstackgerritNuman Siddique proposed openstack/tripleo-heat-templates master: ovn: Add missing configurations required  https://review.openstack.org/43302515:17
*** amoralej|lunch is now known as amoralej15:18
*** jlinkes__ has quit IRC15:20
openstackgerritJiri Tomasek proposed openstack/tripleo-ui master: Add HorizontalNumberInput  https://review.openstack.org/43428215:21
openstackgerritJiri Tomasek proposed openstack/tripleo-ui master: Add HorizontalNumberInput  https://review.openstack.org/43428215:21
*** limao has quit IRC15:23
*** saneax is now known as saneax-_-|AFK15:24
jristhonza: what should be done to injectIntl on this class?15:25
jristhonza: https://review.openstack.org/#/c/433179/2/src/js/components/nodes/NodesTable.js15:25
*** jlinkes__ has joined #tripleo15:25
*** bkopilov has joined #tripleo15:26
*** bkopilov_ has joined #tripleo15:26
jristjtomasek, honza  - can this be simplified15:27
jrist        let checkBoxValue = this.state.checkedEnvironment === environment.get('file')15:27
jrist          ? true15:27
jrist          : false;15:27
jristto just... let checkBoxValue = this.state.checkedEnvironment === environment.get('file')15:27
jristsince that returns true:false15:27
*** karimb has quit IRC15:28
jtomasekjrist: yes15:28
honzajrist: haha yes15:28
jpichjrist, honza: Very curious about the answer as well, I just moved what was already there around, and with my limited javascript skills unfortunately just removing 'default' doesn't seem to work :) Would appreciate any hint/advice on what is expected instead15:28
*** jlinkes_ has quit IRC15:28
*** bnemec is now known as beekneemech15:30
jristmy limited ES6 knowledge tells me that your export is fine15:30
jtomasekhonza: I think I disagree, why not to define a default export in a module?15:30
jristyou need a default15:30
honzabecause you're exporting more than one thing15:31
jtomasekjrist: yuo dont 'need' it:)15:31
jristjtomasek: conventionally15:31
jristhonza: you are?15:31
jristhonza: I don't think so15:31
honzaright below the offending line?15:31
honzaexport class NodesTable... ?15:31
jtomasekhonza: yeah, 2 things are exported but making one of them default seems fine to me15:31
jristyeah15:31
jristhonza: but one of them is default15:31
jristif you imported that class15:32
jristyou could import without importing NodesTableCheckBoxCell15:32
jristimport {default as foo} from 'lib';15:32
jristor15:32
jristimport foo from 'lib';15:32
honzaLOL i don't actually know how to do what i want you to do!15:32
*** eck` is now known as eck`gone15:33
jristwhat are you wanting to do?15:33
*** gbarros has quit IRC15:33
jristhas to do with imports?15:33
jristhttps://tc39.github.io/ecma262/#sec-imports15:33
openstackgerritJames Slagle proposed openstack/python-tripleoclient master: Show stack failures on error  https://review.openstack.org/43549115:33
jristif you don't specify a named imports, it uses a default15:34
honzaOne class per file => export it as default; more than one class in a file => no defaults15:35
*** cylopez has quit IRC15:36
jtomasekhonza: is that convention expressed somewhere? It is valid in some cases, but to me in some cases (such as this) it is fine to export the main component (which carries the same name as module) as default15:37
honzajtomasek: true!  which brings up the point that those other things shouldn't be in that file :)15:37
*** cylopez has joined #tripleo15:38
jristthat's unrelated to this patch tho15:38
honzajtomasek: i only saw SO comments about that convention, nothing official15:38
jtomasekhonza: as I mentioned in the review, that other component is there because it is considered 'private'15:38
jristthe official spec doesn't say https://tc39.github.io/ecma262/#sec-exports15:38
honzasomething like, export NodesTable = injectIntl(NodesTable);15:38
jristheh15:39
openstackgerritJames Slagle proposed openstack/tripleo-heat-templates master: Sort ResourceGroup resource list  https://review.openstack.org/43512115:39
jristthat's even more unconventional honza15:39
socialbandini: it didn't run the 99-empty-iptables at all15:39
jtomasekhonza: oh, wait that is something unrelated to what I thought we're talking about:)15:39
honzajrist: well, yeah, that's obviously wrong15:39
jristjtomasek: no I think it is related15:40
jristjtomasek: he wants to export without having a default15:40
honzajtomasek: i don't know the syntax for what i'm proposing :)15:40
beekneemechEmilienM: panda: derekh: If we merge https://review.openstack.org/#/c/430372/ the 5.5 GB of CentOS images will go away.15:40
honzareading the spec now15:40
bandinisocial: mmh on what tripleo/env are you building the image?15:40
beekneemechNote that it's not tested by ci, so the results don't matter.15:40
gregworkis there a list of 'safe' options to choose in the tripleO web UI installer.  We have been struggling to get an overcloud deployed via this method but depending on the feature we pick in the plan it causes a different failure15:41
socialbandini: delorean manual deploy I just set DIB_YUM_REPO_CONF USE_DELOREAN_TRUNK=0.. and   openstack overcloud image build --all15:41
gregworkbefore we give up on it entirely and go back to using the cli method :/15:41
jtomasekhonza: ah, I see now, not good imho anyway:)15:41
EmilienMbeekneemech: done15:42
jtomasekhonza: maybe export injectIntl(NodesTable) as NodesTable15:42
beekneemechEmilienM: Thanks15:42
jtomasekhonza: but no, imho exporting default is just fine in this case15:42
openstackgerritGabriele Cerami proposed openstack-infra/tripleo-ci master: [WIP] Quickstart transition: replace toci_gate_test.sh  https://review.openstack.org/43102915:43
*** eck`gone is now known as eck`15:44
*** gbarros has joined #tripleo15:44
socialbandini: Expanded element dependencies to: pip-and-virtualenv-override rpm-distro puppet-modules install-bin pip-manifest dib-run-parts pkg-map os-svc-install dhcp-all-interfaces puppet overcloud-base source-repositories package-installs os-refres15:44
jtomasekhonza: also yeah, it is unrelated to the patch15:44
socialh-config os-apply-config dynamic-login centos7 grub2 os-net-config hiera enable-packages-install dib-init-system install-types install-static os-collect-config openssh-server truncate-nova-config stable-interface-names svc-map dib-python15:44
socialelement-manifest openvswitch overcloud-opstools base manifests overcloud-controller redhat-common overcloud-ceph-storage runtime-ssh-host-keys baremetal overcloud-full cache-url hosts yum overcloud-compute15:44
jristjtomasek: that doesn't work either15:44
pandabeekneemech: so we are not using them at all ? I can remove them when this finishes merging ?15:45
beekneemechpanda: Exactly.  None of those images have been requested as far back as the Apache logs go.15:45
beekneemechWe cache the image on the proxy server instead.15:46
jristjtomasek, honza you could injectIntl at the top of the function15:47
pandabeekneemech: what refreshes the local copy of tripleo-ci in the mirror server15:48
panda?15:48
bandinisocial: ok so it seems (i'm no CLI guy, so take this with a pinch of salt ;) that build --all is deprecated and if you use that it will use some older list of elements15:48
jtomasekjrist: ugly:)15:49
jristagreed15:49
jtomasekhonza, jrist: section 3.3. http://www.2ality.com/2014/09/es6-modules-final.html describes using default + named exports as perfectly fine and often used15:49
socialbandini: ah I was expecting this, do we have some updated readme?15:49
jristjtomasek: yeah I was reading that as part of my argument :)15:49
shardymarios: Hey, can you pls share your environment for upgrade including nova?15:50
shardymarios: in particular I'd like to see the service ordering in your ControllerServices15:50
shardyhttps://github.com/openstack/tripleo-heat-templates/blob/master/puppet/services/nova-api.yaml#L19915:50
shardyfor me, that doesn't work because by the time it runs rabbit etc have stopped15:50
jristjtomasek: that page is extremely helpful15:50
bandinisocial: I found this https://review.openstack.org/#/c/385530/ and the deprecation should show up in the --help15:50
shardymarios: so I think we need to move it to step1 (or possibly even the rolling batch upgrades in future)15:51
honzajtomasek: it's funny because i actually read that post15:51
socialbandini: ack  --config-file /usr/share/openstack-tripleo-common/image-yaml/overcloud-images.yaml15:51
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates master: Add checks in ansible upgrade tasks for CephMon and CephOSD  https://review.openstack.org/43550215:51
bandinisocial: yep. let's see if this works15:51
jtomasekhonza: you could do   const NodesTable = injectIntl(NodesTable...)15:51
jristew15:51
jristjpich: are you following all of this? :)15:52
mariosshardy: sure - I am using d/stream, env, lemme fpaste what it looks like sec  (after getting puppet-nova it got further now is ongoing on the latest run)15:52
jpichjrist: I am, I am :) Just reading the arguments, since I'm unable to comment15:52
*** kbyrne has joined #tripleo15:52
jtomasekhonza: and then export { NodesTable, NodesTableCheckBoxCell }15:52
jpichfor now15:52
jtomasekat the end15:52
beekneemechpanda: Part of the mirror-server puppet is pulling the latest tripleo-ci repo.  So it will happen at most 30 minutes after the patch merges.15:53
honzajtomasek: what do you mean it's unrelated to the patch?  a patch that adds i18n should change how things are exported15:53
beekneemech(that goes for basically all of our infrastructure vms, FTR)15:53
pandabeekneemech: ok, I see that now, thanks.15:53
jtomasekhonza: I meant that that patch does not mean to change the way things are exported, it just moves it from class definition to the bottom so it matches the convetion of exporting wrapped components15:54
jristyeah ^15:54
jtomasekbut it is totally fine to have this discussion15:54
honzaand adds a default in the process15:54
jtomasekhonza: no15:54
mariosshardy: so close... now i said that it failed like Feb 17 15:54:15 overcloud-controller-1.localdomain os-collect-config[981956]: Error: /Stage[main]/Aodh::Db::Sync/Exec[aodh-db-sync]: Failed to call refresh: aodh-dbsync --config-file /etc/aodh/aodh.conf returned 1 instead of one of [0]15:54
mariosshardy: so, do you want to see roles_data?15:55
jtomasekhonza: there already was export default class NodesTable15:55
honzajtomasek: huh?15:55
shardymarios: Oh you're just using the default roles_data?15:55
jristand there already is a default by default15:55
shardymarios: No worries if so, I can copy that :)15:55
honzawelp15:55
openstackgerritMerged openstack/tripleo-heat-templates stable/ocata: Add Newton to Ocata UpgradeInitCommonCommand  https://review.openstack.org/43519015:55
honzajtomasek: jrist: jpich: ok, that's the best argument, changing my vote now15:55
jristwoot15:56
jtomasekhonza: thanks for bringing this up!15:56
jristyeah15:56
honzai missed the removal of the "default" in the class definition15:56
jrist+1 to that15:56
jristwas really good to discuss15:56
jpichAlright!15:56
openstackgerritMerged openstack/tripleo-heat-templates master: Add docker_puppet_tasks initialization on primary node  https://review.openstack.org/42656515:56
honzasorry jpich :(15:56
mariosshardy: yeah and whole bunch of patches like http://paste.openstack.org/show/599430/ most of which you'll have because upstream (except last couple perhaps)15:56
jristdon't be sorry15:56
openstackgerritMerged openstack/tripleo-heat-templates stable/mitaka: Ensure working compute node during the whole migration.  https://review.openstack.org/42809315:56
openstackgerritMerged openstack/puppet-tripleo stable/ocata: Update .gitreview for stable/ocata  https://review.openstack.org/43487415:56
jrist+1 workflow plz15:57
jristbackport plz15:57
openstackgerritMerged openstack/puppet-tripleo stable/ocata: Add virtual_packages support to norpm provider  https://review.openstack.org/43518115:57
jpichhonza: No worries, discussion and learning happened :o15:57
*** abehl has joined #tripleo15:57
jristbigly15:57
jpichjrist: It still needs a change to disable the tests actually, it was we talked about with jtomasek the other day (it's dependent on his patch with the new dep)15:57
shardymarios: cool, thanks - my roles_data is customized which I think is why this fails15:57
jristjpich: within that patch?15:58
jtomasekjpich: I am updating that patch now15:58
*** artom has quit IRC15:58
shardymarios: I'll try moving the nova-api upgrade, as that hangs forever failing to restart things15:58
*** Goneri has quit IRC15:58
mariosshardy: well, fwiw i also manually recovered services before this latest run like for i in $(systemctl | grep openstack | grep fail | awk '{print $2}'); do systemctl restart $i; done  (and made sure nova-api and all things were running)15:58
jpichjrist: yeah under related change, "Make connected component" etc. The test update depends on that new stuff... which shall be removed shortly :)15:58
amoralejhi, i've found errors in undercloud install "(os-refresh-config) [ERROR] Could not lock /var/run/os-refresh-config.lock. [Errno 11] Resource temporarily unavailable"15:58
jpichjtomasek: Cool, thank you! I will follow up15:58
amoralejhave you seen something similar ?15:58
derekhbeekneemech: lgtm, I'm sure we used those images at some stage from the mirror server, something changed along the line somewhere15:59
*** yamahata has joined #tripleo15:59
mariosshardy: ok i can grab the patch on a next run once you post it and test it as well16:01
openstackgerritJiri Tomasek proposed openstack/tripleo-ui master: Make connected components update on lang. switch  https://review.openstack.org/43236516:02
beekneemechamoralej: I believe that means the file is already locked.  Was there a previous run of the install on this system?16:04
openstackgerritMichele Baldessari proposed openstack/tripleo-quickstart-extras master: Collect /var/lib/pacemaker/cib/  https://review.openstack.org/42707416:04
*** kbyrne has quit IRC16:04
amoralejbeekneemech, it's in CI jobs16:04
*** sdake has quit IRC16:04
amoralejnop16:04
amoralejno previous runs16:05
*** gbarros has quit IRC16:05
amoralejexample, https://ci.centos.org/artifacts/rdo/jenkins-tripleo-quickstart-promote-master-delorean-minimal_pacemaker-486/undercloud/home/stack/undercloud_install.log.gz16:05
*** sdake has joined #tripleo16:05
socialgah ERROR: At least 7680 MB of memory is required for undercloud installation.  A minimum of 8 GB is recommended. Only detected 6809 MB16:06
socialcan we at least make these configurable so that I don't have to patch it out?16:06
beekneemechsocial: You really want to run with less than 7 GB of memory?16:06
beekneemechI'm pretty sure that's going to be an issue.16:06
*** kbyrne has joined #tripleo16:07
beekneemechEven 8 is not a realistic number, but we get away with it in CI because the vms are short-lived.16:07
beekneemechamoralej: Then I don't know.16:08
amoralejmaybe some kind of race condition16:08
*** adarazs_afk is now known as adarazs16:09
socialbeekneemech: I have swap backed by NVMe, yes I do want to run it on even less ram. I don't understand why it actually needs so much16:11
beekneemechsocial: Ah, maybe we need to consider swap in that check.16:11
beekneemechsocial: It needs so much because we run a lot of stuff on the undercloud. :-)16:12
*** bkopilov_ has quit IRC16:13
*** bkopilov has quit IRC16:13
beekneemechamoralej: 0 hits on that error in tripleo-ci in the past 30 days: http://logstash.openstack.org/#dashboard/file/logstash.json?query=build_name%3A%20*tripleo-ci*%20AND%20build_status%3A%20FAILURE%20AND%20message%3A%20%5C%22Could%20not%20lock%20%2Fvar%2Frun%2Fos-refresh-config.lock%5C%2216:13
beekneemechIf it's a race it's an extremely rare one.16:14
openstackgerritJulie Pichon proposed openstack/tripleo-ui master: Add i18n to Nodes table  https://review.openstack.org/43317916:14
beekneemechNot sure how it could race though.  We only run os-refresh-config once.16:14
amoralejbeekneemech, rdo-ci is particularly slow16:15
*** abehl has quit IRC16:16
*** artom has joined #tripleo16:18
*** jpena is now known as jpena|brb16:20
*** karimb has joined #tripleo16:21
*** pcaruana has quit IRC16:21
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo master: Create /etc/my.cnf.d/tripleo.cnf with proper bind-address  https://review.openstack.org/43142416:22
jristjtomasek: is 433449 ready to go?16:22
jristjtomasek: i.e. workflow16:22
*** gbarros has joined #tripleo16:23
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates master: Make the DB URIs host-independent for all services  https://review.openstack.org/43142516:23
openstackgerritDougal Matthews proposed openstack/tripleo-common master: Use kwargs to pass in 'error' to a Mistral Result  https://review.openstack.org/43310616:28
openstackgerritDougal Matthews proposed openstack/python-tripleoclient master: Respond to multiple messages from workflow executions  https://review.openstack.org/40617816:29
openstackgerritDougal Matthews proposed openstack/python-tripleoclient master: Add a new command to get the overcloudrc files  https://review.openstack.org/40435416:29
*** jaosorior has quit IRC16:30
jristnice, thanks d0ugal ^16:30
jristd0ugal: is that ready to be reviewed now?16:31
jristd0ugal: or still workflow -1 ?16:31
d0ugaljrist: which one?16:31
jristthe one I just pointed to16:31
jrist40435416:31
d0ugaljrist: I need to finish the unit tests16:31
d0ugaljrist: but it can otherwise we reviewed/tested if you want!16:32
jristso I should -1 asking for more unit tests? :)16:32
d0ugaljrist: ... if you want16:32
jrist:)16:32
gfidentechem marios to make things easier I added some more validations as well16:32
jristd0ugal: makes me look good to have more -1s16:32
jristd0ugal: :) jk16:32
*** leanderthal is now known as leanderthal|afk16:32
d0ugaljrist: damn, you only have one open patch that I can -116:33
*** aufi has quit IRC16:33
jristd0ugal: haha16:34
jristmutual -1s16:34
chemgfidente: you're the oneliner master!16:34
chemgfidente: glad I've met you at last :)16:35
gfidentechem no that is a copy/paste16:35
*** jlinkes__ has quit IRC16:35
d0ugalcopy/paste master!16:35
gfidentewill you guys stop blaming me for everything16:35
*** rcernin has quit IRC16:35
gfidenteI add bugs I am the author of16:35
openstackgerritLucas Alvares Gomes proposed openstack/tripleo-docs master: Add some documentation about VirtualBMC and pxe_ipmitool  https://review.openstack.org/43428616:36
chemgfidente: at least we don't have any escape problem ... or do we :) ?16:37
openstackgerritBen Nemec proposed openstack/python-tripleoclient master: Call undercloud install function directly  https://review.openstack.org/43114516:37
mariosgfidente: where i mean did you post something?16:38
openstackgerritBen Nemec proposed openstack/python-tripleoclient master: Call undercloud install function directly  https://review.openstack.org/43114516:38
gfidentechem I guess we'll have to test it16:39
gfidentebut the new hook should have helped us a lot with that16:39
*** sudipto_ has quit IRC16:39
*** sudipto has quit IRC16:39
gfidentemarios https://review.openstack.org/#/c/43550216:39
chemgfidente: the fact that it's used elsewhere give some level of confidence.16:39
*** cylopez has quit IRC16:40
*** gatuus has joined #tripleo16:40
*** ccamacho has quit IRC16:40
chemgfidente: but I have no clue of what it does, it looks like assembler16:40
mariosgfidente: cool16:41
mariosgfidente: but can we continue to blame you for things, anyway?16:41
*** tesseract has quit IRC16:41
chemmarios: gfidente haha16:41
gfidentemarios you have to have the karma for that16:42
mariosgfidente: :( well lets hope we get a happy ending after all ;)16:42
gfidente right I don't think you want to maintain that shell script forever16:43
*** tremble has quit IRC16:44
mariosgfidente: indeed even for those roles we do want the manual run we will use ansible later (the tasks should mostly be there already ) filed a placeholder this morning at https://bugs.launchpad.net/tripleo/+bug/166558516:44
openstackLaunchpad bug 1665585 in tripleo "Ocata to Pike operator driven ansible upgrade" [Undecided,New] - Assigned to Marios Andreou (marios-b)16:44
jtomasekjrist: yes16:45
jristjtomasek: thanks16:45
*** jpena|brb is now known as jpena16:48
*** thrash is now known as thrash|g0ne16:50
*** thrash|g0ne is now known as thrash|f00dz16:50
gfidentemarios ack thanks16:52
*** fragatina has quit IRC16:54
*** artom has quit IRC16:55
openstackgerritBen Nemec proposed openstack/instack-undercloud master: Include swap in memory check  https://review.openstack.org/43553216:56
beekneemechsocial: ^16:56
*** salmankhan has quit IRC16:57
beekneemechAnd if you don't want to wait for that to merge you can pull down the test rpm from https://repos.fedorapeople.org/repos/openstack-m/jenkins/rpm-build/435532/1/instack-undercloud-999.9.9-99999.noarch.rpm16:58
*** salmankhan has joined #tripleo16:58
beekneemechMight be a little simpler than editing the code.16:58
*** kjw3 has joined #tripleo16:58
*** ccamacho has joined #tripleo16:59
*** suuuper has quit IRC17:00
socialbeekneemech: thanks17:01
gfidenteshardy in upgrade_task I can register a value from a get_param value right?17:01
gfidentecause I want to add a conditional in a task based on a value given via parameter17:01
socialbeekneemech: now I'll get used to crying out loud here to get something17:01
shardygfidente: yes, you may need to use str_replace to add it to the string17:02
*** michapma_alt has quit IRC17:02
beekneemechsocial: I can't promise everything will be this easy to fix, but if there's a problem do let us know about it.17:03
gfidenteshardy ok I see17:04
gfidentetks17:04
shardyhttps://github.com/openstack/tripleo-heat-templates/blob/master/puppet/services/nova-conductor.yaml#L8017:06
shardygfidente: ^^ example17:06
gfidenteshardy cool, great17:06
gfidentebut this is probably even simpler I can use 'vars'17:06
*** panda is now known as panda|bbl17:07
*** salmankhan has quit IRC17:12
*** saibarspeis has joined #tripleo17:13
*** salmankhan has joined #tripleo17:16
*** pkovar1 has joined #tripleo17:16
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates master: Add checks in ansible upgrade tasks for CephMon and CephOSD  https://review.openstack.org/43550217:17
gfidenteshardy https://review.openstack.org/#/c/435502/2/puppet/services/ceph-osd.yaml ?17:17
*** pkovar has quit IRC17:18
*** artom has joined #tripleo17:19
shardyhttps://github.com/openstack/tripleo-heat-templates/blob/master/puppet/services/cinder-api.yaml#L15417:20
shardygfidente: looks good, or you could just tag the task with "validation"17:20
gfidenteyeah but then all validations will be skipped right?17:21
shardygfidente: yeah, if you skip that tag17:21
gfidentethe problem with that check is that it is not meant to be skipped17:21
gfidenteexcept when number of osds < 317:22
gfidenteso we need it for testing purposes, but outside testing it shouldn't be skippe17:22
shardygfidente: yeah but you could still use e.g SkipUpgradeConfigTags: ceph_osd_validation17:22
shardyto avoid adding a new parameter and the vars stuff17:22
shardyif you want17:22
shardyeither approach should work :)17:22
gfidenteah right any other tak17:22
gfidentetag17:22
gfidentethe param was there before17:23
gfidenteI agree though, moving it into a tag is better17:23
gfidenteI'll bring it up with chem and marios and see what they prefer :)17:23
gfidentethanks!17:23
mariosgfidente: lets talk next week /me hometime17:24
mariosbai17:24
gfidenteyes17:24
gfidentecool17:24
*** jpich has quit IRC17:24
*** artom has quit IRC17:26
*** athomas has quit IRC17:26
*** artom has joined #tripleo17:26
*** derekh has quit IRC17:27
*** zoli is now known as zoli|gone17:28
*** gbarros has quit IRC17:30
*** gbarros has joined #tripleo17:30
gfidentetty next week17:31
*** gfidente has quit IRC17:32
*** artom has quit IRC17:36
*** dprince has quit IRC17:38
*** dprince has joined #tripleo17:40
*** Goneri has joined #tripleo17:42
*** lucasagomes is now known as lucas-afk17:43
*** b00tcat has quit IRC17:44
openstackgerritMerged openstack/puppet-tripleo master: Add virtual_packages support to norpm provider  https://review.openstack.org/43501117:44
openstackgerritMerged openstack-infra/tripleo-ci master: Add converge update to major upgrade test  https://review.openstack.org/43425017:45
*** jerrygb_ has joined #tripleo17:46
*** rasca has quit IRC17:47
openstackgerritMerged openstack/instack-undercloud stable/ocata: Update .gitreview for stable/ocata  https://review.openstack.org/43487117:48
openstackgerritMerged openstack/tripleo-heat-templates stable/newton: backport multinode_major_upgrade.yaml environment  https://review.openstack.org/43540217:48
openstackgerritMerged openstack/tripleo-heat-templates stable/ocata: Add explicit swift check to tripleo_upgrade_node.sh  https://review.openstack.org/43533817:49
*** sasha2 has quit IRC17:50
*** jerrygb has joined #tripleo17:50
*** sasha2 has joined #tripleo17:51
*** jerrygb_ has quit IRC17:51
mwhahahashardy: I replying in Bug 1665717 but see Bug 163113317:54
openstackbug 1665717 in tripleo "newton->ocata upgrade hangs starting nova-api" [High,Triaged] https://launchpad.net/bugs/166571717:54
*** tosky has quit IRC17:54
openstackbug 1631133 in tripleo "nova's osapi_compute_workers is not valid (must be greater than 1)" [High,Fix released] https://launchpad.net/bugs/1631133 - Assigned to Dan Prince (dan-prince)17:54
mwhahahas/replying/replied/17:54
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Add "install" option to modify-image role  https://review.openstack.org/43480317:56
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Enable novajoin/TLS-everywhere deployment  https://review.openstack.org/42231017:56
*** yamahata has quit IRC18:01
*** jmelvin has quit IRC18:03
*** gbarros has quit IRC18:03
openstackgerritChristian Schwede proposed openstack/puppet-tripleo master: WIP: Migrate Swift ring handling from T-H-T to puppet-tripleo  https://review.openstack.org/43555618:06
cschwede_shardy: i think you had something like this in mind? WIP, just for early feedback18:06
*** artom has joined #tripleo18:07
cschwede_and the corresponding t-h-t would look sth like this: http://paste.openstack.org/show/599450/18:07
diana_clarkeAny tips on where to look for clues as to why this tripleo quickstart failed? Thanks! https://gist.github.com/dianaclarke/57a26f36d69111a2f1eef3c57ebd9ce418:07
*** sdake_ has joined #tripleo18:11
*** jpena is now known as jpena|away18:13
*** sdake has quit IRC18:14
*** thrash|f00dz is now known as thrash18:17
*** fragatina has joined #tripleo18:17
openstackgerritAthlan-Guyot sofer proposed openstack/tripleo-heat-templates stable/mitaka: Fix a typo in the nova db online data migrations during upgrade.  https://review.openstack.org/43556018:20
*** jprovazn has quit IRC18:21
*** pkovar1 has quit IRC18:22
*** pblaho has quit IRC18:23
openstackgerritAthlan-Guyot sofer proposed openstack/tripleo-heat-templates stable/mitaka: Fix a typo in the nova db online data migrations during upgrade.  https://review.openstack.org/43556018:23
*** trown is now known as trown|lunch18:24
chemEmilienM: oh boy, I recheck the patch, and I introduced a typo in the very last patch -> the correction https://review.openstack.org/435560  I'm sorry, I really though I was over.18:24
chembandini: ^18:26
openstackgerritAthlan-Guyot sofer proposed openstack/tripleo-heat-templates stable/mitaka: Fix a typo in the nova db online data migrations during upgrade.  https://review.openstack.org/43556018:33
*** yamahata has joined #tripleo18:38
openstackgerritHonza Pokorny proposed openstack/tripleo-ui master: Add missing i18n strings to the DataTable component  https://review.openstack.org/43556518:39
bandinichem: no worries. done18:43
openstackgerritHonza Pokorny proposed openstack/tripleo-ui master: Add missing i18n strings to the DataTable component  https://review.openstack.org/43556518:46
chembandini: thanks18:46
*** jmelvin has joined #tripleo18:49
*** milan_ has quit IRC18:50
*** owalsh has quit IRC18:55
-openstackstatus- NOTICE: Restarting gerrit due to performance problems19:05
*** jmelvin has quit IRC19:05
*** rbowen has quit IRC19:06
*** jmelvin has joined #tripleo19:09
*** sdake_ has quit IRC19:10
openstackgerritMerged openstack/tripleo-common stable/newton: Use Mistral to create Swift temporary URLs  https://review.openstack.org/43528419:13
openstackgerritMerged openstack/tripleo-heat-templates stable/ocata: Automatically backup and restore Swift rings from the undercloud  https://review.openstack.org/43519119:13
*** colonwq has quit IRC19:14
openstackgerritMerged openstack/tripleo-heat-templates stable/ocata: Deploy Manila with CephMDS in scenario004  https://review.openstack.org/43540819:14
openstackgerritMerged openstack/tripleo-common master: Open log in utf-8 to prevent UnicodeEncodeError  https://review.openstack.org/43452019:14
openstackgerritMerged openstack/tripleo-ui stable/ocata: Include j2 files in plan creation  https://review.openstack.org/43538219:15
openstackgerritMerged openstack/tripleo-ui stable/ocata: Add i18n to nodes page and buttons  https://review.openstack.org/43538119:15
openstackgerritMerged openstack-infra/tripleo-ci master: Remove openstack-heat-templates from bootstrap elements path  https://review.openstack.org/43510219:15
openstackgerritMerged openstack/tripleo-ui master: Add i18n to Register Nodes form  https://review.openstack.org/43425919:15
openstackgerritMerged openstack-infra/tripleo-ci master: Stop mirroring centos images  https://review.openstack.org/43037219:15
*** rbrady is now known as rbrady-afk19:16
*** yprokule has quit IRC19:18
*** eck` is now known as eck`gone19:19
*** salmankhan has quit IRC19:20
openstackgerritAlex Schultz proposed openstack/tripleo-common stable/ocata: Open log in utf-8 to prevent UnicodeEncodeError  https://review.openstack.org/43557219:22
*** colonwq has joined #tripleo19:30
*** owalsh has joined #tripleo19:39
*** mcornea has quit IRC19:39
*** owalsh is now known as owalsh-afk19:39
*** trown|lunch is now known as trown19:43
*** fragatina has quit IRC19:47
*** ccamacho has quit IRC19:48
*** gbarros has joined #tripleo19:49
*** dsneddon is now known as dsneddon_afk19:56
*** dsneddon has joined #tripleo19:58
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart master: Extend devmode to run with OVB  https://review.openstack.org/42544719:58
*** jmelvin has quit IRC20:01
*** abehl has joined #tripleo20:04
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart master: Modify call to quickstart with options for repo include  https://review.openstack.org/42544720:04
*** eck`gone is now known as eck`20:06
openstackgerritRajini Ram proposed openstack/tripleo-heat-templates stable/newton: Composable service support for Cinder Dell EMC Storage Center  https://review.openstack.org/43558120:06
*** jmelvin has joined #tripleo20:09
openstackgerritMerged openstack/diskimage-builder master: Fix requirements update  https://review.openstack.org/43277120:14
openstackgerritRajini Ram proposed openstack/tripleo-heat-templates stable/newton: Composable services support for Cinder Dell EMC PS Series  https://review.openstack.org/43558320:15
openstackgerritJohn Trowbridge proposed openstack/tripleo-quickstart-extras master: publish: don't fail in case the logs dir is a link to a directory  https://review.openstack.org/43495520:15
openstackgerritRajini Ram proposed openstack/puppet-tripleo stable/newton: Rebranding of Eqlx to Dell EMC PS Series  https://review.openstack.org/43558720:22
openstackgerritMerged openstack/tripleo-quickstart master: Replace "ara generate" by "ara generate html"  https://review.openstack.org/43320820:23
*** morazi has quit IRC20:23
openstackgerritRajini Ram proposed openstack/tripleo-heat-templates stable/newton: Composable services support for Cinder Dell EMC PS Series  https://review.openstack.org/43558320:24
*** panda|bbl is now known as panda20:28
*** salmankhan has joined #tripleo20:30
*** morazi has joined #tripleo20:30
*** salmankhan has quit IRC20:34
*** amoralej is now known as amoralej|off20:37
openstackgerritNuman Siddique proposed openstack/tripleo-heat-templates master: ovn: Add missing configurations required  https://review.openstack.org/43302520:37
*** saibarspeis has quit IRC20:38
*** morazi has quit IRC20:38
*** jerrygb has quit IRC20:44
*** dprince has quit IRC20:51
*** morazi has joined #tripleo20:52
*** fragatina has joined #tripleo20:55
*** mhenkel_ has joined #tripleo20:56
*** ansmith has quit IRC21:01
*** bkopilov has joined #tripleo21:04
*** bkopilov_ has joined #tripleo21:04
*** leanderthal|afk has quit IRC21:09
openstackgerritGabriele Cerami proposed openstack-infra/tripleo-ci master: [WIP] Quickstart transition: replace toci_gate_test.sh  https://review.openstack.org/43102921:11
*** ansmith has joined #tripleo21:13
*** gbarros has quit IRC21:13
*** chem has quit IRC21:15
*** salmankhan has joined #tripleo21:17
*** jayg is now known as jayg|g0n321:19
*** zaneb has quit IRC21:20
*** dbecker has quit IRC21:21
openstackgerritMerged openstack/tripleo-quickstart-extras master: Add scenario support to overcloud-deploy and config for 001  https://review.openstack.org/42482721:29
*** fragatin_ has joined #tripleo21:38
*** fragatina has quit IRC21:40
*** kjw3 has quit IRC21:44
*** gbarros has joined #tripleo21:46
*** morazi has quit IRC21:47
*** lblanchard has quit IRC21:52
*** lblanchard has joined #tripleo21:52
openstackgerritMerged openstack/tripleo-heat-templates stable/ocata: Apply puppet in non-controller script in step.  https://review.openstack.org/43531521:53
*** lblanchard has quit IRC21:56
*** jerrygb has joined #tripleo21:57
*** trown is now known as trown|outtypewww22:01
*** jerrygb has quit IRC22:03
*** fragatin_ has quit IRC22:16
*** fragatina has joined #tripleo22:17
*** rlandy has quit IRC22:18
*** sdake has joined #tripleo22:30
*** fultonj has quit IRC22:38
*** rhallisey has quit IRC22:40
*** thrash is now known as thrash|g0ne22:43
*** dsneddon has quit IRC22:48
*** liverpooler has quit IRC22:52
*** jd__ has quit IRC22:59
*** jerrygb has joined #tripleo23:00
*** jmelvin has quit IRC23:00
*** iranzo has quit IRC23:03
*** gatuus has quit IRC23:04
openstackgerritGabriele Cerami proposed openstack-infra/tripleo-ci master: [WIP] Quickstart transition: replace toci_gate_test.sh  https://review.openstack.org/43102923:23
*** panda is now known as panda|Zz23:23
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci master: remove test-environments/multinode_major_upgrade.yaml  https://review.openstack.org/42637523:29
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci master: remove test-environments/multinode_major_upgrade.yaml  https://review.openstack.org/42637523:29
*** salmankhan has quit IRC23:30
*** jkilpatr has quit IRC23:34
*** salmankhan has joined #tripleo23:37
*** chlong has quit IRC23:46
*** b00tcat` has quit IRC23:47
*** saneax-_-|AFK is now known as saneax23:51

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!