Thursday, 2017-01-19

« Wednesday, 2017-01-18 Index Friday, 2017-01-20 »
openstackgerritOliver Walsh proposed openstack/puppet-nova: WIP: Fix cell_v2 setup  https://review.openstack.org/42224800:05
*** acormier has joined #puppet-openstack00:11
*** owalsh is now known as owalsh-afk00:11
*** xarses_ has quit IRC00:14
*** acormier has quit IRC00:15
*** gfidente has quit IRC00:46
openstackgerritAlex Schultz proposed openstack/puppet-nova: Implement a proper cell_v2 setup  https://review.openstack.org/42224800:52
mwhahahaowalsh-afk: -^00:52
owalsh-afkmwhahaha: cool..00:58
owalsh-afkmwhahaha: concern I have with the cell provider is that we really should require a transport_url and database_connection and grep for both to check if a cell exists01:04
mwhahahaowalsh-afk: we can work on improving it but that IMO that's the break out of peices for this process so they can reuse it01:06
mwhahahaowalsh-afk: if we change the nova::cell_v2::cell to a provider we can do what your proposing01:07
mwhahahaVery simply01:07
owalsh-afkmwhahaha: also ensure: absent won't work I think01:07
owalsh-afkno delete (what that implies is being dicussed I believe) and it would require the cell uuid I expect01:08
mwhahahaSo I just wanted to get the structure right so we can swap out the prices as necessary01:08
owalsh-afkcool, I'd comment to keep track of what's needed but gerrit says 503... it's a sign I think. I'll take a closer look tomorrow01:09
mwhahahaWith a provider we can get the uuid no problem. To me the cells resource is wip until I can get an env to work with01:09
*** ducttape_ has joined #puppet-openstack01:15
owalsh-afkcool, gerrit is back... I've just brain-dumped before I forget the details. Thanks mwhahaha... I really gotta go now01:25
EmilienMback01:29
EmilienMand alive01:29
EmilienMhttps://review.openstack.org/#/c/422248/ omg01:29
*** saneax is now known as saneax-_-|AFK01:33
*** sofie has joined #puppet-openstack01:49
*** ducttape_ has quit IRC01:51
*** markvoelker has joined #puppet-openstack01:59
*** ducttape_ has joined #puppet-openstack02:09
*** fc___ has joined #puppet-openstack02:33
*** ducttape_ has quit IRC02:34
*** ducttape_ has joined #puppet-openstack02:35
*** partner_ has joined #puppet-openstack02:36
*** jayg|g0n` has joined #puppet-openstack02:41
*** mtreinish_ has joined #puppet-openstack02:41
*** tlbr_ has joined #puppet-openstack02:41
*** alee has quit IRC02:41
*** fc__ has quit IRC02:41
*** jayg|g0n3 has quit IRC02:41
*** saneax-_-|AFK has quit IRC02:41
*** mjb has quit IRC02:41
*** partner has quit IRC02:41
*** tlbr has quit IRC02:41
*** mtreinish has quit IRC02:41
*** mtreinish_ is now known as mtreinish02:41
*** fc___ is now known as fc__02:41
*** mjb has joined #puppet-openstack02:42
*** ilbot3 has quit IRC02:46
*** agrebennikov_ has joined #puppet-openstack02:47
*** ilbot3 has joined #puppet-openstack02:48
*** alee has joined #puppet-openstack02:50
*** ducttape_ has quit IRC02:53
*** markvoelker has quit IRC02:56
*** markvoelker has joined #puppet-openstack03:10
*** saneax-_-|AFK has joined #puppet-openstack03:10
*** ducttape_ has joined #puppet-openstack03:32
*** ducttape_ has quit IRC03:33
*** agrebennikov_ has quit IRC04:28
*** agrebennikov_ has joined #puppet-openstack04:33
*** nkinder has joined #puppet-openstack04:42
*** furlongm_ has quit IRC04:44
*** furlongm_ has joined #puppet-openstack04:44
*** nkinder has quit IRC05:02
*** saneax-_-|AFK is now known as saneax05:24
*** agrebennikov_ has quit IRC05:31
*** udesale has joined #puppet-openstack05:31
*** skramaja has joined #puppet-openstack05:58
*** dgurtner has joined #puppet-openstack06:03
*** dgurtner has quit IRC06:12
openstackgerritJohn Fulton proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals  https://review.openstack.org/42099206:14
*** agurenko has joined #puppet-openstack06:16
*** acormier has joined #puppet-openstack06:20
*** xarses has joined #puppet-openstack06:22
*** dgurtner has joined #puppet-openstack06:30
*** dgurtner has quit IRC06:30
*** dgurtner has joined #puppet-openstack06:30
*** richm has quit IRC06:42
*** acormier has quit IRC06:44
*** acormier has joined #puppet-openstack06:45
*** dgurtner has quit IRC06:45
*** acormier_ has joined #puppet-openstack07:00
*** acormier_ has quit IRC07:01
*** acormier_ has joined #puppet-openstack07:02
*** acormier has quit IRC07:03
*** acormier_ has quit IRC07:06
*** dgurtner has joined #puppet-openstack07:08
*** dgurtner has quit IRC07:08
*** dgurtner has joined #puppet-openstack07:08
*** owalsh-afk has quit IRC07:15
*** rajinir has quit IRC07:16
*** hjensas has joined #puppet-openstack07:20
*** hjensas has joined #puppet-openstack07:20
*** rasca has joined #puppet-openstack07:58
*** yprokule has joined #puppet-openstack08:01
*** tesseract has joined #puppet-openstack08:13
*** amoralej|off is now known as amoralej08:17
*** owalsh-afk has joined #puppet-openstack08:22
*** jpena|off is now known as jpena08:22
*** ccamacho has joined #puppet-openstack08:23
*** openstackgerrit has quit IRC08:33
*** jpena is now known as jpena|off08:33
*** owalsh-afk is now known as owalsh08:33
*** jpena|off is now known as jpena08:40
*** owalsh has quit IRC08:41
*** gfidente has joined #puppet-openstack08:42
*** gfidente has quit IRC08:42
*** gfidente has joined #puppet-openstack08:42
*** owalsh has joined #puppet-openstack08:42
*** jpich has joined #puppet-openstack08:46
*** openstackgerrit has joined #puppet-openstack08:56
openstackgerritArnaud Morin proposed openstack/puppet-keystone: Make email parameter optional in roles::admin  https://review.openstack.org/42247408:56
*** shardy_afk is now known as shardy09:03
*** rmart04 has joined #puppet-openstack09:08
*** owalsh has quit IRC09:11
*** owalsh has joined #puppet-openstack09:12
*** owalsh has quit IRC09:12
*** owalsh has joined #puppet-openstack09:12
*** ccamacho has quit IRC09:23
*** lucas-afk is now known as lucasagomes09:35
*** ccamacho has joined #puppet-openstack09:40
openstackgerritMykyta Karpin proposed openstack/puppet-keystone: Add default_transport_url option  https://review.openstack.org/42250909:45
*** salmankhan has joined #puppet-openstack09:46
*** derekh has joined #puppet-openstack09:48
*** owalsh_ has joined #puppet-openstack09:49
*** owalsh has quit IRC09:50
*** panda|zZ is now known as panda09:51
*** paramite has joined #puppet-openstack09:57
*** nmathew has joined #puppet-openstack10:00
*** social has joined #puppet-openstack10:01
*** owalsh_ is now known as owalsh10:06
*** owalsh has quit IRC10:10
*** salmankhan has quit IRC10:24
*** salmankhan has joined #puppet-openstack10:30
openstackgerritMykyta Karpin proposed openstack/puppet-glance: Add default_transport_url option  https://review.openstack.org/42254510:43
*** dgurtner has quit IRC10:43
*** hjensas has quit IRC10:47
*** nmathew has quit IRC10:52
*** udesale has quit IRC10:58
*** dtantsur|afk is now known as dtantsur11:05
*** richm has joined #puppet-openstack11:11
*** owalsh has joined #puppet-openstack11:28
*** salmankhan1 has joined #puppet-openstack11:42
*** dgurtner has joined #puppet-openstack11:45
*** dgurtner has quit IRC11:45
*** dgurtner has joined #puppet-openstack11:45
*** salmankhan has quit IRC11:45
*** dgurtner has quit IRC11:46
*** salmankhan has joined #puppet-openstack11:47
*** salmankhan1 has quit IRC11:47
*** salmankhan has quit IRC11:51
*** salmankhan has joined #puppet-openstack11:57
*** jpena is now known as jpena|lunch12:00
*** hjensas has joined #puppet-openstack12:02
*** dgurtner has joined #puppet-openstack12:02
*** dgurtner has quit IRC12:02
*** dgurtner has joined #puppet-openstack12:02
EmilienMhello12:02
*** dgurtner has quit IRC12:12
*** dgurtner has joined #puppet-openstack12:14
*** dgurtner has quit IRC12:14
*** dgurtner has joined #puppet-openstack12:14
openstackgerritSteven Hardy proposed openstack/puppet-openstack-integration: Temporarily remove *_quotas from swift pipeline  https://review.openstack.org/42208412:25
*** lucasagomes is now known as lucas-hungry12:27
*** owalsh is now known as owalsh-lunch12:29
openstackgerritHarald Jensås proposed openstack/puppet-keystone: Federation mellon - spec test wrong filename + typos  https://review.openstack.org/42117312:30
*** shardy is now known as shardy_lunch12:34
iurygregorygood morning12:38
*** dtantsur is now known as dtantsur|lunch12:47
*** saneax is now known as saneax-_-|AFK12:49
*** jpena|lunch is now known as jpena12:55
*** hjensas has quit IRC12:58
mwhahahafyi, ceilometer fix passed CI https://review.openstack.org/#/c/421561/13:02
*** morazi has joined #puppet-openstack13:02
*** jayg|g0n` is now known as jayg13:08
*** morazi has quit IRC13:10
openstackgerritPradeep Kilambi proposed openstack/puppet-ceilometer: Configure compute params in config  https://review.openstack.org/42090813:12
*** ansmith has joined #puppet-openstack13:13
*** lucas-hungry is now known as lucasagomes13:17
*** dtantsur|lunch is now known as dtantsur13:17
*** shardy_lunch is now known as shardy13:20
*** hjensas has joined #puppet-openstack13:21
*** hjensas has quit IRC13:21
*** hjensas has joined #puppet-openstack13:21
*** trown|outtypewww is now known as trown13:23
*** morazi has joined #puppet-openstack13:24
*** sofie has quit IRC13:28
*** nmathew has joined #puppet-openstack13:29
*** agrebennikov_ has joined #puppet-openstack13:31
*** ccamacho is now known as ccamacho|lunch13:35
*** salmankhan has quit IRC13:40
*** acabot has quit IRC13:43
*** acormier has joined #puppet-openstack13:44
*** pradk has joined #puppet-openstack13:44
*** nmathew has quit IRC13:46
*** agrebennikov_ has quit IRC13:47
*** saneax-_-|AFK is now known as saneax13:47
*** owalsh-lunch is now known as owalsh13:48
*** acormier has quit IRC13:51
EmilienMshardy, mwhahaha: I have some thoughts about https://review.openstack.org/#/c/42134113:56
EmilienMI think we should introspect the pipeline parameter and detect the _ and transform in -13:56
EmilienMso it's transparent for our users13:56
*** markvoelker has quit IRC13:57
mwhahahathat could work13:57
EmilienMand we need a release note to communicate on the fact they need to update the pipeline13:57
shardyYeah that sounds reasonable13:57
EmilienMbecause it's clear that https://review.openstack.org/#/c/421341 is not backward compatible13:57
EmilienMI mean, yes we could fix our CI to make it pass13:57
EmilienMbut our users will break their deployments for sure13:57
EmilienMshardy: I can work on a patch but it will be later today, I have urgent tasks to finish before. But I can do it before tomorrow morning for sure.13:58
shardyEmilienM: Ok, if you have time that would be great, if not I can probably do it13:58
EmilienMI'll do it today13:58
shardyOk, sounds good, thanks!13:59
*** markvoelker has joined #puppet-openstack14:01
*** udesale has joined #puppet-openstack14:01
*** dgurtner has quit IRC14:04
*** dprince has joined #puppet-openstack14:05
*** agrebennikov_ has joined #puppet-openstack14:14
*** mkoderer has joined #puppet-openstack14:14
openstackgerritPavel Gluschak (scsnow) proposed openstack/puppet-cinder: Added support of vstorage cinder backend  https://review.openstack.org/41901214:15
amoralejEmilienM, mwhahaha, https://review.openstack.org/#/c/421196/ if you have a chance14:23
EmilienMok14:23
*** udesale has quit IRC14:27
*** salmankhan has joined #puppet-openstack14:30
openstackgerritMerged openstack/puppet-ceilometer: Fix Ubuntu Ocata issues  https://review.openstack.org/42156114:37
aleeEmilienM, mwhahaha - can I get a +W on https://review.openstack.org/#/c/422200/ please ?14:41
*** saneax is now known as saneax-_-|AFK14:46
iurygregoryalee, waitting for TripleO recheck since it has failures14:47
aleeiurygregory, ah gotcha thanks14:47
EmilienMshardy: http://docs.openstack.org/developer/swift/middleware.html14:48
EmilienMshardy: why do we have _ in upstream doc?14:48
shardyhttps://github.com/openstack/swift/blob/master/etc/proxy-server.conf-sample#L73414:50
shardyEmilienM: I don't know, that is also inconsistent with the sample conf14:50
shardyso I guess that needs to be fixed as well14:50
EmilienMsigh14:50
EmilienMshardy: what is the real name? _ or - ?14:51
EmilienM- I guess14:51
shardyEmilienM: it doesn't matter, they just have to match14:51
EmilienMoh ok14:51
shardybut the pipeline is mandatory for puppet-swift if the hard-coded things don't match the service defaults14:51
EmilienMbut since puppet-swift has container_quotas14:52
EmilienMit means we need to use _14:52
EmilienMin that case, why not updating tripleo?14:52
EmilienMshardy: ^14:53
*** nkinder has joined #puppet-openstack14:53
*** dfisher has quit IRC14:54
*** dfisher has joined #puppet-openstack14:55
EmilienMwhy not doing it in swift-proxy service in THT?14:55
shardyEmilienM: I can, but I was trying to make it consistent with the swift internal defaults14:55
shardyalso, why should operators be forced to modify their configuration on upgrade?14:55
shardyit seems like this was a backwards incompatible change to start with14:56
EmilienMwell I think it's the other way around14:56
EmilienMin puppet-swift, we decided to use "_" like it's documented14:56
EmilienMso users need to define the pipeline with _14:56
EmilienMoperators are not forced to declare this middleware14:57
EmilienMbut if they do, they need to use the _14:57
shardyEmilienM: it's not consistent in the swift docs either14:57
EmilienMbecause that's the way puppet-swift enable it14:57
EmilienMshardy: see https://review.openstack.org/#/c/395859/14:57
EmilienMthe problem is Thiago used - instead of _14:58
*** fultonj has joined #puppet-openstack14:58
EmilienM7 weeks ago, we should just use _ and our problem is solved imho14:58
tdasilvasorry14:58
EmilienMtdasilva: no problem, it's not very intuitive in puppet-swift14:59
*** dprince has quit IRC14:59
tdasilvajust realized the use of - and _ seems very inconsistent even in https://github.com/openstack/swift/blob/master/etc/proxy-server.conf-sample14:59
iurygregoryalee, +A ;) TripleO is fine now14:59
shardyEmilienM: meh, whatever14:59
EmilienMwell14:59
EmilienMif you look:14:59
EmilienMhttps://github.com/openstack/puppet-swift/blob/master/manifests/proxy/container_quotas.pp#L2514:59
aleeiurygregory, thanks!14:59
shardyEmilienM: man proxy-server.conf, and the sample config says container-quotas14:59
iurygregoryalee, yw ;)14:59
shardyso do a bunch of other places in the swift docs14:59
shardygrep the swift source to see15:00
EmilienMI added that in 201315:00
EmilienMhttps://github.com/openstack/puppet-swift/commit/4157cb9a4141c071337f7e7c1946f32dd034294515:00
EmilienMsince then, operators have been using "_" to make it work15:00
EmilienMshardy: I'm afraid renaming _ to - will beak backward compatibility15:00
shardyOk, so a bunch of the swift docs, and the sample config is wrong then15:00
EmilienMtdasilva: can we have some help here?15:00
EmilienMmaybe you know which one we should use15:01
shardyEmilienM: Ok, fine I'll fix t-h-t to match puppet-swift, and ignore the swift inconsistencies for now15:01
EmilienMshardy: I would vote for 1) update THT to use "_" in the meantime" 2) create a release note about it and switch to "-" in Pike15:01
EmilienMshardy: and 3) fix doc/code in Swift to be consistent15:01
shardyEmilienM: Ok, thanks for the feedback15:01
EmilienMshardy can do 1), I can do 2) and tdasilva can do 3)15:01
shardyack, sounds good, I'll abandon my puppet patches for now15:02
EmilienMshardy: actually I'll take over them15:02
EmilienMshardy: you can abandon the poi patches15:02
EmilienMshardy: but not the puppet-swift . I'll take over and add deprecated warning & release notes15:03
shardyEmilienM: ack, sounds good, thanks!15:03
tdasilvaEmilienM: I can work with the swift docs to make it more consistent, but I can't guarantee they will be consistent with they way puppet-swift expects15:03
EmilienMtdasilva: I think what we want here in consistency in Swift, not in puppet-swift15:03
EmilienMtdasilva: puppet-switch will adapt to what Swift says15:03
EmilienMbut make it loud and clear before15:03
EmilienMbecause right now it's confusing15:04
tdasilvaEmilienM: but then it might break backwards compatibility with previous puppet-swift installs, no?15:04
EmilienMyes15:04
EmilienMbut I'm working on it15:04
tdasilvaoh ok15:04
EmilienMdon't take care of that15:04
tdasilvasounds good15:04
tdasilva:)15:04
*** udesale has joined #puppet-openstack15:05
*** xarses has quit IRC15:08
*** acormier has joined #puppet-openstack15:21
*** markvoelker_ has joined #puppet-openstack15:28
*** markvoelker has quit IRC15:29
*** markvoelker has joined #puppet-openstack15:31
*** jaosorior has joined #puppet-openstack15:31
*** markvoelker_ has quit IRC15:33
*** rajinir has joined #puppet-openstack15:35
*** ccamacho|lunch is now known as ccamacho15:41
openstackgerritRicardo Noriega proposed openstack/puppet-neutron: Add support for BGPVPN plugin  https://review.openstack.org/42205115:45
*** dprince has joined #puppet-openstack15:47
openstackgerritMerged openstack/puppet-nova: Set correct ownership for nova keytab  https://review.openstack.org/42220015:53
openstackgerritMerged openstack/puppet-vswitch: Add fail_mode parameter to OVS Ports  https://review.openstack.org/42119615:58
*** xarses has joined #puppet-openstack16:00
*** shardy has quit IRC16:00
*** xarses_ has joined #puppet-openstack16:00
*** shardy has joined #puppet-openstack16:01
*** xarses has quit IRC16:04
*** udesale has quit IRC16:11
*** ducttape_ has joined #puppet-openstack16:11
*** agurenko has quit IRC16:13
*** panda is now known as panda|bbl16:28
*** fc__ has left #puppet-openstack16:29
*** rmart04 has quit IRC16:31
vinshhttps://github.com/openstack/puppet-openstack-integration/blob/master/manifests/swift.pp#L43 also related..16:33
vinshhttps://github.com/openstack/puppet-swift/blob/master/manifests/proxy.pp#L25516:35
vinshEmilienM: how is https://review.openstack.org/#/c/421341/2 not backward compatible?16:39
mwhahahavinsh: for people who have the _ it fails16:39
vinshIt really doesn't matter what is set using - or _ in https://review.openstack.org/#/c/421341/2/tests/site.pp16:39
mwhahahait does16:39
vinshproxy.pp will convert it to _16:40
mwhahahaso the pipeline (which the operator sets) has to match the config16:40
mwhahahanot in any of the links you posted :D16:40
vinshSo maybe in proxy.pp we handle account/container_quota as a special case16:40
mwhahahaso that was the proposed fix, was convert them as necessary16:40
mwhahahas/container_quota/container-quota for the pipeline and update the modules16:41
mwhahahaso if the operator provides the old style in the pipeline arg, it gets fixed16:41
vinshOk, just keeping myself up to speed :)16:41
vinshvery good16:41
vinshthe swift example conf files truly are a mish mash as you guys said.16:42
vinshIn other puppet-swift news.. next week is de-templitize object/account/container. move them to ini configs.16:43
vinshalso, add the support for swift crypto settings. is on my list.16:44
*** ccamacho has quit IRC16:48
openstackgerritBrent Eagles proposed openstack/puppet-neutron: Remove logging_facility, allow_pagination, allow_sorting  https://review.openstack.org/42275516:48
dmsimardEmilienM, mwhahaha: are you familiar with that tempest failure in newton ? ceilometer.tests.tempest.api.test_telemetry_notification_api.TelemetryNotificationAPITest.test_check_glance_v1_notifications[id-04b10bfe-a5dc-47af-b22f-0460426bf499,image,smoke]16:50
dmsimardit rings me a bell but I forget why16:50
mwhahahaWe disabled iy16:51
mwhahahaCause it's broken on Ubuntu16:51
openstackgerritHarald Jensås proposed openstack/puppet-keystone: Federation openidc - spec test wrong filename + typos  https://review.openstack.org/42117916:51
mwhahahaI think EmilienM might have also disabled it on centos16:51
mwhahahaGo look in the exclude regex in poi16:52
* mwhahaha is on mobile at the moment16:52
dmsimardmwhahaha: it's excluded in master but not in newton16:52
mwhahahaSomething might have gotten backported16:53
EmilienMI excluded it in master I think because we try to disable ceilometer tests that interact with glance api v116:53
EmilienMwe removed glance api v1 in poi16:54
*** yprokule has quit IRC16:55
EmilienMdmsimard: ^16:56
*** jistr is now known as jistr|afk16:57
mwhahahait's disabled due to packaging things for ubuntu17:02
mwhahahaprobably need to revisit those excludes now that m2 is published17:02
mwhahahait's not excluded for centos17:02
EmilienMah right17:03
*** beagles is now known as beagles-brb17:03
mwhahahaof course need to fix all the ubuntu issues first17:04
mwhahahawhich reminds me to get back to those today17:04
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Update ubuntu excludes for ocata-m2  https://review.openstack.org/42276617:05
*** ccamacho has joined #puppet-openstack17:06
*** aarefiev is now known as aarefiev_afk17:14
*** shardy has quit IRC17:29
openstackgerritRajini Ram proposed openstack/puppet-cinder: Support for ScaleIO Cinder backend  https://review.openstack.org/42277817:33
*** mkoderer has quit IRC17:36
openstackgerritPradeep Kilambi proposed openstack/puppet-ceilometer: Configure compute params in config  https://review.openstack.org/42090817:37
openstackgerritAlex Schultz proposed openstack/puppet-aodh: Fix Ubuntu Ocata-m2 issues  https://review.openstack.org/42156417:43
*** beagles-brb is now known as beagles17:50
*** owalsh has quit IRC17:59
*** _ducttape_ has joined #puppet-openstack18:00
*** jistr|afk is now known as jistr18:01
*** derekh has quit IRC18:03
*** ducttape_ has quit IRC18:03
*** hjensas has quit IRC18:11
*** gfidente has quit IRC18:12
*** jpena is now known as jpena|off18:13
*** lucasagomes is now known as lucas-afk18:15
*** trown is now known as trown|lunch18:16
*** derekh has joined #puppet-openstack18:20
*** dtantsur is now known as dtantsur|afk18:21
*** jpich has quit IRC18:22
*** rasca has quit IRC18:22
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Update ubuntu excludes for ocata-m2  https://review.openstack.org/42276618:23
*** _ducttape_ has quit IRC18:28
*** ducttape_ has joined #puppet-openstack18:28
openstackgerritPradeep Kilambi proposed openstack/puppet-ceilometer: Configure compute params in config  https://review.openstack.org/42090818:33
*** paramite has quit IRC18:34
*** derekh has quit IRC18:35
mwhahahaEmilienM: what's the entry point into the beaker tests in CI? is there a shell script that gets run or what18:39
mwhahahai'm looking at adding the mirror hosts18:39
mwhahahablah i see it's in project-config18:40
EmilienMmwhahaha: back from lunch. Have you found?18:43
mwhahahayea i found it18:43
mwhahahaEmilienM: so do you think we should move the beaker stuff to use a script from p-o-i so we can do the mirror handling?18:44
mwhahahaEmilienM: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/puppet-module-jobs.yaml#L75-L8818:45
mwhahahasince we do already checkout p-o-i, should we just move those bits into like the run_beaker_acceptance.sh?18:45
openstackgerritRicardo Noriega proposed openstack/puppet-neutron: Add support for BGPVPN plugin  https://review.openstack.org/42205118:46
EmilienMmwhahaha: I would support it18:46
*** salmankhan has quit IRC18:46
*** derekh has joined #puppet-openstack19:01
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Create beaker test script  https://review.openstack.org/42281819:04
*** trown|lunch is now known as trown19:05
*** furlongm_ has quit IRC19:12
*** furlongm_ has joined #puppet-openstack19:12
EmilienMmwhahaha: we'll need to backport it to stable branches19:15
mwhahahawell i'm seeing if it works first :D19:15
mwhahahabut yea19:15
EmilienMmwhahaha: we need to take care of other puppet moduels not in Puppet OpenStack who have beaker jobs19:16
EmilienMthey don't use poi19:16
mwhahahabut they check it out19:17
* mwhahaha is aware19:17
* mwhahaha is only testing at the moment19:17
EmilienMcool cool19:18
*** cdelatte has joined #puppet-openstack19:18
*** furlongm_ has quit IRC19:26
*** spredzy has quit IRC19:28
*** furlongm_ has joined #puppet-openstack19:29
*** tesseract has quit IRC19:29
*** dprince has quit IRC19:32
*** openstackgerrit has quit IRC19:33
*** nkinder has quit IRC19:39
*** derekh has quit IRC19:41
*** openstackgerrit has joined #puppet-openstack19:41
openstackgerritHarald Jensås proposed openstack/puppet-keystone: Federation shibboleth - spec test wrong filename + typos  https://review.openstack.org/42121819:41
*** hjensas has joined #puppet-openstack19:59
*** hjensas has joined #puppet-openstack19:59
*** salmankhan has joined #puppet-openstack20:07
*** amoralej is now known as amoralej|off20:09
*** salmankhan has quit IRC20:11
*** dprince has joined #puppet-openstack20:17
*** panda|bbl is now known as panda20:24
*** trown is now known as trown|brb20:33
*** trown|brb is now known as trown20:42
openstackgerritRicardo Noriega proposed openstack/puppet-neutron: Add support for BGPVPN plugin  https://review.openstack.org/42205120:48
*** ccamacho has quit IRC20:59
*** _ducttape_ has joined #puppet-openstack20:59
*** ducttape_ has quit IRC21:01
openstackgerritDavid Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals  https://review.openstack.org/42099221:05
*** ccamacho has joined #puppet-openstack21:09
*** _ducttape_ has quit IRC21:14
*** ducttape_ has joined #puppet-openstack21:15
*** acormier has quit IRC21:28
*** dprince has quit IRC21:28
*** dprince has joined #puppet-openstack21:28
*** jayg is now known as jayg|g0n321:29
*** acormier has joined #puppet-openstack21:30
mwhahahaso EmilienM do we want to https://review.openstack.org/#/c/422285/?  Since we haven't released it should we nuke it before it goes out with 9.5.0?21:37
EmilienMmwhahaha: I guess yes21:39
EmilienMmwhahaha: I can approve it21:39
mwhahahak21:39
mwhahahai think it might be best given how cell v2 has gone21:40
mwhahahaiurygregory, EmilienM: https://review.openstack.org/#/c/421564/ has also passed CI21:41
EmilienMapproved21:43
EmilienMsounds like poi still not happy21:43
EmilienMmwhahaha: https://review.openstack.org/#/c/422766/2/run_tests.sh21:43
EmilienMEXCCLUDES21:43
mwhahahayea i thought i fixed that21:43
mwhahahai guess i forgot to push21:44
EmilienMalso http://logs.openstack.org/66/422766/2/check/gate-puppet-openstack-integration-4-scenario001-tempest-ubuntu-xenial-nv/63d514a/console.html#_2017-01-19_18_46_19_69826421:44
EmilienMwe need to pip install gabbi on ubuntu21:44
EmilienMlike I did in the past for centos21:44
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Update ubuntu excludes for ocata-m2  https://review.openstack.org/42276621:44
EmilienMI'm sure they didn't package it21:44
EmilienMlet me check21:44
EmilienMomg they did21:44
EmilienMhttp://packages.ubuntu.com/xenial/python-gabbi21:44
EmilienMzigo did :D21:45
mwhahahayea they pulled alot of the upstream stuff in as part of ocata-m221:45
EmilienMmwhahaha: we need this package, maybe we can deploy it with puppet-tempest? shrug21:45
mwhahahawhich is partially why we have these problems21:45
EmilienMscenario004 fails to spawn a VM21:46
EmilienMhttp://logs.openstack.org/66/422766/2/check/gate-puppet-openstack-integration-4-scenario004-tempest-ubuntu-xenial-nv/cab67d1/logs/nova/nova-compute.txt.gz#_2017-01-19_18_43_47_57021:46
EmilienMhttp://logs.openstack.org/66/422766/2/check/gate-puppet-openstack-integration-4-scenario004-tempest-ubuntu-xenial-nv/cab67d1/logs/glance/glance-api.txt.gz#_2017-01-19_18_49_15_04821:47
EmilienMsounds like in glance21:47
EmilienMprobably with ceph rgw backend21:47
EmilienMscenario003, designate is not idempotent http://logs.openstack.org/66/422766/2/check/gate-puppet-openstack-integration-4-scenario003-tempest-ubuntu-xenial-nv/2a3e210/logs/puppet.txt.gz#_2017-01-19_18_41_2421:48
*** ansmith has quit IRC21:48
mwhahahaprobably missing something, we had that in centos for a while21:48
EmilienMhttp://logs.openstack.org/66/422766/2/check/gate-puppet-openstack-integration-4-scenario003-tempest-ubuntu-xenial-nv/2a3e210/logs/designate/designate-pool-manager.txt.gz#_2017-01-19_18_36_25_57821:49
EmilienMscenario002 fails to spawn a VM, glance fails to find image http://logs.openstack.org/66/422766/2/check/gate-puppet-openstack-integration-4-scenario002-tempest-ubuntu-xenial-nv/157d5a9/logs/glance/glance-api.txt.gz#_2017-01-19_18_50_41_98621:50
EmilienMthey do a terrible work in testing21:50
mwhahahawell to be fair, we run off their testing repo21:50
EmilienMhttp://logs.openstack.org/66/422766/2/check/gate-puppet-openstack-integration-4-scenario002-tempest-ubuntu-xenial-nv/157d5a9/logs/nova/nova-compute.txt.gz#_2017-01-19_18_50_42_29921:50
mwhahahabut yes21:50
EmilienMscenario002 fails on the vif also21:51
EmilienMet voilà21:51
EmilienMso now? do we spend 24h to fix thing?21:51
mwhahahamaybe tomorrow :D21:52
EmilienMmfisch, _ody, anyone core volunteer to help?21:53
*** cdelatte has quit IRC21:58
*** ccamacho has quit IRC22:05
openstackgerritDavid Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals  https://review.openstack.org/42099222:07
*** xarses_ is now known as xarses22:08
*** ccamacho has joined #puppet-openstack22:21
openstackgerritMerged openstack/puppet-nova: Revert "Add cell_v2 simple_cell_setup"  https://review.openstack.org/42228522:23
*** spredzy has joined #puppet-openstack22:25
*** trown is now known as trown|outtypewww22:25
openstackgerritDavid Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals  https://review.openstack.org/42099222:26
*** imcsk8 has quit IRC22:27
openstackgerritMerged openstack/puppet-aodh: Fix Ubuntu Ocata-m2 issues  https://review.openstack.org/42156422:45
*** owalsh has joined #puppet-openstack22:58
openstackgerritDavid Critch proposed openstack/puppet-ceph: Adding support to force zapping of ceph OSDs and journals  https://review.openstack.org/42099223:00
*** ducttape_ has quit IRC23:16
*** acormier has quit IRC23:18
*** ccamacho has quit IRC23:22
owalshmwhahaha: ping, do you have a few minutes to spare?23:25
mwhahahaowalsh: yea, what's up23:25
mwhahahaowalsh: let me pull up that review, it's been one of those days :D23:25
owalsh:-)23:26
mwhahahaowalsh: so i'm ok with not having delete support initially23:27
owalshmwhahaha: I've been thinking this through a bit more.. hear me out...23:29
mwhahahasure23:29
owalshwe use nova::cell_v2::cell must required transport_url & database_connection. This type manages the optional cells.23:30
owalshthe default cell is not optional23:30
mwhahaharight so that's https://review.openstack.org/#/c/422248/3/manifests/cell_v2/simple_setup.pp@5023:31
owalshyes23:31
owalshalso we add an optional name param23:31
mwhahahawell due to the nature of puppet resources we'll always have a name23:32
owalshit will have a title, but transport_url & database_connection tuple is the unique identifier23:32
mwhahahaonly if managed outside of puppet, correct23:33
owalshno. a cell is another cloud basically, the db & transport url is how the api talks to that cloud23:33
owalshname is just for humans23:33
mwhahahayea i get that23:34
owalshI think that's a valid model for the additonal cells for a multi-cell setup23:35
owalshnow, for ocata a single cell setup is required... to do this we run:23:36
owalshapi_db sync (as we need an api db to add the cells to)23:37
owalshmap_cell023:37
owalshcreate_cell (no args, consider this "map_default_cell")23:37
owalshdb sync (which now syncs the default cell)23:38
*** jaosorior has quit IRC23:38
mwhahahaok yea we've got that23:39
mwhahahaso what's the issue you're running into?23:39
owalshpoint being we don't use the new cell type in the simple_setup class here https://review.openstack.org/#/c/422248/3/manifests/cell_v2/simple_setup.pp23:40
owalshthat's *just* for non-default cells23:41
mwhahahaso we have exactly what you described if you don't pass any transport or database connection into nova::cell_v2::simple_setup23:41
mwhahahathe only different is your default cell gets named cell123:41
owalshwe *must* pass transport or database connection - that's the only way to differentiate two cells23:42
mwhahahai get what you're saying, and it's exactly what we have in that code23:42
mwhahahathe point is to reuse the same bit of logic for both the default and future cells23:42
owalshwe are using name23:42
mwhahahaand?23:42
mwhahahaso you're telling me that nova-manage cell_v2 create_cell --name default != nova-manage cell_v2 create_cell23:43
owalshI saying nova-mange cell_v2 create_cell should be called nova-manage cell_v2 map_default_cell23:44
owalshideally23:44
mwhahahawhat the hell is map_default_cell23:44
owalshthat's what we are doing, mapping the default cell to the nova db23:44
*** panda is now known as panda|Zz23:44
mwhahahaand why is there yet another function to do the same thing23:44
mwhahahaso i must be missing something because i just updated my env and all i've got is: {create_cell,discover_hosts,list_cells,map_cell0,map_cell_and_hosts,map_instances,simple_cell_setup,verify_instance}23:45
mwhahahawhat I would like is the initial flow to leverage the same stuff that scaling actions would take23:45
owalshyea, it will work but it's not correct. add another cell type with a different name but the same transport_url/db23:45
mwhahahabut it wouldn't23:46
mwhahahaso it'd be on the operator to define those when adding new cells23:46
owalshyes, so they use the cell type, and tranport/db are mandatory params23:48
openstackgerritPradeep Kilambi proposed openstack/puppet-gnocchi: Remove carbonara package reference  https://review.openstack.org/42291823:48
mwhahahaowalsh: http://paste.openstack.org/show/595709/23:48
mwhahahaso we don't actually have to make them mandatory in code, we could enforce that via some a class or something. but we could just document that -^23:49
*** pradk has quit IRC23:49
mwhahahaso what you're getting stuck on is a hard requirement that isn't really necessary to the entire process23:49
owalshwhy not make them manditory?23:49
owalshthey are manditory23:49
mwhahahaso we can leverage them in multiple ways23:49
owalshwhat ways, that pastebin is the only way23:49
mwhahahaor use name 'default' with some valiadtion23:49
mwhahahaso that pastebin is how it works now with that code i generated23:50
mwhahahaand it does what you're saying23:50
mwhahahathe only thing you're hung up on is making transport/database required for nova::cell_v2::cell23:50
EmilienMhttps://review.openstack.org/422918 is a promotion blocker23:50
mwhahahaand i'm saying that's a nice to have but not necessary to this entire thing23:50
owalshhow can you check if the cell exists otherwise?23:50
mwhahahaowalsh: so i'm not interested in handling cases where cells are created outside of puppet23:51
mwhahahathis is puppet-nova23:51
mwhahahathe expectation is that you do it via puppet23:51
mwhahahanow over in #tripleo we could have that discussion but if we follow the same flow as what in puppet-nova it's moot23:51
mwhahahawe can clearly document in releasenotes/testing that this is the expectations and limitations23:52
mwhahahabecause simple setup is optional, the operator could handle it all externally and we wouldn't care either23:52
owalshwhy, we can model this correctly?23:52
mwhahahayou're over engineering this23:53
mwhahahait is correct, except you know the implementation details of how nova is handling it23:53
mwhahahafrom an operator standpoint the transport_url/databse_connection thing would get caught in a puppet failure23:54
mwhahahabecause the cell create would fail23:54
owalshbut we know that you can't add a cell without this23:54
mwhahahaadd a cell without what?23:54
owalshtransport_url/databse_connection23:54
mwhahahabut you can, it just reads nova.conf23:55
mwhahahalook i'm trying not to duplicate the code23:55
mwhahahanow here, let me propose what you're talking about23:56
mwhahahagimme 5 mins23:56
owalshwhen it reads from nova.conf it's creating the default cell, not adding a cell23:56
owalshit's unfortunate that it's the same command IMO23:56
mwhahahabut it's not, i think you're getting hung up on these details23:58
*** pradk has joined #puppet-openstack23:59
*** dfisher has quit IRC23:59
« Wednesday, 2017-01-18 Index Friday, 2017-01-20 »

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