| *** limao has joined #tripleo | 00:38 | |
| *** limao has quit IRC | 00:41 | |
| *** limao has joined #tripleo | 00:43 | |
| *** dtrainor has joined #tripleo | 00:46 | |
| *** greghaynes has quit IRC | 00:48 | |
| *** greghaynes has joined #tripleo | 00:49 | |
| *** dtrainor has quit IRC | 00:51 | |
| *** Guest80615 is now known as saneax_AFK | 00:55 | |
| *** dmacpher has joined #tripleo | 00:56 | |
| *** saneax_AFK is now known as saneax | 00:58 | |
| *** saneax is now known as Guest19379 | 00:58 | |
| *** bswartz has joined #tripleo | 01:02 | |
| *** bswartz has quit IRC | 01:13 | |
| *** xuao has joined #tripleo | 01:19 | |
| *** bana_k has joined #tripleo | 01:21 | |
| *** Guest19379 is now known as saneax_AFK | 01:38 | |
| *** yamahata has joined #tripleo | 01:41 | |
| *** dmacpher is now known as dmacpher-lunch | 02:14 | |
| *** apetrich has joined #tripleo | 02:37 | |
| *** bana_k has quit IRC | 02:40 | |
| *** bana_k has joined #tripleo | 02:53 | |
| *** bana_k has quit IRC | 03:08 | |
| *** limao has quit IRC | 03:15 | |
| *** dmacpher-lunch is now known as dmacpher | 03:26 | |
| *** dsariel has quit IRC | 03:45 | |
| *** ramishra has quit IRC | 03:49 | |
| *** ramishra has joined #tripleo | 03:51 | |
| *** limao has joined #tripleo | 04:15 | |
| *** chlong has joined #tripleo | 04:21 | |
| *** limao has quit IRC | 04:22 | |
| openstackgerrit | Honza Pokorny proposed openstack/tripleo-common: Add node tagging workflow https://review.openstack.org/332132 | 04:26 |
|---|---|---|
| *** bana_k has joined #tripleo | 04:30 | |
| *** bana_k has quit IRC | 04:46 | |
| *** jaosorior has joined #tripleo | 04:47 | |
| *** limao has joined #tripleo | 05:00 | |
| *** bswartz has joined #tripleo | 05:07 | |
| *** pabelanger has quit IRC | 05:14 | |
| *** rook has quit IRC | 05:15 | |
| *** markmc has quit IRC | 05:15 | |
| *** toure has quit IRC | 05:15 | |
| *** markmc has joined #tripleo | 05:15 | |
| *** wfoster has quit IRC | 05:15 | |
| *** pabelanger has joined #tripleo | 05:15 | |
| *** lucas-dinner has quit IRC | 05:15 | |
| *** rook has joined #tripleo | 05:17 | |
| *** rook is now known as Guest14039 | 05:18 | |
| *** wfoster has joined #tripleo | 05:19 | |
| *** lucasagomes has joined #tripleo | 05:19 | |
| *** toure has joined #tripleo | 05:22 | |
| *** dmsimard has quit IRC | 05:25 | |
| *** rcernin has joined #tripleo | 05:25 | |
| *** dmsimard has joined #tripleo | 05:26 | |
| *** limao has quit IRC | 05:27 | |
| *** limao has joined #tripleo | 05:29 | |
| *** ebalduf has quit IRC | 05:44 | |
| *** oshvartz has quit IRC | 05:44 | |
| *** r-mibu has quit IRC | 05:46 | |
| *** r-mibu has joined #tripleo | 05:46 | |
| *** radeks has joined #tripleo | 05:50 | |
| *** leanderthal|afk is now known as leanderthal | 05:56 | |
| *** liverpooler has joined #tripleo | 05:59 | |
| *** florianf has joined #tripleo | 06:02 | |
| *** limao_ has joined #tripleo | 06:13 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci: Inject undercloud's CA into python-ironic-agent image https://review.openstack.org/355312 | 06:15 |
| *** limao has quit IRC | 06:16 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci: DO NOT MERGE - Periodic test https://review.openstack.org/355316 | 06:17 |
| *** dmacpher has quit IRC | 06:18 | |
| *** david-lyle has joined #tripleo | 06:36 | |
| *** pcaruana has joined #tripleo | 06:36 | |
| *** david-lyle_ has quit IRC | 06:39 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci: Inject undercloud's CA into python-ironic-agent image https://review.openstack.org/355312 | 06:43 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci: DO NOT MERGE - Periodic test https://review.openstack.org/355316 | 06:43 |
| *** rwsu has joined #tripleo | 06:44 | |
| *** aufi has joined #tripleo | 06:44 | |
| *** shivrao has quit IRC | 06:52 | |
| *** xuao has quit IRC | 06:55 | |
| *** oshvartz has joined #tripleo | 06:56 | |
| *** hjensas has joined #tripleo | 06:58 | |
| *** hjensas has joined #tripleo | 06:58 | |
| *** limao_ has quit IRC | 07:04 | |
| *** limao has joined #tripleo | 07:07 | |
| *** chlong has quit IRC | 07:08 | |
| *** bootsha has joined #tripleo | 07:11 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: DO NOT MERGE - TESTING BARBICAN https://review.openstack.org/353262 | 07:13 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Add Barbican to the overcloud - WIP https://review.openstack.org/352458 | 07:13 |
| *** zoli_gone-proxy is now known as zoliXXL | 07:14 | |
| *** zoliXXL is now known as zoli|wfh | 07:15 | |
| *** hanchao has joined #tripleo | 07:15 | |
| zoli|wfh | good morning | 07:21 |
| *** jpich has joined #tripleo | 07:21 | |
| *** pkovar has joined #tripleo | 07:21 | |
| *** ebarrera has joined #tripleo | 07:24 | |
| *** chlong has joined #tripleo | 07:25 | |
| *** ifarkas_afk is now known as ifarkas | 07:26 | |
| *** Guest30635 has quit IRC | 07:29 | |
| *** Guest30635 has joined #tripleo | 07:29 | |
| *** akuznetsov has joined #tripleo | 07:29 | |
| *** Guest30635 is now known as assassin | 07:29 | |
| *** yamahata has quit IRC | 07:36 | |
| *** ohamada has joined #tripleo | 07:43 | |
| *** snecklifter has joined #tripleo | 07:47 | |
| *** xuao has joined #tripleo | 07:49 | |
| Jokke_ | hi | 08:02 |
| Jokke_ | anyone has idea why this https://review.openstack.org/354047 fails with http://logs.openstack.org/28/353928/5/check/gate-puppet-manila-puppet-unit-3.6-centos-7/cbf7da9/console.html.gz#_2016-08-12_10_25_27_958013 ? :( | 08:02 |
| *** paramite has joined #tripleo | 08:03 | |
| Jokke_ | wupf ... wrong change obviously https://review.openstack.org/#/c/353928/ | 08:03 |
| *** limao has quit IRC | 08:06 | |
| *** chlong has quit IRC | 08:07 | |
| *** saneax_AFK is now known as saneax | 08:07 | |
| *** saneax is now known as Guest34773 | 08:07 | |
| *** limao has joined #tripleo | 08:08 | |
| *** athomas has joined #tripleo | 08:13 | |
| *** zoli|wfh is now known as zoli|brb | 08:16 | |
| jaosorior | Jokke_: Might be worth asking in #puppet-openstack | 08:17 |
| *** dtantsur|afk is now known as dtantsur | 08:18 | |
| *** Guest34773 is now known as saneax_AFK | 08:19 | |
| *** hewbrocca-afk is now known as hewbrocca | 08:23 | |
| dtantsur | morning folks! could you please review my ironic-in-tripleo patch https://review.openstack.org/341433 ? there is plenty of work ahead with iPXE support etc, so I'd really appreciate merging this | 08:23 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Populate HAProxy endpoint ports from EndpointMap https://review.openstack.org/355366 | 08:25 |
| jaosorior | dtantsur: you had a commit testing this in CI, right? can you pass it on? | 08:26 |
| *** ccamacho has joined #tripleo | 08:26 | |
| dtantsur | jaosorior, no longer, I'm testing it manually | 08:26 |
| *** ccamacho has quit IRC | 08:26 | |
| *** ccamacho has joined #tripleo | 08:26 | |
| dtantsur | it takes too much time and effort to baby-sit the CI, and it does not test anything related, except for installation itself | 08:26 |
| dtantsur | (i.e. it does not even call ironic API to make sure it came up) | 08:27 |
| jaosorior | dtantsur: sure, no worries. If you're testing it locally it's fine | 08:28 |
| dtantsur | I have a WIP document for those who want to try it locally: https://review.openstack.org/354016 | 08:30 |
| dtantsur | (but it assumes the change was merged, some magic may be required for a WIP patch) | 08:30 |
| *** aqkhan__ has joined #tripleo | 08:31 | |
| *** aqkhan_ has quit IRC | 08:32 | |
| *** aqkhan_ has joined #tripleo | 08:32 | |
| dtantsur | All this raise an interesting question. Do you folks have some plans to CI the optional services eventually? | 08:35 |
| dtantsur | Otherwise even if I artificially CI-test it now, it can get broken during the next refactoring, and nobody will notice | 08:35 |
| *** aqkhan__ has quit IRC | 08:36 | |
| *** bana_k has joined #tripleo | 08:37 | |
| *** zoli|brb is now known as zoli|wfh | 08:40 | |
| *** bootsha has quit IRC | 08:40 | |
| d0ugal | Morning! | 08:44 |
| hewbrocca | dtantsur: well, I think the more jobs we can add the happier we will be | 08:44 |
| d0ugal | more stable jobs* | 08:45 |
| hewbrocca | but it requires somebody adding the jobs, getting them committed experimental, making sure they don't break all the time | 08:45 |
| hewbrocca | yes, stable jobs | 08:45 |
| dtantsur | I think this is our of question until we get more resources for the CI, right? | 08:45 |
| hewbrocca | not necessarily | 08:45 |
| hewbrocca | we've got a lot more capacity in the rack now | 08:45 |
| hewbrocca | and | 08:45 |
| hewbrocca | with Slagle's multi-node work, we can do more tripleo testing in the regular infra flow | 08:46 |
| hewbrocca | That should mean we have more capacity for testing Inspector stuff that really requires OVB | 08:47 |
| dtantsur | cool, maybe I should start an ML discussion | 08:47 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add ironic inspector as a terminated HAProxy endpoint https://review.openstack.org/345183 | 08:47 |
| hewbrocca | +1 | 08:47 |
| dtantsur | sent | 08:54 |
| *** Goneri has joined #tripleo | 08:55 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci: DO NOT MERGE - Periodic test https://review.openstack.org/355378 | 08:55 |
| *** electrofelix has joined #tripleo | 08:58 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Populate HAProxy endpoint ports from EndpointMap https://review.openstack.org/355366 | 08:58 |
| jaosorior | hewbrocca: anybody around that I can poke for CI related things? | 08:59 |
| hewbrocca | jaosorior: adarazs or sshnaidm should be able to help | 09:00 |
| hewbrocca | or panda | 09:00 |
| hewbrocca | what's up? | 09:00 |
| jaosorior | need to test a patch with the periodic job and I haven't managed to | 09:00 |
| ccamacho | morning guys! | 09:01 |
| jaosorior | sshnaidm, are you around? | 09:02 |
| *** bootsha has joined #tripleo | 09:02 | |
| ccamacho | jaosorior, not 100% sure but today in USA is National holidays also here in Spain, just FYI. | 09:04 |
| jaosorior | I see | 09:05 |
| hewbrocca | ccamacho: US is not off today | 09:05 |
| hewbrocca | next US holiday is labor day, which is the first Monday in September | 09:06 |
| ccamacho | hewbrocca ack, ok but they were speaking last Friday about the Assumption of Mary | 09:07 |
| hewbrocca | that sounds more like a France holiday | 09:07 |
| hewbrocca | definitely not US though | 09:08 |
| ccamacho | oki :) | 09:09 |
| *** xuao has quit IRC | 09:13 | |
| *** bootsha has quit IRC | 09:14 | |
| *** bootsha has joined #tripleo | 09:14 | |
| *** bootsha_ has joined #tripleo | 09:16 | |
| *** xuao has joined #tripleo | 09:17 | |
| *** bootsha__ has joined #tripleo | 09:18 | |
| *** bootsha has quit IRC | 09:20 | |
| *** bootsha_ has quit IRC | 09:22 | |
| *** limao has quit IRC | 09:22 | |
| *** limao has joined #tripleo | 09:24 | |
| *** limao has quit IRC | 09:29 | |
| *** sshnaidm has quit IRC | 09:29 | |
| adarazs | jaosorior: period upsteam jobs or periodic RDO jobs? | 09:30 |
| jaosorior | adarazs: upstream | 09:30 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci: DO NOT MERGE - Periodic test https://review.openstack.org/355378 | 09:30 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci: Inject undercloud's CA into python-ironic-agent image https://review.openstack.org/355312 | 09:30 |
| ccamacho | jaosorior, when having some time mind to check this submission? https://review.openstack.org/#/c/354054/ re to the puppet-lint warning passed CI | 09:31 |
| adarazs | jaosorior: hrm. I don't know how to inject a change into that. I have solutions for rdo image based stuff for now. if there's something you're trying to fix in upstream head, I have a job that builds on the last unpromoted master image. | 09:31 |
| adarazs | jaosorior: that's built from the latest consistent delorean trunk | 09:31 |
| jaosorior | adarazs: well, I'm actually trying to fix something in CI | 09:31 |
| jaosorior | the commits I uploaded ^^ | 09:32 |
| adarazs | jaosorior: oh. :/ so I dont know how to test that. if I had access to those upstream built images that upstream ci started to use, I could help. | 09:33 |
| adarazs | hewbrocca: btw panda is on PTO for this and the next week. | 09:34 |
| jaosorior | hewbrocca: you know if slagle and dprince will be online today? | 09:34 |
| *** jaosorior is now known as jaosorior_brb | 09:38 | |
| ccamacho | hey dtantsur re to checking of the composable roles, Im creating some functional tests for puppet-mistral (POC) and hopefully we will be able to apply those to all other services and the we might have a job running them all, the thing is that even if you enable the service in the overcloud might not even work properly (just checking that puppet syntax it's OK), now we are trusting in people locally testing the services when push | 09:38 |
| jaosorior_brb | ccamacho: will do when I'm back | 09:38 |
| ccamacho | jaosorior_brb thanks! | 09:38 |
| hewbrocca | jaosorior_brb: I'm sure that they will | 09:38 |
| dtantsur | ccamacho, so, are you planning on a job against puppet-mistral that tests mistral in overcloud? | 09:39 |
| ccamacho | dtantsur, Im trying from the test make some API call for each service, to ensure that DB/network/endpoints are going well | 09:40 |
| ccamacho | dtantsur that would be the second part | 09:40 |
| *** bana_k has quit IRC | 09:41 | |
| dtantsur | so, I think we have functional tests for puppet-ironic, I'm more worried about ironic in overcloud, as its installation is way trickier than for most services | 09:43 |
| openstackgerrit | Erno Kuvaja proposed openstack/tripleo-puppet-elements: Install OpenStack Manila UI https://review.openstack.org/355394 | 09:44 |
| ccamacho | dtantsur And for that second part Ill ask for some help as CI jobs are a little unknown for me, but your right I think in the near future we should/must have jobs checking that the services are actually working | 09:45 |
| *** hanchao has quit IRC | 09:49 | |
| hewbrocca | Ideally, those jobs would actually be part of the upstream gate | 09:51 |
| hewbrocca | I mean, we're not the only ones who want to test if Ironic works when deployed :) | 09:51 |
| openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Adds Deployment Action & Workflow https://review.openstack.org/298732 | 09:54 |
| *** tosky has joined #tripleo | 09:55 | |
| d0ugal | jtomasek: Have you tried the latest deployment/template processing TripleO common patches? | 09:56 |
| *** athomas has quit IRC | 09:57 | |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: Added Contrail Composable Services https://review.openstack.org/348905 | 10:02 |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: Sorted variables alphabetically https://review.openstack.org/355400 | 10:02 |
| *** athomas has joined #tripleo | 10:02 | |
| *** xuao has quit IRC | 10:10 | |
| *** radeks has quit IRC | 10:10 | |
| *** zoli|wfh is now known as zoli|brb | 10:11 | |
| openstackgerrit | Tomas Sedovic proposed openstack/tripleo-validations: Validate rogue DHCP servers https://review.openstack.org/353973 | 10:13 |
| *** xuao has joined #tripleo | 10:14 | |
| jtomasek | d0ugal: not yet | 10:16 |
| jtomasek | d0ugal: I'll update my setup with those and let you know | 10:16 |
| openstackgerrit | Tomas Sedovic proposed openstack/tripleo-validations: Validate rabbitmq limits https://review.openstack.org/353970 | 10:16 |
| d0ugal | jtomasek: http://paste.openstack.org/show/557487/ | 10:18 |
| d0ugal | jtomasek: That's what I get. | 10:18 |
| d0ugal | ... but since I had success before, I am not sure if I have done something wrong or if the workflow has. | 10:19 |
| d0ugal | jtomasek: ... nvm, it is something I am doing, just got past that. | 10:21 |
| *** jaosorior_brb is now known as jaosorior | 10:31 | |
| *** xuao has quit IRC | 10:31 | |
| *** radeks has joined #tripleo | 10:42 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci: DO NOT MERGE - Periodic test https://review.openstack.org/355378 | 10:42 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack-infra/tripleo-ci: Inject undercloud's CA into python-ironic-agent image https://review.openstack.org/355312 | 10:42 |
| *** pblaho has joined #tripleo | 10:44 | |
| *** oshvartz has quit IRC | 10:45 | |
| *** mbound has joined #tripleo | 10:47 | |
| *** etp has joined #tripleo | 10:47 | |
| *** birdman has joined #tripleo | 10:49 | |
| *** birdman has left #tripleo | 10:49 | |
| *** mhenkel has joined #tripleo | 10:49 | |
| *** mbound has quit IRC | 10:51 | |
| *** milan has joined #tripleo | 10:58 | |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: Added Contrail Composable Services https://review.openstack.org/348905 | 11:00 |
| *** xuao has joined #tripleo | 11:15 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Populate HAProxy endpoint ports from EndpointMap https://review.openstack.org/355366 | 11:16 |
| *** lucasagomes is now known as lucas-hungry | 11:16 | |
| *** adarazs is now known as adarazs_lunch | 11:20 | |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: Added Contrail Composable Services https://review.openstack.org/348905 | 11:25 |
| *** maeca1 has joined #tripleo | 11:33 | |
| *** bfournie has quit IRC | 11:34 | |
| *** jaosorior has quit IRC | 11:34 | |
| *** jaosorior has joined #tripleo | 11:35 | |
| *** rodrigods has quit IRC | 11:38 | |
| *** rodrigods has joined #tripleo | 11:38 | |
| *** rhallisey has joined #tripleo | 11:38 | |
| *** rhallisey_ has joined #tripleo | 11:39 | |
| slagle | jaosorior: hey, thanks for looking into the ssl issue | 11:40 |
| slagle | jaosorior: i wonder if something else has changed, b/c the nonha job is passing in the check queue | 11:40 |
| slagle | which does use ssl | 11:40 |
| EmilienM | good morning! | 11:43 |
| slagle | morning | 11:43 |
| d0ugal | jtomasek: what does the UI do with the root environment? | 11:45 |
| *** dtrainor has joined #tripleo | 11:45 | |
| jaosorior | slagle: So, one terrible thing I just remembered | 11:46 |
| jtomasek | d0ugal: nothing, why? | 11:46 |
| jaosorior | slagle: we haven't had real SSL coverage until I started working with certmonger | 11:46 |
| d0ugal | jtomasek: Just wondering why it is in the Mistral env, do you read default params from it? | 11:46 |
| slagle | jaosorior: b/c we were using self signed before that? | 11:46 |
| jaosorior | slagle: | 11:47 |
| jaosorior | no | 11:47 |
| d0ugal | jtomasek: Trying to figure out if I can switch it out for a different file path :) | 11:47 |
| jaosorior | because there was a bug in the autogenerate option that didn't really enable SSL endpoints | 11:47 |
| jaosorior | so it would just generate the service certificate, but generate non-tls endpoints | 11:47 |
| slagle | jaosorior: oh. but is that fixed now? | 11:47 |
| jtomasek | d0ugal: you mean why it is in 'environments' section there? http://paste.openstack.org/show/554117/ | 11:47 |
| jaosorior | slagle: https://review.openstack.org/#/c/333121/ | 11:48 |
| slagle | jaosorior: i just wonder how the nonha job is passing currently | 11:48 |
| d0ugal | jtomasek: Yeah | 11:48 |
| jaosorior | slagle: that is a GREAT question | 11:48 |
| jaosorior | I have no clue why | 11:48 |
| jtomasek | d0ugal: because that one is enabled | 11:48 |
| jtomasek | d0ugal: that is a list of enabled environments | 11:48 |
| d0ugal | jtomasek: right, but I thought this was a special case. it is in the capabilities map? | 11:48 |
| *** weshay_afk is now known as weshay | 11:48 | |
| jtomasek | d0ugal: which means, it is a list of environments which are actualy going to get sent to heat | 11:48 |
| jtomasek | d0ugal: it is special in a way that it needs to be first in that list | 11:49 |
| d0ugal | jtomasek: and it is also needed. | 11:49 |
| jtomasek | d0ugal: yes | 11:49 |
| d0ugal | jtomasek: The issue I have, is that for heatclient to merge the env files for me, I need to merge with that one also | 11:49 |
| jaosorior | slagle: I should actually backport that fix to both liberty and mitaka | 11:49 |
| jtomasek | d0ugal: so when user uploads the plan, he needs to identify which environment file is the root one | 11:49 |
| d0ugal | jtomasek: so I think for the CLI I'll need to have a different root environment that includes that one and the users additions. | 11:49 |
| jtomasek | d0ugal: which is done in capabilities map | 11:50 |
| jaosorior | slagle: by the way, when we don't use cache; is there a reason why after copying the images to the undercloud wee remove the images from the bootstrap node? | 11:50 |
| jaosorior | cause what we do after that is fetch the images from the undercloud and then add them to the cache | 11:50 |
| d0ugal | jtomasek: right, I guess I am breaking stuff then. Getting the CLI to work with what we have built is super hacky :( | 11:50 |
| slagle | jaosorior: what do you mean by bootstrap node? | 11:51 |
| *** rhallisey has quit IRC | 11:51 | |
| jtomasek | d0ugal: can't you just add custom user environments into 'environments' in mistral environment? That's how I'd expect it to work | 11:51 |
| slagle | jaosorior: we install the undercloud directly on the instance running the job | 11:51 |
| *** mbound has joined #tripleo | 11:51 | |
| slagle | if that's what you mean | 11:51 |
| d0ugal | jtomasek: It's not really that simple unfortunately | 11:51 |
| d0ugal | jtomasek: because environments need to add files etc. processing needs to be done with heatclient, that merges all the environments into one | 11:52 |
| *** dprince has joined #tripleo | 11:52 | |
| d0ugal | jtomasek: including the root environment | 11:52 |
| jaosorior | slagle: so for instance, here we remove the ironic-python-agent images https://github.com/openstack-infra/tripleo-ci/blob/master/toci_instack.sh#L138 | 11:52 |
| jaosorior | slagle: and here some lines below we fetch it from the undercloud https://github.com/openstack-infra/tripleo-ci/blob/master/toci_instack.sh#L187 | 11:53 |
| jaosorior | slagle: Same seems to be the case for ovb | 11:53 |
| jaosorior | https://github.com/openstack-infra/tripleo-ci/blob/master/toci_instack_ovb.sh#L106 | 11:53 |
| slagle | toci_instack_ovb.sh is what we're using in CI | 11:54 |
| jaosorior | slagle: sure, and that is the case there too | 11:54 |
| slagle | that tar command is just preparing a tarball to upload to the mirror server | 11:54 |
| jtomasek | d0ugal: so you say you can't use the deployment workflow that is in mistral? I see why this is, because the environments can point outside... hmm this is not nice, because it breaks the CLI/GUI compat | 11:54 |
| slagle | jaosorior: it doesnt delete anything | 11:54 |
| d0ugal | jtomasek: Yeah, I don't know how to make this work. Maybe there is a magic way with heatclient I can't find. | 11:55 |
| jaosorior | slagle: right right, now I got that part (got confused because of toci_instack.sh) | 11:55 |
| d0ugal | jtomasek: but this is the closest I've got to something that fully works for the CLI, so I'll push it up and comment on the limitations. | 11:55 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Migrate the deploy command to use the Mistral workflows https://review.openstack.org/342261 | 11:55 |
| jtomasek | d0ugal: I'd favor putting constraint on having the paths correct rather than breaking the GUI/CLI compatibility | 11:55 |
| d0ugal | jtomasek: Sure, that would be nice, but that isn't backward compatible... | 11:56 |
| *** mbound has quit IRC | 11:56 | |
| jtomasek | d0ugal: just to verify... | 11:56 |
| d0ugal | jtomasek: I suspect we can probably fix this somehow, maybe it requires a change to the merging code in heatclient - I'm not sure. | 11:56 |
| jaosorior | slagle: alright, so I put up a fix https://review.openstack.org/#/c/355312/ which I'm trying to test. I copied the same commit you had for the revert you did, and put it above the fix | 11:57 |
| jtomasek | d0ugal: you specify --templates <templates_path> -e <environment_path> | 11:57 |
| d0ugal | jtomasek: Yup | 11:57 |
| jtomasek | d0ugal: is there a constraint that files referenced in that environment need to be included in the --templates path? | 11:58 |
| jtomasek | d0ugal: or not? | 11:58 |
| d0ugal | jtomasek: no | 11:58 |
| jtomasek | :// | 11:58 |
| jtomasek | too bad | 11:58 |
| slagle | jaosorior: i saw. i do think we need a way though to solve this generically w/o modifying the image | 11:58 |
| d0ugal | jtomasek: Simple example here: https://github.com/d0ugal/tripleo-util/tree/master/testenvs/local_resource_registry | 11:58 |
| d0ugal | jtomasek: That is what I am testing with, see it uses the default template path and local env and resource. | 11:58 |
| slagle | jaosorior: otherwise, anyone who enables ssl will have to modify the image we provide the same way | 11:58 |
| jaosorior | slagle: They had to do it anyway. having a self-signed certificate be trusted by the image, and by the overcloud nodes is a pretty bad security issue | 11:59 |
| jaosorior | slagle: so at least they do need to re-generate the ironic-python-agent image | 12:00 |
| *** saneax_AFK is now known as saneax | 12:01 | |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Migrate the deploy command to use the Mistral workflows https://review.openstack.org/342261 | 12:01 |
| jtomasek | d0ugal: so user can rely on the referenced files in environment being automagically uploaded too | 12:01 |
| jaosorior | and actually... I'm still unsure how it's even passing the gate every once in a while. Seems to me that for some reason when it passes it uses a non-TLS endpoint that is open in the undercloud | 12:01 |
| *** saneax is now known as Guest81752 | 12:01 | |
| jaosorior | since we're still listening on the LOCAL_IP (something I am planning to disable) | 12:02 |
| d0ugal | jtomasek: Yup, heatclient handle that for us. | 12:02 |
| jtomasek | d0ugal: what is the heatclient output when it does that? | 12:02 |
| slagle | jaosorior: yea, i dont even know how the job is passing | 12:02 |
| *** apetrich has quit IRC | 12:02 | |
| slagle | jaosorior: i'm setting up an env know to see if i can figure it out | 12:02 |
| d0ugal | jtomasek: A dict of files and one environment that is the result of all envs being merged | 12:02 |
| jaosorior | slagle: how long does it take to build just that python-ironic-agent.initramfs ? | 12:02 |
| slagle | jaosorior: locally it takes me about 10 minutes | 12:03 |
| *** rhallisey has joined #tripleo | 12:03 | |
| d0ugal | jtomasek: See line 105 here: https://review.openstack.org/#/c/337615/14/tripleo_common/actions/templates.py | 12:03 |
| jaosorior | 10 minutes is a bunch for CI... | 12:03 |
| jtomasek | d0ugal: what if CLI did this, but just with the -e environment? would the output include just the files referenced in that single environment? | 12:03 |
| jaosorior | it's not terrible; but taking that into account I opted to modify the image directly instead | 12:03 |
| d0ugal | jtomasek: I guess I could try that. | 12:03 |
| slagle | jaosorior: we build it every time in CI anyway right now | 12:04 |
| jaosorior | slagle: thought that was only the case for periodic jobs | 12:04 |
| d0ugal | jtomasek: Part of the problem is that the CLI creates environments on the fly and adds them | 12:04 |
| *** ohamada has quit IRC | 12:04 | |
| jaosorior | don't we use the cached image? | 12:04 |
| jtomasek | d0ugal: if it did, you could then take that and add this to the plan and continue from there | 12:04 |
| *** bala has joined #tripleo | 12:04 | |
| d0ugal | jtomasek: so there are actually about 4 or 5 on a normal deploy with no user provided environments | 12:04 |
| *** ohamada has joined #tripleo | 12:04 | |
| slagle | jaosorior: the check-tripleo jobs arent using a cache right now afaik | 12:04 |
| slagle | we didnt get it reenabled yet after moving to ovb, and a few other issues | 12:05 |
| jtomasek | d0ugal: I see | 12:05 |
| jaosorior | I see | 12:05 |
| d0ugal | jtomasek: So I guess I can try removing the root environment and merging the others... | 12:05 |
| d0ugal | jtomasek: so many hacks :( | 12:05 |
| *** apetrich has joined #tripleo | 12:05 | |
| jtomasek | d0ugal: does that happen during the deploy command or some other? | 12:06 |
| jaosorior | slagle: then the fix is actually OK according to what we have in CI. The injection of the cert only happens if we have an image already. Else we use the DIB_IPA_CERT which is used for building the image | 12:06 |
| *** ohamada has quit IRC | 12:06 | |
| jtomasek | (creation of additional environments) | 12:06 |
| d0ugal | jtomasek: During the deploy command. | 12:06 |
| *** ohamada has joined #tripleo | 12:06 | |
| d0ugal | jtomasek: everything is in there lol | 12:06 |
| jtomasek | d0ugal: in those additional environments it sets only parameter_defaults or also resources? | 12:07 |
| *** bvandenh has joined #tripleo | 12:07 | |
| slagle | jaosorior: ok, i think we can go with it for now if it passes | 12:07 |
| d0ugal | jtomasek: Only environments, parameter defaults isn't used anymore | 12:07 |
| d0ugal | jtomasek: (it creates an environment with the parameters) | 12:07 |
| d0ugal | jtomasek: https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/overcloud_deploy.py#L333-L362 | 12:07 |
| jaosorior | slagle: I'm guessing people in the field use pre-built images, do they? | 12:07 |
| slagle | jaosorior: yes. | 12:08 |
| slagle | jaosorior: i havent heard any complaints about having to modify those images | 12:08 |
| slagle | jaosorior: which i figured i would have | 12:08 |
| jaosorior | what the hell | 12:09 |
| slagle | so either no one is using undercloud ssl | 12:09 |
| jtomasek | d0ugal: yeah, thats what I meant: if the created environments only include parameter_defaults, then you could replace creating those environments by storing the parameter values in mistral environment parameter_defaults by using updateParameters action | 12:09 |
| jtomasek | which is how GUI does that | 12:09 |
| slagle | jaosorior: or we were not using ssl where we thought we were :) | 12:09 |
| d0ugal | jtomasek: Oh, I see - I'm not sure. I'll need to check them all. | 12:09 |
| *** bfournie has joined #tripleo | 12:10 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/instack-undercloud: Generate TLS endpoints if autogenerate cert is set https://review.openstack.org/355432 | 12:11 |
| slagle | jaosorior: i wonder if it has something to do with a change in how we're configuring ironic | 12:11 |
| slagle | jaosorior: that somehow has switched it to use ssl during the deploy process | 12:11 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/instack-undercloud: Generate TLS endpoints if autogenerate cert is set https://review.openstack.org/355432 | 12:11 |
| jtomasek | d0ugal: I doubt there are environments created at this stage which would change the resource_registry configuration. It would be really hacky. I really thing that those environments just hold the parameters values (such as generated passwords and other ones that were hardcoded in the CLI logic) | 12:12 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/instack-undercloud: Generate TLS endpoints if autogenerate cert is set https://review.openstack.org/355433 | 12:12 |
| jaosorior | slagle: how can we know what endpoint is the initramfs image using for the undercloud? | 12:12 |
| jtomasek | d0ugal: which is what the mistral environment.parameter_defautls is exactly for | 12:12 |
| slagle | jaosorior: that actually gets passed in on the kernel boot cmdline | 12:12 |
| slagle | jaosorior: based on ironic config, that's what i was heading in that direction | 12:13 |
| *** jayg|g0n3 is now known as jayg | 12:13 | |
| slagle | *why | 12:13 |
| d0ugal | jtomasek: True, I'll make that switch and then just try merging user envs. | 12:14 |
| jaosorior | slagle: is that in the ipxe configuration? | 12:15 |
| jtomasek | d0ugal: cool | 12:15 |
| d0ugal | jtomasek: Thnaks! | 12:15 |
| d0ugal | Thanks* | 12:15 |
| jtomasek | np! | 12:16 |
| d0ugal | Two steps forward, one step back etc. :) | 12:16 |
| slagle | jaosorior: not sure. i'm trying to trace it back | 12:18 |
| slagle | or dtantsur can probably tell us pretty quickly :) | 12:18 |
| dtantsur | ? | 12:19 |
| slagle | dtantsur: where does ipa get the url for ironic api/conductor? | 12:19 |
| slagle | i know it's on the kernel cmdline | 12:19 |
| slagle | but where does that value come from | 12:19 |
| dtantsur | right, it gets populated by ironic conductor | 12:19 |
| dtantsur | when it puts the (i)PXE configuration file for a given MAC to its /(http|tftp)boot | 12:19 |
| slagle | oh that's generated from a template somewhere right? | 12:20 |
| dtantsur | and it comes from an option which I need to lookup | 12:20 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/diskimage-builder: Change DIB_IPA_CERT resulting file name https://review.openstack.org/355440 | 12:20 |
| slagle | /usr/lib/python2.7/site-packages/ironic/drivers/modules/ipxe_config.template | 12:20 |
| *** ohamada has quit IRC | 12:20 | |
| dtantsur | slagle, this is for TFTP https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L2583 it goes into Neutron DHCP options IIRC | 12:20 |
| *** ohamada has joined #tripleo | 12:20 | |
| dtantsur | slagle, this is for HTTP/iPXE: https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L943 | 12:21 |
| dtantsur | it gets pasted in the template you've just found | 12:21 |
| slagle | ok | 12:21 |
| dtantsur | what's the tl;dr of the problem? | 12:22 |
| slagle | dtantsur: in our periodic promote jobs, ipa is failing to deploy due to being unable to verify the ssl cert of ironic api/conductor | 12:24 |
| slagle | dtantsur: however, our ssl jobs in check-tripleo are passing | 12:24 |
| slagle | so wondering if something changed | 12:24 |
| dtantsur | hmm, I guess you need the internal endpoint to be passed... | 12:25 |
| *** Guest81752 is now known as saneax_AFK | 12:25 | |
| slagle | oh that might be something ^^ | 12:25 |
| slagle | jaosorior: ^ | 12:25 |
| dtantsur | well, IPA started using different endpoints recently, maybe this is the problem, though I have no clue how it can relate to SSL | 12:25 |
| dtantsur | so either of these 2 | 12:25 |
| slagle | dtantsur: ah, where is that patch? | 12:26 |
| slagle | sounds like it might be the root cause | 12:26 |
| jaosorior | slagle: disturbingly, I think we never tested SSL for liberty and mitaka... | 12:26 |
| dtantsur | slagle, we've been using new API endpoints since https://github.com/openstack/ironic-python-agent/commit/09265ba4b53a186eca21e11bd6ddab2480292cb8 | 12:26 |
| dtantsur | but it should not affect the host in any way | 12:26 |
| *** ohamada has quit IRC | 12:26 | |
| *** sshnaidm|afk has joined #tripleo | 12:26 | |
| jaosorior | We don't have a generate_service_certificate option there. | 12:26 |
| *** ohamada has joined #tripleo | 12:26 | |
| jaosorior | which is what we base our SSL coverage on | 12:26 |
| *** lucas-hungry is now known as lucasagomes | 12:27 | |
| dtantsur | I'd start with looking which endpoint gets accessed by checking ironic.conf | 12:27 |
| dtantsur | using HTTPS in the ramdisk has not so much sense from security standpoint | 12:27 |
| jaosorior | dtantsur: even if it doesn't some client will probably request it cause there exist really crazy regulations. So supporting it is a win for us anyway | 12:28 |
| *** bala has quit IRC | 12:28 | |
| dtantsur | yeah, as soon as they're fine with the fact it does not add security :) | 12:29 |
| dtantsur | if you use self-signed SSL, you have to inject them into the IPA imagine | 12:29 |
| dtantsur | s/imagine/image/ | 12:29 |
| jaosorior | dtantsur: we do use a CA. it's local, but at least it's not as bad as using a self-signed. | 12:30 |
| openstackgerrit | Florian Fuchs proposed openstack/tripleo-ui: Move plan listing to use Mistral https://review.openstack.org/347919 | 12:30 |
| dtantsur | does IPA know about the local CA? | 12:30 |
| jaosorior | dtantsur: I pushed a commit for it to trust that cert | 12:30 |
| *** xuao has quit IRC | 12:30 | |
| jaosorior | dtantsur: do you know which option sets the endpoint? Haven't found it yet. | 12:31 |
| *** bala has joined #tripleo | 12:31 | |
| dtantsur | jaosorior, see above, http_url in [deploy] section | 12:31 |
| jaosorior | dtantsur: sorry, missed that | 12:32 |
| jaosorior | slagle: got a link to a failed deployment? | 12:32 |
| *** pradk has joined #tripleo | 12:33 | |
| slagle | jaosorior: we aren't getting logs for some reason | 12:33 |
| slagle | thats why i set it up locally | 12:33 |
| slagle | jaosorior: the job is getting killed since it hits the timeout | 12:34 |
| jaosorior | wow | 12:34 |
| jaosorior | alright | 12:34 |
| *** rhallisey_ has quit IRC | 12:34 | |
| *** thrash|wknd is now known as thrash | 12:34 | |
| slagle | dtantsur: deploy.http_url is configured via http. but ipa is still trying to use https | 12:36 |
| slagle | dtantsur: could that be due to https://review.openstack.org/#/c/334523/ ? | 12:36 |
| dtantsur | suspicious... can there be a redirect to https? | 12:36 |
| slagle | i dont see one | 12:36 |
| *** rlandy has joined #tripleo | 12:36 | |
| dtantsur | well, this change only affects the path, but not the host/protocol, unless I'm missing something | 12:36 |
| slagle | https://review.openstack.org/#/c/334523/ landed after our last promote fwiw | 12:37 |
| openstackgerrit | Michael Henkel proposed openstack/tripleo-heat-templates: add composable services for Contrail Database, Config, Analytics, Control and Webui https://review.openstack.org/346799 | 12:37 |
| *** adarazs_lunch is now known as adarazs | 12:40 | |
| dtantsur | slagle, could you please find the associated ipxe configuration in /httpboot? just to check the kernel cmdline. also does it fail on lookup or heartbeat? | 12:40 |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: [mitaka-only] Fix up OPM symlinks before AODH migration https://review.openstack.org/355446 | 12:40 |
| slagle | dtantsur: yea i'll check in a few mins. i was brining up an old env to compare the 2 | 12:41 |
| slagle | once that's done, i'll restore the broken one | 12:42 |
| jaosorior | slagle, dtantsur: It shouldn't be cause be a redirect, we don't have anything for that in the haproxy configuration | 12:42 |
| dtantsur | weird. ok, let's see what ends up in the ipxe configuration, so that we're at least sure it happens on the IPA side | 12:43 |
| slagle | yep, will check | 12:44 |
| *** julim has joined #tripleo | 12:44 | |
| *** trozet has joined #tripleo | 12:47 | |
| *** rbrady has joined #tripleo | 12:49 | |
| *** amoralej|off has quit IRC | 12:50 | |
| *** zoli|brb is now known as zoli|wfh | 12:52 | |
| *** amoralej has joined #tripleo | 12:53 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: WIP - Implement undercloud upgrade job - Mitaka -> Newton https://review.openstack.org/346995 | 12:53 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Implement non-ovb overcloud update job - Newton -> Newton https://review.openstack.org/351330 | 12:56 |
| *** tzumainn has joined #tripleo | 12:57 | |
| jtomasek | rbrady: ping | 12:58 |
| rbrady | jtomasek: ack | 12:58 |
| jtomasek | rbrady: does updateParameters overwrite the parameter_defaults or merges/adds to it? | 13:00 |
| jtomasek | rbrady: wf_env.variables['parameter_defaults'].update(self.parameters) | 13:00 |
| d0ugal | jtomasek: That merges/adds | 13:01 |
| *** julim has quit IRC | 13:01 | |
| rbrady | jtomasek: right now it merges | 13:01 |
| *** milan has quit IRC | 13:01 | |
| *** zoli|wfh is now known as zoli|lunch | 13:03 | |
| jtomasek | d0ugal, rbrady: I wonder what is better... Probably merging/adding is best approach. Problem is with cleaning the values in case when value in parameter_default becomes the same as default parameter value and therefore not needed to be included in mistral env parameter_defaults any more | 13:03 |
| jtomasek | rbrady: but anyway, this does not block the patch as it is not that big deal if the value remains there | 13:03 |
| d0ugal | jtomasek: I guess it is only an issue if the template's default is then changed for some reason | 13:04 |
| *** milan has joined #tripleo | 13:04 | |
| *** julim has joined #tripleo | 13:05 | |
| *** ccamacho has quit IRC | 13:05 | |
| jtomasek | d0ugal: yes, although at that point all parameter values in mistral environment should be probably reset as the underlining templates structure changed | 13:05 |
| d0ugal | jtomasek: True, there be dragons :) | 13:06 |
| jtomasek | d0ugal: we don't solve it automatically yet, but rbrady added an action to reset param_defaults in mistral env, so client can use that | 13:06 |
| rbrady | jtomasek: for now, I'm just trying to debug the call to heatclient.stacks.validate and get that working. The larger picture of managing params between templates, previously set values and new values is a later priority | 13:06 |
| *** alee has quit IRC | 13:08 | |
| jtomasek | rbrady: ack. I have the GUI patch almost ready. I am able to filter out parameters I need and also send only the values that I actually want to get stored in mistral env. | 13:08 |
| jtomasek | rbrady: what error do you get? the DeployIdentifier one? | 13:08 |
| rbrady | jtomasek: Bad HTTP Request / malformed | 13:09 |
| jtomasek | rbrady: ok | 13:09 |
| rbrady | jtomasek: trying to search logs to see if there's an actual detail error message somewhere | 13:09 |
| d0ugal | Is shardy off today? | 13:10 |
| *** lblanchard has joined #tripleo | 13:10 | |
| jtomasek | rbrady: since it happens only with the param_defaults set in mistral env, problem probably is in the format of that 'environment' being passed to heatclient | 13:10 |
| openstackgerrit | Michael Henkel proposed openstack/tripleo-heat-templates: add composable services for Contrail Database, Config, Analytics, Control and Webui https://review.openstack.org/346799 | 13:11 |
| rbrady | jtomasek: ack, will compare formats | 13:11 |
| EmilienM | d0ugal: yes, he's on paternity leave | 13:11 |
| EmilienM | d0ugal: all the week at minimum. | 13:11 |
| d0ugal | EmilienM: k, thanks! | 13:11 |
| EmilienM | d0ugal: what is your question? we can also help | 13:12 |
| d0ugal | EmilienM: oh, it wasn't anything specific. I just wanted to chat to him about the Mistral work as he has been keeping up with it. | 13:13 |
| openstackgerrit | Michael Henkel proposed openstack/tripleo-heat-templates: add composable services for Contrail Database, Config, Analytics, Control and Webui https://review.openstack.org/346799 | 13:13 |
| *** jcoufal has joined #tripleo | 13:13 | |
| openstackgerrit | Brad P. Crochet proposed openstack/python-tripleoclient: Throw an error/exception when interactive overcloud update fails https://review.openstack.org/339048 | 13:14 |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: Added Contrail Composable Services https://review.openstack.org/348905 | 13:16 |
| openstackgerrit | Brad P. Crochet proposed openstack/tripleo-heat-templates: Composable Zaqar services https://review.openstack.org/331682 | 13:18 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Create per-role node_names in hieradata for all nodes https://review.openstack.org/348973 | 13:19 |
| openstackgerrit | Brad P. Crochet proposed openstack/tripleo-heat-templates: Composable Mistral services https://review.openstack.org/323436 | 13:21 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Create per-role node_names in hieradata for all nodes https://review.openstack.org/348973 | 13:21 |
| beagles | slagle, dprince: if you get a chance https://review.openstack.org/#/c/335235/ seems like it's been stuck in a rebase spin-cycle. While it is a "bug fix" it is more or less a cleanup in terms of complexity, but might be nice to clear off the plate. | 13:22 |
| *** fultonj has joined #tripleo | 13:22 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Create per-role node_names in hieradata for all nodes https://review.openstack.org/348973 | 13:22 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Create per-role node_names in hieradata for all nodes https://review.openstack.org/348973 | 13:23 |
| openstackgerrit | Brad P. Crochet proposed openstack/tripleo-docs: Switch docs to use tripleo-build-images https://review.openstack.org/337669 | 13:27 |
| openstackgerrit | Emilien Macchi proposed openstack/python-tripleoclient: WIP wire in jinja templating for custom roles https://review.openstack.org/336625 | 13:28 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Convert overcloud.yaml to support jinja2 templating https://review.openstack.org/315679 | 13:28 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Move AllNodesDeployments into jinja template loop https://review.openstack.org/337267 | 13:29 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Move AllNodesValidationDeployments into jinja template loop https://review.openstack.org/337587 | 13:29 |
| bfournie | beagles: thanks! | 13:30 |
| ansiwen | EmilienM: do we need more beaker tests for ec2api? or is it enough? | 13:31 |
| beagles | bfournie, no problem! | 13:31 |
| EmilienM | ansiwen: looks good | 13:32 |
| ansiwen | EmilienM: what about the tempest runs. Do I have to prepare something for puppet-openstack-integration? | 13:33 |
| bfournie | dprince, EmilienM: thanks for reviewing 335235 | 13:34 |
| EmilienM | ansiwen: it would b great ye | 13:34 |
| EmilienM | bfournie: np | 13:34 |
| *** limao has joined #tripleo | 13:35 | |
| *** limao_ has joined #tripleo | 13:37 | |
| jaosorior | slagle: the error manifests as a node registration error, right? | 13:40 |
| slagle | jaosorior: no | 13:40 |
| *** limao has quit IRC | 13:40 | |
| slagle | during instance deployment | 13:40 |
| jaosorior | slagle: "no valid host was found" > | 13:41 |
| jaosorior | ? | 13:41 |
| openstackgerrit | Florian Fuchs proposed openstack/tripleo-common: Exclude more unneeded files from default plan https://review.openstack.org/355472 | 13:42 |
| slagle | jaosorior: yes | 13:42 |
| slagle | dtantsur: in my ipxe config for the instnace, it has https: http://paste.openstack.org/show/557571/ | 13:42 |
| slagle | dtantsur: for ipa-api-url | 13:42 |
| slagle | where is it getting that value from? | 13:42 |
| slagle | https://192.0.2.2:13385 does not exist anywhere in ironic.conf | 13:43 |
| *** [1]cdearborn has joined #tripleo | 13:43 | |
| slagle | perhaps it is querying keystone for the public endpoint? | 13:43 |
| *** tbonds has joined #tripleo | 13:43 | |
| jaosorior | slagle: that seems like the only reasonable explanation | 13:44 |
| slagle | let me go look at ironic patches since the 3rd or so | 13:45 |
| jaosorior | slagle, that's the case | 13:45 |
| jaosorior | slagle: it tries to get the URL from the configuration and if it doesn't get it from there ir queries keystone | 13:45 |
| slagle | jaosorior: so we need to write something to ironic.conf? | 13:46 |
| jaosorior | slagle https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/deploy_utils.py#L99 | 13:46 |
| jaosorior | slagle: on our side, yes, on the ironic side they need to provide an option to get the internal URL instead of the public one | 13:47 |
| *** zoli|lunch is now known as zoli|wfh | 13:47 | |
| *** limao_ has quit IRC | 13:47 | |
| *** limao has joined #tripleo | 13:47 | |
| jaosorior | slagle: checking how to set that url on the undercloud | 13:47 |
| jaosorior | slagle: there is no way to set that option in puppet-ironic. We would need to set it on hieradata_override. | 13:48 |
| slagle | jaosorior: testing setting api_url to the http endpoing now | 13:51 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Migrate the deploy command to use the Mistral workflows https://review.openstack.org/342261 | 13:51 |
| *** chlong has joined #tripleo | 13:51 | |
| d0ugal | jtomasek: The solution was far more obvious after taking a wee break. The workflow includes the root environment, so I can just remove that from the CLI logic. | 13:52 |
| jtomasek | d0ugal: nice | 13:52 |
| ansiwen | EmilienM: so I ask ade lee for advice how to do that? | 13:52 |
| jaosorior | slagle: the fix is green https://review.openstack.org/#/c/355312/4 however, the commit over it that runs the periodic job https://review.openstack.org/#/c/355378/3 got a POST_FAILURE. but it got it in the ha too, which makes no sense as the fix only applies for SSL cases | 13:53 |
| slagle | jaosorior: ok, well i think the right fix is to set api_url to use http | 13:54 |
| EmilienM | ansiwen: yeah you can look what we did https://github.com/openstack/puppet-openstack-integration/commit/abd6b91e05ac6ea2c94b2e9ee00bb8ae074c3e3e | 13:54 |
| slagle | jaosorior: instead of trying to get deployment working over ssl | 13:54 |
| slagle | jaosorior: which has never worked apparently | 13:54 |
| ansiwen | EmilienM: oh, thanks, I was just looking for that commit! | 13:54 |
| slagle | jaosorior: setting api_url fixed it for me | 13:54 |
| *** ayoung has joined #tripleo | 13:55 | |
| jaosorior | slagle: would it be possible for you to test the fix on your deployment? | 13:55 |
| *** dprince has quit IRC | 13:55 | |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart: Adds option to ci-script to mange image in OVB https://review.openstack.org/355476 | 13:55 |
| jaosorior | slagle: we haven't covered it in CI, that's true. However, it's a case that we will support at some point. So I think it's worth it to get that working. | 13:56 |
| slagle | jaosorior: i just tested api_url, it works | 13:56 |
| *** limao_ has joined #tripleo | 13:56 | |
| slagle | i dont have time to test the more involved fix | 13:57 |
| *** egafford has joined #tripleo | 13:57 | |
| *** limao_ has quit IRC | 13:58 | |
| *** limao_ has joined #tripleo | 13:58 | |
| *** limao has quit IRC | 13:59 | |
| *** limao_ has quit IRC | 13:59 | |
| slagle | jaosorior: if we want to support deployment over ssl in the long term, i think we need a generic way to inject the CA so that we can use prebuilt images w/o modification | 13:59 |
| *** limao has joined #tripleo | 13:59 | |
| slagle | via kernel cmdline params perhaps | 14:00 |
| jaosorior | slagle: I honestly don't think that's possible. we can't use a CA that we inject in ALL images. That introduces a vulnerability in our deployments | 14:00 |
| jaosorior | uhm.. | 14:00 |
| openstackgerrit | Merged openstack/puppet-tripleo: Add ironic inspector as a terminated HAProxy endpoint https://review.openstack.org/345183 | 14:00 |
| *** bootsha__ has quit IRC | 14:01 | |
| *** devvesa has joined #tripleo | 14:01 | |
| jaosorior | dtantsur: is it possible to pass a file or something of the sort via kernel cmdline params? | 14:02 |
| jaosorior | dtantsur: we need an alternative way to get python-ironic-agent.initramfs to trust a CA | 14:02 |
| slagle | jaosorior: comparing my 2 environments, one using current-tripleo (working) and one using latest consistent (broken), current-tripleo uses http for ipa-api-url by default | 14:04 |
| slagle | dtantsur: ^ | 14:04 |
| slagle | so something changed on the ironic side in the last 2 weeks or so | 14:04 |
| slagle | or could be puppet-ironic even maybe | 14:04 |
| slagle | my configured endpoints in keystone are the same in both envs. public uses ssl, everything else is non-ssl | 14:04 |
| slagle | so i think just configuring api_url to use http will at least get us back to what we had working before | 14:05 |
| *** dciabrin has quit IRC | 14:06 | |
| *** dtrainor has quit IRC | 14:07 | |
| *** dtrainor has joined #tripleo | 14:08 | |
| *** ebalduf has joined #tripleo | 14:09 | |
| *** myoung|biab is now known as myoung | 14:09 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci: Remove EPEL usage https://review.openstack.org/347499 | 14:10 |
| slagle | EmilienM: can i configure arbitrary ironic config via hieradata? | 14:10 |
| EmilienM | yes | 14:10 |
| slagle | how | 14:11 |
| EmilienM | thanks to https://github.com/openstack/puppet-ironic/blob/master/manifests/config.pp | 14:11 |
| EmilienM | # ironic_config: | 14:11 |
| EmilienM | # DEFAULT/foo: | 14:11 |
| EmilienM | # value: fooValue | 14:11 |
| slagle | ok thanks | 14:11 |
| EmilienM | slagle: you'll also need to patch the .pp to include ::ironic::config | 14:12 |
| d0ugal | jtomasek: So, this is where I am at now: http://paste.openstack.org/show/557578/ | 14:12 |
| d0ugal | jtomasek: user-environment.yaml just contains the bits the CLI adds and the extra environments the user adds | 14:12 |
| d0ugal | jtomasek: If I add support to the getparameters action for this file, does that mean it will work in the UI? | 14:13 |
| *** bala has quit IRC | 14:13 | |
| jtomasek | d0ugal: yes. If you manage to move the parameter_defaults defined in user-environment to mistral environment using updateParameters then you're set with the UI | 14:14 |
| jtomasek | d0ugal: oh, is that different from what you meant?:) | 14:14 |
| d0ugal | jtomasek: no, that is fine | 14:15 |
| jtomasek | d0ugal: ok | 14:15 |
| openstackgerrit | James Slagle proposed openstack/instack-undercloud: Use http for Ironic deployments https://review.openstack.org/355484 | 14:15 |
| d0ugal | jtomasek: Sorry, I am being confusing. Do I need to do anything special for the resource registry in that file? | 14:15 |
| d0ugal | jtomasek: or does the UI never care about the resource_registry? | 14:15 |
| jtomasek | d0ugal: can you paste the contents of that file? | 14:15 |
| openstackgerrit | James Slagle proposed openstack/instack-undercloud: Use http for Ironic deployments https://review.openstack.org/355484 | 14:16 |
| d0ugal | jtomasek: Yeah, just give me a few mins. | 14:16 |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: DO NOT MERGE - Periodic test https://review.openstack.org/346949 | 14:17 |
| ansiwen | EmilienM: I will try scenario001 first, correct? | 14:19 |
| EmilienM | ansiwen: you try to share the load | 14:19 |
| EmilienM | so pick a scenario where the least services run | 14:19 |
| EmilienM | ansiwen: look https://github.com/openstack/puppet-openstack-integration#description | 14:20 |
| ayoung | trown|outtypewww, larsks or any other quickstart Gurus. I need to do a Liberty/OSP8 install with Quickstart to confirm that enableing Multiple LDAPs does not break the install. I thought it was running this way: ~/.quickstart/tripleo-quickstart/quickstart.sh -T -v --config ~/.quickstart//tripleo-quickstart/config/release/liberty.yml ayoung-dell-t1700.test | 14:20 |
| ayoung | but when I look at the Keystone file on the undercloud it is | 14:20 |
| ayoung | openstack-keystone-9.1.1-0.20160711221844.ee74d15.el7.centos.noarch | 14:20 |
| ayoung | which looks like Mitaka, not Liberty | 14:20 |
| *** fragatina has joined #tripleo | 14:20 | |
| ayoung | What am I doing wrong? | 14:20 |
| *** fragatina has quit IRC | 14:21 | |
| slagle | EmilienM: jaosorior : dtantsur : trying this fix: https://review.openstack.org/#/c/355484/ | 14:21 |
| *** fragatina has joined #tripleo | 14:21 | |
| jaosorior | slagle: alright... so the only way I can think of is to give the user the option to use ironic-python-agent with or without TLS. Cause also passing parameters to the kernel cmdline params has limitations. Depending on the architecture we have up to ~4000 characters. a cert file has more than 1000 usually and passing a base64 encoded string would be more than 2000 characters already :/ | 14:21 |
| EmilienM | slagle: ok | 14:22 |
| EmilienM | slagle: lgtm | 14:22 |
| slagle | jaosorior: darn, ok | 14:22 |
| EmilienM | slagle: err wait | 14:22 |
| EmilienM | I did a mistake when I told you how to use it, can I push over? | 14:22 |
| slagle | you want to push over the yaml king??? | 14:22 |
| EmilienM | lol | 14:23 |
| slagle | yea, go for it :) | 14:23 |
| *** jtomasek is now known as jtomasek|biab | 14:23 | |
| EmilienM | k thx | 14:23 |
| jaosorior | slagle: So yeah, if the customer wants to use TLS for ironic-python-agent, they need to rebuild the images. | 14:23 |
| ayoung | jaosorior, we make SSL work with FreeIPA. Then, if someone demands something else, we get them to nail down the requirements | 14:23 |
| ayoung | I'm done trying to build a house without nails | 14:23 |
| slagle | jaosorior: Ok. or we could document how to modify an existing image | 14:23 |
| jaosorior | slagle: sure, that makes sense | 14:23 |
| d0ugal | jtomasek|biab: https://paste.fedoraproject.org/408805/70974147/raw/ and https://paste.fedoraproject.org/408806/14712710/ | 14:24 |
| jaosorior | ayoung: python-ironic-agent won't work with FreeIPA either unless that image is rebuilt | 14:24 |
| jaosorior | ayoung: we're just struggling with that ATM | 14:24 |
| d0ugal | jtomasek|biab: That is the result of my test here: https://github.com/d0ugal/tripleo-util/tree/master/testenvs/local_resource_registry | 14:24 |
| ayoung | jaosorior, why not? You can't install RPMs after launch? | 14:24 |
| openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: Use http for Ironic deployments https://review.openstack.org/355484 | 14:25 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Migrate the deploy command to use the Mistral workflows https://review.openstack.org/342261 | 14:25 |
| mhenkel | hi all. is it normal that CI didn’t kick in after 70 minutes or am I doing something wrong? https://review.openstack.org/#/c/346799/ and https://review.openstack.org/#/c/348905 | 14:26 |
| EmilienM | ayoung: trown is on PTO fyi | 14:26 |
| jaosorior | ayoung: I'm honestly not sure how the python-ironic-agent works. But it's supposed to be a light weight image that is used to only provision nodes https://github.com/openstack/ironic-python-agent | 14:26 |
| ayoung | jaosorior, right, so the order would be bring up nodes, node talks to metadata or config drive, node gets directive that says install package, in this case freeipa-client | 14:27 |
| *** dprince has joined #tripleo | 14:27 | |
| jaosorior | ayoung: Problem is, currently clients use a pre-built image for that... IMO the only way to provide TLS support for ironic-python-agent is to modify the pre-built image and add the CA cert. | 14:27 |
| openstackgerrit | James Slagle proposed openstack/instack-undercloud: Use http for Ironic deployments https://review.openstack.org/355484 | 14:27 |
| ayoung | jaosorior, the rest should be drive from the server side, with the metadata tool that rcrit is working on with the Nova team | 14:27 |
| ayoung | NO CA CERT! | 14:27 |
| jaosorior | ayoung: not even FreeIPA's certificate? | 14:27 |
| ayoung | Heh | 14:27 |
| ayoung | jaosorior, nah, you should not need it | 14:28 |
| ayoung | I mean, it is not a bad thing | 14:28 |
| *** akuznetsov has quit IRC | 14:28 | |
| ayoung | jaosorior, are you trying to get Metadata fetch secured with TLS? | 14:28 |
| jaosorior | ayoung: no | 14:28 |
| ayoung | Yeah, then you should not need it, IFF you enroll with IPA | 14:29 |
| jaosorior | slagle, dtantsur what happens with the ironic-python-agent after ironic starts provisioning a node? | 14:29 |
| slagle | fairies come and sprinke pxe dust | 14:30 |
| *** xuao has joined #tripleo | 14:30 | |
| *** pcaruana has quit IRC | 14:30 | |
| slagle | i'm not sure what you're asking :) | 14:30 |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: align the tripleo-roles.yml playbook w/ quickstart.yml https://review.openstack.org/355495 | 14:30 |
| openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Adds parameters actions https://review.openstack.org/298682 | 14:31 |
| openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Adds Deployment Action & Workflow https://review.openstack.org/298732 | 14:31 |
| openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Adds action for template processing https://review.openstack.org/337615 | 14:31 |
| * slagle thinks ayoung and jaosorior are talking about different IPA's | 14:31 | |
| ayoung | slagle, Heh | 14:31 |
| ayoung | slagle, I should say FreeIPA | 14:31 |
| ayoung | but we had the acronym first... | 14:31 |
| jaosorior | slagle: that's why I always write python-ironic-agent | 14:31 |
| jaosorior | *ironic-python-agent | 14:31 |
| jaosorior | makes it easier to distinguish from FreeIPA | 14:31 |
| ayoung | pia | 14:31 |
| *** awaugama has joined #tripleo | 14:32 | |
| jaosorior | ayoung: people here call ironic-python-agent as IPA. That's why it gets quite confused if we call FreeIPA just IPA as we do in the security channels | 14:32 |
| slagle | EmilienM: did i fix https://review.openstack.org/#/c/355484/ ? | 14:33 |
| jaosorior | IPA is quite overused | 14:33 |
| ayoung | jaosorior, yep. | 14:33 |
| ayoung | jaosorior, IdM is even worse... | 14:33 |
| EmilienM | slagle: you broke it | 14:33 |
| jaosorior | slagle: it shouldn't need the brackets | 14:33 |
| ayoung | calling it ipa-server would make theyre heads asplode | 14:33 |
| EmilienM | https://review.openstack.org/#/c/355484/3..4/elements/puppet-stack-config/puppet-stack-config.yaml.template | 14:33 |
| slagle | so i dont need []'s? | 14:33 |
| EmilienM | slagle: please revert to the previous PS | 14:33 |
| EmilienM | nope | 14:33 |
| slagle | ok ok | 14:33 |
| openstackgerrit | James Slagle proposed openstack/instack-undercloud: Use http for Ironic deployments https://review.openstack.org/355484 | 14:34 |
| EmilienM | slagle: thanks to https://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack_config/ruby.rb#L68-L74 | 14:34 |
| slagle | let me just put your PS 3 back | 14:34 |
| openstackgerrit | James Slagle proposed openstack/instack-undercloud: Use http for Ironic deployments https://review.openstack.org/355484 | 14:35 |
| openstackgerrit | Michael Henkel proposed openstack/tripleo-heat-templates: add composable services for Contrail Database, Config, Analytics, Control and Webui https://review.openstack.org/346799 | 14:35 |
| EmilienM | slagle: is it a permanent fix or a temporary workaround? | 14:35 |
| EmilienM | slagle: because the right thing would be to patch puppet-ironic with the new parameter | 14:35 |
| slagle | EmilienM: i think it's permanent. HOwever, i'm not sure what else ironic might use conductor/api_url for | 14:36 |
| EmilienM | you might want to patch puppet-ironic/manifests/conductor.pp then | 14:36 |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: DO NOT MERGE - Periodic test. https://review.openstack.org/346949 | 14:36 |
| *** zaneb has joined #tripleo | 14:36 | |
| *** radeks has quit IRC | 14:36 | |
| jaosorior | ayoung: anyway, I don't think we can do the FreeIPA enrollment in python-ironic-agent | 14:37 |
| jaosorior | fuck | 14:37 |
| jaosorior | ironic-python-agent | 14:37 |
| ayoung | jaosorior, just say ironic | 14:37 |
| ayoung | Heh | 14:37 |
| jaosorior | ayoung: they're not the same | 14:37 |
| *** Ryjedo has joined #tripleo | 14:38 | |
| ayoung | jaosorior, why not? | 14:38 |
| jaosorior | ayoung: ironic-python-agent is an initramfs that is used by the node on boot to communicate with ironic | 14:38 |
| ayoung | jaosorior, does it not do cloud-init? | 14:38 |
| jaosorior | ayoung: it's before cloud-init as far as I understand | 14:38 |
| ayoung | and you want that channel to be TLS secured, right? | 14:39 |
| jaosorior | ayoung: it's used for many things such as introspection and provisioning | 14:39 |
| jaosorior | ayoung: yes | 14:39 |
| ayoung | it does all that prior to communicating with the ironic head node? | 14:39 |
| jaosorior | ayoung: if we want to do that with FreeIPA we would need to have ALSO provisional instances that register themselves in FreeIPA, and are then discarded since the real instance will be booted after that | 14:40 |
| jaosorior | unless I'm misunderstanding it | 14:40 |
| awaugama | Hey All. I'm trying to deploy 3 controllers and 2 compute nodes with triple o quickstart. I ran the quickstart.sh with a custom ha file that added a second node (- name: compute_1 flavor: compute) but still ended up with one compute node. suggestions? | 14:41 |
| jaosorior | ayoung: That's why I proposed rebuilding that image and just injecting the cert. I don't think it's worth it to have the overhead of enrolling to FreeIPA every time we do introspection or boot a node | 14:41 |
| ayoung | jaosorior, I don't know. | 14:42 |
| ayoung | jaosorior, I suspect that we do not want to customize the images | 14:42 |
| ayoung | it is a lot of workflow to confirm those. Once we customize, we lose the ability to track that the right images are deployed base on their hash | 14:42 |
| jaosorior | well, that's a good point | 14:43 |
| *** akuznetsov has joined #tripleo | 14:43 | |
| ayoung | jaosorior, so, we have to assume that metadata or config drive is secured at the network level, at which point it can fetch a cert | 14:43 |
| ayoung | ensuring that network is locked down becomes a neutron or physical network issue, not crypto | 14:44 |
| ayoung | so for now, provision the CA cert from the ironic head node, and note that there is an assumption of network security. | 14:45 |
| openstackgerrit | Tomas Sedovic proposed openstack/tripleo-validations: Validate haproxy configuration values https://review.openstack.org/353953 | 14:45 |
| openstackgerrit | Julie Pichon proposed openstack/tripleo-common: Add Mistral actions for boot & root device configuration https://review.openstack.org/354174 | 14:51 |
| *** [1]cdearborn has quit IRC | 14:51 | |
| *** [1]cdearborn has joined #tripleo | 14:54 | |
| openstackgerrit | Tomas Sedovic proposed openstack/tripleo-validations: Validate pacemaker status https://review.openstack.org/353969 | 14:56 |
| slagle | dtantsur: jaosorior : ftr, this is the change that broke us: https://review.openstack.org/#/c/236982/ | 14:57 |
| *** dtantsur is now known as dtantsur|mtg | 14:57 | |
| *** devvesa has quit IRC | 14:58 | |
| dtantsur|mtg | ugh... | 14:58 |
| dtantsur|mtg | slagle, do you have any details on what exactly it did? | 14:58 |
| slagle | get_service_url defaults to teh internal endpoint type | 14:58 |
| slagle | in ironic.common.keystone | 14:58 |
| dtantsur|mtg | hmm | 14:58 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Remove NeutronEnableTunnelling from templates https://review.openstack.org/335235 | 14:59 |
| dtantsur|mtg | slagle, on a retro right now, will look into it afterwards (ping me if I don't) | 14:59 |
| *** apetrich has quit IRC | 14:59 | |
| slagle | dtantsur|mtg: actually that looks like it should do the right thing (default to internal) | 15:00 |
| *** xuao has quit IRC | 15:01 | |
| *** xuao has joined #tripleo | 15:01 | |
| jaosorior | slagle: well, now I don't know why the public url was getting there | 15:03 |
| *** akuznetsov has quit IRC | 15:04 | |
| *** egafford has quit IRC | 15:04 | |
| *** mbound has joined #tripleo | 15:05 | |
| dprince | EmilienM: hi, regarding your comment here? https://review.openstack.org/#/c/353748/. I see that nova::network::neutron is used here http://git.openstack.org/cgit/openstack/puppet-tripleo/tree/manifests/profile/base/nova/api.pp#n50 | 15:06 |
| dprince | EmilienM: but do we need it? | 15:06 |
| EmilienM | slagle: commented on https://review.openstack.org/#/c/355484/6/elements/puppet-stack-config/puppet-stack-config.yaml.template - yourcall | 15:06 |
| *** rcernin has quit IRC | 15:06 | |
| EmilienM | dprince: yes, nova api needs those parameters | 15:06 |
| dprince | EmilienM: I guess just for neutron keystone auth | 15:07 |
| dprince | yeah, I'll add them there as well | 15:07 |
| dprince | EmilienM: thanks | 15:07 |
| EmilienM | thanks! | 15:08 |
| dprince | EmilienM: we could just put these into nova-base.yaml then | 15:08 |
| EmilienM | +1 | 15:08 |
| slagle | EmilienM: i'll fix that up after this merges if it works | 15:09 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Convert ServiceNetMap to a nested template https://review.openstack.org/353032 | 15:09 |
| shadower | hey dprince, just to let you know: mandre is away for a week but if you get to look at his mistral validation patches (https://review.openstack.org/#/c/313632/), I can update them etc. | 15:09 |
| *** alhernan has joined #tripleo | 15:09 | |
| dprince | shadower: great, I've got it open and will comment on it | 15:10 |
| shadower | cool | 15:10 |
| EmilienM | slagle: ack | 15:11 |
| *** dprince has quit IRC | 15:12 | |
| *** ebarrera has quit IRC | 15:13 | |
| *** jtomasek|biab is now known as jtomasek | 15:20 | |
| *** limao_ has joined #tripleo | 15:21 | |
| *** limao_ has quit IRC | 15:22 | |
| jtomasek | d0ugal: I see, so I think this is fine, you don't even need to put the parameter_defaults from it into mistral environment | 15:23 |
| jtomasek | d0ugal: it is just another environment which user provided and enabled. Although if CLI adds some parameter_defaults to it during the process, then I'd prefer if it added those using updateParameters action | 15:25 |
| d0ugal | jtomasek: I think it makes sense, otherwise if you edit in the UI then it will be stored in two places. | 15:25 |
| d0ugal | jtomasek: Yeah, so you seen the params in the mistral env, they were all taken out of the custom env file. | 15:25 |
| jtomasek | d0ugal: the rule I try to follow here is that client (CLI/GUI) should only store configuration provided through it in mistal environment and not amend the templates/environments directly | 15:26 |
| d0ugal | jtomasek: Makes sense. | 15:26 |
| jtomasek | d0ugal: so ideally the new environment that user provides remains unchanged (as user provided it) and everything else is in mistral env | 15:27 |
| *** limao has quit IRC | 15:27 | |
| d0ugal | jtomasek: We can't leave it unchanged really. | 15:28 |
| d0ugal | jtomasek: Need to update local paths so they make sense on swift for example | 15:28 |
| *** xuao has quit IRC | 15:31 | |
| jtomasek | d0ugal: I see, that makes sense, yeah | 15:35 |
| jtomasek | d0ugal: how long do you need to support this for backwards compatibility? The GUI is not going to be able to provide this functionality. I think we should move to running heat validate as part of the plan create/update workflow, so user gets notified that heat can't resolve the paths and let the user fix them | 15:38 |
| d0ugal | jtomasek: No idea how long... | 15:39 |
| d0ugal | jtomasek: I guess we can mark it as deprecated in this cycle - but not sure when it can be removed. the next? | 15:39 |
| *** leanderthal is now known as leanderthal|afk | 15:41 | |
| openstackgerrit | greghaynes proposed openstack/diskimage-builder: Add blurb about communication to docs landing page https://review.openstack.org/355533 | 15:42 |
| d0ugal | jtomasek: I guess we need a new CLI story before we can deprecate everything. create plan, deploy plan etc. | 15:44 |
| d0ugal | jtomasek: This is just the old command being powered by the new workflows. | 15:44 |
| jtomasek | d0ugal: yep, that's right | 15:44 |
| d0ugal | s/everything/anything/ | 15:44 |
| *** weshay is now known as weshay_brb | 15:45 | |
| jtomasek | d0ugal: is the user-environment.yaml combined of all that user provides with -e ? | 15:47 |
| *** egafford has joined #tripleo | 15:47 | |
| *** awaugama is now known as awaugama_afk | 15:47 | |
| d0ugal | jtomasek: Yeah, and in theory any that the CLI creates | 15:47 |
| jtomasek | d0ugal: ok | 15:48 |
| d0ugal | jtomasek: I just found out that I can store this file in the mistral env | 15:49 |
| jtomasek | d0ugal: this solution sounds fine to me. Only thing is that CLI should store parameter defaults in mistral env rather than creating new environment itself | 15:49 |
| d0ugal | jtomasek: There is a data option for environments that I hadn't understood. rbrady just explained it to me | 15:49 |
| d0ugal | jtomasek: Yeah, well, re-writing the CLI to not create an environment with parameters is non-trivial. So I just take the result and remove the parameters and add them to the mistral env. | 15:49 |
| jtomasek | d0ugal: I see, ok, add TODOs then:) | 15:50 |
| d0ugal | jtomasek: non-trivial as it would require changes to os-cloud-config and I'm not sure what else. | 15:50 |
| jtomasek | ok | 15:50 |
| d0ugal | but yeah, a TODO makes sense. | 15:50 |
| d0ugal | jtomasek: I am going to move the user-environment.yaml file into the Mistral env. | 15:51 |
| jtomasek | ok | 15:51 |
| *** weshay_brb is now known as weshay | 15:52 | |
| jtomasek | d0ugal: umm, wait...:) this is problematic... you do that, GUI won't be able to list that environment because it is a new concept. | 15:53 |
| d0ugal | jtomasek: well, it is an old concept :-D dprince added it in the original version | 15:54 |
| *** bana_k has joined #tripleo | 15:54 | |
| jtomasek | d0ugal: getCapabilities will then need to include this environment in the listing | 15:54 |
| d0ugal | jtomasek: and I just didn't know about it, I'll show you what it looks like, just a sec | 15:54 |
| *** dsariel has joined #tripleo | 15:54 | |
| d0ugal | jtomasek: https://paste.fedoraproject.org/408838/71276667/raw/ | 15:59 |
| d0ugal | jtomasek: see the environments section. | 15:59 |
| *** jpich has quit IRC | 16:00 | |
| jtomasek | d0ugal: I see, problem really is that there is no way to list this in the list of environments in GUI | 16:00 |
| *** yamahata has joined #tripleo | 16:01 | |
| d0ugal | jtomasek: right, I could use the file approach is that is better but it seems this is what dprince had intended I do | 16:01 |
| *** ifarkas is now known as ifarkas_afk | 16:01 | |
| *** dtantsur|mtg is now known as dtantsur | 16:02 | |
| dtantsur | slagle, so, do our internal endpoints also use https? | 16:02 |
| jtomasek | d0ugal: what if we'd create empty environment in environments/user-environment.yaml which would be also included in capabilities map and CLI would update this one? | 16:02 |
| *** apetrich has joined #tripleo | 16:02 | |
| jtomasek | d0ugal: that way GUI could list that environment no problem | 16:03 |
| jtomasek | and everything would just work | 16:03 |
| d0ugal | jtomasek: Sure, I would be okay with that. | 16:03 |
| jtomasek | your approach would work no problem if we had environment metadata instead of capabilities map. but it is not there for Newton | 16:04 |
| d0ugal | jtomasek: Right, maybe we should remove the data method - I think it might actually be broken, I got an error. | 16:05 |
| mhenkel | Hi all, I need some advice on pep8 validation for composable services. My build fails with | 16:05 |
| mhenkel | 2016-08-15 15:12:27.193049 | ERROR: service_name should match file name for service: ./puppet/services/contrail/contrail-analytics.yaml. | 16:05 |
| mhenkel | 2016-08-15 15:12:27.193076 | Validating ./puppet/services/contrail/contrail-database.yaml | 16:05 |
| *** egafford has quit IRC | 16:05 | |
| jtomasek | d0ugal: I am not against:) | 16:05 |
| mhenkel | however, my service_name matches the file name: | 16:06 |
| d0ugal | jtomasek: lol, I am against broken code tho :-D | 16:06 |
| mhenkel | service_name: contrail-database | 16:06 |
| jtomasek | d0ugal: haha | 16:06 |
| mhenkel | any idea? | 16:06 |
| mhenkel | actually this is the relevant error: | 16:06 |
| mhenkel | 2016-08-15 15:12:27.193076 | Validating ./puppet/services/contrail/contrail-database.yaml | 16:06 |
| mhenkel | 2016-08-15 15:12:27.193110 | ERROR: service_name should match file name for service: ./puppet/services/contrail/contrail-database.yaml. | 16:06 |
| *** cwolferh has joined #tripleo | 16:09 | |
| ansiwen | EmilienM: what would be a good starting point for openstack_integration::ec2api ? can I use the puppet stuff from the beaker test? | 16:09 |
| *** jtomasek is now known as jtomasek|afk | 16:10 | |
| openstackgerrit | John Browning proposed openstack/tripleo-validations: Added check for keystone token_flush cronjob. https://review.openstack.org/355557 | 16:11 |
| *** lucasagomes is now known as lucas-afk | 16:17 | |
| *** ebarrera has joined #tripleo | 16:19 | |
| *** egafford has joined #tripleo | 16:20 | |
| openstackgerrit | John Browning proposed openstack/tripleo-validations: Check to ensure debug is disabled on undercloud services. https://review.openstack.org/355559 | 16:20 |
| jaosorior | slagle: will we then need the fix you did for the undercloud? Or will we use the proposed fix for ironic? | 16:23 |
| *** dprince has joined #tripleo | 16:24 | |
| *** yamahata has quit IRC | 16:29 | |
| slagle | jaosorior: whatever unblocks the promote the fastest | 16:29 |
| slagle | if the ironic patch lands before my i-u patch, i'll abandon it | 16:29 |
| *** bana_k has quit IRC | 16:31 | |
| jaosorior | slagle: so, it feasble to instruct users to rebuild the ironic-python-agent image if needing to use SSL? Even went on to ask in the ironic channel and that does seem to be the expected approach from their end | 16:31 |
| slagle | jaosorior: yes i think so | 16:32 |
| jaosorior | slagle: they are open to other solutions but that would require major changes. | 16:32 |
| dtantsur | slagle, I'll find someone to merge your patch after it passes the CI. thanks for spotting it! | 16:32 |
| dtantsur | also not checking call arguments makes kittens sad :-/ | 16:34 |
| *** aufi has quit IRC | 16:35 | |
| hewbrocca | Poor kittens! | 16:36 |
| *** tosky has quit IRC | 16:37 | |
| * dtantsur shakes fist at keystoneclient | 16:39 | |
| *** florianf has quit IRC | 16:40 | |
| *** ohamada has quit IRC | 16:41 | |
| *** awaugama_afk is now known as awaugama | 16:43 | |
| *** pkovar has quit IRC | 16:49 | |
| *** lucas-afk is now known as lucasagomes | 16:58 | |
| *** jaosorior has quit IRC | 17:06 | |
| *** Goneri has quit IRC | 17:07 | |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Move Nova settings out of puppet/compute.yaml https://review.openstack.org/353748 | 17:09 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Mv Ceilometer settings out of puppet/compute.yaml https://review.openstack.org/354063 | 17:09 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Move neutron settings out of puppet/compute.yaml https://review.openstack.org/354201 | 17:09 |
| *** dprince has quit IRC | 17:09 | |
| *** yamahata has joined #tripleo | 17:09 | |
| *** bana_k has joined #tripleo | 17:09 | |
| *** rajinir has joined #tripleo | 17:12 | |
| *** hewbrocca is now known as hewbrocca-afk | 17:15 | |
| *** ebarrera has quit IRC | 17:16 | |
| *** dhill_ has quit IRC | 17:16 | |
| *** ebarrera has joined #tripleo | 17:16 | |
| *** dhill_ has joined #tripleo | 17:17 | |
| *** zoli|wfh is now known as zoli|gone | 17:19 | |
| *** zoli|gone is now known as zoli_gone-proxy | 17:19 | |
| openstackgerrit | Justin Kilpatrick proposed openstack/tripleo-quickstart: Add documentation for add-on roles https://review.openstack.org/346733 | 17:21 |
| sshnaidm|afk | slagle, hi, could you please merge? https://review.openstack.org/#/c/352598/ | 17:23 |
| sshnaidm|afk | slagle, and this https://review.openstack.org/#/c/353103/ | 17:24 |
| *** lucasagomes is now known as lucas-dinner | 17:25 | |
| *** ebarrera has quit IRC | 17:25 | |
| *** ayoung has quit IRC | 17:25 | |
| *** dtantsur is now known as dtantsur|afk | 17:26 | |
| *** [1]cdearborn has quit IRC | 17:30 | |
| *** dprince has joined #tripleo | 17:36 | |
| *** rcernin has joined #tripleo | 17:37 | |
| *** cdearborn has joined #tripleo | 17:37 | |
| *** milan has quit IRC | 17:39 | |
| *** ayoung has joined #tripleo | 17:41 | |
| *** athomas has quit IRC | 17:45 | |
| EmilienM | ansiwen: look how other services work | 17:47 |
| ansiwen | EmilienM: I did, but it's really not easy to understand what is necessary and what not | 17:48 |
| *** cdearborn has quit IRC | 17:48 | |
| *** [1]cdearborn has joined #tripleo | 17:48 | |
| *** rkolli has joined #tripleo | 17:48 | |
| ansiwen | EmilienM: I have a first version now | 17:48 |
| ansiwen | EmilienM: I will also have to change puppet-tempest, right? | 17:49 |
| EmilienM | ansiwen: well, if you understand puppet it's not hard | 17:49 |
| EmilienM | yes, if tempest.conf needs a change | 17:49 |
| ansiwen | EmilienM: exactly, that's the problem, IF you understand puppet :-) | 17:49 |
| EmilienM | ;) | 17:50 |
| ansiwen | EmilienM: how do I find out which test I have to add to TESTS in run_tests.sh? | 17:58 |
| openstackgerrit | Michael Henkel proposed openstack/tripleo-heat-templates: add composable services for Contrail https://review.openstack.org/346799 | 17:58 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: gnocchi: use new authtoken parameters https://review.openstack.org/355585 | 18:05 |
| ansiwen | EmilienM: I don't understand how I tell the integration test to download the ec2api tempest tests and run them. | 18:06 |
| EmilienM | ansiwen: are they packages in RDO? | 18:07 |
| EmilienM | packaged* | 18:07 |
| ansiwen | EmilienM: I have no idea :-( | 18:08 |
| *** dsariel has quit IRC | 18:09 | |
| EmilienM | ansiwen: do you know where are rdo packages? | 18:09 |
| *** sarath has joined #tripleo | 18:09 | |
| *** maeca1 has quit IRC | 18:13 | |
| *** mbound has quit IRC | 18:16 | |
| slagle | sshnaidm|afk: i dont see where https://review.openstack.org/#/c/352598/ ever passed ovb-ha, do you? | 18:17 |
| *** apetrich has quit IRC | 18:18 | |
| sshnaidm|afk | slagle, I don't see it ever run there | 18:18 |
| *** [1]cdearborn has quit IRC | 18:19 | |
| slagle | it did run. it just never passed | 18:19 |
| ansiwen | EmilienM: you mean http://buildlogs.centos.org/centos/7/cloud/x86_64/rdo-trunk-master-tested/ ? | 18:19 |
| *** cdearborn has joined #tripleo | 18:21 | |
| EmilienM | ansiwen: that's the CDN repo | 18:23 |
| EmilienM | trunk is here: https://trunk.rdoproject.org/ | 18:23 |
| *** rkolli_ has joined #tripleo | 18:23 | |
| EmilienM | so for ex | 18:23 |
| EmilienM | https://trunk.rdoproject.org/centos7-master/93/27/9327cc7f549ef7c7efe07949bb86b0570cd62bcd_282f7f4f/ | 18:23 |
| EmilienM | I don't see a package for ec2-api-tests | 18:23 |
| sshnaidm|afk | slagle, http://logs.openstack.org/98/352598/1/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/f52709a/ | 18:25 |
| *** rkolli has quit IRC | 18:26 | |
| slagle | sshnaidm|afk: ok, please link that into the review | 18:26 |
| slagle | if it's not there | 18:27 |
| sshnaidm|afk | slagle, I took it fom review | 18:27 |
| *** electrofelix has quit IRC | 18:27 | |
| ansiwen | EmilienM: right... so? | 18:27 |
| slagle | ok | 18:27 |
| EmilienM | ansiwen: if there are no tests, we can't test ec2-api except if we install it from pip, thing we don't do in our ci | 18:28 |
| EmilienM | so someone needs to add missing package into ec2-api | 18:29 |
| sshnaidm|afk | slagle, also I don't think this one is worth to recheck: https://review.openstack.org/#/c/353103/ | 18:29 |
| *** apetrich has joined #tripleo | 18:29 | |
| ansiwen | EmilienM: "someone"? | 18:29 |
| *** dsariel has joined #tripleo | 18:30 | |
| *** ayoung has quit IRC | 18:37 | |
| slagle | sshnaidm|afk: agreed | 18:38 |
| slagle | dprince: are you ok with https://review.openstack.org/#/c/353103/ ? | 18:38 |
| slagle | dprince: if so, I will update the ssh keys | 18:38 |
| slagle | on rh{1,2} | 18:39 |
| dprince | slagle: looking now | 18:40 |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: create playbook for oooq replacement of instack-virt-setup https://review.openstack.org/355595 | 18:40 |
| *** rcernin has quit IRC | 18:42 | |
| dprince | slagle: +2 | 18:42 |
| openstackgerrit | Jeff Peeler proposed openstack/tripleo-common: Include environments in capabilities output https://review.openstack.org/355598 | 18:54 |
| sshnaidm|afk | slagle, dprince could you please review/merge this one as well? https://review.openstack.org/#/c/352566/ it's not checked in CI anyway | 18:56 |
| slagle | sshnaidm|afk: i can't actually add ssh keys to the bastion host for rh1. i'll ask around and find out who can | 18:56 |
| sshnaidm|afk | slagle, ok, thanks | 18:56 |
| dprince | sshnaidm|afk: +2 | 18:57 |
| sshnaidm|afk | dprince, thanks | 18:58 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Add tripleo-packages composable service https://review.openstack.org/355605 | 19:13 |
| *** mhenkel has quit IRC | 19:13 | |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Add tripleo-packages composable service https://review.openstack.org/355605 | 19:13 |
| openstackgerrit | greghaynes proposed openstack/diskimage-builder: Clarify OVERWRITE_OLD_IMAGE docs https://review.openstack.org/355607 | 19:14 |
| openstackgerrit | greghaynes proposed openstack/diskimage-builder: Clarify OVERWRITE_OLD_IMAGE docs https://review.openstack.org/355607 | 19:15 |
| *** mbound has joined #tripleo | 19:17 | |
| *** alhernan has quit IRC | 19:18 | |
| *** iogg has joined #tripleo | 19:19 | |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Add tripleo-packages composable service https://review.openstack.org/355605 | 19:20 |
| openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: (mitaka only) enable sysctl nonlocal_bind https://review.openstack.org/355612 | 19:21 |
| *** mbound has quit IRC | 19:22 | |
| *** radeks has joined #tripleo | 19:24 | |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Add tripleo-firewall composable service https://review.openstack.org/355614 | 19:25 |
| EmilienM | dprince: I was thinking how to make https://review.openstack.org/#/c/355614/ required | 19:27 |
| EmilienM | dprince: like, how to make firewall class included everywhere | 19:27 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Defaults 'undef' to kernel_modules and sysctl_settings https://review.openstack.org/355615 | 19:29 |
| openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: Enable sysctl nonlocal_bind https://review.openstack.org/355051 | 19:29 |
| openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: Enable sysctl nonlocal_bind https://review.openstack.org/355051 | 19:30 |
| dprince | EmilienM: it is composable like everything else | 19:31 |
| dprince | EmilienM: We could add a GlobalService I guess? Gfidente had this idea a while back... but I don't think we need it yet | 19:31 |
| EmilienM | dprince: right, wfm. Just wanted to mention that our users will have to think about it | 19:31 |
| dprince | EmilienM: right, well I did add it into the default lists for all the roles | 19:32 |
| EmilienM | ok wfm now | 19:32 |
| dprince | EmilienM: before we add any more features to overcloud.yaml I'd rather have shardy's patches in for the jinja templating I think | 19:32 |
| *** Vijayendra has quit IRC | 19:33 | |
| EmilienM | dprince: ++ | 19:35 |
| openstackgerrit | Harry Rybacki proposed openstack/tripleo-quickstart: Expand templated bash to allow for doc generation https://review.openstack.org/347592 | 19:35 |
| *** ayoung has joined #tripleo | 19:48 | |
| *** yamahata has quit IRC | 19:49 | |
| *** yamahata has joined #tripleo | 19:49 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Remove deprecated hdp plugin and add ambari https://review.openstack.org/352598 | 19:52 |
| *** fragatina has quit IRC | 19:53 | |
| *** radeks has quit IRC | 19:55 | |
| *** ioggstream has joined #tripleo | 19:56 | |
| *** iogg has quit IRC | 19:56 | |
| EmilienM | slagle: sounds like https://review.openstack.org/#/c/355484/ helped to pass promotion jobs | 19:59 |
| EmilienM | slagle: oh, nevermind, I didn't see the "POST_FAILURE" on nonha | 19:59 |
| slagle | EmilienM: it hit a timeout | 20:00 |
| slagle | but it got past deployment, so it worked | 20:00 |
| *** Goneri has joined #tripleo | 20:00 | |
| EmilienM | mhh | 20:00 |
| EmilienM | nice | 20:00 |
| EmilienM | slagle: do you want me to patch puppet-ironic witht the option? | 20:00 |
| *** awaugama has quit IRC | 20:03 | |
| *** liverpooler has quit IRC | 20:07 | |
| *** jayg is now known as jayg|g0n3 | 20:09 | |
| *** ioggstream has quit IRC | 20:14 | |
| *** Goneri has quit IRC | 20:15 | |
| Slower | dprince: you still around? | 20:16 |
| dprince | Slower: yep, what is up dude | 20:16 |
| slagle | EmilienM: i dont think its needed | 20:16 |
| openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: Use http for Ironic deployments https://review.openstack.org/355484 | 20:16 |
| slagle | EmilienM: b/c the real fix is https://review.openstack.org/#/c/355537/ | 20:16 |
| EmilienM | amn | 20:16 |
| EmilienM | damn | 20:16 |
| Slower | dprince: hey so I'm getting someplace with the containers stuff :) | 20:16 |
| EmilienM | slagle: ah ok | 20:16 |
| dprince | Slower: nice | 20:16 |
| EmilienM | I'll abandon my patches then when it's merged & confirm to work | 20:16 |
| Slower | dprince: but I'm noticing I seem to need mysql available for puppet to configure services | 20:16 |
| dprince | Slower: yep, probably a pretty core requirement | 20:17 |
| Slower | right | 20:17 |
| slagle | EmilienM: i still want to land the i-u patch if it's ready before the ironic patch | 20:17 |
| Slower | dprince: I'm thinking I might need to add it to the firstboot | 20:17 |
| dprince | Slower: need my help on that one? | 20:17 |
| slagle | EmilienM: i had hopes of getting the ironic patch landed today, but that does not look likely | 20:17 |
| Slower | it probly needs to be up before the agent container.. | 20:17 |
| slagle | EmilienM: so who knows when that will land | 20:17 |
| dprince | Slower: really? | 20:17 |
| EmilienM | slagle: puppet patch can land today | 20:17 |
| dprince | Slower: why so early? | 20:17 |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: fix the staging image var LOCATION var https://review.openstack.org/355633 | 20:18 |
| EmilienM | dprince: can you review https://review.openstack.org/#/c/355632/ please? | 20:18 |
| Slower | dprince: I added mysql as a service | 20:18 |
| Slower | dprince: but the configure is happening before it tries to start it.. but maybe I have the order wrong | 20:18 |
| slagle | EmilienM: the ironic patch has ~ 4 failed CI jobs on it. and i have no idea if they are actually related to my patch or not | 20:19 |
| slagle | this must be what people feel like looking at tripleo-ci | 20:19 |
| Slower | hrrm, yeah it's trying to configure keystone and failing | 20:19 |
| Slower | so maybe I need to check the order.. | 20:19 |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: fix the staging image var LOCATION var https://review.openstack.org/355633 | 20:19 |
| EmilienM | slagle: ahah | 20:20 |
| EmilienM | slagle: what is scaring me is that your patch is +2 +A and not passing CI | 20:20 |
| Slower | or for now maybe disable keystone | 20:20 |
| slagle | EmilienM: i think that's how most projects do it, b/c they have real voting CI | 20:20 |
| EmilienM | slagle: but your patch is likely the fix we need | 20:20 |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: DO NOT MERGE - Periodic test https://review.openstack.org/346949 | 20:21 |
| bswartz | marios: ping | 20:27 |
| Slower | dprince: so I'll keep messing with it ;) | 20:27 |
| dprince | Slower: sounds good | 20:30 |
| dprince | Slower: we should pare again soon | 20:30 |
| dprince | Slower: or pair even | 20:30 |
| dprince | Slower: maybe even pear !? | 20:31 |
| dprince | EmilienM: does conductor not get it's api_url from keystone? | 20:32 |
| EmilienM | dprince: this patch allows to override it at needs | 20:32 |
| *** ioggstream has joined #tripleo | 20:36 | |
| dprince | EmilienM: +2, no harm there I guess | 20:36 |
| *** tobias_fiberdata has joined #tripleo | 20:38 | |
| *** tobias_fiberdata has quit IRC | 20:38 | |
| EmilienM | dprince: thx | 20:39 |
| *** fragatina has joined #tripleo | 20:41 | |
| *** fragatina has quit IRC | 20:42 | |
| *** fragatina has joined #tripleo | 20:43 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: gnocchi: use new authtoken parameters https://review.openstack.org/355585 | 20:44 |
| *** sarath has quit IRC | 20:48 | |
| openstackgerrit | Emilien Macchi proposed openstack/python-tripleoclient: WIP wire in jinja templating for custom roles https://review.openstack.org/336625 | 20:53 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Convert overcloud.yaml to support jinja2 templating https://review.openstack.org/315679 | 20:54 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Defaults empty hash to kernel_modules and sysctl_settings https://review.openstack.org/355615 | 20:57 |
| openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: Enable sysctl nonlocal_bind https://review.openstack.org/355051 | 20:57 |
| *** dprince has quit IRC | 20:59 | |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: Bump tempest version to latest https://review.openstack.org/355641 | 21:00 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: Bump tempest version to latest https://review.openstack.org/355641 | 21:02 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: WIP: DONT MERGE TESTING https://review.openstack.org/316436 | 21:03 |
| *** julim has quit IRC | 21:03 | |
| *** dsariel has quit IRC | 21:06 | |
| NobodyCam | question is init.d/ the best place to execute a script upon boot up? | 21:09 |
| NobodyCam | and good afternoon Triple-o folks | 21:09 |
| *** lblanchard has quit IRC | 21:10 | |
| *** rhallisey has quit IRC | 21:14 | |
| *** ioggstream has quit IRC | 21:20 | |
| *** apetrich has quit IRC | 21:29 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-pacemaker: Fix CI puppet-lint and type issues https://review.openstack.org/355202 | 21:35 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-pacemaker: Move other-requirements.txt to bindep.txt https://review.openstack.org/355001 | 21:35 |
| *** jcoufal has quit IRC | 21:35 | |
| *** yamahata has quit IRC | 21:38 | |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: fix the instructions that are generated by quickstart.sh https://review.openstack.org/355653 | 21:44 |
| beagles | EmilienM, slagle: can I get an "amen" on a backport? https://review.openstack.org/#/c/354878/ | 22:00 |
| beagles | (or a boo ;)) | 22:00 |
| *** yamahata has joined #tripleo | 22:01 | |
| *** onovy has quit IRC | 22:03 | |
| *** saneax_AFK is now known as saneax | 22:05 | |
| *** saneax is now known as Guest16777 | 22:05 | |
| EmilienM | beagles: done | 22:06 |
| beagles | EmilienM, merci beaucoup monsieur! | 22:07 |
| * beagles is quickly exhausting his French | 22:07 | |
| beagles | EmilienM, did mordred or anybody from infra reach out to tell you that the fix that was breaking the ubuntu builds had landed? | 22:10 |
| *** fragatin_ has joined #tripleo | 22:10 | |
| *** cdearborn has quit IRC | 22:11 | |
| *** beagles is now known as beagles_brb | 22:11 | |
| *** fragatina has quit IRC | 22:13 | |
| *** fragatin_ has quit IRC | 22:14 | |
| *** fragatina has joined #tripleo | 22:15 | |
| *** bswartz has quit IRC | 22:15 | |
| *** jistr has quit IRC | 22:18 | |
| *** jistr has joined #tripleo | 22:19 | |
| *** onovy has joined #tripleo | 22:19 | |
| *** bfournie has quit IRC | 22:22 | |
| *** thrash is now known as thrash|g0ne | 22:28 | |
| *** weshay has quit IRC | 22:32 | |
| *** beagles_brb is now known as beagles | 22:42 | |
| *** weshay has joined #tripleo | 22:44 | |
| *** pabelanger has quit IRC | 22:45 | |
| *** pabelanger has joined #tripleo | 22:45 | |
| *** mbound has joined #tripleo | 22:47 | |
| *** mbound has quit IRC | 22:52 | |
| *** egafford has quit IRC | 22:55 | |
| *** Goneri has joined #tripleo | 23:08 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Defaults empty hash to kernel_modules and sysctl_settings https://review.openstack.org/355615 | 23:24 |
| openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: Use http for Ironic deployments https://review.openstack.org/355484 | 23:25 |
| EmilienM | beagles: nope | 23:27 |
| EmilienM | beagles: but I figured | 23:27 |
| beagles | EmilienM, ack... I didn't really expect mordred to remember me :) | 23:27 |
| *** Vijayendra has joined #tripleo | 23:34 | |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: DO NOT MERGE - Periodic test. https://review.openstack.org/346949 | 23:40 |
| *** myoung is now known as myoung|afk | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!