*** gfidente has quit IRC | 00:02 | |
*** panda has quit IRC | 00:03 | |
*** panda has joined #tripleo | 00:04 | |
*** mcornea has quit IRC | 00:06 | |
*** chlong has quit IRC | 00:09 | |
openstackgerrit | Dan Sneddon proposed openstack/tripleo-heat-templates: Add IPv6 Support to Isolated Networks https://review.openstack.org/235423 | 00:13 |
---|---|---|
*** dmacpher has quit IRC | 00:42 | |
*** olaph has quit IRC | 00:56 | |
*** eil397 has quit IRC | 01:02 | |
*** chlong has joined #tripleo | 01:05 | |
*** yuanying_ has joined #tripleo | 01:09 | |
*** yuanying_ has quit IRC | 01:09 | |
*** yuanying has quit IRC | 01:11 | |
*** chlong has quit IRC | 01:23 | |
*** yuanying has joined #tripleo | 01:23 | |
openstackgerrit | Dan Sneddon proposed openstack/tripleo-heat-templates: Add IPv6 Support to Isolated Networks https://review.openstack.org/235423 | 01:34 |
*** sthillma has quit IRC | 01:41 | |
*** apetrich has quit IRC | 01:47 | |
*** sthillma has joined #tripleo | 01:48 | |
*** apetrich has joined #tripleo | 01:48 | |
*** nijaba has quit IRC | 01:55 | |
*** nijaba has joined #tripleo | 01:56 | |
*** nijaba has quit IRC | 01:56 | |
*** nijaba has joined #tripleo | 01:56 | |
*** rcernin has quit IRC | 02:01 | |
*** panda has quit IRC | 02:01 | |
*** rhallisey has joined #tripleo | 02:09 | |
*** davidlenwell has quit IRC | 02:10 | |
*** dmacpher has joined #tripleo | 02:10 | |
*** davidlenwell has joined #tripleo | 02:13 | |
*** ChanServ sets mode: +v davidlenwell | 02:13 | |
*** rhallisey has quit IRC | 02:13 | |
openstackgerrit | Steve Baker proposed openstack/tripleo-heat-templates: Set the name property for all deployment resources https://review.openstack.org/255644 | 02:29 |
*** alop has quit IRC | 02:29 | |
*** shivrao has quit IRC | 02:32 | |
*** links has joined #tripleo | 02:33 | |
*** cwolferh has quit IRC | 02:42 | |
*** chlong has joined #tripleo | 02:44 | |
*** yuanying has quit IRC | 03:19 | |
*** yuanying has joined #tripleo | 03:37 | |
*** anande has joined #tripleo | 03:58 | |
openstackgerrit | Tzu-Mainn Chen proposed openstack/tripleo-specs: Spec for TripleO overcloud deployment REST API https://review.openstack.org/230432 | 03:59 |
*** yuanying has quit IRC | 04:02 | |
cmyster | trP | 04:05 |
cmyster | sorry | 04:05 |
*** davidlenwell has quit IRC | 04:06 | |
*** yuanying has joined #tripleo | 04:06 | |
*** davidlenwell has joined #tripleo | 04:09 | |
*** ChanServ sets mode: +v davidlenwell | 04:09 | |
*** sthillma has quit IRC | 04:09 | |
*** yamahata has joined #tripleo | 04:10 | |
*** cwolferh has joined #tripleo | 04:47 | |
*** masco has joined #tripleo | 04:56 | |
*** shivrao has joined #tripleo | 05:08 | |
*** shivrao has quit IRC | 05:22 | |
*** sthillma has joined #tripleo | 05:27 | |
*** sthillma_ has joined #tripleo | 05:27 | |
*** sthillma has quit IRC | 05:31 | |
*** sthillma_ is now known as sthillma | 05:31 | |
*** tzumainn has quit IRC | 05:35 | |
*** sthillma has quit IRC | 05:44 | |
*** greghayn1 has joined #tripleo | 05:57 | |
*** greghaynes has quit IRC | 05:58 | |
*** greghayn1 is now known as greghaynes | 05:58 | |
*** greghaynes has quit IRC | 05:59 | |
*** greghaynes has joined #tripleo | 06:00 | |
*** links has quit IRC | 06:01 | |
*** links has joined #tripleo | 06:01 | |
*** greghaynes has quit IRC | 06:05 | |
*** greghaynes has joined #tripleo | 06:06 | |
*** chlong has quit IRC | 06:28 | |
*** jaosorior has joined #tripleo | 06:29 | |
*** links has quit IRC | 06:49 | |
*** chlong has joined #tripleo | 06:49 | |
*** chlong has quit IRC | 07:04 | |
*** links has joined #tripleo | 07:05 | |
*** ukalifon has joined #tripleo | 07:10 | |
*** ukalifon has quit IRC | 07:10 | |
*** rcernin has joined #tripleo | 07:11 | |
*** dmacpher has quit IRC | 07:16 | |
*** rasca has joined #tripleo | 07:18 | |
*** ukalifon has joined #tripleo | 07:19 | |
*** chlong has joined #tripleo | 07:20 | |
*** bvandenh has joined #tripleo | 07:21 | |
*** rasca has quit IRC | 07:24 | |
*** rasca has joined #tripleo | 07:26 | |
*** tserong has joined #tripleo | 07:26 | |
*** tserong has joined #tripleo | 07:26 | |
*** jprovazn has joined #tripleo | 07:43 | |
*** sthillma has joined #tripleo | 07:48 | |
*** tremble has joined #tripleo | 07:56 | |
*** tremble has joined #tripleo | 07:56 | |
*** mkovacik has joined #tripleo | 07:58 | |
*** chlong has quit IRC | 08:05 | |
*** bhunter has quit IRC | 08:05 | |
*** liverpooler has joined #tripleo | 08:07 | |
*** liverpooler has quit IRC | 08:10 | |
jaosorior | tremble: Hey dude, got any idea if apache (used for horizon) is currently managed by pacemaker? | 08:10 |
*** liverpooler has joined #tripleo | 08:10 | |
tremble | jaosorior: I thought it was... There were some discussions around apache taking too long to restart | 08:11 |
*** bhunter has joined #tripleo | 08:11 | |
*** aufi has joined #tripleo | 08:11 | |
*** yamahata has quit IRC | 08:14 | |
*** regebro has quit IRC | 08:14 | |
*** links has quit IRC | 08:14 | |
*** regebro has joined #tripleo | 08:15 | |
*** devvesa has joined #tripleo | 08:19 | |
*** athomas has quit IRC | 08:22 | |
*** mbound_ has joined #tripleo | 08:23 | |
*** athomas has joined #tripleo | 08:23 | |
*** sthillma has quit IRC | 08:26 | |
*** sthillma has joined #tripleo | 08:26 | |
*** ifarkas has joined #tripleo | 08:39 | |
*** ifarkas has joined #tripleo | 08:40 | |
*** shardy has joined #tripleo | 08:43 | |
*** bvandenh has quit IRC | 08:43 | |
*** pblaho has joined #tripleo | 08:44 | |
*** Marga_ has quit IRC | 08:44 | |
*** links has joined #tripleo | 08:52 | |
*** nico_auv has joined #tripleo | 09:00 | |
*** sthillma has quit IRC | 09:02 | |
*** Marga_ has joined #tripleo | 09:03 | |
*** Marga_ has quit IRC | 09:03 | |
*** Marga_ has joined #tripleo | 09:03 | |
*** sthillma has joined #tripleo | 09:04 | |
*** Marga_ has quit IRC | 09:04 | |
*** Marga_ has joined #tripleo | 09:05 | |
*** gfidente has joined #tripleo | 09:10 | |
*** gfidente has quit IRC | 09:10 | |
*** gfidente has joined #tripleo | 09:10 | |
*** yuanying has quit IRC | 09:10 | |
*** gfidente has quit IRC | 09:16 | |
*** gfidente has joined #tripleo | 09:28 | |
*** gfidente has quit IRC | 09:28 | |
*** gfidente has joined #tripleo | 09:28 | |
*** paramite has joined #tripleo | 09:30 | |
*** paramite is now known as paramite|afk | 09:31 | |
*** openstackgerrit has quit IRC | 09:32 | |
*** openstackgerrit has joined #tripleo | 09:32 | |
*** sthillma has quit IRC | 09:33 | |
*** sthillma has joined #tripleo | 09:33 | |
jaosorior | gfidente: Do you remember why this was commented out? https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/manifests/overcloud_controller_pacemaker.pp#L896 | 09:36 |
jaosorior | gfidente: or how does httpd interact with pacemaker? | 09:37 |
gfidente | jaosorior, so it is started/stopped by pcmk | 09:40 |
jaosorior | gfidente: I didn't really understand where in the manifest it's set to be managed as a pacemaker service :/ | 09:41 |
gfidente | jaosorior, the reason for commenting out service_manage is in https://bugzilla.redhat.com/show_bug.cgi?id=1247547#c5 | 09:42 |
openstack | bugzilla.redhat.com bug 1247547 in openstack-tripleo-heat-templates "overcloud: HA: httpd service should not be started on boot by systemd." [High,Post] - Assigned to gfidente | 09:42 |
*** links has quit IRC | 09:42 | |
gfidente | jaosorior, and it's added to pcmk here https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/manifests/overcloud_controller_pacemaker.pp#L1501 | 09:43 |
jaosorior | aah, I understand now | 09:44 |
jaosorior | gfidente: thanks man | 09:44 |
*** mcornea has joined #tripleo | 09:48 | |
gfidente | jaosorior, I think we wanted to get away without importing ::apache at all but wasn't possible | 09:50 |
gfidente | (without importing specifically) | 09:50 |
*** athomas has quit IRC | 09:51 | |
*** jcoufal has joined #tripleo | 09:51 | |
jaosorior | gfidente: I'm running into those issues, cause running keystone in wsgi ends up iporting ::apache | 09:51 |
jaosorior | *importing | 09:51 |
gfidente | yep | 09:51 |
*** paramite|afk is now known as paramite | 09:51 | |
*** sthillma has quit IRC | 09:54 | |
*** links has joined #tripleo | 09:55 | |
*** athomas has joined #tripleo | 09:56 | |
*** jistr has joined #tripleo | 09:59 | |
*** links has quit IRC | 10:04 | |
*** mkovacik has quit IRC | 10:12 | |
*** links has joined #tripleo | 10:17 | |
*** electrofelix has joined #tripleo | 10:19 | |
*** akrivoka has joined #tripleo | 10:27 | |
*** dtantsur|afk is now known as dtantsur | 10:30 | |
*** paramite is now known as paramite|afk | 10:33 | |
openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Fix wrong keypair parameter description https://review.openstack.org/255785 | 10:41 |
openstackgerrit | Martin André proposed openstack/tripleo-specs: Spec for validation API https://review.openstack.org/255792 | 11:00 |
*** tosky has joined #tripleo | 11:06 | |
dtantsur | shardy, hi! do you have release rights? I would really appreciate a DIB release ASAP, if it's possible | 11:13 |
openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: TripleO API v1 https://review.openstack.org/243737 | 11:15 |
openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Create basic API layout https://review.openstack.org/255798 | 11:15 |
shardy | dtantsur: Hi, it looks like I do, but I've never done a TripleO release before | 11:16 |
shardy | https://review.openstack.org/#/admin/groups/200,members | 11:16 |
shardy | marios: Hey, are you familiar with the process? | 11:16 |
shardy | I see some docs in the wiki but I'm not sure if they're current | 11:17 |
marios | shardy: hey, sec, reading back | 11:17 |
marios | shardy: yeah i haven't done a release in a long time i expect the wiki won't include that new stuff like tripleo-common | 11:18 |
marios | dtantsur: sure gimme few mins and i'll try do that for you | 11:19 |
shardy | thanks marios! | 11:19 |
dtantsur | thanks! | 11:19 |
*** mkovacik has joined #tripleo | 11:21 | |
*** lucas-dinner is now known as lucasagomes | 11:28 | |
marios | dtantsur: do you know of any reason not to just bump patch here? current is 1.4.0, prevsiou two are 1.3.0 and 1.2.0 | 11:32 |
dtantsur | marios, well, changes look like features.. but I don't mind | 11:32 |
marios | shardy: don't know if that is a dib convention... at least from my notes from the last release I did we always bumped patch, unless there was/is a known backwards compat thing | 11:32 |
marios | greghaynes: there ^^^ about to cut dib release for dtantsur ... yo udid the last one @ 1.4.0 | 11:33 |
*** anande has quit IRC | 11:34 | |
openstackgerrit | Ana Krivokapic proposed openstack/tripleo-specs: Add spec for TripleO UI https://review.openstack.org/239056 | 11:36 |
*** Marga_ has quit IRC | 11:38 | |
*** Marga_ has joined #tripleo | 11:38 | |
marios | dtantsur: yeah so i see things like "Add dynamic-login element" as well as bugixes/docs etc so ... also going with convention gonna go with 1.5.0 | 11:38 |
marios | greghaynes: ^^^^ | 11:39 |
*** apetrich has quit IRC | 11:41 | |
dtantsur | ++ | 11:46 |
*** paramite|afk is now known as paramite | 11:46 | |
*** anande has joined #tripleo | 11:48 | |
*** paramite is now known as paramite|afk | 11:49 | |
marios | dtantsur: so, sorry for delay, has been so long since release i have lost my pgp key i was using for this. i am just waiting for the new one to be available (may take 10 mins) | 11:49 |
*** paramite|afk is now known as paramite | 11:50 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/instack-undercloud: Set Nova's ram_allocation_ratio configuration option to "1.0" By default https://review.openstack.org/255821 | 11:51 |
gfidente | shardy, on the hostname/ip customization | 12:04 |
*** bvandenh has joined #tripleo | 12:04 | |
marios | dtantsur: so is in progress now | 12:04 |
gfidente | shardy, I had also another question about merging together the pattern logic we currently have with anything provided from a map similar to ControllersIP | 12:04 |
dtantsur | thanks! | 12:04 |
gfidente | shardy, does that sound valid? | 12:04 |
marios | dtantsur: http://status.openstack.org/zuul/ e70138a | 12:04 |
marios | dtantsur: will make its way to pypi and http://tarballs.openstack.org once that finishes | 12:05 |
marios | dtantsur: np sorry for delay. i only ever used that pgp signature for signing git tags and it didn't survive last os migrate it seems. delay was for the new key to showup in the keyserver. anyway, good excuse to have that updated :) | 12:06 |
dtantsur | :) | 12:07 |
marios | dtantsur: http://tarballs.openstack.org/diskimage-builder/ seems there | 12:08 |
shardy | gfidente: Yeah it could be, but I was keeping the hostname/IP/placement thing separate as folks may not want to use all of them | 12:08 |
dtantsur | marios, yeah, I see it, thanks a lot! | 12:08 |
gfidente | shardy, separate yes, in a different map | 12:08 |
*** paramite is now known as paramite|afk | 12:08 | |
shardy | gfidente: athomas made the good observation that if we could make the ctlplance IP's predictable via Ironic/DHCP, we could use a DNS lookup to determine the hostname | 12:08 |
shardy | e.g assuming there is a local DNS that can resolve the provisioning network IP allocation | 12:09 |
gfidente | shardy, do a reverse lookup in heat to determine the name: param? | 12:09 |
shardy | gfidente: Yeah, well not in heat, just on the node | 12:09 |
gfidente | but the IP is a heat param | 12:09 |
shardy | it doesn't look like cloud-init support that, but it'd be easy enough in a script | 12:09 |
shardy | gfidente: not for the provisioning network | 12:09 |
shardy | gfidente: your patch only fixes the IP's for the overlay networks, doesn't it? | 12:10 |
gfidente | shardy, the existing patch only works for overlay networks yes | 12:10 |
gfidente | shardy, but I was looking at it from user perspective | 12:10 |
gfidente | user will probably want to paste the ctlplane IPs in a param as well | 12:10 |
shardy | lucasagomes: Is it possible to make Ironic deploy with predictable DHCP assignments per-node, vs just using a pool of IP's | 12:12 |
shardy | I saw the external DHCP providers patches, but I wondered if you can just configure the undercloud DHCP with mapping of MAC's->IP's | 12:12 |
shardy | I've done that before with hand-rolled DHCP/PXE setups | 12:12 |
*** paramite|afk is now known as paramite | 12:13 | |
shardy | gfidente: I'm not sure it matters that much to operators where it's configured provided placement and IP assignement is predictable | 12:14 |
shardy | I guess it'd be nice if it was all heat parameters, but it's probably not essential for the provisioning network | 12:14 |
lucasagomes | shardy, hi there, I think it may be possible by using neutron + nova | 12:14 |
lucasagomes | if you create a port in neutron with that IP and give it to nova when booting the instance | 12:14 |
lucasagomes | like --nic port_id=<id> (or something like that) | 12:15 |
shardy | lucasagomes: Ah, OK thanks! | 12:15 |
shardy | gfidente: So that is more like the fixed_ip's approach you suggested | 12:15 |
lucasagomes | shardy, I didn't try that, but I don't see why it wouldn't be possible | 12:15 |
lucasagomes | lemme know if it works please | 12:16 |
*** liverpooler has quit IRC | 12:16 | |
shardy | lucasagomes: Ok, thanks - sounds like we need to do some testing | 12:16 |
lucasagomes | yeah | 12:16 |
*** liverpooler has joined #tripleo | 12:16 | |
athomas | Thanks, lucasagomes. That's a great help. | 12:16 |
lucasagomes | athomas, np | 12:17 |
athomas | We'll get someone to try that out. If it works, and we could support setting the hostname from the DNS entry for the IP, we could solve a bunch of problems. | 12:18 |
hewbrocca | very cool | 12:18 |
gfidente | shardy, ack I'll trying the fixed_ips | 12:22 |
gfidente | if you don't get there first :P | 12:23 |
*** weshay_xchat has joined #tripleo | 12:25 | |
openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: TripleO API v1 https://review.openstack.org/243737 | 12:32 |
openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Create basic API layout https://review.openstack.org/255798 | 12:32 |
*** thrash|g0ne is now known as thrash | 12:35 | |
*** weshay_xchat is now known as weshay | 12:39 | |
*** anande has quit IRC | 12:40 | |
openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: Fixup default open_files_limit for galera https://review.openstack.org/255848 | 12:42 |
marios | mandre: ^^^ review please :) | 12:43 |
EmilienM | good morning | 12:43 |
*** apetrich has joined #tripleo | 12:45 | |
*** mbound_ has quit IRC | 12:45 | |
*** jaosorior has quit IRC | 12:47 | |
*** openstackgerrit has quit IRC | 12:47 | |
*** jaosorior has joined #tripleo | 12:47 | |
*** openstackgerrit has joined #tripleo | 12:48 | |
*** devvesa has quit IRC | 12:48 | |
openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: Fixup default open_files_limit for galera https://review.openstack.org/255848 | 12:48 |
*** panda has joined #tripleo | 12:49 | |
*** crinkle has quit IRC | 12:57 | |
*** zaneb has quit IRC | 12:57 | |
*** zaneb has joined #tripleo | 12:57 | |
*** crinkle has joined #tripleo | 12:58 | |
*** mgould has joined #tripleo | 13:05 | |
*** masco has quit IRC | 13:08 | |
jistr | shardy: hi, should we branch stable/liberty for puppet-tripleo? | 13:14 |
gfidente | jistr, yeah I was looking into that but don't have write permissions | 13:15 |
gfidente | slagle, ^^ > | 13:15 |
jistr | OPM folks asked me about it last week too | 13:15 |
jistr | i think we talked about it on the last tripleo mtg | 13:15 |
jistr | that puppet-tripleo branching would be in upstream tripleo team scope rather than upstream puppet team scope | 13:16 |
*** mgrohar has joined #tripleo | 13:17 | |
openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Allows customization of the HAProxy default timeouts https://review.openstack.org/255870 | 13:20 |
*** dprince has joined #tripleo | 13:21 | |
openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: TripleO API v1 https://review.openstack.org/243737 | 13:21 |
openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Create basic API layout https://review.openstack.org/255798 | 13:21 |
shardy | jistr: Hey, yes we should, I've been meaning to cut it but wanted to confirm the SHA, is latest master looking OK to branch from? | 13:23 |
*** trown|outttypeww is now known as trown | 13:24 | |
shardy | trown: ^^ discussing creating stable/liberty branch for puppet-tripleo and looking to confirm the best SHA to branch from | 13:25 |
trown | shardy: I would guess latest puppet-tripleo is in the latest OPM built in delorean? | 13:26 |
trown | that is still a bit magic to me | 13:26 |
*** jprovazn has quit IRC | 13:26 | |
shardy | trown: Yeah I'm pretty sure stable CI is using lastest master, and last time I checked that was working OK | 13:26 |
shardy | I just wanted to ensure you didn't have a downstream branch already we needed to align with | 13:26 |
jistr | stable CI is using master of puppet-tripleo? | 13:26 |
*** jayg|g0n3 is now known as jayg | 13:27 | |
shardy | jistr: it must be, there are no other branches | 13:27 |
jistr | wouldn't it be using master of OPM instead? | 13:27 |
*** mkovacik_ has joined #tripleo | 13:27 | |
shardy | hmm, let me check | 13:27 |
jistr | i'm not 100% sure, but i think the OPM rpm is built from here https://github.com/redhat-openstack/openstack-puppet-modules | 13:27 |
trown | shardy: for RDO I pretty much only care about delorean, and it promoted a repo yesterday so we should be good as long as puppet-tripleo is represented there | 13:27 |
slagle | gfidente: what do you need write permissions for? | 13:28 |
*** tzumainn has joined #tripleo | 13:28 | |
*** rbrady has joined #tripleo | 13:28 | |
*** mkovacik has quit IRC | 13:29 | |
shardy | slagle: the tripleo-release group to cut a puppet-tripleo branch | 13:29 |
*** pradk has joined #tripleo | 13:29 | |
shardy | I'm on it now tho | 13:29 |
slagle | oh ok | 13:29 |
shardy | can add gfidente anyway :) | 13:29 |
gfidente | ack thanks, useful next time | 13:29 |
shardy | jistr: Yeah sorry you're right, I got confused by what goes into o-p-m | 13:30 |
jistr | shadower: actually OPM even has a stable/liberty branch here https://github.com/redhat-openstack/openstack-puppet-modules/tree/stable/liberty | 13:30 |
* jistr pinged social to join this channel | 13:31 | |
shardy | Ok, but that will build from puppet-tripleo master if there's no stable branch right? | 13:31 |
shardy | is there a delorean build log we can check to confirm the sha for the latest package? | 13:31 |
trown | shardy: looking | 13:32 |
jayg | jistr: I was just asking about this on #rdo yesterday too, it seems to me nobody really knows how opm gets there.... | 13:32 |
jayg | we just built for downstream, so we have a new build with lots of fixes, probably another coming today or tomorrow | 13:32 |
jistr | ah ok | 13:32 |
jistr | i recall social pinged me a few days ago re "where's the stable/liberty of puppet-tripleo, should we just use master?" and i think he wanted that ref for the stable/liberty branch of OPM | 13:33 |
gfidente | I think this is the hash which OPM liberty uses of tripleo | 13:34 |
gfidente | https://github.com/redhat-openstack/openstack-puppet-modules/blob/stable/liberty/Puppetfile#L258 | 13:34 |
jistr | +1 | 13:34 |
trown | hmm... not sure how to see what actually goes into OPM... http://trunk.rdoproject.org/centos7-liberty/1b/72/1b721d549994b30794f01c4457a1babab1aca58b_41994f7f/rpmbuild.log is the latest build in liberty delorean | 13:34 |
gfidente | and that is latest from master yes | 13:35 |
gfidente | so I think we're good branching from current master | 13:35 |
EmilienM | should I abandon https://review.openstack.org/#/c/223223/ ? | 13:35 |
shardy | Ok, thanks guys, I'll branch now from 74b589497c9efe400403f2d446aa6335abdb89d5 | 13:35 |
trown | gfidente: shardy I do not think that commit has been built in delorean OPM | 13:36 |
jistr | shardy: ack. I'm not sure if we're actually using the OPM from stable/liberty, but we probably should be, so +1 on branching puppet-tripleo from the ref which is in OPM stable/liberty. | 13:36 |
*** nico_auv has quit IRC | 13:38 | |
jistr | trown: ok that's a good point too :) maybe we should find it somewhere first, like where stable/liberty OPM gets built, if anywhere | 13:38 |
trown | hmm... I have no idea how OPM is being built actually... looks like from master-patches branch | 13:39 |
* shardy holds off branching until we figure it out | 13:39 | |
jayg | trown: I am pretty sure the 7.0.2 tag we just built from for downstream yesterday is on stable/liberty | 13:39 |
EmilienM | gfidente, jistr: I would like your thought on this old patch: https://review.openstack.org/#/c/223223/ -- should I abandon it? Look at the BZ in the patch topic | 13:39 |
trown | https://github.com/redhat-openstack/openstack-puppet-modules/commit/1b721d549994b30794f01c4457a1babab1aca58b is the latest commit in delorean liberty | 13:39 |
trown | which is HEAD of master-patches branch on github | 13:40 |
gfidente | trown, eheh and in there the tripleo hash is different | 13:40 |
jayg | social is on a meetingm, I'll remind him to join when done to help answer questions - but we are a little unclear on how it gets to delorean/rdo as well, think there has been some shuffling of who does what | 13:40 |
gfidente | https://github.com/redhat-openstack/openstack-puppet-modules/blob/master-patches/Puppetfile#L258 | 13:40 |
jayg | https://github.com/redhat-openstack/openstack-puppet-modules/tree/stable/liberty is what you want for liberty | 13:41 |
jistr | i asked this on #rdo-puppet http://fpaste.org/299464/54872144/ | 13:41 |
jaosorior | any +A for this? https://review.openstack.org/#/c/251291/1 | 13:43 |
trown | jayg: that would make sense... however that is not what is currently building in delorean | 13:43 |
tzumainn | dprince, hiya! I addressed your comments for the API spec, let me know if you think there are still -2 worthy issues with it | 13:43 |
jayg | trown: ok, so if we can fix that setup, you should be good | 13:43 |
jayg | note I have no idea how to do so | 13:44 |
jistr | trown: so we're using master-patches branch of OPM in stable/liberty CI TripleO runs? perhaps we should branch puppet-tripleo from that then | 13:44 |
trown | jayg: :) I will check rdoinfo | 13:44 |
dprince | tzumainn: thanks, I'll have a look | 13:44 |
trown | jistr: I would be +1 to that from RDO perspective, since that has passed RDO CI | 13:44 |
jayg | just to make it perfectly clear, whatever you guys recommend for puppet-tripleo hash to be in liberty we can switch to, if the hash in stable/liberty is incorrect, just let us know | 13:45 |
jistr | trown: right... and it also passed the TripleO CI | 13:45 |
*** derekh has joined #tripleo | 13:45 | |
gfidente | trown, jistr I see but I'd follow jayg | 13:45 |
gfidente | if the hash in stable/liberty is wrong, let's update it | 13:46 |
gfidente | and possibly build OPM from stable/liberty for tripleo ci | 13:46 |
jistr | yeah, +1 let's start from what we already test | 13:46 |
jayg | we are going to be working on a few updates to go into another liberty build anyway, so now is the time to coordinate this | 13:46 |
jistr | because updating a hash in OPM is easier than removing commits from a stable/liberty of puppet-tripleo via reverts or even force pushes | 13:47 |
gfidente | jistr, ack | 13:47 |
jayg | and if someone can tell me if we need to do something to help get delorean squared away later, we can do that as well | 13:47 |
openstackgerrit | Merged openstack/instack-undercloud: Remove Ceilometer Alarm from the undercloud https://review.openstack.org/253716 | 13:47 |
gfidente | derekh, do you know where we configure in delorean which branch of OPM to use for the tripleo stable/liberty CI? | 13:47 |
jayg | right, it would be great it opm can have a stable branch in puppet-tripleo to track | 13:48 |
jayg | s/it/if | 13:48 |
trown | gfidente: the issue is https://github.com/redhat-openstack/rdoinfo/blob/master/rdo.yml#L491 | 13:50 |
gfidente | trown++ | 13:51 |
gfidente | trown, that is the 'master' branch of rdoinfo | 13:51 |
gfidente | trown, don't we need a liberty branch of rdoinfo to point at the liberty branch of opm? | 13:51 |
trown | gfidente: ya same branch is used for all deloreans, but with a different delorean config | 13:51 |
trown | gfidente: we do if we have source-branch override in rdoinfo... OPM is the only project doing that | 13:52 |
gfidente | trown, ok can you change the source-branch in rdoinfo? | 13:52 |
gfidente | I will try to update opm stable/liberty to use same hash that master-patches is using | 13:53 |
trown | gfidente: it is unfortunately not so simple... we do not use multiple branches of rdoinfo in delorean, so we would also have to change the delorean server to do that | 13:54 |
jayg | gfidente: are you meaning only for puppet-tripleo? | 13:54 |
derekh | gfidente: that would be in rdoinfo, one sec | 13:54 |
gfidente | jayg, only puppet-tripleo yes | 13:54 |
jayg | gfidente: k | 13:54 |
trown | it would be much better if OPM could use master branch for master and remove that source-branch override from rdoinfo | 13:55 |
jayg | trown: I am not familiar with this, but the top of the file you linked says each package can have its own 'releases' section | 13:55 |
trown | then stable/liberty would just work | 13:55 |
* shardy is confused | 13:55 | |
derekh | gfidente: you probably need to ask apevec to be sure but I think it might be here https://github.com/apevec/rdoinfo/blob/master/rdo.yml#L504 | 13:56 |
trown | jayg: I am not too sure about the stuff at the top of that file :( | 13:56 |
shardy | It may just be easier to update TripleO CI to build the modules from source, for stable/liberty, like it does for master | 13:56 |
trown | jayg: lots of legacy there | 13:56 |
jayg | ah, ok | 13:56 |
shardy | that's on my todo list but not got to it yet | 13:56 |
beagles | marios, thanks for your feedback on the QoS review! | 13:56 |
*** social has joined #tripleo | 13:57 | |
jistr | shardy: hmm but we should have it for dev setups too... this should be fixed on RDO delorean side, not on tripleo side | 13:57 |
social | jang: yere? | 13:57 |
beagles | marios, I need a bit of a sounding board though for my next set update, do you have a second? | 13:57 |
jayg | social: hello! | 13:57 |
*** openstackstatus has quit IRC | 13:57 | |
*** links has quit IRC | 13:57 | |
marios | beagles: np, thanks for looking | 13:58 |
shardy | jistr: Yeah, agreed, I'm just saying from the TripleO perspective from-source may be better | 13:58 |
jistr | social: hi, just to update you on the latest findings, seems like liberty delorean is building from master-patches branch instead of stable/liberty, probably because of this https://github.com/redhat-openstack/rdoinfo/blob/master/rdo.yml#L491 | 13:58 |
*** mgrohar has quit IRC | 13:58 | |
jayg | trown | it would be much better if OPM could use master branch for master and remove that source-branch override from rdoinfo | 13:58 |
jistr | ^ yeah was just pasting that | 13:58 |
*** dmacpher has joined #tripleo | 13:58 | |
social | jistr: that should be changed already by apevec | 13:58 |
jayg | but I am not yet quite sure how these branches are organized to say if that makes any sense for us | 13:58 |
*** jprovazn has joined #tripleo | 13:59 | |
beagles | marios, I find myself weighing two options - add the qos as default to N-AgentExtensions, etc and remove the NeutronEnableQoS flag or keep the flag and remove the N-AgentExtensions N-ExtensionDriver parameters ... | 13:59 |
social | jistr: because he patches build-opm.sh | 13:59 |
social | ah I don't see any patch | 14:00 |
beagles | marios, my thinking is this - if I keep the string lists and default them to QoS then I open the door for the unexpected if a deployer removes qos from the list, but leaves the NeutronEnableQoS as true... | 14:00 |
marios | beagles: yeah i don't know. if you see it as something that will be/remain 'qos' for a while (e.g. cos new feature) then maybe just add the NeutronEnableQoS flag. | 14:00 |
beagles | marios, if I want to keep the flag, I'm probably better to remove the N-ExtensionDriver and N-AgentLists | 14:00 |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Switch to F22 jobs https://review.openstack.org/255887 | 14:01 |
jayg | social: where is this build-opm.sh? | 14:01 |
marios | beagles: yeah so my primary concern was user facing interface with the params. but in general wrt sanity checking, that is aproblem, and not one we always can/do deal with | 14:01 |
jistr | social: hmm sorry i don't know what's build-opm.sh, but let me just follow up on what trown said as it sounded right to me, from what i can tell as a non-delorean person :) IIUC if we'd rename 'master-patches' to 'master', and 'master' to something else (in OPM repo), and remove that source-branch line from rdoinfo, then things could just work without further patching | 14:01 |
jistr | trown: ^ sounds correct? | 14:02 |
derekh | Tripleo jobs are now all F22 can somebody review ^^ to keep the status page looking at the correct thing................... >> https://review.openstack.org/255887 | 14:02 |
beagles | marios, yeah.. wow. I forget how much your brain can twist when you have to think about human-facing-interfaces | 14:02 |
social | I don't see change done by apvec | 14:02 |
beagles | marios, okay thanks, I'll keep it simple! | 14:02 |
marios | beagles: yeah, i think we can make it configurable (the drivers/extensions) if/when that is necessary/requested i think | 14:03 |
*** julim has quit IRC | 14:03 | |
beagles | marios, right | 14:03 |
*** julim has joined #tripleo | 14:03 | |
jistr | social, trown: sounds to me like there's some convention on branch names, 'master' going to delorean master, 'stable/liberty' going to delorean liberty etc. so perhaps it would be easiest if OPM followed that convention | 14:04 |
* jayg has to walk the dogs, back in about 20 minutes | 14:04 | |
trown | jistr: that would be my vote | 14:04 |
social | jistr: we do follow that, opm in rhos8 is from stable/liberty liberty-patches | 14:05 |
*** rlandy has joined #tripleo | 14:05 | |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: [NO MERGY] Test a update in trunk repository version https://review.openstack.org/229789 | 14:06 |
openstackgerrit | Ryan Brady proposed openstack/tripleo-common: Ensures plan exists https://review.openstack.org/255400 | 14:06 |
*** julim has quit IRC | 14:08 | |
*** morazi has joined #tripleo | 14:08 | |
jistr | social: hehe now i'm getting confused :) mostly about what's the difference between stable/liberty vs. liberty-patches, and other RDO packages also have these two branches | 14:09 |
social | I'll try to explain but I'm in meeting | 14:09 |
jistr | sure thing, let's pause this for a bit | 14:10 |
*** paramite is now known as paramite|afk | 14:13 | |
*** paramite|afk is now known as paramite | 14:15 | |
social | jistr: 15:18 < apevec> social, on hold until testday(s) are over, please do not rebase master-patches this week! | 14:24 |
social | so yeah I'm going to bump opm master for you guys | 14:24 |
jistr | let's discuss this in person to make sure we're on the same page what the issue is :) | 14:25 |
EmilienM | slagle: hey, you asked me to come back if no reviews -- could you help on https://etherpad.openstack.org/p/tripleo-aodh-patches ? | 14:27 |
*** devvesa has joined #tripleo | 14:28 | |
*** jcoufal has quit IRC | 14:34 | |
*** jcoufal has joined #tripleo | 14:34 | |
jistr | alright this is way more complicated than i thought :D | 14:35 |
gfidente | ahahaha | 14:35 |
*** lucasagomes is now known as lucas-hungry | 14:35 | |
gfidente | and if we wanted to take it as a black box | 14:35 |
gfidente | what is that we need to know? | 14:35 |
jistr | * the other projects should be following what OPM does for mitaka, so changing conventions on OPM at this point doesn't make sense | 14:36 |
social | jistr: trown: so opm, we have stable/liberty for first time only because RDO moved out of fedora. It would be f24 and f24-patches instead | 14:37 |
*** rhallisey has joined #tripleo | 14:37 | |
*** rpothier has joined #tripleo | 14:37 | |
social | jistr: trown: yet the branch without -patches is mere copy of upstream while -patches contains our patchset and is used by rdopkg for package building | 14:37 |
openstackgerrit | Miles Gould proposed openstack/python-tripleoclient: [WIP] Support new ironic "enroll" state https://review.openstack.org/235158 | 14:38 |
jistr | and the change to make liberty delorean build liberty OPM is ready i think, but held off for now due to upcoming test day | 14:38 |
social | jistr: trown: It's easy now to hack it temporarly in delorean as opm in mitaka+ will change a lot switching to opm-ci which means it'll get bit more automated and will contain stable stable-tag and stable-patches branches | 14:38 |
social | where stable-tag will be the one to be used and tracked | 14:38 |
openstackgerrit | Ryan Brady proposed openstack/tripleo-common: Ensures plan exists https://review.openstack.org/255400 | 14:40 |
openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates: Enable setting Neutron QoS feature https://review.openstack.org/247056 | 14:41 |
*** liverpooler has quit IRC | 14:41 | |
*** lblanchard has joined #tripleo | 14:44 | |
*** nico_auv has joined #tripleo | 14:44 | |
*** paramite is now known as paramite|afk | 14:45 | |
gfidente | jistr, so it'll be something like liberty-patches but it's not there yet? | 14:47 |
gfidente | social ^^ | 14:48 |
jistr | so the gist of what we just discussed in person with social and paramite is: we're testing everything with puppet-tripleo ref which is in master-patches branch of OPM. liberty-patches of OPM is ahead of master-patches wrt puppet-tripleo ref, as it got branched just a few days ago from puppet-tripleo master, because there was no stable/liberty to branch from. What we want to do is un-bump puppet-tripleo in liberty-patches branch of OPM to the | 14:48 |
jistr | version we're testing now, so that when liberty delorean is fixed to build OPM from liberty-patches and not master-patches, we'd keep using the same version of puppet-tripleo | 14:48 |
jistr | sounds ok? | 14:48 |
gfidente | yes | 14:49 |
gfidente | that is great | 14:49 |
gfidente | so we need to change the hash in liberty-patches to match master-patches | 14:49 |
rhallisey | shardy, ping. I need some wisdom | 14:50 |
gfidente | and we need to branch puppet-tripleo from the hash in master-patches | 14:50 |
jistr | gfidente: yea | 14:50 |
gfidente | jistr tks | 14:50 |
trown | jistr: gfidente, makes sense to me. thanks! | 14:50 |
gfidente | shardy, so the branch hash is ae595ce7731f2b286d4ffd280ed735b61d86b63c | 14:50 |
jistr | so we'd branch puppet-tripleo stable/liberty from ae595ce7731f2b286d4ffd280ed735b61d86b63c | 14:50 |
jistr | :) | 14:51 |
gfidente | hurrah | 14:51 |
shardy | gfidente: thanks, done! | 14:53 |
derekh | Tripleo jobs are now all F22 can somebody review status page update to keep it looking at the correct thing https://review.openstack.org/255887 | 14:53 |
shardy | derekh: done | 14:54 |
*** julim has joined #tripleo | 14:54 | |
*** Goneri has quit IRC | 14:54 | |
derekh | shardy: gfidente thanks, going to approve as CI does nothing on this file | 14:55 |
openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Set defaultbranch in .gitreview to stable/liberty https://review.openstack.org/255919 | 14:56 |
*** pradk has quit IRC | 14:57 | |
*** pradk_ has joined #tripleo | 14:57 | |
d0ugal | tripleo_common has a bunch of exceptions in tripleo_common.core.exception. Does it make sense for the API to use these? Or should the API catch and re-raise an exception for each that controls the message and status code. | 14:57 |
d0ugal | rbrady: ^ thoughts? | 14:57 |
*** pradk_ is now known as pradk | 14:58 | |
rbrady | d0ugal: If it uses the exceptions directly, we'd need to change them to add a status code and treat them more like HTTP exceptions, right? | 15:00 |
d0ugal | rbrady: Yeah | 15:01 |
d0ugal | rbrady: I guess that is a bit of an abuse of the exceptions | 15:01 |
rbrady | d0ugal: If the CLI is going to use the API anyway, then I don't see an issue with changing the exceptions and just using them | 15:01 |
d0ugal | rbrady: I added status code's to a couple here to test things out: https://review.openstack.org/#/c/243737/33/tripleo_common/core/exception.py,cm | 15:01 |
rbrady | d0ugal: If the CLI or any other code will use the library directly, then there might be an issue with it | 15:01 |
d0ugal | so only status code are needed | 15:01 |
d0ugal | The CLI currently uses bits, but not these exceptions and I understand we are migrating everything to using the API | 15:02 |
d0ugal | so I think that is fine. | 15:02 |
rbrady | d0ugal: okay...let's add the attributes needed. seems like a small patch atm | 15:03 |
d0ugal | rbrady: ok, thanks. I guess we can always split it out if it feels weird after doing a few more | 15:03 |
rbrady | d0ugal: ack | 15:04 |
openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Remove not needed completion-signal https://review.openstack.org/255927 | 15:09 |
*** mgould has quit IRC | 15:10 | |
openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: Sample environment with old ServiceNetMap value https://review.openstack.org/255929 | 15:11 |
gfidente | marios, slagle ^^ you know that using internal_api other than being the default before is also the default for the external lb | 15:12 |
gfidente | marios, slagle and the only reason why we switched that to ctlplane was os-cloud-config ... | 15:12 |
openstackgerrit | Merged openstack-infra/tripleo-ci: Switch to F22 jobs https://review.openstack.org/255887 | 15:13 |
gfidente | marios, slagle so I think internal_api was actually a better default, except we had to cope with os-cloud-config | 15:13 |
slagle | gfidente: i've argued for that and ctlplane was still chosen | 15:13 |
gfidente | :( | 15:13 |
slagle | imo, it doesnt even need a specific example | 15:13 |
slagle | it's an internal api | 15:13 |
gfidente | and can we make internal_api the default for 8 though? | 15:13 |
slagle | hmm, i wonder what network it should go on???? :) | 15:14 |
devvesa | dprince: can you ask you to do some reviews for the midonet integration with TripleO? | 15:18 |
devvesa | They are small | 15:18 |
openstackgerrit | Lukas Bezdicka proposed openstack/puppet-tripleo: Set tunnel timeout for nova_novncproxy https://review.openstack.org/255933 | 15:22 |
openstackgerrit | Lukas Bezdicka proposed openstack/puppet-tripleo: MidoNet services manifests https://review.openstack.org/255934 | 15:22 |
openstackgerrit | Lukas Bezdicka proposed openstack/puppet-tripleo: Fix unit tests failing against Puppet 4.3.0 https://review.openstack.org/255935 | 15:22 |
dprince | devvesa: I will try in a bit, link them? | 15:22 |
social | <.< | 15:23 |
devvesa | dprince: https://review.openstack.org/#/c/250763/, https://review.openstack.org/#/c/251319/, https://review.openstack.org/#/c/250777/ | 15:23 |
*** trown is now known as trown|brb | 15:23 | |
dprince | gfidente: once we use Puppet to configure keystone we can forget about os-cloud-config | 15:23 |
dprince | gfidente: but that patch was reverted | 15:24 |
dprince | gfidente: forget about os-cloud-config for keystone that is | 15:24 |
gfidente | dprince, yeah was just speculating to set back the default to internal_api for mitaka | 15:24 |
dprince | gfidente: please don't, until we get the Puppet keystone configuration landed | 15:24 |
gfidente | dprince, sure, only after we get that | 15:24 |
gfidente | and IF we get that | 15:25 |
gfidente | actually, what is the status of that patch? anybody working on it? | 15:25 |
*** mgould has joined #tripleo | 15:25 | |
dprince | gfidente: I think bnemec posted it | 15:25 |
dprince | gfidente: but is is blocked on "updates testing" or something | 15:25 |
gfidente | yeah I don't know the details of that though | 15:26 |
*** trown|brb is now known as trown | 15:27 | |
shardy | dprince: It was proven to break things on update/upgrade, which is why it was reverted | 15:29 |
openstackgerrit | Merged openstack/python-tripleoclient: Increase test coverage for utils and plugin https://review.openstack.org/253040 | 15:30 |
dprince | shardy: yep, I approved the revert :) | 15:31 |
dprince | shardy: I'm just very keen to have it back because it solves some problems. Namely keystone v3 and gets us of os-cloud-config | 15:31 |
*** paramite|afk is now known as paramite | 15:32 | |
shardy | dprince: Cool - IMHO nothing should be blocked on automated updates testing, we just can't merge anything risky without someone testing the process manually and proving the world doesn't break | 15:32 |
shardy | obviously automating it is preferable in the long term | 15:32 |
dprince | shardy: sure, I spoke to slagle with regards to this yesterday as well. manual is fine for now | 15:33 |
dprince | shardy: anyways, it is blocked until someone does this I guess | 15:34 |
*** Goneri has joined #tripleo | 15:44 | |
*** lucas-hungry is now known as lucasagomes | 15:51 | |
*** paramite is now known as paramite|afk | 15:52 | |
*** paramite|afk is now known as paramite | 15:55 | |
openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: TripleO API v1 https://review.openstack.org/243737 | 15:56 |
openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: Set default start/stop timeout for pcmk services to 95s https://review.openstack.org/255949 | 15:56 |
openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: Bump further the stop/start timeout for pcmk/systemd services https://review.openstack.org/255950 | 15:56 |
EmilienM | if you have some time, please help me to have https://etherpad.openstack.org/p/tripleo-aodh-patches reviewed | 16:00 |
openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: Fixup default open_files_limit for galera https://review.openstack.org/255848 | 16:01 |
openstackgerrit | Ronelle Landy proposed openstack/tripleo-common: Merge "Fix node group name when deleting a node" https://review.openstack.org/255955 | 16:03 |
openstackgerrit | Merged openstack/puppet-tripleo: loadbalancer: add Aodh API support https://review.openstack.org/241395 | 16:06 |
*** dtantsur is now known as dtantsur|brb | 16:08 | |
openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: honor the rabbit user and password provided https://review.openstack.org/255959 | 16:10 |
*** apetrich has quit IRC | 16:14 | |
*** pblaho has quit IRC | 16:14 | |
openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: TripleO API v1 https://review.openstack.org/243737 | 16:16 |
openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Create basic API layout https://review.openstack.org/255798 | 16:16 |
openstackgerrit | John Trowbridge proposed openstack/tripleo-common: [stable/liberty]Fix node group name when deleting a node https://review.openstack.org/255955 | 16:18 |
*** dprince has quit IRC | 16:19 | |
*** aufi has quit IRC | 16:19 | |
*** tremble has quit IRC | 16:20 | |
*** rohitpagedar__ has joined #tripleo | 16:20 | |
*** dprince has joined #tripleo | 16:21 | |
jpeeler | d0ugal: is there anybody that can be poked to get https://review.openstack.org/#/c/254885/ merged? | 16:21 |
d0ugal | jpeeler: We need CI to pass and we need another code to +2 | 16:23 |
d0ugal | s/code/core/ | 16:23 |
jpeeler | d0ugal: it won't pass CI until the delorean packaging is fixed though, which can't be merged before, right? | 16:23 |
d0ugal | oh yeah | 16:23 |
d0ugal | sorry, I forgot it was *this* one | 16:23 |
openstackgerrit | Lennart Regebro proposed openstack/python-tripleoclient: More tests for overcloud deploy https://review.openstack.org/254767 | 16:24 |
d0ugal | Any cores around with a few mins to take a look? | 16:24 |
openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: TripleO API v1 https://review.openstack.org/243737 | 16:24 |
openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Create basic API layout https://review.openstack.org/255798 | 16:24 |
jpeeler | dprince: can you give a quick look at https://review.openstack.org/#/c/254885/? (my review is 3 dependencies deep currently) | 16:26 |
*** Marga_ has quit IRC | 16:27 | |
openstackgerrit | Derek Higgins proposed openstack/tripleo-common: Don't install git-hg https://review.openstack.org/255970 | 16:31 |
derekh | shardy: stable jobs are failing because of the switch to F22, fix is there ^^ | 16:32 |
shardy | derekh: kk, thanks | 16:41 |
dprince | jpeeler: yes, in a bit. I've got to run out now... I've got it up now though | 16:41 |
*** dprince has quit IRC | 16:41 | |
shardy | derekh: +2, feel free to approve when it passes CI if I don't check back first | 16:42 |
*** rcernin has quit IRC | 16:42 | |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Manage keystone initialization directly in t-h-t manifests https://review.openstack.org/244162 | 16:44 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: WIP - Deploy Aodh services, replacing Ceilometer Alarm https://review.openstack.org/241408 | 16:44 |
rhallisey | derekh, ya that's why the containers job was failing :( | 16:45 |
rhallisey | It's run on f22 | 16:45 |
derekh | rhallisey: for liberty yes probably, it should be ok for master as that patch was already in. | 16:49 |
*** paramite is now known as paramite|afk | 16:50 | |
*** paramite|afk is now known as paramite | 16:52 | |
*** david-lyle has quit IRC | 16:53 | |
*** dtantsur|brb is now known as dtantsur | 17:01 | |
*** dprince has joined #tripleo | 17:02 | |
*** ibravo has joined #tripleo | 17:04 | |
*** yamahata has joined #tripleo | 17:04 | |
*** jprovazn has quit IRC | 17:05 | |
openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Pacemaker maintenance mode for the duration of Puppet run on update https://review.openstack.org/245093 | 17:08 |
openstackgerrit | Derek Higgins proposed openstack/instack-undercloud: Change undercloud replicat count to 1 https://review.openstack.org/255995 | 17:15 |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: [NO MERGY] Test a update in trunk repository version https://review.openstack.org/229789 | 17:15 |
*** ayoung has joined #tripleo | 17:15 | |
ayoung | jaosorior, so, on https://review.openstack.org/#/c/213175/11/puppet/manifests/overcloud_controller_pacemaker.pp,cm do we have a catch 22? We need to affect change in both puppet and triple-heat at the same time, or can we do this all in the heat template? | 17:17 |
openstackgerrit | Ben Swartzlander proposed openstack/tripleo-heat-templates: Enable Manila integration https://review.openstack.org/188137 | 17:18 |
*** dprince has quit IRC | 17:22 | |
*** alop has joined #tripleo | 17:24 | |
openstackgerrit | Ben Swartzlander proposed openstack/tripleo-heat-templates: Enable Manila integration https://review.openstack.org/188137 | 17:25 |
openstackgerrit | Ben Swartzlander proposed openstack/tripleo-heat-templates: Add NetApp integration to Manila https://review.openstack.org/188138 | 17:25 |
openstackgerrit | Ben Swartzlander proposed openstack/tripleo-heat-templates: Add Gluster integration to Manila https://review.openstack.org/230936 | 17:25 |
rhallisey | shardy, every so often my deployment hangs here: http://fpaste.org/299579/44976853/ | 17:29 |
openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: WIP: Allow predictable IPs for Controllers on the ctlplane https://review.openstack.org/256003 | 17:29 |
openstackgerrit | greghaynes proposed openstack/diskimage-builder: Allow grub2 to build with opensuse https://review.openstack.org/251912 | 17:29 |
rhallisey | any idea why? Heat is constantly signaling, but never passing a create complete when a `heat stack-list` shows a create-complete | 17:30 |
shardy | rhallisey: I'm confused, if heat stack-list says CREATE_COMPLETE, it is complete? | 17:32 |
rhallisey | shardy, ya | 17:32 |
rhallisey | except it never signals back | 17:32 |
*** richm has joined #tripleo | 17:33 | |
rhallisey | it just hangs there | 17:33 |
ayoung | richm, cool, you are here | 17:33 |
shardy | Oh, you mean the event polling in tripleoclient? | 17:33 |
ayoung | so https://review.openstack.org/#/c/213175/11/puppet/manifests/overcloud_controller_pacemaker.pp,cm | 17:33 |
shardy | rhallisey: If heat stack-list show CREATE_COMPLETE, tripleoclient should complete and exit | 17:33 |
*** mkovacik_ has quit IRC | 17:33 | |
shardy | I suspect it's more a client issue, but it's hard to say what | 17:34 |
rhallisey | shardy, ok will look there. Thanks | 17:34 |
ayoung | richm, trying to get a handle on what the puppet manifest will do if we change the heat template. jaosorior has a comment, but I am not clear on what it means | 17:34 |
shardy | rhallisey: heat doesn't signal anything on completion, tripleoclient is polling the heat event API | 17:34 |
shardy | maybe that is what's getting stuck | 17:34 |
rhallisey | shardy, oh that's what I see in the logs then | 17:34 |
shardy | rhallisey: Yeah, if you run tripleoclient with --debug it'll show each call to heat | 17:34 |
shardy | hopefully you'll see where it's getting hung up then | 17:35 |
ayoung | richm, "That is because httpd will be managed by pacemaker instead." is that as significant a statement as I think it is, or does that just mean that heat is trying to do too much, and we can remove things | 17:35 |
*** Marga_ has joined #tripleo | 17:35 | |
rhallisey | shardy, cool thx | 17:35 |
richm | I don't know how pacemaker works - is it ok to restart httpd "out from under" pacemaker? | 17:36 |
*** dmacpher is now known as dmacpher-afk | 17:36 | |
*** athomas has quit IRC | 17:37 | |
gfidente | richm, nope it isn't | 17:37 |
*** ukalifon has quit IRC | 17:37 | |
*** openstackstatus has joined #tripleo | 17:38 | |
*** ChanServ sets mode: +v openstackstatus | 17:38 | |
gfidente | richm, it might work on occasion, but it might trigger bad events in others | 17:38 |
richm | Then I'm not sure how puppet is supposed to work in this case - you modify a config that requires a service restart to take effect - what service is puppet supposed to know to notify? | 17:38 |
richm | ~> Service['some service provided by caller'] | 17:39 |
richm | or | 17:39 |
richm | tell puppet that service management is outside of puppet, then | 17:39 |
richm | ~> Service['/dev/null'] | 17:40 |
gfidente | richm, we used to pass something like manage_service false to the puppet module so that it won't trigger restart when notified | 17:41 |
*** dprince has joined #tripleo | 17:41 | |
*** weshay has quit IRC | 17:41 | |
richm | gfidente: then how does pacemaker know that it needs to restart httpd if some config changes, if it is not getting those puppet notifies? | 17:42 |
gfidente | richm, currently the sad story is that we can't do this on a per-service basis and tell pcmk to restart the services | 17:44 |
gfidente | https://review.openstack.org/#/c/245093/18/extraconfig/tasks/pacemaker_resource_restart.sh,cm | 17:44 |
richm | gfidente: so do you just tell pacemaker to restart httpd unconditionally? | 17:45 |
*** masco has joined #tripleo | 17:45 | |
richm | every time puppet runs? | 17:45 |
gfidente | richm, yes unfortunately that is the effect | 17:45 |
richm | ok | 17:45 |
richm | gfidente: ayoung: Then this is a bug (or rfe) for puppet-keystone - need to be able to disable all ~> and notifies for Service['httpd'] and/or Service['keystone'] | 17:46 |
ayoung | Fcuking Hell. | 17:46 |
ayoung | Can we please not make things impossible? | 17:46 |
*** weshay_xchat has joined #tripleo | 17:46 | |
ayoung | Pretty please? | 17:47 |
*** weshay_xchat is now known as weshay | 17:47 | |
*** paramite has quit IRC | 17:48 | |
richm | I don't know if it is possible to work around this - maybe there is some puppet hack whereby you can define/alias/redirect notifies to Service['httpd'] to a no-op | 17:48 |
ayoung | richm, so let me get this straight. I took us a couple years to finally get HTTPD support into puppet-keystone, and, now that we have it, we can't use it because of Pacemaker. I assume this change is a small change? | 17:48 |
richm | will need a puppet guru for that | 17:48 |
ayoung | Puppet needs to die | 17:48 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: WIP - Deploy Aodh services, replacing Ceilometer Alarm https://review.openstack.org/241408 | 17:49 |
gfidente | I think if you pass service_manage false to ::apache the -> won't trigger a restart | 17:49 |
* ayoung gonna go rewrite everything in ansible. | 17:49 | |
*** trown is now known as trown|lunch | 17:49 | |
richm | ayoung: a) right - can't use it because pacemaker b) it is a small change c) puppet cannot die | 17:49 |
ayoung | I know, you shoot the puppet, all you do is injure the puppeteer | 17:49 |
EmilienM | lol | 17:49 |
richm | gfidente: I don't think that will suffice - https://github.com/openstack/puppet-keystone/blob/master/manifests/wsgi/apache.pp#L168 | 17:49 |
richm | gfidente: afaict, keystone::wsgi::apache needs to be fixed | 17:50 |
EmilienM | richm: for what? | 17:50 |
pino|work | (don't kill puppet, think about the jistr involved) | 17:50 |
richm | EmilienM: Then this is a bug (or rfe) for puppet-keystone - need to be able to disable all ~> and notifies for Service['httpd'] and/or Service['keystone'] | 17:50 |
jistr | :D | 17:50 |
EmilienM | richm: oh I can imagine that's because of pacemaker? | 17:51 |
richm | EmilienM: correct | 17:51 |
jistr | i think what we really need is a faux service provider for services managed by pacemaker | 17:51 |
EmilienM | I agree, we need to fix it | 17:51 |
jistr | which wouldn't react on restarts | 17:51 |
richm | jrist: +1 | 17:51 |
EmilienM | jistr: yeah | 17:51 |
ayoung | could we tell puppet somehow that Keystone == pacemaker? | 17:52 |
richm | ayoung: not exactly | 17:52 |
ayoung | So the puppet modules wouldn't know what was implementing it | 17:52 |
ayoung | just defere to a later definition of Keystone? | 17:52 |
richm | ayoung: gfidente says that you cannot notify pacemaker via a puppet notify | 17:52 |
gfidente | jistr, faux service provider ... | 17:52 |
richm | ayoung: what jrist is proposing is better | 17:52 |
jistr | richm: btw jrist vs jistr :)) | 17:53 |
jrist | nothing I'm not used to | 17:53 |
ayoung | jistr, I learned the diff between you two the hard way | 17:53 |
jistr | yeah me too :) | 17:53 |
jistr | lol | 17:53 |
jistr | ayoung: everybody does it seems :) | 17:53 |
ayoung | jistr should be read as Jester | 17:53 |
richm | jistr: I think you meant gfidente | 17:53 |
ayoung | jrist should be read as Jay Wrist | 17:54 |
jrist | for the record I'm older than jistr | 17:54 |
jrist | so I think I came first? | 17:54 |
ayoung | For the record I'm older than all of you excpet maybe for richm, and even that is close | 17:55 |
jrist | right but we're not comparing your name to mine :) | 17:55 |
jistr | :P | 17:55 |
richm | jistr: sorry, yes, I meant jistr | 17:55 |
ayoung | jrist, when I first stared on openstack, everyone thought I as Andrew Young. But his irc was "sleepsonthefloor" | 17:55 |
devvesa | dprince: please remember reviews. | 17:56 |
gfidente | so if the keystone module was notifying class[::apache:service] instead of service[keystone] it should work? | 17:56 |
devvesa | My managers are pushing me to have the changes merged by tuesday... | 17:57 |
ayoung | gfidente, I thought that was what I was doing, but apparently not | 17:57 |
richm | gfidente: Does notifying class[::apache:service] restart httpd? If not, what exactly does it do? | 17:57 |
*** devvesa has quit IRC | 17:57 | |
jistr | richm, ayoung: btw we don't want to necessarily notify pacemaker from puppet anyway, as some config changes can mean a two-way dependency between service restarts. They need to be restarted at the same time, or things break. | 17:57 |
*** athomas has joined #tripleo | 17:58 | |
gfidente | richm it'd let the class decide, based on the service_manage thing | 17:58 |
ayoung | jistr, agreed. And this is all initial configuration stuff. We should not start/stop/restart 1000000 times | 17:58 |
gfidente | and pcmk is pretty good in maintaining that relationship amongst the services | 17:59 |
gfidente | let's give it what has to be given | 17:59 |
ayoung | This is why I really don't like the all-in-one approach. While Keystone should be running in apache, it really should be able to vary separately from the resot of openstack. Should be in its own container | 17:59 |
ayoung | would make upgrades much easier | 17:59 |
ayoung | and allow people to run later keystones than the rest of openstack... | 17:59 |
*** ifarkas has quit IRC | 18:00 | |
gfidente | ayoung, so we have also other issues with restarting httpd | 18:00 |
gfidente | it's going to affect other services, poentially more and more while the apis all move to wsgi | 18:00 |
ayoung | gfidente, what is in Apache besides Keystone and Horizon? | 18:00 |
gfidente | and I think we wanted to explore the possibility of running the wsgis in some lightweight server which we would spawn in multiple copies anf configurations, depending on service | 18:01 |
jistr | ayoung: currently nothing i think, but everybody plans to move there, at least that's what the rumor is :) | 18:01 |
ayoung | Can HTTPD be told to re-read config without restarting? | 18:02 |
ayoung | Seems to me that is the heart of the problem | 18:02 |
gfidente | I think running an isolated server also works better with containers | 18:03 |
ayoung | according to this yes http://www.cyberciti.biz/faq/apache-2-reload-httpd-config-file-unix-linux-command/ | 18:03 |
gfidente | but on the other hand, -HUP is a good interim solution | 18:03 |
gfidente | ayoung, for now, I would just try noticing ::apache::service | 18:04 |
ayoung | gfidente, here? https://github.com/openstack/puppet-keystone/blob/master/manifests/wsgi/apache.pp#L168 | 18:04 |
gfidente | yes | 18:05 |
gfidente | jistr, ^^ ? | 18:05 |
richm | will that work both with and without pacemaker? | 18:05 |
*** shardy has quit IRC | 18:05 | |
jistr | i don't think this is guaranteed to work with pacemaker | 18:05 |
jistr | what are we trying to achieve anyway? i don't think i noticed it in the conversation, sorry | 18:06 |
gfidente | richm, yes in non-pacemaker service_manage is true | 18:06 |
ayoung | jistr, run Keystone in HTTPD instead of eventlet | 18:06 |
richm | is someone going to file a bug against puppet-keystone? | 18:06 |
ayoung | http://adam.younglogic.com/2012/03/keystone-should-move-to-apache-httpd/ We are coing up on 4 years, so I figured it was time | 18:07 |
ayoung | richm, I can do that | 18:07 |
richm | ayoung: ok - please add me to cc or just send me a link | 18:07 |
ayoung | richm, can you take a tuick look at https://bugs.launchpad.net/puppet-keystone and se if it is actually covered under one of the exsitng? | 18:08 |
ayoung | "when WSGI enabled: domain service is unavailable during first Puppet run" almost seems to be it | 18:08 |
*** dtantsur is now known as dtantsur|afk | 18:08 | |
jistr | gfidente: i don't know what notifying ::apache::service does, not *that* much of a puppeteer. It might do the service restart. | 18:08 |
jistr | but i don't think any of this is going to play well with pacemaker | 18:09 |
jistr | we can just prevent it from freaking out if we put it into maintenance mode, which we do for stack-updates | 18:09 |
gfidente | jistr, I think I've seen it working for rabbitmq module | 18:11 |
richm | ayoung: no, that bug is the "set default domain bootstrap problem" | 18:12 |
richm | ayoung: looks like this problem isn't covered by an existing bug | 18:12 |
ayoung | richm, the whole "we don't know the id until after we create it" cycle? | 18:12 |
richm | ayoung: yes | 18:13 |
ayoung | richm, https://bugs.launchpad.net/puppet-keystone/+bug/1524927 | 18:13 |
openstack | Launchpad bug 1524927 in puppet-keystone "Puppet keystone cannot restart HTTPD managed by Pacemaker" [Undecided,New] | 18:13 |
ayoung | I used jaosorior 's comment to populate | 18:13 |
richm | ayoung: looks good - thanks! | 18:13 |
ayoung | richm, I have a git checkout of the tripleo heat templates that I was running with. I'll roll back to master there, and then, I want to try with that change to the puppe modules | 18:15 |
ayoung | I assume I can clone them to the undercloud stack users dir and run overcloud deploy pointing at them somehow? | 18:15 |
openstackgerrit | Ryan Hallisey proposed openstack/tripleo-heat-templates: Convert JSON generations from bash to python https://review.openstack.org/253135 | 18:17 |
richm | ayoung: If you're talking to me, I have no idea | 18:17 |
ayoung | richm, you were not testing them via tripleo deploys, I take it then | 18:18 |
richm | ayoung: I've never even touched tripleo - any puppet-keystone testing I did was with spec or packstack | 18:20 |
*** dsneddon has quit IRC | 18:20 | |
ayoung | richm, OK...I'll figure this out. I know jistr has made something like this work | 18:21 |
patrickeast | Hi, maybe a simple question... I'm trying to add in a new package to my nodes so I've added in a definition like package { 'device-mapper-multipath': ensure => installed}" to the compute.pp manifest thats in tripleo-heat-templates. I'm seeing in the log that puppet is picking up the package http://paste.openstack.org/show/481544/ | 18:21 |
patrickeast | any reason why the package then would not actually be installed? | 18:21 |
dprince | jpeeler, trown|lunch: this looks fine, but have we added the actual RDO package update yet? https://review.openstack.org/#/c/254885/ | 18:22 |
dprince | thrash: ^^ | 18:22 |
jpeeler | dprince: my understanding is the delorean change had to happen afterwards - https://review.gerrithub.io/#/c/254527/ | 18:23 |
jistr | ayoung, richm: openstack overcloud deploy --templates <path-to-templates> | 18:23 |
ayoung | jistr, for puppet modules | 18:23 |
ayoung | jistr, I need to change https://github.com/openstack/puppet-keystone/blob/master/manifests/wsgi/apache.pp#L168 | 18:23 |
jistr | ohh, i've had that working some time ago via rsync but that was in devtest times, and i don't think i got it to work after moving to instack, for some reason | 18:24 |
*** trown|lunch is now known as trown | 18:24 | |
jistr | i've heard someone saves them to swift and fetches them (instead of rsync), but i haven't seen the code/snippets to make this happen yet | 18:24 |
*** dsneddon has joined #tripleo | 18:25 | |
dprince | jpeeler, thrash, trown okay, I commented on https://review.gerrithub.io/#/c/254527/2. I think we could probably handle this in the .spec file to get CI passing. I've done similar things in the tripleo-heat-templates packaging before... | 18:25 |
trown | dprince: the packaging update is waiting on that change you just reviewed :P | 18:25 |
jistr | ayoung: also you can use virt-customize or virt-edit to edit the image, then `openstack overcloud image upload --update-existing` | 18:25 |
jistr | but that can get heavy handed if you do a lot of changes | 18:26 |
jistr | sorry to not have a good workflow here ATM | 18:26 |
*** dsneddon has quit IRC | 18:26 | |
dprince | trown: ideally I think we'd do it the other way around. In other words make the packaging change work with both the old and new upstream code. | 18:26 |
*** dsneddon has joined #tripleo | 18:26 | |
dprince | trown: a bit odd in the packaging world, yes but I think it is fine. Plus we can always drop any extra code afterwards. | 18:26 |
jistr | dprince: did you do deploy-time OPM updates via swift? | 18:26 |
trown | dprince: I think for the /* idea we should get opinions on #rdo | 18:27 |
ayoung | jistr, hmmmm | 18:27 |
*** dsneddon has quit IRC | 18:27 | |
trown | dprince: merging that stub in tripleo-common with failing CI was meant to be the workaround | 18:27 |
ayoung | jistr, when building the overcload images, we use the puppet modules as they are installed on the hypervisor? | 18:28 |
*** dsneddon has joined #tripleo | 18:28 | |
dprince | trown: right, I'd rather see CI passing first though | 18:28 |
trown | dprince: the real solution here is to have packaging in the same gerrit as the code so we can do depends-on ... | 18:28 |
gfidente | ayoung, I think that alternatively you can use something like virt-customize to edit the overcloud image, reupload in undercloud glance, and reattempt deployment | 18:28 |
dprince | trown: I think we can do this | 18:28 |
dprince | trown: exactly | 18:28 |
dprince | trown: that is the real solution | 18:28 |
dprince | trown: without it we have to do package munging | 18:29 |
dprince | trown: which I've been doing in t-h-t here: https://github.com/openstack-packages/tripleo-heat-templates | 18:29 |
jistr | ayoung: hmm i don't know for sure but i don't think so. I think it's going to ensure that it's up to date with the repo location and ref, usually master branch. | 18:29 |
trown | dprince: but I am not super opinionated on which less ideal solution to go with | 18:29 |
trown | dprince: I am not even the maintainer of tripleo-common so even less opinionated :) | 18:30 |
* jistr in conversations for longer than intended, it's 7:30 PM, probably should go home :)) | 18:30 | |
dprince | trown: given this only affects tripleo* packages I think it is fine to treat the packages this way for now | 18:30 |
ayoung | jistr, this call right here https://github.com/jistr/inlunch/blob/master/answers.yml.example#L96 | 18:30 |
ayoung | /home/stack/tripleo-common/scripts/tripleo.sh --overcloud-images | 18:30 |
dprince | jistr: I need more context. Swift updates for puppet? | 18:30 |
ayoung | jistr, where does that get executed | 18:31 |
jistr | dprince: yeah update a puppet module (puppet-keystone for ayoung / richm use case) on deploy-time, without rebuilding the image | 18:31 |
jistr | ayoung: on undercloud | 18:31 |
dprince | jistr: oh, yeah. That is all done | 18:31 |
dprince | jistr: reviews are posted, just stalled out in the review process now | 18:32 |
thrash | dprince: i did add a .spec review | 18:32 |
dprince | thrash: right, I commented on it | 18:33 |
thrash | dprince: thanks | 18:33 |
dprince | jistr: https://blueprints.launchpad.net/tripleo/+spec/puppet-modules-deployment-via-swift | 18:34 |
jistr | i see | 18:34 |
jistr | https://review.openstack.org/#/c/245310/ | 18:34 |
dprince | jistr: the mechanism would now support generic deployment of artifacts | 18:34 |
jistr | https://review.openstack.org/#/c/245314 | 18:34 |
dprince | jistr: so either a tarball (my favorite) or an RPM if you prefer | 18:34 |
jistr | tarball ftw for dev work | 18:34 |
dprince | jistr: exactly | 18:35 |
jistr | i used to use this https://github.com/openstack/tripleo-heat-templates/blob/master/firstboot/userdata_dev_rsync.yaml | 18:35 |
dprince | jistr: this is super useful to me for composable roles work BTW (moving things out of t-h-t --> puppet-tripleo) | 18:35 |
jistr | yeah | 18:35 |
dprince | jistr: upload new puppet-modules tarball (contains etc/puppet/modules/*) | 18:35 |
dprince | jistr: then heat stack-update | 18:36 |
dprince | boom | 18:36 |
jistr | yeah the firstboot only worked for stack-create iirc | 18:36 |
jistr | ok, i think i'll explore the swift way, would be good to converge on a single way to do these things | 18:36 |
dprince | jistr: yep, check it out and see what you think | 18:37 |
jistr | ayoung, richm ^^ | 18:37 |
* jistr leaving for real now :) | 18:37 | |
jistr | o/ | 18:37 |
dprince | jistr: see ya | 18:38 |
*** jistr has quit IRC | 18:38 | |
*** nico_auv has quit IRC | 18:38 | |
*** olap has quit IRC | 18:38 | |
ayoung | jistr, dprince we have swift in the undercloud, right ? | 18:39 |
* ayoung neophyte | 18:39 | |
dprince | ayoung: yes, and we currently use it for very little | 18:39 |
dprince | ayoung: introspection data goes there and that is it really | 18:40 |
*** masco has quit IRC | 18:40 | |
dprince | ayoung: there are however patches posted to make Glance use it as a backend, which would potentially allow image data to go directly from Swift -> IPA (our deployment ramdisk) skipping the conductor conversion bit in Ironic | 18:41 |
jpeeler | thrash: you leaning towards putting that wildcard in? | 18:44 |
*** shivrao_ has joined #tripleo | 18:44 | |
thrash | jpeeler: I'm currently debating that with myself. :) | 18:44 |
thrash | or trying to figure out how the conditional stuff would work. | 18:44 |
jpeeler | i say do it! | 18:44 |
thrash | jpeeler: the wildcard? | 18:45 |
jpeeler | yep | 18:45 |
dprince | ayoung: and then the new idea of using it to deploy tarballs/RPMs is now posted as well | 18:45 |
jpeeler | thrash: easy for a packager to remove it and explicitly list files if something needs to not go in the RPM. but chances are that won't happen in my opinion | 18:45 |
thrash | jpeeler: agreed. | 18:46 |
ayoung | dprince, I get it, but I think it does not help me trying to do it from git. Just adds another place I need to stick them | 18:46 |
*** Marga_ has quit IRC | 18:46 | |
*** Marga_ has joined #tripleo | 18:47 | |
openstackgerrit | Merged openstack/tripleo-common: Ensures plan exists https://review.openstack.org/255400 | 18:48 |
ayoung | dprince, It seems to me "A Swift Temp URL" could be replaced with a git repo and a tag, hash, or branch. | 18:48 |
ayoung | dprince, can I git push to Swift? I'm only 1/3 joking | 18:49 |
dprince | ayoung: yeah, so if we'd stuck with the initial interface I had then yes. Initially the Heat parameter was called PuppetModulesURL | 18:50 |
dprince | ayoung: and the deployment script would know where to deploy said module, be it a git URL, or a swift temp URL tarball | 18:50 |
*** apetrich has joined #tripleo | 18:50 | |
dprince | ayoung: but now (per comments in the spec) the mechanism is "generic". We *can* deploy RPMs now though | 18:50 |
dprince | ayoung: so it is a trade off | 18:50 |
dprince | ayoung: the difficulty with the generic ArtifactsURL deployment is you don't know what they are, or where to install them if it is a git repo | 18:51 |
dprince | ayoung: so the question here is really do we specialize our Heat parameters for Puppet modules? or keep them generic. Most want them generic I think | 18:52 |
jpeeler | thrash: unforunately, it still fails CI. forgot that it doesn't solve the chicken egg problem | 18:54 |
ayoung | dprince, I'm still wrpping my head around what the mechanism flow is here. I can't quite figure out where jistr's Ansible playbook is generating the images. I assume that the scriopt it calls to do that is then getting the puppet modules from somewhere, but I don't want to guess where | 18:54 |
openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: TripleO API v1 https://review.openstack.org/243737 | 18:54 |
openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Add environment capabilities https://review.openstack.org/232534 | 18:54 |
openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Create basic API layout https://review.openstack.org/255798 | 18:54 |
*** shivrao_ has quit IRC | 18:54 | |
jpeeler | d0ugal: would you now be okay with approving https://review.openstack.org/#/c/254885/ ? | 18:56 |
ayoung | openstack overcloud image build --all | 18:56 |
ayoung | --tripleo-puppet-elements | 18:57 |
*** jcoufal has quit IRC | 18:58 | |
*** dprince has quit IRC | 18:58 | |
*** dprince has joined #tripleo | 18:59 | |
thrash | jpeeler: thought so... It's complaining that nothing in in %{_bindir} | 19:00 |
rhallisey | has only ever run into an issue with the triplo client polling heat forever even after heat is 'CREATE_COMPLETE'? | 19:01 |
rhallisey | s/only/anyone/ | 19:01 |
trown | thrash: dprince, I think we should just merge the stub as originally planned... In the extremely unlikely chance it breaks CI we can revert | 19:02 |
*** shivrao has joined #tripleo | 19:03 | |
*** tosky has quit IRC | 19:03 | |
jpeeler | trown: the stub actually has two +2s | 19:03 |
thrash | trown: I think leaving the %{_bindir}/* should be fine though | 19:05 |
jpeeler | +1 | 19:05 |
thrash | once we have *one* script in there, it alleviates future chicken-egg problems. | 19:05 |
trown | thrash: ya I do not have a problem with that | 19:05 |
jpeeler | so who can approve the stub patch? | 19:06 |
thrash | d0ugal: ^^^? | 19:06 |
thrash | theoretically I can +A it, but I hate doing that on my own patch. | 19:07 |
jpeeler | i'm imagine d0ugal is done for the day if he's still in the UK | 19:07 |
jpeeler | really hoping we can get this sorted like... nowish | 19:07 |
*** ukalifon1 has joined #tripleo | 19:09 | |
trown | jpeeler: thrash it would probably be good for dprince to +A if he agrees... he was the one with objections to that approach | 19:10 |
thrash | trown: ack | 19:10 |
dprince | trown: link me the patch again? | 19:10 |
trown | dprince: https://review.openstack.org/#/c/254885/ | 19:11 |
dprince | trown: commented | 19:12 |
*** bvandenh has quit IRC | 19:13 | |
jpeeler | thrash: ok i guess you're stuck figuring out the conditional. | 19:13 |
trown | dprince: the * trick does not work though with 0 files | 19:14 |
trown | it works with any number of files >0 | 19:14 |
*** Marga_ has quit IRC | 19:15 | |
*** mgould has quit IRC | 19:15 | |
*** ukalifon1 has quit IRC | 19:16 | |
openstackgerrit | Dan Sneddon proposed openstack/tripleo-heat-templates: Add IPv6 Support to Isolated Networks https://review.openstack.org/235423 | 19:16 |
*** gfidente has quit IRC | 19:21 | |
*** Marga_ has joined #tripleo | 19:25 | |
dprince | trown: can we put a conditional around it then? | 19:26 |
*** ukalifon has joined #tripleo | 19:26 | |
thrash | jpeeler: dprince trown have a possibility on the conditional | 19:26 |
thrash | I don't have a working delorean setup locally, so I'll push it up and see what it does. :) | 19:26 |
thrash | it rpmlint's fine. :) | 19:26 |
*** bvandenh has joined #tripleo | 19:27 | |
*** shivrao has quit IRC | 19:32 | |
*** shivrao_ has joined #tripleo | 19:32 | |
thrash | heyyooo | 19:32 |
*** olap has joined #tripleo | 19:32 | |
thrash | dprince: trown jpeeler https://review.gerrithub.io/#/c/254527/ passed | 19:33 |
thrash | lmkwyt | 19:33 |
jpeeler | yes, can trown approve it? | 19:33 |
trown | thrash: thanks I will +A | 19:33 |
jpeeler | one step closer! | 19:33 |
thrash | trown: theoretically, we can drop the 'add only' commit. | 19:34 |
thrash | aka collapse the two. | 19:34 |
thrash | trown: should I collapse them? | 19:34 |
trown | thrash: ya now we should be able to... I would like to remove that conditional once this all resolved as well... that is a bit ugly | 19:34 |
thrash | trown: agreed, but i'd rather have the ugly be in the packaging than in the upstream repo. :) | 19:35 |
thrash | let me collapse them. | 19:35 |
jpeeler | another rebase for me then | 19:35 |
trown | thrash: ya as a temporary hack to get past chicken egg problem I have no issue with it | 19:36 |
openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: Deploy Monitoring on the undercloud with Puppet https://review.openstack.org/253709 | 19:37 |
openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: Manage keystone related configuration via Puppet https://review.openstack.org/234704 | 19:37 |
openstackgerrit | Brad P. Crochet proposed openstack/tripleo-common: Use the new yaml syntax in CI https://review.openstack.org/254886 | 19:37 |
openstackgerrit | Brad P. Crochet proposed openstack/tripleo-common: Build image files from definitions in yaml https://review.openstack.org/235569 | 19:37 |
thrash | jpeeler: base your change on this: https://review.openstack.org/#/c/235569/ | 19:38 |
thrash | not the CI one. | 19:39 |
thrash | trown: jpeeler squashed and rebased.. Let's see what happens. :) | 19:39 |
openstackgerrit | greghaynes proposed openstack/diskimage-builder: Add dib-python element https://review.openstack.org/256071 | 19:39 |
*** electrofelix has quit IRC | 19:39 | |
jpeeler | thrash: thanks, yeah hopefully all is well and these patches get merged soon too | 19:40 |
thrash | jpeeler: hold off on your rebase. | 19:40 |
thrash | jpeeler: got a requirement mismatch. | 19:40 |
thrash | well.. | 19:41 |
thrash | hmmm | 19:41 |
*** ibravo has quit IRC | 19:42 | |
thrash | gonna ignore that for now. | 19:44 |
jpeeler | why not just go ahead and bump them | 19:45 |
thrash | alrighty | 19:46 |
openstackgerrit | Derek Higgins proposed openstack/instack-undercloud: Change undercloud replica count to 1 https://review.openstack.org/255995 | 19:47 |
openstackgerrit | Brad P. Crochet proposed openstack/tripleo-common: Use the new yaml syntax in CI https://review.openstack.org/254886 | 19:47 |
openstackgerrit | Brad P. Crochet proposed openstack/tripleo-common: Build image files from definitions in yaml https://review.openstack.org/235569 | 19:47 |
jpeeler | i mean, the job will restart immediately. so you won't lose that much testing time | 19:47 |
jpeeler | thrash: hopefully my involvement has not been annoying on all this | 19:48 |
thrash | jpeeler: not at all. :) | 19:48 |
jpeeler | good | 19:48 |
*** Marga_ has quit IRC | 19:50 | |
*** rcernin has joined #tripleo | 20:00 | |
openstackgerrit | greghaynes proposed openstack/diskimage-builder: Run package-intalls with py3k if we must https://review.openstack.org/254386 | 20:02 |
greghaynes | SpamapS: pino|work ^ coded up what we chatted about | 20:03 |
openstackgerrit | Jeff Peeler proposed openstack/tripleo-common: Upload docker image files from yaml config file https://review.openstack.org/247724 | 20:04 |
thrash | jpeeler: trown will check on status when I get home. | 20:09 |
jpeeler | thrash: appreciate it - i have high hopes though | 20:09 |
*** shivrao_ has quit IRC | 20:12 | |
openstackgerrit | greghaynes proposed openstack/diskimage-builder: Add a new element hpdsa https://review.openstack.org/220888 | 20:12 |
*** Marga_ has joined #tripleo | 20:12 | |
*** shivrao has joined #tripleo | 20:14 | |
*** lucasagomes is now known as lucas-beer | 20:19 | |
*** ibravo has joined #tripleo | 20:26 | |
*** bvandenh has quit IRC | 20:28 | |
*** sthillma has joined #tripleo | 20:29 | |
*** athomas has quit IRC | 20:32 | |
*** lucas-beer has quit IRC | 20:34 | |
*** dprince has quit IRC | 20:37 | |
*** lucasagomes has joined #tripleo | 20:40 | |
*** ibravo has quit IRC | 20:40 | |
*** mcornea has quit IRC | 20:42 | |
*** sthillma has quit IRC | 20:44 | |
*** Hazelesque_ has joined #tripleo | 20:47 | |
*** Hazelesque has quit IRC | 20:47 | |
*** jaosorior has quit IRC | 20:47 | |
*** jaosorior has joined #tripleo | 20:47 | |
*** cwolferh has quit IRC | 20:53 | |
*** sthillma has joined #tripleo | 20:58 | |
*** akrivoka has quit IRC | 21:04 | |
*** rebrego has joined #tripleo | 21:05 | |
*** regebro has quit IRC | 21:05 | |
*** rebrego is now known as regebro | 21:05 | |
openstackgerrit | greghaynes proposed openstack/diskimage-builder: Run package-intalls with py3k if we must https://review.openstack.org/254386 | 21:05 |
trown | derekh: I would assume all of the tripleo jobs would fail on the undercloud install issue you found if they were not all failing on 'No package git-hg available' during delorean setup | 21:06 |
trown | derekh: I thought we fixed that a while ago... | 21:06 |
*** Hazelesque_ has quit IRC | 21:10 | |
*** cwolferh has joined #tripleo | 21:10 | |
*** ukalifon has quit IRC | 21:13 | |
*** Hazelesque has joined #tripleo | 21:13 | |
*** julim has quit IRC | 21:14 | |
*** sthillma has quit IRC | 21:17 | |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: pacemaker: run neutron-server-start-wait-stop only at step 4 https://review.openstack.org/256099 | 21:24 |
openstackgerrit | Merged openstack/diskimage-builder: Add proliant-tools element https://review.openstack.org/163754 | 21:24 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: pacemaker: run neutron-server-start-wait-stop only at step 4 https://review.openstack.org/256099 | 21:24 |
openstackgerrit | Merged openstack/diskimage-builder: Follow up patch for 25d3ee547176528e86d42eb026c99a134dff9452 https://review.openstack.org/251879 | 21:25 |
openstackgerrit | Dan Sneddon proposed openstack/tripleo-heat-templates: Add IPv6 Support to Isolated Networks https://review.openstack.org/235423 | 21:25 |
*** rpothier has quit IRC | 21:37 | |
openstackgerrit | Merged openstack/diskimage-builder: Add a new element hpdsa https://review.openstack.org/220888 | 21:46 |
*** jayg is now known as jayg|g0n3 | 21:57 | |
*** mcornea has joined #tripleo | 22:01 | |
*** alop has quit IRC | 22:03 | |
openstackgerrit | Ryan Brady proposed openstack/tripleo-common: Fixes enabled bug https://review.openstack.org/254844 | 22:05 |
*** lblanchard has quit IRC | 22:05 | |
*** derekh has quit IRC | 22:07 | |
*** shivrao has quit IRC | 22:09 | |
*** shivrao has joined #tripleo | 22:10 | |
*** shakamunyi has quit IRC | 22:12 | |
*** penick has joined #tripleo | 22:12 | |
*** dsneddon has quit IRC | 22:12 | |
*** mcornea has quit IRC | 22:13 | |
*** trown is now known as trown|outttypeww | 22:14 | |
*** shakamunyi has joined #tripleo | 22:15 | |
*** dsneddon has joined #tripleo | 22:16 | |
*** dsneddon has quit IRC | 22:18 | |
*** dsneddon has joined #tripleo | 22:18 | |
*** david-lyle has joined #tripleo | 22:20 | |
*** derekh has joined #tripleo | 22:24 | |
*** yamahata has quit IRC | 22:24 | |
*** yamahata has joined #tripleo | 22:24 | |
*** Goneri has quit IRC | 22:32 | |
*** thrash is now known as thrash|g0ne | 22:38 | |
*** pradk has quit IRC | 22:40 | |
openstackgerrit | Merged openstack/tripleo-common: Don't install git-hg https://review.openstack.org/255970 | 22:40 |
*** rcernin has quit IRC | 22:45 | |
*** alop has joined #tripleo | 22:51 | |
*** david-lyle has quit IRC | 22:55 | |
openstackgerrit | Ryan Hallisey proposed openstack/tripleo-heat-templates: Remove hack the pulls latest docker https://review.openstack.org/256129 | 23:00 |
*** openstackstatus has quit IRC | 23:01 | |
*** rhallisey has quit IRC | 23:12 | |
*** alop has quit IRC | 23:12 | |
*** apetrich has quit IRC | 23:16 | |
*** yuanying has joined #tripleo | 23:18 | |
*** morazi has quit IRC | 23:26 | |
*** derekh has quit IRC | 23:35 | |
openstackgerrit | Derek Higgins proposed openstack/instack-undercloud: Change undercloud replica count to 1 https://review.openstack.org/255995 | 23:40 |
*** weshay has quit IRC | 23:40 | |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: [NO MERGY] Test a update in trunk repository version https://review.openstack.org/229789 | 23:43 |
*** sthillma has joined #tripleo | 23:49 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!