Friday, 2016-07-29

*** TSCHAK has joined #tripleo00:05
*** mburned is now known as mburned_out00:12
*** mburned_out is now known as mburned00:15
*** yamahata has quit IRC00:17
*** jcoufal has quit IRC00:19
*** jcoufal has joined #tripleo00:20
openstackgerritMatthew Thode proposed openstack/diskimage-builder: Optionally remove portage files  https://review.openstack.org/34853600:27
*** jcoufal has quit IRC00:28
*** dtrainor has quit IRC00:35
*** jcoufal has joined #tripleo00:39
*** limao has joined #tripleo00:40
*** yamahata has joined #tripleo00:46
*** limao_ has joined #tripleo00:59
*** limao has quit IRC01:02
*** jcoufal has quit IRC01:06
*** jcoufal has joined #tripleo01:08
*** yamahata has quit IRC01:10
*** saneax is now known as saneax_AFK01:10
*** jcoufal has quit IRC01:20
*** jcoufal has joined #tripleo01:20
*** tzumainn has quit IRC01:20
*** seanatcisco has joined #tripleo01:27
*** yamahata has joined #tripleo01:44
*** rlandy has quit IRC01:50
*** jcoufal has quit IRC01:51
*** jcoufal has joined #tripleo01:52
*** jcoufal has quit IRC02:12
*** yamahata has quit IRC02:15
*** seanatcisco has quit IRC02:17
*** seanatcisco has joined #tripleo02:18
openstackgerritBrent Eagles proposed openstack/tripleo-heat-templates: Properly direct MTU parameters and allow service defaults  https://review.openstack.org/33333302:19
*** dmacpher is now known as dmacpher-lunch02:40
*** seanatcisco has quit IRC02:43
*** dtrainor has joined #tripleo02:47
*** dtrainor has quit IRC02:59
*** thrash is now known as thrash|g0ne03:00
*** dmacpher-lunch is now known as dmacpher03:31
*** coolsvap_ has joined #tripleo03:56
*** coolsvap_ is now known as coolsvap03:57
openstackgerritMerged openstack/diskimage-builder: Generalize logic for skipping final image generation  https://review.openstack.org/33404203:59
*** links has joined #tripleo04:12
*** links has quit IRC04:13
*** links has joined #tripleo04:13
*** numans has joined #tripleo04:53
*** masco has joined #tripleo05:07
*** numans has quit IRC05:07
*** saneax_AFK is now known as saneax05:32
*** radek_ has joined #tripleo05:40
*** pgadiya has joined #tripleo05:50
saneaxkarthiks, https://review.openstack.org/#/c/333882/13/manifests/plugins/ml2/mech_driver.pp@5705:54
openstackgerritSanjay Upadhyay proposed openstack/tripleo-specs: tripleo-sriov  https://review.openstack.org/31387206:04
*** numans has joined #tripleo06:36
*** tesseract- has joined #tripleo06:37
*** pcaruana has joined #tripleo06:45
*** leanderthal|afk is now known as leanderthal06:49
*** dculik|gone has joined #tripleo06:59
*** ccamacho has joined #tripleo06:59
*** paramite has joined #tripleo06:59
*** jprovazn has joined #tripleo07:02
*** anshul has joined #tripleo07:05
bandinimorning07:07
ccamachomorning bandini! man yesterday I ended buying a portable a/c :P07:08
bandiniccamacho: ehehe in the end you do what you have to do to survive ;)07:09
*** zoli_gone-proxy is now known as zoliXXL07:09
ccamachoyeahp, I'm starting to think about selling Madrid's Sahara air bottled...07:10
bandiniehehehe, I will buy some for my winters ;)07:11
ccamachobandini and * I'm going to invest my time with composable aodh if needed some reviews just let me know07:12
bandiniccamacho: ah awesome. so I need a last ack here (and its dep): https://review.openstack.org/34731007:13
bandiniccamacho: after that goes in I will rebase the HA NG stuff on the aodh profile review07:13
bandiniso I will test that aodh profile one as well07:14
ccamachobandini in that case Ill test yours first, ha 1+1 works for you or ha 3+1 ?07:14
ccamachoafaik the aodh submission is not working.. so I need to check whats up there..07:15
bandiniccamacho: I always test HA 3+1 or 3+207:17
ccamachodon't have enough ram for 3+2 :P07:18
ccamacho3+1 then07:18
bandiniyeah RAM is always an issue :) I am lucky I can test on baremetal these days07:18
bandininot all the time, but often enough07:19
openstackgerrityolanda.robla proposed openstack-infra/tripleo-ci: Add --yes flags to heat stack-delete for liberty  https://review.openstack.org/34720207:19
*** akuznetsov has joined #tripleo07:22
*** davidlenwell has quit IRC07:22
*** gfidente has joined #tripleo07:23
*** gfidente has quit IRC07:23
*** gfidente has joined #tripleo07:23
ccamachobandini, **deploying one thing after the deployment, do you need any manual sanity check?07:25
bandiniccamacho: what do you mean?07:26
bandini(I am especially thick in the morning ;)07:27
marioshappy friday *07:27
ccamachoi.e. the overcloud pingtest or any other manual test that you might want to execute there07:28
bandiniccamacho: for the HA NG? nope07:28
bandinimarios: happy friday to you too! (and greetings from ANJ)07:28
ccamachomarios happy friday man! http://www.kappit.com/img/pics/201510_0922_addcd_sm.jpg07:28
ccamachobandini ack :)07:28
hewbroccamorning folks07:31
ccamachoo/07:32
*** mcornea has joined #tripleo07:33
*** ccamacho is now known as ccamacho|afk07:33
saneaxGM Folks07:33
bandinio/07:33
*** davidlenwell has joined #tripleo07:34
*** ChanServ sets mode: +v davidlenwell07:34
*** shardy has joined #tripleo07:39
*** florianf has joined #tripleo07:40
*** fzdarsky|afk has joined #tripleo07:44
openstackgerritGiulio Fidente proposed openstack/python-tripleoclient: Generate CephX keys via generate_overcloud_passwords  https://review.openstack.org/34841207:47
*** ccamacho|afk is now known as ccamacho07:51
*** jpich has joined #tripleo07:55
*** dbecker has joined #tripleo07:58
*** panda|Zz is now known as panda08:00
*** dbecker has quit IRC08:02
*** dbecker has joined #tripleo08:03
gfidentehey guys, maybe we have bash expertise around here :)08:04
gfidentequestion08:04
gfidenteif we have tool which doesn't output to stdout but can write into a file08:05
gfidentewhich syntax is best08:05
gfidente-o /dev/stdout  or  >(cat -) ?08:05
gfidenteor what else?08:05
gfidentesounds like outputting to /dev/stdout is easier but doesn't give any guarantee there won't be other processes doing the same mixing the output08:06
gfidenteso I'd prefer the >() syntax08:06
gfidente+1 ?08:06
ccamacho>() +108:07
gfidenteccamacho tks :)08:08
*** fragatina has quit IRC08:09
*** derekh has joined #tripleo08:09
ccamachogfidente but never echoing outputs  directly08:09
gfidenteccamacho never do >(echo) you mean?08:10
ccamachoas it will open/close the file each time you use it08:10
gfidentebandini sounds like we need someone to approve the nova constraints?08:10
ccamachoyeahp, try to save it into a var and then saving08:10
gfidenteccamacho then echoing yes08:11
gfidentethanks08:11
*** fzdarsky has joined #tripleo08:11
*** fzdarsky|afk has quit IRC08:11
ccamacho:)08:12
*** electrofelix has joined #tripleo08:13
bandinigfidente: yep we need one volunteer08:14
mariosgfidente: bandini gimme few looking at the nova one now08:14
bandinimarios++08:14
gfidentemarios https://review.openstack.org/#/c/347310/08:14
mariosyes and https://review.openstack.org/#/c/347309/08:14
gfidenteneeds to be merged with its dependency which won't pass alone08:14
gfidenteyeah08:14
gfidenteso it's an act of courage08:15
mariosgfidente: tis ok, bandini already pimped the change last night08:15
gfidenteyeah it's like the openstack-core change we had as well08:15
gfidentesame purpose, same problem08:15
bandinisame shit different day08:17
bandiniI have a mug with that phrase ;)08:17
*** jbadiapa has joined #tripleo08:18
gfidentebandini that's just one way to think about it08:18
mariosgfidente: bandini so simple enough lgtm guys... i count 10 constraints removed and 10 added ... :) i trust you've sanity checked them a bit more than me08:21
gfidenteplus the one moving the constraints into the role passes so we have some ci coverage08:21
ccamachobandini, my deployment is taking while, buuut its already on step4, when completes Ill put some feedback in both reviews.08:22
bandinimarios: yeah I did double check them08:22
*** akuznetsov has quit IRC08:23
gfidenteccamacho fyi https://review.openstack.org/#/c/347860/8/manifests/key.pp08:24
mariosbandini: hah thanks bandini i didn't really need a signed contract it was mostly rhetorical i know obviously they've been checked by sane individuals, its not just gfidente, but bandini is involved to... so. ok08:24
marioss/to/too08:24
*** akuznetsov has joined #tripleo08:24
bandiniahahahah08:24
bandinimarios: you're the first person in years to call me sane. you should question your sanity now ;)08:25
marios"what is sane"?08:25
marioswhoami08:25
* bandini chuckles08:25
gfidentemarios also, remember https://review.openstack.org/#/c/348415/ ?08:26
mariosgfidente: yeah sure man will look but only if you will leave me alone after that08:27
gfidenteI am giving you a chance to improve your stats08:27
marioswhat are you trying to say about my stats?08:28
bandinithanks ccamacho08:30
ccamachonp man!08:30
gfidentebandini so I w+208:31
gfidente+1 actually08:31
bandiniah so that does not happen automagically with 2x+208:31
gfidenteno so then we'll have to wait for the aodh role08:32
ccamachogfidente, Im stating to check the errors there, but need to increase the nodes mem.. 6GB OC nodes.. getting all the times... Cannot allocate memory - fork(2) stuff..08:34
bandiniyeah, I have a call now and then I will jump on the aodh stuff as well and rebase on top of it08:36
*** yamahata has joined #tripleo08:37
openstackgerritMerged openstack/puppet-tripleo: Move nova constraints from controller manifest to each service  https://review.openstack.org/34730908:39
openstackgerritMerged openstack-infra/tripleo-ci: Stop running dstat twice  https://review.openstack.org/34791708:40
*** r-mibu has quit IRC08:41
*** akuznetsov has quit IRC08:41
shardyGuys can I please get some reviews on https://review.openstack.org/#/c/347925/ and https://review.openstack.org/#/c/348440/ ?08:41
shardyboth are passing CI and the second one is big so I'd rather not rebase it08:42
shardy(big but trivial, sed replacement)08:42
gfidenteshardy I think in firewall.pp we were even replacing - with _ somewhere08:44
shardygfidente: Ah, cool, I guess we can remove that then08:44
gfidentehttps://github.com/openstack/puppet-tripleo/blob/master/manifests/firewall/service_rules.pp#L2708:45
gfidenteyeah :)08:45
shardygfidente: I'm working on a patch that makes the service_foo_ips lists dynamically generated:08:45
shardyhttps://github.com/openstack/tripleo-heat-templates/blob/master/overcloud.yaml#L91108:45
shardyso we need this change to enable that08:46
gfidenteack08:46
shardyI'm currently working around it by a nasty combination of list_join, str_replace and str_split :D08:46
openstackgerritMerged openstack/tripleo-heat-templates: Move nova constraints from controller manifest to each service  https://review.openstack.org/34731008:46
ccamachoso service names with _ right? /me needs to update aodh and mistral then08:47
shardyccamacho: Yeah, sorry any pending patches will need to be updated when we land my patch08:47
shardywe need the names to be consistent08:47
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Use delorean's exit code to decide on retry  https://review.openstack.org/34879008:48
gfidenteand there it is our first yaql as well08:48
ccamachoshardy, np08:48
*** ramishra has quit IRC08:48
shardygfidente: Yeah08:48
shardythe dynamic generation of *ips uses nested map_replace fundtions combined with repeat and map_merge :D08:48
shardyhttp://paste.openstack.org/show/543854/08:50
*** r-mibu has joined #tripleo08:51
openstackgerritMerged openstack/tripleo-heat-templates: Convert service_name to underscore syntax  https://review.openstack.org/34844008:52
openstackgerritAdriano Petrich proposed openstack/tripleo-quickstart: WIP gate upgrade  https://review.openstack.org/34216108:59
gfidenteshardy so that goes this way?08:59
gfidentecinder_api_node_ips: cinder_api_network08:59
gfidentecinder_api_node_ips: internal_api08:59
gfidentecinder_api_node_ips: 172.X.Y.Z08:59
openstackgerritMerged openstack/tripleo-ui: Add es6-promise polyfill to webpack config  https://review.openstack.org/34720108:59
shardygfidente: Yup08:59
shardygfidente: I'm still trying to figure out how we can maintain compatibility with the current ServiceNetMap syntax tho09:00
shardyideally we'd move to service_name_network in there too09:01
shardybut obviously we'll have to allow for deprecation of the old format09:01
gfidenteshardy wait though one thing I don't get is09:01
gfidenteto get the list of ips hosting a service09:02
gfidenteis this generated within the resourcegroup09:02
gfidenteand then joined in overcloud.yaml?09:02
shardygfidente: Yes, that says NetIpMap, but it will move into overcloud.yaml and use ControllerIpListMap09:03
shardywe'll generate a map of lists for each role, with a key for every enabled service09:03
gfidentewow09:03
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Add pacemaker mistral profile  https://review.openstack.org/34334609:04
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Implement aodh profiles  https://review.openstack.org/33285409:05
*** jbadiapa has quit IRC09:05
* shardy needs another coffee before attempting to make that actually work ;)09:06
mariosgfidente: so i figure i'm allowed at least *one* review request from you... https://review.openstack.org/#/c/188137/ afaics it is good, but the test run at https://review.openstack.org/#/c/342961/ wont pass with "CREATE_FAILED Resource CREATE failed: The Referenced Attribute (ControllerServiceChain role_data) is incorrect" has passed before. but, can you please eyes on the manila particularly the puppet/se09:08
mariosrvices/ incase i missed sthing there?09:08
mariosgfidente: hah sorry for gigaping.09:08
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates: Composable Mistral services  https://review.openstack.org/32343609:08
mariosgfidente: i spent ages staring at those definitinos yesterday but couldn't spot some issue09:08
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates: Composable Mistral services  https://review.openstack.org/32343609:09
gfidentemarios I think it's a syntax error in role_data09:09
gfidenteI've been there before09:09
mariosgfidente: ack thanks appreciate, whenever you ahve time no rush/now09:10
gfidenteshardy so in https://review.openstack.org/#/c/347925/1/puppet/services/services.yaml data: is the json to which $ refers to?09:10
*** mgould|afk is now known as mgould09:11
gfidenteguys can you check the trove submission as well: https://review.openstack.org/#/c/233240/09:11
gfidente^_^09:11
shardygfidente: Yeah, that's the input data to the yaql expression09:12
shardyI'm basically running a yaql query to filter the null values we get from OS::Heat::None09:13
shardyhttp://docs.openstack.org/developer/heat/template_guide/hot_spec.html#yaql09:14
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates: Add Aodh composable roles  https://review.openstack.org/33355609:14
shardyThe yaql docs are not good but it seems to work pretty well09:14
gfidenteack +209:15
gfidentemarios I think I found it09:15
gfidentenow you have to review shit for a week!09:15
shardylol09:15
mariosgfidente: yeah as if there is a day when you don't hasle me for reviews09:16
mariosgfidente: thanks man :)  where?09:16
gfidentehttps://review.openstack.org/#/c/188137/87/puppet/services/manila-base.yaml09:16
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates: Add Aodh composable roles  https://review.openstack.org/33355609:16
mariosgfidente: thanks man :) gfidente ++09:16
mariosbandini: prepare for tornados and hail today ^^09:17
marioshale even? probably09:17
*** mbound has joined #tripleo09:17
bandinieheheh09:18
*** yolanda has quit IRC09:19
*** jbadiapa has joined #tripleo09:19
*** yolanda has joined #tripleo09:20
mariosccamacho: did the _ thing land already? pointer?09:22
ccamachomarios https://review.openstack.org/#/c/348440/09:24
ccamachoyeahp today in the morning09:24
mariosccamacho: ack thanks09:25
ccamachonp09:25
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: Enable Manila integration - as a composable controller service  https://review.openstack.org/18813709:30
*** chem|away is now known as chem09:31
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: DO NOT MERGE testing manila  https://review.openstack.org/34296109:33
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Remve old cache files on the mirror server  https://review.openstack.org/34880609:37
*** fragatina has joined #tripleo09:38
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Next generation HA architecture work  https://review.openstack.org/34265009:38
ccamachoExtreme problems often require extreme solutions    http://petitions.moveon.org/sign/a-memory-upgrade-for?source=c.em.cp&r_by=16249607   :P09:44
hewbroccaROFL09:48
hewbroccaccamacho: nice09:48
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates: Next generation HA architecture work  https://review.openstack.org/34881109:52
derekhIf only we had a OVB deployment with enough capacity of all to use09:53
*** akrivoka has joined #tripleo09:57
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates: Next generation HA architecture work  https://review.openstack.org/31420809:57
*** ramishra has joined #tripleo10:00
openstackgerritKarthik S proposed openstack/tripleo-heat-templates: Customize vcpu_pin_set in nova.conf  https://review.openstack.org/34377010:01
bandinigfidente: there are two more cleanups we need to do for NG, just noticed now (same as before just moving constraints to their respective tripleo pacemaker profiles)10:08
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Move openstack-core-then-httpd constraint in the apache manifest  https://review.openstack.org/34881510:09
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates: Move openstack-core-then-httpd constraint in the apache manifest  https://review.openstack.org/34881610:10
gfidentebandini ack10:11
*** limao_ has quit IRC10:13
*** zoliXXL is now known as zoli|lunch10:15
*** yamahata has quit IRC10:16
*** milan has quit IRC10:16
*** tosky has joined #tripleo10:18
openstackgerritMerged openstack/tripleo-docs: Remove enabling epel  https://review.openstack.org/34853810:20
openstackgerritMerged openstack/tripleo-heat-templates: We don't need to set a default for the CephX keys and cluster FSID  https://review.openstack.org/34841510:20
openstackgerritGiulio Fidente proposed openstack/python-tripleoclient: Generate CephX keys via generate_overcloud_passwords  https://review.openstack.org/34841210:21
openstackgerritMerged openstack-infra/tripleo-ci: Refactor mirror logic  https://review.openstack.org/32614310:22
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Record testenv wait times in ovb jobs  https://review.openstack.org/34703710:23
openstackgerritMerged openstack/python-tripleoclient: Removes setting NeutronL3HA and NeutronAllowL3AgentFailover  https://review.openstack.org/34566310:23
*** milan has joined #tripleo10:25
openstackgerritSanjay Upadhyay proposed openstack/tripleo-heat-templates: SR-IOV nic agent changes in line with composable roles.  https://review.openstack.org/33219010:26
gfidenteshardy trozet ^^ neutronl3ha thing is wrong10:27
gfidenteI don't think we can enable l3ha with a single neutron node10:27
gfidentewhich is why we were enabling it based on the number of controllers10:27
shardygfidente: https://review.openstack.org/#/c/345648/2/puppet/services/neutron-server.yaml defaults it to false10:28
shardybut yeah, we're assuming pacemaker won't ever be enabled for single nodes10:28
gfidenteshardy yeah I mean the pacemaker scenario10:28
gfidentemaybe it's fine to keep it as is10:29
shardyhttp://docs.openstack.org/developer/heat/template_guide/hot_spec.html#equals10:29
shardygfidente: we could use equals to convert a node count to a boolean, e.g equals 110:29
shardybut then we'd have coupling between the ControllerCount and the service10:29
gfidenteshardy ah yaql!10:30
shardyNo, it's another new function ;)10:30
* shardy is using all the new shiny heat things today ;)10:30
gfidentein that respect, we had the coupling in the client before so it wasn't great anyway10:31
shardyMaybe we just document you need to disable it if you want to deploy a 1 node pacemaker setup?10:31
shardyseems like something of a corner case to me10:31
*** mbound has quit IRC10:31
gfidenteyeah10:31
pandawho's the CI mentor this week ? :)10:33
shardypanda: derekh is in the etherpad10:33
derekhpanda: how can I help?10:34
pandashardy: I see that ssl jobs are using a static certificate with 192.2.0.6 as CN. That's strange because I've seen the deployment trying to contact the controller on the vlan10 IP (10.0.0.5) I don't know how to change my parameters to have the Ip in CN  as the public VIP of the controller. If I have to change the certificate instead I don't know if using 10.0.0.5 as CN is a good idea.10:42
pandashardy: does anything that I've said make any sense ?10:47
shardypanda: where do you see traffic going via 10.0.0.5 vs the public vip?10:47
shardyis this for internal SSL?10:47
shardypanda: it does, but tbh you'd probably be best dicussing this with either jaosorior or bnemec10:48
shardyI think jaosorior is out on PTO for a week tho10:48
derekhpanda: you may have seen it but I think this is the template CI is using to set the VIP to 192.0.2.6 http://git.openstack.org/cgit/openstack-infra/tripleo-ci/tree/test-environments/enable-tls.yaml10:53
pandashardy: I don't see traffic, a deploy that enables SSL stops with this error: SSL exception connecting to https://10.0.0.5:13000/v3/auth/tokens: hostname '10.0.0.5' doesn't match u'192.0.2.6'. I don't know why the puppet module is trying to contact the controller on that IP.10:54
pandaIt's probably an issue with my local environment setup, but I'd like to understand how to set the right parameters10:54
*** tzumainn has joined #tripleo10:55
*** pkovar has joined #tripleo10:55
gfidenteshardy sorry but I am having a hard time figuring what is wrong here:10:57
gfidentehttps://review.openstack.org/#/c/346796/2/puppet/services/ceph-mon.yaml10:57
gfidenteI'm turning it from list into hash so it can be overridden via map_merge10:58
gfidentebut it's failing with http://logs.openstack.org/96/346796/2/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/83c81ea/console.html#_2016-07-28_11_13_56_09937310:58
gfidenteany hint?10:58
*** fragatina has quit IRC11:00
*** panda is now known as panda|lunch11:05
*** jprovazn has quit IRC11:09
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: DO NOT MERGE testing manila  https://review.openstack.org/34296111:10
shardypanda|lunch: it should be using the keystone public endpoint from the service catalog, so I'd check the IP that points to11:12
shardygfidente: You need a list of maps, but the get_param's are referencing string parameters?11:13
shardyso you need somekey: {get_param: ...} ?11:14
*** lucasagomes is now known as lucas-hungry11:14
ccamachogfidente, really not sure, but something like http://paste.openstack.org/show/543873/11:15
shardyccamacho: You can't map_merge a list of lists11:15
shardyit merges a list of maps11:15
ccamachommm :(11:16
shardythe problem is the maps in the list are malformed11:16
*** rodrigods has quit IRC11:19
*** rodrigods has joined #tripleo11:19
*** jprovazn has joined #tripleo11:29
*** tbonds has quit IRC11:30
openstackgerritMiles Gould proposed openstack/diskimage-builder: Allow installing proliant tools from an OS package  https://review.openstack.org/34421411:30
*** mbound has joined #tripleo11:32
openstackgerritMiles Gould proposed openstack/diskimage-builder: Allow installing proliant tools from an OS package  https://review.openstack.org/34421411:36
*** saneax is now known as saneax_AFK11:36
*** mbound has quit IRC11:36
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Move constraints to their respective services  https://review.openstack.org/34881511:39
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates: Move constraints to their respective services  https://review.openstack.org/34881611:39
*** thrash|g0ne is now known as thrash11:41
openstackgerritMiles Gould proposed openstack/diskimage-builder: Allow installing proliant tools from an OS package  https://review.openstack.org/34421411:41
*** rhallisey has joined #tripleo11:42
*** dsariel has joined #tripleo11:45
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates: WIP Move rabbit's clustering port away from the ephemeral port range  https://review.openstack.org/34585111:46
*** zoli|lunch is now known as zoli11:47
hewbroccafolks, seen beekhof's latest blog re "composable HA"?11:47
hewbroccahttp://blog.clusterlabs.org/blog/2016/composable-openstack-ha11:47
gfidenteshardy get_param is getting a string parameter yes11:51
gfidenteI was thinking to build the map 'on the fly'11:51
gfidente- get_param11:52
gfidente  key: value11:52
gfidente- get_param11:52
gfidente  key: value11:52
gfidenteand I wanted map_merge to merge the two dicts11:52
shardygfidente: but the get_param isn't a valid key: value for the map11:53
gfidenteah shit11:53
gfidenteright11:53
gfidente;)11:53
shardy:)11:53
gfidentethanks11:54
shardyhewbrocca: No, looks very interesting, thanks for the heads-up!11:54
*** pradk has joined #tripleo11:55
hewbroccafabbione: just linked me to it11:57
*** weshay is now known as weshay_mtg12:01
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Support arbitrary attributes for the Ceph pools  https://review.openstack.org/34679612:10
*** zoli is now known as zoli|brb12:10
*** masco has quit IRC12:11
*** panda|lunch is now known as panda12:11
*** jayg|g0n3 is now known as jayg12:12
*** morazi has joined #tripleo12:13
*** trown|outtypewww is now known as trown12:13
openstackgerritFlavio Percoco proposed openstack/tripleo-heat-templates: Update heat-agents setup files  https://review.openstack.org/34685912:19
openstackgerritFlavio Percoco proposed openstack/tripleo-heat-templates: Add default value for `RoleData`  https://review.openstack.org/34686012:19
openstackgerritFlavio Percoco proposed openstack/tripleo-heat-templates: WIP: Containerized Services for Composable Roles  https://review.openstack.org/33065912:19
openstackgerritFlavio Percoco proposed openstack/tripleo-heat-templates: Bind mount files to run DiD in latest atomic host  https://review.openstack.org/34721812:19
*** lucas-hungry is now known as lucasagomes12:20
*** ccamacho is now known as ccamacho|lunch12:28
*** tbonds has joined #tripleo12:29
gfidentederekh do we have a copy of the tripleo-overcloud-passwords file from ci jobs that you know?12:29
derekhgfidente: hmm, that would be in ~ wouldn't it? I don't think so12:30
gfidenteyeah12:30
derekhya, we only get files from /var/log and /etc ,12:31
gfidentederekh can I add it to host_info ?12:31
*** dsariel has quit IRC12:31
derekhgfidente: yup, go for it12:31
*** weshay_mtg is now known as weshay12:31
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: move hieradata/ceph into ceph-base service  https://review.openstack.org/34848912:32
*** jdob has quit IRC12:33
*** jdob has joined #tripleo12:33
openstackgerritHonza Pokorny proposed openstack/tripleo-ui: Add packaging spec and config files  https://review.openstack.org/34493212:36
openstackgerritGiulio Fidente proposed openstack-infra/tripleo-ci: Add 'ceph status' and tripleo-overcloud-passwords to host_info  https://review.openstack.org/34886212:38
*** rlandy has joined #tripleo12:44
EmilienMgfidente: can you approve https://review.openstack.org/#/c/347455/ please?12:50
shardyEmilienM: Hi, good morning, could you please check https://review.openstack.org/#/c/347925/ when you get a moment?12:54
shardyI'm going to push some patches which depend on that soon and I'd rather land it than cause a rebase12:54
EmilienMshardy: let me check12:54
shardythanks!12:55
EmilienMshardy: +A12:55
*** zoli|brb is now known as zoli12:57
*** zoli is now known as zoliXXL12:58
*** jcoufal has joined #tripleo12:58
openstackgerritHonza Pokorny proposed openstack/tripleo-ui: Add packaging spec and config files  https://review.openstack.org/34493213:00
*** maeca1 has joined #tripleo13:00
openstackgerritMerged openstack/tripleo-heat-templates: Filter null/None service names  https://review.openstack.org/34792513:00
openstackgerritJulie Pichon proposed openstack/tripleo-common: Add boot option argument to the Node Registration Mistral workflow  https://review.openstack.org/34887513:00
*** pradk has quit IRC13:02
*** jeckersb_gone is now known as jeckersb13:04
*** bvandenh has quit IRC13:05
*** rcrit has left #tripleo13:05
*** saneax_AFK is now known as saneax13:06
*** links has quit IRC13:07
*** numans has quit IRC13:07
beaglesEmilienM, shardy, if/when you can spare a moment https://review.openstack.org/#/c/333333/13:07
EmilienMbeagles: sure, will look shortly13:08
*** ccamacho|lunch is now known as ccamacho13:08
shardybeagles: this is a mitaka backport, but I can't see the master patch?13:08
beaglesEmilienM, thx!13:09
shardythe change id doesn't match anything in master, and grepping the title gives me no hits13:09
*** ianw has quit IRC13:10
*** ianw has joined #tripleo13:10
openstackgerritJohn Trowbridge proposed openstack/python-tripleoclient: Remove another private OSC attribute usage  https://review.openstack.org/34846013:10
trownthrash:  fixed up UT on ^13:10
*** numans has joined #tripleo13:10
*** bvandenh has joined #tripleo13:11
beaglesshardy, it wasn't really a backport per-se. The counterpart for newtown was quite different - but the related patch is https://review.openstack.org/#/c/325926/13:11
beaglesshardy, basically we completely removed all of the properties in newton13:11
shardybeagles: Ok, you'll need to explain why it's a mitaka-only patch in the commit message, and link the related master patch please13:12
beaglesshardy, ack13:12
ccamachoEmilienM morning! found several issues in the aodh submission, have fixed the non-ha locally, now im starting to work with the ha profiles.13:12
shardyalso please add a Closes-Bug to the commit message while you're updating :)13:12
hewbroccaderekh: so we got rh1 running?13:12
hewbroccaWhen do we go turn back on all the jobs?13:13
beagleshah.. there wasn't a bug reported against it when it was orgiinally filed :) but IIRC there are some now.. will do13:13
hewbroccaI'm a little afraid...13:13
shardybeagles: all patches landing on stable branches must have bugs associated with them13:13
EmilienMccamacho: excellent13:14
derekhhewbrocca: its running, nodepool is trying to use it but for soem reason deleting its VMs after a minute, I'm not sure why yet, I'm waiting for the infra people to wake up to find out why13:14
beaglesIIRC = bugs are actually assigned to me at this point and forgot to amend my commit message to include them ;)13:14
shardyhehe, no worries :)13:14
hewbroccaderekh: Well OK, forward progress13:14
derekhhewbrocca: I think this is all thats needed to get it working https://review.openstack.org/#/c/348778/13:15
derekhhewbrocca: once its working, I have two patches lined up13:15
hewbroccaohhh were we using nova-net for floating IPs before?13:15
derekhhewbrocca: 1 to switch all our current jobs from rh2 to rh113:15
hewbroccaIt would make sense13:15
derekhhewbrocca: then assuming thats ok, another to restore all the origional jobs13:16
hewbroccacool13:16
*** Tristitia has joined #tripleo13:17
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart: Adds ci-scripts to for tripleo-quickstart on OVB  https://review.openstack.org/34739313:19
openstackgerritJulie Pichon proposed openstack/python-tripleoclient: Add 'openstack overcloud node import' command  https://review.openstack.org/33972713:20
openstackgerritJulie Pichon proposed openstack/python-tripleoclient: [WIP] Add 'openstack overcloud node configure' command  https://review.openstack.org/34487513:20
openstackgerritBrent Eagles proposed openstack/tripleo-heat-templates: Properly direct MTU parameters and allow service defaults  https://review.openstack.org/33333313:21
openstackgerritJustin Kilpatrick proposed openstack/tripleo-quickstart: Add documentation for add-on roles  https://review.openstack.org/34673313:21
*** egafford has joined #tripleo13:24
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Source rh1.env to decide how many testenv workers to run  https://review.openstack.org/34888313:25
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Switch back to the net iso HA job  https://review.openstack.org/34888413:25
*** saneax is now known as saneax_AFK13:26
EmilienMslagle: I don't understand https://review.openstack.org/#/c/346147/20/scripts/deploy.sh13:32
EmilienMif I put an exit 0, I'm breaking CI13:32
slaglehow does that break CI?13:34
slagleif you're doing an undercloud job, exit after undercloud stuff is done13:35
trozetgfidente: is there a use case for having pacemaker on a single controller node? wouldn't most people use noha?13:35
*** mbound has joined #tripleo13:36
hewbroccatrozet: It dramatically simplifies deployment for us13:37
hewbroccasince we don't have to test systemd-management on the non-ha path vs. pacemaker management on the HA path13:38
hewbroccaif we use pacemaker on HA and non-HA there's half as many paths to test13:38
trozethewbrocca: well sure...but imo that isn't a reason to do it13:38
trozethewbrocca: along the same lines I don't get why there is a VIP used in noha case13:39
hewbroccaLOL seems like a pretty good reason to me, but go ahead13:39
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Implement aodh profiles  https://review.openstack.org/33285413:44
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates: Add Aodh composable roles  https://review.openstack.org/33355613:44
gfidentetrozet yeah the purpose is to be able to test the pacemaker codepath with a single controller13:45
gfidenteanyway, for dev purposes one can set that parameter to false as needed13:45
trozetgfidente: yeah, it's just defaulted13:45
*** myoung|remote has joined #tripleo13:46
*** jeckersb is now known as jeckersb_gone13:49
*** seanatcisco has joined #tripleo13:50
*** akshai has joined #tripleo13:50
*** dculik|gone has quit IRC13:51
*** seanatcisco_ has joined #tripleo13:52
rodrigodsquick (dummy) question: when running "bash quickstart.sh $HOST", it is skipping overcloud install/deploy13:53
rodrigodsis there any extra param needed?13:53
trownrodrigods: ya by default it stops before the overcloud deploy, so that a user could make changes/verify the deploy command before continuing13:54
trownrodrigods: "--tags all" will do the full deploy13:54
openstackgerritEmilien Macchi proposed openstack/instack-undercloud: Fix Aodh httpd binding  https://review.openstack.org/34889313:54
rodrigodstrown, hmm thanks13:54
*** pgadiya has quit IRC13:54
EmilienMhey, I found this bug in master and stable/mitaka ^13:55
*** seanatcisco has quit IRC13:55
*** seanatcisco_ is now known as seanatcisco13:55
EmilienMit's kind of urgent for our mitaka release, please review it asap13:55
shardyEmilienM: lgtm, but curious why it's a release blocker, don't we disable all metering services by default?13:57
EmilienMit's a blocker to upgrade the undercloud13:59
EmilienMfrom liberty to mitaka13:59
*** myoung|remote has quit IRC13:59
*** myoung|remote has joined #tripleo14:00
*** pradk has joined #tripleo14:00
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci: Implement a new job for undercloud testing only  https://review.openstack.org/34614714:02
trowncore folks, https://review.openstack.org/#/c/348460 is the only thing blocking promote atm, but zuul is super slow today. only UT failed on PS3 and I only fixed up tests in PS4. Would anyone be willing to manually verify I fixed UT (I ran locally and they pass) and we can merge?14:02
bnemecpanda: The CI setup is probably different because we weren't running net-iso in the ssl-enabled jobs, and it sounds like you are.14:03
EmilienMtrown: promote master?14:03
bnemecpanda: There was also a change in how you have to configure the vip for overcloud ssl.  See my change at: https://review.openstack.org/#/c/344475/14:03
trownEmilienM: ya14:03
EmilienMtrown: I've notice mitaka hasn't promoted also14:03
EmilienMis there a blocker?14:03
trownEmilienM: ya, mitaka failed on a spurious error in one of the packstack jobs last night14:04
*** fultonj has joined #tripleo14:04
trownEmilienM: it only runs every 12h by default, but I kicked this morning and tripleo jobs are in deploy14:04
EmilienMtrown: packstack only? we should promote14:04
EmilienMok14:04
trownfingers crossed14:04
pandabnemec: yes, I am using network isolation ... so the only way is to regenerate the certificate ?14:04
pandabnemec: I'm running in circles trying to find the exact place where this IP is specified ...14:05
trownshardy: is it ok if I merge https://review.openstack.org/#/c/348460 for the logic above?14:05
trown[10:02:36] <trown> core folks, https://review.openstack.org/#/c/348460 is the only thing blocking promote atm, but zuul is super slow today. only UT failed on PS3 and I only fixed up tests in PS4. Would anyone be willing to manually verify I fixed UT (I ran locally and they pass) and we can merge?14:05
EmilienMshardy: also, can you look https://review.openstack.org/#/c/347455/ please?14:06
bnemecpanda: Yes, there are instructions in the overcloud ssl docs for generating an appropriate certificate too.14:06
pandaKeystonAdminIP .. points to KeystoneAdminVirtualIP .. that points to IP_ADDRESS .. that points to PublicVIP, that points to KeystonAdminIP ... that points to ...14:06
bnemecpanda: The IP wasn't previously specified.  You just used the first one in the external range.14:06
*** akshai has quit IRC14:06
bnemecNow something changed and that doesn't work.14:06
bnemecSo you explicitly assign the same IP you used for the certificate.14:07
*** akshai has joined #tripleo14:07
bnemecOr you set up DNS and point that at the IP, but that's extra hassle in a dev/test environment.14:07
pandabnemec: thanks, the fact that network isolation is not used in CI explains a lot, the IPs were completely different from what I expected14:07
pandabnemec: assigning the Ip would mean to change everything, because I woul have to assign the public controller IP to the External vlan10 interface , or something like that, right ?14:08
trownthanks bnemec14:09
pandaIf only I knew how to do it, of course14:09
bnemecpanda: Did you read the link I posted?  It explains how to set the public vip.14:09
openstackgerritEmilien Macchi proposed openstack/instack-undercloud: Fix Aodh httpd binding  https://review.openstack.org/34889614:10
pandabnemec: Yes, I've seen that, I was talking about using that certificate anyway, and instead rotating all the IPs to match what the certificate contains.14:11
bnemecpanda: Oh, I think it's probably easier to just generate your own cert.14:12
* bnemec expenses ccamacho a http://www.ebay.com/itm/Dell-PowerEdge-R710-3-5-Server-12-Cores-2-93-GHz-SLBV7-144GB-H700-6x-Caddies-RPS-/331915267681?hash=item4d47b10e61:g:OvYAAOSwwbdWN9qR14:13
bnemec;-)14:13
pandabnemec: it is, but since I'm struggling to find IP parameters around, that would have been an interesting excercise14:13
openstackgerritMerged openstack/tripleo-quickstart: Adds ci-scripts to for tripleo-quickstart on OVB  https://review.openstack.org/34739314:14
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: added contrail database puppet module for tripleO  https://review.openstack.org/34840614:14
ccamachojajaja bnemec, thanks!14:14
bnemecpanda: You would change IP ranges in network-environment.yaml: http://docs.openstack.org/developer/tripleo-docs/advanced_deployment/network_isolation.html#create-network-environment-file14:15
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: added contrail analytics puppet module for tripleO  https://review.openstack.org/34889914:16
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: We don't need to set a default for the CephX keys and cluster FSID  https://review.openstack.org/34890014:17
EmilienMgfidente: lol the commit message ^14:18
bnemecderekh: Are the new rh1 environment details somewhere?  Want to make sure I have access before you disappear. :-)14:19
derekhbnemec: did you see the mail I just sent?14:19
bnemecderekh: lol, I just did14:19
gfidenteEmilienM yeah I talk bad about myself14:19
bnemecderekh: Thanks14:20
derekhbnemec: I *think* it has everything you need but worth a look over to make sure14:20
openstackgerritMerged openstack/instack-undercloud: Drop broken keystone pki setup  https://review.openstack.org/32717114:20
hewbroccaderekh: should we ask pabelanger to watch over the infra requests?14:21
pandabnemec: very interesting, thanks! sometimes one assumes that since he has seen and used a file a lot of times it's able to understand or modify its content .. I'll study this.14:21
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: added contrail analytics puppet module for tripleO  https://review.openstack.org/34773914:21
openstackgerritMerged openstack-infra/tripleo-ci: Don't spam postci with deployment time loop  https://review.openstack.org/34699014:22
derekhhewbrocca: sure, pabelanger I'm forwarding you an email  about the rh1 switch over14:22
openstackgerritMerged openstack-infra/tripleo-ci: Record testenv wait times in ovb jobs  https://review.openstack.org/34703714:22
*** mcornea has quit IRC14:28
*** mcornea has joined #tripleo14:29
trownpanda: did not read your comment before merging https://review.openstack.org/#/c/348554/1 I think the idea there is just to get the test harness up for a lint job, and fix linting in subsequent patches14:31
openstackgerritMerged openstack/tripleo-quickstart: Add tox -elinters  https://review.openstack.org/34855414:32
pandatrown: yes, "before proceding" I intended "before making it voting", sorry for not being more specific14:32
trownah cool14:32
trownyou +1, or I would have noticed :)14:32
slaglederekh: what tenant should I use if I just want to create some instances on rh2 for multinode ci job testing?14:34
slagleit'd be 4 8GB instances14:35
derekhslagle: "tripleo-user" , I think I sent you the creds at one stage14:35
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: Added Contrail Composable Services  https://review.openstack.org/34890514:35
derekhslagle: if not shout and I'll send them on14:35
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Support arbitrary attributes for the Ceph pools  https://review.openstack.org/34679614:35
slaglederekh: can you resend, the previous ones for "toci" tenant14:36
slaglewhich i think was before the last redeploy14:36
slaglederekh: wait, i have them14:37
slaglederekh: yea, i got them. thx14:37
derekhslagle: ack, just sent now anyways14:37
*** myoung|remote has quit IRC14:37
derekhslagle: if you use the template-centos-7-1469732382 image, thats the same image infra use14:38
slaglecool, yea14:40
slaglei'm going to use this to test some multinode HA jobs14:40
*** dsariel has joined #tripleo14:40
pabelangermorning14:42
*** david-lyle has joined #tripleo14:42
pabelangerderekh: replied to the email, but issue with tripleo-test-cloud-rh1 is we still need to launch AFS mirror in -rh114:42
hewbroccaahhh the AFS mirror!14:43
pabelangerwe should also create the 2nd tenant for nodepool to only launch nodes there, instead of a single tenant where everything g14:43
openstackgerritMerged openstack/tripleo-heat-templates: Stop passing charset=utf8 for neutron database connection option  https://review.openstack.org/33049014:43
*** jcoufal_ has joined #tripleo14:44
pabelangerit could be something like openstack-nodepool as the user14:44
pabelangersorry, some other user, as openstack-nodepool exists today14:45
pabelangeropenstackjenkins seems to be what otherclouds are using14:45
derekhpabelanger: flavor created14:46
EmilienMcan I have a review on https://review.openstack.org/309140 please?14:46
EmilienMand a last approve on https://review.openstack.org/347455 ?14:46
pabelangerslagle: EmilienM: https://review.openstack.org/#/c/348493/ ready for review now. Reverts the facter FQDN patch14:46
*** jcoufal has quit IRC14:47
derekhpabelanger: I can create the second user now but wont have a chance to switch everything over, if it isn't pressing maybe somebody else could pick it up next week?14:47
EmilienMpabelanger: +214:47
*** david-lyle has quit IRC14:47
pabelangerderekh: we can leave what you have today in the first tenant, as long as the 2nd tenant can still access that network I think we'll be fine14:48
shardyEmilienM: was there a reason to recheck https://review.openstack.org/#/c/348493 when all the jobs passed?14:49
EmilienMshardy: yes14:49
EmilienMshardy: I was to make sure it works on different clouds but it looks fine, since it fixed rh2 job14:49
EmilienMshardy: we can safely land it14:49
shardyEmilienM: Ok, thanks for clarifying14:50
bandinijistr: https://review.openstack.org/#/c/348816/ easiest review you will have today *PROMISE* (look at the depends-on too) ;)14:50
EmilienMgfidente: I dont get why you "diff" in https://review.openstack.org/347860 ?14:51
jistrhehe14:51
derekhpabelanger: ok, so to confirm, I'll create a user "openstackjenkins" in a "openstackjenkins" project and you should be able to use that? the network is usable by all tenants14:51
* jistr looking14:51
* bandini brings beer to jistr's desk14:51
gfidenteEmilienM basically I diff against the pre-existing keyfile to see if there are differences with the new one14:51
gfidenteif there are, then a new keyring is generated14:51
gfidentethe purpose is to cover with a single 'onlyif' all possible differences, secret key and/or caps14:52
pabelangerderekh: yup, then your local mirror and AFS will be the the current tenant.14:52
gfidenteEmilienM it's same purpose the sed had, but the sed was only grepping for the secret key so it wouldn't in fact regenerate the keyfile if the caps were changed14:52
jistrbandini, gfidente: looks good. We need to land these in sync again. Ready? :)14:53
gfidentejistr :)14:53
bandinijistr: w00t!14:54
EmilienMgfidente: wow, that's super smart14:54
EmilienMgfidente: but so complex :P14:54
EmilienMgfidente: should I +2 ?14:54
gfidenteit's not super smart though14:55
gfidentedon't lie to me14:55
slaglederekh: I added my public key to the default keypair. Is that going to get updated automatically on the jumphost so I can ssh in?14:55
gfidenteit does the job14:55
gfidentethe sed was smarter, but it didn't14:55
openstackgerritJulie Pichon proposed openstack/tripleo-common: Mistral workflow for creating a deployment plan from JSON input  https://review.openstack.org/34783014:55
openstackgerritMerged openstack/python-tripleoclient: Remove another private OSC attribute usage  https://review.openstack.org/34846014:56
gfidenteEmilienM honestly, given the commands are idempotent, I was going to throw in the idea of removing the onlyif entirely14:56
EmilienMgfidente: +214:56
derekhslagle: I've updated the jump host14:56
EmilienMtrown: trunk blocker landed^14:56
derekhslagle: no was the answer ;-)14:56
slaglederekh: thanks :). i'm in now14:57
gfidenteEmilienM but then I think puppet idempotency would kill me because it doesn't want exec launched twice with same arguments14:57
gfidenteso ...14:57
*** jdob has quit IRC14:57
*** jdob has joined #tripleo14:57
derekhpabelanger: user created, same password as the other nodepool user, want me to generate a new password?14:58
derekhbrb14:58
pabelangerderekh: ya, lets do a new password14:58
*** dtrainor has joined #tripleo14:59
*** dtrainor has quit IRC15:00
*** dtrainor has joined #tripleo15:01
openstackgerritMerged openstack/instack-undercloud: Revert "Explicitly set $FACTER_fqdn"  https://review.openstack.org/34849315:02
openstackgerritMerged openstack/instack-undercloud: Allow to configure Ironic [ssh] section  https://review.openstack.org/34745515:02
openstackgerritAttila Darazs proposed openstack/tripleo-quickstart: Remove the obsolete rebuild-inventory role  https://review.openstack.org/34891715:04
openstackgerritMerged openstack/puppet-tripleo: Move constraints to their respective services  https://review.openstack.org/34881515:04
*** jdob has quit IRC15:07
*** jdob has joined #tripleo15:07
*** jistr is now known as jistr|call15:10
*** jcoufal_ has quit IRC15:11
derekhpabelanger: new password set, where do you want it?15:11
*** jcoufal has joined #tripleo15:12
*** jistr|call is now known as jistr15:17
*** eggmaster has quit IRC15:20
gfidentejistr bandini sounds like we want a further -1 +1 ?15:22
gfidentehttps://review.openstack.org/#/c/348816/15:22
*** ayoung has joined #tripleo15:22
bandinigfidente: I have no idea what to do there15:23
bandinibut we need to merge it quickly because the puppet-tripleo counterpart went in already15:23
pabelangerderekh: can we both SSH into some common node?15:23
gfidenteI think jistr can -1 +1 again15:23
*** david-lyle has joined #tripleo15:23
gfidenteunless recheck does it15:24
jistrgfidente: i did15:24
* bandini passes the magic wand to jistr15:24
jistrgfidente: reverify should do it15:24
bandiniwe tried that15:24
gfidenteahahah15:24
gfidentecrazy moments15:24
jistrgfidente, bandini: if we aren't able to merge it then we need to push a revert for the other one in the interim15:24
bandinigfidente: argh you did a recheck?15:25
bandiniwon't that trigger the whole CI now?15:25
gfidenteyeah I tried that too15:25
*** weshay is now known as weshay_bbiab15:25
gfidenteit's ongoing no panic http://status.openstack.org/zuul/15:26
gfidentejust didn't move to gate queue yet15:26
bandiniack let's give it a bit then15:27
jistrgfidente, bandini: "easiest review you will have today *PROMISE*"15:27
derekhpabelanger: ssh root@66.187.229.46 cat password15:27
jistr^ quoted literally15:27
bandinijistr: ahahahah15:27
jistrwell played bandini15:27
gfidenteahaha15:27
bandinijistr: this was zuul's fault. my innocence is impeccable on this one ;)15:28
jistryea i know :)15:28
gfidentebandini are you admitting there have been other circumstances where you weren't innocent?15:28
bandiniI am at fault like *all the time* except this one ;)15:28
gfidenteI knew we had something in common15:29
bandiniahahah15:29
hewbroccagfidente: knows the underside of the bus so well he could draw you a picture of it15:29
EmilienMslagle: https://review.openstack.org/34892715:30
EmilienMbnemec: ^15:30
gfidentehewbrocca I got marios to explain me what was the story with the bus15:31
gfidentehewbrocca you're unfair15:31
hewbroccaLOL15:31
gfidenteonly those who don't do don't see the bus!15:31
bnemecEmilienM: The idea is we'll run it on stable/newton some day when that branch is a thing?15:31
slagleayoung: yesterday, were you expressing some concerns about having multiple OpenStack services running under httpd/wsgi alongside keystone?15:32
hewbroccaI would blame marios, he's the one who's always throwing you under there15:32
EmilienMbnemec: yeah, see my regex15:32
ayoungslagle, actually, just the opposite15:32
EmilienMbnemec: we just don't run it for stable/liberty and stable/mitaka15:32
ayoungslagle, I want Keystone to run in its own container15:32
EmilienMbnemec: I did a typo15:32
EmilienMbnemec: now it's good.15:32
ayoungslagle, so I don't want a "HTTPD" role, I want a Keystone role when we split things out15:33
bnemecEmilienM: Okay, just wanted to confirm that was the plan.15:33
*** rwsu has quit IRC15:33
slagleayoung: ok. but are there known issues if we put multiple services running under the same httpd?15:33
slagleayoung: i'm curious b/c of https://review.openstack.org/#/c/309140/ for the undercloud15:33
slaglewhere we already have keystone under httpd15:33
ayoungslagle, Not now there is not.  Keystone and HTTPD run side by side just fine15:33
ayoungslagle, undercloud should be OK that way, too15:34
slagleok15:34
*** anshul has quit IRC15:34
*** leanderthal is now known as leanderthal|afk15:34
marioshewbrocca: i am in a call so no idea what you're discussing, but YES it was gfidente fault for sure.15:34
ayoungslagle, I am thinking minimal downtime upgrades.  WHen upgrading a system, Keystone needs to be upgradable first and we are working hard to make that work without bringing down the cluster15:35
jistrbandini, gfidente: well you know what's bad, that until the lint gate is fixed, we aren't able to push a revert either15:35
jistrit will fail on the same thing15:35
ayoungslagle, but part of that is that we need to upgrade the code before the database, and doing will likely potentially bring down the other services.  THere are library dependdencies (all the oslo-*) that force a lockstep upgrade15:36
gfidentejistr but people will blame gate15:36
* bnemec throws the gate under the bus with gfidente15:36
gfidentelike marios15:36
ayoungslagle, and Horizon links in quite a few dependencies itself, especially all of the clients, so I would not want to force it into the same container as Keystone.  Undermining ourselves if we do15:37
bandinijistr: but it failed on a timeout, so chances are it will work on a retry15:37
bandinijistr: it was a timeout reaching some ruby gem15:37
bandiniunless the whole ruby gem download ecosystem is down that is :)15:37
gfidentebnemec the "dictionary update sequence element #3 has length 4;"15:37
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Configure Nova [cache] options correctly (Mitaka only)  https://review.openstack.org/34825515:37
gfidentestrange story where dict['key'] fails when value contains equals15:37
gfidentepassword=randomstring==15:38
gfidente.get() is safe though15:38
hewbroccathe ruby gem download ecosystem is not infrequently down15:38
slagleayoung: ok, i understand15:38
gfidentebnemec ever seen it?15:38
hewbroccagfidente: ^^^ see??15:38
bnemecgfidente: Wha?15:38
bnemecThat's insane.15:38
slagleayoung: i think these are all things to consider for the overcloud going forward. but i suspect for the single node undercloud, we're likely ok15:38
* bnemec tries it15:38
gfidentebnemec yeah but I just tried it locally and that's it15:39
ayoungslagle, correct15:39
gfidentebnemec yes please :)15:39
jistrbandini, gfidente: hmm i wasn't able to find a single running lint job ATM, they seem to be all "queued" in zuul, at least as for what i was able to dig up15:40
gfidenteneedless to say cephx keys are base64 encoded and append those ==15:40
bnemecgfidente: I can't reproduce it on my laptop.  I wonder if this is a weird bug in the Python on centos 7.15:42
bnemecfoo = {}15:42
bnemecfoo['foo'] = 'something=='15:42
bnemecprint foo['foo']15:42
bnemecWorks fine for me.15:42
*** radek_ has quit IRC15:42
gfidentebnemec though try append an = to one of the passwords in tripleo-overcloud-passwords15:42
gfidenteif you put the value in quotes it also works yes15:42
gfidentebut we don't put them in quotes15:42
gfidenteI mean password="string==" is also fine15:43
gfidentebnemec if you edit tripleo-overcloud-passwords appending a = and try a deploy it will fail like that15:43
bandinijistr: do you know who we can ping here to unblock things?15:43
jistri guess some combination of #puppet-openstack and #openstack-infra15:44
bnemecgfidente: Oh, it's reading the file back in, not the Python dict that is failing.15:44
gfidenteyeah15:44
jistrEmilienM might know15:44
bnemecI understand now.15:44
gfidenteI realized it now as well15:44
bnemecI think we can fix that.  One second15:44
gfidenteas I was talking about the file15:44
bnemecgfidente: We need to pass maxsplit=1 when reading the file back in: https://docs.python.org/2/library/stdtypes.html#str.split15:45
gfidentebnemec ack15:45
EmilienMjistr, bandini: what's up?15:46
gfidentebnemec thanks :)15:46
pabelangerderekh: nice! need to remember that15:47
jistrEmilienM: this https://bugs.launchpad.net/tripleo/+bug/160783615:47
openstackLaunchpad bug 1607836 in tripleo "CI down on chicken-and-egg patch unable to merge because of puppet-lint job" [Critical,Triaged]15:47
jistri tagged it with alert so that it pops up on IRC here until the patch is able to merge15:48
EmilienMjistr: why can't we land the tht now?15:48
gfidentebnemec though it says the rightmost split is done15:48
jistrEmilienM: we'd love to but puppet-lint blocks it15:48
gfidentewhich is not what we want :P15:48
gfidenteoh no lsplit15:48
gfidenteis not rsplit15:48
EmilienMjistr: it's in check queue now, let's see15:49
gfidentesplit is not rsplit15:49
jistryea15:49
jistractually that one will just update the check, won't it15:49
jistri'll try reverify again15:49
jistrso that we get another pass at the 'gate' section15:49
EmilienMjistr: let it going now15:49
EmilienMit's in check, and I think it will work15:50
openstackgerritGiulio Fidente proposed openstack/python-tripleoclient: Generate CephX keys via generate_overcloud_passwords  https://review.openstack.org/34841215:50
bnemecgfidente: I'm a little confused why we're even reading the passwords back in on the initial deploy though.15:50
EmilienMwe can't skip 'check'15:50
jistrEmilienM: we didn't, check is green15:50
EmilienMjistr: will see how it works15:51
EmilienMwe have no choice to wait now :)15:51
gfidentebnemec generate_ just always does it to see if the password is already there I think15:51
bnemecgfidente: Yeah, but the file isn't there the first time.15:51
jistrok15:51
bnemecSo it should skip that block entirely.15:51
bnemecI mean, I'm glad we caught this now.  But I don't understand why.15:52
bnemecgfidente: Oh, it's because we create a password cache that also calls generate: https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/utils.py#L52515:54
gfidente_password_cache15:54
dmsimardgfidente: ping15:56
bnemecgfidente: I think that's going to fail pep8 due to line length.15:56
gfidentedmsimard pong15:56
dmsimardgfidente: what ceph version is expected/tested to work with newton ?15:56
gfidentebnemec amending15:56
dmsimardgfidente: it doesn't look like documentation mentions setting up a particular ceph repository so if it's pulling from EPEL, that's 0.80 and much older than even hammer (0.94) or jewel15:57
dmsimardgfidente: we're testing puppet-openstack-integration against jewel now.15:57
EmilienMgfidente: FYI https://review.openstack.org/#/c/348489/2/puppet/services/ceph-base.yaml15:57
gfidentedmsimard we need to land https://review.openstack.org/#/c/340504/ and https://review.openstack.org/#/c/343667/15:58
gfidenteEmilienM so we had a BZ tracking that15:58
*** panda is now known as panda|bbl15:58
EmilienMgfidente: yea, just an fyi15:58
gfidenteand we can't switch to the new path as easily because it's problematic on upgrades15:58
gfidenteso we need a plan for that15:58
EmilienMI had selinux issues because of it15:58
EmilienMgfidente: right.15:59
EmilienMjust an fyi :)15:59
gfidente:(15:59
*** jcoufal has quit IRC15:59
*** jcoufal has joined #tripleo15:59
dmsimardgfidente: ack, so is it hammer or jewel then ? :p16:00
dmsimardgfidente: someone in #rdo needs to know because he's trying things out for rdo test day.16:00
*** pcaruana has quit IRC16:01
gfidenteEmilienM see what you think of the comment I left, marios can help as well :)16:01
gfidentedmsimard it's hammer from EPEL16:02
dmsimardgfidente: EPEL provides 0.80, that's not hammer16:02
dmsimardhammer is 0.9416:02
EmilienMgfidente: right now, we should just merge the patch16:02
EmilienMand see later16:02
*** dsariel has quit IRC16:02
openstackgerritMerged openstack/instack-undercloud: Deploy and Upgrade Ironic to run in mod_wsgi  https://review.openstack.org/30914016:02
dmsimard0.80 is firefly16:03
*** mcornea has quit IRC16:04
gfidentedmsimard oh it's 0.94 in the rhel image ...16:04
dmsimardgfidente: so centos-release-ceph-hammer should work then16:04
dmsimardthanks16:04
gfidentedmsimard so I even had a submission for that16:04
gfidenteone of the previous patchsets for that submission was using the centos sig packages16:05
gfidentedmsimard https://review.openstack.org/#/c/340504/3/scripts/tripleo.sh16:06
gfidenteI think we also have to figure if our preference is for the centos/sig version or the ceph.com version of the packages16:06
*** paramite has quit IRC16:06
dmsimardgfidente: yeah, the release package is not yet available for jewel last I know16:06
dmsimardgfidente: the problem with using ceph.com repositories16:06
*** hewbrocca is now known as hewbrocca-afk16:06
dmsimardgfidente: is that the dependencies are not self contained and not available in CentOS base OS16:07
dmsimardfor example libleveldb is only in EPEL16:07
gfidentebut centos+epel right16:07
dmsimardso if you use ceph.com repo, you need to have epel16:07
zoliXXLhave a good weekend16:07
dmsimardbut if you have the SIG repo, libleveldb is there16:07
gfidentethough I think we're good with that because we'd be using those for the upstream CI only16:07
dmsimardit's self contained16:07
dmsimardEPEL shouldn't be used anywhere16:07
dmsimardit's the devil16:07
gfidenteoh well I think we need it in tripleo16:07
EmilienMnope16:08
EmilienMthere is some WIP to remove it in tripleo-ci16:08
dmsimardif you need it, you need to identify what you need16:08
*** zoliXXL is now known as zoli|gone16:08
dmsimardso we can pull the dependencies either in RDO or elsewhere16:08
EmilienMbelieve me, you don't need EPEL to deploy TripleO16:08
dmsimardwe can't control EPEL, they've broken us more than once16:08
gfidentebut the ceph.com is going to depend on EPEL16:08
openstackgerritGiulio Fidente proposed openstack/python-tripleoclient: Generate CephX keys via generate_overcloud_passwords  https://review.openstack.org/34841216:09
*** zoli|gone is now known as zoli_gone-proxy16:09
*** ooolpbot has joined #tripleo16:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION16:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/160783616:10
*** ooolpbot has quit IRC16:10
openstackLaunchpad bug 1607836 in tripleo "CI down on chicken-and-egg patch unable to merge because of puppet-lint job" [Critical,Triaged]16:10
gfidenteand centos/sig doesn't have jewel yet16:10
*** jeckersb_gone is now known as jeckersb16:11
dmsimardgfidente: it does16:11
EmilienMgfidente: they do16:11
EmilienMwe use it16:11
EmilienMa sec16:12
dmsimardgfidente: http://buildlogs.centos.org/centos/7/storage/x86_64/ceph-jewel/16:12
dmsimardgfidente: there's just no release RPM for it yet, not sure why I forget16:12
EmilienMhttps://github.com/openstack/puppet-ceph/blob/master/manifests/repo.pp#L126-L13216:12
EmilienMjust set ceph::repo::enable_sig: true and it will deploy the repo for you :)16:12
*** numans has quit IRC16:12
openstackgerritayoung proposed openstack/python-tripleoclient: Clear the old environment in rc file  https://review.openstack.org/34755316:12
openstackgerritayoung proposed openstack/python-tripleoclient: Add an overcloud rc.v3 file  https://review.openstack.org/34755416:12
derekhpabelanger: you got that password? /me is going to delete the instance16:12
gfidentedmsimard so you liked https://review.openstack.org/#/c/340504/3/scripts/tripleo.sh more?16:13
EmilienMgfidente: /me likes :)16:13
dmsimardgfidente: yeah16:13
gfidenteif that looks good we can just restore patchset #316:13
gfidentefine :)16:13
*** jcoufal has quit IRC16:13
pabelangerderekh: yup, delete away16:13
derekhpabelanger: ack, done16:14
*** jcoufal has joined #tripleo16:14
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Implement aodh profiles  https://review.openstack.org/33285416:15
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates: Add Aodh composable roles  https://review.openstack.org/33355616:17
ccamachobandini, EmilienM Im hoping this is the last error from composable aodh http://paste.openstack.org/show/543930/  this is related to the pacemaker constraints, but Im not fully aware about how they work, when having time mid to check how they should be referenced?16:19
bandiniccamacho: looking16:19
ccamachos/mid/mind/16:19
EmilienMccamacho: ok I'll monitor that in the week end16:19
ccamachoEmilienM thanks!16:20
openstackgerritGiulio Fidente proposed openstack-infra/tripleo-ci: Install Ceph Hammer and/or Jewel from CentOS Storage SIG repos  https://review.openstack.org/34050416:20
ccamachobandini they are defined here https://review.openstack.org/#/c/332854/10/manifests/profile/pacemaker/aodh.pp16:20
pabelangergfidente: we have ceph mirrors in openstack-infra, can we use them?16:21
bandiniccamacho: i see the problem, sec16:22
gfidentepabelanger ceph.com mirrors or centos storage sig mirrors?16:22
pabelangergfidente: ceph.com mirrors16:23
gfidentepabelanger ^^ so we just discussed switching to centos storage sig16:23
pabelangerI believe that is what EmilienM is using for puppet testing too16:23
ccamachobandini  <316:23
gfidentepabelanger I think EmilienM is using SIG?16:23
*** florianf has quit IRC16:23
*** weshay_bbiab is now known as weshay16:24
pabelangerdmsimard: is there any way we can fix this in ceph.com repos?16:24
EmilienMgfidente: I do16:25
EmilienMwe use RDO + SIG without EPEL16:25
dmsimardpabelanger: upstream ceph doesn't want to carry/build/package/maintain all of their dependencies16:25
dmsimardpabelanger: I asked16:25
bandiniccamacho: see PM16:25
*** trown is now known as trown|lunch16:25
ccamachobandini looking16:25
pabelangerdmsimard: ya, that make sense. So they depend on EPEL16:25
beaglesis dprince on pto or something this week?16:25
dmsimardpabelanger: right. Ceph won't even install on CentOS without EPEL due to lack of dependencies.16:26
*** jpich has quit IRC16:26
dmsimardpabelanger: but the CentOS storage SIG repos for Ceph maintain all the dependencies themselves and they are self-contained16:26
dmsimardso no need for EPEL.16:27
pabelangeroh wait16:27
pabelangerwhat is the URL for the SIG repo?16:27
pabelangerhttp://mirror.centos.org/centos/7/storage/x86_64/ceph-hammer/ ?16:27
pabelangerdmsimard: gfidente ^16:28
pabelangerif that is the URL, we I can add it to AFS16:28
dmsimardpabelanger: that's hammer, there is one for jewel that will be available soon16:28
dmsimardpabelanger: there are signed packages to install the repos16:28
pabelangerdmsimard: will it be in mirror.centos.org?16:28
dmsimardpabelanger: yeah16:28
pabelangerokay16:28
pabelangergreat16:28
pabelangerthen I can mirror that into our AFS16:28
pabelangerand you'll be able to use it16:28
dmsimardpabelanger: it's on buildlogs until released: http://buildlogs.centos.org/centos/7/storage/x86_64/16:28
pabelangerlet me add it16:28
pabelangerdmsimard: right16:29
openstackgerritMerged openstack-infra/tripleo-ci: Source rh1.env to decide how many testenv workers to run  https://review.openstack.org/34888316:29
pabelangerdmsimard: so, you'll have to use buildlogs for now16:29
pabelangerbut I can mirror hammer16:29
dmsimardpabelanger: that's what we're doing in puppet, yeah16:29
gfidentepabelanger though I'd have to mangle the .repo file to point to afs then right?16:29
*** tesseract- has quit IRC16:30
*** dcotton has quit IRC16:30
pabelangergfidente: for the moment yes, but my patch will drop the .repo file into the folder disabled. you'll just need to enable it16:30
pabelangergfidente: that will take a few days to land hower16:30
pabelangerhowever*16:30
gfidentepabelanger ok can you comment on the submission then with some instructions in case it isn't landed yet?16:30
pabelangergfidente: yes16:30
gfidentepabelanger or even follow up on it with the needed changes16:30
gfidenteyour call16:30
pabelangergfidente: if you can way a few more days, lets fix your patch. Not sure if that holds up development16:31
gfidenteack16:31
openstackgerritJeff Peeler proposed openstack/tripleo-common: Change output for GetCapabilitiesAction  https://review.openstack.org/34853716:32
openstackgerritJeff Peeler proposed openstack/tripleo-common: Change output for GetCapabilitiesAction  https://review.openstack.org/34853716:33
openstackgerritJeff Peeler proposed openstack/tripleo-common: Change output for GetCapabilitiesAction  https://review.openstack.org/34853716:34
*** paramite has joined #tripleo16:36
gfidenteso I am leaving guys, thanks dmsimard for helping with the repos16:36
dmsimardgfidente: np16:37
gfidentetrown|lunch on the docs issue I agree but I am not sure where to push the docs16:37
gfidentetrown|lunch if you have 10 mins please comment on the same submission with a few info on where to push the docs and what you'd like to see documented16:38
gfidenteunless you meant the tripleo-docs?16:38
*** gfidente has quit IRC16:39
*** lucasagomes is now known as lucas-dinner16:41
EmilienMjistr: ok indeed, all lint jobs are now broken :)16:44
EmilienMah no, it's a timeout tis time http://logs.openstack.org/93/348893/1/check/gate-instack-undercloud-puppet-lint/6a5feb9/console.html#_2016-07-29_16_24_47_67823216:44
*** rhallisey has quit IRC16:52
*** jbadiapa has quit IRC16:52
*** yamahata has joined #tripleo16:55
*** thrash is now known as thrash|biab16:56
*** derekh has quit IRC16:57
beaglesEmilienM, got bit by that as well17:01
beagleshttps://youtu.be/rWIcubkPaxo17:02
openstackgerritHonza Pokorny proposed openstack/tripleo-ui: Add visual indicator that a deploy was requested  https://review.openstack.org/33912017:05
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Implement aodh profiles  https://review.openstack.org/33285417:08
openstackgerritMerged openstack/instack-undercloud: Remove nova-compute restart at end of uc install  https://review.openstack.org/34738917:09
*** snecklifter has joined #tripleo17:09
*** paramite has quit IRC17:09
*** ooolpbot has joined #tripleo17:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION17:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/160783617:10
*** ooolpbot has quit IRC17:10
openstackLaunchpad bug 1607836 in tripleo "CI down on chicken-and-egg patch unable to merge because of puppet-lint job" [Critical,Triaged]17:10
ccamachobandini, your wise advise made the trick, the aodh submission is working for me both ha and nonha locally, EmilienM we can check it later.17:12
bandiniccamacho: w00t!17:12
*** tosky has quit IRC17:14
ccamacho**me going for some beers!  Have a nice weekend guys!!!17:14
bandiniccamacho: enjoy ;)17:14
*** ccamacho has quit IRC17:15
*** milan has quit IRC17:17
socialEmilienM: http://logs.openstack.org/69/232369/11/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/a168c17/console.html so I guess patch the patch?17:24
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates: Add EnabledServices output to overcloud.yaml  https://review.openstack.org/34844117:25
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates: Convert AllNodesConfig hosts config to a map  https://review.openstack.org/34897217:25
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates: Create per-role node_names in hieradata for all nodes  https://review.openstack.org/34897317:25
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates: WIP Create composable mapping between enabled services and role ips  https://review.openstack.org/34897417:25
EmilienMsocial: what is the context, please.17:28
*** rwsu has joined #tripleo17:31
socialEmilienM: gfidente's patch https://review.openstack.org/#/c/23236917:32
*** shardy has quit IRC17:33
EmilienMhttp://logs.openstack.org/69/232369/11/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/a168c17/console.html#_2016-07-29_14_18_18_20500717:33
EmilienMit fails here17:34
socialegafford: 2016-07-29 14:18:18.911631 | Notice: /Stage[main]/Nova::Compute::Libvirt::Services/Service[libvirt]/ensure: ensure changed 'stopped' to 'running'17:36
openstackgerritHonza Pokorny proposed openstack/tripleo-ui: Add npm version to webpack output  https://review.openstack.org/34897617:36
socialEmilienM: so it starts libvirt after the command17:36
EmilienMit's something you can easily fix with puppet17:37
socialhah 4 lines lower it's fixed17:37
egaffordsocial: Did you mean to ping me there or was there a namespace collision?17:38
*** trown|lunch is now known as trown17:39
socialegafford: collision17:39
egaffordsocial: ^517:39
*** mgould is now known as mgould|afk17:44
*** rhallisey_ has joined #tripleo17:48
EmilienMbnemec, slagle : I have difficulties to understand how puppet modules are deploying in tripleo stable jobs17:53
*** links has joined #tripleo17:53
EmilienMthe undercloud upgrade job is failing because of puppet modules are too recent for puppet-oslo17:54
*** jprovazn has quit IRC17:59
*** jprovazn has joined #tripleo18:02
openstackgerritPaul Belanger proposed openstack/tripleo-quickstart: Fix E226 missing whitespace around arithmetic operator  https://review.openstack.org/34899918:08
openstackgerritPaul Belanger proposed openstack/tripleo-quickstart: Use ansible 2.1 importing of boilerplate code  https://review.openstack.org/34900018:08
openstackgerritPaul Belanger proposed openstack/tripleo-quickstart: Fix H102 Apache 2.0 license header not found  https://review.openstack.org/34900118:08
*** ooolpbot has joined #tripleo18:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION18:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/160783618:10
*** ooolpbot has quit IRC18:10
openstackLaunchpad bug 1607836 in tripleo "CI down on chicken-and-egg patch unable to merge because of puppet-lint job" [Critical,Triaged]18:10
ayoungEmilienM, is there support in the puppet modules for managing the horizon file /usr/share/openstack-dashboard/openstack_dashboard/local/local_settings.py18:19
ayoungDo we really keep a copy of the that file in the puppet modules?  http://git.openstack.org/cgit/openstack/puppet-horizon/tree/templates/local_settings.py.erb18:21
ayoungShudder18:21
EmilienMayoung: it is how the module was written, yes.18:23
ayoungEmilienM, that might be problematic.  It means that we drop any changes that happen in upstream Horizon and need to resync.  I can see why it is a template, due to the complexity of parsing, but I would have thought Puppet (written in Ruby) would have included a Full python parser inside it....18:24
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci: Use delorean-current for puppet modules  https://review.openstack.org/34142918:24
ayoungAnyway, from a WebSSO perspective, that template needs triggers to be able to handle any webSSO specific parameters, and we need to be able to pass them through Hiera18:25
EmilienMbnemec, slagle: I rebased shardy's patch about puppet modules deployment, feedback is welcome.18:27
EmilienMayoung: just send a patch to puppet-keystone18:27
EmilienMor file a bug and I'll do it18:27
*** electrofelix has quit IRC18:28
ayoungEmilienM, puppet-horizon, will do18:30
EmilienMerr18:30
EmilienMsorry18:30
EmilienMjust an FYI for everyone, current status of OVB HA job is broken, we need https://review.openstack.org/#/c/348816 which is in gate now18:31
*** thrash|biab is now known as thrash18:34
ayoungEmilienM, https://bugs.launchpad.net/puppet-horizon/+bug/160791218:35
openstackLaunchpad bug 1607912 in puppet-horizon "WebSSO config options" [Undecided,New]18:35
*** akrivoka has quit IRC18:36
EmilienMslagle: tripleo CI is currently red until 348816 gets merged18:38
EmilienMslagle: I'm about to ask a promotion of the zuul jobs on infra channel18:38
*** tzumainn has quit IRC18:40
beaglesEmilienM, thanks for the heads up18:42
*** jcoufal has quit IRC18:43
*** jcoufal has joined #tripleo18:44
openstackgerritMerged openstack/instack-undercloud: Fix Aodh httpd binding  https://review.openstack.org/34889618:46
*** yamahata has quit IRC18:46
*** panda|bbl is now known as panda18:46
EmilienMslagle: fyi, no need to do rechecks, like I said, we need the patch merged before18:51
slagleEmilienM: Huh?18:51
slaglei havent done any rechecks. since you told me 13 minutes ago18:51
EmilienMslagle: yeah you did before, just an fyi in case you missed my message18:52
EmilienMI asked to promote the CI job18:52
EmilienMbut nobody replied to me18:52
slagleEmilienM: where is the bug on this?18:52
EmilienMso our gate is currently red18:52
slaglecan you add the alert tag?18:52
EmilienMhttps://bugs.launchpad.net/tripleo/+bug/160783618:52
openstackLaunchpad bug 1607836 in tripleo "CI down on chicken-and-egg patch unable to merge because of puppet-lint job" [Critical,Triaged]18:52
slaglethen people will know18:52
EmilienMit's in alert18:52
slagleok great18:52
EmilienMI'm going try again on infra channel18:52
slagleEmilienM: the multinode job also uses pacemaker, yet it is passing19:05
slagleis it bacause it's not ha?19:06
EmilienMyeah probably something required on mutli node19:06
bandinithe review failed on unrelated stuff it seems19:06
*** pkovar has quit IRC19:09
*** ooolpbot has joined #tripleo19:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION19:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/160783619:10
*** ooolpbot has quit IRC19:10
openstackLaunchpad bug 1607836 in tripleo "CI down on chicken-and-egg patch unable to merge because of puppet-lint job" [Critical,In progress] - Assigned to Emilien Macchi (emilienm)19:10
*** bnemec is now known as beekneemech19:12
*** penick has joined #tripleo19:14
*** fzdarsky is now known as fzdarsky|afk19:19
*** fzdarsky|afk has quit IRC19:19
*** links has quit IRC19:20
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci: Implement a new job for undercloud testing only  https://review.openstack.org/34614719:22
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci: Implement a new job for undercloud testing only  https://review.openstack.org/34614719:22
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci: WIP - Implement undercloud upgrade job - Mitaka -> Newton  https://review.openstack.org/34699519:22
*** rwsu has quit IRC19:27
*** myoung|remote has joined #tripleo19:29
*** rhallisey_ is now known as rhallisey19:42
*** yamahata has joined #tripleo19:43
*** jprovazn has quit IRC19:54
*** jdob has quit IRC20:00
openstackgerritMerged openstack/tripleo-heat-templates: Move constraints to their respective services  https://review.openstack.org/34881620:06
*** ooolpbot has joined #tripleo20:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION20:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/160783620:10
*** ooolpbot has quit IRC20:10
openstackLaunchpad bug 1607836 in tripleo "CI down on chicken-and-egg patch unable to merge because of puppet-lint job" [Critical,In progress] - Assigned to Emilien Macchi (emilienm)20:10
EmilienMoh wow it landed20:10
EmilienMCI should be back20:11
*** jeckersb is now known as jeckersb_gone20:15
*** snecklifter has quit IRC20:17
*** rwsu has joined #tripleo20:18
EmilienMslagle: thx20:20
EmilienMI was about doing it20:20
*** coolsvap has quit IRC20:21
weshayslagle, do you want the undercloud to be a libvirt guest, or directly installed on the node?20:34
weshayre: https://review.openstack.org/#/c/348530/20:34
openstackgerritEmilien Macchi proposed openstack/instack-undercloud: Fix Aodh httpd binding  https://review.openstack.org/34889320:36
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: move hieradata/ceph into ceph-base service  https://review.openstack.org/34848920:36
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Enable keystone to use the SSL middleware  https://review.openstack.org/34135420:37
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Enable glance to use the SSL middleware  https://review.openstack.org/34136720:37
EmilienMslagle: did you already seen http://logs.openstack.org/95/346995/16/experimental/gate-tripleo-ci-centos-7-undercloud-upgrades-nv/64ef2b7/console.html#_2016-07-29_20_06_03_806767 ?20:40
*** tbonds has quit IRC20:41
*** akshai has quit IRC20:42
slagleweshay: directly installed20:42
weshayslagle, k.. we may need to bring in a 3rd party role to do that20:43
slagleno we don't, what i did works :)20:43
weshayok20:44
slagleEmilienM: no20:44
slagleEmilienM: i'm not sure what it's trying to do there20:47
slagleEmilienM: hmm, that shouldn't even cause os-refresh-config to fail afaict20:50
slagleEmilienM: the real error is earlier up in the undercloud install log20:51
slagle20551 2016-07-29 20:06:00.000 | IOError: [Errno 2] No such file or directory: '/sys/class/net/eth1/address'20:51
openstackgerritJustin Kilpatrick proposed openstack/tripleo-quickstart: Add documentation for add-on roles  https://review.openstack.org/34673320:51
slagleEmilienM: oh snap20:51
slagleEmilienM: so...the net config override feature we use to install the undercloud in the multinode job is only on master :/20:51
slagleyou could possibly hack something in tripleo-ci20:52
slagleEmilienM: i had to add this for the multinode job: https://review.openstack.org/#/c/305957/20:53
slagleso that we can specify a custom net-config.json20:53
slaglewhat you could possibly do...is overwrite the packaged file when you install the mitaka undercloud since this feature is not there20:53
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Next generation HA architecture work  https://review.openstack.org/34265020:54
beekneemechEmilienM: slagle: Oh, I actually had a workaround for the eth1 thing.  I just create a dummy interface on the node, since in undercloud-only we don't care if the ctlplane address is actually accessible.20:56
beekneemechsudo ip l a dummy0 type dummy20:56
beekneemechsudo ip l set dummy0 up20:56
beekneemechWe could probably rename that to eth1 if we don't want to have to write a custom undercloud.conf entry for local_interface.20:56
beekneemechIt's what I was using for my local undercloud jobs and it worked fine.20:56
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates: Next generation HA architecture work  https://review.openstack.org/31420820:57
slaglewell i'll be darned20:57
*** jayg is now known as jayg|g0n320:59
*** trown is now known as trown|outtypewww20:59
weshayslagle, so undercloud directly on the node, and the overcloud are guests on that node or different machines?21:05
slagleweshay: they wouldn't be guests on that node21:08
slagleweshay: they could be anywhere, as tripleo.sh supports21:08
weshayok.. thanks21:08
slagleovb, bm, multinode21:08
weshayyup21:09
slaglei dont think that part matters for just installing the undercloud21:09
weshayjust getting a better idea of what you are going after21:09
slagleweshay: basically to see if the equivalent of --undercloud could be done with oooq21:12
weshayk21:12
EmilienMslagle: sorry I was afk 10 min21:13
EmilienMslagle: why not backporting it?21:14
EmilienMso much code conflict in the cherry-pick21:17
EmilienMI'm not sure it's a good idea to backport it21:17
openstackgerritBen Nemec proposed openstack-infra/tripleo-ci: Don't try to get stack details if the stack doesn't exist  https://review.openstack.org/31933721:17
openstackgerritBen Nemec proposed openstack-infra/tripleo-ci: TEST: Delete the overcloud when finished  https://review.openstack.org/29732821:17
openstackgerritBen Nemec proposed openstack-infra/tripleo-ci: Test overcloud deletion in periodic job  https://review.openstack.org/29676521:17
EmilienMI'll try beekneemech's hack21:17
EmilienMslagle: you have an opinion on backporting it?21:19
slagleEmilienM: we don't backport features21:20
EmilienMok, end of story. I'll do the hack21:20
slaglei wouldnt support backporting this for what is effectively CI purposes21:20
EmilienMokidoc21:20
openstackgerritBen Nemec proposed openstack/tripleo-docs: Group advanced topics by type  https://review.openstack.org/33523221:21
EmilienMslagle: sorry for dumb question but why does it fail?21:26
slagleEmilienM: eth1 doesn't exist21:29
EmilienMhow mitaka jobs work then?21:29
slagleinfra spawned nodes only have an eth021:30
EmilienMah we don't have multi node jobs ok21:30
EmilienMgot it...21:30
slaglemitaka jobs run on ovb21:30
EmilienMI need to sleep21:30
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci: WIP - Implement undercloud upgrade job - Mitaka -> Newton  https://review.openstack.org/34699521:33
openstackgerritPaul Belanger proposed openstack/tripleo-quickstart: Update directory layout per playbook best practices  https://review.openstack.org/34907621:35
EmilienMpabelanger: do you mind looking https://review.openstack.org/#/c/348927/ ?21:36
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Remove usage of ::nova class in THT  https://review.openstack.org/32598321:38
openstackgerritPaul Belanger proposed openstack/tripleo-quickstart: Update directory layout per playbook best practices  https://review.openstack.org/34907621:39
*** rhallisey has quit IRC21:40
openstackgerritPaul Belanger proposed openstack/tripleo-quickstart: Update directory layout per playbook best practices  https://review.openstack.org/34907621:40
openstackgerritPaul Belanger proposed openstack/tripleo-quickstart: Move the library folder into the playbooks directory  https://review.openstack.org/34908021:41
*** ayoung has quit IRC21:44
*** weshay is now known as weshay_afk21:51
*** limao has joined #tripleo21:52
*** limao_ has joined #tripleo21:55
*** limao has quit IRC21:57
*** maeca1 has quit IRC22:01
*** pradk has quit IRC22:04
*** limao has joined #tripleo22:05
openstackgerritPaul Belanger proposed openstack/tripleo-quickstart: Rename docs folder to doc  https://review.openstack.org/34908322:05
openstackgerritPaul Belanger proposed openstack/tripleo-quickstart: Initial support for sphinxdoc  https://review.openstack.org/34908422:05
*** limao_ has quit IRC22:08
openstackgerritPaul Belanger proposed openstack/tripleo-quickstart: Remove unneeded setup-hook  https://review.openstack.org/34909022:14
*** fultonj has quit IRC22:18
*** limao has quit IRC22:19
*** limao has joined #tripleo22:21
*** limao_ has joined #tripleo22:29
*** limao__ has joined #tripleo22:30
*** limao has quit IRC22:31
*** dtrainor has quit IRC22:32
*** jcoufal_ has joined #tripleo22:33
*** limao_ has quit IRC22:33
*** jcoufal has quit IRC22:34
*** limao has joined #tripleo22:40
*** limao_ has joined #tripleo22:41
*** jcoufal_ has quit IRC22:43
*** limao__ has quit IRC22:44
*** limao has quit IRC22:45
*** dtrainor has joined #tripleo22:47
*** limao has joined #tripleo22:52
*** rlandy has quit IRC22:53
*** limao__ has joined #tripleo22:54
*** limao_ has quit IRC22:55
*** limao has quit IRC22:57
*** maeca1 has joined #tripleo22:58
*** saneax_AFK is now known as saneax23:05
*** jcoufal has joined #tripleo23:07
*** limao__ has quit IRC23:07
*** limao has joined #tripleo23:09
*** limao has quit IRC23:11
*** limao has joined #tripleo23:11
openstackgerritPaul Belanger proposed openstack/tripleo-quickstart: Add bindep support  https://review.openstack.org/34911423:14
*** Ryjedo has quit IRC23:15
*** morazi has quit IRC23:28
*** thrash is now known as thrash|wknd23:32
*** rwsu has quit IRC23:34
*** limao_ has joined #tripleo23:35
*** limao has quit IRC23:38
*** limao has joined #tripleo23:44
*** limao_ has quit IRC23:48
*** jcoufal_ has joined #tripleo23:48
*** jcoufal has quit IRC23:49
*** limao_ has joined #tripleo23:51
*** limao__ has joined #tripleo23:54
*** limao has quit IRC23:54
*** limao_ has quit IRC23:58
*** limao__ has quit IRC23:59

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