Thursday, 2016-06-30

*** gfidente|afk has quit IRC00:03
openstackgerritDan Sneddon proposed openstack/tripleo-heat-templates: Enable IPv4/IPv6 dual-stack Public API endpoints  https://review.openstack.org/28927900:13
*** limao has joined #tripleo00:24
*** dmacpher has joined #tripleo00:39
*** yamahata has quit IRC00:41
*** bfournie has joined #tripleo00:43
*** fpan has joined #tripleo00:48
*** rwsu has quit IRC00:49
*** nico_auv has quit IRC00:49
*** julim has quit IRC00:52
*** numans has joined #tripleo00:54
*** numans has quit IRC00:54
*** numans has joined #tripleo00:54
*** links has joined #tripleo00:57
*** julim has joined #tripleo00:58
*** Goneri has joined #tripleo01:13
*** greghaynes has quit IRC01:16
openstackgerritIan Wienand proposed openstack/diskimage-builder: package-installs : add list to arch and "not-arch" list  https://review.openstack.org/33574801:21
openstackgerritIan Wienand proposed openstack/diskimage-builder: package-installs : add list to arch and "not-arch" list  https://review.openstack.org/33574801:25
*** mburned is now known as mburned_out01:27
*** Goneri has quit IRC01:28
openstackgerritIan Wienand proposed openstack/diskimage-builder: package-installs : add list to arch and "not-arch" list  https://review.openstack.org/33574801:30
*** greghaynes has joined #tripleo01:30
*** xinwu has quit IRC01:34
openstackgerritJames Slagle proposed openstack-infra/tripleo-ci: Add mulitnode CI job support to tripleo-ci  https://review.openstack.org/32477701:39
*** julim has quit IRC01:41
*** dmacpher_ has joined #tripleo01:41
*** julim has joined #tripleo01:41
*** dmacpher_ has quit IRC01:41
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-pacemaker: Remove 12.04 from metadata  https://review.openstack.org/33576201:44
*** shivrao has quit IRC01:47
*** yamahata has joined #tripleo01:54
*** rajinir has quit IRC01:54
*** shivrao has joined #tripleo02:01
openstackgerritKeith Schincke proposed openstack/tripleo-heat-templates: Add support for deploying RGW on controllers. (Composable)  https://review.openstack.org/28902702:04
openstackgerritKeith Schincke proposed openstack/tripleo-heat-templates: Switch Ceph Monitor/OSD/Client/External to composable roles  https://review.openstack.org/30469202:04
openstackgerritKeith Schincke proposed openstack/tripleo-heat-templates: Configure BlockStorage services via resource chains  https://review.openstack.org/32285802:04
*** shivrao has quit IRC02:20
*** jraju has joined #tripleo02:33
*** links has quit IRC02:36
openstackgerritliyuanzhen proposed openstack/diskimage-builder: Fix the bug that "mktemp: failed to create directory"  https://review.openstack.org/33577502:36
*** julim has quit IRC02:37
*** coolsvap has joined #tripleo02:56
*** jraju has quit IRC03:03
*** shivrao has joined #tripleo03:31
*** shivrao has quit IRC03:40
*** xinwu has joined #tripleo03:49
cmyster.904:01
*** fpan has quit IRC04:04
*** xinwu has quit IRC04:05
*** xinwu has joined #tripleo04:09
*** tzumainn has quit IRC04:14
*** ayoung has quit IRC04:29
*** xinwu has quit IRC04:29
openstackgerritliyuanzhen proposed openstack/diskimage-builder: Fix the bug that "mktemp: failed to create directory"  https://review.openstack.org/33577504:35
*** shivrao has joined #tripleo04:39
*** numans has quit IRC04:41
*** jprovazn has joined #tripleo04:43
*** xinwu has joined #tripleo04:56
openstackgerritMerged openstack/diskimage-builder: dmidecode does not exist for ppc64/ppc64el  https://review.openstack.org/33393205:00
*** noslzzp has joined #tripleo05:08
*** masco has joined #tripleo05:11
*** noslzzp has quit IRC05:17
*** noslzzp has joined #tripleo05:17
*** marios has joined #tripleo05:25
*** marios has quit IRC05:27
*** marios has joined #tripleo05:27
*** jprovazn has quit IRC05:29
*** marios has quit IRC05:31
*** marios has joined #tripleo05:31
*** marios has quit IRC05:32
*** marios has joined #tripleo05:32
*** liverpooler has joined #tripleo05:33
*** fragatina has joined #tripleo05:46
*** bswartz has quit IRC05:47
*** dsariel has joined #tripleo05:51
*** dtantsur|afk is now known as dtantsur05:55
*** fragatina has quit IRC05:57
*** fragatina has joined #tripleo05:57
saneaxGM Folks06:03
*** pradk has quit IRC06:03
*** Larion has quit IRC06:06
*** tremble has joined #tripleo06:07
*** tremble has joined #tripleo06:07
*** rcernin has joined #tripleo06:09
openstackgerritliyuanzhen proposed openstack/diskimage-builder: Fix the bug that "mktemp: failed to create directory"  https://review.openstack.org/33577506:09
*** bootsha has joined #tripleo06:13
*** rook- has quit IRC06:17
*** andreas-f has joined #tripleo06:23
*** liverpooler has quit IRC06:24
*** liverpooler has joined #tripleo06:24
*** bootsha has quit IRC06:24
*** bootsha has joined #tripleo06:27
*** dsneddon has quit IRC06:31
*** florianf has joined #tripleo06:35
*** pcaruana has joined #tripleo06:37
*** florianf has joined #tripleo06:40
*** dtantsur is now known as dtantsur|brb06:43
*** ramishra has quit IRC06:46
*** xinwu has quit IRC06:47
*** ramishra has joined #tripleo06:48
*** shardy has joined #tripleo06:51
*** bootsha has quit IRC06:51
*** jprovazn has joined #tripleo06:54
*** chem|off is now known as chem06:55
*** amoralej|off is now known as amoralej06:55
*** xinwu has joined #tripleo06:56
*** xinwu has quit IRC07:00
*** bootsha has joined #tripleo07:04
*** zoli_gone-proxy is now known as zoliXXL07:10
*** mcornea has joined #tripleo07:10
*** cschwede has joined #tripleo07:13
*** ifarkas has joined #tripleo07:15
*** tesseract- has joined #tripleo07:15
*** numans has joined #tripleo07:18
*** abehl has joined #tripleo07:33
*** panda has quit IRC07:34
*** panda has joined #tripleo07:35
*** ebarrera has joined #tripleo07:35
*** fragatina has quit IRC07:36
*** jpich has joined #tripleo07:39
*** itamarl has joined #tripleo07:41
*** jpena|off is now known as jpena07:43
*** liverpooler has quit IRC07:43
*** liverpooler has joined #tripleo07:43
ccamacho|awayMorning guys!07:43
*** ccamacho|away is now known as ccamacho07:43
*** penick_ has joined #tripleo07:44
*** penick has quit IRC07:44
*** penick_ is now known as penick07:44
*** myoung is now known as myoung|afk07:48
*** fragatina has joined #tripleo07:50
shardy'morning ccamacho!07:52
ccamachohey!07:52
ccamacho:)07:52
*** dmacpher has quit IRC07:53
*** aufi has joined #tripleo07:55
*** fzdarsky has joined #tripleo07:56
*** shivrao has quit IRC07:57
*** coolsvap is now known as coolsvap_08:02
*** gfidente has joined #tripleo08:04
*** derekh has joined #tripleo08:04
gfidenteccamacho, that blockstorage patch08:07
gfidentepassed all jobs in between the last two attempts08:07
gfidente(rechecks)08:07
gfidenteI think we should just merge it08:07
*** athomas has joined #tripleo08:09
gfidenteanyone wants to +2/A https://review.openstack.org/#/c/322858 ?08:09
shardyYes yes, that would be good - marios perhaps you can take a look?08:12
*** snecklifter has joined #tripleo08:13
openstackgerritJulie Pichon proposed openstack/python-tripleoclient: Add 'openstack overcloud node provide' command  https://review.openstack.org/33441108:13
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Switch Ceph Monitor/OSD/Client/External to composable roles  https://review.openstack.org/30469208:14
marioso/ shardy gfidente sure will do08:14
*** numans has quit IRC08:15
shardythanks!08:15
gfidentemarios++ <308:16
gfidentethis passed all jobs as well on the first attempt  https://review.openstack.org/#/c/335419/08:17
shardygfidente: is there a bug associated with that?08:21
shardythe commit message is pretty light on detail08:21
gfidenteyeah08:21
shardypatch lgtm though08:21
gfidentedidn't we have a README explaining the steps?08:21
gfidentemaybe I can just point to that?08:21
shardyyeah there is a readme in puppet/services IIRC08:22
gfidenteack08:22
gfidenteupdating08:22
shardybut if there's a bug we should update the commit message08:22
shardywe can just land it after without waiting for CI08:22
*** kbyrne has joined #tripleo08:25
*** ohamada has joined #tripleo08:26
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Add support for OVB based CI  https://review.openstack.org/33478708:28
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Refactor common code for tripleo-ci  https://review.openstack.org/33562208:28
derekhOVB job on rh2 working and ready for review ^^08:28
*** lucas-afk is now known as lucasagomes08:30
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Nova API and Neutron server should run database creation in step 3  https://review.openstack.org/33541908:34
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Add mulitnode CI job support to tripleo-ci  https://review.openstack.org/32477708:35
*** pradk has joined #tripleo08:37
gfidenteshardy, updated commit08:41
shardygfidente: thanks!08:42
*** electrofelix has joined #tripleo08:43
shardyThis can land now I think https://review.openstack.org/#/c/318699/08:43
shardyjust updates the ips-from-pool sample environment to show the Management network via comments08:44
openstackgerritMerged openstack/tripleo-heat-templates: Configure BlockStorage services via resource chains  https://review.openstack.org/32285808:45
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Add support for OVB based CI  https://review.openstack.org/33478708:51
*** bootsha has quit IRC08:52
*** andreas-f has quit IRC08:52
*** osp has joined #tripleo08:52
dtantsur|brbshardy, gfidente, hey! the CI passed on https://review.openstack.org/316128 please rereview when you have a minute08:53
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates: Composable Timezone service - Controller  https://review.openstack.org/33152808:53
openstackgerritMerged openstack/tripleo-heat-templates: Increase cluster sync timeout for M->N major upgrades  https://review.openstack.org/33566608:54
gfidenteccamacho, thanks for the updates on the timezone rol08:55
gfidente:)08:55
ccamachogfidente, np, Im rebasing all the patches for the storage role landed :)08:55
*** ohamada has quit IRC08:55
openstackgerritMerged openstack/tripleo-heat-templates: Drop unused VIP params to controller.yaml  https://review.openstack.org/33351108:55
*** ohamada has joined #tripleo08:55
gfidenteccamacho, yeah unfortunately the problem there is that the blockstorage role got updated after your tree built on top of it was submitted08:56
gfidentebut at least the timezone submissions won't have to be rebased one by one when they merge08:57
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: [NO MERGY] Test a fake periodic job  https://review.openstack.org/22978908:58
derekhsshnaidm: upgrades job failed last night ^^ will rurun a fake periodic job, if the upgrades job passes then the current-tripleo pin should update08:59
derekhtrown|outtypewww: ^08:59
*** mandre_ has joined #tripleo09:00
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Install packages for components introduced in Mitaka  https://review.openstack.org/33586709:00
*** fragatina has quit IRC09:00
ccamachogfidente, np with the manual rebase :)09:00
*** numans has joined #tripleo09:02
*** aufi_ has joined #tripleo09:04
*** Slower_ has joined #tripleo09:04
*** mgould|afk is now known as mgould09:04
*** markmc` has joined #tripleo09:06
*** fzdarsky has quit IRC09:07
*** afazekas_ has joined #tripleo09:07
*** leifmadsen_ has joined #tripleo09:07
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates: Composable Timezone service - Controller  https://review.openstack.org/33152809:08
*** ohamada has quit IRC09:08
*** kbyrne has quit IRC09:08
*** aufi has quit IRC09:08
*** mandre has quit IRC09:08
*** afazekas has quit IRC09:08
*** Slower has quit IRC09:08
*** lmiccini has quit IRC09:08
*** leifmadsen has quit IRC09:08
*** mordred has quit IRC09:08
*** markmc has quit IRC09:08
*** mandre_ is now known as mandre09:08
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates: Composable Timezone service - Compute  https://review.openstack.org/33153009:09
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates: Composable Timezone service - CephStorage  https://review.openstack.org/33153409:09
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates: Composable Timezone service - ObjectStorage  https://review.openstack.org/33153609:09
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates: Wire NTP and Timezone services into BlockStorage nodes  https://review.openstack.org/33274009:09
*** stendulker has joined #tripleo09:10
*** kbyrne has joined #tripleo09:11
*** bootsha has joined #tripleo09:12
*** mordred has joined #tripleo09:13
*** ohamada has joined #tripleo09:13
*** lmiccini has joined #tripleo09:13
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Don't install zaqarclient on upgrades  https://review.openstack.org/33587209:15
shardySimple review here if anyone has a moment (passed CI): https://review.openstack.org/#/c/334150/09:18
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Increase cluster sync timeout for M->N major upgrades  https://review.openstack.org/33587609:23
*** osp has quit IRC09:33
*** psanchez has joined #tripleo09:34
*** snecklifter has quit IRC09:34
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Add support for OVB based CI  https://review.openstack.org/33478709:38
*** osp has joined #tripleo09:46
*** ohamada has quit IRC09:46
*** ohamada has joined #tripleo09:47
*** dtantsur|brb is now known as dtantsur09:49
*** ohamada has quit IRC09:51
*** snecklifter has joined #tripleo09:54
derekh<lucasagomes> derekh, you can add whatever port to the http server, by default ironic looks for 8088, but can be changed in the [deploy] http_url configuration option09:54
derekhlucasagomes: sorry, never replied to this yesterday09:54
derekhlucasagomes: so basically, this port clashes with the port that infra now use for console access to running CI jobs09:55
lucasagomesderekh, no worries, right...09:55
derekhlucasagomes: and I'm wondering which is easier to change09:56
lucasagomesderekh, the http server configuration will be under /etc/httpd/conf.d/ironic.conf09:56
lucasagomesthe port will be there, once you change the port and restart apache09:57
lucasagomesyou can just put the url + port in the ironic configuration09:57
lucasagomese.g: http_url = http://192.168.122.156:392809:57
*** jprovazn has quit IRC09:57
derekhlucasagomes: yes, bet then tripeo wont be using the default port that ironic uses09:57
derekhlucasagomes: not sure if thats a big deal or not09:57
lucasagomesderekh, that's fine, the port is suppose to be configured09:58
lucasagomeseven in devstack I think we don't use the 808809:58
derekhlucasagomes: ok, thanks09:58
lucasagomesderekh, we should make it easily configurable tho,... lemme see if the ironic puppet modules already parametize the http port09:59
lucasagomes1 sec09:59
dtantsurderekh, fix this: https://github.com/openstack/puppet-ironic/blob/e3c2ae631c4108bfb01e1f696d38a5881f9ef513/manifests/drivers/deploy.pp#L1709:59
dtantsurto use another port09:59
dtantsurlucasagomes, ^^09:59
*** bootsha has quit IRC09:59
dtantsurderekh, we're already using non-default port: https://github.com/openstack/instack-undercloud/blob/b8c5ac736733e28315364a0c9e70465b6f41166d/elements/puppet-stack-config/puppet-stack-config.yaml.template#L42009:59
dtantsurjust fix this line to use something else10:00
derekhdtantsur: ack, thanks10:00
lucasagomesdtantsur, cool10:00
*** tosky has joined #tripleo10:04
*** dciabrin has quit IRC10:05
*** dciabrin has joined #tripleo10:05
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Add support for OVB based CI  https://review.openstack.org/33478710:06
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Add the OVB based job to the triple ci report  https://review.openstack.org/33591610:10
*** fragatina has joined #tripleo10:10
*** fragatina has quit IRC10:17
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: [NO MERGY] Test a fake periodic job  https://review.openstack.org/22978910:17
*** snecklifter has quit IRC10:18
*** zoliXXL is now known as zoli|lunch10:22
*** thrash|g0ne is now known as thrash10:26
openstackgerritMerged openstack/tripleo-common: Add upload-puppet-modules script  https://review.openstack.org/29964310:27
*** akrivoka has joined #tripleo10:27
*** jefrite has joined #tripleo10:28
*** fragatina has joined #tripleo10:28
openstackgerritMartin AndrĂ© proposed openstack/tripleo-common: Allow running validation against different plans  https://review.openstack.org/31819410:29
openstackgerritMartin AndrĂ© proposed openstack/tripleo-common: Disable retry files for ansible validations  https://review.openstack.org/32903910:29
openstackgerritMartin AndrĂ© proposed openstack/tripleo-common: Validations actions and workbook  https://review.openstack.org/31363210:29
openstackgerritMartin AndrĂ© proposed openstack/tripleo-common: Deployment workflows for remote execution, etc.  https://review.openstack.org/31395310:29
openstackgerritMartin AndrĂ© proposed openstack/tripleo-common: Mistral action to drive Heat software deployments  https://review.openstack.org/31096610:29
*** bootsha has joined #tripleo10:29
*** fragatina has quit IRC10:32
openstackgerritDougal Matthews proposed openstack/tripleo-common: Remove the underscore from the base action client methods  https://review.openstack.org/33592010:33
*** fragatina has joined #tripleo10:33
openstackgerritMerged openstack-infra/tripleo-ci: Fix openstack-tripleo-common package name  https://review.openstack.org/33446710:33
*** jprovazn has joined #tripleo10:33
*** bootsha has quit IRC10:34
*** limao has quit IRC10:36
*** jschlueter has joined #tripleo10:40
*** fzdarsky has joined #tripleo10:40
*** chem has quit IRC10:42
*** chem has joined #tripleo10:42
*** thrash is now known as thrash|bbl10:49
openstackgerritDougal Matthews proposed openstack/tripleo-common: Remove the underscore from the base action client methods  https://review.openstack.org/33592010:54
d0ugalthrash|bbl: Thanks for the quick review! ^ fixed10:54
*** stendulker has quit IRC10:57
sshnaidmderekh, hi, seems like CI is stable today10:58
d0ugal\o/10:58
derekhsshnaidm: yup it does10:59
sshnaidmand almost all periodic passed \o/10:59
*** xinwu has joined #tripleo10:59
d0ugalDon't tempt fate!10:59
derekhsshnaidm: but the bad new is that new jobs stopped getting queued up about an hour ago http://status.openstack.org/zuul/10:59
derekhsshnaidm: see the check-tripleo queue10:59
sshnaidmderekh, yeah, I saw10:59
*** xinwu has quit IRC10:59
sshnaidmderekh, but all of them in merge-check queue11:00
derekhsshnaidm: I asked in #openstack-infra but no answer,11:00
sshnaidmderekh,  my patches at least there11:00
sshnaidmand merge-check queue is huge now..11:00
derekhsshnaidm: hmm, maybe that queue is stuck or something11:00
derekhwill ask on #infra see if that but of info helps11:01
*** fzdarsky has quit IRC11:01
derekhsshnaidm: hmm, merge-check getting smaller now, lets see what happens11:01
sshnaidmderekh, somebody cleared it now completely11:02
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Add support for OVB based CI  https://review.openstack.org/33478711:04
openstackgerritMerged openstack/instack-undercloud: Drop deprecated nova network options  https://review.openstack.org/32709711:06
*** rook- has joined #tripleo11:06
openstackgerritMerged openstack/tripleo-heat-templates: Increase RabbitMQ maximum file descriptors.  https://review.openstack.org/33212711:11
openstackgerritMerged openstack/python-tripleoclient: Updated from global requirements  https://review.openstack.org/33238311:11
*** weshay has joined #tripleo11:11
*** osp has quit IRC11:15
*** jpena is now known as jpena|lunch11:16
*** rhallisey has joined #tripleo11:22
openstackgerritGabriele Cerami proposed openstack/tripleo-quickstart: Devmode: rename delorean current so any existing repo is overwritten  https://review.openstack.org/33595311:23
*** osp has joined #tripleo11:28
openstackgerritDan Prince proposed openstack/puppet-tripleo: Resolve duplicate base::ceilometer issues  https://review.openstack.org/33596011:30
*** hewbrocca-afk is now known as hewbrocca11:30
*** zoli|lunch is now known as zoli11:32
*** zoli is now known as zoliXXL11:32
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Re-enable Ceilometer composable roles for controller  https://review.openstack.org/33349811:32
*** mburned_out is now known as mburned11:33
*** panda has quit IRC11:34
shardygfidente: hey, have you seen https://review.openstack.org/#/c/332926 from dprince?11:35
*** panda has joined #tripleo11:35
shardyit changes the StepConfig interface to pass all of the role_data mapping from services/*11:35
shardywhich gets you a step closer to having the interface needed for creating a merged list of enabled services, as we discussed yesterday11:36
d0ugaldtantsur: Thanks for the review re: osc_lib. Do you have a link where I can read more?11:36
d0ugaldtantsur: python-mistralclient is using it - so we are going to be stuck with it as a dep already.11:36
dtantsurwell, inspector-client is also using it11:37
d0ugalaha11:37
dtantsurbut on ironicclient review we were told it's premature11:37
dtantsurlemme find11:37
*** ccamacho is now known as ccamacho|lunch11:39
d0ugaldtantsur: https://review.openstack.org/#/c/333311/11:40
d0ugaland https://review.openstack.org/#/c/334803/11:40
dtantsurcorrect. damned gertty is too slow today >_<11:40
d0ugal:)11:40
gfidenteshardy, looking thanks11:42
*** akrivoka has quit IRC11:42
gfidenteright so we could just merge RoleData11:42
*** akrivoka has joined #tripleo11:42
shardygfidente: Yeah, and have some key like global_config or something11:42
*** numans has quit IRC11:42
gfidenteadded as output11:43
shardyor yup11:43
gfidenteso we only "share" what we want to be global11:43
gfidenteok , nice , let me try11:43
*** athomas has quit IRC11:44
shardyOne problem is map_merge doesn't yet support deep merging, so we'd either need to have an output like enabled_services: {glance-api: True}, or a list inside global_config which could be list_join'd for all roles11:45
shardyOr, we could make map_merge do deep merging I guess11:46
*** paramite has quit IRC11:46
* shardy -> lunch11:46
*** paramite has joined #tripleo11:46
*** athomas has joined #tripleo11:50
EmilienMhello11:51
*** hewbrocca is now known as hewbrocca-afk11:58
gfidenteshardy, can't we do map_merge: {get_addr: [ServiceChain, role_data, global_config]} ?11:58
*** jcoufal has joined #tripleo11:59
dtantsurg'afternoon folks. Please review the ironic-in-overcloud patch https://review.openstack.org/316128 when you have a second. Passed the gate.12:00
*** fragatina has quit IRC12:01
EmilienMdtantsur: it passed the gate because you don't enable it ;-)12:01
EmilienMbut this is a really good start12:02
dtantsurEmilienM, there is a follow-up patch testing it: https://review.openstack.org/#/c/329872/12:02
shardygfidente: Yes, but I think if you had, say, an "enabled_services" key inside global_config, each service would overwrite (rather than append) to that list/map12:02
dtantsur(no HA support yet, thus only non-HA job passes, I guess)12:02
EmilienMyeah, it's passing nonha, which is a good start. Though failing on HA & upgrade12:02
EmilienMdtantsur: I'll look at it12:02
dtantsurright, I'm yet to write HA support for ironic (after I'm done with non-HA, that is)12:02
shardygfidente: so you'll need either an "enabled_services" output, which is then merged, or some key inside global_config which can be joined together12:03
dtantsurthanks!12:03
*** lucasagomes is now known as lucas-hungry12:03
openstackgerritMiles Gould proposed openstack/instack-undercloud: Fix inspection_enable_uefi description  https://review.openstack.org/32455512:03
openstackgerritMiles Gould proposed openstack/instack-undercloud: Add option to enable introspection of UEFI nodes  https://review.openstack.org/32111912:03
gfidenteshardy, ok I see what you mean12:05
gfidenteinitially I didn't think about same key emitted by multiple roles12:05
*** coolsvap_ has quit IRC12:05
shardyit may be easiest to just have an enabled_service output or something (which could be either a list or a map12:06
gfidentemy use-case would be for example, emitting from the cinder role the list of enabled backends so that nova role can behave differently based on that12:06
gfidenteor, another example, emit from each role if its endpoint has to be created or not12:06
*** akuznetsov has joined #tripleo12:06
shardyah, yeah I was just thinking about the keystone endpoints12:06
gfidenteso that we could from the keystone role create the endpoints for those which demand to12:06
shardyif we went with the global_role_data approach the map it outputs could have a key that's the name of the service12:07
gfidenteyep12:07
shardye.g nova-api: {blah}12:07
shardycool, sounds good12:07
gfidenteor even a flat12:07
gfidenteenable_nova_api_endpoint: true12:07
openstackgerritMerged openstack/puppet-tripleo: Resolve duplicate base::ceilometer issues  https://review.openstack.org/33596012:08
gfidenteactually12:08
gfidentetripleo::profile::base::keystone::create_nova_api_endpoint: true12:08
shardygfidente: Ah, yes that could work, and we can use yaql to transform the mapping after it's merged12:08
shardyAh, so create some hieradata, like a global_config_settings ?12:08
shardywhich is then deployed to all nodes?12:09
shardyallnodes_config_settings perhaps12:09
EmilienMshardy: hey12:09
gfidentedeployed on all nodes exactly12:09
EmilienMabout https://review.openstack.org/#/c/335663/1/templates/tenantvm_floatingip.yaml12:09
EmilienMI already read12:09
EmilienMport_id = ID of the port associated with this IP.12:09
EmilienMit's not the router_id but the VM port12:09
EmilienMyou don't connect a fip to a router interface12:10
shardyOh so we actually depend on the RouterInterface12:10
shardyhmm12:10
socialchem: do you have issues with RDO Liberty overcloud deployment failing on mongodb?12:11
shardyOk, we can add the depends_on, but I think it may mean there's something broken in our heat modelling of the neutron resources12:11
EmilienMshardy: maybe we can go for it *now* so our CI gets a bit more stable, and I can add a todo to investigate it in the patch12:11
shardyEmilienM: sure, that's fine - I'd just like to understand if there's something wrong with the heat resources12:12
shardynormally depends_on shouldn't be required for resources which interact in the declarative model, only for operator defined serialization12:12
shardyI'm fine with merging it if it'll help with CI stability12:12
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci: pingtest/tenantvm_floatingip.yaml: connect vrouter before creating fip  https://review.openstack.org/33566312:13
EmilienMshardy: ok, I added a comment on the code, we can safely land it. I'll follow up with a patch that test something else, but at least now our CI is more stable12:14
EmilienMshardy: I'll let you approve it12:14
openstackgerritMerged openstack-infra/tripleo-ci: pingtest/tenantvm_floatingip.yaml: connect vrouter before creating fip  https://review.openstack.org/33566312:16
*** paramite is now known as paramite|afk12:17
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci: pingtest: don't show error if no images needs to be cleaned-up  https://review.openstack.org/33565512:18
EmilienMshardy: better ^  ?12:18
EmilienMshardy: also, can you look https://review.openstack.org/#/c/332897/ please?12:20
shardyEmilienM: Yes, thanks, and will do12:21
*** paramite|afk is now known as paramite12:21
*** egafford has joined #tripleo12:21
shardyhttps://review.openstack.org/#/c/318699 https://review.openstack.org/#/c/334150/12:22
shardyEmilienM: done, if you have time, could you please check those two ^^ ?12:23
shardyboth trivial and passing CI12:23
EmilienMyes12:23
shardythanks!12:23
EmilienMshardy: I always have time for composable services :)12:23
shardyThose actually aren't composable services, but thanks for looking :)12:23
EmilienMshardy: oh !!! you got me12:24
shardyhehe12:24
EmilienMshardy: your snmp now pass12:24
EmilienMI feel like CI is more stable12:24
shardy\o/12:24
EmilienMlot of green http://tripleo.org/cistatus.html12:24
shardyit has been much more stable today12:24
shardyalthough the jobs seem much slower than a few weeks ago after the hardware upgrade12:25
EmilienMshardy: did we increase testenv per host?12:25
EmilienMshardy: because the swift issue on the undercloud makes me think about perfs too12:25
shardyI'm not sure, derekh would know12:26
*** bswartz has joined #tripleo12:26
EmilienMthough we could have inject some regressions that would cause some slow performance :(12:26
shardybut we're seeing nonha taking over 2hrs again, and some upgrades jobs near the infra timeout at 2hr4012:26
shardyanyway green==good12:26
EmilienMthat's a lot12:27
EmilienMdon't we have metrics?12:27
*** jdob has joined #tripleo12:28
shardythere are some timing metrics in the logs per job, and we can use tripleo-jobs-gerrit.py to generate data for the walltime overall12:28
shardyI graphed centos7-nonha earlier, but really we need to combine with the pre-centos7 results to get a clear picture12:29
shardyI may do that later if I have time12:29
openstackgerritMerged openstack/tripleo-heat-templates: Convert noop NetworkExtraConfig to OS::Heat::None  https://review.openstack.org/33415012:29
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: First iteration of libvirt and nova-compute as a composable services  https://review.openstack.org/32968612:29
shardya graph on tripleo.org would be ideal12:29
EmilienMshardy: my nova/compute/libvirt stuff is not ready yet, see http://logs.openstack.org/86/329686/17/check-tripleo/gate-tripleo-ci-centos-7-nonha/c2c2edb/logs/overcloud-novacompute-0/var/log/nova/nova-compute.txt.gz#_2016-06-30_02_07_36_41912:30
EmilienMI just found this log12:30
openstackgerritMerged openstack/tripleo-heat-templates: Add Management net to ips-from-pool-all example  https://review.openstack.org/31869912:30
EmilienMyeah, I'll try to look at what takes time now12:30
shardyEmilienM: ack, I wasn't sure if that was ready, hence my only comments were cosmetic re the map_merge12:31
EmilienMI fixed it but I found in logs this error12:31
*** jeckersb is now known as jeckersb_gone12:32
*** rlandy has joined #tripleo12:33
openstackgerritMartin AndrĂ© proposed openstack/tripleo-common: Allow running validation against different plans  https://review.openstack.org/31819412:34
openstackgerritMartin AndrĂ© proposed openstack/tripleo-common: Disable retry files for ansible validations  https://review.openstack.org/32903912:34
openstackgerritMartin AndrĂ© proposed openstack/tripleo-common: Validations actions and workbook  https://review.openstack.org/31363212:34
*** fragatina has joined #tripleo12:34
*** akuznetsov has quit IRC12:37
openstackgerritMerged openstack/tripleo-heat-templates: Convert SNMP configuration to composable services format  https://review.openstack.org/32059712:39
*** jayg|g0n3 is now known as jayg12:40
openstackgerritLukas Bezdicka proposed openstack/tripleo-image-elements: Deprecate old RDO release  https://review.openstack.org/33598412:42
*** pradk has quit IRC12:43
openstackgerritMerged openstack/puppet-tripleo: Remove base/neutron/ml2.pp  https://review.openstack.org/33287012:43
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: First iteration of libvirt and nova-compute as a composable services  https://review.openstack.org/32968612:44
EmilienMshardy: ok I found out nova/libvirt thing, should be good now :)12:44
shardycool :)12:45
derekh<EmilienM> lot of green http://tripleo.org/cistatus.html12:45
derekh<shardy> it has been much more stable today12:45
*** jpena|lunch is now known as jpena12:46
derekhEmilienM: shardy but nothing is running at the moment12:46
EmilienMhow that nothing?12:46
*** ccamacho|lunch is now known as ccamacho12:46
derekhI've asked in infra a couple of times but nobody is around there yet that seems to know why12:46
gfidenteEmilienM, a questions about this one https://review.openstack.org/#/c/333326/12:46
derekhEmilienM: the check-tripleo zuul queue has been empty for hours12:47
gfidenteI agree with what you're saying but my purpose for the change was just to switch the default from true to falsde12:47
gfidentewould you prefer me to leave the selinux execs untouched?12:47
EmilienMgfidente: replied12:47
EmilienMgfidente: why disabling selinux? :(12:47
EmilienMderekh: oh, I didn't know that12:48
gfidenteEmilienM, we had to disable it in the past12:48
gfidenteI'm thinking that the execs at least won't conflict with the class parameter12:48
EmilienMit's odd to disable selinux12:48
gfidentewould maybe override it, but not cause compilatin errors12:48
rhalliseygfidente, can you give me some AVCs?  I may be able to fix the problem12:49
gfidenteyeah we have been asked to do it in the past because of known issues with ceph-osd12:49
EmilienMccamacho: I'm about to rebase your patches for timezone, ok?12:49
ccamachoEmilienM np I /me can do it12:49
gfidenterhallisey, hey thanks for jumping in, the submission is actually switching the default so that we *don't* disable selinux anymore12:49
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Composable Timezone service - Controller  https://review.openstack.org/33152812:49
derekh<shardy> although the jobs seem much slower than a few weeks ago after the hardware upgrade12:50
ccamachoIt is good that patches are landing :)12:50
gfidenterhallisey, https://review.openstack.org/#/c/333326/12:50
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Composable Timezone service - Compute  https://review.openstack.org/33153012:50
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Composable Timezone service - CephStorage  https://review.openstack.org/33153412:50
ccamachoEmilienM hey, thanks for the rebase :P12:50
rhalliseygfidente, gotcha12:50
derekhshardy: I'm slowley comming to the conclusion that no matter what our limit is in CI for time or RAM  we're just going to use it12:50
openstackgerritMerged openstack/puppet-tripleo: neutron plugins: include neutron base  https://review.openstack.org/33289712:51
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Composable Timezone service - ObjectStorage  https://review.openstack.org/33153612:51
shardyderekh: did we turn off image caching or something though, e.g we at one point had nonha jobs close to one hour, now they're back over two again12:51
EmilienMccamacho: my pleasure :)12:51
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Wire NTP and Timezone services into BlockStorage nodes  https://review.openstack.org/33274012:51
derekhshardy: a few weeks ago when the caching stuff was added the nonha job wend down to just over an hour, now here we are a few weeks later and we're using that time up somewhere else12:51
shardyderekh: Yeah, that's what I'm confused by :)12:52
*** zoliXXL is now known as zoli|brb12:52
derekhshardy: caching is still happening, but something else has taken up the time12:52
openstackgerritMerged openstack/puppet-tripleo: Nova API and Neutron server should run database creation in step 3  https://review.openstack.org/33541912:53
shardyderekh: Ok, I guess we'll have to figure out what, I just wanted to sanity-check nothing else had changed12:53
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Add gnocchi profiles  https://review.openstack.org/31552712:54
*** amoralej is now known as amoralej|lunch12:56
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Re-enable Ceilometer composable roles for controller  https://review.openstack.org/33349812:57
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Default ceph_osd_selinux_permissive to false  https://review.openstack.org/33332612:58
*** rodrigods has quit IRC12:58
*** rodrigods has joined #tripleo12:58
derekhshardy: the middle of may is when we started using the cached overcloud images https://23.253.94.71/render?from=-60days&until=now&width=400&height=250&target=tripleo.ci.total.seconds&_uniq=0.02441948416753137&title=tripleo.ci.total.seconds12:59
derekhshardy: from looking at that graph since then we've added about 33 minutes to the runtime13:00
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Ceilometer Compute Agent composable service  https://review.openstack.org/33180213:01
EmilienMwow13:02
EmilienMthat's probably related13:02
EmilienMderekh: how using cached overcloud images would add runtime? It should be opposite, no?13:02
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: services/ceilometer: switch to polling agent  https://review.openstack.org/33250413:02
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: compute: deploy composable neutron service  https://review.openstack.org/33123413:03
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: compute: moving out iscsi bits, now in puppet-tripleo  https://review.openstack.org/33124713:03
derekhEmilienM: it is the opposite, my point is that we started using the image caching, that saved us run time, and then instead of  keeping the run time down we added other things that increased the runtime13:04
EmilienMderekh: other things == new services?13:04
*** dsariel has quit IRC13:04
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: keystone: deploy composable firewall rules  https://review.openstack.org/33078513:05
derekhEmilienM: probably, or we add more step to the deployment or a combination of these13:05
EmilienMgfidente: any idea about it^ you're working on steps quite often13:05
*** lucas-hungry is now known as lucasagomes13:06
gfidenteright but not adding steps though :)13:06
derekhEmilienM: We do the same with RAM, I'f I Give use an additional 2GB of RAM CI today, we'll probably be using it ALL in a few weeks because nothing stopped us from doing it13:06
gfidenteI it maybe one of the steps taking too much?13:08
gfidente*is it13:08
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates: Gnocchi composable roles  https://review.openstack.org/31841313:08
* gfidente back in a moment13:08
shardyderekh: just after the hardware upgrade we switch to centos7, I assume that's not related?13:09
gfidenteEmilienM, not sure if we can gather from compile time how long it takes to apply the manifest?13:09
shardymy local deployment times haven't changed much13:09
gfidenteyeah was about to say the same, locally it's not taking twice the time13:09
derekhshardy: it may be related, its hard to know for sure13:10
*** tzumainn has joined #tripleo13:10
derekhshardy: and we're kind of committed to centos now anyways13:10
shardyderekh: do we have any stats re the testenv load, e.g loadavg during the test etc?13:10
shardyderekh: sure, but maybe there's some tuning needed or something13:11
derekhshardy: We had some dstat stats from the testenvs before the HW upgarde, but we don't have any since then13:12
derekhshardy: yup, maybe there is tunning to be done13:12
*** trown|outtypewww is now known as trown13:13
EmilienMgfidente: Ill investigate13:13
shardyhttps://23.253.94.71/render/?width=586&height=308&_salt=1467292422.987&target=tripleo.overcloud.nonha.deploy.seconds&from=00%3A00_20160526&until=23%3A59_2016063013:14
shardyovercloud deploy time doesn't seem to have changed much really13:14
*** fzdarsky has joined #tripleo13:16
shardyhttps://23.253.94.71/render/?width=586&height=308&_salt=1467292561.71&from=00%3A00_20160526&until=23%3A59_20160630&target=tripleo.overcloud.nonha.images.seconds&target=tripleo.overcloud.nonha.ping_test.seconds13:16
shardywhen we do build images it seems like the time taken is not at all well bounded13:16
shardyhttps://23.253.94.71/render/?width=586&height=308&_salt=1467292620.103&from=00%3A00_20160526&until=23%3A59_20160630&target=tripleo.overcloud.nonha.ping_test.seconds13:17
shardythe pingtest is taking much longer...13:17
* shardy checks when heat enabled convergence13:17
slaglei wouldnt be surprised if we've lost a couple of minutes on the undercloud install as well since moving more things to puppet13:18
derekhshardy: the image build time can be explained by the caching, looks like we're loggin times even if the cache is used13:18
shardyIt'd be good to work out when https://github.com/openstack/heat/commit/909c842e3dc1d13fc104f3f054dad9dfc339bcc5 hit current-tripleo13:19
shardyit's about a week before the spike in pingtest time13:20
shardyMaybe we should land https://review.openstack.org/#/c/333890/ and see what happens to the graph ;)13:21
trownthe overcloud deploy is taking more cpu now for sure... RDO HA jobs are unpassable on the 3 of 4 chassis that have slower CPU13:21
trownbut it is true for mitaka as well... though not liberty13:22
*** electrofelix has quit IRC13:25
derekhshardy: we move the pin on June 15th, I think that when we started using the commit you linked to13:25
derekh*moved13:25
shardyhttps://23.253.94.71/render/?width=586&height=308&_salt=1467293032.471&from=00%3A00_20160526&until=23%3A59_20160630&target=tripleo.overcloud.nonha.ping_test.seconds&target=tripleo.undercloud.install.seconds13:28
shardyderekh: interesting, well it definitely seems like a combination of undercloud and pingtest are costing us about 10mins13:28
derekhshardy: yup looks pretty definite13:30
dciabrinjistr, marios hey guys can you help on ci failures http://logs.openstack.org/05/325205/25/check-tripleo/gate-tripleo-ci-centos-7-upgrades/6972b2a/ ?13:31
*** zoli|brb is now known as zoli13:33
*** electrofelix has joined #tripleo13:34
jistrdciabrin: Error: /Stage[main]/Keystone::Roles::Admin/Keystone_user[admin]: Could not evaluate: Command: 'openstack ["user", "show", "--format", "shell", ["admin", "--domain", "default"]]' has been running for more than 20 seconds (tried 0, for a total of 0 seconds)13:34
jistrin http://logs.openstack.org/05/325205/25/check-tripleo/gate-tripleo-ci-centos-7-upgrades/6972b2a/logs/overcloud-controller-0/var/log/messages13:34
jistri think this is an intermittent error13:35
jistrIIRC chem submitted a fix for that (increasing the timeout)13:35
*** jeckersb_gone is now known as jeckersb13:35
chemjistr: the fix is currently blocked by the gate as the puppet-keystone CI is broken13:36
dciabrinjistr, chem damned :)13:36
chemdciabrin: this should be completly fixed when this patch https://review.openstack.org/335600 land13:36
chemjistr: dciabrin it seems to be some rdo repo weirdeness, I'm tracking the issue there https://bugs.launchpad.net/puppet-keystone/+bug/159775313:38
openstackLaunchpad bug 1597753 in puppet-keystone "Broken CI: python-pysaml2 is always re-installed" [Critical,Confirmed]13:38
chemdciabrin: so you can recheck and if you're lucky it will be green13:39
dciabrinchem, ok thx for the pointer. i'll recheck and roll a d20 :)13:39
chemdciabrin:hehe13:39
*** amoralej|lunch is now known as amoralej13:40
derekhWould appreciate some eyeballs on the patch to run CI on RH2, https://review.openstack.org/#/c/334787/13:41
mgouldhi everyone13:41
derekhRH1 is being switched off on tuesday next so we need to get it merged with a few days run time if we can befor that13:41
mgouldI have some backports to stable/liberty open on instack-undercloud which have been broken by the release of puppet-lint 2.0.013:41
mgouldwhat's my best option? Backport https://review.openstack.org/#/c/333347/ ?13:42
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: profile/base/nova: declare nova class properly  https://review.openstack.org/32834713:43
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Remove usage of ::nova class in THT  https://review.openstack.org/32598313:44
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: nova/compute: add NFS bits  https://review.openstack.org/33295413:44
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Remove NFS bits from overcloud_compute manifest  https://review.openstack.org/33295513:44
openstackgerritMartin MĂ¡gr proposed openstack/tripleo-heat-templates: Availability monitoring support  https://review.openstack.org/25478813:48
*** ayoung has joined #tripleo13:48
*** r-mibu has quit IRC13:48
*** r-mibu has joined #tripleo13:48
openstackgerritMiles Gould proposed openstack/instack-undercloud: Fix inspection_enable_uefi description  https://review.openstack.org/32455513:51
openstackgerritMiles Gould proposed openstack/instack-undercloud: Add option to enable introspection of UEFI nodes  https://review.openstack.org/32111913:51
*** pradk has joined #tripleo13:52
*** jmiu_ has joined #tripleo13:52
openstackgerritPradeep Kilambi proposed openstack/tripleo-heat-templates: Add Aodh composable roles  https://review.openstack.org/33355613:55
*** akshai has joined #tripleo13:57
colonwqgfidente, I see how I made a mistake on my patch last night. I compared against HEAD instead of your version. It did not look right when I did it but HEAD is always correct.... sometimes.13:59
gfidentecolonwq, ack no worries13:59
*** social has quit IRC14:00
colonwqIt reminds me of the software development dance: 5 steps forward, 2 steps back.14:01
mgouldcolonwq: on a good day...14:03
*** derekh changes topic to "TripleO CI Jobs are not running, investigating why... : http://tripleo.org/ | https://wiki.openstack.org/wiki/TripleO | Meetings On Tuesdays at 14:00 UTC in #openstack-meeting-alt"14:06
openstackgerritMerged openstack/instack-undercloud: Increase swift-proxy node_timeout  https://review.openstack.org/33201414:10
dtantsurshardy, hey, 2nd +2 needed on the ironic patch we've discussed yesterday, if it looks sane now: https://review.openstack.org/#/c/316128/14:11
*** fultonj has joined #tripleo14:12
*** xinwu has joined #tripleo14:14
*** rcernin has quit IRC14:14
*** xinwu has quit IRC14:18
shardydtantsur: there one remaining issue that I mentioned yesterday - if you'd like to fix that via a followup patch I think we can land it14:19
dtantsurshardy, yeah, let me follow-up with it please14:19
shardydtantsur: Ok, thanks, approved!14:20
dtantsurthanks shardy!14:20
derekhshardy: I gotta remove the 80 node overcloud now, once this merges we'll be running rh2 jobs on all changes https://review.openstack.org/#/c/335862/14:21
shardyderekh: Ok, no problem, thanks for the access to it, been very useful14:21
derekhshardy: assuming we get jobs running again14:21
derekhshardy: no problem14:21
derekhshardy: you don't need to get anything from it first?14:22
shardyderekh: No, thanks, I copied what I need off already14:22
derekhshardy: ack14:22
*** noslzzp has quit IRC14:23
*** itamarl has quit IRC14:24
mgouldhey everyone, I just submitted https://review.openstack.org/#/c/336049/ to turn off non-doc CI for docs-only changes to python-tripleoclient14:24
mgouldso as to avoid horrors like https://review.openstack.org/#/c/321117/14:24
mgouldplease review...14:24
openstackgerritMerged openstack/tripleo-heat-templates: Basic support for deploying Ironic in overcloud  https://review.openstack.org/31612814:24
pradkcan we get this backport in https://review.openstack.org/#/c/335612/ please14:25
openstackgerritMerged openstack/tripleo-quickstart: Add extra-vars for enable pacamaker to upgrade ci script  https://review.openstack.org/33437114:27
*** rcernin has joined #tripleo14:29
openstackgerritDmitry Tantsur proposed openstack/tripleo-heat-templates: DO NOT MERGE: testing ironic  https://review.openstack.org/32987214:31
openstackgerritDmitry Tantsur proposed openstack/tripleo-heat-templates: Move ironic::db data within service template  https://review.openstack.org/33605314:31
bnemecmgould: That's fine, but there's no reason to recheck a docs only change anyway.  Cores should use their judgment in that case and recognize that tripleo-ci isn't testing anything.14:31
*** jistr is now known as jistr|mtg14:31
mgouldbnemec: I agree, but that's not what happened to me14:32
bnemecWe really need to resurrect the reviewer guidelines.14:32
mgouldalso, running heavly tripleo-ci jobs for docs-only patches even once is a waste of gate resources14:33
mgouldbnemec: that sounds like a great idea14:33
bnemecAgreed.  Your change is definitely good regardless.14:33
derekhmgould: comment added, can you add a new job to the regex "gate-tripleo-ci-centos-7-ovb-ha"14:34
bnemecderekh: Will all of the jobs be getting a -ovb- when rh1 is back up?14:35
mgouldderekh: sure. Any reason I shouldn't match gate-tripleo-ci-.*?14:35
derekhbnemec: the current plan was to bring rh1 back up as is, i.e. nothing changes, but I think if the OVB job works well then I'd like to approach the subject of redeploying it with OVB instead of whats there at the moment14:36
bnemecderekh: Yeah, it would be really nice to modernize that environment.14:37
derekhmgould: I think gate-tripleo-ci-.* should be ok, I'll double check and get back to you14:37
openstackgerritMerged openstack/python-tripleoclient: Don't pass None via UpdateIdentifier  https://review.openstack.org/33459814:38
derekhbnemec: yup, and we could remove a bunch of code from toci_*14:38
mgouldderekh: excellent14:38
derekhmgould: best I can tell "gate-tripleo-ci-.*" will match http://paste.openstack.org/show/524275/  , so looks ok to me14:43
mgouldexcellent14:44
mgouldI've pushed up a new version14:44
*** rcernin has quit IRC14:46
*** pcaruana has quit IRC14:46
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Re-enable Ceilometer composable roles for controller  https://review.openstack.org/33349814:49
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Ceilometer Compute Agent composable service  https://review.openstack.org/33180214:49
*** saneax is now known as saneax_AFK14:49
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: services/ceilometer: switch to polling agent  https://review.openstack.org/33250414:49
*** rbrady has quit IRC14:50
*** akuznetsov has joined #tripleo14:51
openstackgerritJulie Pichon proposed openstack/python-tripleoclient: Add 'openstack overcloud node provide' command  https://review.openstack.org/33441114:52
openstackgerritmathieu bultel proposed openstack/tripleo-quickstart: add bootstrap into the upgade ci script  https://review.openstack.org/33606614:54
openstackgerritMerged openstack/tripleo-quickstart: add bootstrap into the upgade ci script  https://review.openstack.org/33606614:54
*** rbrady has joined #tripleo14:55
*** skramaja is now known as skramaja|afk14:58
*** dsneddon has joined #tripleo14:58
*** rcernin has joined #tripleo14:58
*** dsneddon has quit IRC14:58
*** dsneddon has joined #tripleo14:58
*** akshai has quit IRC14:59
*** akshai has joined #tripleo15:00
ccamachohey pradk are you arround?15:01
pradkccamacho, yea15:01
openstackgerritRaoul Scarazzini proposed openstack/tripleo-heat-templates: Add Rabbit/epmd port to firewall  https://review.openstack.org/33607215:02
ccamachopradk when have some time can you check my last comment on https://review.openstack.org/#/c/315527/  ? It is just to clarify a manifest call, also I need to move the db sync to another step15:02
*** thrash|bbl is now known as thrash15:05
pradkccamacho, i replied to your previous comment on api is that the one?15:06
openstackgerritMarek Aufart proposed openstack/tripleo-docs: Add note with boot config image options  https://review.openstack.org/33607815:07
*** masco has quit IRC15:07
*** dtantsur is now known as dtantsur|afk15:08
ccamachoprad, thanks! I still need to move some code to another deployment step, based on https://review.openstack.org/#/c/335419/ but in any other case the submission I think is good for reviews after the last change.15:08
*** noslzzp has joined #tripleo15:08
*** noslzzp has quit IRC15:08
*** noslzzp has joined #tripleo15:09
pradkccamacho, cool, I moved the database code from hieradata into service template in tht15:10
pradkyou mean on puppet-tripleo side?15:10
*** isq_ has quit IRC15:13
*** fzdarsky has quit IRC15:13
pradkEmilienM, slagle, can you guys +2 https://review.openstack.org/#/c/335612/ .. we need this for next puddle15:14
EmilienMderekh: can we land https://review.openstack.org/#/c/334787/ ?15:15
EmilienMpradk: landed15:15
pradkEmilienM, ty sir15:15
*** ebarrera has quit IRC15:17
*** rcernin has quit IRC15:19
openstackgerritBrad P. Crochet proposed openstack/tripleo-heat-templates: Composable Zaqar services  https://review.openstack.org/33168215:21
openstackgerritMerged openstack/tripleo-heat-templates: Fix ceilometer auth url  https://review.openstack.org/33561215:21
*** jistr|mtg is now known as jistr15:23
derekhCI jobs are back running again, unser why, what changed15:25
*** derekh changes topic to "TripleO : http://tripleo.org/ | https://wiki.openstack.org/wiki/TripleO | Meetings On Tuesdays at 14:00 UTC in #openstack-meeting-alt"15:25
derekhEmilienM: we should probably just wait for the current CI job to finish, but ya if people are happy with with its good to merge15:26
derekhEmilienM: also we need the dependent patch https://review.openstack.org/#/c/335622/315:26
EmilienMderekh: ok will wait CI15:26
rbradyjtomasek: wrt to the last comment in this file about labels/descriptions https://review.openstack.org/#/c/330283/5/tripleo_common/actions/plan.py , does this mean you're talking about adding something like http://paste.openstack.org/show/524291/ to https://github.com/openstack/tripleo-heat-templates/blob/master/overcloud.yaml#L1007-L103715:27
*** tesseract- has quit IRC15:27
*** aufi_ has quit IRC15:29
derekhThis has been sitting there for a while, it gives us more accurate timestamps in the console logs https://review.openstack.org/#/c/320905/15:29
jtomasekrbrady: yes15:29
rbradyjpeeler: ^^15:30
*** florianf has quit IRC15:31
*** akuznetsov has quit IRC15:35
*** akuznetsov has joined #tripleo15:35
openstackgerritMerged openstack-infra/tripleo-ci: Prevent python from buffering stdout  https://review.openstack.org/32090515:35
*** akrivoka has quit IRC15:36
openstackgerritPradeep Kilambi proposed openstack/tripleo-heat-templates: Re-enable Ceilometer composable roles for controller  https://review.openstack.org/33349815:37
*** akrivoka has joined #tripleo15:38
*** akuznetsov has quit IRC15:39
*** florianf has joined #tripleo15:44
*** fragatina has quit IRC15:54
*** rasca_ has joined #tripleo15:55
*** rasca has quit IRC15:55
*** fragatina has joined #tripleo15:55
*** florianf has quit IRC15:56
*** akrivoka has quit IRC15:57
*** zoli is now known as zoli|gone16:00
*** noslzzp has quit IRC16:00
*** zoli|gone is now known as zoli_gone-proxy16:01
jidarhey guys, is there any way to prevent `openstack undercloud install` from performing all of the package upgrades?16:02
jidarI'm trying to rebuild a lab box to match my production systems16:03
*** TSCHAK has quit IRC16:03
jidarI can hitup a bigass yum install <package list from prod> but as soon as I hit that undercloud install it does a bunch of yum -y install <foo>16:04
trownjidar: not without some hacking... it has been a pet peave of mine for some time as well, just havent ever filed a bug/patch for it16:05
jidar:( :( :(16:05
jidareven if I opened a bug there's no way it would be backported to this very early 7 release I'm on16:06
trownjidar: you can hack this line with a noop command https://github.com/openstack/instack-undercloud/blob/master/elements/puppet-stack-config/puppet-stack-config.pp#L2016:06
trownoh, on OSP7, that is in a different spot16:06
jidartrown: you sure it's only the puppet stuff that's doing the instals16:06
bnemecNo, I'm pretty sure it's not.16:06
jidarI ask beause I'm fairly sure it's built into other parts (ie: no -e 0 -d stuff in some of my yum history)16:07
jidargahhh16:07
trownno on kilo it is lots of elements16:07
trownon master, hacking out that yum update line definitely works.. I have tried it16:07
jidarfair enough16:07
bnemecI believe we run an yum update because we've had a lot of trouble with people having outdated packages that are incompatible.16:07
trownI have a similar hack for older stuff though16:07
shardyjidar: you could probably just disable all the yum repos if the box already has versions of everything you need installed?16:08
jidarI could try16:08
trownshardy: that doesnt work (or at least has not for me in the past), it complains about missing packages16:08
jidarI wonder if the install would just fall over though16:08
shardytrown: Yeah, I was interpreting this as re-running puppet on a cloned box16:08
*** tremble has quit IRC16:09
jidarsounds like the kilo release is going to call out a few install -y's though, prior to the puppet stuff16:09
trownshardy: well in my case it was running against an image that actually did have all packages installed... it has been a while though, so I dont totally remember what fell over16:09
*** shivrao has joined #tripleo16:09
*** florianf has joined #tripleo16:09
jidarI see git, iscsi, and a few other packages being installed16:10
jidarprior to the puppet apply stuff16:10
shardyIt is possible to just run puppet via /usr/libexec/os-refresh-config/configure.d/50-puppet-stack-config16:11
shardyit's just a script16:11
bnemecjidar: Install matching versions of those packages first too?16:11
shardybut you would need the hieradata in place from a previous run of undercloud install16:11
jidarbnemec: I've got that part worked out16:11
trownjidar: this hack works on older versions of instack-undercloud https://github.com/radez/apex/blob/d79a80135056e4badbd891167ee51ec38be56f4c/ci/deploy.sh#L753-L76016:12
jidarI've got the hieradata too, but something is weird about it, some hiera lookups fail.. which might be another issue16:12
trownits not pretty :)16:12
jidarheh16:12
trownbut opnfv apex project had that issue, and that ^ was their solution16:12
jidarlet me spend some time on this, thanks for all the heads up - you guys rock16:13
*** ifarkas has quit IRC16:14
*** fragatina has quit IRC16:15
shardyNow CI is working again we might consider merging https://review.openstack.org/#/c/32088316:15
shardymakes validation warnings/errors fatal in CI16:15
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates: Replace NodeConfigIdentifiers with DeployIdentifier  https://review.openstack.org/31561616:17
mgouldbnemec, shardy: would it be possible to merge https://review.openstack.org/#/c/324555/ and the following patch despite the puppet-lint failure, which is caused by an unrelated problem?16:18
openstackgerritMerged openstack-infra/tripleo-ci: Make deploy validation errors/warnings fatal for CI  https://review.openstack.org/32088316:18
mgouldalternatively, can you suggest the most appropriate fix?16:19
bnemecmgould: No, we can't ignore puppet-lint failures.16:19
mgoulddammit16:19
bnemecmgould: I think the correct thing to do would be to just push a fix for the lint failure.16:20
mgouldthe problem is that the Gemfile is pulling in the HEAD commit of puppet-lint from a random GitHub repo, which (as of the 22nd) no longer matches the spec of one of its reverse dependencies16:20
mgouldso it's not actually a lint failure16:20
bnemec\o/ random github repos16:20
mgouldit's a "Gemfile assumes puppet-lint 2.0.0 will never be released" error16:21
bnemecEmilienM: Thoughts? ^16:21
mgouldwhich wasn't a *totally* unreasonable assumption, given their release tempo...16:21
mgouldI could maybe backport the changes to Gemfile in master16:21
jidaris "/usr/share/diskimage-builder/elements/yum/bin/install-packages" what yum is calling under the hood to execute the install of a package?16:21
jidarholy crap, is that why this is so ugly? hahahah16:21
openstackgerritBrad P. Crochet proposed openstack/tripleo-heat-templates: Composable Mistral services  https://review.openstack.org/32343616:21
bnemecjidar: No, that's what calls yum.16:22
jidarclose enough16:22
jidarthat's an amazing solution16:22
*** florianf has quit IRC16:22
EmilienMbnemec: where?16:23
mgouldEmilienM: my comment from three minutes ago16:23
bnemecEmilienM: mgould's question about the puppet-lint failure on https://review.openstack.org/#/c/324555/16:23
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci: pingtest: don't show error if no images needs to be cleaned-up  https://review.openstack.org/33565516:24
EmilienMok looking16:24
*** shivrao has quit IRC16:24
EmilienMmgould, bnemec: I would do recheck16:25
EmilienMit sounds like a random error16:25
EmilienMTypeError: no implicit conversion from nil to integer16:25
EmilienMoO16:25
EmilienMno  idea but I think lint is good16:25
mgouldOK, rechecking16:26
bnemecEmilienM: Thanks16:26
mgouldah, wait, that's because I altered the Gemfile in16:27
mgouldhttps://review.openstack.org/#/c/32111916:27
dsneddonEmilienM, Got a sec? I'm trying to finish up the dual-stack public API work, but I'm getting a parsing error when I supply an array: http://pastebin.test.redhat.com/38892216:28
mgould...which is causing that problem16:28
mgoulddammit16:28
*** tbonds has quit IRC16:28
openstackgerritgreghaynes proposed openstack/diskimage-builder: Add specs dir  https://review.openstack.org/33610916:29
EmilienMmgould: why did you do that?16:29
mgouldbecause of the "Gemfile points at GitHub HEAD" issue described above16:29
*** tbonds has joined #tripleo16:29
mgouldit seemed worth trying16:30
*** trown is now known as trown|lunch16:30
mgouldOK, so I'll revert the change to the Gemfile; can you suggest a next step?16:31
EmilienMdsneddon: ack, after lunch16:31
dsneddonEmilienM, No problem.16:32
EmilienMdsneddon: oh yes quotes probably16:32
EmilienM:)16:32
EmilienMmgould: what next step?16:32
EmilienMdon't touch Gemfile, never16:32
dsneddonEmilienM, I'll try it, thanks.16:32
EmilienMit's status16:32
EmilienMstatic*16:32
mgouldOK16:32
mgould'what can I do to get my patch merged, given that the puppet-lint check will always fail during "bundle install"?'16:33
EmilienMwhy does it fail??16:33
openstackgerritBen Nemec proposed openstack-infra/tripleo-ci: Add undercloud idempotency test  https://review.openstack.org/27921816:34
openstackgerritBen Nemec proposed openstack-infra/tripleo-ci: TEST: Run undercloud idempotency test on all jobs  https://review.openstack.org/33611216:34
openstackgerritMiles Gould proposed openstack/instack-undercloud: Fix inspection_enable_uefi description  https://review.openstack.org/32455516:34
openstackgerritMiles Gould proposed openstack/instack-undercloud: Add option to enable introspection of UEFI nodes  https://review.openstack.org/32111916:34
mgould^^^ undoing change to Gemfile16:34
EmilienMyeah16:34
EmilienMit won't fail16:34
EmilienMand if it does, it's not because of your patch16:34
mgouldyes, exactly16:34
EmilienMdon't edit Gemfile except if you need to add gems or bump them16:34
EmilienMbut don't drop gems, never, our CI relies on it16:35
mgouldsome other puppet-lint-* module depends on puppet-lint 1.x16:35
mgouldOK, I won't do that again16:35
EmilienMmgould: I fixed Gemfile last week, maybe related16:35
openstackgerritMerged openstack/tripleo-puppet-elements: Add mistral packages to controller image  https://review.openstack.org/32950416:35
EmilienMok /me afk lunch now16:35
mgouldbecause of puppet-lint issue 409, we were tracking Gemfile HEAD16:35
mgouldwhich is now pointing to 2.0.016:35
mgouldhence the install dependencies are unsatisfiable16:35
mgouldEmilienM: yes, I think so16:36
*** akuznetsov has joined #tripleo16:36
mgouldshall I try backporting your patch to Gemfile?16:36
*** jpich has quit IRC16:36
*** ramishra has quit IRC16:37
EmilienMmgould: I'll look later, sorry I have to go now16:37
mgouldEmilienM: np, thanks!16:38
sshnaidmEmilienM, hi, do you know where to look if some openstack service is not started in deployment?16:40
*** akuznetsov has quit IRC16:40
sshnaidmEmilienM, I mean: sahara is installed, but the service itself is disabled and stopped, who is responsible for starting services in tripleo?16:41
toskyiirc sahara was always installed and enabled by tripleo up to mitaka, not sure if it was migrated to composable roles yet16:42
*** chem is now known as chem|off16:42
*** tbonds has quit IRC16:43
sshnaidmtosky, yes, until last month it was running, what are composable roles?16:44
toskysshnaidm: https://blueprints.launchpad.net/tripleo/+spec/composable-services-within-roles16:45
*** mcornea has quit IRC16:46
shardysshnaidm: I think it was a mistake in https://review.openstack.org/#/c/318840/16:48
shardythe new services weren't added to the ControllerServices list, sec I'll push a patch16:49
*** lucasagomes is now known as lucas-afk16:50
sshnaidmshardy, thanks, you can refer to bug in your patch: https://bugs.launchpad.net/tripleo/+bug/159228416:50
openstackLaunchpad bug 1592284 in tripleo "Sahara is not started in TripleO deployment" [Undecided,Triaged]16:50
ayoungshardy, is it possible to do a Tripleo deploy using DNS everywhere instead of IP address, or at least for the service catalog?16:51
*** rwsu has joined #tripleo16:52
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates: Add Sahara services to ControllerServices list  https://review.openstack.org/33611916:52
*** julim has joined #tripleo16:52
shardysshnaidm: ^^16:53
ayoungshardy, and, if so,  I want to use that to set the FQDN for the controller, and, prior to launching the controller, I want to register it as a host with IPA, get an OTP, and pass the OTP in the metadata to the controller node.  It will make the TLS Everywhere story much, much simpler16:54
bnemecayoung: It's possible for the public endpoints.  That's actually the default in the SSL case: https://github.com/openstack/tripleo-heat-templates/blob/master/environments/enable-tls.yaml16:54
ayoungbnemec, how do I set that up?  I was reading that doc, and it referes to it, but does not indicate how16:54
ayoungI can do the nameserver set up here: http://docs.openstack.org/developer/tripleo-docs/basic_deployment/basic_deployment_cli.html#configure-a-nameserver-for-the-overcloud16:54
sshnaidmshardy, cool, so ControllerServices is responsible for enabling and starting services in controller, right?16:55
shardysshnaidm: in the new composable services model, yes16:55
bnemecayoung: You set the CloudName parameter to your FQDN and then it gets used everywhere that CLOUDNAME appears in the EndpointMap.16:55
bnemecFor the internal and admin endpoints we'd probably need something different.16:56
ayoungbnemec, where is the source of CLOUDNAME?16:56
bnemecAlthough you could actually hard-code an FQDN in the endpoint map.  You don't have to use the parameters at all.16:56
ayoungbnemec, talk to me like I am just learning Tripleo...16:56
ayoungwhen you say these things, it implies I am changing....environment files?16:57
bnemecayoung: It's a top-level parameter: https://github.com/openstack/tripleo-heat-templates/blob/master/overcloud.yaml#L7316:57
bnemecayoung: Yes, you'd set it with an environment file.16:57
ayoungbnemec, ok, I'm using quickstart.  Let me see if it even has that variable set yet...16:58
gfidenteshardy, dah good catch!16:58
gfidentethanks16:58
shardygfidente: sshnaidm found the problem :)16:58
shardygood to see tempest catching these sorts of issues we miss with pingtest16:59
gfidente:)17:00
sshnaidmshardy, right! :)17:00
*** markmc` is now known as markmc17:00
ayoungbnemec, is there a way to set the IP address used for the controller so it is the same each time I do a deploy?17:01
bnemecayoung: With or without network isolation?17:01
shardyayoung: yes, there's details in the docs here: http://docs.openstack.org/developer/tripleo-docs/advanced_deployment/node_placement.html17:01
shardyheh, good question :)17:02
*** derekh has quit IRC17:02
bnemecI'm also assuming it isn't actually the controller address that he cares about, but the public VIP.17:02
openstackgerritJames Slagle proposed openstack-infra/tripleo-ci: Add mulitnode CI job support to tripleo-ci  https://review.openstack.org/32477717:02
openstackgerritJames Slagle proposed openstack-infra/tripleo-ci: Refactor common code for tripleo-ci  https://review.openstack.org/33562217:02
*** shivrao has joined #tripleo17:03
openstackgerritJames Slagle proposed openstack-infra/tripleo-ci: Add support for OVB based CI  https://review.openstack.org/33478717:04
openstackgerritJames Slagle proposed openstack-infra/tripleo-ci: Refactor common code for tripleo-ci  https://review.openstack.org/33562217:04
bnemecayoung: Without net-iso, you can use ControlFixedIPs like we do in CI: http://git.openstack.org/cgit/openstack-infra/tripleo-ci/tree/test-environments/enable-tls.yaml#n417:04
bnemecWith net-iso it will always use the first IP of the external allocation range.17:05
bnemecAlthough I believe you can make that explicit with the ExternalNetworkVip parameter: https://github.com/openstack/tripleo-heat-templates/blob/98f19c17a6a562cfcfed2dc9edc7391933492bc9/environments/external-loadbalancer-vip.yaml#L2217:05
ayoungbnemec, OK, nothing seems to be setting CLOUDNAME.    However, quickstart generates a file like this: https://paste.fedoraproject.org/386777/06254146/   Now, the controller is overcloud-controller-0.ayoung-dell-t1700.test   but I would want the CLOUDNAME to be more generic, like  cloud.ayoung-dell-t1700.test right?17:05
ayounggranted, I am testing non-HA for now, resource constrained,17:06
bnemecayoung: CloudName can be whatever you register in your DNS server.  It's not tied to the name of the controllers.17:07
ayoungbut, you are saying that whatever I make the CLOUDNAME will be the FQDN used for external endpoints.17:07
*** jpena is now known as jpena|off17:08
ayoungso from a DNS perspective, I need to create 1 hostname for the cloud, and then one per controller node17:08
ayoungthe CLOUDNAME fqdn needs to match the external VIP17:09
bnemecayoung: Right.  You don't necessarily need to register the individual nodes though.  They already get /etc/hosts entries for each other, so they don't need to be in the external DNS.17:09
ayoungbnemec, they do for TLS17:10
ayoungbnemec, we really need to stop using IP addresses everywhere...its an antipattern.17:10
ayoungand copying host entries around is so 199817:10
bnemecayoung: All of the nodes can already reference each other by DNS name.17:10
ayoungbnemec, really, what I need to get to is to have the hosts registered in IPA, and have IPA issue them a cert.  For that, I need a hostname, and the HTTP listeners need to accept that host name and cert17:11
EmilienMsshnaidm: ok back from lunch17:11
EmilienMsshnaidm: you got your answer? backlog is huge17:12
sshnaidmEmilienM, yeah, all right now :)17:12
EmilienMok17:12
ayoungbnemec, so, assuming I need to do work during the heat stack create for the overcloud, prior to posting the hiera data to the metadata server, where would I put that call?17:13
*** numans has joined #tripleo17:14
EmilienMok so stable/liberty gate is broken because of syntax/lint jobs, see https://review.openstack.org/#/c/32111917:16
EmilienMI have no time to work on it in short term, but might look tonight probably. Feel free to jump into it17:16
EmilienMI'm filling a bug17:16
EmilienMhttps://bugs.launchpad.net/tripleo/+bug/159784917:18
openstackLaunchpad bug 1597849 in tripleo "stable/liberty: puppet lint/syntax jobs failing" [Critical,Confirmed]17:18
bnemecayoung: I would guess something like ControllerPrePuppet: https://github.com/openstack/tripleo-heat-templates/blob/master/overcloud-resource-registry-puppet.yaml#L2817:18
bnemecBut honestly I don't think I'm the best one to be giving advice on that.17:19
bnemect-h-t isn't really my thing.17:19
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Output generic role_data in services.yaml  https://review.openstack.org/33286217:19
ayoungbnemec, the number of people that understand this is vanishingly small.  And shardy does not scale.17:19
ayoungI'm trying to get end-to-end knowledge so I can help guide others.17:20
*** julim has quit IRC17:21
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Pass RoleData into -post.yaml stacks  https://review.openstack.org/33292617:22
bnemecI have no good answer.17:22
bnemecThe whole thing is one giant Rube Goldberg machine.17:22
*** yamahata has quit IRC17:24
openstackgerritEmilien Macchi proposed openstack/instack-undercloud: Update Gemfile to work in gate  https://review.openstack.org/33612917:25
dsneddonEmilienM, I think I may have found a legitimate bug in the list_to_hash function. Not sure why loadbalancer.pp is trying to concatenate the two IP addresses, but they got smashed together: http://pastebin.test.redhat.com/38893817:26
openstackgerritEmilien Macchi proposed openstack/instack-undercloud: Update Gemfile to work in gate  https://review.openstack.org/33612917:26
EmilienMdsneddon: not good17:26
*** mgould is now known as mgould|afk17:27
shardyayoung: to inject hieradata before puppet runs you'll need to use either a firstboot (cloud-init, run once) script, or one of the ExtraConfigPre interfaces:17:27
*** numans has quit IRC17:27
shardyhttp://docs.openstack.org/developer/tripleo-docs/advanced_deployment/extra_config.html17:27
shardySimilar to what we do for rhel registration perhaps17:27
shardyhttps://github.com/openstack/tripleo-heat-templates/tree/master/extraconfig/pre_deploy/rhel-registration17:27
openstackgerritEmilien Macchi proposed openstack/instack-undercloud: Update Gemfile to work in gate  https://review.openstack.org/33613017:28
EmilienMmgould|afk: ^17:28
chem|offEmilienM: if you can get traction for review for this one https://review.openstack.org/#/c/335600/ should fix the last timeout in tripleo ci (when it's passing ci ...)17:29
shardyayoung: if it's something to run on all nodes, you can use OS::TripleO::NodeExtraConfig, if it's just controllers then use OS::TripleO::ControllerExtraConfigPre17:29
EmilienMchem|off: ack17:29
chem|offEmilienM: thanks17:29
shardyayoung: more examples of creating additional hieradata here:17:31
shardyhttps://github.com/openstack/tripleo-heat-templates/tree/master/puppet/extraconfig/pre_deploy/controller17:31
shardywe may want to rework things into the composable services format in due course, but if you need to run a script on each node, get some data, then update the hieradata, that interface should work17:32
openstackgerritGiulio Fidente proposed openstack/tripleo-common: Remove sysctl element from overcloud image  https://review.openstack.org/33613317:33
*** penick_ has joined #tripleo17:35
openstackgerritGiulio Fidente proposed openstack/python-tripleoclient: Remove sysctl element from overcloud image  https://review.openstack.org/33613417:36
*** penick has quit IRC17:36
*** penick_ is now known as penick17:36
gfidenteslagle, mburned ^^ so if I got it right we need to change it in two repos, and upstream CI will be testing it via tripleoclient17:36
*** akuznetsov has joined #tripleo17:36
gfidentemaybe I can add a depends-on to the tripleo-common change, just to make sure17:36
* mburned wonders if tripleo ci should switch to tripleo-common image building17:37
mburnedisn't that the future?17:37
slagleyes17:37
slaglewe need to17:37
slaglei recently pushed a patch to get the 2 back in sync17:38
openstackgerritGiulio Fidente proposed openstack/tripleo-common: Remove sysctl element from overcloud image  https://review.openstack.org/33613317:38
openstackgerritgreghaynes proposed openstack/diskimage-builder: Add specs dir  https://review.openstack.org/33610917:38
slagleprobably need to switch tripleo.sh over to use tripleo-build-images17:38
slaglethen deprecated the functionality in python-tripleoclient17:38
gfidentewhere is sync happening? needs votes?17:38
slaglegfidente: it already merged a week ago17:39
gfidenteok good17:39
*** trown|lunch is now known as trown17:40
*** akuznetsov has quit IRC17:40
openstackgerritgreghaynes proposed openstack/diskimage-builder: Add specs dir  https://review.openstack.org/33610917:41
*** gfidente has quit IRC17:44
*** athomas has quit IRC17:44
*** tosky has quit IRC17:46
*** andreas-f has joined #tripleo17:50
*** yamahata has joined #tripleo17:53
*** electrofelix has quit IRC17:55
thrashslagle: want me to tackle that?17:57
slaglethrash: sure :)17:58
thrashslagle: word. :P17:59
*** abehl has quit IRC18:02
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates: WIP prototyping fully composable/custom roles  https://review.openstack.org/31567918:03
*** Jabadia has joined #tripleo18:05
*** akuznetsov has joined #tripleo18:07
*** akuznetsov has quit IRC18:11
*** akuznetsov has joined #tripleo18:12
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates: WIP prototyping fully composable/custom roles  https://review.openstack.org/31567918:13
*** hjensas has joined #tripleo18:15
*** hjensas has quit IRC18:15
*** hjensas has joined #tripleo18:15
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates: Remove depends_on: Networks from ControllerServiceChain  https://review.openstack.org/33615018:16
*** noslzzp has joined #tripleo18:16
*** akuznetsov has quit IRC18:16
openstackgerritJohn Trowbridge proposed openstack/tripleo-quickstart: Add a config file for single node HA with ceph  https://review.openstack.org/33615318:21
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates: Add Sahara services to ControllerServices list  https://review.openstack.org/33611918:22
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates: Add Sahara services to ControllerServices list  https://review.openstack.org/33611918:23
*** amoralej is now known as amoralej|off18:24
*** thrash is now known as thrash|biab18:38
thrash|biabshardy: have this too: https://review.openstack.org/#/c/326012/18:39
thrash|biabshardy: Didn't realize I missed it in overcloud.yaml too... :(18:39
thrash|biabI will abandon my change.18:40
shardythrash|biab: ah, missed that - feel free to squash my patch into yours18:40
shardyI'm happy to abandon mine, but if you can reference the bug that'd be good18:40
thrash|biabshardy: nah. Yours does the job. :)18:40
ayoungshardy, bnemec check me on this:  with the TLS stuff, since it is a heat template, we have to clone the whole directory.  This is not a desirable end state. Instead, iff a user wants to do TLS, they should provide an env file that has the data that the template would use.  Something that has only the cert (and key...yuck) in it, and a link to say "and activate the TLS template".  Right?18:40
shardykk, thanks18:40
shardyayoung: right, that's exactly how the environment interface works, there's no need to copy the entire tree unless you're a developer wanting to hack on things18:41
* shardy gtg18:41
*** shardy has quit IRC18:41
*** jeckersb is now known as jeckersb_gone18:49
*** fpan has joined #tripleo18:51
*** hjensas has quit IRC18:58
ayoungbnemec, OK...I just did a test.  In the env file passed to openstack overcloud deploy I added   CLOUDNAME: overcloud.ayoung-dell-t1700.test .  sure enough, it shows up in the overcloudrc file...but nowhere else that I can see19:01
ayounghmmm19:01
ayoungno, in that file it is19:01
ayoungexport OS_CLOUDNAME=overcloud19:01
ayoungI bet that is from an earlier run19:01
bnemecayoung: It's CloudName.  http://docs.openstack.org/developer/tripleo-docs/advanced_deployment/ssl.html#dns-endpoint-configuration has an example.19:02
ayoungbnemec, ah...19:02
ayoungbnemec, and if I set that, I should see it in all the endpoints?  Should I teardown the overcloud and redeploy, or is just adding that enoough?19:03
ayoungtearing down....19:05
ayoungnuke it from orbit.  its the only way19:05
bnemecYeah, I think you would need to redeploy.19:07
*** noslzzp has quit IRC19:19
*** Jabadia has quit IRC19:24
openstackgerritBen Nemec proposed openstack-infra/tripleo-ci: Don't try to get stack details if the stack doesn't exist  https://review.openstack.org/31933719:27
openstackgerritBen Nemec proposed openstack-infra/tripleo-ci: TEST: Delete the overcloud when finished  https://review.openstack.org/29732819:27
openstackgerritBen Nemec proposed openstack-infra/tripleo-ci: Stop piping yes to Heat  https://review.openstack.org/31773019:27
openstackgerritBen Nemec proposed openstack-infra/tripleo-ci: Test overcloud deletion in periodic job  https://review.openstack.org/29676519:27
*** panda has quit IRC19:35
*** panda has joined #tripleo19:35
*** jeckersb_gone is now known as jeckersb19:37
openstackgerritMerged openstack/tripleo-specs: [Trivial] Remove unnecessary executable privilege  https://review.openstack.org/31101819:39
*** hjensas has joined #tripleo19:49
jidarwooo! boom it works19:49
jidartrown: you roooooock!19:50
trownjidar: lol, not my hack, but thanks19:50
trownradez to the rescue19:50
jidarradez: you rock!!19:50
jidarit still does the git and iscsi upgrade but that doesn't really change anything I care about19:50
trownradez: context is the opnfv hack to not install packages during undercloud install19:50
jidarradez: yea, I needed to mimic my prod env in a lab and somebody boned up my old lab so building from scratch I needed to hack out the upgrade's performed by `openstack undercloud install`19:51
jidarmy yum install line is 1000 packages long :/19:52
openstackgerritBen Nemec proposed openstack/diskimage-builder: Generalize logic for skipping final image generation  https://review.openstack.org/33404220:01
*** fragatina has joined #tripleo20:03
*** mgunjan has joined #tripleo20:05
mgunjanhi20:05
*** jprovazn has quit IRC20:06
mgunjani am working on installing rdo liberty using tripleo but my undercloud installation is failing20:06
mgunjani would like to know if someone has recently installed liberty using tripleo on baremetal20:06
mgunjanplease help. i have gone through the tripleo guide and am using default values during installation and still undercloud installation fails20:07
mgunjanError: /Stage[main]/Neutron::Keystone::Auth/Keystone::Resource::Service_identity[neutron]/Keystone_user[neutron]: Could not evaluate: Execution of '/bin/openstack domain show --format shell Default' returned 1: Could not find resource Default Error: /Stage[main]/Heat::Keystone::Auth/Keystone::Resource::Service_identity[heat]/Keystone_user[heat]: Could not evaluate: Execution of '/bin/openstack domain show --format shell Default' ret20:09
mgunjanthis is the error which shows up at the time of the install20:09
*** noslzzp has joined #tripleo20:10
*** thrash|biab is now known as thrash20:12
openstackgerritJohn Trowbridge proposed openstack/tripleo-quickstart: Add a config file for single node HA with ceph  https://review.openstack.org/33615320:14
*** jayg is now known as jayg|g0n320:21
*** noslzzp has quit IRC20:22
*** noslzzp has joined #tripleo20:22
*** noslzzp has quit IRC20:29
*** zxcv has joined #tripleo20:32
*** zxcv has quit IRC20:33
*** bootsha has joined #tripleo20:39
*** rcernin has joined #tripleo20:41
*** bootsha has quit IRC20:45
*** agu_ has joined #tripleo20:49
*** agu_ has quit IRC20:50
*** fultonj has quit IRC20:58
*** xinwu has joined #tripleo20:59
*** bootsha has joined #tripleo21:01
*** lblanchard1 has quit IRC21:05
*** mburned is now known as mburned_out21:08
*** trozet has quit IRC21:11
*** akshai has quit IRC21:16
*** trown is now known as trown|outtypewww21:23
*** trozet has joined #tripleo21:26
*** lucas-afk has quit IRC21:28
*** lucasagomes has joined #tripleo21:28
*** kbyrne has quit IRC21:29
*** kbyrne has joined #tripleo21:31
*** ayoung has quit IRC21:34
*** yamahata has quit IRC21:34
*** jefrite has quit IRC21:34
*** jefrite_ has joined #tripleo21:36
openstackgerritJeff Peeler proposed openstack/tripleo-common: Add List Roles Action  https://review.openstack.org/33028321:40
*** HenryG_ has joined #tripleo21:41
*** hjensas has quit IRC21:42
*** jroll|dupe has joined #tripleo21:44
*** jroll|dupe has quit IRC21:44
*** jroll|dupe has joined #tripleo21:44
*** rwsu has quit IRC21:45
*** mordred has quit IRC21:45
*** HenryG has quit IRC21:45
*** jeckersb has quit IRC21:45
*** jroll has quit IRC21:45
*** ianw has quit IRC21:45
*** trown|outtypewww has quit IRC21:45
*** hogepodge has quit IRC21:45
*** jroll|dupe is now known as jroll21:45
*** yamahata has joined #tripleo21:47
*** fragatina has quit IRC21:47
*** xinwu has quit IRC21:49
*** jeckersb has joined #tripleo21:49
*** HenryG_ is now known as HenryG21:52
openstackgerritJeff Peeler proposed openstack/tripleo-common: Add List Roles Action  https://review.openstack.org/33028321:57
*** bootsha has quit IRC22:05
*** julim has joined #tripleo22:16
*** noslzzp has joined #tripleo22:17
*** rhallisey has quit IRC22:17
*** mgunjan has quit IRC22:19
openstackgerritKen'ichi Ohmichi proposed openstack-infra/tripleo-ci: Use tempest.lib tenants_client  https://review.openstack.org/33630422:21
*** derekh has joined #tripleo22:22
*** mordred has joined #tripleo22:23
derekhAnybody against merging the tripleo-ci common code refactor? https://review.openstack.org/#/c/335622/522:24
derekhall passed except upgrades job22:24
derekhwhich passed in the OVB support patch https://review.openstack.org/#/c/334787/4322:25
*** bootsha has joined #tripleo22:25
*** bootsha has quit IRC22:27
bnemecderekh: +1.  It looks like it got almost all the way through the upgrades job and then died in the restart step of the deploy, which seems unlikely to be related to the CI change.22:28
derekhbnemec: yup22:28
derekhbnemec: ok, merging22:28
openstackgerritMerged openstack-infra/tripleo-ci: Refactor common code for tripleo-ci  https://review.openstack.org/33562222:28
bnemecderekh: Means we can merge OVB too, right?22:28
openstackgerritMerged openstack-infra/tripleo-ci: Add support for OVB based CI  https://review.openstack.org/33478722:29
derekhbnemec: yup, I just did, we should see it passing soon22:29
bnemecNice22:29
derekhbnemec: I've been testing the script to use quintupleo , looks like it should be ok, just one thing I want wondering about22:31
derekhbnemec: If I stub out the undercloud creation, it still creates the public net "public_net: public"22:31
derekhbnemec: do you think we could stub that out also?22:31
derekhbnemec: I'll push a WIP patch up in a minute to show what I've been doing22:34
bnemecderekh: Probably.  It's kind of hard-coded because it was how I was testing net-iso before we had the configurable extra networks.22:34
openstackgerritDan Sneddon proposed openstack/os-net-config: Add support for Infiniband interfaces  https://review.openstack.org/30503922:34
bnemecBut I suspect we could make that a resource and noop it.22:34
*** pradk has quit IRC22:34
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: First iteration of libvirt and nova-compute as a composable services  https://review.openstack.org/32968622:34
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: compute: deploy composable neutron service  https://review.openstack.org/33123422:35
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: compute: moving out iscsi bits, now in puppet-tripleo  https://review.openstack.org/33124722:35
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: [WIP] Update to use quintupleo  https://review.openstack.org/33630922:35
derekhbnemec: ^ thats what I thinking of, now longer need to externally create network stuff, just a quintupleo stack then attach an interface22:37
derekhbnemec: and update the PR https://github.com/cybertron/openstack-virtual-baremetal/pull/24/files22:38
derekh*updated22:38
bnemecderekh: Cool, merged.  Good catch.22:39
derekhbnemec: thanks22:40
*** trozet has quit IRC22:41
openstackgerritBrad P. Crochet proposed openstack-infra/tripleo-ci: Use tripleo-build-images for CI  https://review.openstack.org/33631222:45
derekhbnemec: I forgot to say, I'll take a look at making public_net option in the morning22:46
*** abehl has joined #tripleo22:47
bnemecderekh: Cool, sounds good.22:48
bnemecderekh: One thought about that though.22:48
derekhbnemec: just saw your comments on my patch, iirc I haven't seen the delete fail much and it was at least 3 weeks ago,22:49
bnemecUnless we want to have separate testenvs for net-iso and non-net-iso jobs, we're going to need to deploy all the networks anyway.22:49
derekhbnemec: ok22:50
bnemecderekh: Okay, that's good to hear.  I've occasionally had issues with the stack, but I'm running devstack Heat so it's probably more prone to bugs than stable/mitaka.22:50
derekhbnemec: Also I have a heat bm stack that I can't seem to get ride of, but I reckon thats a heat bug, its seems to be trying to delete a VM that doesn't exist22:52
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Composable Timezone service - Controller  https://review.openstack.org/33152822:53
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Composable Timezone service - Compute  https://review.openstack.org/33153022:54
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Composable Timezone service - CephStorage  https://review.openstack.org/33153422:54
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Composable Timezone service - ObjectStorage  https://review.openstack.org/33153622:54
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Wire NTP and Timezone services into BlockStorage nodes  https://review.openstack.org/33274022:54
EmilienMpradk: ceilo still failing, I'll investigate that22:54
bnemecderekh: Hmm, not sure I've seen that one. :-/22:54
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Remove NFS bits from overcloud_compute manifest  https://review.openstack.org/33295522:55
openstackgerritDan Sneddon proposed openstack/os-net-config: Make os-net-config idempotent with Linux bonds and MAC mapping  https://review.openstack.org/33631323:02
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci: Add gate-tripleo-ci-centos-7-ovb-ha in list of jobs  https://review.openstack.org/33631423:03
EmilienMslagle bnemec derekh ^23:03
EmilienMdsneddon: hey I haven't looked your issue yet23:03
EmilienMdo you still have it?23:03
derekhEmilienM: hmm, looks like I  did the wrong file earlier https://review.openstack.org/#/c/335916/1/scripts/tripleo-jobs.py23:04
EmilienMderekh: feel free to abandon mine, I didn't know you did it before23:05
EmilienMsorry23:05
derekhEmilienM: merging, ci doesn't touch it, I'll submit a seperate one to rmove the containers job23:05
EmilienMok23:06
openstackgerritMerged openstack-infra/tripleo-ci: Add gate-tripleo-ci-centos-7-ovb-ha in list of jobs  https://review.openstack.org/33631423:06
EmilienMpradk: I figure ceilometer profile now set mysql by default, it's wrong, I'll fix it tonight23:06
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Remove the containers job from the ci status page  https://review.openstack.org/33591623:09
*** ayoung has joined #tripleo23:09
derekhbnemec: ovb/rh2 doing its thing http://chunk.io/f/6a015d3224b54433928363d9db19169423:10
*** noslzzp has quit IRC23:11
bnemecderekh: Nice!23:12
*** dsariel has joined #tripleo23:12
*** apetrich has quit IRC23:16
*** apetrich has joined #tripleo23:19
*** adarazs has quit IRC23:19
*** adarazs has joined #tripleo23:20
*** saneax_AFK is now known as saneax23:22
*** panda has quit IRC23:34
*** panda has joined #tripleo23:35
*** yamahata has quit IRC23:36
*** noslzzp has joined #tripleo23:37
*** abehl has quit IRC23:40
*** derekh has quit IRC23:47
*** trozet has joined #tripleo23:49
*** apetrich has quit IRC23:53
*** apetrich has joined #tripleo23:55
*** hogepodge has joined #tripleo23:59

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