*** nijaba has joined #tripleo | 00:02 | |
*** nijaba has quit IRC | 00:02 | |
*** nijaba has joined #tripleo | 00:02 | |
*** nijaba_ has quit IRC | 00:03 | |
*** gfidente has quit IRC | 00:12 | |
*** zaneb has quit IRC | 00:15 | |
*** zaneb has joined #tripleo | 00:16 | |
*** shadower has quit IRC | 00:23 | |
*** rlandy has quit IRC | 00:23 | |
*** shadower has joined #tripleo | 00:23 | |
*** MasterPiece has quit IRC | 00:27 | |
*** sthillma has quit IRC | 00:30 | |
*** thrash is now known as thrash|g0ne | 00:31 | |
*** yamahata has quit IRC | 01:27 | |
*** rhallisey has quit IRC | 01:31 | |
*** chlong has joined #tripleo | 01:46 | |
*** adrianopetrich has quit IRC | 01:50 | |
*** chlong_ has joined #tripleo | 01:53 | |
*** yog_ has joined #tripleo | 01:53 | |
*** yog_ has quit IRC | 02:09 | |
*** yamahata has joined #tripleo | 02:18 | |
*** rlandy has joined #tripleo | 02:30 | |
*** david-lyle has joined #tripleo | 02:34 | |
*** shivrao has joined #tripleo | 02:37 | |
*** yog_ has joined #tripleo | 02:38 | |
*** chlong_ has quit IRC | 02:50 | |
*** untriaged-bot has joined #tripleo | 03:00 | |
untriaged-bot | No untriaged bugs so far! \o/ | 03:00 |
---|---|---|
*** untriaged-bot has quit IRC | 03:00 | |
*** yog_ has quit IRC | 03:08 | |
*** yog_ has joined #tripleo | 03:09 | |
*** sthillma has joined #tripleo | 03:10 | |
*** sthillma_ has joined #tripleo | 03:11 | |
*** sthillma has quit IRC | 03:15 | |
*** sthillma_ is now known as sthillma | 03:15 | |
*** penick has joined #tripleo | 03:40 | |
*** olaph has quit IRC | 03:41 | |
*** rlandy has quit IRC | 03:43 | |
*** yog_ has quit IRC | 03:50 | |
*** sdake_ has joined #tripleo | 03:51 | |
*** yog_ has joined #tripleo | 03:52 | |
*** sdake has quit IRC | 03:54 | |
*** yamahata has quit IRC | 04:03 | |
*** shivrao has quit IRC | 04:14 | |
*** chlong has quit IRC | 04:16 | |
*** olaph has joined #tripleo | 04:21 | |
*** tzumainn has quit IRC | 04:29 | |
*** chlong has joined #tripleo | 04:32 | |
*** sthillma has quit IRC | 04:40 | |
*** sdake has joined #tripleo | 04:46 | |
*** chlong has quit IRC | 04:48 | |
*** sdake_ has quit IRC | 04:49 | |
*** penick has quit IRC | 05:10 | |
*** shivrao has joined #tripleo | 05:14 | |
*** adrianopetrich has joined #tripleo | 05:28 | |
openstackgerrit | imain proposed openstack/tripleo-heat-templates: Docker compute role configured via Puppet https://review.openstack.org/209505 | 05:30 |
openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Allow for usage of pre-allocated IPs for the Controller nodes https://review.openstack.org/222497 | 05:51 |
*** gfidente has joined #tripleo | 05:51 | |
*** jaosorior has joined #tripleo | 05:54 | |
*** ukalifon has joined #tripleo | 06:00 | |
*** lsmola has joined #tripleo | 06:01 | |
*** shivrao has quit IRC | 06:02 | |
*** chlong has joined #tripleo | 06:17 | |
*** yamahata has joined #tripleo | 06:28 | |
*** jprovazn has joined #tripleo | 06:31 | |
*** akrivoka has joined #tripleo | 07:02 | |
*** jaosorior has quit IRC | 07:04 | |
*** akrivoka has quit IRC | 07:08 | |
*** akrivoka has joined #tripleo | 07:16 | |
*** jaosorior has joined #tripleo | 07:20 | |
*** shardy has joined #tripleo | 07:25 | |
*** yamahata has quit IRC | 07:26 | |
openstackgerrit | Merged openstack-infra/tripleo-ci: Use a specific fedora mirror on the jenkins node https://review.openstack.org/223868 | 07:26 |
*** dtantsur|afk is now known as dtantsur | 07:27 | |
*** aufi has joined #tripleo | 07:31 | |
*** hashar has joined #tripleo | 07:35 | |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Use a specific mirror for CentOS builds https://review.openstack.org/224000 | 07:37 |
shardy | Hey folks, what's the status wrt landing t-h-t changes atm, are we holding off due to CI issues still? | 07:39 |
shardy | I see quite a lot of patches with recent green CI that can probably land, so wanted to check before pushing them in.. | 07:39 |
*** shardy_ has joined #tripleo | 07:47 | |
*** shardy has quit IRC | 07:47 | |
*** shardy_ has quit IRC | 07:51 | |
*** shardy has joined #tripleo | 07:52 | |
*** Marga_ has quit IRC | 07:53 | |
*** ifarkas has joined #tripleo | 07:59 | |
-openstackstatus- NOTICE: Gate is currently stuck, failing grenade upgrade tests due the release of oslo.utils 1.4.1 for Juno. | 08:02 | |
*** paramite has joined #tripleo | 08:07 | |
*** jistr has joined #tripleo | 08:20 | |
*** jaosorior has quit IRC | 08:23 | |
*** jaosorior has joined #tripleo | 08:24 | |
*** lucasagomes has joined #tripleo | 08:29 | |
openstackgerrit | Jan Provaznik proposed openstack/tripleo-specs: Spec for running overcloud nodes reboot https://review.openstack.org/224481 | 08:32 |
*** derekh has joined #tripleo | 08:33 | |
*** mcornea has joined #tripleo | 08:34 | |
*** Marga_ has joined #tripleo | 08:39 | |
*** paramite is now known as paramite|afk | 08:44 | |
*** paramite|afk is now known as paramite | 08:46 | |
*** Marga_ has quit IRC | 08:47 | |
*** untriaged-bot has joined #tripleo | 09:00 | |
untriaged-bot | No untriaged bugs so far! \o/ | 09:00 |
*** untriaged-bot has quit IRC | 09:00 | |
openstackgerrit | Derek Higgins proposed openstack/diskimage-builder: Download a compressed centos cloud images https://review.openstack.org/224155 | 09:04 |
*** paramite is now known as paramite|afk | 09:05 | |
*** jaosorior has quit IRC | 09:05 | |
*** jaosorior has joined #tripleo | 09:09 | |
*** athomas has joined #tripleo | 09:17 | |
*** pelix has joined #tripleo | 09:27 | |
*** paramite|afk is now known as paramite | 09:28 | |
jistr | one more reviewer is needed for derekh's CI stability improvements | 09:41 |
jistr | https://review.openstack.org/#/c/223996 | 09:41 |
jistr | https://review.openstack.org/#/c/223997/ | 09:42 |
jistr | https://review.openstack.org/#/c/224000/ | 09:42 |
*** beagles has joined #tripleo | 09:44 | |
openstackgerrit | Yanis Guenane proposed openstack/tripleo-heat-templates: Allow one to specify horizon ALLOWED_HOSTS https://review.openstack.org/221255 | 09:44 |
*** beagles is now known as Guest73647 | 09:44 | |
pino|work | jistr: will that be enough? | 09:46 |
jistr | pino|work: yes :) | 09:48 |
jistr | also this one just passed CI and will be helpful too, esp. as we pin to a single mirror https://review.openstack.org/224155/ | 09:49 |
*** paramite is now known as paramite|afk | 10:05 | |
*** Guest73647 is now known as b3nt_pin | 10:13 | |
openstackgerrit | Antoine Musso proposed openstack/diskimage-builder: Prettify 'Caches and offline mode' documentation https://review.openstack.org/224554 | 10:16 |
-openstackstatus- NOTICE: Gate back to normal, thanks to the backlisting of the problematic version | 10:16 | |
shardy | derekh: Hey, I was wondering what the status is wrt approving t-h-t patches, are we holding off due to CI issues? | 10:20 |
shardy | derekh: there's a bunch of patches that look green except for gate-tripleo-ironic-overcloud-f21puppet-ha | 10:20 |
shardy | it'd be good to start clearing the backlog soon, but I don't want to introduce more entropy unneccesarily ;) | 10:20 |
openstackgerrit | Merged openstack/diskimage-builder: Don't create a centos yum repository https://review.openstack.org/223996 | 10:23 |
derekh | shardy: As of tuesday morning the HA job has been mostly passing (as much as the others have at least), when did it fail on those patches? | 10:24 |
derekh | shardy: officially its non voting but if it was after tuesday, it would be nice to see a green result (not necessarily all green all at the same time) | 10:25 |
shardy | derekh: I'm looking at stuff like https://review.openstack.org/#/c/213142/ https://review.openstack.org/#/c/214740/ https://review.openstack.org/#/c/215694/ and https://review.openstack.org/#/c/223271/ | 10:26 |
shardy | derekh: there are a few failures (all yesterday) in either the ha or ceph jobs, but mostly they look ready to land AFAICS | 10:26 |
derekh | shardy: looking | 10:26 |
shardy | the last patch is all green, which I think proves the series of all three nexus/allnodes patches | 10:26 |
derekh | shardy: takeing this first one here https://review.openstack.org/#/c/214740 , is the different between PS11 and PS12 just a rebase ? | 10:32 |
shardy | derekh: just a rebase | 10:35 |
shardy | I think gerrit shows trivial rebases from ps10 actually as the votes have carried over | 10:36 |
derekh | shardy: ok, given that you have the passing ci test in PS11 (and its non voting ) I think its reasonable to say its ok to a CI point of view, I'll take a proper look now and see if I can add a +2 | 10:36 |
shardy | derekh: great, thanks! | 10:37 |
*** dtantsur is now known as dtantsur|lunch | 10:55 | |
*** lsmola has quit IRC | 11:01 | |
*** timbyr_ has joined #tripleo | 11:02 | |
*** lsmola has joined #tripleo | 11:02 | |
*** timbyr_ has quit IRC | 11:03 | |
*** matbu_ has joined #tripleo | 11:17 | |
*** matbu has quit IRC | 11:20 | |
openstackgerrit | Antoine Musso proposed openstack/diskimage-builder: Prettyfy source-repositories doc https://review.openstack.org/224578 | 11:25 |
*** hashar has quit IRC | 11:25 | |
*** dprince has joined #tripleo | 11:28 | |
dprince | marios: hi, are you still +2 here? https://review.openstack.org/#/c/204781/ | 11:28 |
slagle | jroll: lucasagomes : you've got your +A here: https://review.openstack.org/#/c/209079/, but it's failing devstack check/gate. it's passed tripleo | 11:29 |
lucasagomes | slagle, oh lemme take a look | 11:30 |
slagle | jroll: lucasagomes : leaving it up to you all to look at the failures and push it through | 11:30 |
lucasagomes | slagle, ack | 11:30 |
marios | dprince: looking | 11:30 |
marios | dprince: oh, yeah, probably not. i mean, it will fail when no network isolation, so we should probably find a way to work around it | 11:31 |
marios | dprince: sorry, i updated the BZ but forgot to add note there, are you planning to update? | 11:31 |
dprince | marios: it should pass regardless though, and it actually worked in CI... it is enabled for all configurations | 11:31 |
marios | dprince:so, i only just got my centos setup going yesterday(and still haven't managed to deploy ha yet). the issue was with all downstream env | 11:32 |
dprince | marios: no update yet, I actually think in your case the validation was probably just working correct | 11:32 |
dprince | marios: and something about your networking wasn't correct | 11:32 |
marios | dprince: sec... lemme find the bz | 11:32 |
marios | dprince: no, it is a created port that doesn't get used in non network isolation case (controller externalport) https://bugzilla.redhat.com/show_bug.cgi?id=1246596 | 11:34 |
openstack | bugzilla.redhat.com bug 1246596 in openstack-tripleo-heat-templates "Add support for network validation tests" [High,On_qa] - Assigned to dprince | 11:34 |
*** weshay_xchat has quit IRC | 11:35 | |
*** weshay_xchat has joined #tripleo | 11:37 | |
*** sdake_ has joined #tripleo | 11:43 | |
*** sdake has quit IRC | 11:47 | |
marios | dprince: so i'm waiting for a deply to finish on my centos env, will try applying this locally in a minute | 11:48 |
*** paramite|afk is now known as paramite | 11:49 | |
*** trown|outttypeww is now known as trown | 11:55 | |
*** jayg|g0n3 is now known as jayg | 11:57 | |
*** thrash|g0ne is now known as thrash | 12:01 | |
*** rhallisey has joined #tripleo | 12:03 | |
*** tzumainn has joined #tripleo | 12:08 | |
openstackgerrit | Merged openstack/tripleo-heat-templates: Add YAML sanity check https://review.openstack.org/223766 | 12:09 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/diskimage-builder: Updated from global requirements https://review.openstack.org/224606 | 12:11 |
trown | EmilienM: good morning! for https://review.openstack.org/#/c/223690/ ... How do you decide what config options should go on the class? i.e. if we are just using upstream default, then do we need any options directly on the class? | 12:13 |
*** sdake_ is now known as sdake | 12:13 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-apply-config: Updated from global requirements https://review.openstack.org/193995 | 12:15 |
EmilienM | trown: you are ironic expect | 12:15 |
EmilienM | expert* | 12:15 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-cloud-config: Updated from global requirements https://review.openstack.org/193997 | 12:15 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-collect-config: Updated from global requirements https://review.openstack.org/193998 | 12:15 |
trown | EmilienM: I am thinking maybe just credentials/URLs and making them required ... wdyt? | 12:15 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-refresh-config: Updated from global requirements https://review.openstack.org/192302 | 12:15 |
EmilienM | so you decide which param we will eventually need to change | 12:15 |
EmilienM | that's why puppet is cool, you decide the interface | 12:16 |
EmilienM | you can put 1000 params or 5 | 12:16 |
trown | EmilienM: right, but with the inspector_config type, the interface is already totally flexible | 12:16 |
EmilienM | I suggest keeping focus on what we actually need for a real production deployment | 12:16 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-tuskarclient: Updated from global requirements https://review.openstack.org/194031 | 12:17 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/tuskar: Updated from global requirements https://review.openstack.org/186939 | 12:17 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/tuskar-ui: Updated from global requirements https://review.openstack.org/224694 | 12:17 |
trown | EmilienM: ok...I think I should probably keep what I have then, since I took them directly from instack-undercloud bash element, I will change defaults back to upstream defaults though | 12:18 |
trown | EmilienM: the convention of setting passwords to false and having them "optional" is a bit odd to me | 12:19 |
trown | what is the downside to setting them to required? inspector will not work without setting them | 12:19 |
EmilienM | trown: have you investigated how work other Puppet OpenStack modules? | 12:21 |
trown | EmilienM: I looked at puppet-neutron and puppet-ironic and they do it this way of false,optional | 12:22 |
openstackgerrit | Merged openstack/tripleo-heat-templates: network validation to ping test each interface https://review.openstack.org/204781 | 12:23 |
trown | EmilienM: actually looking at the api.pp in ironic, it does have a password as required | 12:23 |
trown | EmilienM: cool, I will do it that way...thanks for your help | 12:24 |
*** paramite is now known as paramite|afk | 12:25 | |
EmilienM | trown: cool | 12:27 |
*** Goneri has joined #tripleo | 12:28 | |
*** jaosorior has quit IRC | 12:31 | |
*** weshay_xchat is now known as weshay | 12:32 | |
openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Add validation to ping the default gateway https://review.openstack.org/204806 | 12:33 |
*** lblanchard has joined #tripleo | 12:33 | |
*** paramite|afk is now known as paramite | 12:34 | |
jistr | dprince: hi, i was looking at this https://review.openstack.org/#/c/221881/2/puppet/controller-config-pacemaker.yaml | 12:35 |
jistr | i'm thinking we could have a separate hiera file for the pacemaker stuff in the end | 12:35 |
jistr | if we pass through the 'servers' parameter to the ControllerPuppetConfig here https://github.com/openstack/tripleo-heat-templates/blob/9096f375fff7b0a6e8d015d8313da0fdbff41457/puppet/controller-post-puppet.yaml#L16-L17 | 12:36 |
*** morazi has joined #tripleo | 12:37 | |
jistr | then we could create a new hiera similarly like we do for the ControllerExtraConfigPre resources, e.g. to implement 3rd party plugins | 12:37 |
jistr | like here for example https://review.openstack.org/#/c/213142/8/puppet/extraconfig/pre_deploy/controller/neutron-ml2-bigswitch.yaml | 12:37 |
jistr | dprince: do you think such approach is worth pursuing for the pacemaker bits? (i could look into it) | 12:38 |
*** mcornea has quit IRC | 12:38 | |
*** hashar has joined #tripleo | 12:38 | |
openstackgerrit | Merged openstack/tripleo-heat-templates: Big Switch Neutron ML2 plugin integration https://review.openstack.org/213142 | 12:39 |
jistr | dprince: i'm a bit discouraged from the direct approach of feeding the params via 'inputs' directly to a SoftwareConfig resource | 12:39 |
jistr | it's more verbose in code, and it's less neatly structured on the actual deployed server, than if we'd have a separate hiera file | 12:40 |
dprince | jistr: good points | 12:40 |
jistr | i can look into that if you don't want to spend time on it | 12:40 |
dprince | jistr: however it does give us a nice way to overlay (compose) the hiera | 12:40 |
dprince | jistr: perhaps there is a way to solve the verbosity issues... | 12:41 |
dprince | jistr: gotta run an errand, back in a bit if you'd like to chat more about this | 12:42 |
jistr | dprince: sure :) | 12:42 |
openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Rename -puppet.yaml templates. https://review.openstack.org/203452 | 12:43 |
*** dprince has quit IRC | 12:47 | |
pino|work | jistr: https://review.openstack.org/#/c/223997/ worth pushing again? | 12:47 |
*** dtantsur|lunch is now known as dtantsur | 12:48 | |
jistr | pino|work: i think it's already running, i see it in zuul | 12:49 |
pino|work | \o/ | 12:49 |
jistr | pino|work: seems like it started both check and verify jobs (i don't know why that's done but ok) | 12:49 |
pino|work | maybe because you wrote "reverify"? /me dunno's | 12:49 |
jistr | pino|work: yeah but i thought it would just run verify jobs, not check | 12:50 |
*** julim has joined #tripleo | 12:50 | |
pino|work | check all the things! | 12:50 |
jistr | ah perhaps the reason is that verify jobs are more "expensive" | 12:50 |
jistr | in terms of serialization, iirc | 12:50 |
jistr | well anyway, it should be on its way if i see correctly | 12:50 |
*** pradk has joined #tripleo | 12:58 | |
trown | dtantsur: I put all of the config options back to upstream defaults here https://review.openstack.org/#/c/223690/ | 12:59 |
dtantsur | cool! | 12:59 |
trown | dtantsur: I also removed managing dnsmasq...I cant see a way to handle PXE way without making iPXE way a pain in instack | 13:00 |
trown | I think it is fair to say "manage your own tftp mechanism" | 13:00 |
dtantsur | trown, I think eventually we need it configurable with a switch, but not in this patch | 13:01 |
trown | dtantsur: I went back and forth on requiring passwords/urls...but if we did that it would break using this for noauth setups | 13:02 |
dtantsur | yeah | 13:02 |
trown | hence the "optional"/false | 13:02 |
*** rlandy has joined #tripleo | 13:04 | |
openstackgerrit | Gael Chamoulaud proposed openstack/tripleo-heat-templates: Enable Manila integration https://review.openstack.org/188137 | 13:11 |
dtantsur | trown, won't "$processing_hooks = ['ramdisk_error','scheduler','validate_interfaces']" set anything? | 13:15 |
trown | dtantsur: yes, but that is the upstream default | 13:16 |
dtantsur | trown, it might change any time, and will change soon. can we NOT set it at all, just leave as it is? | 13:16 |
dtantsur | trown, also upstream default is "$default_processing_hooks" | 13:17 |
openstackgerrit | Gael Chamoulaud proposed openstack/tripleo-heat-templates: Enable Manila integration https://review.openstack.org/188137 | 13:17 |
openstackgerrit | Gael Chamoulaud proposed openstack/tripleo-heat-templates: Add NetApp integration to Manila https://review.openstack.org/188138 | 13:17 |
openstackgerrit | Merged openstack/diskimage-builder: Add centos7 support for DIB_DISTRIBUTION_MIRROR https://review.openstack.org/223997 | 13:17 |
*** jprovazn has quit IRC | 13:17 | |
trown | dtantsur: ah good catch on the name...I changed it in the comment but not the actual name... as far as the actual question, I am not opposed to removing it from the class | 13:18 |
trown | dtantsur: but it seems like something alot of people would want to change. no? | 13:18 |
trown | dtantsur: fwiw, it is totally possible to change it without it being set on the class | 13:19 |
dtantsur | trown, not really a lot. I'm not even sure that we will want to change it | 13:19 |
dtantsur | ok, good, that works for me | 13:19 |
dtantsur | (hmm, we probably will. but default use case is perfectly fine without changing this option) | 13:19 |
trown | ok, I will remove that from the class then. Is that the only option I should remove? | 13:20 |
dtantsur | lemme check | 13:21 |
dtantsur | trown, keep_ports is the same as default, so it's useless. also dnsmasq_interface assumes tripleo, not sure if it's a good thing | 13:22 |
trown | dtantsur: both are the same as upstream though... my understanding is that we put things directly on the class that many users of the module would like to change | 13:23 |
trown | but we set the value to upstream default | 13:23 |
dtantsur | trown, hmm, I'm fine with these 2 | 13:24 |
trown | it is a bit odd, because really we could put nothing on the class and just do everything via ironic_inspector_config type... | 13:24 |
trown | but I think putting nothing there would be bad from a puppet best practices pov | 13:25 |
dtantsur | I see | 13:25 |
trown | dtantsur: but really... http://goo.gl/8wj2zZ | 13:25 |
dtantsur | LOL | 13:26 |
trown | whats a ruby | 13:26 |
*** lblanchard has quit IRC | 13:27 | |
openstackgerrit | Merged openstack-infra/tripleo-ci: Use a specific mirror for CentOS builds https://review.openstack.org/224000 | 13:30 |
*** openstackgerrit has quit IRC | 13:31 | |
*** openstackgerrit has joined #tripleo | 13:31 | |
pino|work | jistr: all the three landed, i think | 13:34 |
jistr | yup \o/ | 13:34 |
jistr | thanks derekh for making those :) | 13:34 |
*** julim has quit IRC | 13:37 | |
derekh | jistr: no prob, thanks for reviews | 13:38 |
*** paramite is now known as paramite|afk | 13:38 | |
pino|work | jistr: in https://review.openstack.org/#/c/211535/, there's no way to group the "which foo && foo ..." toghether? | 13:39 |
pino|work | like "which foo && { ... }" or with subshells | 13:39 |
derekh | dare I say it, I think our reliability should be a bit better now with the ci stuff that has merged in the last 2 days, only one left https://review.openstack.org/#/c/224155/ | 13:40 |
derekh | next up, figure out why docker fails to start sometimes, https://trello.com/c/WOBpS7L7/32-figure-out-whats-up-with-docker-not-starting | 13:41 |
jistr | pino|work: could be, not sure if it's so much better given that those commands are run only once, except pcs which is run three times | 13:45 |
*** dprince has joined #tripleo | 13:46 | |
*** jprovazn has joined #tripleo | 13:47 | |
*** jpeeler has joined #tripleo | 13:49 | |
*** jpeeler has quit IRC | 13:49 | |
*** jpeeler has joined #tripleo | 13:49 | |
jistr | dprince: do you have bandwidth to continue the pacemaker/hiera chat? | 13:50 |
dprince | jistr: yes, lets chat | 13:53 |
jistr | dprince: cool. could you explain what you meant "it does give us a nice way to overlay (compose) the hiera"? | 13:54 |
openstackgerrit | Merged openstack-infra/tripleo-ci: Remove code to pull clone repositories https://review.openstack.org/222406 | 13:54 |
*** rpothier has joined #tripleo | 13:55 | |
dprince | jistr: right, what I meant was if you are trying to reuse or share one of the role manifests. Say overcloud_controller.pp or whatever | 13:55 |
dprince | jistr: adding 'inputs' directly on the Software config gives you a point to override any hiera variables used to execute that | 13:56 |
dprince | jistr: this is because of this http://git.openstack.org/cgit/openstack/tripleo-heat-templates/tree/puppet/controller-puppet.yaml#n898 | 13:57 |
dprince | jistr: that heat_config_%{::deploy_config_name} overrides all other hiera settings, and was intended exactly for this purpose | 13:57 |
*** absubram has joined #tripleo | 13:57 | |
jistr | dprince: hmm i actually consider this a drawback of that solution | 13:58 |
jistr | dprince: anything set by pacemaker config would now be unoverridable by ExtraConfig / controllerExtraConfig | 13:58 |
dprince | jistr: then perhaps a simple re-ordering of our hiera would help fix this | 13:59 |
dprince | jistr: so you would propose using ExtraConfig for this instead? | 14:00 |
dprince | jistr: I view ExtraConfig as something that is user specified | 14:01 |
*** egafford has joined #tripleo | 14:01 | |
jistr | dprince: not as a solution for the pacemaker problem, there's too much that would have to go there | 14:01 |
dprince | jistr: so we should allow that to override things, I think we can | 14:01 |
dprince | do it | 14:01 |
jistr | dprince: yeah i agree re ExtraConfig, my only concern is that we should keep things overridable by the user | 14:01 |
dprince | jistr: if we simply put controller_extraconfig, and extraconfig first I think we'd be okay | 14:02 |
jistr | still trying to think through "what I meant was if you are trying to reuse or share one of the role manifests. Say overcloud_controller.pp or whatever" | 14:02 |
dprince | jistr: sorry, I suppose my end goal here would be to... if at all possible get us back down to a single controller template | 14:03 |
dprince | jistr: the ability to specifiy all of the extra pacemaker stuff with an overlay mechanism like this would be a perfect solution | 14:03 |
dprince | jistr: the patch we are talking about here is only a slight step in that direction... but it demonstrates the approach | 14:03 |
dprince | jistr: your points about it being verbose are totally valid though | 14:04 |
jistr | dprince: right, with a new hiera file this would still be possible afaik, only thing we need is that the hiera hierarchy includes an entry... so we have "controller_pacemaker" in the hierarchy right above "controller" | 14:04 |
jistr | trying to think if such a solution has a drawback vs. the direct inputs | 14:05 |
dprince | jistr: okay, if a new hiera file works I'm fine w/ it | 14:05 |
dprince | jistr: I wasn't sure the hiera sections would merge though | 14:05 |
*** julim has joined #tripleo | 14:05 | |
dprince | jistr: ideally we wouldn't have to duplicate them | 14:05 |
dprince | jistr: I feel another nested stack resource might be in order here ;) | 14:06 |
* dprince feels partially to blame for the nested stack explosion in t-h-t | 14:06 | |
dprince | jistr: totally open to ideas here on this, if you have an alternative that you think works better I'm happy to go with it | 14:08 |
dprince | jistr: my patch was just a suggestion based on an idea I had w/ the docker approach | 14:08 |
jistr | dprince: cool, i'll try to explore this solution. Nested stack resource could be a solution probably (not sure where exactly you meant it right now), but i hope we can do it quite simply. Lemme try to come up with something and we can hash out the details on a review, it's going to be easier to talk over some actual code. | 14:08 |
*** paramite|afk is now known as paramite | 14:09 | |
*** panda has quit IRC | 14:09 | |
jistr | dprince: thx :) | 14:10 |
*** panda has joined #tripleo | 14:10 | |
openstackgerrit | Brad P. Crochet proposed openstack/python-tripleoclient: Remove baremetal client code in favor of upstream client https://review.openstack.org/224744 | 14:10 |
dprince | jistr: cool, thanks | 14:11 |
jistr | and +1 for trying to get back to a single controller manifest, that would be neat | 14:12 |
trown | congrats dprince on a well run election campaign :) | 14:17 |
trown | https://wiki.openstack.org/wiki/PTL_Elections_September_2015#Confirmed_Candidates | 14:17 |
trown | same to EmilienM | 14:19 |
dprince | trown: thanks | 14:19 |
EmilienM | trown: thanks | 14:19 |
openstackgerrit | Brad P. Crochet proposed openstack/python-tripleoclient: Remove baremetal client code in favor of upstream client https://review.openstack.org/224744 | 14:22 |
openstackgerrit | Brad P. Crochet proposed openstack/python-tripleoclient: Turn the orchestration client reference into a property https://review.openstack.org/224752 | 14:25 |
*** mcornea has joined #tripleo | 14:27 | |
*** lblanchard has joined #tripleo | 14:42 | |
openstackgerrit | Pino Toscano proposed openstack-infra/tripleo-ci: [WIP] Generate overcloud image outside the undercloud https://review.openstack.org/224166 | 14:43 |
thrash | slagle: what's the status of d0ugal and I added to the cores list? | 14:47 |
*** penick has joined #tripleo | 14:49 | |
openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Support new form of servers config for ML2 cisco nexus allnodes config. https://review.openstack.org/223271 | 14:50 |
openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Add "AllNodes" ExtraConfig interface https://review.openstack.org/214740 | 14:50 |
openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Port Cisco Nexus/UCSM ExtraConfig to AllNodes https://review.openstack.org/215694 | 14:50 |
openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Allow for usage of pre-allocated IPs for the Controller nodes https://review.openstack.org/222497 | 14:54 |
*** mburned has quit IRC | 14:54 | |
dprince | gfidente: hi, any progress/thoughts on the validation script? | 14:55 |
slagle | thrash: i was going to give folks a week to comment. | 14:55 |
slagle | which i think is today | 14:55 |
gfidente | dprince, hey no sorry, haven't started that yet | 14:55 |
thrash | slagle: ok. Just checking. :) | 14:55 |
thrash | slagle: And I was trying to give you a week before I bugged you. lol | 14:56 |
slagle | thrash: no worries, thanks for reminding me | 14:56 |
*** mburned has joined #tripleo | 14:57 | |
*** penick has quit IRC | 14:58 | |
*** paramite has quit IRC | 14:59 | |
*** Marga_ has joined #tripleo | 14:59 | |
*** untriaged-bot has joined #tripleo | 15:00 | |
untriaged-bot | No untriaged bugs so far! \o/ | 15:00 |
*** untriaged-bot has quit IRC | 15:00 | |
*** Marga_ has quit IRC | 15:00 | |
*** Marga_ has joined #tripleo | 15:01 | |
slagle | d0ugal: what is your gerrit email? | 15:01 |
slagle | gerrit is telling me Account Not Found: dougal@redhat.com | 15:02 |
slagle | even though it auto-completed that for me :) | 15:02 |
slagle | dmatthew@redhat.com? | 15:02 |
slagle | yea there we go | 15:02 |
slagle | thrash: d0ugal : you're added to core. make sure CI has passed on patches before you approve :) | 15:04 |
*** mburned has quit IRC | 15:04 | |
*** mburned has joined #tripleo | 15:10 | |
*** ukalifon has quit IRC | 15:10 | |
*** aufi has quit IRC | 15:11 | |
d0ugal | slagle: Awesome, thanks :) | 15:15 |
openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Rename -puppet.yaml templates. https://review.openstack.org/203452 | 15:15 |
d0ugal | slagle: hmm, I should be dougal@redhat.com | 15:18 |
d0ugal | slagle: Just checked the settings | 15:18 |
d0ugal | I wonder if I have two accounts somehow | 15:18 |
dprince | shardy, bnemec: just a rebase, you guys still +2 on https://review.openstack.org/#/c/203452/ | 15:30 |
thrash | slagle: awesome, thanks! | 15:32 |
shardy | dprince: yup, it'd be good to clean that up before we start landing e.g docker trees | 15:32 |
dprince | shardy: exactly, would be nice to get it in | 15:33 |
dprince | shardy: unfortanely each rebase drops the +2's on this one | 15:34 |
shardy | dprince: it'll be good to start clearing the backlog, quite a lot of stuff has been around for a while now | 15:35 |
dprince | shardy: yep, trying to work through things | 15:35 |
dprince | shardy: a lot of comments aren't exactly clear... we've had an odd mix of downstream CI with some of the upstream patches | 15:36 |
dprince | shardy: I'm going to speak w/ derekh again soon about re-spinning the testenv's so we can properly CI some of the network isolation stuff | 15:36 |
trown | thrash: I am honored that https://review.openstack.org/#/c/222223/ is your first +A :) | 15:36 |
trown | thanks! | 15:36 |
thrash | trown: :) | 15:37 |
shardy | dprince: sounds good, I should have more time to help out next week, been fully loaded with pre-rc1 heat stuff this week | 15:38 |
shardy | I'm particularly interested in reviving the release branch discussion to see if there's a way we can feasably support installing stable releases from Liberty | 15:38 |
dprince | shardy: cool, would be nice to have that in the mix | 15:39 |
egafford | slagle, trown: Success on installing an upstreamish overcloud from the tripleo dev docs. :) As a note, though, the current instack flow does install rdomanager-oscplugin rather than tripeloclient, which poses a moderate challenge in terms of upstream dev when it touches the client (new passwords, etc.) | 15:40 |
trown | egafford: nice one, ya that should be a simple doc bug | 15:41 |
derekh | dprince: all for it, we should be able todo it with no downtime | 15:41 |
derekh | trown: no, I *think* some packages are pulling it in as a dep | 15:42 |
trown | egafford: ironic-inspector is now in delorean http://trunk.rdoproject.org/centos7/report.html which fixes the RDO is breaking tripleo issue | 15:42 |
dprince | derekh: cool, let me push the need tripleo-ci stuff. I think we'll need one patch there first. | 15:42 |
derekh | dprince: ok | 15:42 |
trown | derekh: there is no provides in the tripleoclient spec? | 15:42 |
trown | derekh: ohh...you are right...that is not so good | 15:43 |
trown | I can fix that | 15:43 |
dprince | shardy: this would be helpful https://review.openstack.org/#/c/222549/ | 15:43 |
derekh | trown: nope, I've been meaning to circle back and do it but others stuff took my attention | 15:43 |
derekh | trown: cool, go for it | 15:43 |
egafford | trown: What's the workaround to tripleoclient v rdomanager-oscplugin? Simple pip install or something deeper? Also, nice: good deal (re: delorean) | 15:44 |
dprince | shardy: In your domain man. I know you love to review puppet stuff :) | 15:44 |
dprince | slagle: thanks for clarification on https://review.openstack.org/#/c/204806/. I will try and accomidate those suggestions | 15:44 |
thrash | anyone know how we can get jenkins to apply updated jobs on old reviews? | 15:45 |
dprince | slagle: FWIW I don't mind if you co-author any of these things too if you see fit for it | 15:45 |
trown | egafford: well I have just been installing python-tripleoclient in place of python-rdomanager-oscplugin and it has worked for me | 15:45 |
d0ugal | thrash: tried a rebase? | 15:45 |
openstackgerrit | Merged openstack/python-tripleoclient: DIB includes the "in-tree" elements by default https://review.openstack.org/222223 | 15:46 |
trown | egafford: but I will submit a package change that will make stuff that depends on rdomanager-oscplugin install tripleoclient instead | 15:46 |
dprince | derekh: could you review this one. I think it should help lucas w/ some testing with regards to Ironic stuff https://review.openstack.org/#/c/194161/ | 15:46 |
thrash | d0ugal: i have not... | 15:46 |
dprince | derekh: he has a patch depending on it there for some time | 15:46 |
*** ryansb has quit IRC | 15:46 | |
dprince | derekh: or maybe it moved... I don't see it anymore | 15:46 |
dprince | derekh: either than or I'm abandoning this entirely because only a few people still use it now I think | 15:48 |
derekh | dprince: looking....isn't incubaor dead.... | 15:48 |
shardy | dprince: sure, let me check the status of our heat-templates gate now, it was broken yesterday | 15:48 |
dprince | derekh: yeah, I think just a few people still use it for odd local testing of things | 15:48 |
derekh | dprince: it wont do any harm, lemme take a quit look | 15:48 |
*** ryansb has joined #tripleo | 15:48 | |
*** ryansb has joined #tripleo | 15:48 | |
dprince | derekh: incubator isn't quite dead until we get instack bits to do similar sorts of things for local setup | 15:49 |
dprince | derekh: or at least I do anyways | 15:49 |
thrash | no idea why the py27 check is failing on a mock file open... | 15:51 |
dprince | gfidente: have you done any work on the validation script for this series https://review.openstack.org/#/c/222189/ | 15:51 |
gfidente | dprince, nope | 15:51 |
dprince | gfidente: given the complexity of the external load balancer I would have serious concerns about landing it without one | 15:51 |
dprince | gfidente: i.e. very little confidence it would ever work without clear validations for an end user... | 15:52 |
dprince | gfidente: I mean it could work. but this stuff is way trickier than most things and easy to get wrong | 15:52 |
trown | egafford: derekh, I think I see the issue...there is no python-rdomanager in liberty delorean, I think that is getting pulled in because we enable kilo-release in the docs | 15:52 |
gfidente | dprince, yes | 15:52 |
gfidente | dprince, so I thought to leave validation at the end | 15:53 |
gfidente | to have a clear picture of what should be 'validated' | 15:53 |
gfidente | I got a custom env file which makes testing not-so-terrible | 15:53 |
dprince | gfidente: yep, totally fine. Just wondering if we should potentially block the series until we have that | 15:53 |
openstackgerrit | Brad P. Crochet proposed openstack/python-tripleoclient: Cleanup some strangling references to rdomanager-oscplugin https://review.openstack.org/221785 | 15:54 |
gfidente | dprince, actually the static assignment of ips is testable in CI easily | 15:54 |
gfidente | that would be the topmost change | 15:54 |
openstackgerrit | Brad P. Crochet proposed openstack/python-tripleoclient: Stop false positive validation warning https://review.openstack.org/221940 | 15:55 |
dprince | gfidente: you mean: pre-configured external LB Ips rights? | 15:55 |
trown | derekh: egafford ...actually that package is not in rdo-release-kilo either... where is it getting pulled in from | 15:55 |
dprince | gfidente: "static IPs" is a different thing that this I think... | 15:56 |
trown | we should still do provides/obsoletes, but I dont see where the old package is coming in | 15:56 |
gfidente | dprince, yes the mechanism which uses from_pool.yaml | 15:56 |
gfidente | to pick the ips from a list | 15:56 |
gfidente | can be tested standalone with a small env file | 15:56 |
dprince | gfidente: sure, what I'd want the validation script to do is test that the externally configured VIPs actually work | 15:57 |
gfidente | yeah exactly the other half of the work | 15:57 |
dprince | gfidente: early enough that we don't see failures in provisioning from things like MySQL or Rabbit | 15:57 |
gfidente | ack | 15:57 |
dprince | gfidente: rather we get a clear error message that says "your external VIPs are misconfigured" | 15:57 |
trown | egafford: nevermind found it... we removed it from delorean, but the instructions link to an old repo that still has it | 15:57 |
trown | lastest delorean would not have that issue | 15:58 |
dprince | gfidente: otherwise... someone will use this and thing the installer is broken when everything is just fine there, it is just an end user config issue | 15:58 |
egafford | trown: atm I'm using the delorean in toci (figured that'd have a decent chance of success.) | 15:58 |
egafford | trown: If i use delorean/current, do I have a reasonable chance of successfully fielding an overcloud atm? | 15:58 |
gfidente | dprince, ack I'll try working on a validation script as well as soon as the series is stable | 15:59 |
*** Goneri has quit IRC | 15:59 | |
trown | egafford: for some definition of reasonable...I have been testing on current, but only to the point of undercloud install | 15:59 |
egafford | trown: Heh. Is there a reasonable way I can hack the planet and map in tripleoclient to my current env? | 16:00 |
*** mcornea has quit IRC | 16:00 | |
egafford | I'm pretty happy with the degree to which my unmodified overcloud currently exists. | 16:00 |
trown | egafford: you could install just tripleoclient from the lastest delorean directly | 16:00 |
egafford | It is fully 100% more existent than any previous upstream overcloud. | 16:00 |
trown | yum install http://trunk.rdoproject.org/centos7/78/df/78dfa2ba52b5da3768d66b5bc53e4f341a911406_37a16b8f/python-tripleoclient-0.0.11-dev3.el7.centos.noarch.rpm | 16:01 |
egafford | trown: Cool. Hopefully that'll just beat up rdomanager_oscplugin and take its stuff. | 16:01 |
trown | egafford: not yet... but maybe later today :) | 16:01 |
egafford | trown: Life moves fast in TripleO... | 16:02 |
*** rlandy has quit IRC | 16:03 | |
openstackgerrit | Merged openstack/tripleo-incubator: Drop USE_IRONIC from tripleo-incubator https://review.openstack.org/194161 | 16:04 |
openstackgerrit | Merged openstack/python-tripleoclient: image build: separate the envvars for dib https://review.openstack.org/222539 | 16:04 |
*** dtantsur is now known as dtantsur|afk | 16:06 | |
dprince | seeing CI failures due to 'tuskar plan-add-role' errors | 16:10 |
dprince | BadRequest: There are no parameters named Controller-1::Flavor, Controller-1::CloudName, Ceph-Storage | 16:10 |
*** shivrao has joined #tripleo | 16:10 | |
thrash | d0ugal: any thoughts on why this is failing? (The tests run fine locally...) https://review.openstack.org/#/c/222541/ | 16:10 |
thrash | d0ugal: and I'm talking about the redhat-release part... | 16:11 |
thrash | d0ugal: also looks like we will need to be explicit about python3-jobs | 16:12 |
*** Prophidys has quit IRC | 16:13 | |
trown | derekh: care to review https://review.gerrithub.io/246757 | 16:13 |
*** shivrao_ has joined #tripleo | 16:13 | |
*** hashar has quit IRC | 16:14 | |
*** shivrao has quit IRC | 16:15 | |
*** shivrao_ is now known as shivrao | 16:15 | |
openstackgerrit | Pino Toscano proposed openstack/python-tripleoclient: image build: add support for virt-dib https://review.openstack.org/222541 | 16:17 |
openstackgerrit | Pino Toscano proposed openstack/python-tripleoclient: image build: prepare for different generation methods https://review.openstack.org/222540 | 16:17 |
dprince | jdob: any thoughts on this one https://bugs.launchpad.net/tripleo/+bug/1496919 | 16:18 |
openstack | Launchpad bug 1496919 in tripleo "instack install failures: tuskar plan-add-role: error: argument -r/--role-uuid: expected one argument" [High,Triaged] | 16:18 |
*** rlandy has joined #tripleo | 16:18 | |
jdob | hrm, that command changed so instead of providing a list of roles, it's multiple instances of -r | 16:19 |
jdob | what I don't see is where that's being called incorrectly | 16:19 |
*** sdake_ has joined #tripleo | 16:19 | |
jdob | though it may be because of 2015-09-17 15:48:42.154 | No plan with name 'overcloud' exists. | 16:19 |
dprince | jdob: I think perhaps the issue is the $ROLES in the script isn't getting set correctly | 16:19 |
dprince | jdob: also, one clarification so I don't send you on a wild goose chase, it was this patch (previously passing in CI, perhaps before instack) that I noticed the failure https://review.openstack.org/#/c/203452/ | 16:20 |
dprince | jdob: perhaps there is some hard coded patch in tuskar with regards to these templates I'm unaware of | 16:20 |
*** ifarkas has quit IRC | 16:20 | |
derekh | trown: looking | 16:21 |
jdob | ah, perhaps (i like that patch though, glad to remove the -puppet stuff) | 16:21 |
jdob | it's likely breaking 90-tuskar-db-sync | 16:21 |
trown | derekh: it did not pass CI, sorry for the early ping :( | 16:22 |
jdob | load_roles "$(echo ${ROLES[@]})" $PUPPET_DIR "-puppet.yaml" "$ROLE_EXTRA_DATA" | 16:22 |
jdob | its how tuskar distinguished the role templates from the others | 16:22 |
dprince | jdob: where is 90-tuskar-db-sync? | 16:22 |
dprince | jdob: t-i-e or instack maybe? | 16:23 |
*** sdake has quit IRC | 16:23 | |
jdob | yes, sorry, tie/tuskar/configure.d | 16:23 |
jdob | er, os-refresh-config/configure.d | 16:23 |
*** penick has joined #tripleo | 16:25 | |
dprince | jdob: got it. lost my mind for a minute there and was in the wrong project directory | 16:26 |
dprince | jdob: okay, so I'm going to propose a conditional in the script to properly load the new location | 16:26 |
dprince | jdob: with that approach I think it should work either way | 16:27 |
jdob | sounds good, i'll keep an eye for the review | 16:27 |
slagle | dare i ask if we still need this element at all? | 16:27 |
dprince | jdob: thanks | 16:27 |
*** tzumainn has quit IRC | 16:27 | |
jdob | slagle: depends on when we're officially pulling the plug on tuskar | 16:27 |
slagle | jdob: is there anyone "officially" working on it | 16:28 |
*** pblaho has quit IRC | 16:28 | |
openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Use a separate hiera file for pacemaker deployments https://review.openstack.org/224815 | 16:28 |
slagle | can decided later i suppose | 16:28 |
jistr | dprince: ^^ worked locally :) | 16:28 |
jdob | i think i'd feel more comfortable waiting to delete shit until after the UI is no longer using it at all | 16:28 |
slagle | ok | 16:29 |
*** tzumainn has joined #tripleo | 16:29 | |
jistr | dprince: i'll be heading home but if you find time to do the review, please provide any feedback | 16:29 |
jdob | i think part of me wonders if we'll run out of time and just hack a patch into tuskar to limp through 7.2 | 16:29 |
jdob | er, 8 | 16:29 |
jdob | one of those | 16:29 |
dprince | slagle, jdob: I'm keen to "cut the fat" if we don't need things. I'm also happy to have CI tell me that I've broken something... so having it in CI late is better than never I guess | 16:30 |
dprince | slagle, jdob: for this particular change we can probably leave it in place and update t-i-e for now... | 16:30 |
jistr | dprince: one downside is that to allow the config stack to write an extra hiera file, it needs a reference to the servers, so i added an extra param there. We could get around that e.g. by creating a new "nested stack type" which could be switchable in resource registry, but it doesn't look like it would make things clearer in this case (at least for me, as i see it now :) ). | 16:31 |
* jistr will stop hijacking the conversation and go home :) | 16:31 | |
dprince | jistr: oh, right. The extra server thing was why I avoided this | 16:31 |
jdob | the other thing to keep in mind is that we really should sacrifice a goat or something when we delete tuskar to make sure the evil spirits are fully vanquished | 16:31 |
dprince | jistr: perhaps you've talked me into it... I'll have a look | 16:31 |
jdob | so we need to make sure we have a goat before that review goes up | 16:31 |
jistr | dprince: thx | 16:31 |
derekh | trown: am not 100% sure about my comment, added apevec and number80 to take a look also | 16:33 |
*** sdake has joined #tripleo | 16:34 | |
trown | derekh: I went with what I recently did for discoverd -> inspector, but probably best to have one of those guys take a look | 16:34 |
*** jistr has quit IRC | 16:35 | |
*** david-lyle has quit IRC | 16:36 | |
derekh | trown: ya, I saw it down in a few repositories and not done in others, | 16:36 |
*** ifarkas has joined #tripleo | 16:37 | |
d0ugal | thrash: I think the tests must be runiing on a non-redhat distro? | 16:37 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/diskimage-builder: Updated from global requirements https://review.openstack.org/224606 | 16:38 |
*** sdake_ has quit IRC | 16:38 | |
*** penick has quit IRC | 16:43 | |
derekh | do not merge that update to requirments ^^ | 16:43 |
openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Use a separate hiera file for pacemaker deployments https://review.openstack.org/224815 | 16:43 |
derekh | https://trello.com/c/CzyU5uu0/33-update-pbr-in-the-delorean-build-container | 16:43 |
*** shivrao has quit IRC | 16:44 | |
*** david-lyle has joined #tripleo | 16:47 | |
*** derekh has quit IRC | 16:50 | |
*** lucasagomes has quit IRC | 16:53 | |
openstackgerrit | Dan Prince proposed openstack/tripleo-image-elements: Update tuskar to support renamed heat roles https://review.openstack.org/224827 | 16:58 |
openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Rename -puppet.yaml templates. https://review.openstack.org/203452 | 16:59 |
dprince | jdob, slagle ^^ | 16:59 |
*** david-lyle has quit IRC | 17:03 | |
*** dprince has quit IRC | 17:06 | |
*** dprince has joined #tripleo | 17:08 | |
*** trown is now known as trown|lunch | 17:10 | |
*** Goneri has joined #tripleo | 17:10 | |
*** Marga_ has quit IRC | 17:15 | |
*** Marga_ has joined #tripleo | 17:16 | |
*** lsmola has quit IRC | 17:17 | |
*** ifarkas has quit IRC | 17:31 | |
*** penick has joined #tripleo | 17:34 | |
rhallisey | dprince, slagle jpeeler roll over to dans bluejeans? lol | 17:36 |
dprince | rhallisey: cool | 17:36 |
rhallisey | slagle, oh sorry meant Slower_ | 17:36 |
Slower_ | oh ok | 17:38 |
Slower_ | I want to hear what jpeeler has been up to | 17:38 |
*** julim has quit IRC | 17:48 | |
*** julim has joined #tripleo | 17:48 | |
*** Prophidys has joined #tripleo | 17:49 | |
*** dprince has quit IRC | 17:53 | |
*** trown|lunch is now known as trown | 18:03 | |
*** dprince has joined #tripleo | 18:05 | |
dprince | rhallisey, Slower_ : sorry IRC dropped me. | 18:08 |
dprince | rhallisey, Slower_ : did you answer my previous question? | 18:09 |
rhallisey | dprince, what was it? I don't see it | 18:09 |
dprince | rhallisey: oh I see, IRC really dropped me :) | 18:10 |
dprince | rhallisey, Slower_ : question was does docker provide an easy way to get the files in a container | 18:10 |
openstackgerrit | Ben Nemec proposed openstack/instack-undercloud: Add check for sufficient memory to undercloud install https://review.openstack.org/224852 | 18:11 |
dprince | rhallisey, Slower_ : thinking about modifying our workflow slightly such that we pre-download containers before generating the config files... compile a list of "vanilla" files they have... and pre-create those in a script before the puppet run | 18:11 |
*** pelix has quit IRC | 18:11 | |
dprince | rhallisey, Slower_ : sorry, thinking again about optimizations here. I can't help myself | 18:11 |
rhallisey | dprince, so we could just use volume mounts in a location that containers will always look at | 18:13 |
Slower_ | dprince: or 'docker cp' | 18:13 |
rhallisey | that would be 2 steps though | 18:14 |
rhallisey | but a possibility | 18:14 |
dprince | rhallisey: steps don't bother my so much... just trying to avoid baking as much into the agent container | 18:14 |
dprince | rhallisey: keeping that guy as light as possible would be cool | 18:14 |
*** b3nt_pin has quit IRC | 18:15 | |
dprince | slagle: have you used IPA yet, what do we need to do to enable it? | 18:17 |
dprince | slagle: since the alternative is getting deprecated... | 18:18 |
trown | dprince: I have been working on it | 18:18 |
dprince | trown: nice | 18:18 |
dprince | trown: working well? | 18:18 |
trown | dprince: ya we got inspection ported to it in liberty | 18:19 |
*** sdake has quit IRC | 18:19 | |
trown | so now 1 ramdisk to rule them all, | 18:19 |
*** beagles has joined #tripleo | 18:20 | |
*** beagles is now known as Guest6478 | 18:20 | |
*** athomas has quit IRC | 18:20 | |
slagle | dprince: there was a patch to devtest to switch to ipa. that got abandoned given the switch. lucas said he would work on another for instack | 18:20 |
dprince | slagle: might be nice to still land that one... | 18:21 |
dprince | slagle: as an example | 18:21 |
trown | dprince: https://review.openstack.org/#/c/222231/ will build it with tripleoclient | 18:21 |
trown | or adds support to anyways, it does not get built in the default yet | 18:22 |
dprince | trown: cool | 18:22 |
*** sdake has joined #tripleo | 18:23 | |
slagle | dprince: sure, i think he just abandoned it b/c of the lack of ci coverage, but i didnt say he had to | 18:24 |
slagle | it appears straightforward: https://review.openstack.org/#/c/210461 | 18:24 |
slagle | not sure about delorean packaging | 18:24 |
slagle | trown: do you know about packaging? | 18:24 |
trown | it is packaged in delorean and rawhide | 18:25 |
slagle | awesome! | 18:25 |
trown | the tripleoclient patch builds it from packages too | 18:25 |
*** jprovazn has quit IRC | 18:27 | |
*** yog_ has quit IRC | 18:27 | |
trown | as soon as I get a working undercloud with no undocumented workarounds, my next focus was going to be the switch to IPA | 18:27 |
trown | dprince: ^ | 18:27 |
dprince | trown: great, curious what workarounds are you hitting today? | 18:29 |
trown | dprince: mostly packaging stuff to do with inspector and tripleoclient renames | 18:29 |
trown | but tripleoclient also uses discoverd python client, so I have a bug opened to change it to inspector | 18:30 |
slagle | the one i'm aware of that there's no patch for yet is the enabled=false for ironic-inspector | 18:30 |
dprince | trown: yeah, I skip some of that stuff too, otherwise I'm operational w/ upstream instack | 18:30 |
dprince | trown: would be nice to make some of those things opt-in I think anyways | 18:31 |
trown | slagle: ya I am close to a fix for that... https://review.gerrithub.io/#/c/246757/ is the last piece I think to get everything installed right | 18:31 |
dprince | trown: because not all deployments require them | 18:31 |
*** sdake has quit IRC | 18:31 | |
trown | although tripleoclient still wont work without fixing https://bugs.launchpad.net/tripleo/+bug/1496937 | 18:32 |
openstack | Launchpad bug 1496937 in tripleo "[tripleoclient] discoverd renamed to inspector" [Undecided,In progress] - Assigned to John Trowbridge (trown) | 18:32 |
trown | since there is no discoverd package in the latest delorean | 18:32 |
trown | dprince: there is one more issue that I need to file a bug for, but you would not see it unless you checked swift-api service | 18:33 |
*** sdake has joined #tripleo | 18:33 | |
trown | https://github.com/openstack/ceilometer/commit/191f7bf9ccee33d8444f7dac5c09ceccce72ca29 removed swift proxy ceilometer middleware | 18:34 |
trown | but we are still including it in the pipeline in puppet-stack-config | 18:34 |
trown | so swift-proxy wont start | 18:34 |
slagle | any bets on how many attempts it's actually going to try? :) http://paste.openstack.org/show/466672/ | 18:35 |
trown | lol | 18:36 |
trown | 101 | 18:36 |
*** dprince has quit IRC | 18:37 | |
openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: external loadbalancer: Add net_vip_map_external https://review.openstack.org/222189 | 18:38 |
*** penick has quit IRC | 18:41 | |
openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Make load balancer deployment optional via template param https://review.openstack.org/222305 | 18:42 |
*** jaosorior has joined #tripleo | 18:43 | |
openstackgerrit | Mike Burns proposed openstack/python-tripleoclient: change default to vxlan https://review.openstack.org/224870 | 18:47 |
*** greghaynes has quit IRC | 18:49 | |
*** penick has joined #tripleo | 18:51 | |
*** penick has quit IRC | 18:51 | |
openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Pull tripleo related packages from delorean/current https://review.openstack.org/224168 | 18:53 |
openstackgerrit | Merged openstack-infra/tripleo-ci: Stop building tuskar-ui-extras https://review.openstack.org/222407 | 18:54 |
openstackgerrit | Mike Burns proposed openstack/python-tripleoclient: change default to vxlan https://review.openstack.org/224870 | 18:54 |
*** greghaynes has joined #tripleo | 18:56 | |
*** Goneri has quit IRC | 18:57 | |
*** dprince has joined #tripleo | 19:00 | |
*** Guest6478 has quit IRC | 19:02 | |
*** beagles_ has joined #tripleo | 19:03 | |
*** beagles_ is now known as b3nt_pin | 19:03 | |
*** b3nt_pin is now known as Guest31268 | 19:04 | |
*** Guest31268 has quit IRC | 19:04 | |
*** beagles_ has joined #tripleo | 19:05 | |
*** beagles_ has quit IRC | 19:06 | |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Stop building ironic-discoverd https://review.openstack.org/222408 | 19:06 |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Remove support for mgt-master branches https://review.openstack.org/223599 | 19:06 |
*** akrivoka has quit IRC | 19:10 | |
*** yog_ has joined #tripleo | 19:10 | |
openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: external loadbalancer: Add net_vip_map_external https://review.openstack.org/222189 | 19:11 |
openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Make load balancer deployment optional via template param https://review.openstack.org/222305 | 19:11 |
openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Allow for usage of pre-allocated IPs for the Controller nodes https://review.openstack.org/222497 | 19:11 |
*** david-lyle has joined #tripleo | 19:11 | |
*** olaph has quit IRC | 19:18 | |
openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: external loadbalancer: Add net_vip_map_external https://review.openstack.org/222189 | 19:19 |
openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Pull tripleo related packages from delorean/current https://review.openstack.org/224168 | 19:20 |
*** beagles has joined #tripleo | 19:25 | |
*** david-lyle has quit IRC | 19:25 | |
*** beagles is now known as Guest49018 | 19:25 | |
*** Marga_ has quit IRC | 19:26 | |
*** bvandenh has joined #tripleo | 19:35 | |
*** Guest49018 has quit IRC | 19:37 | |
*** sdake has quit IRC | 19:46 | |
*** yog_ has quit IRC | 19:46 | |
*** yog_ has joined #tripleo | 19:47 | |
openstackgerrit | Tzu-Mainn Chen proposed openstack/tripleo-specs: Spec for TripleO overcloud deployment library and API https://review.openstack.org/219754 | 19:48 |
*** sdake has joined #tripleo | 19:48 | |
*** derekh has joined #tripleo | 19:51 | |
derekh | ci failing >> https://trello.com/c/Rqt7AOQY/34-tripleo-ci-failing-requires-python-ironic-discoverd | 19:51 |
openstackgerrit | Ben Nemec proposed openstack/diskimage-builder: Update rhel7 element readme https://review.openstack.org/222351 | 19:52 |
derekh | I'm on a train, can somebody take a look | 19:52 |
trown | looking | 19:52 |
derekh | trown: thanks, I'm wondering if we maybe need to add to the list of packages we build http://git.openstack.org/cgit/openstack-infra/tripleo-ci/tree/toci_instack.sh#n44 | 19:54 |
openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: external loadbalancer: Add net_vip_map_external https://review.openstack.org/222189 | 19:55 |
trown | derekh: I think it is because we are installing python-rdomanager-plugin | 19:57 |
*** sdake has quit IRC | 19:57 | |
*** sdake has joined #tripleo | 20:00 | |
trown | derekh: also...we should be installing openstack-ironic-inspector instead of openstack-ironic-discoverd | 20:00 |
*** lblanchard has quit IRC | 20:00 | |
derekh | trown: we also got this "pip install python-ironic-inspector-client" , it probably doesn't help | 20:01 |
trown | I think that is the root actually...because we are trying to install openstack-ironic-discoverd alongside python-ironic-inspector-client | 20:02 |
trown | and those are conflicting packages | 20:02 |
trown | derekh: I am noob when it comes to CI, where would I submit a patch? | 20:03 |
derekh | trown: http://git.openstack.org/cgit/openstack-infra/tripleo-ci/tree/ | 20:03 |
derekh | trown: the majority of things happen in this file | 20:03 |
derekh | trown: http://git.openstack.org/cgit/openstack-infra/tripleo-ci/tree/toci_instack.sh | 20:03 |
*** beagles_ has joined #tripleo | 20:03 | |
trown | cool | 20:03 |
*** jaosorior has quit IRC | 20:05 | |
*** beagles_ has quit IRC | 20:08 | |
*** trellooobot has joined #tripleo | 20:10 | |
trellooobot | URGENT TRIPLEO TASKS NEED ATTENTION | 20:10 |
trellooobot | +-------------------------------------+-------------------------------+-------------------+-------------+ | 20:10 |
trellooobot | | Title | URL | Members | Last Active | | 20:10 |
trellooobot | +-------------------------------------+-------------------------------+-------------------+-------------+ | 20:10 |
trellooobot | | tripleo-ci failing : Requires: pyth | https://trello.com/c/Rqt7AOQY | **NEEDS MEMBERS** | 18 min | | 20:10 |
trellooobot | +-------------------------------------+-------------------------------+-------------------+-------------+ | 20:10 |
*** trellooobot has quit IRC | 20:10 | |
derekh | trown: gonna add you to the tripleo trello org and to that card if its ok? so the bot will stop spamming | 20:12 |
trown | sure | 20:12 |
trown | this is a bit of a mess because tripleoclient is actually still using discoverd python module, so none of the "baremetal" commands are going to work even if we fix the package issue | 20:13 |
derekh | trown: if you stop looking into the problem update the card and remove your name then when the bot spams again people will know nobody is looking into it | 20:13 |
trown | fixing tripleoclient was what I was working on when you pinged | 20:13 |
trown | derekh: will do | 20:14 |
trown | to the time machine | 20:14 |
trown | sweet we did not remove instack-ironic-deployment from the packaging | 20:15 |
slagle | i'm not sure i can detect sarcasm | 20:16 |
*** lucasagomes has joined #tripleo | 20:17 | |
*** penick has joined #tripleo | 20:21 | |
*** derekh has quit IRC | 20:21 | |
openstackgerrit | John Trowbridge proposed openstack-infra/tripleo-ci: Fix broken CI due to discoverd/inspector rename https://review.openstack.org/224887 | 20:22 |
trown | slagle: no it is a lifesaver right now ^ | 20:22 |
trown | without those old scripts I would have had to rewrite them myself in toci_instack.sh | 20:22 |
trown | s/rewrite/copy-paste/ but still | 20:25 |
*** Marga_ has joined #tripleo | 20:26 | |
*** bvandenh has quit IRC | 20:26 | |
*** lucasagomes has quit IRC | 20:29 | |
trown | slagle: should I file a CI bug for that ^ against tripleo? for the sake of speed I just submitted without a bug | 20:29 |
slagle | trown: sure, probably would be helpful | 20:30 |
slagle | i dont think i fully grok what started the failure | 20:30 |
slagle | did a change in the packaging spec just land or something? | 20:31 |
*** julim has quit IRC | 20:31 | |
*** penick has quit IRC | 20:31 | |
trown | ya, I changed packaging for inspector and inspector-client | 20:31 |
trown | but CI is installing discoverd from the rdo-management repo | 20:32 |
trown | and pip installing the inspector-client | 20:32 |
trown | tl;dr it is a mess, but hopefully that patch fixes most of it | 20:33 |
trown | just need to finish the patch to fix tripleoclient | 20:33 |
slagle | ok | 20:33 |
slagle | was mostly just curious what it just started failing, but that explains it :) | 20:33 |
slagle | *why | 20:33 |
*** MasterPiece has joined #tripleo | 20:37 | |
openstackgerrit | James Slagle proposed openstack/tripleo-heat-templates: Use already Deployed/Installed servers https://review.openstack.org/222772 | 20:42 |
*** lucasagomes has joined #tripleo | 20:44 | |
*** MasterPiece has quit IRC | 20:46 | |
trown | slagle do instack-undercloud bugs also get filed against tripleo? I think we are actually hitting something else that I have a workaround for. | 20:47 |
trown | there is an rdomanager element being included in all the defaults | 20:48 |
*** jayg is now known as jayg|g0n3 | 20:49 | |
slagle | trown: yes, if it's affecting tripleo | 20:54 |
openstackgerrit | John Trowbridge proposed openstack/instack-undercloud: Remove outdated elements https://review.openstack.org/224897 | 20:56 |
*** penick has joined #tripleo | 20:57 | |
*** penick has quit IRC | 20:59 | |
*** untriaged-bot has joined #tripleo | 21:00 | |
untriaged-bot | Untriaged bugs so far: | 21:00 |
untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1497008 | 21:00 |
openstack | Launchpad bug 1497008 in tripleo "instack-undercloud includes outdated elements" [Undecided,In progress] - Assigned to John Trowbridge (trown) | 21:00 |
untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1497004 | 21:00 |
openstack | Launchpad bug 1497004 in tripleo "[CI] failing because of discoverd to inspector rename" [Undecided,In progress] - Assigned to John Trowbridge (trown) | 21:00 |
*** untriaged-bot has quit IRC | 21:00 | |
*** morazi has quit IRC | 21:03 | |
*** david-lyle has joined #tripleo | 21:06 | |
*** david-lyle has quit IRC | 21:09 | |
openstackgerrit | John Trowbridge proposed openstack-infra/tripleo-ci: Fix broken CI due to discoverd/inspector rename https://review.openstack.org/224887 | 21:09 |
trown | gotta pick up the little one... I will check back later on | 21:11 |
*** trown is now known as trown|outttypeww | 21:11 | |
*** Guest47613 has joined #tripleo | 21:12 | |
*** absubram has quit IRC | 21:16 | |
*** olaph has joined #tripleo | 21:16 | |
*** gfidente has quit IRC | 21:17 | |
*** rhallisey has quit IRC | 21:19 | |
*** Guest47613 has quit IRC | 21:26 | |
*** dprince has quit IRC | 21:27 | |
*** rpothier has quit IRC | 21:31 | |
openstackgerrit | Dan Prince proposed openstack/tripleo-image-elements: Update tuskar to support renamed heat roles https://review.openstack.org/224827 | 21:31 |
*** LokeshJain has joined #tripleo | 21:32 | |
LokeshJain | Hi, for hiera value of service_plugins, is list_join required? | 21:33 |
*** thrash is now known as thrash|g0ne | 21:55 | |
*** trellooobot has joined #tripleo | 22:10 | |
trellooobot | URGENT TRIPLEO TASKS NEED ATTENTION | 22:10 |
trellooobot | +-------------------------------------+-------------------------------+-------------------+-------------+ | 22:10 |
trellooobot | | Title | URL | Members | Last Active | | 22:10 |
trellooobot | +-------------------------------------+-------------------------------+-------------------+-------------+ | 22:10 |
trellooobot | | tripleo-ci failing : Requires: pyth | https://trello.com/c/Rqt7AOQY | **NEEDS MEMBERS** | 58 min | | 22:10 |
trellooobot | +-------------------------------------+-------------------------------+-------------------+-------------+ | 22:10 |
*** trellooobot has quit IRC | 22:10 | |
*** beagles has joined #tripleo | 22:11 | |
*** beagles is now known as Guest62063 | 22:11 | |
*** shivrao has joined #tripleo | 22:15 | |
*** Guest62063 has quit IRC | 22:17 | |
*** weshay has quit IRC | 22:20 | |
*** beagles_ has joined #tripleo | 22:21 | |
*** beagles_ has quit IRC | 22:26 | |
*** panda has quit IRC | 22:37 | |
*** panda has joined #tripleo | 22:38 | |
*** egafford has quit IRC | 22:49 | |
*** david-lyle has joined #tripleo | 22:59 | |
*** trellooobot has joined #tripleo | 23:10 | |
trellooobot | URGENT TRIPLEO TASKS NEED ATTENTION | 23:10 |
trellooobot | +-------------------------------------+-------------------------------+-------------------+-------------+ | 23:10 |
trellooobot | | Title | URL | Members | Last Active | | 23:10 |
trellooobot | +-------------------------------------+-------------------------------+-------------------+-------------+ | 23:10 |
trellooobot | | tripleo-ci failing : Requires: pyth | https://trello.com/c/Rqt7AOQY | **NEEDS MEMBERS** | 118 min | | 23:10 |
trellooobot | +-------------------------------------+-------------------------------+-------------------+-------------+ | 23:10 |
*** trellooobot has quit IRC | 23:10 | |
*** lucasagomes has quit IRC | 23:16 | |
*** trown|outttypeww is now known as trown | 23:25 | |
*** sthillma has joined #tripleo | 23:35 | |
*** penick has joined #tripleo | 23:38 | |
*** alop has quit IRC | 23:50 | |
*** Prophidy_ has joined #tripleo | 23:50 | |
openstackgerrit | John L. Villalovos proposed openstack/diskimage-builder: Fix variable misspelling error https://review.openstack.org/224928 | 23:53 |
*** Prophidys has quit IRC | 23:54 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!