*** TSCHAK has joined #tripleo | 00:05 | |
*** mburned is now known as mburned_out | 00:12 | |
*** mburned_out is now known as mburned | 00:15 | |
*** yamahata has quit IRC | 00:17 | |
*** jcoufal has quit IRC | 00:19 | |
*** jcoufal has joined #tripleo | 00:20 | |
openstackgerrit | Matthew Thode proposed openstack/diskimage-builder: Optionally remove portage files https://review.openstack.org/348536 | 00:27 |
---|---|---|
*** jcoufal has quit IRC | 00:28 | |
*** dtrainor has quit IRC | 00:35 | |
*** jcoufal has joined #tripleo | 00:39 | |
*** limao has joined #tripleo | 00:40 | |
*** yamahata has joined #tripleo | 00:46 | |
*** limao_ has joined #tripleo | 00:59 | |
*** limao has quit IRC | 01:02 | |
*** jcoufal has quit IRC | 01:06 | |
*** jcoufal has joined #tripleo | 01:08 | |
*** yamahata has quit IRC | 01:10 | |
*** saneax is now known as saneax_AFK | 01:10 | |
*** jcoufal has quit IRC | 01:20 | |
*** jcoufal has joined #tripleo | 01:20 | |
*** tzumainn has quit IRC | 01:20 | |
*** seanatcisco has joined #tripleo | 01:27 | |
*** yamahata has joined #tripleo | 01:44 | |
*** rlandy has quit IRC | 01:50 | |
*** jcoufal has quit IRC | 01:51 | |
*** jcoufal has joined #tripleo | 01:52 | |
*** jcoufal has quit IRC | 02:12 | |
*** yamahata has quit IRC | 02:15 | |
*** seanatcisco has quit IRC | 02:17 | |
*** seanatcisco has joined #tripleo | 02:18 | |
openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates: Properly direct MTU parameters and allow service defaults https://review.openstack.org/333333 | 02:19 |
*** dmacpher is now known as dmacpher-lunch | 02:40 | |
*** seanatcisco has quit IRC | 02:43 | |
*** dtrainor has joined #tripleo | 02:47 | |
*** dtrainor has quit IRC | 02:59 | |
*** thrash is now known as thrash|g0ne | 03:00 | |
*** dmacpher-lunch is now known as dmacpher | 03:31 | |
*** coolsvap_ has joined #tripleo | 03:56 | |
*** coolsvap_ is now known as coolsvap | 03:57 | |
openstackgerrit | Merged openstack/diskimage-builder: Generalize logic for skipping final image generation https://review.openstack.org/334042 | 03:59 |
*** links has joined #tripleo | 04:12 | |
*** links has quit IRC | 04:13 | |
*** links has joined #tripleo | 04:13 | |
*** numans has joined #tripleo | 04:53 | |
*** masco has joined #tripleo | 05:07 | |
*** numans has quit IRC | 05:07 | |
*** saneax_AFK is now known as saneax | 05:32 | |
*** radek_ has joined #tripleo | 05:40 | |
*** pgadiya has joined #tripleo | 05:50 | |
saneax | karthiks, https://review.openstack.org/#/c/333882/13/manifests/plugins/ml2/mech_driver.pp@57 | 05:54 |
openstackgerrit | Sanjay Upadhyay proposed openstack/tripleo-specs: tripleo-sriov https://review.openstack.org/313872 | 06:04 |
*** numans has joined #tripleo | 06:36 | |
*** tesseract- has joined #tripleo | 06:37 | |
*** pcaruana has joined #tripleo | 06:45 | |
*** leanderthal|afk is now known as leanderthal | 06:49 | |
*** dculik|gone has joined #tripleo | 06:59 | |
*** ccamacho has joined #tripleo | 06:59 | |
*** paramite has joined #tripleo | 06:59 | |
*** jprovazn has joined #tripleo | 07:02 | |
*** anshul has joined #tripleo | 07:05 | |
bandini | morning | 07:07 |
ccamacho | morning bandini! man yesterday I ended buying a portable a/c :P | 07:08 |
bandini | ccamacho: ehehe in the end you do what you have to do to survive ;) | 07:09 |
*** zoli_gone-proxy is now known as zoliXXL | 07:09 | |
ccamacho | yeahp, I'm starting to think about selling Madrid's Sahara air bottled... | 07:10 |
bandini | ehehehe, I will buy some for my winters ;) | 07:11 |
ccamacho | bandini and * I'm going to invest my time with composable aodh if needed some reviews just let me know | 07:12 |
bandini | ccamacho: ah awesome. so I need a last ack here (and its dep): https://review.openstack.org/347310 | 07:13 |
bandini | ccamacho: after that goes in I will rebase the HA NG stuff on the aodh profile review | 07:13 |
bandini | so I will test that aodh profile one as well | 07:14 |
ccamacho | bandini in that case Ill test yours first, ha 1+1 works for you or ha 3+1 ? | 07:14 |
ccamacho | afaik the aodh submission is not working.. so I need to check whats up there.. | 07:15 |
bandini | ccamacho: I always test HA 3+1 or 3+2 | 07:17 |
ccamacho | don't have enough ram for 3+2 :P | 07:18 |
ccamacho | 3+1 then | 07:18 |
bandini | yeah RAM is always an issue :) I am lucky I can test on baremetal these days | 07:18 |
bandini | not all the time, but often enough | 07:19 |
openstackgerrit | yolanda.robla proposed openstack-infra/tripleo-ci: Add --yes flags to heat stack-delete for liberty https://review.openstack.org/347202 | 07:19 |
*** akuznetsov has joined #tripleo | 07:22 | |
*** davidlenwell has quit IRC | 07:22 | |
*** gfidente has joined #tripleo | 07:23 | |
*** gfidente has quit IRC | 07:23 | |
*** gfidente has joined #tripleo | 07:23 | |
ccamacho | bandini, **deploying one thing after the deployment, do you need any manual sanity check? | 07:25 |
bandini | ccamacho: what do you mean? | 07:26 |
bandini | (I am especially thick in the morning ;) | 07:27 |
marios | happy friday * | 07:27 |
ccamacho | i.e. the overcloud pingtest or any other manual test that you might want to execute there | 07:28 |
bandini | ccamacho: for the HA NG? nope | 07:28 |
bandini | marios: happy friday to you too! (and greetings from ANJ) | 07:28 |
ccamacho | marios happy friday man! http://www.kappit.com/img/pics/201510_0922_addcd_sm.jpg | 07:28 |
ccamacho | bandini ack :) | 07:28 |
hewbrocca | morning folks | 07:31 |
ccamacho | o/ | 07:32 |
*** mcornea has joined #tripleo | 07:33 | |
*** ccamacho is now known as ccamacho|afk | 07:33 | |
saneax | GM Folks | 07:33 |
bandini | o/ | 07:33 |
*** davidlenwell has joined #tripleo | 07:34 | |
*** ChanServ sets mode: +v davidlenwell | 07:34 | |
*** shardy has joined #tripleo | 07:39 | |
*** florianf has joined #tripleo | 07:40 | |
*** fzdarsky|afk has joined #tripleo | 07:44 | |
openstackgerrit | Giulio Fidente proposed openstack/python-tripleoclient: Generate CephX keys via generate_overcloud_passwords https://review.openstack.org/348412 | 07:47 |
*** ccamacho|afk is now known as ccamacho | 07:51 | |
*** jpich has joined #tripleo | 07:55 | |
*** dbecker has joined #tripleo | 07:58 | |
*** panda|Zz is now known as panda | 08:00 | |
*** dbecker has quit IRC | 08:02 | |
*** dbecker has joined #tripleo | 08:03 | |
gfidente | hey guys, maybe we have bash expertise around here :) | 08:04 |
gfidente | question | 08:04 |
gfidente | if we have tool which doesn't output to stdout but can write into a file | 08:05 |
gfidente | which syntax is best | 08:05 |
gfidente | -o /dev/stdout or >(cat -) ? | 08:05 |
gfidente | or what else? | 08:05 |
gfidente | sounds like outputting to /dev/stdout is easier but doesn't give any guarantee there won't be other processes doing the same mixing the output | 08:06 |
gfidente | so I'd prefer the >() syntax | 08:06 |
gfidente | +1 ? | 08:06 |
ccamacho | >() +1 | 08:07 |
gfidente | ccamacho tks :) | 08:08 |
*** fragatina has quit IRC | 08:09 | |
*** derekh has joined #tripleo | 08:09 | |
ccamacho | gfidente but never echoing outputs directly | 08:09 |
gfidente | ccamacho never do >(echo) you mean? | 08:10 |
ccamacho | as it will open/close the file each time you use it | 08:10 |
gfidente | bandini sounds like we need someone to approve the nova constraints? | 08:10 |
ccamacho | yeahp, try to save it into a var and then saving | 08:10 |
gfidente | ccamacho then echoing yes | 08:11 |
gfidente | thanks | 08:11 |
*** fzdarsky has joined #tripleo | 08:11 | |
*** fzdarsky|afk has quit IRC | 08:11 | |
ccamacho | :) | 08:12 |
*** electrofelix has joined #tripleo | 08:13 | |
bandini | gfidente: yep we need one volunteer | 08:14 |
marios | gfidente: bandini gimme few looking at the nova one now | 08:14 |
bandini | marios++ | 08:14 |
gfidente | marios https://review.openstack.org/#/c/347310/ | 08:14 |
marios | yes and https://review.openstack.org/#/c/347309/ | 08:14 |
gfidente | needs to be merged with its dependency which won't pass alone | 08:14 |
gfidente | yeah | 08:14 |
gfidente | so it's an act of courage | 08:15 |
marios | gfidente: tis ok, bandini already pimped the change last night | 08:15 |
gfidente | yeah it's like the openstack-core change we had as well | 08:15 |
gfidente | same purpose, same problem | 08:15 |
bandini | same shit different day | 08:17 |
bandini | I have a mug with that phrase ;) | 08:17 |
*** jbadiapa has joined #tripleo | 08:18 | |
gfidente | bandini that's just one way to think about it | 08:18 |
marios | gfidente: bandini so simple enough lgtm guys... i count 10 constraints removed and 10 added ... :) i trust you've sanity checked them a bit more than me | 08:21 |
gfidente | plus the one moving the constraints into the role passes so we have some ci coverage | 08:21 |
ccamacho | bandini, my deployment is taking while, buuut its already on step4, when completes Ill put some feedback in both reviews. | 08:22 |
bandini | marios: yeah I did double check them | 08:22 |
*** akuznetsov has quit IRC | 08:23 | |
gfidente | ccamacho fyi https://review.openstack.org/#/c/347860/8/manifests/key.pp | 08:24 |
marios | bandini: hah thanks bandini i didn't really need a signed contract it was mostly rhetorical i know obviously they've been checked by sane individuals, its not just gfidente, but bandini is involved to... so. ok | 08:24 |
marios | s/to/too | 08:24 |
*** akuznetsov has joined #tripleo | 08:24 | |
bandini | ahahahah | 08:24 |
bandini | marios: you're the first person in years to call me sane. you should question your sanity now ;) | 08:25 |
marios | "what is sane"? | 08:25 |
marios | whoami | 08:25 |
* bandini chuckles | 08:25 | |
gfidente | marios also, remember https://review.openstack.org/#/c/348415/ ? | 08:26 |
marios | gfidente: yeah sure man will look but only if you will leave me alone after that | 08:27 |
gfidente | I am giving you a chance to improve your stats | 08:27 |
marios | what are you trying to say about my stats? | 08:28 |
bandini | thanks ccamacho | 08:30 |
ccamacho | np man! | 08:30 |
gfidente | bandini so I w+2 | 08:31 |
gfidente | +1 actually | 08:31 |
bandini | ah so that does not happen automagically with 2x+2 | 08:31 |
gfidente | no so then we'll have to wait for the aodh role | 08:32 |
ccamacho | gfidente, Im stating to check the errors there, but need to increase the nodes mem.. 6GB OC nodes.. getting all the times... Cannot allocate memory - fork(2) stuff.. | 08:34 |
bandini | yeah, I have a call now and then I will jump on the aodh stuff as well and rebase on top of it | 08:36 |
*** yamahata has joined #tripleo | 08:37 | |
openstackgerrit | Merged openstack/puppet-tripleo: Move nova constraints from controller manifest to each service https://review.openstack.org/347309 | 08:39 |
openstackgerrit | Merged openstack-infra/tripleo-ci: Stop running dstat twice https://review.openstack.org/347917 | 08:40 |
*** r-mibu has quit IRC | 08:41 | |
*** akuznetsov has quit IRC | 08:41 | |
shardy | Guys can I please get some reviews on https://review.openstack.org/#/c/347925/ and https://review.openstack.org/#/c/348440/ ? | 08:41 |
shardy | both are passing CI and the second one is big so I'd rather not rebase it | 08:42 |
shardy | (big but trivial, sed replacement) | 08:42 |
gfidente | shardy I think in firewall.pp we were even replacing - with _ somewhere | 08:44 |
shardy | gfidente: Ah, cool, I guess we can remove that then | 08:44 |
gfidente | https://github.com/openstack/puppet-tripleo/blob/master/manifests/firewall/service_rules.pp#L27 | 08:45 |
gfidente | yeah :) | 08:45 |
shardy | gfidente: I'm working on a patch that makes the service_foo_ips lists dynamically generated: | 08:45 |
shardy | https://github.com/openstack/tripleo-heat-templates/blob/master/overcloud.yaml#L911 | 08:45 |
shardy | so we need this change to enable that | 08:46 |
gfidente | ack | 08:46 |
shardy | I'm currently working around it by a nasty combination of list_join, str_replace and str_split :D | 08:46 |
openstackgerrit | Merged openstack/tripleo-heat-templates: Move nova constraints from controller manifest to each service https://review.openstack.org/347310 | 08:46 |
ccamacho | so service names with _ right? /me needs to update aodh and mistral then | 08:47 |
shardy | ccamacho: Yeah, sorry any pending patches will need to be updated when we land my patch | 08:47 |
shardy | we need the names to be consistent | 08:47 |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Use delorean's exit code to decide on retry https://review.openstack.org/348790 | 08:48 |
gfidente | and there it is our first yaql as well | 08:48 |
ccamacho | shardy, np | 08:48 |
*** ramishra has quit IRC | 08:48 | |
shardy | gfidente: Yeah | 08:48 |
shardy | the dynamic generation of *ips uses nested map_replace fundtions combined with repeat and map_merge :D | 08:48 |
shardy | http://paste.openstack.org/show/543854/ | 08:50 |
*** r-mibu has joined #tripleo | 08:51 | |
openstackgerrit | Merged openstack/tripleo-heat-templates: Convert service_name to underscore syntax https://review.openstack.org/348440 | 08:52 |
openstackgerrit | Adriano Petrich proposed openstack/tripleo-quickstart: WIP gate upgrade https://review.openstack.org/342161 | 08:59 |
gfidente | shardy so that goes this way? | 08:59 |
gfidente | cinder_api_node_ips: cinder_api_network | 08:59 |
gfidente | cinder_api_node_ips: internal_api | 08:59 |
gfidente | cinder_api_node_ips: 172.X.Y.Z | 08:59 |
openstackgerrit | Merged openstack/tripleo-ui: Add es6-promise polyfill to webpack config https://review.openstack.org/347201 | 08:59 |
shardy | gfidente: Yup | 08:59 |
shardy | gfidente: I'm still trying to figure out how we can maintain compatibility with the current ServiceNetMap syntax tho | 09:00 |
shardy | ideally we'd move to service_name_network in there too | 09:01 |
shardy | but obviously we'll have to allow for deprecation of the old format | 09:01 |
gfidente | shardy wait though one thing I don't get is | 09:01 |
gfidente | to get the list of ips hosting a service | 09:02 |
gfidente | is this generated within the resourcegroup | 09:02 |
gfidente | and then joined in overcloud.yaml? | 09:02 |
shardy | gfidente: Yes, that says NetIpMap, but it will move into overcloud.yaml and use ControllerIpListMap | 09:03 |
shardy | we'll generate a map of lists for each role, with a key for every enabled service | 09:03 |
gfidente | wow | 09:03 |
openstackgerrit | Carlos Camacho proposed openstack/puppet-tripleo: Add pacemaker mistral profile https://review.openstack.org/343346 | 09:04 |
openstackgerrit | Carlos Camacho proposed openstack/puppet-tripleo: Implement aodh profiles https://review.openstack.org/332854 | 09:05 |
*** jbadiapa has quit IRC | 09:05 | |
* shardy needs another coffee before attempting to make that actually work ;) | 09:06 | |
marios | gfidente: so i figure i'm allowed at least *one* review request from you... https://review.openstack.org/#/c/188137/ afaics it is good, but the test run at https://review.openstack.org/#/c/342961/ wont pass with "CREATE_FAILED Resource CREATE failed: The Referenced Attribute (ControllerServiceChain role_data) is incorrect" has passed before. but, can you please eyes on the manila particularly the puppet/se | 09:08 |
marios | rvices/ incase i missed sthing there? | 09:08 |
marios | gfidente: hah sorry for gigaping. | 09:08 |
openstackgerrit | Carlos Camacho proposed openstack/tripleo-heat-templates: Composable Mistral services https://review.openstack.org/323436 | 09:08 |
marios | gfidente: i spent ages staring at those definitinos yesterday but couldn't spot some issue | 09:08 |
openstackgerrit | Carlos Camacho proposed openstack/tripleo-heat-templates: Composable Mistral services https://review.openstack.org/323436 | 09:09 |
gfidente | marios I think it's a syntax error in role_data | 09:09 |
gfidente | I've been there before | 09:09 |
marios | gfidente: ack thanks appreciate, whenever you ahve time no rush/now | 09:10 |
gfidente | shardy so in https://review.openstack.org/#/c/347925/1/puppet/services/services.yaml data: is the json to which $ refers to? | 09:10 |
*** mgould|afk is now known as mgould | 09:11 | |
gfidente | guys can you check the trove submission as well: https://review.openstack.org/#/c/233240/ | 09:11 |
gfidente | ^_^ | 09:11 |
shardy | gfidente: Yeah, that's the input data to the yaql expression | 09:12 |
shardy | I'm basically running a yaql query to filter the null values we get from OS::Heat::None | 09:13 |
shardy | http://docs.openstack.org/developer/heat/template_guide/hot_spec.html#yaql | 09:14 |
openstackgerrit | Carlos Camacho proposed openstack/tripleo-heat-templates: Add Aodh composable roles https://review.openstack.org/333556 | 09:14 |
shardy | The yaql docs are not good but it seems to work pretty well | 09:14 |
gfidente | ack +2 | 09:15 |
gfidente | marios I think I found it | 09:15 |
gfidente | now you have to review shit for a week! | 09:15 |
shardy | lol | 09:15 |
marios | gfidente: yeah as if there is a day when you don't hasle me for reviews | 09:16 |
marios | gfidente: thanks man :) where? | 09:16 |
gfidente | https://review.openstack.org/#/c/188137/87/puppet/services/manila-base.yaml | 09:16 |
openstackgerrit | Carlos Camacho proposed openstack/tripleo-heat-templates: Add Aodh composable roles https://review.openstack.org/333556 | 09:16 |
marios | gfidente: thanks man :) gfidente ++ | 09:16 |
marios | bandini: prepare for tornados and hail today ^^ | 09:17 |
marios | hale even? probably | 09:17 |
*** mbound has joined #tripleo | 09:17 | |
bandini | eheheh | 09:18 |
*** yolanda has quit IRC | 09:19 | |
*** jbadiapa has joined #tripleo | 09:19 | |
*** yolanda has joined #tripleo | 09:20 | |
marios | ccamacho: did the _ thing land already? pointer? | 09:22 |
ccamacho | marios https://review.openstack.org/#/c/348440/ | 09:24 |
ccamacho | yeahp today in the morning | 09:24 |
marios | ccamacho: ack thanks | 09:25 |
ccamacho | np | 09:25 |
openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: Enable Manila integration - as a composable controller service https://review.openstack.org/188137 | 09:30 |
*** chem|away is now known as chem | 09:31 | |
openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: DO NOT MERGE testing manila https://review.openstack.org/342961 | 09:33 |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Remve old cache files on the mirror server https://review.openstack.org/348806 | 09:37 |
*** fragatina has joined #tripleo | 09:38 | |
openstackgerrit | Michele Baldessari proposed openstack/puppet-tripleo: Next generation HA architecture work https://review.openstack.org/342650 | 09:38 |
ccamacho | Extreme problems often require extreme solutions http://petitions.moveon.org/sign/a-memory-upgrade-for?source=c.em.cp&r_by=16249607 :P | 09:44 |
hewbrocca | ROFL | 09:48 |
hewbrocca | ccamacho: nice | 09:48 |
openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates: Next generation HA architecture work https://review.openstack.org/348811 | 09:52 |
derekh | If only we had a OVB deployment with enough capacity of all to use | 09:53 |
*** akrivoka has joined #tripleo | 09:57 | |
openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates: Next generation HA architecture work https://review.openstack.org/314208 | 09:57 |
*** ramishra has joined #tripleo | 10:00 | |
openstackgerrit | Karthik S proposed openstack/tripleo-heat-templates: Customize vcpu_pin_set in nova.conf https://review.openstack.org/343770 | 10:01 |
bandini | gfidente: there are two more cleanups we need to do for NG, just noticed now (same as before just moving constraints to their respective tripleo pacemaker profiles) | 10:08 |
openstackgerrit | Michele Baldessari proposed openstack/puppet-tripleo: Move openstack-core-then-httpd constraint in the apache manifest https://review.openstack.org/348815 | 10:09 |
openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates: Move openstack-core-then-httpd constraint in the apache manifest https://review.openstack.org/348816 | 10:10 |
gfidente | bandini ack | 10:11 |
*** limao_ has quit IRC | 10:13 | |
*** zoliXXL is now known as zoli|lunch | 10:15 | |
*** yamahata has quit IRC | 10:16 | |
*** milan has quit IRC | 10:16 | |
*** tosky has joined #tripleo | 10:18 | |
openstackgerrit | Merged openstack/tripleo-docs: Remove enabling epel https://review.openstack.org/348538 | 10:20 |
openstackgerrit | Merged openstack/tripleo-heat-templates: We don't need to set a default for the CephX keys and cluster FSID https://review.openstack.org/348415 | 10:20 |
openstackgerrit | Giulio Fidente proposed openstack/python-tripleoclient: Generate CephX keys via generate_overcloud_passwords https://review.openstack.org/348412 | 10:21 |
openstackgerrit | Merged openstack-infra/tripleo-ci: Refactor mirror logic https://review.openstack.org/326143 | 10:22 |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Record testenv wait times in ovb jobs https://review.openstack.org/347037 | 10:23 |
openstackgerrit | Merged openstack/python-tripleoclient: Removes setting NeutronL3HA and NeutronAllowL3AgentFailover https://review.openstack.org/345663 | 10:23 |
*** milan has joined #tripleo | 10:25 | |
openstackgerrit | Sanjay Upadhyay proposed openstack/tripleo-heat-templates: SR-IOV nic agent changes in line with composable roles. https://review.openstack.org/332190 | 10:26 |
gfidente | shardy trozet ^^ neutronl3ha thing is wrong | 10:27 |
gfidente | I don't think we can enable l3ha with a single neutron node | 10:27 |
gfidente | which is why we were enabling it based on the number of controllers | 10:27 |
shardy | gfidente: https://review.openstack.org/#/c/345648/2/puppet/services/neutron-server.yaml defaults it to false | 10:28 |
shardy | but yeah, we're assuming pacemaker won't ever be enabled for single nodes | 10:28 |
gfidente | shardy yeah I mean the pacemaker scenario | 10:28 |
gfidente | maybe it's fine to keep it as is | 10:29 |
shardy | http://docs.openstack.org/developer/heat/template_guide/hot_spec.html#equals | 10:29 |
shardy | gfidente: we could use equals to convert a node count to a boolean, e.g equals 1 | 10:29 |
shardy | but then we'd have coupling between the ControllerCount and the service | 10:29 |
gfidente | shardy ah yaql! | 10:30 |
shardy | No, it's another new function ;) | 10:30 |
* shardy is using all the new shiny heat things today ;) | 10:30 | |
gfidente | in that respect, we had the coupling in the client before so it wasn't great anyway | 10:31 |
shardy | Maybe we just document you need to disable it if you want to deploy a 1 node pacemaker setup? | 10:31 |
shardy | seems like something of a corner case to me | 10:31 |
*** mbound has quit IRC | 10:31 | |
gfidente | yeah | 10:31 |
panda | who's the CI mentor this week ? :) | 10:33 |
shardy | panda: derekh is in the etherpad | 10:33 |
derekh | panda: how can I help? | 10:34 |
panda | shardy: I see that ssl jobs are using a static certificate with 192.2.0.6 as CN. That's strange because I've seen the deployment trying to contact the controller on the vlan10 IP (10.0.0.5) I don't know how to change my parameters to have the Ip in CN as the public VIP of the controller. If I have to change the certificate instead I don't know if using 10.0.0.5 as CN is a good idea. | 10:42 |
panda | shardy: does anything that I've said make any sense ? | 10:47 |
shardy | panda: where do you see traffic going via 10.0.0.5 vs the public vip? | 10:47 |
shardy | is this for internal SSL? | 10:47 |
shardy | panda: it does, but tbh you'd probably be best dicussing this with either jaosorior or bnemec | 10:48 |
shardy | I think jaosorior is out on PTO for a week tho | 10:48 |
derekh | panda: you may have seen it but I think this is the template CI is using to set the VIP to 192.0.2.6 http://git.openstack.org/cgit/openstack-infra/tripleo-ci/tree/test-environments/enable-tls.yaml | 10:53 |
panda | shardy: I don't see traffic, a deploy that enables SSL stops with this error: SSL exception connecting to https://10.0.0.5:13000/v3/auth/tokens: hostname '10.0.0.5' doesn't match u'192.0.2.6'. I don't know why the puppet module is trying to contact the controller on that IP. | 10:54 |
panda | It's probably an issue with my local environment setup, but I'd like to understand how to set the right parameters | 10:54 |
*** tzumainn has joined #tripleo | 10:55 | |
*** pkovar has joined #tripleo | 10:55 | |
gfidente | shardy sorry but I am having a hard time figuring what is wrong here: | 10:57 |
gfidente | https://review.openstack.org/#/c/346796/2/puppet/services/ceph-mon.yaml | 10:57 |
gfidente | I'm turning it from list into hash so it can be overridden via map_merge | 10:58 |
gfidente | but it's failing with http://logs.openstack.org/96/346796/2/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/83c81ea/console.html#_2016-07-28_11_13_56_099373 | 10:58 |
gfidente | any hint? | 10:58 |
*** fragatina has quit IRC | 11:00 | |
*** panda is now known as panda|lunch | 11:05 | |
*** jprovazn has quit IRC | 11:09 | |
openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: DO NOT MERGE testing manila https://review.openstack.org/342961 | 11:10 |
shardy | panda|lunch: it should be using the keystone public endpoint from the service catalog, so I'd check the IP that points to | 11:12 |
shardy | gfidente: You need a list of maps, but the get_param's are referencing string parameters? | 11:13 |
shardy | so you need somekey: {get_param: ...} ? | 11:14 |
*** lucasagomes is now known as lucas-hungry | 11:14 | |
ccamacho | gfidente, really not sure, but something like http://paste.openstack.org/show/543873/ | 11:15 |
shardy | ccamacho: You can't map_merge a list of lists | 11:15 |
shardy | it merges a list of maps | 11:15 |
ccamacho | mmm :( | 11:16 |
shardy | the problem is the maps in the list are malformed | 11:16 |
*** rodrigods has quit IRC | 11:19 | |
*** rodrigods has joined #tripleo | 11:19 | |
*** jprovazn has joined #tripleo | 11:29 | |
*** tbonds has quit IRC | 11:30 | |
openstackgerrit | Miles Gould proposed openstack/diskimage-builder: Allow installing proliant tools from an OS package https://review.openstack.org/344214 | 11:30 |
*** mbound has joined #tripleo | 11:32 | |
openstackgerrit | Miles Gould proposed openstack/diskimage-builder: Allow installing proliant tools from an OS package https://review.openstack.org/344214 | 11:36 |
*** saneax is now known as saneax_AFK | 11:36 | |
*** mbound has quit IRC | 11:36 | |
openstackgerrit | Michele Baldessari proposed openstack/puppet-tripleo: Move constraints to their respective services https://review.openstack.org/348815 | 11:39 |
openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates: Move constraints to their respective services https://review.openstack.org/348816 | 11:39 |
*** thrash|g0ne is now known as thrash | 11:41 | |
openstackgerrit | Miles Gould proposed openstack/diskimage-builder: Allow installing proliant tools from an OS package https://review.openstack.org/344214 | 11:41 |
*** rhallisey has joined #tripleo | 11:42 | |
*** dsariel has joined #tripleo | 11:45 | |
openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates: WIP Move rabbit's clustering port away from the ephemeral port range https://review.openstack.org/345851 | 11:46 |
*** zoli|lunch is now known as zoli | 11:47 | |
hewbrocca | folks, seen beekhof's latest blog re "composable HA"? | 11:47 |
hewbrocca | http://blog.clusterlabs.org/blog/2016/composable-openstack-ha | 11:47 |
gfidente | shardy get_param is getting a string parameter yes | 11:51 |
gfidente | I was thinking to build the map 'on the fly' | 11:51 |
gfidente | - get_param | 11:52 |
gfidente | key: value | 11:52 |
gfidente | - get_param | 11:52 |
gfidente | key: value | 11:52 |
gfidente | and I wanted map_merge to merge the two dicts | 11:52 |
shardy | gfidente: but the get_param isn't a valid key: value for the map | 11:53 |
gfidente | ah shit | 11:53 |
gfidente | right | 11:53 |
gfidente | ;) | 11:53 |
shardy | :) | 11:53 |
gfidente | thanks | 11:54 |
shardy | hewbrocca: No, looks very interesting, thanks for the heads-up! | 11:54 |
*** pradk has joined #tripleo | 11:55 | |
hewbrocca | fabbione: just linked me to it | 11:57 |
*** weshay is now known as weshay_mtg | 12:01 | |
openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Support arbitrary attributes for the Ceph pools https://review.openstack.org/346796 | 12:10 |
*** zoli is now known as zoli|brb | 12:10 | |
*** masco has quit IRC | 12:11 | |
*** panda|lunch is now known as panda | 12:11 | |
*** jayg|g0n3 is now known as jayg | 12:12 | |
*** morazi has joined #tripleo | 12:13 | |
*** trown|outtypewww is now known as trown | 12:13 | |
openstackgerrit | Flavio Percoco proposed openstack/tripleo-heat-templates: Update heat-agents setup files https://review.openstack.org/346859 | 12:19 |
openstackgerrit | Flavio Percoco proposed openstack/tripleo-heat-templates: Add default value for `RoleData` https://review.openstack.org/346860 | 12:19 |
openstackgerrit | Flavio Percoco proposed openstack/tripleo-heat-templates: WIP: Containerized Services for Composable Roles https://review.openstack.org/330659 | 12:19 |
openstackgerrit | Flavio Percoco proposed openstack/tripleo-heat-templates: Bind mount files to run DiD in latest atomic host https://review.openstack.org/347218 | 12:19 |
*** lucas-hungry is now known as lucasagomes | 12:20 | |
*** ccamacho is now known as ccamacho|lunch | 12:28 | |
*** tbonds has joined #tripleo | 12:29 | |
gfidente | derekh do we have a copy of the tripleo-overcloud-passwords file from ci jobs that you know? | 12:29 |
derekh | gfidente: hmm, that would be in ~ wouldn't it? I don't think so | 12:30 |
gfidente | yeah | 12:30 |
derekh | ya, we only get files from /var/log and /etc , | 12:31 |
gfidente | derekh can I add it to host_info ? | 12:31 |
*** dsariel has quit IRC | 12:31 | |
derekh | gfidente: yup, go for it | 12:31 |
*** weshay_mtg is now known as weshay | 12:31 | |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: move hieradata/ceph into ceph-base service https://review.openstack.org/348489 | 12:32 |
*** jdob has quit IRC | 12:33 | |
*** jdob has joined #tripleo | 12:33 | |
openstackgerrit | Honza Pokorny proposed openstack/tripleo-ui: Add packaging spec and config files https://review.openstack.org/344932 | 12:36 |
openstackgerrit | Giulio Fidente proposed openstack-infra/tripleo-ci: Add 'ceph status' and tripleo-overcloud-passwords to host_info https://review.openstack.org/348862 | 12:38 |
*** rlandy has joined #tripleo | 12:44 | |
EmilienM | gfidente: can you approve https://review.openstack.org/#/c/347455/ please? | 12:50 |
shardy | EmilienM: Hi, good morning, could you please check https://review.openstack.org/#/c/347925/ when you get a moment? | 12:54 |
shardy | I'm going to push some patches which depend on that soon and I'd rather land it than cause a rebase | 12:54 |
EmilienM | shardy: let me check | 12:54 |
shardy | thanks! | 12:55 |
EmilienM | shardy: +A | 12:55 |
*** zoli|brb is now known as zoli | 12:57 | |
*** zoli is now known as zoliXXL | 12:58 | |
*** jcoufal has joined #tripleo | 12:58 | |
openstackgerrit | Honza Pokorny proposed openstack/tripleo-ui: Add packaging spec and config files https://review.openstack.org/344932 | 13:00 |
*** maeca1 has joined #tripleo | 13:00 | |
openstackgerrit | Merged openstack/tripleo-heat-templates: Filter null/None service names https://review.openstack.org/347925 | 13:00 |
openstackgerrit | Julie Pichon proposed openstack/tripleo-common: Add boot option argument to the Node Registration Mistral workflow https://review.openstack.org/348875 | 13:00 |
*** pradk has quit IRC | 13:02 | |
*** jeckersb_gone is now known as jeckersb | 13:04 | |
*** bvandenh has quit IRC | 13:05 | |
*** rcrit has left #tripleo | 13:05 | |
*** saneax_AFK is now known as saneax | 13:06 | |
*** links has quit IRC | 13:07 | |
*** numans has quit IRC | 13:07 | |
beagles | EmilienM, shardy, if/when you can spare a moment https://review.openstack.org/#/c/333333/ | 13:07 |
EmilienM | beagles: sure, will look shortly | 13:08 |
*** ccamacho|lunch is now known as ccamacho | 13:08 | |
shardy | beagles: this is a mitaka backport, but I can't see the master patch? | 13:08 |
beagles | EmilienM, thx! | 13:09 |
shardy | the change id doesn't match anything in master, and grepping the title gives me no hits | 13:09 |
*** ianw has quit IRC | 13:10 | |
*** ianw has joined #tripleo | 13:10 | |
openstackgerrit | John Trowbridge proposed openstack/python-tripleoclient: Remove another private OSC attribute usage https://review.openstack.org/348460 | 13:10 |
trown | thrash: fixed up UT on ^ | 13:10 |
*** numans has joined #tripleo | 13:10 | |
*** bvandenh has joined #tripleo | 13:11 | |
beagles | shardy, it wasn't really a backport per-se. The counterpart for newtown was quite different - but the related patch is https://review.openstack.org/#/c/325926/ | 13:11 |
beagles | shardy, basically we completely removed all of the properties in newton | 13:11 |
shardy | beagles: Ok, you'll need to explain why it's a mitaka-only patch in the commit message, and link the related master patch please | 13:12 |
beagles | shardy, ack | 13:12 |
ccamacho | EmilienM morning! found several issues in the aodh submission, have fixed the non-ha locally, now im starting to work with the ha profiles. | 13:12 |
shardy | also please add a Closes-Bug to the commit message while you're updating :) | 13:12 |
hewbrocca | derekh: so we got rh1 running? | 13:12 |
hewbrocca | When do we go turn back on all the jobs? | 13:13 |
beagles | hah.. there wasn't a bug reported against it when it was orgiinally filed :) but IIRC there are some now.. will do | 13:13 |
hewbrocca | I'm a little afraid... | 13:13 |
shardy | beagles: all patches landing on stable branches must have bugs associated with them | 13:13 |
EmilienM | ccamacho: excellent | 13:14 |
derekh | hewbrocca: its running, nodepool is trying to use it but for soem reason deleting its VMs after a minute, I'm not sure why yet, I'm waiting for the infra people to wake up to find out why | 13:14 |
beagles | IIRC = bugs are actually assigned to me at this point and forgot to amend my commit message to include them ;) | 13:14 |
shardy | hehe, no worries :) | 13:14 |
hewbrocca | derekh: Well OK, forward progress | 13:14 |
derekh | hewbrocca: I think this is all thats needed to get it working https://review.openstack.org/#/c/348778/ | 13:15 |
derekh | hewbrocca: once its working, I have two patches lined up | 13:15 |
hewbrocca | ohhh were we using nova-net for floating IPs before? | 13:15 |
derekh | hewbrocca: 1 to switch all our current jobs from rh2 to rh1 | 13:15 |
hewbrocca | It would make sense | 13:15 |
derekh | hewbrocca: then assuming thats ok, another to restore all the origional jobs | 13:16 |
hewbrocca | cool | 13:16 |
*** Tristitia has joined #tripleo | 13:17 | |
openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart: Adds ci-scripts to for tripleo-quickstart on OVB https://review.openstack.org/347393 | 13:19 |
openstackgerrit | Julie Pichon proposed openstack/python-tripleoclient: Add 'openstack overcloud node import' command https://review.openstack.org/339727 | 13:20 |
openstackgerrit | Julie Pichon proposed openstack/python-tripleoclient: [WIP] Add 'openstack overcloud node configure' command https://review.openstack.org/344875 | 13:20 |
openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates: Properly direct MTU parameters and allow service defaults https://review.openstack.org/333333 | 13:21 |
openstackgerrit | Justin Kilpatrick proposed openstack/tripleo-quickstart: Add documentation for add-on roles https://review.openstack.org/346733 | 13:21 |
*** egafford has joined #tripleo | 13:24 | |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Source rh1.env to decide how many testenv workers to run https://review.openstack.org/348883 | 13:25 |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Switch back to the net iso HA job https://review.openstack.org/348884 | 13:25 |
*** saneax is now known as saneax_AFK | 13:26 | |
EmilienM | slagle: I don't understand https://review.openstack.org/#/c/346147/20/scripts/deploy.sh | 13:32 |
EmilienM | if I put an exit 0, I'm breaking CI | 13:32 |
slagle | how does that break CI? | 13:34 |
slagle | if you're doing an undercloud job, exit after undercloud stuff is done | 13:35 |
trozet | gfidente: is there a use case for having pacemaker on a single controller node? wouldn't most people use noha? | 13:35 |
*** mbound has joined #tripleo | 13:36 | |
hewbrocca | trozet: It dramatically simplifies deployment for us | 13:37 |
hewbrocca | since we don't have to test systemd-management on the non-ha path vs. pacemaker management on the HA path | 13:38 |
hewbrocca | if we use pacemaker on HA and non-HA there's half as many paths to test | 13:38 |
trozet | hewbrocca: well sure...but imo that isn't a reason to do it | 13:38 |
trozet | hewbrocca: along the same lines I don't get why there is a VIP used in noha case | 13:39 |
hewbrocca | LOL seems like a pretty good reason to me, but go ahead | 13:39 |
openstackgerrit | Carlos Camacho proposed openstack/puppet-tripleo: Implement aodh profiles https://review.openstack.org/332854 | 13:44 |
openstackgerrit | Carlos Camacho proposed openstack/tripleo-heat-templates: Add Aodh composable roles https://review.openstack.org/333556 | 13:44 |
gfidente | trozet yeah the purpose is to be able to test the pacemaker codepath with a single controller | 13:45 |
gfidente | anyway, for dev purposes one can set that parameter to false as needed | 13:45 |
trozet | gfidente: yeah, it's just defaulted | 13:45 |
*** myoung|remote has joined #tripleo | 13:46 | |
*** jeckersb is now known as jeckersb_gone | 13:49 | |
*** seanatcisco has joined #tripleo | 13:50 | |
*** akshai has joined #tripleo | 13:50 | |
*** dculik|gone has quit IRC | 13:51 | |
*** seanatcisco_ has joined #tripleo | 13:52 | |
rodrigods | quick (dummy) question: when running "bash quickstart.sh $HOST", it is skipping overcloud install/deploy | 13:53 |
rodrigods | is there any extra param needed? | 13:53 |
trown | rodrigods: ya by default it stops before the overcloud deploy, so that a user could make changes/verify the deploy command before continuing | 13:54 |
trown | rodrigods: "--tags all" will do the full deploy | 13:54 |
openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: Fix Aodh httpd binding https://review.openstack.org/348893 | 13:54 |
rodrigods | trown, hmm thanks | 13:54 |
*** pgadiya has quit IRC | 13:54 | |
EmilienM | hey, I found this bug in master and stable/mitaka ^ | 13:55 |
*** seanatcisco has quit IRC | 13:55 | |
*** seanatcisco_ is now known as seanatcisco | 13:55 | |
EmilienM | it's kind of urgent for our mitaka release, please review it asap | 13:55 |
shardy | EmilienM: lgtm, but curious why it's a release blocker, don't we disable all metering services by default? | 13:57 |
EmilienM | it's a blocker to upgrade the undercloud | 13:59 |
EmilienM | from liberty to mitaka | 13:59 |
*** myoung|remote has quit IRC | 13:59 | |
*** myoung|remote has joined #tripleo | 14:00 | |
*** pradk has joined #tripleo | 14:00 | |
openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Implement a new job for undercloud testing only https://review.openstack.org/346147 | 14:02 |
trown | core folks, https://review.openstack.org/#/c/348460 is the only thing blocking promote atm, but zuul is super slow today. only UT failed on PS3 and I only fixed up tests in PS4. Would anyone be willing to manually verify I fixed UT (I ran locally and they pass) and we can merge? | 14:02 |
bnemec | panda: The CI setup is probably different because we weren't running net-iso in the ssl-enabled jobs, and it sounds like you are. | 14:03 |
EmilienM | trown: promote master? | 14:03 |
bnemec | panda: There was also a change in how you have to configure the vip for overcloud ssl. See my change at: https://review.openstack.org/#/c/344475/ | 14:03 |
trown | EmilienM: ya | 14:03 |
EmilienM | trown: I've notice mitaka hasn't promoted also | 14:03 |
EmilienM | is there a blocker? | 14:03 |
trown | EmilienM: ya, mitaka failed on a spurious error in one of the packstack jobs last night | 14:04 |
*** fultonj has joined #tripleo | 14:04 | |
trown | EmilienM: it only runs every 12h by default, but I kicked this morning and tripleo jobs are in deploy | 14:04 |
EmilienM | trown: packstack only? we should promote | 14:04 |
EmilienM | ok | 14:04 |
trown | fingers crossed | 14:04 |
panda | bnemec: yes, I am using network isolation ... so the only way is to regenerate the certificate ? | 14:04 |
panda | bnemec: I'm running in circles trying to find the exact place where this IP is specified ... | 14:05 |
trown | shardy: is it ok if I merge https://review.openstack.org/#/c/348460 for the logic above? | 14:05 |
trown | [10:02:36] <trown> core folks, https://review.openstack.org/#/c/348460 is the only thing blocking promote atm, but zuul is super slow today. only UT failed on PS3 and I only fixed up tests in PS4. Would anyone be willing to manually verify I fixed UT (I ran locally and they pass) and we can merge? | 14:05 |
EmilienM | shardy: also, can you look https://review.openstack.org/#/c/347455/ please? | 14:06 |
bnemec | panda: Yes, there are instructions in the overcloud ssl docs for generating an appropriate certificate too. | 14:06 |
panda | KeystonAdminIP .. points to KeystoneAdminVirtualIP .. that points to IP_ADDRESS .. that points to PublicVIP, that points to KeystonAdminIP ... that points to ... | 14:06 |
bnemec | panda: The IP wasn't previously specified. You just used the first one in the external range. | 14:06 |
*** akshai has quit IRC | 14:06 | |
bnemec | Now something changed and that doesn't work. | 14:06 |
bnemec | So you explicitly assign the same IP you used for the certificate. | 14:07 |
*** akshai has joined #tripleo | 14:07 | |
bnemec | Or you set up DNS and point that at the IP, but that's extra hassle in a dev/test environment. | 14:07 |
panda | bnemec: thanks, the fact that network isolation is not used in CI explains a lot, the IPs were completely different from what I expected | 14:07 |
panda | bnemec: assigning the Ip would mean to change everything, because I woul have to assign the public controller IP to the External vlan10 interface , or something like that, right ? | 14:08 |
trown | thanks bnemec | 14:09 |
panda | If only I knew how to do it, of course | 14:09 |
bnemec | panda: Did you read the link I posted? It explains how to set the public vip. | 14:09 |
openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: Fix Aodh httpd binding https://review.openstack.org/348896 | 14:10 |
panda | bnemec: Yes, I've seen that, I was talking about using that certificate anyway, and instead rotating all the IPs to match what the certificate contains. | 14:11 |
bnemec | panda: Oh, I think it's probably easier to just generate your own cert. | 14:12 |
* bnemec expenses ccamacho a http://www.ebay.com/itm/Dell-PowerEdge-R710-3-5-Server-12-Cores-2-93-GHz-SLBV7-144GB-H700-6x-Caddies-RPS-/331915267681?hash=item4d47b10e61:g:OvYAAOSwwbdWN9qR | 14:13 | |
bnemec | ;-) | 14:13 |
panda | bnemec: it is, but since I'm struggling to find IP parameters around, that would have been an interesting excercise | 14:13 |
openstackgerrit | Merged openstack/tripleo-quickstart: Adds ci-scripts to for tripleo-quickstart on OVB https://review.openstack.org/347393 | 14:14 |
openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: added contrail database puppet module for tripleO https://review.openstack.org/348406 | 14:14 |
ccamacho | jajaja bnemec, thanks! | 14:14 |
bnemec | panda: You would change IP ranges in network-environment.yaml: http://docs.openstack.org/developer/tripleo-docs/advanced_deployment/network_isolation.html#create-network-environment-file | 14:15 |
openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: added contrail analytics puppet module for tripleO https://review.openstack.org/348899 | 14:16 |
openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: We don't need to set a default for the CephX keys and cluster FSID https://review.openstack.org/348900 | 14:17 |
EmilienM | gfidente: lol the commit message ^ | 14:18 |
bnemec | derekh: Are the new rh1 environment details somewhere? Want to make sure I have access before you disappear. :-) | 14:19 |
derekh | bnemec: did you see the mail I just sent? | 14:19 |
bnemec | derekh: lol, I just did | 14:19 |
gfidente | EmilienM yeah I talk bad about myself | 14:19 |
bnemec | derekh: Thanks | 14:20 |
derekh | bnemec: I *think* it has everything you need but worth a look over to make sure | 14:20 |
openstackgerrit | Merged openstack/instack-undercloud: Drop broken keystone pki setup https://review.openstack.org/327171 | 14:20 |
hewbrocca | derekh: should we ask pabelanger to watch over the infra requests? | 14:21 |
panda | bnemec: very interesting, thanks! sometimes one assumes that since he has seen and used a file a lot of times it's able to understand or modify its content .. I'll study this. | 14:21 |
openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: added contrail analytics puppet module for tripleO https://review.openstack.org/347739 | 14:21 |
openstackgerrit | Merged openstack-infra/tripleo-ci: Don't spam postci with deployment time loop https://review.openstack.org/346990 | 14:22 |
derekh | hewbrocca: sure, pabelanger I'm forwarding you an email about the rh1 switch over | 14:22 |
openstackgerrit | Merged openstack-infra/tripleo-ci: Record testenv wait times in ovb jobs https://review.openstack.org/347037 | 14:22 |
*** mcornea has quit IRC | 14:28 | |
*** mcornea has joined #tripleo | 14:29 | |
trown | panda: did not read your comment before merging https://review.openstack.org/#/c/348554/1 I think the idea there is just to get the test harness up for a lint job, and fix linting in subsequent patches | 14:31 |
openstackgerrit | Merged openstack/tripleo-quickstart: Add tox -elinters https://review.openstack.org/348554 | 14:32 |
panda | trown: yes, "before proceding" I intended "before making it voting", sorry for not being more specific | 14:32 |
trown | ah cool | 14:32 |
trown | you +1, or I would have noticed :) | 14:32 |
slagle | derekh: what tenant should I use if I just want to create some instances on rh2 for multinode ci job testing? | 14:34 |
slagle | it'd be 4 8GB instances | 14:35 |
derekh | slagle: "tripleo-user" , I think I sent you the creds at one stage | 14:35 |
openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: Added Contrail Composable Services https://review.openstack.org/348905 | 14:35 |
derekh | slagle: if not shout and I'll send them on | 14:35 |
openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Support arbitrary attributes for the Ceph pools https://review.openstack.org/346796 | 14:35 |
slagle | derekh: can you resend, the previous ones for "toci" tenant | 14:36 |
slagle | which i think was before the last redeploy | 14:36 |
slagle | derekh: wait, i have them | 14:37 |
slagle | derekh: yea, i got them. thx | 14:37 |
derekh | slagle: ack, just sent now anyways | 14:37 |
*** myoung|remote has quit IRC | 14:37 | |
derekh | slagle: if you use the template-centos-7-1469732382 image, thats the same image infra use | 14:38 |
slagle | cool, yea | 14:40 |
slagle | i'm going to use this to test some multinode HA jobs | 14:40 |
*** dsariel has joined #tripleo | 14:40 | |
pabelanger | morning | 14:42 |
*** david-lyle has joined #tripleo | 14:42 | |
pabelanger | derekh: replied to the email, but issue with tripleo-test-cloud-rh1 is we still need to launch AFS mirror in -rh1 | 14:42 |
hewbrocca | ahhh the AFS mirror! | 14:43 |
pabelanger | we should also create the 2nd tenant for nodepool to only launch nodes there, instead of a single tenant where everything g | 14:43 |
openstackgerrit | Merged openstack/tripleo-heat-templates: Stop passing charset=utf8 for neutron database connection option https://review.openstack.org/330490 | 14:43 |
*** jcoufal_ has joined #tripleo | 14:44 | |
pabelanger | it could be something like openstack-nodepool as the user | 14:44 |
pabelanger | sorry, some other user, as openstack-nodepool exists today | 14:45 |
pabelanger | openstackjenkins seems to be what otherclouds are using | 14:45 |
derekh | pabelanger: flavor created | 14:46 |
EmilienM | can I have a review on https://review.openstack.org/309140 please? | 14:46 |
EmilienM | and a last approve on https://review.openstack.org/347455 ? | 14:46 |
pabelanger | slagle: EmilienM: https://review.openstack.org/#/c/348493/ ready for review now. Reverts the facter FQDN patch | 14:46 |
*** jcoufal has quit IRC | 14:47 | |
derekh | pabelanger: I can create the second user now but wont have a chance to switch everything over, if it isn't pressing maybe somebody else could pick it up next week? | 14:47 |
EmilienM | pabelanger: +2 | 14:47 |
*** david-lyle has quit IRC | 14:47 | |
pabelanger | derekh: we can leave what you have today in the first tenant, as long as the 2nd tenant can still access that network I think we'll be fine | 14:48 |
shardy | EmilienM: was there a reason to recheck https://review.openstack.org/#/c/348493 when all the jobs passed? | 14:49 |
EmilienM | shardy: yes | 14:49 |
EmilienM | shardy: I was to make sure it works on different clouds but it looks fine, since it fixed rh2 job | 14:49 |
EmilienM | shardy: we can safely land it | 14:49 |
shardy | EmilienM: Ok, thanks for clarifying | 14:50 |
bandini | jistr: https://review.openstack.org/#/c/348816/ easiest review you will have today *PROMISE* (look at the depends-on too) ;) | 14:50 |
EmilienM | gfidente: I dont get why you "diff" in https://review.openstack.org/347860 ? | 14:51 |
jistr | hehe | 14:51 |
derekh | pabelanger: ok, so to confirm, I'll create a user "openstackjenkins" in a "openstackjenkins" project and you should be able to use that? the network is usable by all tenants | 14:51 |
* jistr looking | 14:51 | |
* bandini brings beer to jistr's desk | 14:51 | |
gfidente | EmilienM basically I diff against the pre-existing keyfile to see if there are differences with the new one | 14:51 |
gfidente | if there are, then a new keyring is generated | 14:51 |
gfidente | the purpose is to cover with a single 'onlyif' all possible differences, secret key and/or caps | 14:52 |
pabelanger | derekh: yup, then your local mirror and AFS will be the the current tenant. | 14:52 |
gfidente | EmilienM it's same purpose the sed had, but the sed was only grepping for the secret key so it wouldn't in fact regenerate the keyfile if the caps were changed | 14:52 |
jistr | bandini, gfidente: looks good. We need to land these in sync again. Ready? :) | 14:53 |
gfidente | jistr :) | 14:53 |
bandini | jistr: w00t! | 14:54 |
EmilienM | gfidente: wow, that's super smart | 14:54 |
EmilienM | gfidente: but so complex :P | 14:54 |
EmilienM | gfidente: should I +2 ? | 14:54 |
gfidente | it's not super smart though | 14:55 |
gfidente | don't lie to me | 14:55 |
slagle | derekh: I added my public key to the default keypair. Is that going to get updated automatically on the jumphost so I can ssh in? | 14:55 |
gfidente | it does the job | 14:55 |
gfidente | the sed was smarter, but it didn't | 14:55 |
openstackgerrit | Julie Pichon proposed openstack/tripleo-common: Mistral workflow for creating a deployment plan from JSON input https://review.openstack.org/347830 | 14:55 |
openstackgerrit | Merged openstack/python-tripleoclient: Remove another private OSC attribute usage https://review.openstack.org/348460 | 14:56 |
gfidente | EmilienM honestly, given the commands are idempotent, I was going to throw in the idea of removing the onlyif entirely | 14:56 |
EmilienM | gfidente: +2 | 14:56 |
derekh | slagle: I've updated the jump host | 14:56 |
EmilienM | trown: trunk blocker landed^ | 14:56 |
derekh | slagle: no was the answer ;-) | 14:56 |
slagle | derekh: thanks :). i'm in now | 14:57 |
gfidente | EmilienM but then I think puppet idempotency would kill me because it doesn't want exec launched twice with same arguments | 14:57 |
gfidente | so ... | 14:57 |
*** jdob has quit IRC | 14:57 | |
*** jdob has joined #tripleo | 14:57 | |
derekh | pabelanger: user created, same password as the other nodepool user, want me to generate a new password? | 14:58 |
derekh | brb | 14:58 |
pabelanger | derekh: ya, lets do a new password | 14:58 |
*** dtrainor has joined #tripleo | 14:59 | |
*** dtrainor has quit IRC | 15:00 | |
*** dtrainor has joined #tripleo | 15:01 | |
openstackgerrit | Merged openstack/instack-undercloud: Revert "Explicitly set $FACTER_fqdn" https://review.openstack.org/348493 | 15:02 |
openstackgerrit | Merged openstack/instack-undercloud: Allow to configure Ironic [ssh] section https://review.openstack.org/347455 | 15:02 |
openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart: Remove the obsolete rebuild-inventory role https://review.openstack.org/348917 | 15:04 |
openstackgerrit | Merged openstack/puppet-tripleo: Move constraints to their respective services https://review.openstack.org/348815 | 15:04 |
*** jdob has quit IRC | 15:07 | |
*** jdob has joined #tripleo | 15:07 | |
*** jistr is now known as jistr|call | 15:10 | |
*** jcoufal_ has quit IRC | 15:11 | |
derekh | pabelanger: new password set, where do you want it? | 15:11 |
*** jcoufal has joined #tripleo | 15:12 | |
*** jistr|call is now known as jistr | 15:17 | |
*** eggmaster has quit IRC | 15:20 | |
gfidente | jistr bandini sounds like we want a further -1 +1 ? | 15:22 |
gfidente | https://review.openstack.org/#/c/348816/ | 15:22 |
*** ayoung has joined #tripleo | 15:22 | |
bandini | gfidente: I have no idea what to do there | 15:23 |
bandini | but we need to merge it quickly because the puppet-tripleo counterpart went in already | 15:23 |
pabelanger | derekh: can we both SSH into some common node? | 15:23 |
gfidente | I think jistr can -1 +1 again | 15:23 |
*** david-lyle has joined #tripleo | 15:23 | |
gfidente | unless recheck does it | 15:24 |
jistr | gfidente: i did | 15:24 |
* bandini passes the magic wand to jistr | 15:24 | |
jistr | gfidente: reverify should do it | 15:24 |
bandini | we tried that | 15:24 |
gfidente | ahahah | 15:24 |
gfidente | crazy moments | 15:24 |
jistr | gfidente, bandini: if we aren't able to merge it then we need to push a revert for the other one in the interim | 15:24 |
bandini | gfidente: argh you did a recheck? | 15:25 |
bandini | won't that trigger the whole CI now? | 15:25 |
gfidente | yeah I tried that too | 15:25 |
*** weshay is now known as weshay_bbiab | 15:25 | |
gfidente | it's ongoing no panic http://status.openstack.org/zuul/ | 15:26 |
gfidente | just didn't move to gate queue yet | 15:26 |
bandini | ack let's give it a bit then | 15:27 |
jistr | gfidente, bandini: "easiest review you will have today *PROMISE*" | 15:27 |
derekh | pabelanger: ssh root@66.187.229.46 cat password | 15:27 |
jistr | ^ quoted literally | 15:27 |
bandini | jistr: ahahahah | 15:27 |
jistr | well played bandini | 15:27 |
gfidente | ahaha | 15:27 |
bandini | jistr: this was zuul's fault. my innocence is impeccable on this one ;) | 15:28 |
jistr | yea i know :) | 15:28 |
gfidente | bandini are you admitting there have been other circumstances where you weren't innocent? | 15:28 |
bandini | I am at fault like *all the time* except this one ;) | 15:28 |
gfidente | I knew we had something in common | 15:29 |
bandini | ahahah | 15:29 |
hewbrocca | gfidente: knows the underside of the bus so well he could draw you a picture of it | 15:29 |
EmilienM | slagle: https://review.openstack.org/348927 | 15:30 |
EmilienM | bnemec: ^ | 15:30 |
gfidente | hewbrocca I got marios to explain me what was the story with the bus | 15:31 |
gfidente | hewbrocca you're unfair | 15:31 |
hewbrocca | LOL | 15:31 |
gfidente | only those who don't do don't see the bus! | 15:31 |
bnemec | EmilienM: The idea is we'll run it on stable/newton some day when that branch is a thing? | 15:31 |
slagle | ayoung: yesterday, were you expressing some concerns about having multiple OpenStack services running under httpd/wsgi alongside keystone? | 15:32 |
hewbrocca | I would blame marios, he's the one who's always throwing you under there | 15:32 |
EmilienM | bnemec: yeah, see my regex | 15:32 |
ayoung | slagle, actually, just the opposite | 15:32 |
EmilienM | bnemec: we just don't run it for stable/liberty and stable/mitaka | 15:32 |
ayoung | slagle, I want Keystone to run in its own container | 15:32 |
EmilienM | bnemec: I did a typo | 15:32 |
EmilienM | bnemec: now it's good. | 15:32 |
ayoung | slagle, so I don't want a "HTTPD" role, I want a Keystone role when we split things out | 15:33 |
bnemec | EmilienM: Okay, just wanted to confirm that was the plan. | 15:33 |
*** rwsu has quit IRC | 15:33 | |
slagle | ayoung: ok. but are there known issues if we put multiple services running under the same httpd? | 15:33 |
slagle | ayoung: i'm curious b/c of https://review.openstack.org/#/c/309140/ for the undercloud | 15:33 |
slagle | where we already have keystone under httpd | 15:33 |
ayoung | slagle, Not now there is not. Keystone and HTTPD run side by side just fine | 15:33 |
ayoung | slagle, undercloud should be OK that way, too | 15:34 |
slagle | ok | 15:34 |
*** anshul has quit IRC | 15:34 | |
*** leanderthal is now known as leanderthal|afk | 15:34 | |
marios | hewbrocca: i am in a call so no idea what you're discussing, but YES it was gfidente fault for sure. | 15:34 |
ayoung | slagle, I am thinking minimal downtime upgrades. WHen upgrading a system, Keystone needs to be upgradable first and we are working hard to make that work without bringing down the cluster | 15:35 |
jistr | bandini, gfidente: well you know what's bad, that until the lint gate is fixed, we aren't able to push a revert either | 15:35 |
jistr | it will fail on the same thing | 15:35 |
ayoung | slagle, but part of that is that we need to upgrade the code before the database, and doing will likely potentially bring down the other services. THere are library dependdencies (all the oslo-*) that force a lockstep upgrade | 15:36 |
gfidente | jistr but people will blame gate | 15:36 |
* bnemec throws the gate under the bus with gfidente | 15:36 | |
gfidente | like marios | 15:36 |
ayoung | slagle, and Horizon links in quite a few dependencies itself, especially all of the clients, so I would not want to force it into the same container as Keystone. Undermining ourselves if we do | 15:37 |
bandini | jistr: but it failed on a timeout, so chances are it will work on a retry | 15:37 |
bandini | jistr: it was a timeout reaching some ruby gem | 15:37 |
bandini | unless the whole ruby gem download ecosystem is down that is :) | 15:37 |
gfidente | bnemec the "dictionary update sequence element #3 has length 4;" | 15:37 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Configure Nova [cache] options correctly (Mitaka only) https://review.openstack.org/348255 | 15:37 |
gfidente | strange story where dict['key'] fails when value contains equals | 15:37 |
gfidente | password=randomstring== | 15:38 |
gfidente | .get() is safe though | 15:38 |
hewbrocca | the ruby gem download ecosystem is not infrequently down | 15:38 |
slagle | ayoung: ok, i understand | 15:38 |
gfidente | bnemec ever seen it? | 15:38 |
hewbrocca | gfidente: ^^^ see?? | 15:38 |
bnemec | gfidente: Wha? | 15:38 |
bnemec | That's insane. | 15:38 |
slagle | ayoung: i think these are all things to consider for the overcloud going forward. but i suspect for the single node undercloud, we're likely ok | 15:38 |
* bnemec tries it | 15:38 | |
gfidente | bnemec yeah but I just tried it locally and that's it | 15:39 |
ayoung | slagle, correct | 15:39 |
gfidente | bnemec yes please :) | 15:39 |
jistr | bandini, gfidente: hmm i wasn't able to find a single running lint job ATM, they seem to be all "queued" in zuul, at least as for what i was able to dig up | 15:40 |
gfidente | needless to say cephx keys are base64 encoded and append those == | 15:40 |
bnemec | gfidente: I can't reproduce it on my laptop. I wonder if this is a weird bug in the Python on centos 7. | 15:42 |
bnemec | foo = {} | 15:42 |
bnemec | foo['foo'] = 'something==' | 15:42 |
bnemec | print foo['foo'] | 15:42 |
bnemec | Works fine for me. | 15:42 |
*** radek_ has quit IRC | 15:42 | |
gfidente | bnemec though try append an = to one of the passwords in tripleo-overcloud-passwords | 15:42 |
gfidente | if you put the value in quotes it also works yes | 15:42 |
gfidente | but we don't put them in quotes | 15:42 |
gfidente | I mean password="string==" is also fine | 15:43 |
gfidente | bnemec if you edit tripleo-overcloud-passwords appending a = and try a deploy it will fail like that | 15:43 |
bandini | jistr: do you know who we can ping here to unblock things? | 15:43 |
jistr | i guess some combination of #puppet-openstack and #openstack-infra | 15:44 |
bnemec | gfidente: Oh, it's reading the file back in, not the Python dict that is failing. | 15:44 |
gfidente | yeah | 15:44 |
jistr | EmilienM might know | 15:44 |
bnemec | I understand now. | 15:44 |
gfidente | I realized it now as well | 15:44 |
bnemec | I think we can fix that. One second | 15:44 |
gfidente | as I was talking about the file | 15:44 |
bnemec | gfidente: We need to pass maxsplit=1 when reading the file back in: https://docs.python.org/2/library/stdtypes.html#str.split | 15:45 |
gfidente | bnemec ack | 15:45 |
EmilienM | jistr, bandini: what's up? | 15:46 |
gfidente | bnemec thanks :) | 15:46 |
pabelanger | derekh: nice! need to remember that | 15:47 |
jistr | EmilienM: this https://bugs.launchpad.net/tripleo/+bug/1607836 | 15:47 |
openstack | Launchpad bug 1607836 in tripleo "CI down on chicken-and-egg patch unable to merge because of puppet-lint job" [Critical,Triaged] | 15:47 |
jistr | i tagged it with alert so that it pops up on IRC here until the patch is able to merge | 15:48 |
EmilienM | jistr: why can't we land the tht now? | 15:48 |
gfidente | bnemec though it says the rightmost split is done | 15:48 |
jistr | EmilienM: we'd love to but puppet-lint blocks it | 15:48 |
gfidente | which is not what we want :P | 15:48 |
gfidente | oh no lsplit | 15:48 |
gfidente | is not rsplit | 15:48 |
EmilienM | jistr: it's in check queue now, let's see | 15:49 |
gfidente | split is not rsplit | 15:49 |
jistr | yea | 15:49 |
jistr | actually that one will just update the check, won't it | 15:49 |
jistr | i'll try reverify again | 15:49 |
jistr | so that we get another pass at the 'gate' section | 15:49 |
EmilienM | jistr: let it going now | 15:49 |
EmilienM | it's in check, and I think it will work | 15:50 |
openstackgerrit | Giulio Fidente proposed openstack/python-tripleoclient: Generate CephX keys via generate_overcloud_passwords https://review.openstack.org/348412 | 15:50 |
bnemec | gfidente: I'm a little confused why we're even reading the passwords back in on the initial deploy though. | 15:50 |
EmilienM | we can't skip 'check' | 15:50 |
jistr | EmilienM: we didn't, check is green | 15:50 |
EmilienM | jistr: will see how it works | 15:51 |
EmilienM | we have no choice to wait now :) | 15:51 |
gfidente | bnemec generate_ just always does it to see if the password is already there I think | 15:51 |
bnemec | gfidente: Yeah, but the file isn't there the first time. | 15:51 |
jistr | ok | 15:51 |
bnemec | So it should skip that block entirely. | 15:51 |
bnemec | I mean, I'm glad we caught this now. But I don't understand why. | 15:52 |
bnemec | gfidente: Oh, it's because we create a password cache that also calls generate: https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/utils.py#L525 | 15:54 |
gfidente | _password_cache | 15:54 |
dmsimard | gfidente: ping | 15:56 |
bnemec | gfidente: I think that's going to fail pep8 due to line length. | 15:56 |
gfidente | dmsimard pong | 15:56 |
dmsimard | gfidente: what ceph version is expected/tested to work with newton ? | 15:56 |
gfidente | bnemec amending | 15:56 |
dmsimard | gfidente: it doesn't look like documentation mentions setting up a particular ceph repository so if it's pulling from EPEL, that's 0.80 and much older than even hammer (0.94) or jewel | 15:57 |
dmsimard | gfidente: we're testing puppet-openstack-integration against jewel now. | 15:57 |
EmilienM | gfidente: FYI https://review.openstack.org/#/c/348489/2/puppet/services/ceph-base.yaml | 15:57 |
gfidente | dmsimard we need to land https://review.openstack.org/#/c/340504/ and https://review.openstack.org/#/c/343667/ | 15:58 |
gfidente | EmilienM so we had a BZ tracking that | 15:58 |
*** panda is now known as panda|bbl | 15:58 | |
EmilienM | gfidente: yea, just an fyi | 15:58 |
gfidente | and we can't switch to the new path as easily because it's problematic on upgrades | 15:58 |
gfidente | so we need a plan for that | 15:58 |
EmilienM | I had selinux issues because of it | 15:58 |
EmilienM | gfidente: right. | 15:59 |
EmilienM | just an fyi :) | 15:59 |
gfidente | :( | 15:59 |
*** jcoufal has quit IRC | 15:59 | |
*** jcoufal has joined #tripleo | 15:59 | |
dmsimard | gfidente: ack, so is it hammer or jewel then ? :p | 16:00 |
dmsimard | gfidente: someone in #rdo needs to know because he's trying things out for rdo test day. | 16:00 |
*** pcaruana has quit IRC | 16:01 | |
gfidente | EmilienM see what you think of the comment I left, marios can help as well :) | 16:01 |
gfidente | dmsimard it's hammer from EPEL | 16:02 |
dmsimard | gfidente: EPEL provides 0.80, that's not hammer | 16:02 |
dmsimard | hammer is 0.94 | 16:02 |
EmilienM | gfidente: right now, we should just merge the patch | 16:02 |
EmilienM | and see later | 16:02 |
*** dsariel has quit IRC | 16:02 | |
openstackgerrit | Merged openstack/instack-undercloud: Deploy and Upgrade Ironic to run in mod_wsgi https://review.openstack.org/309140 | 16:02 |
dmsimard | 0.80 is firefly | 16:03 |
*** mcornea has quit IRC | 16:04 | |
gfidente | dmsimard oh it's 0.94 in the rhel image ... | 16:04 |
dmsimard | gfidente: so centos-release-ceph-hammer should work then | 16:04 |
dmsimard | thanks | 16:04 |
gfidente | dmsimard so I even had a submission for that | 16:04 |
gfidente | one of the previous patchsets for that submission was using the centos sig packages | 16:05 |
gfidente | dmsimard https://review.openstack.org/#/c/340504/3/scripts/tripleo.sh | 16:06 |
gfidente | I think we also have to figure if our preference is for the centos/sig version or the ceph.com version of the packages | 16:06 |
*** paramite has quit IRC | 16:06 | |
dmsimard | gfidente: yeah, the release package is not yet available for jewel last I know | 16:06 |
dmsimard | gfidente: the problem with using ceph.com repositories | 16:06 |
*** hewbrocca is now known as hewbrocca-afk | 16:06 | |
dmsimard | gfidente: is that the dependencies are not self contained and not available in CentOS base OS | 16:07 |
dmsimard | for example libleveldb is only in EPEL | 16:07 |
gfidente | but centos+epel right | 16:07 |
dmsimard | so if you use ceph.com repo, you need to have epel | 16:07 |
zoliXXL | have a good weekend | 16:07 |
dmsimard | but if you have the SIG repo, libleveldb is there | 16:07 |
gfidente | though I think we're good with that because we'd be using those for the upstream CI only | 16:07 |
dmsimard | it's self contained | 16:07 |
dmsimard | EPEL shouldn't be used anywhere | 16:07 |
dmsimard | it's the devil | 16:07 |
gfidente | oh well I think we need it in tripleo | 16:07 |
EmilienM | nope | 16:08 |
EmilienM | there is some WIP to remove it in tripleo-ci | 16:08 |
dmsimard | if you need it, you need to identify what you need | 16:08 |
*** zoliXXL is now known as zoli|gone | 16:08 | |
dmsimard | so we can pull the dependencies either in RDO or elsewhere | 16:08 |
EmilienM | believe me, you don't need EPEL to deploy TripleO | 16:08 |
dmsimard | we can't control EPEL, they've broken us more than once | 16:08 |
gfidente | but the ceph.com is going to depend on EPEL | 16:08 |
openstackgerrit | Giulio Fidente proposed openstack/python-tripleoclient: Generate CephX keys via generate_overcloud_passwords https://review.openstack.org/348412 | 16:09 |
*** zoli|gone is now known as zoli_gone-proxy | 16:09 | |
*** ooolpbot has joined #tripleo | 16:10 | |
ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 16:10 |
ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1607836 | 16:10 |
*** ooolpbot has quit IRC | 16:10 | |
openstack | Launchpad bug 1607836 in tripleo "CI down on chicken-and-egg patch unable to merge because of puppet-lint job" [Critical,Triaged] | 16:10 |
gfidente | and centos/sig doesn't have jewel yet | 16:10 |
*** jeckersb_gone is now known as jeckersb | 16:11 | |
dmsimard | gfidente: it does | 16:11 |
EmilienM | gfidente: they do | 16:11 |
EmilienM | we use it | 16:11 |
EmilienM | a sec | 16:12 |
dmsimard | gfidente: http://buildlogs.centos.org/centos/7/storage/x86_64/ceph-jewel/ | 16:12 |
dmsimard | gfidente: there's just no release RPM for it yet, not sure why I forget | 16:12 |
EmilienM | https://github.com/openstack/puppet-ceph/blob/master/manifests/repo.pp#L126-L132 | 16:12 |
EmilienM | just set ceph::repo::enable_sig: true and it will deploy the repo for you :) | 16:12 |
*** numans has quit IRC | 16:12 | |
openstackgerrit | ayoung proposed openstack/python-tripleoclient: Clear the old environment in rc file https://review.openstack.org/347553 | 16:12 |
openstackgerrit | ayoung proposed openstack/python-tripleoclient: Add an overcloud rc.v3 file https://review.openstack.org/347554 | 16:12 |
derekh | pabelanger: you got that password? /me is going to delete the instance | 16:12 |
gfidente | dmsimard so you liked https://review.openstack.org/#/c/340504/3/scripts/tripleo.sh more? | 16:13 |
EmilienM | gfidente: /me likes :) | 16:13 |
dmsimard | gfidente: yeah | 16:13 |
gfidente | if that looks good we can just restore patchset #3 | 16:13 |
gfidente | fine :) | 16:13 |
*** jcoufal has quit IRC | 16:13 | |
pabelanger | derekh: yup, delete away | 16:13 |
derekh | pabelanger: ack, done | 16:14 |
*** jcoufal has joined #tripleo | 16:14 | |
openstackgerrit | Carlos Camacho proposed openstack/puppet-tripleo: Implement aodh profiles https://review.openstack.org/332854 | 16:15 |
openstackgerrit | Carlos Camacho proposed openstack/tripleo-heat-templates: Add Aodh composable roles https://review.openstack.org/333556 | 16:17 |
ccamacho | bandini, EmilienM Im hoping this is the last error from composable aodh http://paste.openstack.org/show/543930/ this is related to the pacemaker constraints, but Im not fully aware about how they work, when having time mid to check how they should be referenced? | 16:19 |
bandini | ccamacho: looking | 16:19 |
ccamacho | s/mid/mind/ | 16:19 |
EmilienM | ccamacho: ok I'll monitor that in the week end | 16:19 |
ccamacho | EmilienM thanks! | 16:20 |
openstackgerrit | Giulio Fidente proposed openstack-infra/tripleo-ci: Install Ceph Hammer and/or Jewel from CentOS Storage SIG repos https://review.openstack.org/340504 | 16:20 |
ccamacho | bandini they are defined here https://review.openstack.org/#/c/332854/10/manifests/profile/pacemaker/aodh.pp | 16:20 |
pabelanger | gfidente: we have ceph mirrors in openstack-infra, can we use them? | 16:21 |
bandini | ccamacho: i see the problem, sec | 16:22 |
gfidente | pabelanger ceph.com mirrors or centos storage sig mirrors? | 16:22 |
pabelanger | gfidente: ceph.com mirrors | 16:23 |
gfidente | pabelanger ^^ so we just discussed switching to centos storage sig | 16:23 |
pabelanger | I believe that is what EmilienM is using for puppet testing too | 16:23 |
ccamacho | bandini <3 | 16:23 |
gfidente | pabelanger I think EmilienM is using SIG? | 16:23 |
*** florianf has quit IRC | 16:23 | |
*** weshay_bbiab is now known as weshay | 16:24 | |
pabelanger | dmsimard: is there any way we can fix this in ceph.com repos? | 16:24 |
EmilienM | gfidente: I do | 16:25 |
EmilienM | we use RDO + SIG without EPEL | 16:25 |
dmsimard | pabelanger: upstream ceph doesn't want to carry/build/package/maintain all of their dependencies | 16:25 |
dmsimard | pabelanger: I asked | 16:25 |
bandini | ccamacho: see PM | 16:25 |
*** trown is now known as trown|lunch | 16:25 | |
ccamacho | bandini looking | 16:25 |
pabelanger | dmsimard: ya, that make sense. So they depend on EPEL | 16:25 |
beagles | is dprince on pto or something this week? | 16:25 |
dmsimard | pabelanger: right. Ceph won't even install on CentOS without EPEL due to lack of dependencies. | 16:26 |
*** jpich has quit IRC | 16:26 | |
dmsimard | pabelanger: but the CentOS storage SIG repos for Ceph maintain all the dependencies themselves and they are self-contained | 16:26 |
dmsimard | so no need for EPEL. | 16:27 |
pabelanger | oh wait | 16:27 |
pabelanger | what is the URL for the SIG repo? | 16:27 |
pabelanger | http://mirror.centos.org/centos/7/storage/x86_64/ceph-hammer/ ? | 16:27 |
pabelanger | dmsimard: gfidente ^ | 16:28 |
pabelanger | if that is the URL, we I can add it to AFS | 16:28 |
dmsimard | pabelanger: that's hammer, there is one for jewel that will be available soon | 16:28 |
dmsimard | pabelanger: there are signed packages to install the repos | 16:28 |
pabelanger | dmsimard: will it be in mirror.centos.org? | 16:28 |
dmsimard | pabelanger: yeah | 16:28 |
pabelanger | okay | 16:28 |
pabelanger | great | 16:28 |
pabelanger | then I can mirror that into our AFS | 16:28 |
pabelanger | and you'll be able to use it | 16:28 |
dmsimard | pabelanger: it's on buildlogs until released: http://buildlogs.centos.org/centos/7/storage/x86_64/ | 16:28 |
pabelanger | let me add it | 16:28 |
pabelanger | dmsimard: right | 16:29 |
openstackgerrit | Merged openstack-infra/tripleo-ci: Source rh1.env to decide how many testenv workers to run https://review.openstack.org/348883 | 16:29 |
pabelanger | dmsimard: so, you'll have to use buildlogs for now | 16:29 |
pabelanger | but I can mirror hammer | 16:29 |
dmsimard | pabelanger: that's what we're doing in puppet, yeah | 16:29 |
gfidente | pabelanger though I'd have to mangle the .repo file to point to afs then right? | 16:29 |
*** tesseract- has quit IRC | 16:30 | |
*** dcotton has quit IRC | 16:30 | |
pabelanger | gfidente: for the moment yes, but my patch will drop the .repo file into the folder disabled. you'll just need to enable it | 16:30 |
pabelanger | gfidente: that will take a few days to land hower | 16:30 |
pabelanger | however* | 16:30 |
gfidente | pabelanger ok can you comment on the submission then with some instructions in case it isn't landed yet? | 16:30 |
pabelanger | gfidente: yes | 16:30 |
gfidente | pabelanger or even follow up on it with the needed changes | 16:30 |
gfidente | your call | 16:30 |
pabelanger | gfidente: if you can way a few more days, lets fix your patch. Not sure if that holds up development | 16:31 |
gfidente | ack | 16:31 |
openstackgerrit | Jeff Peeler proposed openstack/tripleo-common: Change output for GetCapabilitiesAction https://review.openstack.org/348537 | 16:32 |
openstackgerrit | Jeff Peeler proposed openstack/tripleo-common: Change output for GetCapabilitiesAction https://review.openstack.org/348537 | 16:33 |
openstackgerrit | Jeff Peeler proposed openstack/tripleo-common: Change output for GetCapabilitiesAction https://review.openstack.org/348537 | 16:34 |
*** paramite has joined #tripleo | 16:36 | |
gfidente | so I am leaving guys, thanks dmsimard for helping with the repos | 16:36 |
dmsimard | gfidente: np | 16:37 |
gfidente | trown|lunch on the docs issue I agree but I am not sure where to push the docs | 16:37 |
gfidente | trown|lunch if you have 10 mins please comment on the same submission with a few info on where to push the docs and what you'd like to see documented | 16:38 |
gfidente | unless you meant the tripleo-docs? | 16:38 |
*** gfidente has quit IRC | 16:39 | |
*** lucasagomes is now known as lucas-dinner | 16:41 | |
EmilienM | jistr: ok indeed, all lint jobs are now broken :) | 16:44 |
EmilienM | ah no, it's a timeout tis time http://logs.openstack.org/93/348893/1/check/gate-instack-undercloud-puppet-lint/6a5feb9/console.html#_2016-07-29_16_24_47_678232 | 16:44 |
*** rhallisey has quit IRC | 16:52 | |
*** jbadiapa has quit IRC | 16:52 | |
*** yamahata has joined #tripleo | 16:55 | |
*** thrash is now known as thrash|biab | 16:56 | |
*** derekh has quit IRC | 16:57 | |
beagles | EmilienM, got bit by that as well | 17:01 |
beagles | https://youtu.be/rWIcubkPaxo | 17:02 |
openstackgerrit | Honza Pokorny proposed openstack/tripleo-ui: Add visual indicator that a deploy was requested https://review.openstack.org/339120 | 17:05 |
openstackgerrit | Carlos Camacho proposed openstack/puppet-tripleo: Implement aodh profiles https://review.openstack.org/332854 | 17:08 |
openstackgerrit | Merged openstack/instack-undercloud: Remove nova-compute restart at end of uc install https://review.openstack.org/347389 | 17:09 |
*** snecklifter has joined #tripleo | 17:09 | |
*** paramite has quit IRC | 17:09 | |
*** ooolpbot has joined #tripleo | 17:10 | |
ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 17:10 |
ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1607836 | 17:10 |
*** ooolpbot has quit IRC | 17:10 | |
openstack | Launchpad bug 1607836 in tripleo "CI down on chicken-and-egg patch unable to merge because of puppet-lint job" [Critical,Triaged] | 17:10 |
ccamacho | bandini, your wise advise made the trick, the aodh submission is working for me both ha and nonha locally, EmilienM we can check it later. | 17:12 |
bandini | ccamacho: w00t! | 17:12 |
*** tosky has quit IRC | 17:14 | |
ccamacho | **me going for some beers! Have a nice weekend guys!!! | 17:14 |
bandini | ccamacho: enjoy ;) | 17:14 |
*** ccamacho has quit IRC | 17:15 | |
*** milan has quit IRC | 17:17 | |
social | EmilienM: http://logs.openstack.org/69/232369/11/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/a168c17/console.html so I guess patch the patch? | 17:24 |
openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Add EnabledServices output to overcloud.yaml https://review.openstack.org/348441 | 17:25 |
openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Convert AllNodesConfig hosts config to a map https://review.openstack.org/348972 | 17:25 |
openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Create per-role node_names in hieradata for all nodes https://review.openstack.org/348973 | 17:25 |
openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: WIP Create composable mapping between enabled services and role ips https://review.openstack.org/348974 | 17:25 |
EmilienM | social: what is the context, please. | 17:28 |
*** rwsu has joined #tripleo | 17:31 | |
social | EmilienM: gfidente's patch https://review.openstack.org/#/c/232369 | 17:32 |
*** shardy has quit IRC | 17:33 | |
EmilienM | http://logs.openstack.org/69/232369/11/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/a168c17/console.html#_2016-07-29_14_18_18_205007 | 17:33 |
EmilienM | it fails here | 17:34 |
social | egafford: 2016-07-29 14:18:18.911631 | Notice: /Stage[main]/Nova::Compute::Libvirt::Services/Service[libvirt]/ensure: ensure changed 'stopped' to 'running' | 17:36 |
openstackgerrit | Honza Pokorny proposed openstack/tripleo-ui: Add npm version to webpack output https://review.openstack.org/348976 | 17:36 |
social | EmilienM: so it starts libvirt after the command | 17:36 |
EmilienM | it's something you can easily fix with puppet | 17:37 |
social | hah 4 lines lower it's fixed | 17:37 |
egafford | social: Did you mean to ping me there or was there a namespace collision? | 17:38 |
*** trown|lunch is now known as trown | 17:39 | |
social | egafford: collision | 17:39 |
egafford | social: ^5 | 17:39 |
*** mgould is now known as mgould|afk | 17:44 | |
*** rhallisey_ has joined #tripleo | 17:48 | |
EmilienM | bnemec, slagle : I have difficulties to understand how puppet modules are deploying in tripleo stable jobs | 17:53 |
*** links has joined #tripleo | 17:53 | |
EmilienM | the undercloud upgrade job is failing because of puppet modules are too recent for puppet-oslo | 17:54 |
*** jprovazn has quit IRC | 17:59 | |
*** jprovazn has joined #tripleo | 18:02 | |
openstackgerrit | Paul Belanger proposed openstack/tripleo-quickstart: Fix E226 missing whitespace around arithmetic operator https://review.openstack.org/348999 | 18:08 |
openstackgerrit | Paul Belanger proposed openstack/tripleo-quickstart: Use ansible 2.1 importing of boilerplate code https://review.openstack.org/349000 | 18:08 |
openstackgerrit | Paul Belanger proposed openstack/tripleo-quickstart: Fix H102 Apache 2.0 license header not found https://review.openstack.org/349001 | 18:08 |
*** ooolpbot has joined #tripleo | 18:10 | |
ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 18:10 |
ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1607836 | 18:10 |
*** ooolpbot has quit IRC | 18:10 | |
openstack | Launchpad bug 1607836 in tripleo "CI down on chicken-and-egg patch unable to merge because of puppet-lint job" [Critical,Triaged] | 18:10 |
ayoung | EmilienM, is there support in the puppet modules for managing the horizon file /usr/share/openstack-dashboard/openstack_dashboard/local/local_settings.py | 18:19 |
ayoung | Do we really keep a copy of the that file in the puppet modules? http://git.openstack.org/cgit/openstack/puppet-horizon/tree/templates/local_settings.py.erb | 18:21 |
ayoung | Shudder | 18:21 |
EmilienM | ayoung: it is how the module was written, yes. | 18:23 |
ayoung | EmilienM, that might be problematic. It means that we drop any changes that happen in upstream Horizon and need to resync. I can see why it is a template, due to the complexity of parsing, but I would have thought Puppet (written in Ruby) would have included a Full python parser inside it.... | 18:24 |
openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Use delorean-current for puppet modules https://review.openstack.org/341429 | 18:24 |
ayoung | Anyway, from a WebSSO perspective, that template needs triggers to be able to handle any webSSO specific parameters, and we need to be able to pass them through Hiera | 18:25 |
EmilienM | bnemec, slagle: I rebased shardy's patch about puppet modules deployment, feedback is welcome. | 18:27 |
EmilienM | ayoung: just send a patch to puppet-keystone | 18:27 |
EmilienM | or file a bug and I'll do it | 18:27 |
*** electrofelix has quit IRC | 18:28 | |
ayoung | EmilienM, puppet-horizon, will do | 18:30 |
EmilienM | err | 18:30 |
EmilienM | sorry | 18:30 |
EmilienM | just an FYI for everyone, current status of OVB HA job is broken, we need https://review.openstack.org/#/c/348816 which is in gate now | 18:31 |
*** thrash|biab is now known as thrash | 18:34 | |
ayoung | EmilienM, https://bugs.launchpad.net/puppet-horizon/+bug/1607912 | 18:35 |
openstack | Launchpad bug 1607912 in puppet-horizon "WebSSO config options" [Undecided,New] | 18:35 |
*** akrivoka has quit IRC | 18:36 | |
EmilienM | slagle: tripleo CI is currently red until 348816 gets merged | 18:38 |
EmilienM | slagle: I'm about to ask a promotion of the zuul jobs on infra channel | 18:38 |
*** tzumainn has quit IRC | 18:40 | |
beagles | EmilienM, thanks for the heads up | 18:42 |
*** jcoufal has quit IRC | 18:43 | |
*** jcoufal has joined #tripleo | 18:44 | |
openstackgerrit | Merged openstack/instack-undercloud: Fix Aodh httpd binding https://review.openstack.org/348896 | 18:46 |
*** yamahata has quit IRC | 18:46 | |
*** panda|bbl is now known as panda | 18:46 | |
EmilienM | slagle: fyi, no need to do rechecks, like I said, we need the patch merged before | 18:51 |
slagle | EmilienM: Huh? | 18:51 |
slagle | i havent done any rechecks. since you told me 13 minutes ago | 18:51 |
EmilienM | slagle: yeah you did before, just an fyi in case you missed my message | 18:52 |
EmilienM | I asked to promote the CI job | 18:52 |
EmilienM | but nobody replied to me | 18:52 |
slagle | EmilienM: where is the bug on this? | 18:52 |
EmilienM | so our gate is currently red | 18:52 |
slagle | can you add the alert tag? | 18:52 |
EmilienM | https://bugs.launchpad.net/tripleo/+bug/1607836 | 18:52 |
openstack | Launchpad bug 1607836 in tripleo "CI down on chicken-and-egg patch unable to merge because of puppet-lint job" [Critical,Triaged] | 18:52 |
slagle | then people will know | 18:52 |
EmilienM | it's in alert | 18:52 |
slagle | ok great | 18:52 |
EmilienM | I'm going try again on infra channel | 18:52 |
slagle | EmilienM: the multinode job also uses pacemaker, yet it is passing | 19:05 |
slagle | is it bacause it's not ha? | 19:06 |
EmilienM | yeah probably something required on mutli node | 19:06 |
bandini | the review failed on unrelated stuff it seems | 19:06 |
*** pkovar has quit IRC | 19:09 | |
*** ooolpbot has joined #tripleo | 19:10 | |
ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 19:10 |
ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1607836 | 19:10 |
*** ooolpbot has quit IRC | 19:10 | |
openstack | Launchpad bug 1607836 in tripleo "CI down on chicken-and-egg patch unable to merge because of puppet-lint job" [Critical,In progress] - Assigned to Emilien Macchi (emilienm) | 19:10 |
*** bnemec is now known as beekneemech | 19:12 | |
*** penick has joined #tripleo | 19:14 | |
*** fzdarsky is now known as fzdarsky|afk | 19:19 | |
*** fzdarsky|afk has quit IRC | 19:19 | |
*** links has quit IRC | 19:20 | |
openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Implement a new job for undercloud testing only https://review.openstack.org/346147 | 19:22 |
openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Implement a new job for undercloud testing only https://review.openstack.org/346147 | 19:22 |
openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: WIP - Implement undercloud upgrade job - Mitaka -> Newton https://review.openstack.org/346995 | 19:22 |
*** rwsu has quit IRC | 19:27 | |
*** myoung|remote has joined #tripleo | 19:29 | |
*** rhallisey_ is now known as rhallisey | 19:42 | |
*** yamahata has joined #tripleo | 19:43 | |
*** jprovazn has quit IRC | 19:54 | |
*** jdob has quit IRC | 20:00 | |
openstackgerrit | Merged openstack/tripleo-heat-templates: Move constraints to their respective services https://review.openstack.org/348816 | 20:06 |
*** ooolpbot has joined #tripleo | 20:10 | |
ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 20:10 |
ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1607836 | 20:10 |
*** ooolpbot has quit IRC | 20:10 | |
openstack | Launchpad bug 1607836 in tripleo "CI down on chicken-and-egg patch unable to merge because of puppet-lint job" [Critical,In progress] - Assigned to Emilien Macchi (emilienm) | 20:10 |
EmilienM | oh wow it landed | 20:10 |
EmilienM | CI should be back | 20:11 |
*** jeckersb is now known as jeckersb_gone | 20:15 | |
*** snecklifter has quit IRC | 20:17 | |
*** rwsu has joined #tripleo | 20:18 | |
EmilienM | slagle: thx | 20:20 |
EmilienM | I was about doing it | 20:20 |
*** coolsvap has quit IRC | 20:21 | |
weshay | slagle, do you want the undercloud to be a libvirt guest, or directly installed on the node? | 20:34 |
weshay | re: https://review.openstack.org/#/c/348530/ | 20:34 |
openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: Fix Aodh httpd binding https://review.openstack.org/348893 | 20:36 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: move hieradata/ceph into ceph-base service https://review.openstack.org/348489 | 20:36 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Enable keystone to use the SSL middleware https://review.openstack.org/341354 | 20:37 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Enable glance to use the SSL middleware https://review.openstack.org/341367 | 20:37 |
EmilienM | slagle: did you already seen http://logs.openstack.org/95/346995/16/experimental/gate-tripleo-ci-centos-7-undercloud-upgrades-nv/64ef2b7/console.html#_2016-07-29_20_06_03_806767 ? | 20:40 |
*** tbonds has quit IRC | 20:41 | |
*** akshai has quit IRC | 20:42 | |
slagle | weshay: directly installed | 20:42 |
weshay | slagle, k.. we may need to bring in a 3rd party role to do that | 20:43 |
slagle | no we don't, what i did works :) | 20:43 |
weshay | ok | 20:44 |
slagle | EmilienM: no | 20:44 |
slagle | EmilienM: i'm not sure what it's trying to do there | 20:47 |
slagle | EmilienM: hmm, that shouldn't even cause os-refresh-config to fail afaict | 20:50 |
slagle | EmilienM: the real error is earlier up in the undercloud install log | 20:51 |
slagle | 20551 2016-07-29 20:06:00.000 | IOError: [Errno 2] No such file or directory: '/sys/class/net/eth1/address' | 20:51 |
openstackgerrit | Justin Kilpatrick proposed openstack/tripleo-quickstart: Add documentation for add-on roles https://review.openstack.org/346733 | 20:51 |
slagle | EmilienM: oh snap | 20:51 |
slagle | EmilienM: so...the net config override feature we use to install the undercloud in the multinode job is only on master :/ | 20:51 |
slagle | you could possibly hack something in tripleo-ci | 20:52 |
slagle | EmilienM: i had to add this for the multinode job: https://review.openstack.org/#/c/305957/ | 20:53 |
slagle | so that we can specify a custom net-config.json | 20:53 |
slagle | what you could possibly do...is overwrite the packaged file when you install the mitaka undercloud since this feature is not there | 20:53 |
openstackgerrit | Michele Baldessari proposed openstack/puppet-tripleo: Next generation HA architecture work https://review.openstack.org/342650 | 20:54 |
beekneemech | EmilienM: slagle: Oh, I actually had a workaround for the eth1 thing. I just create a dummy interface on the node, since in undercloud-only we don't care if the ctlplane address is actually accessible. | 20:56 |
beekneemech | sudo ip l a dummy0 type dummy | 20:56 |
beekneemech | sudo ip l set dummy0 up | 20:56 |
beekneemech | We could probably rename that to eth1 if we don't want to have to write a custom undercloud.conf entry for local_interface. | 20:56 |
beekneemech | It's what I was using for my local undercloud jobs and it worked fine. | 20:56 |
openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates: Next generation HA architecture work https://review.openstack.org/314208 | 20:57 |
slagle | well i'll be darned | 20:57 |
*** jayg is now known as jayg|g0n3 | 20:59 | |
*** trown is now known as trown|outtypewww | 20:59 | |
weshay | slagle, so undercloud directly on the node, and the overcloud are guests on that node or different machines? | 21:05 |
slagle | weshay: they wouldn't be guests on that node | 21:08 |
slagle | weshay: they could be anywhere, as tripleo.sh supports | 21:08 |
weshay | ok.. thanks | 21:08 |
slagle | ovb, bm, multinode | 21:08 |
weshay | yup | 21:09 |
slagle | i dont think that part matters for just installing the undercloud | 21:09 |
weshay | just getting a better idea of what you are going after | 21:09 |
slagle | weshay: basically to see if the equivalent of --undercloud could be done with oooq | 21:12 |
weshay | k | 21:12 |
EmilienM | slagle: sorry I was afk 10 min | 21:13 |
EmilienM | slagle: why not backporting it? | 21:14 |
EmilienM | so much code conflict in the cherry-pick | 21:17 |
EmilienM | I'm not sure it's a good idea to backport it | 21:17 |
openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Don't try to get stack details if the stack doesn't exist https://review.openstack.org/319337 | 21:17 |
openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: TEST: Delete the overcloud when finished https://review.openstack.org/297328 | 21:17 |
openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Test overcloud deletion in periodic job https://review.openstack.org/296765 | 21:17 |
EmilienM | I'll try beekneemech's hack | 21:17 |
EmilienM | slagle: you have an opinion on backporting it? | 21:19 |
slagle | EmilienM: we don't backport features | 21:20 |
EmilienM | ok, end of story. I'll do the hack | 21:20 |
slagle | i wouldnt support backporting this for what is effectively CI purposes | 21:20 |
EmilienM | okidoc | 21:20 |
openstackgerrit | Ben Nemec proposed openstack/tripleo-docs: Group advanced topics by type https://review.openstack.org/335232 | 21:21 |
EmilienM | slagle: sorry for dumb question but why does it fail? | 21:26 |
slagle | EmilienM: eth1 doesn't exist | 21:29 |
EmilienM | how mitaka jobs work then? | 21:29 |
slagle | infra spawned nodes only have an eth0 | 21:30 |
EmilienM | ah we don't have multi node jobs ok | 21:30 |
EmilienM | got it... | 21:30 |
slagle | mitaka jobs run on ovb | 21:30 |
EmilienM | I need to sleep | 21:30 |
openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: WIP - Implement undercloud upgrade job - Mitaka -> Newton https://review.openstack.org/346995 | 21:33 |
openstackgerrit | Paul Belanger proposed openstack/tripleo-quickstart: Update directory layout per playbook best practices https://review.openstack.org/349076 | 21:35 |
EmilienM | pabelanger: do you mind looking https://review.openstack.org/#/c/348927/ ? | 21:36 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Remove usage of ::nova class in THT https://review.openstack.org/325983 | 21:38 |
openstackgerrit | Paul Belanger proposed openstack/tripleo-quickstart: Update directory layout per playbook best practices https://review.openstack.org/349076 | 21:39 |
*** rhallisey has quit IRC | 21:40 | |
openstackgerrit | Paul Belanger proposed openstack/tripleo-quickstart: Update directory layout per playbook best practices https://review.openstack.org/349076 | 21:40 |
openstackgerrit | Paul Belanger proposed openstack/tripleo-quickstart: Move the library folder into the playbooks directory https://review.openstack.org/349080 | 21:41 |
*** ayoung has quit IRC | 21:44 | |
*** weshay is now known as weshay_afk | 21:51 | |
*** limao has joined #tripleo | 21:52 | |
*** limao_ has joined #tripleo | 21:55 | |
*** limao has quit IRC | 21:57 | |
*** maeca1 has quit IRC | 22:01 | |
*** pradk has quit IRC | 22:04 | |
*** limao has joined #tripleo | 22:05 | |
openstackgerrit | Paul Belanger proposed openstack/tripleo-quickstart: Rename docs folder to doc https://review.openstack.org/349083 | 22:05 |
openstackgerrit | Paul Belanger proposed openstack/tripleo-quickstart: Initial support for sphinxdoc https://review.openstack.org/349084 | 22:05 |
*** limao_ has quit IRC | 22:08 | |
openstackgerrit | Paul Belanger proposed openstack/tripleo-quickstart: Remove unneeded setup-hook https://review.openstack.org/349090 | 22:14 |
*** fultonj has quit IRC | 22:18 | |
*** limao has quit IRC | 22:19 | |
*** limao has joined #tripleo | 22:21 | |
*** limao_ has joined #tripleo | 22:29 | |
*** limao__ has joined #tripleo | 22:30 | |
*** limao has quit IRC | 22:31 | |
*** dtrainor has quit IRC | 22:32 | |
*** jcoufal_ has joined #tripleo | 22:33 | |
*** limao_ has quit IRC | 22:33 | |
*** jcoufal has quit IRC | 22:34 | |
*** limao has joined #tripleo | 22:40 | |
*** limao_ has joined #tripleo | 22:41 | |
*** jcoufal_ has quit IRC | 22:43 | |
*** limao__ has quit IRC | 22:44 | |
*** limao has quit IRC | 22:45 | |
*** dtrainor has joined #tripleo | 22:47 | |
*** limao has joined #tripleo | 22:52 | |
*** rlandy has quit IRC | 22:53 | |
*** limao__ has joined #tripleo | 22:54 | |
*** limao_ has quit IRC | 22:55 | |
*** limao has quit IRC | 22:57 | |
*** maeca1 has joined #tripleo | 22:58 | |
*** saneax_AFK is now known as saneax | 23:05 | |
*** jcoufal has joined #tripleo | 23:07 | |
*** limao__ has quit IRC | 23:07 | |
*** limao has joined #tripleo | 23:09 | |
*** limao has quit IRC | 23:11 | |
*** limao has joined #tripleo | 23:11 | |
openstackgerrit | Paul Belanger proposed openstack/tripleo-quickstart: Add bindep support https://review.openstack.org/349114 | 23:14 |
*** Ryjedo has quit IRC | 23:15 | |
*** morazi has quit IRC | 23:28 | |
*** thrash is now known as thrash|wknd | 23:32 | |
*** rwsu has quit IRC | 23:34 | |
*** limao_ has joined #tripleo | 23:35 | |
*** limao has quit IRC | 23:38 | |
*** limao has joined #tripleo | 23:44 | |
*** limao_ has quit IRC | 23:48 | |
*** jcoufal_ has joined #tripleo | 23:48 | |
*** jcoufal has quit IRC | 23:49 | |
*** limao_ has joined #tripleo | 23:51 | |
*** limao__ has joined #tripleo | 23:54 | |
*** limao has quit IRC | 23:54 | |
*** limao_ has quit IRC | 23:58 | |
*** limao__ has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!