Wednesday, 2016-08-17

*** limao_ has joined #tripleo00:46
*** Guest67611 is now known as saneax_AFK00:53
*** chlong has joined #tripleo00:58
openstackgerritMerged openstack/tripleo-heat-templates: Set coordination url for gnocchi storage  https://review.openstack.org/35437801:00
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Pass ServiceNetMap to services  https://review.openstack.org/35441401:03
*** bana_k has quit IRC01:13
*** Ryjedo_ has joined #tripleo01:20
*** Ryjedo has quit IRC01:22
*** Ryjedo_ is now known as Ryjedo01:22
*** fragatina has quit IRC01:34
*** dtrainor has quit IRC01:36
*** julim has joined #tripleo01:46
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: gnocchi: use new authtoken parameters  https://review.openstack.org/35558501:55
*** padkrish has joined #tripleo02:06
*** julim has quit IRC02:17
*** yamahata has quit IRC02:20
*** bkopilov has quit IRC02:21
*** tzumainn has quit IRC02:25
*** dmacpher has joined #tripleo02:29
*** thrash is now known as thrash|g0ne02:29
*** padkrish has quit IRC02:49
*** padkrish has joined #tripleo02:50
*** padkrish has quit IRC02:54
*** fragatina has joined #tripleo03:00
*** fragatina has quit IRC03:04
*** limao_ has quit IRC03:12
*** egafford has left #tripleo03:17
*** r-mibu has quit IRC03:19
*** r-mibu has joined #tripleo03:25
*** bswartz has joined #tripleo03:26
*** padkrish has joined #tripleo03:28
*** bkopilov has joined #tripleo03:37
*** trozet has quit IRC03:45
*** bana_k has joined #tripleo03:47
*** jeckersb is now known as jeckersb_gone03:49
openstackgerritTim Rozet proposed openstack/tripleo-heat-templates: Fixes Neutron metadata parameters in THT  https://review.openstack.org/35598703:49
*** padkrish_ has joined #tripleo03:49
*** padkrish has quit IRC03:52
*** links has joined #tripleo03:54
*** trozet has joined #tripleo03:59
*** coolsvap has joined #tripleo04:04
*** limao_ has joined #tripleo04:13
*** limao_ has quit IRC04:17
*** apetrich has joined #tripleo04:22
*** masco has joined #tripleo04:26
*** bswartz has quit IRC04:27
*** davidlenwell has quit IRC04:42
*** davidlenwell has joined #tripleo04:55
*** ChanServ sets mode: +v davidlenwell04:55
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Now fly and set cinder::api::bind_host within service template  https://review.openstack.org/31834405:08
*** jaosorior has joined #tripleo05:13
*** limao_ has joined #tripleo05:23
*** trozet has quit IRC05:25
*** limao_ has quit IRC05:25
*** limao_ has joined #tripleo05:26
*** numans has joined #tripleo05:26
*** liverpooler has joined #tripleo05:40
*** rcernin has joined #tripleo05:46
*** benoit_ has quit IRC05:49
*** benoit_ has joined #tripleo05:51
*** saneax_AFK is now known as saneax05:52
*** saneax is now known as Guest1805905:52
openstackgerritKarthik S proposed openstack/tripleo-heat-templates: Customize the reservation of memory for host processes  https://review.openstack.org/32770505:53
openstackgerritKarthik S proposed openstack/tripleo-heat-templates: Customize the reservation of memory for host processes  https://review.openstack.org/32770505:55
openstackgerritKarthik S proposed openstack/tripleo-heat-templates: Configure the pci_passthrough_whitelist via THT  https://review.openstack.org/32941506:03
*** jprovazn has joined #tripleo06:03
openstackgerritMerged openstack/puppet-tripleo: Defaults empty hash to kernel_modules and sysctl_settings  https://review.openstack.org/35561506:06
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: DO NOT MERGE - TESTING BARBICAN  https://review.openstack.org/35326206:06
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Add Barbican to the overcloud - WIP  https://review.openstack.org/35245806:06
openstackgerritMerged openstack/puppet-tripleo: Align hiera keys with service names  https://review.openstack.org/35358206:06
openstackgerritHonza Pokorny proposed openstack/tripleo-ui: Add packaging spec and config files  https://review.openstack.org/34493206:13
jaosorioradarazs: Hey, can you check out this commit? https://review.openstack.org/#/c/351683/06:13
*** Guest18059 is now known as saneax_AFK06:15
*** bvandenh has quit IRC06:16
*** gfidente has quit IRC06:18
*** ebalduf has quit IRC06:21
bandinimorning06:33
openstackgerritSanjay Upadhyay proposed openstack/tripleo-specs: tripleo-sriov  https://review.openstack.org/31387206:34
*** aufi has joined #tripleo06:35
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Enable aodh systemd services on non-bootstrap nodes  https://review.openstack.org/35611006:36
openstackgerritKarthik S proposed openstack/tripleo-heat-templates: Customize vcpu_pin_set in nova.conf  https://review.openstack.org/34377006:38
*** milan has joined #tripleo06:44
*** pgadiya has joined #tripleo06:44
openstackgerritKarthik S proposed openstack/tripleo-heat-templates: Customize vcpu_pin_set in nova.conf  https://review.openstack.org/34377006:48
*** radeks has joined #tripleo06:50
*** bana_k has quit IRC06:51
*** leanderthal|afk is now known as leanderthal06:54
*** tesseract- has joined #tripleo07:00
*** gfidente has joined #tripleo07:00
*** matbu|afk is now known as matbu07:03
*** ioggstream has joined #tripleo07:05
* gfidente earlier today07:05
gfidenteioggstream !07:05
ioggstreamgfidente: giulivo!07:05
gfidentewhat do you do around here and how come I only noticed today!07:05
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci: Don't use AFS mirrors in no nodepool environment  https://review.openstack.org/35000307:15
*** sshnaidm|afk has left #tripleo07:15
*** sshnaidm|afk has joined #tripleo07:15
*** zoli_gone-proxy is now known as zoliXXL07:18
*** bootsha has joined #tripleo07:19
*** ebarrera has joined #tripleo07:19
*** Goneri has joined #tripleo07:20
openstackgerritMerged openstack/puppet-tripleo: Add cinder-backup profiles  https://review.openstack.org/31121807:20
*** zoliXXL is now known as zoli|wfh07:22
jaosoriorgfidente: hey man, can you help me out figuring out what's up with this? https://review.openstack.org/#/c/352458/907:22
gfidentejaosorior that is unfair!07:22
gfidenteI ask for votes not reviews!07:23
gfidentethough I owe you some07:23
gfidente:)07:23
*** jpich has joined #tripleo07:23
jaosoriorlol07:23
gfidentejaosorior thanks for checking the net_ip_stuff indeed07:24
gfidentehopefully it passes07:24
*** athomas has joined #tripleo07:24
gfidenteathomas fwiw looks like we're set for the ceph upgrade process and will start pushing actual stuff today07:26
gfidentejaosorior you mean "ERROR: Property error: : resources.VipDeployment.properties.input_values: : The Parameter (ServiceNetMap) was not provided." right?07:28
*** yamahata has joined #tripleo07:29
gfidenteseems misleading, can't think why VipDeployment should know about ServiceNetMap07:29
jaosoriorgfidente: yeah dude, what the hell :/07:30
*** akuznetsov has joined #tripleo07:31
openstackgerritJuan Antonio Osorio Robles proposed openstack-infra/tripleo-ci: Update TLS endpoints  https://review.openstack.org/35628607:35
*** florianf has joined #tripleo07:37
jaosoriorgfidente: we need that ^^ some services such as ironic and manila will have issues in the nonha CI (which I guess ironic might be encountering at some point) because that's the mapping we actually use07:38
*** atom_ has joined #tripleo07:38
*** radeks has quit IRC07:38
gfidentejaosorior right that's the usual sync_up_in_ci_task07:40
jaosoriorgfidente: yeah :(07:41
gfidenteon barbican issue I think BarbicanApiNetwork will sort it07:41
gfidentebut then it might race for merge conflict with the ServiceNetMap submission07:41
gfidentenot sure if it makes sense to rebase07:41
gfidentewdyt?07:41
jaosoriorgfidente: no biggie, I don't think barbican is close to merging07:41
jaosoriorI'll just rebase as needed07:42
jaosoriorlets merge the servicenetmap stuff first07:42
*** dtantsur|afk is now known as dtantsur07:49
dtantsurmorning07:50
gfidenteyou guys spam openstack-dev with so much stuff07:55
*** ioggstream has quit IRC07:55
gfidentelooks like I should write there more07:55
*** abehl has joined #tripleo07:58
*** hewbrocca-afk is now known as hewbrocca07:58
jaosoriorgfidente: let there be merges!07:59
gfidentewohooo :)08:01
hewbroccaMorning all08:02
openstackgerritJiri Tomasek proposed openstack/tripleo-ui: Stop Validation  https://review.openstack.org/33656108:02
openstackgerritJiri Tomasek proposed openstack/tripleo-ui: Triggering validations and messaging  https://review.openstack.org/33656008:02
openstackgerritJiri Tomasek proposed openstack/tripleo-ui: List Validations and Validation results  https://review.openstack.org/33477008:02
*** akuznetsov has quit IRC08:02
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates: Availability monitoring agents support  https://review.openstack.org/25478808:03
*** xuao has joined #tripleo08:04
*** padkrish_ has quit IRC08:05
jtomasekflorianf: Hi, could you please check this one? https://review.openstack.org/#/c/355937/ That erorr breaks current master08:06
gfidentejaosorior it actually worked, cinder bind_host is set correctly08:06
*** abehl has quit IRC08:06
gfidenteso looks like we can start switching all bind settings in the services! :)08:07
*** abehl has joined #tripleo08:07
florianfjtomasek: sure. gimme a sec...08:08
jaosoriorgfidente: awwww yeaaaaah08:08
*** ccamacho has joined #tripleo08:08
*** coolsvap is now known as coolsvap_08:09
*** chem has joined #tripleo08:09
florianfjtomasek: done08:10
jtomasekflorianf: thanks!08:12
dtantsurjistr, morning! if you have some time: https://review.openstack.org/#/c/341433/ (the patch I was talking about yesterday)08:12
*** stendulker has joined #tripleo08:13
*** lucas-dinner is now known as lucasagomes08:17
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Fix deployment steps for AODH profiles  https://review.openstack.org/35237208:21
*** ohamada has joined #tripleo08:22
*** bootsha_ has joined #tripleo08:23
*** rlandy has quit IRC08:24
*** bootsha has quit IRC08:24
openstackgerritMerged openstack/tripleo-ui: Fix Error on Version loader  https://review.openstack.org/35593708:24
*** paramite has joined #tripleo08:31
*** jbadiapa has quit IRC08:32
openstackgerritMerged openstack/tripleo-heat-templates: Add cinder-backup service as role  https://review.openstack.org/30456308:33
*** Jaison has joined #tripleo08:35
*** links has quit IRC08:37
jaosorioreasy commit https://review.openstack.org/#/c/355768/08:37
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates: Make sahara run via systemd  https://review.openstack.org/35590408:39
jistrdtantsur: added myself there, will try to review but not promising anything :) the BZ force is strong today08:45
*** limao_ has quit IRC08:48
*** jbadiapa has joined #tripleo08:50
dtantsurno worries, thanks :)08:50
dtantsuraha, so now your gate is also affected by check-osc-plugins instability...08:52
*** limao has joined #tripleo08:56
*** derekh has joined #tripleo08:57
*** oneswig has joined #tripleo08:59
*** atom_ has quit IRC08:59
*** gfidente has quit IRC09:07
openstackgerritMerged openstack/tripleo-heat-templates: Remove deprecated net_ip_subnet_map output  https://review.openstack.org/34605509:12
*** sshnaidm|afk is now known as sshnaidm09:13
openstackgerritMerged openstack/tripleo-heat-templates: Remove deprecated net_ip_uri_map output  https://review.openstack.org/35438109:13
*** atom_ has joined #tripleo09:15
openstackgerritMerged openstack/tripleo-heat-templates: Remove unneeded net_ip_map dump into hieradata  https://review.openstack.org/35439109:17
*** yamahata has quit IRC09:24
*** gfidente has joined #tripleo09:26
*** bootsha_ has quit IRC09:28
*** bootsha has joined #tripleo09:30
adarazsjaosorior: checking out the change you sent.09:34
*** aqkhan__ has joined #tripleo09:35
*** aqkhan_ has quit IRC09:36
*** aqkhan_ has joined #tripleo09:36
*** atom_ has quit IRC09:37
jaosorioradarazs: thanks09:37
*** pkovar has joined #tripleo09:38
*** electrofelix has joined #tripleo09:39
adarazsjaosorior: is there any reason you create the file as 0600? or just copied the task above?09:40
*** aqkhan__ has quit IRC09:40
openstackgerritJiri Tomasek proposed openstack/tripleo-ui: Stop Validation  https://review.openstack.org/33656109:40
openstackgerritJiri Tomasek proposed openstack/tripleo-ui: Triggering validations and messaging  https://review.openstack.org/33656009:40
openstackgerritJiri Tomasek proposed openstack/tripleo-ui: List Validations and Validation results  https://review.openstack.org/33477009:40
openstackgerritJiri Tomasek proposed openstack/tripleo-ui: Parameters listing and updating  https://review.openstack.org/35155409:41
adarazsor because it can potentially have passwords in it later on?09:41
*** tosky has joined #tripleo09:41
jaosorioradarazs: it's been copied. Though I think that makes sense, we only want the owner of the file to be able to do operations on it09:42
jaosorioradarazs: hieradata_override can be used for several other things, so a user might start putting passwords or something of the sort there09:42
adarazsyeah, that's what I thought, okay.09:43
jaosoriorgfidente: crap, seems that with something that merged recently the ServiceNetMap commit no longer passes :(https://review.openstack.org/#/c/354414/09:44
gfidentejaosorior right cinder-backup didn't have those09:49
gfidenteadding now09:49
adarazsjaosorior: found a problem with the patch but it's easy to fix09:51
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Pass ServiceNetMap to services  https://review.openstack.org/35441409:52
jaosorioradarazs: that's interesting09:52
jaosorioradarazs: now I'm confused how it's working09:53
gfidentejaosorior ^^ that's cool it means validations were working as intended :)09:53
adarazsjaosorior: yeah, it's only defined in roles/libvirt/defaults/main.yml09:53
jaosorioradarazs: but it's being deployed with the correct value https://ci.centos.org/artifacts/rdo/jenkins-tripleo-quickstart-gate-master-delorean-quick-basic-464/undercloud/home/stack/quickstart-hieradata-overrides.yaml.gz09:53
adarazsjaosorior: not sure. solution could be that we pass it on the command line (and it would become global), but I don't see it in any of the usual config files we pass.09:53
*** openstack has joined #tripleo10:16
*** oneswig has quit IRC10:18
*** xuao has quit IRC10:24
*** bootsha has quit IRC10:28
*** milan has quit IRC10:32
*** bootsha has joined #tripleo10:37
*** bootsha has quit IRC10:39
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates: Composable Mistral services  https://review.openstack.org/32343610:39
openstackgerritGiulio Fidente proposed openstack/instack-undercloud: Enable sysctl nonlocal_bind  https://review.openstack.org/35637210:43
openstackgerritCarlos Camacho proposed openstack-infra/tripleo-ci: Set Mistral workers to 1  https://review.openstack.org/35637510:45
*** hjensas has quit IRC10:45
*** hjensas has joined #tripleo10:46
*** hjensas has quit IRC10:46
*** hjensas has joined #tripleo10:46
*** limao has quit IRC10:48
tbarronping Jokke_ re manila deployment and what we're missing ...10:49
*** pkovar has quit IRC10:50
*** paramite has quit IRC10:52
dtantsurfolks, what does this do: https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/services/heat-api.yaml#L44-L48 ?10:53
*** paramite has joined #tripleo10:53
*** bootsha has joined #tripleo10:56
slagledtantsur: that's hieradata10:59
slagleit opens those ports in iptables10:59
*** masco has quit IRC11:00
shadowerd0ugal: do you happen to know how does the node registration json file get passed to mistral?11:02
shadowerd0ugal: pinging you because you're the mistral expert :-)11:02
openstackgerritMerged openstack/tripleo-quickstart: Expand templated bash to allow for doc generation  https://review.openstack.org/34759211:03
dtantsurslagle, interesting, ironic-api works without it for me. is it always required?11:03
ccamachodtantsur moved to composable firewall rules here  (Prev. here: puppet/hieradata/controller.yaml) https://github.com/openstack/tripleo-heat-templates/commit/5195d7f8910f7d1ce0895caa133b028a727f862211:04
openstackgerritMerged openstack/tripleo-heat-templates: Update Ironic composable services  https://review.openstack.org/34143311:08
dtantsur\o/11:08
openstackgerritMerged openstack-infra/tripleo-ci: Update TLS endpoints  https://review.openstack.org/35628611:08
dtantsurccamacho, interesting, it didn't have ironic support as well..11:08
dtantsurok, have to run now, I guess I'll figure out later11:08
dtantsurthanks for review on the ironic thing jistr!11:08
*** zoli|wfh is now known as zoli_gone-proxy11:09
ccamachosy!11:09
openstackgerritTomas Sedovic proposed openstack/tripleo-validations: Validate the instackenv format  https://review.openstack.org/35395611:10
openstackgerritMerged openstack/tripleo-docs: Fix indentation for SSL CA configuration  https://review.openstack.org/35576811:10
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Configure galera-monitor on all controller nodes  https://review.openstack.org/35639011:11
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Populate HAProxy endpoint ports from EndpointMap  https://review.openstack.org/35536611:11
*** dtantsur is now known as dtantsur|bbl11:11
*** stendulker has quit IRC11:14
jpichshadower: Once the environment file is parsed, the JSON is passed as-is, as a normal input parameter - is this what you are asking about? https://github.com/openstack/tripleo-common/blob/master/workbooks/baremetal.yaml#L30 - https://github.com/openstack/tripleo-common/blob/master/tripleo_common/actions/baremetal.py#L2411:15
*** matbu is now known as matbu|lch11:16
*** zoli_gone-proxy is now known as zoliXXL11:19
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates: Composable Mistral services  https://review.openstack.org/32343611:20
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Now fly and set cinder::api::bind_host within service template  https://review.openstack.org/31834411:21
*** lucasagomes is now known as lucas-hungry11:23
*** limao has joined #tripleo11:25
shadowerjpich: I've seen that but what I don't understand is how does the json file (assuming it is still a file) get passed into that workbook11:26
shadoweri.e. what code reads it, how does it know which path to check, etc.11:26
*** bootsha has quit IRC11:27
*** tzumainn has joined #tripleo11:27
shadowerjpich: I have a validation that checks the nodes file for potential errors and I'd like to make sure the validation reads the same file that is going to be used for the registration11:27
*** bootsha has joined #tripleo11:27
jpichshadower: It's no longer a file by then, just a dictionary AFAICT -> https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/baremetal.py#L23011:28
jpichthat file calls to https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/workflows/baremetal.py#L18 which just passes the dict along11:29
*** thrash|g0ne is now known as thrash11:29
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates: Do not merge, checking composable mistral.  https://review.openstack.org/35294711:30
*** ramishra has quit IRC11:30
shadowerjpich: so I'm asking about the file being read from: https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/baremetal.py#L191-L19611:31
*** ramishra has joined #tripleo11:32
slaglegfidente: jistr : see https://review.openstack.org/#/c/355612/11:34
jpichshadower: Ok! Personally I use instackenv.json because I'm still only following the docs. The client makes magic from that file, and passes the resulting node dictionary into the workflow. A.k.a. the existing validation should work??11:35
*** pino|work has left #tripleo11:35
*** jaosorior has quit IRC11:35
shadowerjpich: yeah. I just wouldn't want to hardcode `instackenv.json` any more11:36
shadowerbut that does work so  far11:36
jistrslagle: ack, thanks11:36
*** jaosorior has joined #tripleo11:36
*** bootsha has quit IRC11:37
d0ugalshadower: Sorry, was getting some lunch.11:37
jpichshadower: Cool! Maybe d0ugal will be able to clarify better if my half-baked answer is a bit suspicious :)11:38
d0ugalshadower: Did you get an answer?11:38
jpichAnswer fragments and loadsa links11:38
d0ugallol11:38
* d0ugal reads11:38
openstackgerritMerged openstack/instack-undercloud: Enable sysctl nonlocal_bind  https://review.openstack.org/35505111:38
openstackgerritMerged openstack/instack-undercloud: (mitaka only) enable sysctl nonlocal_bind  https://review.openstack.org/35561211:38
shadowerd0ugal: basically, how does the instackenv.json file gets to mistral? And is that filename/path hardcoded?11:39
d0ugalshadower, jpich - yeah, so it is just a python dict. Mistral doesn't accept files.11:39
*** bkopilov has quit IRC11:39
d0ugalshadower: The CLI reads it and sends it as JSON, it isn't stored on the Mistral side.11:39
shadowerd0ugal: this is where the cli reads it, right? https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/baremetal.py#L191-L19611:39
d0ugalshadower: Yup, and then passes to to the workflow at L23211:40
d0ugalshadower: (well, passes it to a function that wraps the workflow, but yeah)11:40
d0ugalshadower: The file name is specified by the user11:41
shadowerd0ugal: right, okay. Thanks11:41
*** maeca1 has joined #tripleo11:41
d0ugalshadower: Do you need to do something similar?11:41
shadowerd0ugal: yeah, we have a validation that makes sure the file is correct, the nodes accessible etc. And I just wanted to make sure it uses the same file11:42
shadowerd0ugal: I suppose the GUI will generate the contents and pass them to mistral directly, but in the CLI workflow, someone has to write it first, right?11:43
d0ugalshadower: Yeah, except for a virt setup when instack creates it11:44
shadowerright11:44
*** bootsha has joined #tripleo11:45
*** sshnaidm is now known as sshnaidm|afk11:46
openstackgerritJulie Pichon proposed openstack/python-tripleoclient: [WIP] Add 'openstack overcloud node configure' command  https://review.openstack.org/34487511:48
*** weshay has joined #tripleo11:48
weshaysshnaidm|afk, ping me when you're back re: the tripleo-pin11:49
EmilienMgood morning11:52
EmilienMcan someone look https://review.openstack.org/#/c/355585/ please?11:52
weshayslagle, EmilienM wondering if we can trigger the periodic jobs to hit https://trunk.rdoproject.org/centos7/97/0d/970d046f1ad304b309a6b6b610faf7a52477fbf1_073a7efe to update the tripleo pin11:53
* weshay looks into it11:53
EmilienMweshay: to run periodic job, slagle did like this https://review.openstack.org/#/c/346949/11:54
openstackgerritTomas Sedovic proposed openstack/tripleo-validations: Check for differences between introspected HW  https://review.openstack.org/35397811:55
*** xuao has joined #tripleo11:56
*** morazi has joined #tripleo11:59
slagleweshay: why?12:00
slagleweshay: we're blocked on https://bugs.launchpad.net/tripleo/+bug/161308812:01
openstackLaunchpad bug 1613088 in tripleo "tripleo-ci: periodic nonha job failing to promote due to SSL" [Critical,In progress] - Assigned to Emilien Macchi (emilienm)12:01
weshayslagle, the tripleo-pin is older than rdo and rhosp as soon as the import is done12:01
*** julim has joined #tripleo12:01
slaglethere's no reason to promote a repo that doesnt work with ssl12:01
*** jayg|g0n3 is now known as jayg12:01
slagleweshay: does rdo test with ssl on the undercloud?12:01
weshayno :(12:01
slaglethen that's how you promoted12:01
*** xuao has quit IRC12:02
weshayya12:02
weshayok.. glad I checked12:02
weshaythanks12:02
*** coolsvap_ is now known as coolsvap12:03
*** rhallisey has joined #tripleo12:06
openstackgerritTomas Sedovic proposed openstack/tripleo-validations: Validation to verify connectivity to overcloud  https://review.openstack.org/35393412:07
*** rhallisey_ has joined #tripleo12:08
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Enable usage of FQDNs for the endpoints  https://review.openstack.org/35641612:09
*** xuao has joined #tripleo12:09
*** xuao has quit IRC12:10
*** florianf has quit IRC12:11
*** rhallisey has quit IRC12:12
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates: Availability monitoring agents support  https://review.openstack.org/25478812:13
*** florianf has joined #tripleo12:13
*** rhallisey_ is now known as rhallisey12:13
*** rodrigods has quit IRC12:14
*** rodrigods has joined #tripleo12:14
*** jpena is now known as jpena|lunch12:15
*** dprince has joined #tripleo12:16
*** asbishop has joined #tripleo12:18
*** chem has quit IRC12:18
*** chem has joined #tripleo12:19
openstackgerritPradeep Kilambi proposed openstack/puppet-tripleo: Move ceilometer api to run under apache wsgi  https://review.openstack.org/35247612:19
*** matbu|lch is now known as matbu12:20
*** milan has joined #tripleo12:24
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add tripleo-firewall composable service  https://review.openstack.org/35561412:25
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add tripleo-packages composable service  https://review.openstack.org/35560512:25
*** lucas-hungry is now known as lucasagomes12:26
jaosoriordprince how do we plan to enforce that firewall and packages are always included?12:27
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Add HAProxy TLS handled by certmonger as composable service  https://review.openstack.org/35643012:28
*** weshay is now known as weshay_mtg12:28
dprincejaosorior: I added them to all the default lists12:28
dprincejaosorior: I don't think we need a global include12:28
openstackgerritMerged openstack/puppet-tripleo: Fix deployment steps for AODH profiles  https://review.openstack.org/35237212:29
dprincejaosorior: for example... with docker I don't actually need tripleo::packages (Because I used 'tags' to only run part of the catalog anyway)12:29
jaosoriordprince: well, if someone forgets to include those we're gonna run into some really weird errors12:29
dprincejaosorior: the defaults are in there, They are included by default12:30
jaosoriordprince: I planned to add global configurations for CAs done via hieradata (instead of the bash script we use now) and I was told to do it with composable services instead12:30
jaosoriordprince: so there's no way to remove them?12:30
jaosoriordprince:12:31
jaosoriorno12:31
jaosoriorbut that's the default12:31
dprincejaosorior: sorry, not following that last bit. Got a patch?12:31
jaosoriorwhich we expect people to change12:31
jaosoriordprince: so people will configure their services that by overriding the 'ControllerServices' parameter12:31
dprincejaosorior: it is composable. If I don't want firewall enabled then I can exclude tripleo::firewall entirely.12:31
dprincejaosorior: There are multiple ways people can control enabled services12:32
openstackgerritwes hayutin proposed openstack/tripleo-quickstart: add ssl option as default to True for the undercloud  https://review.openstack.org/35643112:32
dprincejaosorior: that is one way12:32
jaosoriordprince: ok, I guess that applies there.12:32
dprincejaosorior: another is to use OS::Heat::None to noop out a service that is enabled for a role by default12:32
jaosoriordprince: ok, I get your point12:33
jaosoriordprince: now, regarding the CA stuff. I could easily do that as a composable service12:33
dprincegfidente: hi, I'm working on a patch which adds a mechanism to deal with the "top level" passwords for composable services. Things like the MySQL password, rabbit cookie, etc.12:33
jaosoriordprince: but... if we don't include the CA in some node. With services that communicate via TLS, we will get very funky errors with SSL validation12:33
jaosoriordprince: any idea how to enforce the usage of that one?12:34
dprincejaosorior: do it composable I think12:34
jaosoriordprince: that doesn't address the issue12:34
dprincejaosorior: I don't want to add any more "features" to overcloud.yaml (i.e. a global mechanism) until we get the Mistral templating done12:34
openstackgerritBrad P. Crochet proposed openstack/tripleo-heat-templates: Composable Zaqar services  https://review.openstack.org/33168212:34
dprincejaosorior: they way I view your problem is we should validate it at the appropriate time12:35
dprincejaosorior: so we'll have a Mistral workflow, that generates overcloud.yaml12:35
dprincejaosorior: if we enable the CA TLS stuff for any roles... you are saying it needs to be enabled for all roles. SO perhaps we have a validation to catch this early on?12:36
jaosoriordprince: that sounds reasonable12:36
dprincejaosorior: or, like you suggest we have a global services section we could add and use it for things like tripleo::packages, tripleo::firewall, tripleo::ca_tls_stuff....12:37
dprincejaosorior: or we perhaps do both12:37
dprincejaosorior: validations, and global config section12:37
*** jprovazn has quit IRC12:38
dprincejaosorior: I'd rather have composable roles than the global validations feature at this point. So I'd say hold on it until we get the templating work done12:38
jaosoriordprince: or I guess there could be a set of base services that are always enabled. And it will throw you a big warning if you try to disable something of the sort12:38
dprincejaosorior: just be careful with your assumuptions about something always being enabled. With new deployment types (containers for example) we need to be flexible12:39
dprincejaosorior: so long as it is configurable though it should be fine12:39
jaosoriordprince: if you set no CA certs then none will be configured12:39
dprincejaosorior: sounds reasonable12:40
openstackgerritPradeep Kilambi proposed openstack/tripleo-heat-templates: Upgrade scripts to migrate aodh alarm data  https://review.openstack.org/35304212:40
jaosoriordprince: but if you only enable it for some nodes, and a node without that CA cert configured tries to access a TLS endpoint, then we will get issues12:40
jaosoriorand OpenSSL is not explicit at all with the validation errors it gives12:40
jaosoriorso the bug reports we'll get (and similar things have happened before) are gonna be cryptic as hell12:40
jaosoriordprince: been down that road, that's why I point it out12:41
dprincejaosorior: gotcha, so a high level validation sounds like it might fit here. If the feature is enabled... it should be in use everywhere12:41
*** jprovazn has joined #tripleo12:41
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add manifests to inject and trust CA certificates  https://review.openstack.org/35302112:43
*** zoliXXL is now known as zoli|lunch12:46
*** fultonj_ has joined #tripleo12:46
*** tosky has quit IRC12:47
*** rlandy has joined #tripleo12:47
*** bswartz has joined #tripleo12:48
openstackgerritwes hayutin proposed openstack/tripleo-quickstart: [WIP] add ssl option as default to True for the undercloud  https://review.openstack.org/35643112:48
*** weshay_mtg is now known as weshay12:49
*** tosky has joined #tripleo12:49
*** aqkhan__ has joined #tripleo12:50
*** aqkhan_ has quit IRC12:51
*** aqkhan_ has joined #tripleo12:51
*** sshnaidm|afk is now known as sshnaidm12:52
sshnaidmweshay, hi12:53
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Add deployment of CAs via hieradata  https://review.openstack.org/35303712:55
*** aqkhan__ has quit IRC12:55
jaosoriordprince ^^12:55
weshaysshnaidm, can you help drive https://bugs.launchpad.net/tripleo/+bug/1613088 and the related reviews so we can get a tripleo pin update12:55
openstackLaunchpad bug 1613088 in tripleo "tripleo-ci: periodic nonha job failing to promote due to SSL" [Critical,In progress] - Assigned to Emilien Macchi (emilienm)12:55
weshayvia the periodic jobs12:55
sshnaidmweshay, I think it's on ironic side12:56
sshnaidmdtantsur|bbl, ^^12:56
EmilienMI'm not sure there is anything to drive12:56
EmilienMslagle is waiting for his patch to land in ironic12:56
EmilienMand I'm waiting for my fix to land in puppet12:56
weshayk12:56
EmilienMeverything is under control already12:57
EmilienMbut thanks!12:57
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates: Monitoring node support  https://review.openstack.org/34969012:58
openstackgerritSaravanan KR proposed openstack/python-tripleoclient: NTP arg should be optional when specified in an environment  https://review.openstack.org/33269412:59
hewbroccathere is no telling how long we will wait for Ironic :(12:59
*** lblanchard has joined #tripleo13:00
*** paramite has quit IRC13:03
*** jprovazn has quit IRC13:04
*** bkopilov has joined #tripleo13:05
*** jprovazn has joined #tripleo13:06
*** pradk has quit IRC13:06
jtomasekshadower: is this the set of validations patches that mandre reworked? https://review.openstack.org/#/c/318194/13:10
shadowerjtomasek: yep!13:11
*** coolsvap is now known as _coolsvap_13:11
jtomasekshadower: ok, I've followed the README and validations work ok with it in GUI13:11
shadowerjtomasek: \o/13:11
shadowerjtomasek: could you say so on the patches?13:12
*** bootsha has quit IRC13:12
jtomasekshadower: done13:13
shadowerthanks!13:13
*** trozet has joined #tripleo13:15
*** fultonj_ has quit IRC13:15
*** bootsha has joined #tripleo13:15
*** florianf has quit IRC13:16
*** fultonj_ has joined #tripleo13:16
*** fultonj_ has quit IRC13:16
*** jpena|lunch is now known as jpena13:18
*** Vijayendra has quit IRC13:18
*** limao has quit IRC13:19
*** florianf has joined #tripleo13:20
*** fultonj has quit IRC13:20
*** egafford has joined #tripleo13:21
*** fultonj has joined #tripleo13:22
openstackgerritKevin Carter (cloudnull) proposed openstack/diskimage-builder: Disabled IPv6 privacy extensions  https://review.openstack.org/35645613:23
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Use modulepath for Puppet  https://review.openstack.org/35645713:23
ansiwenEmilienM: thanks for the review13:23
EmilienMansiwen: yw13:24
*** dtantsur|bbl is now known as dtantsur13:24
dtantsursshnaidm, EmilienM, and we're broken by nova :( and nova patch is still in gate :( unlucky week13:25
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Refresh puppet module symlinks after every package update  https://review.openstack.org/35602813:25
sshnaidmdtantsur, and what is nova issue?13:26
EmilienMdtantsur: yeah I  noticed13:26
dtantsurlemme find13:26
shadowerd0ugal: paths in here: https://github.com/openstack/tripleo-common/blob/master/tripleo_common/constants.py -- are they overridable by the user?13:26
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Permit usage of raw disks as backends for Swift storage  https://review.openstack.org/35084113:26
EmilienMdtantsur: it would be so cool to integrate a tripleo-bug-dashboard into tripleo.org13:27
openstackgerritFlorian Fuchs proposed openstack/tripleo-ui: Use Mistral API for plan deletion  https://review.openstack.org/35645813:27
dtantsurEmilienM, +113:27
*** bootsha has quit IRC13:27
openstackgerritKarthik S proposed openstack/tripleo-heat-templates: Add sriov_numvfs in THT  https://review.openstack.org/34535213:27
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add DefaultPasswords to composable services  https://review.openstack.org/35646113:28
sshnaidmEmilienM, dtantsur I started to do some "current status", currently manual: https://etherpad.openstack.org/p/tripleo-ci-status13:28
dtantsurEmilienM, sshnaidm, so we need https://review.openstack.org/#/c/355659/ to merge. it's been in the gate for 11 hours already13:28
dprincegfidente: https://review.openstack.org/356461, this is how I'd like to solve the passwords problem13:28
EmilienMdtantsur: right, I'm monitoring it13:28
dprinceEmilienM, jaosorior: you too. This passwords issue will allow me to proceed with cleaning out controller.yaml now13:29
dprinceEmilienM, jaosorior: https://review.openstack.org/35646113:29
*** tbonds has quit IRC13:30
gfidentedprince oh to get them from where they are and pass as param so they don't change, I see13:30
*** tbonds has joined #tripleo13:30
dprincegfidente: yes, and I can use YAQL to make it possible to provide a custom (override) parameter too within the composable services themselves. If it isn't set then it'll use the top level historical heat default for these though13:31
EmilienMdprince: oh nice13:31
dtantsurfolks, how does haproxy work in tripleo? I remember adding ironic:true to https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/services/haproxy.yaml and then ironic-api failed to start because something was already listening on its port13:31
dtantsurdo we use some different port in this case or what?13:32
gfidentedprince so we can move the password setting code in the service too13:32
gfidentethanks13:32
openstackgerritFlorian Fuchs proposed openstack/tripleo-ui: Use Mistral API for plan deletion  https://review.openstack.org/35645813:32
dprincegfidente: yes, I'm doing it now... assuming you buy my approach13:33
* dprince assumes :)13:33
d0ugalshadower: I guess it depends where and how they are used13:33
d0ugalshadower: That file is a bit of a mixed bag of things13:33
gfidentedprince sure13:33
gfidentealso we need to figure how to get some icecream in barcelona13:33
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add DefaultPasswords to composable services  https://review.openstack.org/35646113:33
hewbroccagfidente: have jistr bring you some zmrzlina from here13:35
*** bootsha has joined #tripleo13:35
shadowerd0ugal: ah, ok. I'm thinking about a new value that's not been merged yet13:35
jtomasekd0ugal, rbrady: I've added comment in https://review.openstack.org/#/c/29873213:36
EmilienMdprince: can you +A https://review.openstack.org/#/c/355585/ please?13:38
jistryea let's export central-european ice cream to the southern countries :D13:38
hewbroccaA little dry ice and you'll be golden13:39
jistrmaybe we'll sell a piece or two before getting laughed at13:39
hewbroccawhat could go wrong?13:39
openstackgerritFlorian Fuchs proposed openstack/tripleo-ui: Move plan listing to use Mistral  https://review.openstack.org/34791913:39
*** jcoufal has joined #tripleo13:39
gfidentehewbrocca jistr I think you are invited13:40
bandinijistr: if it has beer in it, I'll buy13:40
jistrif it doesn't have beer in it, it probably has at least garlic13:41
gfidentejistr++13:41
d0ugalshadower: right, I would use that for defaults, but allow users to replace probably. depends what it is I guess :)13:41
openstackgerritwes hayutin proposed openstack/tripleo-quickstart: add ssl option as default to True for the undercloud  https://review.openstack.org/35643113:41
gfidentebandini ok you can come as well13:41
gfidentebut please don't extend too much the circle13:41
gfidenteI know what type you are13:42
bandinijistr: ahahah13:42
bandinigfidente: am in the car already!13:42
bandinitype o-negative13:42
gfidentethe band?13:42
bandinimy type13:42
gfidenteor the blood?13:42
gfidentehewbrocca the other day I read flavour13:43
gfidenteand it made me very sad13:43
gfidenteI will never understand13:43
gfidente(I meant to say read the word 'flavour' on a random paper)13:43
gfidenteI will remember, but not understand13:44
*** jeckersb_gone is now known as jeckersb13:44
*** zoli|lunch is now known as zoli|wfh13:44
*** Jaison has quit IRC13:46
hewbroccaathomas: see what your country has done to us?13:48
social  echo '/etc/yum.repos.d//CentOS-Ceph-Jewel.repo is not a valid yum repo configuration file.'13:48
hewbroccagfidente: doesn't know how to spell any more13:48
dprinceEmilienM: +A13:48
EmilienMdprince: thx13:48
socialsince when we depend on CentOS-Ceph-Jewel.repo in RDO? where do I get this from?13:48
hewbroccasocial: that's gfidente fault13:48
jtomasekd0ugal, rbrady: just to clarify: does the ProcessTemplatesAction include the 'environment' from mistral environment parameter_defaults?13:48
* social chews on gfidente 13:49
gfidenteI knew13:49
gfidenteit would turn against me13:49
* athomas wonders if his country can really be blames for variations from English.13:49
athomasblamed, even13:49
jaosoriorbandini: what was the reason again why HAProxy is still managed by pacemaker?13:49
jaosoriorbandini: I don't think it shouldn't; just asking13:49
gfidentesocial https://review.openstack.org/#/c/352910/ , we use it to install ceph/jewel in ci13:50
gfidentepreviously we were using the version in epel13:50
*** karthiks_afk is now known as karthiks13:50
EmilienMI'm wondering if we still use EPEL in tripleo CI13:51
gfidenteEmilienM we don't judging from tripleo.sh13:51
jaosoriorCan someone check this commit? https://review.openstack.org/#/c/348037/13:52
bandinijaosorior: sec, am on a call13:53
socialgfidente: but dib looks fro jewel repo13:53
openstackgerritHarry Rybacki proposed openstack/tripleo-quickstart: use jinja to update ironic vs. ansible  https://review.openstack.org/34616813:53
gfidentejaosorior looks nice but people will have to pass a new env file right?13:55
gfidentein addition to the one they already use13:55
gfidentesocial to make DIB use the repo change is a bit more intrusive https://review.openstack.org/#/c/340504/13:56
gfidenteyou have to export DIB_YUM_REPO_CONF13:56
EmilienMgfidente: I'll let you +A if you're ok13:56
gfidenteI think it's worth merging but we need to keep track of these in the docs somehow13:56
jaosoriorgfidente: That's correct. Either way, if I mange to pull out TLS everywhere... another env file will be needed anyway...13:56
EmilienMgfidente: ok13:57
d0ugalrbrady: Good question, I am not sure. Looking13:57
bandinijaosorior: back. so the reason is that that way we can guarantee that the VIP runs where haproxy is up (it could crash and whatnot)13:58
*** apetrich has quit IRC13:58
rbradyjtomasek: if you're asking if the ProcessTemplateActions includes the user set parameter values when it calls into heatclient for the environment merge...no it doesn't13:58
jaosoriorbandini: I see14:00
bandinidprince: do you think you will have some time this week (or beginning of the next one) to clue me and larsks on the split stack stuff? (see email). I offer beer and pizza (I do make an awesome pizza!) ;)14:00
jtomasekrbrady: why not?14:00
rbradyjtomasek: because14:00
d0ugaljtomasek, rbrady: it sounds like it should.14:00
dprincebandini: I replied to your email already I think14:01
dprincebandini: see that stuff, and then yes we can chat14:01
rbradyd0ugal, jtomasek: the short answer is that this has been an iterative process.  I'm looking at the 3 patches in the series now wrt to this14:01
dprincebandini: my heat undercloud installer uses the "split stack" approach14:02
bandinidprince: faster than light! I will take a look and come back if I have any questions. Thanks14:02
*** Vijayendra has joined #tripleo14:03
karthiksHello EmilienM14:03
Ngccamacho: bandini: was the change to the comment intentionally removed in patch set 3 of https://review.openstack.org/#/c/355904/ ?14:03
dprincebandini: I have lots of ideas here. Like if we used Zaqar for our transport it would be better for split stack I think14:04
dprincebandini: I've recently pushed patches to make that work via websockets https://review.openstack.org/#/c/344365/14:04
bandinidprince: oh wow!14:04
dprincebandini: zaqar is nice because it allows us to pre-know our queue14:04
dprincebandini: which means os-collect-config is easier to deal with for the deployed-server trick14:04
rbradyd0ugal: at first glance, do you think lines 50-53 in https://review.openstack.org/#/c/298682/21/tripleo_common/actions/parameters.py could just move to after line 109 in https://review.openstack.org/#/c/337615/16/tripleo_common/actions/templates.py14:05
*** bootsha has quit IRC14:05
dprincebandini: lots of goodness at the end of these tunnels I think.14:05
bandiniNg: agreed it would be nice to readd that comment14:05
d0ugalrbrady: Yeah, but that was the hacky solution I done :)14:05
rbradyd0ugal: or maybe rather be treated like line 84-90 in https://review.openstack.org/#/c/337615/16/tripleo_common/actions/templates.py14:05
bandinidprince: I will need to get more familiar with it. Sounds indeed promising14:05
*** bootsha has joined #tripleo14:06
rbradyd0ugal: treat it as a temporary file14:06
ccamachohey Ng, you mean from patchet 2 to 3? I just have rebased as It was on merge conflict14:06
ccamachoNg but you areg right14:06
d0ugalrbrady: +114:06
jaosoriorgfidente: With this we can finally stop maintaining the EndpointMap list in the CI repo :D https://review.openstack.org/#/c/356488/14:07
rbradyd0ugal: I guess it would save 8 - 10 lines in other places14:07
gfidentejaosorior I figured it was going there14:08
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates: Make sahara run via systemd  https://review.openstack.org/35590414:08
*** pradk has joined #tripleo14:08
Ngccamacho: \o/14:08
d0ugalrbrady: yup, that's true. We should do it. You happy to update or should I?14:09
jaosoriorgfidente: maintaining that list in two repos hurt14:09
gfidentejaosorior which is nice :)14:09
bandiniccamacho++14:09
rbradyd0ugal: I can update it.14:09
ccamachobandini Ng sorry for cutting off the message in the rebase14:09
bandiniccamacho: ah no worries ;)14:10
d0ugalrbrady: let me know when you have, ready to re-review14:10
rbradyd0ugal: what happens to GetParametersAction though? https://review.openstack.org/#/c/298682/21/tripleo_common/actions/parameters.py14:10
*** bootsha has quit IRC14:11
*** florianf has quit IRC14:12
*** xuao has joined #tripleo14:12
rbradyd0ugal: nm...I think I see what needs to be done there too14:13
d0ugalrbrady: Just remove the parameter_defaults update?14:14
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Pass ServiceNetMap to services  https://review.openstack.org/35441414:14
ansiwenEmilienM: why you think I have to add a test in run_test.sh? I understood that the plugins for the tempest runs are autodetected, and that it's enough to set the tempest.conf parameter?14:14
EmilienMansiwen: yeah go ahead14:15
rbradyd0ugal: yep14:15
EmilienMansiwen: does ec2api has tests tagged "smoke"?14:15
d0ugalrbrady: cool, thanks!14:15
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Add deployment of CAs via hieradata  https://review.openstack.org/35303714:15
ansiwenEmilienM: I'll check14:15
*** florianf has joined #tripleo14:15
openstackgerritMerged openstack/tripleo-quickstart: fix the staging image var LOCATION var  https://review.openstack.org/35563314:17
*** jeckersb is now known as jeckersb_gone14:17
karthiksEmilienM, I am sorry, I couldn't clearly understand your comment on review https://review.openstack.org/#/c/344829/1/puppet/services/nova-scheduler.yaml@1314:18
EmilienMkarthiks: can you try to set the default to '' ?14:18
karthiksWill it not override the puppet's default ?14:19
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Move MySQL settings out of puppet/controller.yaml  https://review.openstack.org/35650514:19
dprinceEmilienM, jaosorior, gfidente: that is what using the DefaultPasswords looks like14:19
*** jeckersb_gone is now known as jeckersb14:20
EmilienMoh wow14:20
EmilienMkarthiks: let's try14:20
EmilienMkarthiks: tbh I'm not sure14:20
karthikssure EmilienM . will check14:20
EmilienMdprince: nice stuff14:21
*** maeca1 has quit IRC14:22
gfidentedprince I think I did similar without yaql for the ceph pools https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/services/ceph-mon.yaml#L63-L7914:25
gfidenteusing map_replace14:25
gfidente(map_merge)14:25
dprincegfidente: cool, I might like YAQL better here I think14:25
dprincegfidente: if it fits that is14:25
jtomasekrbrady, d0ugal: so the result is that mistral parameter_defaults is going to get added as another environment in  ProcessTemplatesAction, right? GetParameters and Deploy action are going to get more lightweight then14:27
*** yamahata has joined #tripleo14:27
rbradyjtomasek: yes14:29
jtomasekrbrady: great, thanks!14:29
gfidentedprince ok yes as we'd need to merge a string with a map key so it won't be any shorter than that anyway14:29
gfidentewith pools it was maps on maps14:29
dprincegfidente: I'm using YAQL to provide a way to select the first non-empty password. pretty simple case... which goes away when Heat gets support for simple logic14:30
*** maeca1 has joined #tripleo14:31
gfidenteright with pools it's slightly different, map_merge is taking last map items, if any (non-empty)14:31
gfidentebut again those are both maps, so it's convenient to use _merge14:32
*** xuao has quit IRC14:32
gfidentenice thing is that all this stuff is getting upgrade safe14:33
jtomasekflorianf: thanks for a review on that validation patch. See my reply. I'll try to look into that problem again though14:36
*** limao has joined #tripleo14:37
*** pgadiya has quit IRC14:38
florianfjtomasek: Sure, creating a bug sounds fine to me. It's not really critical for that particular patch.14:38
*** limao_ has joined #tripleo14:39
*** padkrish has joined #tripleo14:39
*** limao has quit IRC14:41
*** padkrish_ has joined #tripleo14:42
jtomasekflorianf: ok14:43
*** padkrish has quit IRC14:44
florianfjtomasek: +2'd14:45
slaglerbrady: d0ugal : hey, do you guys have any plans to document the API in tripleo-docs?14:45
rbradyslagle: https://review.openstack.org/#/c/341572/14:46
d0ugalslagle, rbrady: we really need some higher level docs, walking users through how to use it14:46
slagleyea14:47
slaglethis looks like a good start though!14:47
d0ugalbut the generated docs should be a useful compliment to that14:47
jtomasekflorianf: thanks, as I look at it, I could try to improve it so the whole processing is done in the runValidation and runValidationMessage actions and make them dispatch the same 'addWorkflowExecution' action14:47
slaglerbrady: d0ugal : just wanted to make sure it was on the radar14:47
jtomasekflorianf: which is probably what you suggested:D14:47
jtomasekflorianf: I'll do it in separate patch tomorrow14:47
d0ugalslagle: somewhere on the radar, but it's fair to say we have neglected it a bit thus far :)14:47
florianfjtomasek: sure :-) sounds good. thanks14:48
*** limao_ has quit IRC14:49
d0ugalslagle, rbrady: I'll have a go at writing some docs once the CLI patch issues are resolved.14:49
slaglerbrady: d0ugal : i've been testing out the deployment patches as well, and got it all running locally yesterday. it's looking good14:49
d0ugalslagle: Would you add docs to tripleo-docs?14:49
*** limao has joined #tripleo14:49
d0ugalslagle: Great14:49
gfidenteslagle do we have ideas about branching tripleo-docs?14:50
florianfdprince: ping! May I pick your brain quickly about an undercloud.conf issue?14:50
slaglerbrady: d0ugal : i did see one issue, where it assumes we can create a new plan every time: Exception creating plan: Error occurred creating plan: Duplicate entry for Environment: ['name']14:50
d0ugalslagle: Yup, working on that now :)14:50
slagleeven after deleting the previous stack14:50
slaglek, cool :)14:50
gfidenteslagle don't think it will ever happen as we use colors for stable releases right?14:50
dtantsurfolks, how does haproxy in overcloud co-exist with actual services?14:50
dtantsuri.e. they seem to listen on the same port...14:50
gfidentedtantsur they bind on different IPs14:51
slagled0ugal: for the docs, I think we could have some high level docs in tripleo-docs.14:51
dtantsurgfidente, could you point me to an example? I think ironic lacks something to do that.14:51
slagled0ugal: just pointing out the API and where to get the generated API docs14:51
openstackgerritPradeep Kilambi proposed openstack/puppet-tripleo: Clean up unused params from aodh profiles  https://review.openstack.org/35652314:51
slagled0ugal: and perhaps some curl'ish type examples14:51
d0ugalslagle: mistral client example? :)14:52
*** derekh has quit IRC14:52
slagled0ugal: sure, that would be fine.14:52
dprinceflorianf: what is up14:53
gfidentedtantsur eg cinder/api bind address is given via https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/controller.yaml#L68114:54
gfidentedtantsur but haproxy bind address is given by https://github.com/openstack/puppet-tripleo/blob/master/manifests/haproxy.pp#L56014:54
florianfdprince: Do you know of an easy way/setting to make all services set up by `openstack undercloud install` to listen on an accessible interface (rather than 192.0.2.1)?14:54
*** dtrainor has joined #tripleo14:54
florianfdprince: like keystone, heat, ironic...14:54
dprinceflorianf: I set local_ip = 172.19.0.3/2414:55
dtantsurgfidente, thanks, I think I know what to fix then :)14:55
dprinceflorianf: because my undercloud is running on that IP and that is where I have it on my network14:55
gfidentedtantsur coming from https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/vip-config.yaml#L4414:56
dprinceflorianf: you can set 'local_ip' to whatever you'd like and I think that is used as the bind IP for all the relevant services14:56
florianfdprince: excellent, thanks! I tried that, but wasn't sure if there are potential side effects that aren't obvious.14:56
dprinceflorianf: there are side effects, but I think most things should be controllable via other config parameters14:56
dprinceflorianf: you might need to set local_ip, network_gateway, network_cidr, and local_interface accordingly14:57
dprinceflorianf: what I mean is so long as your network setting match you should be fine14:59
florianfdprince: what about undercloud_(public|admin)_vip?14:59
jaosoriorlarsks: hey man, can you check this commit out? https://review.openstack.org/#/c/351683/14:59
dprinceflorianf: only used if you want SSL certs I think14:59
larsksjaosorior: looking...14:59
dprinceflorianf: so if you don't configure they cert I think those aren't used15:00
dprincethe cert15:00
florianfdprince: OK, got it.15:00
florianfdprince: thanks a ton!15:00
*** limao has quit IRC15:00
jaosoriorflorianf: right, those are only used if TLS is enabled at the moment.15:00
*** xuao has joined #tripleo15:00
*** limao has joined #tripleo15:00
pradkEmilienM, can you set the milestone for this to n3 https://bugs.launchpad.net/tripleo/+bug/159049515:01
openstackLaunchpad bug 1590495 in tripleo "gnocchi requires bulk enabled in swift" [Undecided,Fix committed] - Assigned to Pradeep Kilambi (pkilambi)15:01
EmilienMpradk: sure15:01
*** _coolsvap_ is now known as coolsvap15:01
florianfjaosorior: which sounds like a good idea, but probably not strictly needed in a poc environment?15:01
EmilienMpradk: let me add you to the list of people allowed to do it15:01
pradkEmilienM, cool, thx15:01
pradkEmilienM, also https://bugs.launchpad.net/tripleo/+bug/1604860 can be n315:02
openstackLaunchpad bug 1604860 in tripleo "Add a base constraint so gnocchi services should start after galera" [Undecided,Fix committed] - Assigned to Pradeep Kilambi (pkilambi)15:02
EmilienMpradk: oh I can't15:02
EmilienMpradk: no permissions :(15:02
openstackgerritMerged openstack/tripleo-heat-templates: gnocchi: use new authtoken parameters  https://review.openstack.org/35558515:02
EmilienMboth bugs are targeted now15:02
jaosoriorflorianf: not needed. Though you can easily test it out with a local CA if you set  generate_service_certificate = True in your undercloud.conf15:02
pradkEmilienM, cool15:02
florianfjaosorior: Oh, that will auto create a self-signed one?15:03
*** openstackgerrit has quit IRC15:03
jaosoriorflorianf: it will use certmonger's local CA (which is self-signed), and that one will sign the cert used for the undercloud15:04
*** openstackgerrit has joined #tripleo15:04
florianfjaosorior: ok. thanks!15:04
*** eggmaster has joined #tripleo15:04
*** limao has quit IRC15:05
jaosoriorlarsks: I was actually recommended by adarazs to set that variable there15:07
larsksadarazs: can you comment? ^^^^15:07
larsksadarazs: re: https://review.openstack.org/#/c/351683/615:07
larsksThat makes no sense to me.15:07
jaosoriorlarsks: with the reason that one might skip the libvirt part, and just run the undercloud deployment. And in that case, that variable will not be set15:07
larsksIt's not okay to set the same variable in two different places, becuaes if they are out of sync, confusion and sadness will result.15:07
adarazslarsks: so, that role uses this variable, but it's not set globally. roles are able to access defaults *if* those roles run beofre.15:07
jaosoriorlarsks: adarazs also commented in that review in a past patchset15:07
larsksadarazs: also, right now, libvirt is a dependency of the undercloud module.15:07
larsksadarazs: so nobody is going to be using one without the other.15:07
openstackgerritPradeep Kilambi proposed openstack/tripleo-heat-templates: Set the default gnocchi driver to swift  https://review.openstack.org/35438515:07
* larsks just really hates seeing the same information being provided in multiple places.15:08
adarazslarsks: yeah, but I can easily see it being used without it for baremetal in the future. what's the problem with setting a default there? if somebody wants to override it with --extra-vars then it will work.15:08
larsksadarazs: I guess I just like having a "set the default in one place only" model.15:08
adarazslarsks: yeah, but then we shouldn't access information outside of them then, we need sensible defaults per roles.15:08
*** dtrainor has quit IRC15:09
larsksFair enough.I will withdraw my -1.15:09
*** dtrainor has joined #tripleo15:09
adarazslarsks: my alternative is to set this default somewhere global like in the config/general_config config files.15:09
adarazsand remove it from both places.15:09
adarazsbut that doesn't seem so good either.15:09
openstackgerritMerged openstack/tripleo-heat-templates: Decouple EndpointMap from SSL certificate params  https://review.openstack.org/34803715:09
larsksEh, I +2d it for now.15:10
larsksjaosorior: you're all set.15:10
jaosoriorgfidente, can you check the keystone endpoints commit? https://review.openstack.org/#/c/352364/ it's been passing CI :D that one and the one using a flag to skip the post-config https://review.openstack.org/#/c/353307/15:10
jaosoriorlarsks: thanks15:10
jaosoriorEmilienM ^^15:11
*** xuao has quit IRC15:11
*** dprince has quit IRC15:11
*** xuao has joined #tripleo15:11
EmilienMjaosorior: looking15:12
dmsimardIs it possible to setup particular partitions for Ironic bare metal images in the context of TripleO ? Someone wants to know, can anyone get back to him ? https://ask.openstack.org/en/question/95318/how-to-setup-partitions-for-ironic-baremetal-images-in-tripleo/15:12
EmilienMjaosorior: wait, I might see a use case where it wouldn't work15:12
jaosoriorEmilienM: sure, what's up?15:12
EmilienMjaosorior: let me 5 min please15:12
openstackgerritDmitry Tantsur proposed openstack/tripleo-heat-templates: Ironic: add missing haproxy and firewall configuration  https://review.openstack.org/35653915:13
openstackgerritMerged openstack/tripleo-quickstart: Persist ironic configuration changes  https://review.openstack.org/35168315:13
pradkEmilienM, jaosorior, any thoughts why apache is failing to start on https://review.openstack.org/#/c/352476/ .. the patch tht patch that depends on this passes fine https://review.openstack.org/#/c/293642/15:14
pradkis it a circular dependency thingy?15:14
EmilienMpradk: a sec plz15:15
openstackgerritDmitry Tantsur proposed openstack/tripleo-heat-templates: Ironic: add missing haproxy and firewall configuration  https://review.openstack.org/35653915:16
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Permit usage of raw disks as backends for Swift storage  https://review.openstack.org/35084115:21
*** maeca1 has quit IRC15:22
*** florianf has quit IRC15:25
*** saneax_AFK is now known as saneax15:26
*** saneax is now known as Guest6646615:27
openstackgerritMerged openstack/tripleo-heat-templates: Remove deprecated node_ips hiera keys  https://review.openstack.org/35506415:28
*** xuao has quit IRC15:29
*** florianf has joined #tripleo15:32
openstackgerritGael Chamoulaud proposed openstack/tripleo-quickstart: Replace the native quickstart inventory with ansible-role-tripleo-inventory  https://review.openstack.org/35596215:35
ansiwenEmilienM: so, andrey-mp claims the ec2api under [service_available] in tempest.conf is not necessary... :-/15:36
*** milan has quit IRC15:36
ansiwenEmilienM: instead I need a [aws] section like here, he says: http://logs.openstack.org/57/356257/1/check/gate-functional-neutron-dsvm-ec2api/9cd6367/logs/tempest_conf.txt.gz15:37
EmilienMansiwen: ok15:37
ansiwenit's so confusing, because I get contradicting information...15:38
EmilienMpradk: yes15:38
EmilienMpradk: you need both patches together :(15:38
*** leanderthal is now known as leanderthal|afk15:39
EmilienMpradk: oh wait15:39
pradkhmm15:39
EmilienMpradk: let me dig into the CI failures15:39
EmilienMin fact no, you should be able to do it without THT15:39
EmilienMthe puppet-tripleo should be a noop patch in term of CI15:39
pradkEmilienM, k the ci failures indicate that httpd is failing to start on keystone api15:40
EmilienMError: /Stage[main]/Apache::Service/Service[httpd]: Failed to call refresh: Could not restart Service[httpd]: Execution of '/usr/bin/systemctl restart httpd' returned 1: Job for httpd.service failed because the control process exited with error code. See \"systemctl status httpd.service\" and \"journalctl -xe\" for details.\u001b[0m\n\u001b[1;31mError: /15:40
pradkyep15:40
EmilienMok what I suggess is to declare the class manually in puppet-tripleo15:40
openstackgerritGael Chamoulaud proposed openstack/tripleo-quickstart: Configure Undercloud VM to start at virthost boot.  https://review.openstack.org/35166515:40
EmilienMand when the THT patch will be merged, we'll just do include ::15:40
EmilienMdo you see?15:40
EmilienMwhat is missing now are the parameters in the wsgi class15:40
pradkEmilienM, ah the ones i'm setting in tht should be explicitly set here15:41
EmilienMjaosorior: I'm not sure you can create endpoints etc outside keystone node15:41
ansiwenEmilienM: and the tests fails because botocore.exceptions is not installed: http://logs.openstack.org/42/356442/1/check/gate-puppet-openstack-integration-3-scenario003-tempest-centos-7/fea293d/console.html#_2016-08-17_13_18_31_74294215:41
EmilienMjaosorior: that's the tl;dr, but I'm not sure, let me check again15:41
EmilienMpradk: yes15:41
EmilienMpradk: please try that first15:41
EmilienMand add a note saying we'll include later15:41
EmilienMwhen THT lands15:41
pradkEmilienM, interesting will try15:42
EmilienMansiwen: really bad time for me to help on ec2 api tbh15:42
ansiwenEmilienM: ok15:42
*** ebarrera has quit IRC15:42
*** numans has quit IRC15:42
jaosoriorEmilienM: I gotta go, but if you can investigate that I would really appreciate, so I could finally drop that effort and focus on getting something usable via mistral or something of the sort.15:43
EmilienMjaosorior: sure, let me investigate15:44
EmilienMjaosorior: sorry for -2 in the meantime but I needed to block it15:45
*** maeca1 has joined #tripleo15:47
jaosoriorEmilienM: No problem; is the right thing to do15:47
EmilienMjaosorior: I think it will be good but it might need one adjustment15:48
EmilienMI'll comment with good details today15:48
*** zoli|wfh is now known as zoli|gone15:49
*** zoli|gone is now known as zoli_gone-proxy15:50
jaosoriorEmilienM: thanks for checking it out man15:51
jaosorioranyway, I gotta go15:51
jaosoriorhave a good day folks!15:51
*** jaosorior has quit IRC15:51
openstackgerritGael Chamoulaud proposed openstack/tripleo-quickstart: Replace the native quickstart inventory with ansible-role-tripleo-inventory  https://review.openstack.org/35596215:55
pradkEmilienM, alternately what if i set depends on tht patch in here https://review.openstack.org/#/c/352476/4 .. that should get the params the wsgi apache class needs.. but not sure how ci handles merging both15:56
EmilienMpradk: there is an alternative15:57
*** padkrish_ has quit IRC15:57
*** ebalduf has joined #tripleo15:57
EmilienMpradk: step 1: patch THT with ceilometer::wsgi::apache with the required parameters15:57
EmilienMpradk: step 2: patch puppet-tripleo by adding include ::ceilometer::wsgi::apache15:58
EmilienMoh wait15:58
pradkwhich is https://review.openstack.org/#/c/293642/ for step 115:58
EmilienMit won't work but I have another proposal :)15:58
EmilienMok, it's overkill but it will work :15:58
*** rcernin has quit IRC15:58
EmilienMstep 1: patch THT with ceilometer::wsgi::apache parameters only15:59
pradkok which is https://review.openstack.org/#/c/29364215:59
*** dtantsur is now known as dtantsur|brb16:00
EmilienMstep 2: patch puppet-tripleo with the include of this class, but in a conditional if hiera('ceilometer_wsgi') { include ::ceilometer::wsgi::apache }16:00
EmilienMboth will pass CI because they are noop16:00
EmilienMstep 3: patch THT and set ceilometer_wsgi to true in the service16:00
EmilienMif it pass CI, ceilometer WSGI works16:00
*** asbishop has quit IRC16:01
EmilienMstep 4: remove the conditional in pupet-tripleo and step 5: remove the param in the service16:01
EmilienMI know it's overkill but our CI is now voting and we have to make everything backward compatible16:01
*** asbishop has joined #tripleo16:01
pradkhmm yea a bit round about but sounds like a good workaround16:01
pradki could just set the params in puppet tripleo ceilo::wsgi::apache class .. something like:16:02
EmilienMoh and use depends on :)16:02
pradk+    class { '::ceilometer::wsgi::apache':16:02
pradk+      bind_host => hiera('ceilometer_api_network'),16:02
pradk+      ssl       => false,16:02
pradk+    }16:02
dtantsur|brbgfidente, corrected ironic haproxy: https://review.openstack.org/#/c/356539/ (tested locally)16:02
pradkbut would it both tht if i set those again16:02
pradkbother*16:03
EmilienMyes but CI will fail16:03
EmilienMbecause apache will start ceilometer and ceilometer will start it's wsgi process too16:03
EmilienMsince ceilometer::api::service_name needs to be set to 'httpd'16:03
*** weshay is now known as weshay_bbiab16:03
pradkah so i need to set ceilometer api class params too16:03
pradkk i16:03
pradkll try the above approach16:03
EmilienMit should work16:04
EmilienMbandini: what does it means now the notion of bootstrap node?16:05
bandiniEmilienM: I need to open a bug about our current notion of bootstrap node because we landed a change where bootstrap_node is per node type (compute,controller,..) so our if hostname==bootstrap_node tests are not too robust16:06
bandinidoes that make sense?16:09
d0ugaljtomasek: Are you using list_plans?16:09
d0ugaljtomasek: (tripleo.list_plans, the Mistral action)16:10
EmilienMbandini: please look https://review.openstack.org/#/c/352364/16:12
gfidenteslagle social was about to add the use of DIB_YUM_REPO_CONF into docs16:12
gfidentethough I noticed in tripleo.sh we do delorean.repo, delorean-current.repo and delorean-deps.repo16:13
gfidentewhile in tripleo-docs we do export DELOREAN_REPO_FILE="delorean.repo"16:13
gfidentewhat is going to happen with the other two files?16:13
slaglegfidente: we document the same in the repo setup16:13
*** ebarrera has joined #tripleo16:14
slaglegfidente: and we add them all to DIB_YUM_REPO_CONF for the image build16:14
gfidentedon't see references to YUM_REPO_CONF in docs16:14
slaglethat's what you're adding :)16:14
slaglewe didnt document the image build part16:14
slaglebut we do document adding those 3 repos for the repo setup16:15
slagleon the undercloud16:15
gfidenteright16:15
socialEmilienM: https://review.openstack.org/#/c/354125 I'm interested in your opinion about creating the ironic::pxe class and using it's options in ironic::inspector16:15
EmilienMsocial: ok I'll look16:15
EmilienMsocial: sounds like dtantsur|brb already commented16:16
socialEmilienM: yes he did, I'm asking for your opinion as I don't recall openstack puppet doing what he proposes eg creating config only class16:17
EmilienMno, we do it16:18
EmilienMsocial: https://github.com/openstack/puppet-nova/blob/master/manifests/ironic/common.pp16:18
EmilienMsocial: https://github.com/openstack/puppet-nova/blob/master/manifests/vncproxy/common.pp16:19
jpichd0ugal: Are you talking about in the UI?16:19
*** padkrish has joined #tripleo16:19
socialEmilienM: ironic::pxe would only contain http_boot_dir variable and ironic::inspector http_boot_dir would default to ::ironic::pxe::http_boot_dir16:20
EmilienMsocial: wfm16:20
d0ugaljpich: are16:21
d0ugaljpich: aye16:21
socialEmilienM: ack :) than don't review, I'll make another version16:21
d0ugaljpich: but I figured out what I was doing wrong16:21
jpichd0ugal: Scottish Pirate day?16:21
EmilienMok I'll review it after16:21
openstackgerritPradeep Kilambi proposed openstack/puppet-tripleo: Move ceilometer api to run under apache wsgi  https://review.openstack.org/35247616:21
openstackgerritgreghaynes proposed openstack/diskimage-builder: Add element for setting sysctl values  https://review.openstack.org/35658216:21
d0ugaljpich: I was creating the swift container incorrectly16:21
jpichd0ugal: Ok! Well for the record florianf wrote that one :) https://review.openstack.org/#/c/347919/16:21
d0ugaljpich: aha, thanks!16:22
jpichd0ugal: Also I wrote a client patch for it \o/ We're tracking the work around plan management at -> https://blueprints.launchpad.net/tripleo-ui/+spec/tripleo-ui-mistral-refactoring16:22
d0ugaljpich: Neat, I shall take a look16:26
openstackgerritDougal Matthews proposed openstack/python-tripleoclient: Migrate the deploy command to use the Mistral workflows  https://review.openstack.org/34226116:27
*** dprince has joined #tripleo16:27
pabelangerAfternoon!16:31
pabelangerI want to update tripleo-test-cloud-rh1 to use the new openstackzuul credentials, wanted to make sure everybody is away what is going to happen for that change.16:31
pabelangerbasically, running jobs _should_ continue to run but will require manually cleanup in openstackjenkins project16:32
pabelangerand new jobs will be launched in the openstackzuul project16:32
*** bana_k has joined #tripleo16:33
*** abehl has quit IRC16:34
*** milan has joined #tripleo16:34
gfidenteslagle for image building the RHN channels from REG_REPOS are available doing the packages install?16:35
*** jpich has quit IRC16:35
gfidente*during16:36
jtomasekd0ugal: yes16:37
*** rwsu has quit IRC16:38
openstackgerritPradeep Kilambi proposed openstack/tripleo-heat-templates: DO NOT MERGE: Testing Ceilometer wsgi  https://review.openstack.org/35659916:38
*** florianf has quit IRC16:39
pradkEmilienM, ^^ lets see16:39
pabelangerslagle: bnemec-pto: EmilienM: See ^ about tripleo-test-cloud-rh1 credential updates16:41
pabelangerI haven't made the change yet16:41
pabelangerbut would like too16:41
openstackgerritGiulio Fidente proposed openstack/tripleo-docs: Document Ceph repositories to be enabled  https://review.openstack.org/35291016:42
openstackgerritGiulio Fidente proposed openstack/tripleo-docs: Document Ceph repositories to be enabled  https://review.openstack.org/35291016:43
slaglepabelanger: it sounds fine. but just bear in mind i'm about to leave for pto for thurs/fri, as is emilien. and ben is also out16:45
slaglepabelanger: maybe we could hold off until next week? just in case..16:45
pabelangerslagle: I we should be fine, I don't expect problems. If so, I can roll back to the original project16:46
pabelangers/we/think/16:46
slagleok16:47
slaglegfidente: yes they should be. but honestly i've not done image building while registering to rhn/portal in a *long* time16:48
gfidenteslagle right my point is that it should be installing the packages from the rhn channel16:49
slagleEmilienM: https://review.openstack.org/#/c/355537/ entered the gate!16:49
*** hewbrocca is now known as hewbrocca-afk16:49
*** dtantsur|brb is now known as dtantsur16:49
gfidenteso I've updated docs accordingly for ceph as well16:49
*** padkrish has quit IRC16:49
*** maeca1 has quit IRC16:51
pabelangerslagle: EmilienM: I've added the new credentials, the change should land in the next 25mins16:51
openstackgerritMerged openstack/tripleo-ui: List Validations and Validation results  https://review.openstack.org/33477016:51
openstackgerritMerged openstack/tripleo-ui: Triggering validations and messaging  https://review.openstack.org/33656016:51
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Move MySQL settings out of puppet/controller.yaml  https://review.openstack.org/35650516:52
*** pabelanger has quit IRC16:54
*** pabelanger has joined #tripleo16:54
*** bootsha has joined #tripleo16:56
*** yamahata has quit IRC16:58
*** weshay_bbiab is now known as weshay16:58
*** cdearborn has joined #tripleo16:59
*** bkopilov has quit IRC16:59
*** padkrish has joined #tripleo17:00
*** lucasagomes is now known as lucas-dinner17:00
*** tesseract- has quit IRC17:00
openstackgerritwes hayutin proposed openstack/tripleo-quickstart: add ssl option as default to True for the undercloud  https://review.openstack.org/35643117:02
openstackgerritDmitry Tantsur proposed openstack/tripleo-docs: [WIP] Documentation for installing and using Ironic in overcloud  https://review.openstack.org/35401617:03
*** ebarrera has quit IRC17:04
*** egafford has quit IRC17:04
*** ebarrera has joined #tripleo17:05
*** fragatina has joined #tripleo17:05
*** liverpooler has quit IRC17:07
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates: Enable gnocchi systemd services  https://review.openstack.org/35611117:07
*** padkrish has quit IRC17:08
openstackgerritGiulio Fidente proposed openstack/tripleo-docs: Document Ceph repositories to be enabled  https://review.openstack.org/35291017:10
*** bana_k has quit IRC17:14
*** bootsha has quit IRC17:18
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates: Remove OS::Tripleo::Services::ManilaShare: from puppet-pacemaker.yaml  https://review.openstack.org/35662217:19
*** egafford has joined #tripleo17:20
*** bootsha has joined #tripleo17:21
*** bootsha has quit IRC17:22
*** maeca1 has joined #tripleo17:23
*** bootsha has joined #tripleo17:23
*** oshvartz has joined #tripleo17:24
*** rcernin has joined #tripleo17:26
*** bootsha has quit IRC17:26
*** electrofelix has quit IRC17:27
*** bana_k has joined #tripleo17:28
dprincegfidente: hi, in my move configs out of controller.yaml I'm now doing the IP stuff too. Just FYI17:28
dprincegfidente: For each service I'm taking care of everything left... otherwise it is just too many patches to split this up I think17:28
gfidentedprince right I wanted to help there as well17:30
gfidenteadd me on reviews , I can +2 at least17:30
dprincegfidente: rebasing now. I've got MySQL and Rabbit done and I'm working through the rest of controller.yaml now too17:30
*** fragatina has quit IRC17:30
dprincegfidente: we are running out of time so I'd like all this settled17:30
dprincegfidente: the composable service work is totally unusable (incomplete) without it17:31
*** fragatina has joined #tripleo17:31
dprincegfidente: turns out those passwords and bind ports or pretty important ;)17:31
dprincegfidente: I'm going to leave inline comments on the bind ports to indicate they are str-replaced to avoid confusion17:32
*** bootsha has joined #tripleo17:32
gfidentedprince ack17:32
dprincegfidente: the mechanism works... but could be a big step for someone ramping up to understand this on first read17:33
gfidente_uri _subnet you mean?17:33
dprincegfidente: https://review.openstack.org/#/c/318344/10/puppet/services/cinder-api.yaml17:33
dprincegfidente: add a comment there to say explicitly that magic happens17:33
gfidenteright so we need to say17:33
gfidentewhen in need of the bracketed version, or of the subnet17:34
dprincegfidente: "magic happens here" is fine :)17:34
gfidentethe lookup gets a bit weird cause you need to append the _uri or _subnet suffix to network name too17:34
dprincegfidente: yeah, all magic17:34
gfidentelooks like examples are best bet17:34
gfidenteI'll try to set ceph subnet as it needs _subnet and comment there too17:34
*** padkrish has joined #tripleo17:37
*** dtantsur is now known as dtantsur|afk17:37
openstackgerritwes hayutin proposed openstack/tripleo-quickstart: replace json network-environment w/ yaml  https://review.openstack.org/35614117:37
*** bootsha has quit IRC17:39
*** yamahata has joined #tripleo17:40
*** maeca1 has quit IRC17:41
*** bootsha has joined #tripleo17:43
pabelangerslagle: EmilienM: new nodes coming on line for openstackzuul project17:43
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Permit usage of raw disks as backends for Swift storage  https://review.openstack.org/35084117:44
*** egafford1 has joined #tripleo17:45
gfidenteEmilienM can you check if this is fine merging https://review.openstack.org/#/c/350840/17:45
*** ohamada has quit IRC17:46
*** ebarrera has quit IRC17:46
openstackgerritwes hayutin proposed openstack/tripleo-quickstart: replace qemu-kvm w/ qemu-kvm-ev  https://review.openstack.org/35663917:47
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Move MySQL settings out of puppet/controller.yaml  https://review.openstack.org/35650517:47
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add DefaultPasswords to composable services  https://review.openstack.org/35646117:47
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Move RabbitMQ settings out of controller.yaml  https://review.openstack.org/35664017:47
dprincegfidente: ^^^ those include bind_ip magic17:48
gfidentedprince cool17:48
*** egafford has quit IRC17:48
gfidenteI just realized something like that for the swift ring17:48
gfidenteas it's coming from the controller template too17:48
dprincegfidente: yep, I will get all of these. controller will be clean when I finish17:49
*** bkopilov has joined #tripleo17:49
gfidenteright sorry, swift ring is just the ip we need to replace so it's same stuff17:49
EmilienMback from lunch17:49
EmilienMgfidente: ok looking17:49
EmilienMslagle: amazing!17:50
gfidenteEmilienM thanks!17:50
gfidenteEmilienM would like to merge the swift disks management which depends on it https://review.openstack.org/#/c/350841/17:50
gfidente*me17:50
openstackgerritwes hayutin proposed openstack/tripleo-quickstart: [WIP] replace qemu-kvm w/ qemu-kvm-ev  https://review.openstack.org/35663917:50
EmilienMslagle: my puppet patch should also land today17:51
*** tosky has quit IRC17:51
EmilienMgfidente: +A17:52
EmilienMpabelanger: ack, excellent & thx for the update17:53
EmilienMpabelanger: do you have an example of job running atm?17:54
EmilienMpabelanger: is everything running fine?17:55
*** aufi has quit IRC17:55
pabelangerEmilienM: still launching nodes, I have to clean up resources from the old project17:55
EmilienMcan we monitor that in grafana?17:55
pabelangeryes17:55
EmilienMmaybe in http://grafana.openstack.org/dashboard/db/nodepool17:55
pabelangerhttp://grafana.openstack.org/dashboard/db/nodepool-tripleo-test-cloud17:55
EmilienMin http://grafana.openstack.org/dashboard/db/nodepool-tripleo-test-cloud17:56
EmilienMyeah ! thx17:56
EmilienMpabelanger: but how to see which jobs are using this cloud?17:56
*** jcoufal has quit IRC17:56
pabelangeryou'll have to look on status.o.o/zuul for new jobs that have just been launched17:57
pabelangeractually17:57
EmilienMyeah but it's manual17:57
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Move RabbitMQ settings out of controller.yaml  https://review.openstack.org/35664017:57
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Move MySQL settings out of puppet/controller.yaml  https://review.openstack.org/35650517:57
pabelangerEmilienM: nc 66.187.229.132 1988517:58
openstackgerritLars Kellogg-Stedman proposed openstack/tripleo-heat-templates: [WIP] Add fluentd client service  https://review.openstack.org/35350617:58
pradkEmilienM, argh .. this dint help https://review.openstack.org/#/c/352476/4..5/manifests/profile/base/ceilometer/api.pp17:59
*** jcoufal has joined #tripleo17:59
pradkah nm18:00
openstackgerritSwapnil Kulkarni (coolsvap) proposed openstack/tripleo-quickstart: Add undercloud reboot  https://review.openstack.org/35048918:00
pradki'll pass in false as default18:00
EmilienMpabelanger: it's gate-tripleo-ci-centos-7-ovb-ha running18:01
EmilienMI thought you were running another job outside tripleo18:01
pabelangerEmilienM: not yet, first we need to clean up the projects in openstack18:01
openstackgerritPradeep Kilambi proposed openstack/puppet-tripleo: Move ceilometer api to run under apache wsgi  https://review.openstack.org/35247618:02
EmilienMpabelanger: ok18:02
pabelangerEmilienM: once this works, we can then create one for other projects18:02
EmilienMgreat18:02
EmilienMslagle: last job is gate-grenade-dsvm-ironic and it's running18:03
EmilienMslagle: once it merges, we need to wait for delorean to build a repo18:03
openstackgerritwes hayutin proposed openstack/tripleo-quickstart: add ssl option as default to True for the undercloud  https://review.openstack.org/35643118:04
EmilienMslagle: and then recheck https://review.openstack.org/#/c/346949/ ?18:04
EmilienMslagle: do we still want the puppet fixes?18:05
EmilienMthey are ready to land too18:05
*** bootsha has quit IRC18:07
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Set cinder::api::bind_host within service template  https://review.openstack.org/31834418:10
gfidentedprince ^^ good?18:10
dprincegfidente: very, I will update mine to match18:14
gfidenteok so I am pushing the ceph/subnet cleanup too last one for me18:15
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Move out of roles any Ceph setting  https://review.openstack.org/35665718:19
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Move out of roles any Ceph setting  https://review.openstack.org/35665718:19
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Move RabbitMQ settings out of controller.yaml  https://review.openstack.org/35664018:21
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Move MySQL settings out of puppet/controller.yaml  https://review.openstack.org/35650518:22
*** gfidente has quit IRC18:24
*** padkrish has quit IRC18:34
*** padkrish has joined #tripleo18:35
*** padkrish has quit IRC18:37
*** liverpooler has joined #tripleo18:38
openstackgerritEmilien Macchi proposed openstack/python-tripleoclient: WIP wire in jinja templating for custom roles  https://review.openstack.org/33662518:42
openstackgerritEmilien Macchi proposed openstack/python-tripleoclient: WIP wire in jinja templating for custom roles  https://review.openstack.org/33662518:42
EmilienMslagle: ironic patch is about to land, all jobs passed18:43
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Convert overcloud.yaml to support jinja2 templating  https://review.openstack.org/31567918:43
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Move AllNodesDeployments into jinja template loop  https://review.openstack.org/33726718:44
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Move AllNodesValidationDeployments into jinja template loop  https://review.openstack.org/33758718:44
openstackgerritLars Kellogg-Stedman proposed openstack/puppet-tripleo: [WIP] updates to fluentd support  https://review.openstack.org/35350718:48
EmilienMcan anyone approve this patch please? https://review.openstack.org/#/c/354606/18:49
EmilienMslagle: oh no... POST_FAILURE18:50
*** padkrish has joined #tripleo18:50
EmilienMslagle: merging puppet patch now18:51
*** padkrish has quit IRC18:51
*** padkrish has joined #tripleo18:56
*** padkrish has quit IRC18:59
*** paramite has joined #tripleo18:59
pabelangerslagle: EmilienM: can you confirm the quota on openstackzuul?19:00
pabelangerslagle: EmilienM: that is was the same as openstack-nodepool19:00
EmilienM32 you mean?19:01
pabelangerit should be the same RAM, HDD, and instances quota19:01
EmilienMahh19:01
pabelangerI'm not sure it is19:01
EmilienMlet me check19:01
*** athomas has quit IRC19:01
EmilienMpabelanger: we need to wait for a job to finish, right?19:03
EmilienMto see logs19:03
EmilienMI'm not sure we see that in the console19:03
EmilienMBuilding remotely on tripleo-centos-7-tripleo-test-cloud-rh1-356295319:04
EmilienMthis one is good?19:04
pabelangerForbidden: Quota exceeded for ram: Requested 6144, but already used 49152 of 51200 ram (HTTP 403) (Request-ID: req-c5e4acfc-1a27-4f8a-b43e-c5dd81b1f0b2)19:07
pabelangeris what I am seeing19:07
EmilienMcontroller has 5.2G of RAM and 41G of disk19:07
EmilienMIm not sure I'm looking at the right place19:08
EmilienMhttp://logs.openstack.org/04/355904/4/check-tripleo/gate-tripleo-ci-centos-7-ovb-nonha/e1569b8/logs/overcloud-controller-0/var/log/host_info.txt.gz19:08
*** beagles is now known as beagles_biab19:08
pabelangerthis is in the openstackzuul project19:08
pabelangerin tripleo-test-cloud-rh119:08
*** padkrish has joined #tripleo19:08
pabelangerthe quota needs to be up'd to match the openstack-nodepool user19:08
*** paramite has quit IRC19:08
openstackgerritwes hayutin proposed openstack/tripleo-quickstart: pin the undercloud-post and overcloud roles  https://review.openstack.org/35666919:09
*** padkrish has quit IRC19:10
*** jeckersb is now known as jeckersb_gone19:11
*** padkrish has joined #tripleo19:11
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Create per-role node_names in hieradata for all nodes  https://review.openstack.org/34897319:12
*** jbadiapa has quit IRC19:15
*** padkrish has quit IRC19:19
pabelangerEmilienM: are you an admin for tripleo-test-cloud-rh1?19:19
pabelangerif not, we'll need one19:19
EmilienMpabelanger: I'm not19:19
EmilienMpabelanger: dprince, sshnaidm and slagle are19:19
*** maeca1 has joined #tripleo19:19
*** hk_ has joined #tripleo19:22
hk_hi19:22
hk_hi all, is it best way to install openstack? https://keithtenzer.com/2016/05/30/red-hat-openstack-platform-8-lab-configuration-using-openstack-director/19:24
*** ayoung has quit IRC19:27
dprincehk_: TripleO is the best hands down man. do it19:29
hk_i didn't understood "hands down" still I assume that You wanna say it's best19:31
hk_aren't I19:31
hk_?19:31
dprincethe best, yes19:31
*** padkrish has joined #tripleo19:32
*** lblanchard has quit IRC19:33
*** abehl has joined #tripleo19:34
*** jeckersb_gone is now known as jeckersb19:34
hk_thank you dprince19:35
*** liverpooler has quit IRC19:42
*** jprovazn has quit IRC19:44
*** hk_ has quit IRC19:46
-openstackstatus- NOTICE: The volume for logs.openstack.org filled up rather suddenly, causing a number of jobs to fail with a POST_FAILURE result and no logs; we're manually expiring some logs now to buy breathing room, but any changes which hit that in the past few minutes will need to be rechecked and/or approved again19:46
*** jcoufal has quit IRC19:48
EmilienMdprince: can you review https://review.openstack.org/#/c/354606/ please?19:54
dprinceEmilienM: interesting, I assumed it was puppet causing the slowness19:55
*** ioggstream has joined #tripleo19:56
*** toure is now known as toure|biab19:56
dprinceEmilienM: would be nice to quantify that, if it actually does help CI runtimes19:58
EmilienMdprince: ok19:59
*** rwsu has joined #tripleo20:02
*** maeca1 has quit IRC20:04
*** coolsvap has quit IRC20:11
cwolferhcurious if either of these would be useful in tht: https://review.openstack.org/#/c/349136/ (hopefully avoid nested stacks that only exist for data transformations) and https://review.openstack.org/#/c/355143/ (maybe help with debugging by catching str_replace issues earlier)20:13
*** rhallisey_ has joined #tripleo20:15
dprincecwolferh: yes, it will be. We've been waiting on such features for some time20:16
*** ayoung has joined #tripleo20:18
*** jayg is now known as jayg|g0n320:18
*** dprince has quit IRC20:19
cwolferhdprince, ok, thx. some in heat may be skeptical of either, so good to know20:19
*** rhallisey_ has quit IRC20:20
openstackgerritEmilien Macchi proposed openstack/instack-undercloud: Work around deletion of _member_ role assignments on upgrade  https://review.openstack.org/30735220:20
openstackgerritLars Kellogg-Stedman proposed openstack/tripleo-specs: [wip] Spec: Centralized logging support  https://review.openstack.org/35669120:23
*** jbadiapa has joined #tripleo20:31
*** Jokke_ has quit IRC20:35
*** jbadiapa has quit IRC20:36
*** Jokke_ has joined #tripleo20:37
ccamacho Guys mind to review these two submissions?  (sahara runs via systemd)https://review.openstack.org/#/c/355904/ and (remove duplicated resource registry entry) https://review.openstack.org/#/c/356622/20:37
openstackgerritHarry Rybacki proposed openstack/tripleo-quickstart: Update configuration docs  https://review.openstack.org/35670020:38
*** rcernin has quit IRC20:40
*** bootsha has joined #tripleo20:41
*** ioggstream has quit IRC20:45
*** toure|biab is now known as toure20:49
*** egafford1 has quit IRC20:53
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci: Deploy minimal services in multinode job  https://review.openstack.org/35509720:56
*** rhallisey has quit IRC21:04
*** dcain has joined #tripleo21:08
*** bootsha has quit IRC21:10
*** jeckersb is now known as jeckersb_gone21:16
*** dcain has quit IRC21:17
*** asbishop has quit IRC21:20
*** dcain has joined #tripleo21:21
*** dcain has quit IRC21:23
openstackgerritwes hayutin proposed openstack/tripleo-quickstart: update devmode doc, remove method that is not working  https://review.openstack.org/35674721:26
openstackgerritEmilien Macchi proposed openstack/instack-undercloud: Use http for Ironic deployments  https://review.openstack.org/35548421:27
*** gfidente has joined #tripleo21:27
*** Guest66466 is now known as saneax_AFK21:29
*** pradk has quit IRC21:29
openstackgerritwes hayutin proposed openstack/tripleo-quickstart: update devmode doc, remove method that is not working  https://review.openstack.org/35674721:33
*** padkrish has quit IRC21:35
openstackgerritEmilien Macchi proposed openstack/instack-undercloud: (mitaka only) Fix sysctl_settings configuration  https://review.openstack.org/35675421:40
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: (mitaka only) loadbalancer: manage ipv6 sysctl  https://review.openstack.org/35675521:42
*** penick has joined #tripleo21:43
*** padkrish has joined #tripleo21:44
*** abehl has quit IRC21:51
*** cdearborn has quit IRC22:14
*** ayoung has quit IRC22:14
*** dhill_ has quit IRC22:16
*** dhill_ has joined #tripleo22:17
*** lucas-dinner has quit IRC22:23
*** jpeeler has quit IRC22:25
*** Slower has quit IRC22:25
*** zaneb has quit IRC22:25
*** ansiwen has quit IRC22:25
*** jeckersb_gone has quit IRC22:25
*** ansiwen has joined #tripleo22:27
*** jeckersb_gone has joined #tripleo22:30
*** lucasagomes has joined #tripleo22:30
*** Slower has joined #tripleo22:32
*** saneax_AFK is now known as saneax22:33
*** saneax is now known as Guest6566322:34
*** zaneb has joined #tripleo22:38
*** jpeeler has joined #tripleo22:39
*** weshay has quit IRC22:41
*** padkrish has quit IRC22:42
*** beagles_biab is now known as beagles22:42
*** padkrish has joined #tripleo22:44
beagleshas anyone seen overcloud image build with something like https://paste.fedoraproject.org/409920/47147388/22:45
beaglesI'm seeing it on mitaka but I think I've seen it from time to time on master22:45
*** limao has joined #tripleo22:46
beagleserg.. meant to write "overcloud image build fail"...22:46
*** limao has quit IRC22:57
*** limao has joined #tripleo22:57
openstackgerritMerged openstack-infra/tripleo-ci: Try increasing KeystoneWorkers to improve performance  https://review.openstack.org/35460623:00
*** penick has quit IRC23:02
*** limao has quit IRC23:03
*** limao has joined #tripleo23:04
*** sshnaidm is now known as sshnaidm|afk23:08
*** lblanchard has joined #tripleo23:13
*** limao has quit IRC23:15
*** limao has joined #tripleo23:15
*** ayoung has joined #tripleo23:18
*** padkrish has quit IRC23:25
*** limao has quit IRC23:26
*** limao has joined #tripleo23:27
*** padkrish has joined #tripleo23:29
*** fultonj has quit IRC23:37
*** limao_ has joined #tripleo23:38
*** limao has quit IRC23:43
pabelangerbeagles: do have DLRN repos enabled?23:45
pabelangerbeagles: I had the same error without them23:45
beaglespabelanger, plot thickens.. just ran "tripleo.sh --overcloud-images" instead of openstack image build --all and it worked23:49
beaglespabelanger, now I have to double check that it built the *right* images :)23:49
*** limao_ has quit IRC23:49
pabelangerI've only done overcloud using tripleoclient23:50
pabelangerbeagles: http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/tripleo.yaml#n64 is an experimental job I am working on23:50
pabelangerbeagles: I've only tested master, but should be easy to check experimental on other branches23:51
*** rwsu has quit IRC23:51
beaglespabelanger, mmm.. my environment was slightly different than yours23:51
beagles.. I might've even had a stupid typo in my env and just can't see it23:53
beaglespabelanger, actually it *has* to be something like that. I presume we are successfully running mitaka CI so .. :)23:54

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