*** bana_k has quit IRC | 00:20 | |
*** pmannidi has joined #tripleo | 00:25 | |
*** bana_k has joined #tripleo | 00:25 | |
*** tzumainn has joined #tripleo | 00:39 | |
*** limao has joined #tripleo | 00:48 | |
*** bana_k has quit IRC | 00:51 | |
*** michapma_alt has joined #tripleo | 01:00 | |
*** ecerquei_ has joined #tripleo | 01:28 | |
*** limao has quit IRC | 01:31 | |
*** limao has joined #tripleo | 01:31 | |
*** limao has quit IRC | 01:32 | |
*** limao has joined #tripleo | 01:32 | |
*** asalkeld has joined #tripleo | 01:42 | |
*** noslzzp has quit IRC | 01:44 | |
*** noslzzp has joined #tripleo | 01:45 | |
*** asalkeld has quit IRC | 01:47 | |
*** dmacpher has joined #tripleo | 02:03 | |
*** noslzzp has quit IRC | 02:11 | |
openstackgerrit | Dan Prince proposed openstack/tripleo-puppet-elements: Disable old hiera.yaml o-a-c script https://review.openstack.org/392615 | 02:20 |
---|---|---|
openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Horizon service cleanups for hiera json hook https://review.openstack.org/393258 | 02:21 |
openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Hiera optimization: use a new hiera hook https://review.openstack.org/384757 | 02:22 |
*** liverpooler has quit IRC | 02:22 | |
*** ecerquei_ has quit IRC | 03:04 | |
*** limao has quit IRC | 03:47 | |
*** chandankumar has joined #tripleo | 03:50 | |
*** sudipto has joined #tripleo | 04:02 | |
*** sudipto_ has joined #tripleo | 04:02 | |
*** links has joined #tripleo | 04:15 | |
*** yolanda has quit IRC | 04:18 | |
*** limao has joined #tripleo | 04:27 | |
*** limao has quit IRC | 04:34 | |
*** limao has joined #tripleo | 04:50 | |
*** sudswas__ has joined #tripleo | 04:57 | |
*** sudipto_ has quit IRC | 04:57 | |
*** sudipto has quit IRC | 04:58 | |
*** sudipto_ has joined #tripleo | 04:58 | |
*** bana_k has joined #tripleo | 04:59 | |
*** sudipto_ has quit IRC | 04:59 | |
*** sudswas__ has quit IRC | 04:59 | |
*** pgadiya has joined #tripleo | 05:04 | |
*** oshvartz has quit IRC | 05:22 | |
*** coolsvap has joined #tripleo | 05:24 | |
*** masco has joined #tripleo | 05:27 | |
*** sudipto has joined #tripleo | 05:38 | |
*** sudipto_ has joined #tripleo | 05:38 | |
*** bana_k has quit IRC | 05:50 | |
*** sudipto_ has quit IRC | 05:50 | |
*** sudipto has quit IRC | 05:50 | |
*** bkopilov has joined #tripleo | 05:55 | |
*** skramaja_ has quit IRC | 05:59 | |
*** skramaja has joined #tripleo | 06:03 | |
*** abregman has joined #tripleo | 06:05 | |
*** jaosorior has joined #tripleo | 06:09 | |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/instack-undercloud: Always deploy HAProxy https://review.openstack.org/360368 | 06:13 |
*** TSCHAK has joined #tripleo | 06:15 | |
*** Vijayendra has joined #tripleo | 06:15 | |
*** bkopilov_ has joined #tripleo | 06:17 | |
*** pabelanger has quit IRC | 06:28 | |
*** pabelanger has joined #tripleo | 06:34 | |
*** abehl has joined #tripleo | 06:36 | |
*** dsariel has joined #tripleo | 06:38 | |
*** jaosorior has quit IRC | 06:39 | |
*** jaosorior has joined #tripleo | 06:40 | |
*** ealcaniz has joined #tripleo | 06:45 | |
*** udesale has joined #tripleo | 06:45 | |
*** oshvartz has joined #tripleo | 06:58 | |
openstackgerrit | Merged openstack/diskimage-builder: elements: zypper-minimal: Refresh repositories after adding the cache https://review.openstack.org/392004 | 07:01 |
*** lmiccini has joined #tripleo | 07:01 | |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for Barbican API https://review.openstack.org/397021 | 07:05 |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for Barbican API https://review.openstack.org/397021 | 07:06 |
*** iranzo has joined #tripleo | 07:08 | |
openstackgerrit | Merged openstack/diskimage-builder: elements: zypper-minimal: Mount common pseudo filesystems https://review.openstack.org/392986 | 07:08 |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Enable internal TLS for Barbican API https://review.openstack.org/397025 | 07:10 |
*** bana_k has joined #tripleo | 07:11 | |
openstackgerrit | Merged openstack/diskimage-builder: Install lsb package by map name instead of package name https://review.openstack.org/396762 | 07:13 |
*** tzumainn has quit IRC | 07:16 | |
*** rasca has joined #tripleo | 07:17 | |
*** jprovazn has joined #tripleo | 07:19 | |
jaosorior | B/exit | 07:27 |
*** jaosorior has quit IRC | 07:27 | |
openstackgerrit | Merged openstack/diskimage-builder: Updated from global requirements https://review.openstack.org/395311 | 07:31 |
*** bana_k has quit IRC | 07:32 | |
*** florianf has joined #tripleo | 07:33 | |
*** yolanda has joined #tripleo | 07:33 | |
openstackgerrit | Merged openstack/puppet-tripleo: Ensure keepalived is restarted when necessary. https://review.openstack.org/396731 | 07:36 |
openstackgerrit | Merged openstack/instack-undercloud: Make inspection also respect whether iPXE is enabled https://review.openstack.org/393418 | 07:36 |
openstackgerrit | Merged openstack/tripleo-heat-templates: Fixes missing OVS Firewall config with OpenDaylight https://review.openstack.org/396734 | 07:36 |
*** mcornea has joined #tripleo | 07:38 | |
*** fzdarsky has joined #tripleo | 07:39 | |
*** tdasilva has quit IRC | 07:41 | |
*** dsariel has quit IRC | 07:44 | |
openstackgerrit | Noam Angel proposed openstack/diskimage-builder: redhat-common add option to select networking service NetworkManager/network https://review.openstack.org/392170 | 07:46 |
*** jbadiapa has joined #tripleo | 07:47 | |
*** jaosorior has joined #tripleo | 07:55 | |
openstackgerrit | Merged openstack/tripleo-heat-templates: Fix inconsistent Manila service naming https://review.openstack.org/396662 | 07:56 |
openstackgerrit | Merged openstack/instack-undercloud: Prevent pxe_ilo driver from guessing the boot mode to use https://review.openstack.org/392181 | 07:56 |
openstackgerrit | Merged openstack/python-tripleoclient: Default the stack name to 'overcloud' in overcloud node delete https://review.openstack.org/395522 | 07:56 |
*** dsariel has joined #tripleo | 07:56 | |
*** pcaruana has joined #tripleo | 07:57 | |
*** shardy has joined #tripleo | 07:58 | |
*** ccamacho has joined #tripleo | 08:01 | |
jaosorior | easy commits if anybody has time to review https://review.openstack.org/#/c/396517/ https://review.openstack.org/#/c/396516/ https://review.openstack.org/#/c/396518/ | 08:02 |
*** tdasilva has joined #tripleo | 08:03 | |
*** cylopez has joined #tripleo | 08:07 | |
*** ohamada has joined #tripleo | 08:09 | |
openstackgerrit | mathieu bultel proposed openstack-infra/tripleo-ci: Implement overcloud upgrade job - Mitaka -> Newton https://review.openstack.org/323750 | 08:09 |
jaosorior | shardy: hey dude, do you have some minutes to explain to me how the workflow with heat requesting the ec2 credentials work? I'm still stuck in the same issue as friday, and hopefully getting an explanation of that workflow will help me out debugging it | 08:09 |
openstackgerrit | Adriano Petrich proposed openstack/tripleo-heat-templates: GATE TEST, please ignore https://review.openstack.org/365449 | 08:10 |
*** bkopilov has quit IRC | 08:10 | |
shardy | jaosorior: sure, I think I covered it in quite a bit of detail last week - did you want to read the IRC log and get back to me with specific questions, or should I summarise it again? | 08:11 |
jaosorior | shardy: I would sure use a summary, I tried going back to the logs but honestly ended up a bit confused. | 08:12 |
shardy | jaosorior: Ok, there are basically two steps | 08:12 |
shardy | 1. Heat calculates a signature so it can add a pre-signed URL for a SoftwareDeployment into the server metadata | 08:13 |
shardy | that happens here: | 08:13 |
*** bana_k has joined #tripleo | 08:14 | |
shardy | https://github.com/openstack/heat/blob/master/heat/engine/resources/signal_responder.py#L174 | 08:14 |
*** jlinkes has joined #tripleo | 08:14 | |
shardy | see the "request" map there, it contains all the details about the request which keystoneclient uses to calculate the signature | 08:14 |
*** dsariel has quit IRC | 08:14 | |
shardy | 2. We get an API request, containing the previously calculated signature. We call the keystone ec2tokens API with the exact same data used in "request" above | 08:15 |
shardy | and keystone gives us an ack/nack on the signature based on calculating a signature and comparing with what we pass from the request | 08:15 |
shardy | https://github.com/openstack/heat/blob/master/heat/api/aws/ec2token.py#L211 | 08:16 |
shardy | that happens there | 08:16 |
shardy | jaosorior: last week I think our discussion stalled because it wasn't clear if the signature calculation was the reason for the API request sending the heat code down the "AWS authentication failure" path | 08:17 |
*** zoli|gone is now known as zoli | 08:19 | |
jaosorior | shardy: that's correct, at some point I was seeing success there for the signature validation, which was pretty strange. So I thought I screwed my deployment. I'm currently deploying another one from scratch to test it again. | 08:19 |
openstackgerrit | Merged openstack/os-refresh-config: Updated from global requirements https://review.openstack.org/389925 | 08:20 |
openstackgerrit | Merged openstack/os-collect-config: Updated from global requirements https://review.openstack.org/389924 | 08:20 |
shardy | jaosorior: Yeah, that's where I got confused tbh - you pasted some stuff which seemed inconsistent so I guess the first step is to get back to a consistent reproducer and debug from there | 08:20 |
openstackgerrit | Merged openstack/tripleo-common: Updated from global requirements https://review.openstack.org/389957 | 08:20 |
*** percevalbot has joined #tripleo | 08:21 | |
openstackgerrit | mathieu bultel proposed openstack-infra/tripleo-ci: Implement overcloud upgrade job - Mitaka -> Newton https://review.openstack.org/323750 | 08:22 |
jaosorior | anyway, it's deploying currently,s o hopefully I can reproduce it again in a bit | 08:22 |
zoli | good morning | 08:22 |
*** arxcruz has joined #tripleo | 08:23 | |
*** dsariel has joined #tripleo | 08:27 | |
*** ebarrera has joined #tripleo | 08:28 | |
*** TSCHAK has quit IRC | 08:28 | |
*** bana_k has quit IRC | 08:29 | |
*** aufi has joined #tripleo | 08:31 | |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/instack-undercloud: Deploy heat APIs over httpd https://review.openstack.org/394837 | 08:35 |
*** milan has joined #tripleo | 08:35 | |
*** abregman_ has joined #tripleo | 08:37 | |
*** jpena|off is now known as jpena | 08:37 | |
*** abregman has quit IRC | 08:39 | |
*** tremble has joined #tripleo | 08:40 | |
*** nyechiel has joined #tripleo | 08:42 | |
*** hewbrocca_afk is now known as hewbrocca | 08:42 | |
sshnaidm | hi all | 08:42 |
jaosorior | sshnaidm: sup dude | 08:43 |
*** d0ugal has joined #tripleo | 08:43 | |
*** amoralej|off is now known as amoralej | 08:44 | |
sshnaidm | doe anybody know what can cause error like: Error: Duplicate declaration: Nova_config[DEFAULT/host] is already declared in file /etc/puppet/modules/nova/manifests/init.pp | 08:44 |
jaosorior | sshnaidm: undercloud or overcloud? | 08:45 |
jaosorior | sshnaidm: and in what job did this error appear? | 08:45 |
sshnaidm | jaosorior, overcloud | 08:45 |
jaosorior | shardy: can you +A this? https://review.openstack.org/#/c/396434/ | 08:46 |
sshnaidm | jaosorior, it's try of running oooq with stable branch on CI environment: http://logs.openstack.org/94/381094/31/check-tripleo/gate-tripleo-ci-centos-7-ovb-nonha-mitaka/e0c5bd2/logs/undercloud/var/log/oooq/failed_deployments.txt.gz | 08:47 |
*** abregman_ has quit IRC | 08:47 | |
sshnaidm | jaosorior, interesting that it doesn't happen in master | 08:47 |
*** abregman has joined #tripleo | 08:47 | |
*** jpich has joined #tripleo | 08:48 | |
jaosorior | sshnaidm: it doesn't appear there cause we removed it from the manifests | 08:49 |
jaosorior | sshnaidm: this is the issue https://github.com/openstack/tripleo-heat-templates/blob/stable/mitaka/puppet/manifests/overcloud_compute.pp#L100 | 08:49 |
*** lmiccini has quit IRC | 08:50 | |
jaosorior | uhm... interesting | 08:51 |
jaosorior | sshnaidm: so, DEFAULT/host wasn't set in puppet nova in stable/mitaka | 08:51 |
jaosorior | it was introduced, and so we took it off the tripleo manifests | 08:51 |
*** sudipto has joined #tripleo | 08:52 | |
*** lmiccini has joined #tripleo | 08:52 | |
jaosorior | now, checking the stable/mitaka puppet-nova, that value is not set there, so I'm thinking you might have a puppet manifest that's too new | 08:52 |
sshnaidm | jaosorior, I'll check the repos.. | 08:53 |
jaosorior | sshnaidm: this is how that value is set in puppet-nova master https://github.com/openstack/puppet-nova/blob/master/manifests/init.pp#L643 and it's not existent in stable/mitaka https://github.com/openstack/puppet-nova/blob/stable/mitaka/manifests/init.pp | 08:53 |
*** lmiccini has quit IRC | 08:56 | |
sshnaidm | jaosorior, how can I check if I use the right version? I'm on the host | 08:57 |
sshnaidm | puppet modules version seems right.. http://paste.openstack.org/show/589076/ | 09:00 |
*** zoli is now known as zoli|doctor | 09:00 | |
jaosorior | no idea | 09:01 |
jaosorior | sshnaidm: but you could check if DEFAULT/host is being set in init.pp | 09:01 |
*** flepied has joined #tripleo | 09:01 | |
jaosorior | sshnaidm: etc/puppet/modules/nova/manifests/init.pp | 09:01 |
sshnaidm | jaosorior, checking.. | 09:01 |
jaosorior | sshnaidm: /etc/puppet/modules/nova/manifests/init.pp | 09:01 |
sshnaidm | jaosorior, on overcloud it's there | 09:03 |
sshnaidm | 'DEFAULT/host': value => $host;\ | 09:04 |
sshnaidm | jaosorior, so, could it be that I install modules for master in mitaka..? | 09:05 |
*** dmacpher has quit IRC | 09:06 | |
jaosorior | sshnaidm: or the newton ones | 09:06 |
sshnaidm | jaosorior, thanks, will check this.. | 09:07 |
sshnaidm | jaosorior, how is going with approving our network range patch? | 09:07 |
jaosorior | sshnaidm: you got a +2 from my side, you need another core | 09:09 |
*** lmiccini has joined #tripleo | 09:12 | |
*** abregman has quit IRC | 09:12 | |
*** jtomasek has joined #tripleo | 09:12 | |
*** dbecker has joined #tripleo | 09:13 | |
*** chem has joined #tripleo | 09:15 | |
*** abregman has joined #tripleo | 09:18 | |
*** limao has quit IRC | 09:20 | |
hewbrocca | are we finally fixing that ridiculous 192.0.2.1 thing? | 09:24 |
*** limao has joined #tripleo | 09:24 | |
openstackgerrit | yolanda.robla proposed openstack/tripleo-docs: Add documentation on how to use whole disk images https://review.openstack.org/397075 | 09:24 |
openstackgerrit | yolanda.robla proposed openstack/python-tripleoclient: Support whole disk images in TripleO https://review.openstack.org/394426 | 09:26 |
shardy | bandini: Hey can we declare https://bugs.launchpad.net/tripleo/+bug/1628998 complete or is there more work to do? | 09:29 |
openstack | Launchpad bug 1628998 in tripleo "Change rabbitmq queues HA mode from ha-all to ha-exactly" [High,In progress] - Assigned to Michele Baldessari (michele) | 09:29 |
*** athomas has joined #tripleo | 09:30 | |
bandini | shardy: we need to land this one https://review.openstack.org/#/c/389309/ (although if we land the ansible composable upgrade we will need to remember to migrate that to ansible) | 09:32 |
openstackgerrit | Merged openstack/diskimage-builder: Avoid disabling rhel-7-server-rh-common-rpms https://review.openstack.org/384533 | 09:32 |
bandini | mmh I see that the commented code in that review needs updateing | 09:32 |
shardy | bandini: ack, thanks - if you push an update I'll review/approve | 09:33 |
shardy | If that closes the bug, can you also add Closes-Bug to the commit message please? | 09:34 |
shardy | bandini: re the ansible upgrades, yeah I've been wondering how to handle that | 09:34 |
shardy | one option would be just to call the script functions from ansible and not try to refactor everything into native ansible right away | 09:35 |
openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates: Fix up Newton->Ocata rabbitmq ha policy https://review.openstack.org/389309 | 09:35 |
bandini | shardy: ^ | 09:35 |
openstackgerrit | Alejandro Andreu proposed openstack/tripleo-heat-templates: WIP: Create MidoNet services https://review.openstack.org/396692 | 09:35 |
bandini | shardy: yeah I guess it will depend a bit on how much time we have for this. Likely given the short cycle we will need to resort to calling bash in our first iteration | 09:35 |
shardy | bandini: Yeah possibly - I'd suggest we do that first, get it working and see how much we can clean up before the end of the cycle | 09:36 |
openstackgerrit | Merged openstack/puppet-tripleo: Fix barbican server name to not use aodh hiera https://review.openstack.org/396434 | 09:36 |
bandini | shardy: ack, sounds reasonable! | 09:37 |
*** bogdando has quit IRC | 09:38 | |
*** ccamacho is now known as ccamacho|afk | 09:39 | |
openstackgerrit | Merged openstack/diskimage-builder: Fix ironic-python-agent image not loading vfat mod https://review.openstack.org/391535 | 09:41 |
*** rbowen has joined #tripleo | 09:44 | |
*** rbowen has left #tripleo | 09:44 | |
*** TSCHAK has joined #tripleo | 09:45 | |
*** openstackgerrit has quit IRC | 09:47 | |
*** openstackgerrit has joined #tripleo | 09:48 | |
jaosorior | shardy: reproduced the issue, seems that it's not an issue with the two things that are checked in the "reason" string https://github.com/openstack/heat/blob/master/heat/api/aws/ec2token.py#L231 | 09:49 |
jaosorior | sshnaidm: just trying to set up logging in the generation of the signature to see where it doesn't match | 09:49 |
jaosorior | shardy: the initial call to "generate" for the signature is called in heat-engine or in cfn? | 09:49 |
jaosorior | sshnaidm: it was meant for shardy, sorry | 09:50 |
*** jlinkes has quit IRC | 09:57 | |
*** jlinkes has joined #tripleo | 09:58 | |
*** rain has joined #tripleo | 09:59 | |
*** derekh has joined #tripleo | 09:59 | |
*** leanderthal has quit IRC | 09:59 | |
shardy | jaosorior: the call to generate the signature and update the server metadata happens in heat-engine | 09:59 |
*** rain is now known as leanderthal | 10:00 | |
shardy | jaosorior: then the validation (ec2tokens call to keystone) happens in the heat-api-cfn process | 10:00 |
jaosorior | the URLs don't seem the same | 10:00 |
jaosorior | http://paste.openstack.org/show/589084/ | 10:00 |
openstackgerrit | Merged openstack/diskimage-builder: elements: opensuse-minimal: Add support for building Tumbleweed images https://review.openstack.org/392003 | 10:01 |
openstackgerrit | Merged openstack/diskimage-builder: elements: runtime-ssh-host-keys: Add openssh-client mapping for SUSE https://review.openstack.org/388242 | 10:01 |
jaosorior | shardy: keystone calculates the signature with a URL that has more query parameters than what heat-engine calculates with | 10:02 |
shardy | jaosorior: Hmm, there's a bunch of additional query arguments in the heat one? | 10:02 |
shardy | deploy_stdout%22%3A%20%22os-apply-config%20deployment%207300bf95-302e-4aa7-b861-69dc5620700f%20completed%22%2C%20%22deploy_status_code%22% ... | 10:02 |
jaosorior | yeah | 10:02 |
shardy | Ok, so that explains it | 10:02 |
shardy | Oh | 10:02 |
shardy | that's the keystone one | 10:02 |
jaosorior | that's correct | 10:02 |
shardy | Ok, so the move to httpd has somehow added those, I'm not sure why | 10:03 |
shardy | that stuff should be in the post body | 10:03 |
shardy | not part of the path | 10:04 |
shardy | or query string rather | 10:04 |
*** limao has quit IRC | 10:04 | |
openstackgerrit | Merged openstack/diskimage-builder: Drop MANIFEST.in - it's not needed by pbr https://review.openstack.org/391497 | 10:08 |
openstackgerrit | Merged openstack/diskimage-builder: Don't include openstack/common in flake8 exclude list https://review.openstack.org/391498 | 10:08 |
*** udesale has quit IRC | 10:08 | |
jaosorior | shardy: strange, this is what creds_json contains, which i what cfn is passing keystone http://paste.openstack.org/show/589091/ | 10:11 |
*** numans has joined #tripleo | 10:11 | |
shardy | https://github.com/openstack/tripleo-image-elements/blob/master/elements/os-refresh-config/os-refresh-config/post-configure.d/99-refresh-completed#L32 | 10:13 |
shardy | jaosorior: Yeah, so it's adding the post body to the params, which is wrong | 10:13 |
*** panda|Zz is now known as panda | 10:14 | |
jaosorior | shardy: seems to only be an issue with cfn though. I posted a commit only moving heat-api to run over httpd and that seems fine. Anyway, still investigating | 10:16 |
shardy | jaosorior: I suspect we wouldn't break if heat-api is configured to do the same, because we don't use the query string in the same way | 10:17 |
shardy | probably the extra data would just be ignored | 10:17 |
openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates: M/N upgrade fix galera restart. https://review.openstack.org/397102 | 10:21 |
jaosorior | Can someone merge the introduction of scenario004 to our CI? https://review.openstack.org/#/c/396056/ | 10:21 |
yolanda | hi, good morning. Can i get some reviews on https://review.openstack.org/394426 and https://review.openstack.org/397075 ? | 10:21 |
*** akrivoka has joined #tripleo | 10:22 | |
*** owalsh has joined #tripleo | 10:26 | |
jaosorior | shardy: wow, so actually the auth_params contain that http://paste.openstack.org/show/589097/ https://github.com/openstack/heat/blob/master/heat/api/aws/ec2token.py#L190 | 10:30 |
*** dtantsur|afk is now known as dtantsur | 10:30 | |
*** katkapilatova has joined #tripleo | 10:31 | |
shardy | jaosorior: yep, so the API is appending the post body to the req.params | 10:32 |
shardy | https://github.com/openstack/heat/blob/master/heat/api/aws/ec2token.py#L190 | 10:32 |
shardy | which I assume means API misconfiguration? | 10:32 |
openstackgerrit | Dmitry Tantsur proposed openstack/instack-undercloud: Prevent pxe_ilo driver from guessing the boot mode to use https://review.openstack.org/397112 | 10:33 |
*** katkapilatova has left #tripleo | 10:33 | |
shardy | it's weird, the post body is being used as a key in the dict | 10:33 |
shardy | with a zero length string as a value | 10:33 |
jaosorior | shardy: well, as per the documentation. req.params should contain both the query params and the POST variables | 10:34 |
openstackgerrit | Dmitry Tantsur proposed openstack/instack-undercloud: Make inspection also respect whether iPXE is enabled https://review.openstack.org/397114 | 10:35 |
openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: Show in console which repositories were downloaded https://review.openstack.org/397115 | 10:36 |
panda | can someone confirm this is just a promotion blocker ? I see the gates passing normally https://bugs.launchpad.net/tripleo/+bug/1641571 | 10:37 |
openstack | Launchpad bug 1641571 in tripleo "CI: master jobs fail on neutron-db-manage" [High,Confirmed] | 10:37 |
dtantsur | morning owlers :) folks who know about os-net-config, could you please review https://review.openstack.org/#/c/396446/ ? this will hopefully fix a blocker for us.. | 10:37 |
*** therve_ has joined #tripleo | 10:38 | |
dtantsur | reference: https://bugzilla.redhat.com/show_bug.cgi?id=1382653 (I hope it's public) | 10:39 |
openstack | bugzilla.redhat.com bug 1382653 in rhel-osp-director "Interfaces configure IP address via DHCP and do not honour the configuration done by os-net-config" [Urgent,Assigned] - Assigned to dsneddon | 10:39 |
*** tosky has joined #tripleo | 10:40 | |
*** iranzo has quit IRC | 10:40 | |
*** iranzo has joined #tripleo | 10:40 | |
*** iranzo has joined #tripleo | 10:40 | |
shardy | jaosorior: Hrm, I see, but in this case it's just dumping the entire req.post in there | 10:40 |
shardy | which we could pop in the heat code I guess | 10:40 |
jaosorior | shardy: so... that code should only care about the GET? | 10:41 |
shardy | jaosorior: No, we only care about the query string, not the post body in this case | 10:42 |
shardy | that's where the auth related params are passed (or in headers for aws v4 signatures) | 10:42 |
shardy | https://github.com/openstack/heat/blob/master/heat/api/aws/ec2token.py#L192 | 10:42 |
shardy | I suspect adding auth_params.pop(req.body, None) there might fix it | 10:43 |
shardy | alternatively we might access req.environ['QUERY_STRING'] | 10:44 |
shardy | but ideally we don't want to have to parse the string | 10:44 |
shardy | it's odd that the old non http API doesn't do this tho | 10:44 |
jaosorior | shardy: well, you could achieve that by using req.GET http://docs.webob.org/en/stable/reference.html#query-post-variables | 10:45 |
jaosorior | shardy: GET and POST are wrongly named in Webob, and refer to the query parameters and post variables | 10:45 |
shardy | jaosorior: ah, yeah, that would probably also work | 10:45 |
*** assassin is now known as ubijtsa_afk | 10:45 | |
jaosorior | shardy: I'm doing another run to print exactly what does variables have stored | 10:46 |
*** therve has quit IRC | 10:46 | |
*** therve_ is now known as therve | 10:46 | |
*** ubijtsa_afk is now known as ubijtsa | 10:47 | |
*** ubijtsa is now known as assassin | 10:47 | |
*** ccamacho|afk is now known as ccamacho | 10:49 | |
*** bogdando has joined #tripleo | 10:52 | |
jaosorior | ccamacho: hey dude, could you check these out? https://review.openstack.org/#/c/396517/ https://review.openstack.org/#/c/396516/ https://review.openstack.org/#/c/396518/ | 10:52 |
*** owalsh_ has joined #tripleo | 10:52 | |
*** owalsh has quit IRC | 10:53 | |
*** owalsh has joined #tripleo | 10:54 | |
*** owalsh__ has joined #tripleo | 10:55 | |
*** d0ugal has quit IRC | 10:56 | |
*** owalsh_ has quit IRC | 10:57 | |
*** zoli|doctor is now known as zoli | 10:57 | |
*** owalsh_ has joined #tripleo | 10:58 | |
*** owalsh has quit IRC | 10:58 | |
jaosorior | shardy: yeah, the body is repeated for some reason, being both in the req.body and in the req.POST http://paste.openstack.org/show/589099/ | 10:59 |
*** panda is now known as panda|afk | 10:59 | |
jaosorior | shardy: I have the feeling that something is not complying with what mod_wsgi expects | 11:00 |
*** owalsh__ has quit IRC | 11:00 | |
jaosorior | shardy: anyway, incoming heat patch | 11:01 |
shardy | jaosorior: if switching to req.GET works for both wsgi and non wsgi I think a heat patch will be perfectly acceptable | 11:02 |
shardy | jaosorior: thanks for sticking with it to discover the root cause! :) | 11:02 |
*** zoli is now known as zoli|lunch | 11:04 | |
*** d0ugal has joined #tripleo | 11:05 | |
*** athomas has quit IRC | 11:08 | |
openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart: WIP: shell script to merge roles into extras https://review.openstack.org/396694 | 11:09 |
*** fultonj has joined #tripleo | 11:09 | |
jaosorior | shardy: well, this also settles it, it seems that only deploying heat-api over httpd does work https://review.openstack.org/#/c/394837/ | 11:10 |
*** owalsh_ is now known as owalsh | 11:11 | |
*** pblaho has joined #tripleo | 11:11 | |
*** athomas has joined #tripleo | 11:13 | |
openstackgerrit | Alejandro Andreu proposed openstack/tripleo-heat-templates: WIP: Create MidoNet services https://review.openstack.org/396692 | 11:26 |
*** limao has joined #tripleo | 11:28 | |
jaosorior | shardy: nice! It's going further with that patch. Deploying Step3 at the moment. | 11:29 |
openstackgerrit | Raoul Scarazzini proposed openstack/tripleo-quickstart: WIP: shell script to merge roles into extras https://review.openstack.org/396694 | 11:31 |
*** limao_ has joined #tripleo | 11:32 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-tripleoclient: Updated from global requirements https://review.openstack.org/395379 | 11:35 |
*** limao has quit IRC | 11:35 | |
*** zoli|lunch is now known as zoli | 11:37 | |
*** pkovar has joined #tripleo | 11:49 | |
*** dmacpher has joined #tripleo | 11:53 | |
*** panda|afk is now known as panda | 11:53 | |
*** jkilpatr has joined #tripleo | 12:00 | |
jaosorior | derekh: could you check this out? https://review.openstack.org/#/c/396056/2 | 12:02 |
derekh | jaosorior: looking | 12:02 |
jaosorior | shardy: well, I wasn't planning on having heat running over httpd in stable/newton; for heat, I actually don't know if people would like to have it in stable/master | 12:05 |
jaosorior | *stable/newton | 12:05 |
jaosorior | shardy: what do you think? | 12:06 |
shardy | jaosorior: I don't think we should change the stable/newton for TripleO, my comment was if the fix may make sense for the heat stable branch | 12:09 |
shardy | given that it could break operators I think backporting may be a good idea, even if we're not using it for TripleO | 12:09 |
*** shardy is now known as shardy_lunch | 12:09 | |
openstackgerrit | Noam Angel proposed openstack/diskimage-builder: simplify ARCH param for rhel/centos param can be x86_64 and amd64 https://review.openstack.org/392176 | 12:13 |
*** sudipto has quit IRC | 12:16 | |
*** dprince has joined #tripleo | 12:21 | |
*** pblaho has quit IRC | 12:24 | |
derekh | jaosorior: I've only +1'd it, I don't think I can approve as I have zero experience with the scenario tests, is it in a rush? if so I'll do a bit of digging to figure out how it all works to approve | 12:26 |
openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart-extras: Add essential repo files https://review.openstack.org/396652 | 12:27 |
jaosorior | derekh: no biggie, there is no rush just wanted to move it along. Seems not too many people are familiar with the scenario tests | 12:30 |
*** lucasagomes is now known as lucas-hungry | 12:30 | |
openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart-extras: Add essential repo files https://review.openstack.org/396652 | 12:31 |
derekh | jaosorior: ok, if it looks stuck ping me again and I'll spend some time figuring it all out | 12:31 |
*** rhallisey has joined #tripleo | 12:32 | |
*** abehl has quit IRC | 12:39 | |
*** thrash|g0ne is now known as thrash | 12:41 | |
*** milan has quit IRC | 12:46 | |
dprince | shardy_lunch, slagle: any ideas on what I'm missing here https://review.openstack.org/#/c/396171/. Surprised to see that failed CI (the new apply-config hook package seems to be installed, etc.) | 12:46 |
openstackgerrit | Katerina Pilatova proposed openstack/tripleo-validations: validation_output.py: Provided output for ansible debug module https://review.openstack.org/397177 | 12:46 |
dtantsur | folks, could you please merge one more small backport? https://review.openstack.org/#/c/397112/ | 12:46 |
dprince | shardy_lunch: also, this https://review.openstack.org/#/c/393258/ | 12:47 |
*** rodrigods has quit IRC | 12:47 | |
*** rodrigods has joined #tripleo | 12:47 | |
*** owalsh has quit IRC | 12:47 | |
slagle | dprince: that InstanceId resource was added to satisfy this: https://github.com/openstack/tripleo-image-elements/blob/master/elements/os-refresh-config/os-refresh-config/post-configure.d/99-refresh-completed#L11-L13 | 12:49 |
slagle | so does that os-apply-config command still return something after the group has been switched in the templates? | 12:49 |
*** amoralej is now known as amoralej|lunch | 12:50 | |
dprince | slagle: the new apply-config hook doesn't require 99-refresh-completed at all actually | 12:50 |
slagle | it looks like something is waiting for a signal on the Heat side | 12:51 |
slagle | that never came | 12:51 |
dprince | slagle: the two can co-exist, but the more I look at this the more it seems like switching one off and the other on is better | 12:51 |
*** owalsh has joined #tripleo | 12:52 | |
dprince | slagle: thanks for having a look. I'll revisit the logs again and see if I can rework it to make this transition happier | 12:53 |
slagle | yea it seems there might be soemthing somewhere still waiting for a signal from 99-refresh-completed | 12:54 |
dprince | slagle: oh, there is. There are lots more os-apply-config's in the tree still | 12:54 |
dprince | slagle: perhaps I should go after them all at once (like I'm doing for hiera) | 12:54 |
*** ayoung has quit IRC | 12:54 | |
dprince | slagle: I did that before but the patch ends up being really large | 12:55 |
slagle | ok, in that case i'd guess that $ID is coming up null now in that script, so it bails out early | 12:55 |
* dprince wonders if we are missing a depends on or something | 12:56 | |
*** bfournie has quit IRC | 12:57 | |
*** sudipto has joined #tripleo | 13:02 | |
*** sudipto has quit IRC | 13:03 | |
*** pgadiya has quit IRC | 13:04 | |
*** jayg|g0n3 is now known as jayg | 13:04 | |
sshnaidm | amoralej|lunch, hi, do you know if last failures in periodic jobs with neutron-db-manage are related to bug you opened about cisco plugin? | 13:06 |
sshnaidm | amoralej|lunch, I don't see this trace there: http://logs.openstack.org/periodic/periodic-tripleo-ci-centos-7-ovb-ha/e15e49b/logs/postci.txt.gz#_2016-11-14_07_39_19_000 | 13:06 |
*** zaneb has joined #tripleo | 13:07 | |
panda | sshnaidm: I think cisco plugin is involved | 13:10 |
*** ccamacho is now known as ccamacho|lunch | 13:11 | |
sshnaidm | panda, I too, but want to be sure it is | 13:11 |
panda | sshnaidm: where's the bug ? | 13:12 |
sshnaidm | panda, https://bugs.launchpad.net/networking-cisco/+bug/1641311 | 13:13 |
openstack | Launchpad bug 1641311 in networking-cisco "neutron-db-manage fails after https://review.openstack.org/#/c/394201/" [High,Confirmed] | 13:13 |
sshnaidm | panda, 24 in https://etherpad.openstack.org/p/tripleo-ci-status | 13:13 |
sshnaidm | but I don't see the trace like in bug, that's suspicious | 13:13 |
panda | sshnaidm: yep, it's the same | 13:14 |
*** jtomasek has quit IRC | 13:14 | |
panda | sshnaidm: search for ATTR | 13:15 |
panda | sshnaidm: it's there | 13:15 |
sshnaidm | panda, you're right, missed it.. | 13:15 |
panda | ok, what do I do with https://bugs.launchpad.net/tripleo/+bug/1641571 then ? duplicate ? invalid ? | 13:16 |
openstack | Launchpad bug 1641571 in tripleo "CI: master jobs fail on neutron-db-manage" [High,Confirmed] | 13:16 |
sshnaidm | it's in stdout | 13:16 |
sshnaidm | panda, duplicate I think | 13:16 |
openstackgerrit | Brad P. Crochet proposed openstack/tripleo-docs: Switch docs to use yaml with overcloud image build https://review.openstack.org/337669 | 13:18 |
openstackgerrit | Katerina Pilatova proposed openstack/tripleo-validations: validation_output.py: Provided output for ansible debug module https://review.openstack.org/397177 | 13:18 |
openstackgerrit | Merged openstack/tripleo-heat-templates: Use default Sensu redact https://review.openstack.org/394847 | 13:18 |
*** fultonj has quit IRC | 13:22 | |
*** fultonj has joined #tripleo | 13:22 | |
*** jprovazn has quit IRC | 13:23 | |
*** bfournie has joined #tripleo | 13:27 | |
*** amoralej|lunch is now known as amoralej | 13:28 | |
amoralej | sshnaid, yes it's the same one | 13:29 |
*** rbrady-afk has quit IRC | 13:29 | |
jaosorior | dprince, slagle, shardy_lunch: Do you guys know what's calling this? http://paste.openstack.org/show/589121/ | 13:30 |
jaosorior | what makes that POST over there | 13:30 |
jaosorior | I see no trace of a "Signalling os-apply-config deployment" in the os-collect-config code-base | 13:31 |
*** shardy_lunch is now known as shardy | 13:31 | |
shardy | jaosorior: https://github.com/openstack/tripleo-image-elements/blob/master/elements/os-refresh-config/os-refresh-config/post-configure.d/99-refresh-completed#L79 | 13:32 |
*** rlandy has joined #tripleo | 13:35 | |
jaosorior | shardy: any specific reason that call is a POST? | 13:36 |
*** jeckersb_gone is now known as jeckersb | 13:39 | |
openstackgerrit | Athlan-Guyot sofer proposed openstack/tripleo-docs: Fix typo. https://review.openstack.org/397190 | 13:41 |
*** lucas-hungry is now known as lucasagomes | 13:43 | |
rook-out | dtantsur: hey - thanks for the feedback on the commit (396667) let me know if I am missing anything else. | 13:46 |
*** ccamacho|lunch is now known as ccamacho | 13:47 | |
*** jpena is now known as jpena|lunch | 13:48 | |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-image-elements: Set correct content-type for o-r-c curl calls https://review.openstack.org/397195 | 13:48 |
jaosorior | shardy: ^^ | 13:48 |
*** dsariel has quit IRC | 13:48 | |
*** jcoufal has joined #tripleo | 13:48 | |
jaosorior | how can I test that? | 13:48 |
jaosorior | in the overcloud images | 13:49 |
shardy | jaosorior: I don't recall exactly why we used POST - IIRC waitconditions use PUT, but after a signal their state is absolute, whereas you can signal e.g a scaling policy resource multiple times thus it's implicitly not idempotent | 13:49 |
dtantsur | rook-out, thanks, will get to it later today | 13:49 |
rook-out | dtantsur: no rush, I am still on leave. | 13:51 |
jaosorior | shardy: well, that CFN endpoint is expecting a GET :/ and completely discards the binary data that's set in that curl request | 13:51 |
shardy | jaosorior: I'd use virt-copy-in to copy the new 99-refresh-completed into your overcloud-full.qcow then update the image in glance | 13:51 |
jaosorior | shardy: sure... but where does that 99-refresh-completed end up? | 13:52 |
shardy | https://github.com/openstack/heat/blob/master/heat/api/cfn/v1/__init__.py#L79 | 13:52 |
shardy | jaosorior: when we wrote it (years and years ago FWIW) it expected a POST | 13:52 |
dtantsur | TIL you can schedule to a specific Ironic node by using availability zones: http://paste.openstack.org/show/550163/ | 13:53 |
*** ecerquei_ has joined #tripleo | 13:55 | |
shardy | jaosorior: /usr/libexec/os-refresh-config/post-configure.d/99-refresh-completed | 13:55 |
*** tzumainn has joined #tripleo | 13:56 | |
openstackgerrit | Arx Cruz proposed openstack/tripleo-quickstart: Enable cinder-backup service on minimal pacemaker https://review.openstack.org/396587 | 13:56 |
*** ecerquei__ has joined #tripleo | 13:59 | |
*** ohamada has quit IRC | 13:59 | |
*** ohamada has joined #tripleo | 14:00 | |
*** jprovazn has joined #tripleo | 14:00 | |
*** jlinkes_ has joined #tripleo | 14:00 | |
*** milan has joined #tripleo | 14:02 | |
*** ecerquei_ has quit IRC | 14:02 | |
*** jlinkes has quit IRC | 14:04 | |
*** tiswanso has joined #tripleo | 14:07 | |
*** oshvartz has quit IRC | 14:08 | |
matbu | chem: marios hey guys, one question, during the upgrade with a ceph, did you remove ceph-hammer for ceph-jewel ? | 14:10 |
matbu | (i mean manually) | 14:10 |
openstackgerrit | Brent Eagles proposed openstack/os-net-config: Add support for enabling hotplug on interfaces https://review.openstack.org/394660 | 14:10 |
jaosorior | shardy: ok, so it did do the trick, setting the content-type | 14:10 |
jaosorior | shardy: I'll abandon the heat commit | 14:11 |
chem | matbu: hum ... nope it should be done automatically ... | 14:11 |
shardy | jaosorior: ack, sounds good! | 14:11 |
*** ElCoyote_ has quit IRC | 14:11 | |
jaosorior | ccamacho: hey dude, thanks for the commit, though the three commits go in a row, so the workflow you did isn't gonna merge it unless the other two merge as well :/ (wasn't on purpose, kinda went that way) | 14:11 |
openstackgerrit | Lukas Bezdicka proposed openstack/instack-undercloud: Update puppet dependancies before running puppet https://review.openstack.org/328264 | 14:12 |
matbu | chem: hm interesting, it looks like not the case on upstream | 14:12 |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-image-elements: Set correct content-type for o-r-c curl calls https://review.openstack.org/397195 | 14:12 |
marios | matbu: my dev/test doesn't have room for ceph ... also gfidente is the person to ask afor that and i believe it is already handled as chem says | 14:12 |
openstackgerrit | Lukas Bezdicka proposed openstack/python-tripleoclient: Don't update whole system on undercloud upgrade https://review.openstack.org/327176 | 14:13 |
matbu | marios: right, i will ask to gfidente when come up online | 14:13 |
*** [1]cdearborn has joined #tripleo | 14:13 | |
matbu | marios: dev/test is rhos ? | 14:13 |
*** lblanchard has joined #tripleo | 14:14 | |
*** [1]cdearborn has quit IRC | 14:14 | |
chem | matbu: the problem is you ceph env is not updated ? | 14:14 |
chem | matbu: the osd, or on the controller ? | 14:14 |
openstackgerrit | Lukas Bezdicka proposed openstack/python-tripleoclient: Remove workaround for openstack-nova-api https://review.openstack.org/397205 | 14:14 |
marios | matbu: on i meant my dev environment (is virt based, instack undercloud 3 control compute setup) | 14:15 |
*** sudipto has joined #tripleo | 14:15 | |
matbu | chem: on the controller there is no systemd for starting ceph | 14:15 |
matbu | chem: puppet is crying about that | 14:15 |
*** Goneri has joined #tripleo | 14:16 | |
matbu | chem: so i guess it's because ceph is not updated | 14:16 |
openstackgerrit | Lukas Bezdicka proposed openstack/instack-undercloud: Update puppet dependancies before running puppet https://review.openstack.org/328264 | 14:16 |
*** pcaruana has quit IRC | 14:16 | |
*** spredzy has quit IRC | 14:17 | |
*** spredzy has joined #tripleo | 14:17 | |
*** noslzzp has joined #tripleo | 14:17 | |
*** pkovar has quit IRC | 14:18 | |
openstackgerrit | Brent Eagles proposed openstack/os-net-config: Add support for name replacement in OVS_EXTRA https://review.openstack.org/395795 | 14:21 |
*** udesale has joined #tripleo | 14:21 | |
*** rbrady has joined #tripleo | 14:23 | |
*** florianf has quit IRC | 14:23 | |
*** dsariel has joined #tripleo | 14:25 | |
openstackgerrit | Alejandro Andreu proposed openstack/tripleo-heat-templates: WIP: Create MidoNet services https://review.openstack.org/396692 | 14:26 |
*** sudipto has quit IRC | 14:27 | |
ccamacho | jaosorior, man sorry for not updating all the patches, not feeling good, yahp tested locally the 3 patches | 14:28 |
jaosorior | ah! | 14:29 |
jaosorior | no biggie dude | 14:29 |
jaosorior | thanks for checking them out | 14:29 |
jaosorior | rest duuuude | 14:29 |
ccamacho | I have borrowed a box with 64GB so in home I have upstream TripleO and then a virtual env with downstream :P | 14:29 |
*** morazi has joined #tripleo | 14:31 | |
openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Move the overcloudrc generation from tripleoclient to a Mistral action https://review.openstack.org/397211 | 14:32 |
jaosorior | shardy: nevermind, that didn't help, some stuff is passing but there are still authentication errors | 14:32 |
*** zoli is now known as zoli|brb | 14:32 | |
openstackgerrit | Lukas Bezdicka proposed openstack/python-tripleoclient: Don't update whole system on undercloud upgrade https://review.openstack.org/327176 | 14:34 |
b00tcat | when implementing new, custom services in puppet-tripleo - how do you debug this code? I can't build a new base image everytime :-/ (since AFAIK the puppet-tripleo module is already on the image) | 14:35 |
*** dmarlin is now known as dmarlin_ | 14:35 | |
*** Tristitia has joined #tripleo | 14:35 | |
openstackgerrit | Dan Prince proposed openstack/tripleo-puppet-elements: Disable old hiera.yaml o-a-c script https://review.openstack.org/392615 | 14:36 |
*** ayoung has joined #tripleo | 14:37 | |
openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Horizon service cleanups for hiera json hook https://review.openstack.org/393258 | 14:37 |
openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Hiera optimization: use a new hiera hook https://review.openstack.org/384757 | 14:37 |
openstackgerrit | Saverio Proto proposed openstack/diskimage-builder: debian: install dialog package https://review.openstack.org/397218 | 14:37 |
*** florianf has joined #tripleo | 14:38 | |
*** dsariel has quit IRC | 14:39 | |
*** isq has quit IRC | 14:39 | |
*** jaosorior has quit IRC | 14:39 | |
*** isq has joined #tripleo | 14:40 | |
*** jaosorior has joined #tripleo | 14:40 | |
*** cdearborn has joined #tripleo | 14:41 | |
*** noslzzp has quit IRC | 14:41 | |
openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart: WIP: shell script to merge roles into extras https://review.openstack.org/396694 | 14:42 |
*** noslzzp has joined #tripleo | 14:42 | |
openstackgerrit | Tim Rozet proposed openstack/tripleo-heat-templates: Fixes missing OVS Firewall config with OpenDaylight https://review.openstack.org/397228 | 14:44 |
*** mhenkel has joined #tripleo | 14:45 | |
jaosorior | shardy: any idea where else could we be doing calls without content-type? I keep seeing this http://paste.openstack.org/show/589131/ | 14:46 |
jaosorior | some calls actually started working, but it's still getting stuck | 14:46 |
shardy | https://github.com/openstack/tripleo-heat-templates/blob/master/extraconfig/tasks/yum_update.yaml#L11 | 14:47 |
shardy | jaosorior: that's coming from the script run by that config | 14:48 |
shardy | so it must be from the heat-config-notify script run after heat-config-script ends | 14:48 |
mhenkel | one question on the layout of hiera variable names of services. E.g. If I have a service definition like this: | 14:48 |
mhenkel | OS::TripleO::Services::ContrailAnalyticsDatabase: puppet/services/network/contrail-analytics-database.yaml | 14:48 |
shardy | jaosorior: https://github.com/openstack/heat-templates/blob/master/hot/software-config/elements/heat-config/bin/heat-config-notify | 14:48 |
*** pkovar has joined #tripleo | 14:49 | |
therve | Weird that we're setting the wrong content type on purpose | 14:49 |
mhenkel | will the the nodes_ip variable in the hiera files be: contrail_analytics_database_node_ips ? | 14:49 |
mhenkel | or contrail_analyticsdatabase_node_ips? | 14:49 |
*** chem has quit IRC | 14:49 | |
shardy | mhenkel: It depends on the service_name defined in puppet/services/network/contrail-analytics-database.yaml | 14:50 |
mhenkel | ah ok | 14:50 |
*** zoli|brb is now known as zoli | 14:51 | |
shardy | https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/services/network/contrail-analytics.yaml#L75 | 14:51 |
shardy | mhenkel: so it'll be contrail_analytics_node_ips | 14:51 |
shardy | in that case | 14:51 |
mhenkel | service_name: contrail_analytics_database | 14:51 |
shardy | ah, so contrail_analytics_database_node_ips in that case | 14:51 |
mhenkel | shardy: yes, I will have to add a separate role | 14:52 |
*** pblaho has joined #tripleo | 14:52 | |
mhenkel | shardy: for the analytics database | 14:52 |
mhenkel | now that everything works with my local templates | 14:52 |
shardy | mhenkel: ack, that should work fine provided the service_name is unique, and you add an entry in the ServiceNetMap or ServiceNetMapDefaults for the new service | 14:52 |
*** dsariel has joined #tripleo | 14:52 | |
mhenkel | btw. the ntp issues I had were because my undercloud was outdated | 14:53 |
*** jpena|lunch is now known as jpena | 14:53 | |
shardy | mhenkel: ah, good to hear the update fixed it :) | 14:53 |
*** zoli is now known as zoliXXL | 14:53 | |
mhenkel | shardy: yeah, it was a re-installation rather than an upgrade | 14:53 |
mhenkel | we are testing on newton-testing now | 14:54 |
jaosorior | therve, shardy I see. I'll try to add the content-type there and test it on my deployment | 14:55 |
jaosorior | patch coming in a bit | 14:55 |
therve | jaosorior, Still, there is something we're missing. The issue is in the apache conf | 14:55 |
*** nyechiel has quit IRC | 14:56 | |
jaosorior | therve: well, it is a fact that those content-type settings were wrong for the requests | 14:56 |
therve | jaosorior, Maybe, but they work without apache | 14:56 |
*** abehl has joined #tripleo | 14:57 | |
jaosorior | therve: alright, if you find the missing configuration let me know and I can test it out. Meanwhile I'll see if this helps. | 14:58 |
therve | jaosorior, Sorry I didn't want to be a dick about it | 14:59 |
*** chem has joined #tripleo | 15:00 | |
*** dsariel has quit IRC | 15:02 | |
mhenkel | another quick question: is there any elegant way in tripleo to define different internalApi networks on a per VM basis | 15:04 |
openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart: Add libffi devel package for the upstream gates https://review.openstack.org/397234 | 15:04 |
mhenkel | e.g. in case my compute nodes are distributed across different subnets | 15:04 |
mhenkel | so not only internalApi but also ControlPlane | 15:04 |
openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart: Add libffi devel package for the upstream gates https://review.openstack.org/397234 | 15:09 |
openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Deployed server: switch to apply-config hook https://review.openstack.org/396171 | 15:09 |
*** chem has quit IRC | 15:09 | |
jaosorior | therve, shardy: where does hot/software-config/elements/heat-config/bin/heat-config-notify end up in the resulting image? | 15:11 |
*** chem has joined #tripleo | 15:12 | |
jaosorior | therve: would it be that with eventlet it just assumes the content-type to be application/json? | 15:12 |
therve | jaosorior, Possibly yes. Or something different, at least | 15:13 |
*** links has quit IRC | 15:13 | |
openstackgerrit | Merged openstack/tripleo-quickstart: Add libffi devel package for the upstream gates https://review.openstack.org/397234 | 15:14 |
shardy | jaosorior: find shows it in /usr/bin | 15:14 |
jaosorior | shardy: I sshed into the running node and it doesn't seem to be there | 15:15 |
openstackgerrit | Merged openstack/instack-undercloud: Configure keystone ec2 conf for heat behind haproxy https://review.openstack.org/396516 | 15:15 |
openstackgerrit | Merged openstack/instack-undercloud: Move glance_api_servers conf for nova to hiera https://review.openstack.org/396517 | 15:16 |
openstackgerrit | Merged openstack/instack-undercloud: Pass keystone configuration via hiera https://review.openstack.org/396518 | 15:16 |
*** coolsvap has quit IRC | 15:17 | |
openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart-extras: Add essential repo files https://review.openstack.org/396652 | 15:17 |
shardy | jaosorior: http://paste.openstack.org/show/589135/ | 15:17 |
shardy | unless you're running a really old image it should be there (it was moved from a dib element to a package some time ago) | 15:17 |
*** dsariel has joined #tripleo | 15:17 | |
*** pradk has joined #tripleo | 15:18 | |
*** sudipto has joined #tripleo | 15:18 | |
jaosorior | shardy: seems to be in /usr/local/bin | 15:18 |
shardy | jaosorior: Ok I suspect you're running a pretty old image then? | 15:19 |
jaosorior | shardy: same images as the ones that are being built by tripleo-quickstart | 15:19 |
jaosorior | shardy: deployed master just today | 15:19 |
jaosorior | trown: ^^ | 15:20 |
shardy | jaosorior: Ok, sounds like something funky is happening with those images then, as it should be using the python-heat-agent package | 15:20 |
jaosorior | it isn't for some reason | 15:20 |
shardy | we added it to tripleoclient, but I'm not completely sure how those images get built (for quickstart) | 15:20 |
jaosorior | adarazs: ^^ | 15:21 |
jaosorior | or larsks ^^ | 15:21 |
* adarazs reads back | 15:21 | |
*** jtomasek has joined #tripleo | 15:21 | |
trown | they get built using the yaml in tripleo-common | 15:22 |
therve | jaosorior, Bingo. The content-type comes from eventlet. It sets it to text/plain by default | 15:23 |
jaosorior | god dammit eventlet! | 15:24 |
rook-out | heya is there a issue upstream? -- gate-puppet-openstack-integration-4-scenario002-tempest-centos-7 | 15:24 |
jaosorior | rook-out: might want to ask in #puppet-openstack | 15:24 |
*** Goneri has quit IRC | 15:25 | |
*** dhill_ has joined #tripleo | 15:27 | |
ansiwen | mwhahaha: I'll add the releasenotes, but note, that I couldn't find another release notes for such an paramter... so it doesn't seem to be standard yet ;-) | 15:27 |
*** masco has quit IRC | 15:27 | |
openstackgerrit | Ben Nemec proposed openstack/tripleo-docs: Add Newton repo details https://review.openstack.org/396736 | 15:27 |
*** dsariel has quit IRC | 15:28 | |
mwhahaha | ansiwen: ok we should get in the habbit of announcing new things via the release notes. i thought we had started that. I know we do for deprecations | 15:28 |
ansiwen | mwhahaha: yes, I see a couple of deprecations | 15:28 |
openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart-extras: Add essential repo files https://review.openstack.org/396652 | 15:29 |
openstackgerrit | Pradeep Kilambi proposed openstack/instack-undercloud: Add gnocchi support on undercloud https://review.openstack.org/392992 | 15:31 |
jaosorior | pradk: I noticed you submitted panko for the overcloud, is that running over httpd? | 15:32 |
*** numans has quit IRC | 15:33 | |
trozet | trown: hi | 15:34 |
trown | shardy: jaosorior... actually just realized the image build role in RDO is still using YAML generated from that role and not directly from tripleo-common https://github.com/redhat-openstack/ansible-role-tripleo-image-build/blob/master/templates/dib-manifest-default.yaml.j2 | 15:34 |
trozet | trown: did the images in http://buildlogs.centos.org/centos/7/cloud/x86_64/tripleo_images/newton/delorean move to CentOS 7.3 yet? | 15:34 |
jaosorior | therve: well, we could try to force the content type in apache using ForceType application/json | 15:34 |
jaosorior | but I'm not sure if that's the way to go | 15:34 |
trown | shardy: jaosorior, it is high on the priority list to stop using that role for image building all together (it is not getting ported to tripleo-quickstart-extras repo), but I will patch the current role for the short term | 15:35 |
*** liverpooler has joined #tripleo | 15:35 | |
*** rajinir has joined #tripleo | 15:35 | |
therve | jaosorior, Frankly, it would make sense from a compatibility point of vien | 15:36 |
therve | view | 15:36 |
trown | trozet: depends if http://cloud.centos.org/centos/7/images/CentOS-7-x86_64-GenericCloud.qcow2 points to 7.3 | 15:36 |
trown | trozet: not sure | 15:36 |
shardy | trown: ack, thanks for the info | 15:36 |
trown | trozet: we could ask on #centos-devel | 15:37 |
trozet | trown: ok for NFV/SDN we need 7.3 | 15:38 |
trozet | trown: i can download the generic and see | 15:38 |
*** Goneri has joined #tripleo | 15:38 | |
therve | jaosorior, Although I guess your fix in heat may be a valuable approach. I'm worried about the other places where content type can influence things | 15:38 |
trozet | trown: it's still 7.2.1511 | 15:40 |
trown | trozet: from #centos-devel [10:38:59] <Evolution> trown: it is not. | 15:40 |
trown | [10:39:09] <Evolution> 7.3 is not out officially yet | 15:40 |
*** udesale has quit IRC | 15:40 | |
*** dsariel has joined #tripleo | 15:41 | |
trozet | trown: crap | 15:42 |
trozet | trown: did they mention when it will be released? | 15:42 |
jaosorior | therve: looking on how to do it... but ForceType doesn't seem to promising for this purpose http://httpd.apache.org/docs/2.2/mod/core.html#forcetype since it seems quite invasive | 15:42 |
therve | jaosorior, Ah, it's not when not present | 15:43 |
jaosorior | therve, shardy: But FWIW, my deployment already got to the puppet steps with both the fix to os-refresh-config and heat-templates | 15:44 |
therve | jaosorior, Also it's for responses it seems | 15:44 |
trown | trozet: nope... has RHEL7.3 even GA'd? CentOS is usually a couple weeks after | 15:44 |
*** abregman is now known as abregman|mtg | 15:44 | |
trozet | trown: yeah RHEL 7.3 came out Nov 2nd i think | 15:45 |
lucasagomes | hey folks, mind taking a look at this patch here: https://review.openstack.org/#/c/396086/ ? It's fix a problem importing Cisco hardware with tripleo | 15:46 |
trown | trozet: cool, so likely this week or next then :) | 15:46 |
trozet | trown: alright so should just automatically update when they update the link - good | 15:46 |
openstackgerrit | Arx Cruz proposed openstack/tripleo-quickstart: Enable cinder-backup service on minimal pacemaker https://review.openstack.org/396587 | 15:47 |
jaosorior | therve: true, seems to be for responses, missed that part | 15:47 |
trown | lucasagomes: we will likely want to backport that to newton ya? | 15:48 |
jaosorior | shardy, therve meanwhile, could you review these? https://review.openstack.org/#/c/397232/1 https://review.openstack.org/#/c/397195/ | 15:49 |
lucasagomes | trown, yes! | 15:50 |
*** dtantsur is now known as dtantsur|brb | 15:50 | |
*** aufi has quit IRC | 15:50 | |
lucasagomes | trown, I will propose a patch once that one gets merged | 15:50 |
trown | lucasagomes: cool, added newton-backport-potential to bug | 15:51 |
*** chandankumar has quit IRC | 15:51 | |
lucasagomes | trown, thanks | 15:51 |
jaosorior | ayoung: do you happen to know if it's possible to set a default content-type for the incoming requests for an apache vhost? | 15:52 |
openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: WIP prototyping composable upgrades with Heat+Ansible https://review.openstack.org/393448 | 15:52 |
openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: Show in console which repositories were downloaded https://review.openstack.org/397115 | 15:55 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/tripleo-common: Replace ucs_hostname with ucs_address https://review.openstack.org/397266 | 15:55 |
lucasagomes | trown, ^ (the backport) | 15:55 |
openstackgerrit | Dan Radez proposed openstack/puppet-tripleo: OVN agent typo correction https://review.openstack.org/397267 | 15:55 |
social | marios: re https://review.openstack.org/#/c/327176 we really should leave the update work for puppet code | 15:57 |
openstackgerrit | Pradeep Kilambi proposed openstack/puppet-tripleo: Add panko service support https://review.openstack.org/396435 | 15:57 |
*** yamahata has quit IRC | 15:57 | |
*** yamahata has joined #tripleo | 15:58 | |
jaosorior | pradk: thanks for the update, that LGTM | 15:58 |
ayoung | jaosorior, I think it is possible... | 15:58 |
pradk | jaosorior, cool thx for review .. i dint enable tls stuff yet, we can do that as a following patch? | 15:59 |
pradk | if needed | 15:59 |
jaosorior | pradk: it will be needed ;) but sure, it can come subsequently | 15:59 |
pradk | jaosorior, ok I'll look into it today | 16:00 |
jaosorior | ayoung: any idea how? therve and me have been looking into it and haven't figured it out yet. | 16:00 |
ayoung | jaosorior, request or response content type? | 16:00 |
*** limao_ has quit IRC | 16:01 | |
therve | ayoung, request | 16:01 |
ayoung | therve, I think the default is {X}HTML but...isn't it up to the application what it accepts? | 16:01 |
therve | ayoung, It is. But eventlet wsgi container uses text/plain by default | 16:02 |
ayoung | -H "Content-Type:application/json" is the client telling the application "I am sending you JSON" | 16:02 |
jaosorior | ayoung: content: I'm trying to get heat to run over httpd. and started running into issues related to the content-type | 16:02 |
ayoung | which wsgi app are we talking about here? | 16:02 |
therve | Heat | 16:02 |
ayoung | HEat. | 16:02 |
jaosorior | *context | 16:02 |
ayoung | Hmmm. We handle that in Keystone deeper in the stack | 16:03 |
openstackgerrit | Pradeep Kilambi proposed openstack/tripleo-heat-templates: Add panko api support to service templates https://review.openstack.org/396439 | 16:03 |
ayoung | it might be heat specific, | 16:03 |
jaosorior | so, at the same time I'm trying to fix the instances where the lack of content type affect tripleo | 16:03 |
jaosorior | but we thought having a default would make the transition smoother | 16:04 |
therve | Yeah we need a fix in heat itself | 16:04 |
ayoung | Let me pull out the Keystone code...one sec | 16:06 |
ayoung | I know how I rewrote it...but that got -2ed | 16:06 |
ayoung | jaosorior, http://git.openstack.org/cgit/openstack/keystone/tree/keystone/middleware/core.py#n76 | 16:06 |
ayoung | JsonBodyMiddleware | 16:06 |
jaosorior | Is ee | 16:07 |
jaosorior | * I see | 16:07 |
ayoung | Icy | 16:07 |
jaosorior | yep | 16:07 |
jaosorior | and also snowy | 16:07 |
* jaosorior such is Finland right now | 16:07 | |
jaosorior | therve: could you look into the fix for heat? | 16:10 |
therve | jaosorior, Sure! | 16:11 |
jaosorior | therve: thanks dude | 16:11 |
therve | jaosorior, No problem, thanks for you patience. I'm glad we found the root cause, though it was painful :) | 16:11 |
jaosorior | slagle: could you take a look at https://review.openstack.org/#/c/396056/ ? | 16:12 |
*** ealcaniz has quit IRC | 16:14 | |
*** ramishra has quit IRC | 16:18 | |
*** tremble has quit IRC | 16:19 | |
openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: WIP prototyping composable upgrades with Heat+Ansible https://review.openstack.org/393448 | 16:20 |
*** ramishra has joined #tripleo | 16:20 | |
*** ealcaniz has joined #tripleo | 16:21 | |
*** phpcodemonkey has joined #tripleo | 16:22 | |
*** fultonj has quit IRC | 16:22 | |
*** fultonj has joined #tripleo | 16:23 | |
*** Goneri has quit IRC | 16:24 | |
*** absubram has joined #tripleo | 16:24 | |
*** pcaruana has joined #tripleo | 16:24 | |
*** abregman|mtg is now known as abregman | 16:27 | |
*** dtantsur|brb is now known as dtantsur | 16:27 | |
*** mhenkel has quit IRC | 16:29 | |
*** milan has quit IRC | 16:30 | |
*** liverpooler has quit IRC | 16:30 | |
openstackgerrit | Lukas Bezdicka proposed openstack/python-tripleoclient: Stop openvswitch service before running yum update https://review.openstack.org/397283 | 16:32 |
*** jlinkes_ has quit IRC | 16:33 | |
*** mcornea has quit IRC | 16:33 | |
*** ebarrera has quit IRC | 16:34 | |
*** phpcodemonkey has quit IRC | 16:36 | |
*** Goneri has joined #tripleo | 16:36 | |
*** fragatina has quit IRC | 16:37 | |
openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Always use cached images in updates job https://review.openstack.org/386713 | 16:38 |
openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Temporarily disable ceph in the updates job https://review.openstack.org/397288 | 16:38 |
openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Always use cached images in updates job https://review.openstack.org/386713 | 16:40 |
openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Temporarily disable ceph in the updates job https://review.openstack.org/397288 | 16:40 |
beekneemech | Now with 100% more bug reference! | 16:40 |
jaosorior | beekneemech: thanks for the reviews dude | 16:40 |
*** beekneemech is now known as bnemec | 16:40 | |
bnemec | jaosorior: Seemed only fair since I was the one who wanted to disable it in the first place. :-) | 16:40 |
*** weshay is now known as weshay_brb | 16:46 | |
ansiwen | every THT puppet service seems to have a "Base" service... is that a requirement or good practice? | 16:48 |
*** stendulker has joined #tripleo | 16:49 | |
jaosorior | ansiwen: depends on your needs | 16:50 |
ansiwen | that is, if I have a simple service (EC2API), do I need a Ec2ApiBase as well? | 16:50 |
jaosorior | probably not | 16:51 |
*** shardy has quit IRC | 16:51 | |
ansiwen | is there another example of such a simple service? I'm a bit overwhelmed by the pretty large yaml files. I have no idea what I need and what not. | 16:52 |
jaosorior | ansiwen: base files are usually used if there's some configurations you need to share | 16:52 |
ansiwen | well, they are not really large, but a kind of sceleton would be nice | 16:53 |
jaosorior | ansiwen: for instance, heat-base.yaml is used by heat-api.yaml, heat-engine.yaml and heat-cfn.yaml | 16:53 |
bnemec | Having a base module for something that only has one service probably doesn't make sense. | 16:54 |
bnemec | ansiwen: Take a look at barbican-api. No base file that I see. | 16:55 |
jaosorior | ansiwen: but yeah, if it's a small service you might not need it | 16:55 |
bnemec | Also horizon, keepalived, keystone... | 16:55 |
*** michapma_alt has quit IRC | 16:55 | |
ansiwen | bnemec, jaosorior, thanks guys! | 16:56 |
ansiwen | probably I should also read some THT documentation, to understand what this yaml file tells me :-) | 16:56 |
ansiwen | I was only dealing with puppet for weeks now, completely list in THT :-/ | 16:57 |
ansiwen | ...lost... | 16:57 |
*** aufi has joined #tripleo | 16:57 | |
jaosorior | ansiwen: understandable, it can be quite overwhelming at first :/ | 16:59 |
openstackgerrit | Merged openstack/tripleo-image-elements: Include lsb_release in the hosts element since it's used there. https://review.openstack.org/396759 | 16:59 |
openstackgerrit | Ana Krivokapic proposed openstack/tripleo-specs: GUI: Import/Export Deployment Plan https://review.openstack.org/397296 | 16:59 |
ansiwen | for instance, can I freely chose a service_name? what default value do I have to use for monitoring_subscription... and so on... | 16:59 |
ansiwen | I'm naivly thinking: meh, yaml file, I will understand that just by looking at it... but not the case here >_< | 17:02 |
openstackgerrit | Merged openstack/tripleo-heat-templates: Fix typo in Keystone Sensu subscription https://review.openstack.org/394824 | 17:02 |
*** ayoung has quit IRC | 17:02 | |
bnemec | ansiwen: You probably want to read through http://docs.openstack.org/developer/tripleo-docs/developer/tht_walkthrough/tht_walkthrough.html | 17:04 |
bnemec | ansiwen: And if you find anything missing or unclear there, please submit improvements or bugs to request further explanation. | 17:05 |
*** jaosorior has quit IRC | 17:05 | |
ansiwen | bnemec: I looked at that a few months ago and couldn't follow. but probably I'm ready for it now. :-) | 17:06 |
ansiwen | and at that time it was just a gerrit change | 17:06 |
ansiwen | so probably it's in better shape now | 17:06 |
ansiwen | I'll give it a try | 17:06 |
ansiwen | thanks | 17:06 |
ansiwen | bnemec: ^ | 17:06 |
bnemec | ansiwen: Yeah, I think the goal of that doc is to help people in exactly your situation, so if it needs further tweaking then that's good feedback. | 17:07 |
*** abregman has quit IRC | 17:08 | |
*** jaosorior has joined #tripleo | 17:08 | |
*** rbrady is now known as rbrady-afk | 17:08 | |
*** zoliXXL is now known as zoli|gone | 17:11 | |
*** fragatina has joined #tripleo | 17:12 | |
*** milan has joined #tripleo | 17:12 | |
*** nyechiel has joined #tripleo | 17:17 | |
*** ayoung has joined #tripleo | 17:17 | |
*** ealcaniz has quit IRC | 17:17 | |
*** panda is now known as panda|bbl | 17:18 | |
openstackgerrit | Merged openstack/diskimage-builder: Add element for setting sysctl values https://review.openstack.org/356582 | 17:19 |
*** chandankumar has joined #tripleo | 17:22 | |
*** dmarlin has joined #tripleo | 17:23 | |
openstackgerrit | Brad P. Crochet proposed openstack/python-tripleoclient: Remove code deprecated since Liberty https://review.openstack.org/397312 | 17:24 |
*** bana_k has joined #tripleo | 17:27 | |
*** weshay_brb is now known as weshay | 17:29 | |
*** ebarrera has joined #tripleo | 17:29 | |
*** dsariel has quit IRC | 17:30 | |
*** ohamada has quit IRC | 17:30 | |
openstackgerrit | Tomas Sedovic proposed openstack/tripleo-validations: Fix the DHCP validations https://review.openstack.org/397314 | 17:31 |
*** iranzo has quit IRC | 17:34 | |
*** jpich has quit IRC | 17:36 | |
*** lucasagomes is now known as lucas-afk | 17:42 | |
*** athomas has quit IRC | 17:42 | |
openstackgerrit | Pradeep Kilambi proposed openstack/instack-undercloud: Add gnocchi support on undercloud https://review.openstack.org/392992 | 17:48 |
bnemec | Gerrit and launchpad both hate me today. | 17:49 |
*** yamahata has quit IRC | 17:50 | |
*** ctayal has joined #tripleo | 17:50 | |
*** jaosorior has quit IRC | 17:51 | |
*** ayoung has quit IRC | 17:55 | |
*** trown is now known as trown|lunch | 17:56 | |
openstackgerrit | Merged openstack-infra/tripleo-ci: Introduce scenario004 https://review.openstack.org/396056 | 17:58 |
*** stendulker has quit IRC | 17:59 | |
openstackgerrit | Merged openstack-infra/tripleo-ci: Add note about scenario004 in README https://review.openstack.org/396238 | 17:59 |
*** nyechiel has quit IRC | 17:59 | |
*** derekh has quit IRC | 18:01 | |
*** fragatin_ has joined #tripleo | 18:01 | |
*** fragatina has quit IRC | 18:05 | |
*** rasca has quit IRC | 18:06 | |
openstackgerrit | Brad P. Crochet proposed openstack/tripleo-heat-templates: Composable Zaqar services https://review.openstack.org/331682 | 18:06 |
*** ayoung has joined #tripleo | 18:08 | |
*** hewbrocca is now known as hewbrocca_afk | 18:09 | |
*** dtantsur is now known as dtantsur|afk | 18:09 | |
*** jkilpatr has quit IRC | 18:14 | |
*** _milan_ has joined #tripleo | 18:14 | |
*** milan has quit IRC | 18:15 | |
*** jkilpatr has joined #tripleo | 18:21 | |
*** jpena is now known as jpena|off | 18:23 | |
*** tosky has quit IRC | 18:23 | |
*** sudipto has quit IRC | 18:24 | |
openstackgerrit | Merged openstack/python-tripleoclient: Switch overcloud build images to use ImageBuildManager https://review.openstack.org/385530 | 18:25 |
thrash | bnemec: little love to https://review.openstack.org/#/c/395726/ ? | 18:26 |
thrash | bnemec: it's a quick one. | 18:26 |
*** assassin has quit IRC | 18:26 | |
bnemec | thrash: +2A | 18:27 |
thrash | bnemec: gracias | 18:27 |
*** rcernin has joined #tripleo | 18:33 | |
openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Default $STABLE_RELEASE in bootstrap-overcloud-full.sh https://review.openstack.org/385159 | 18:34 |
openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Add configurable mtu to --mulitnode-setup https://review.openstack.org/385162 | 18:34 |
openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Add --setup-nodepool-files option https://review.openstack.org/385160 | 18:34 |
openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Create dummy overcloud-full in --boostrap-subnodes https://review.openstack.org/385161 | 18:34 |
*** cylopez has quit IRC | 18:34 | |
*** ubijtsa has joined #tripleo | 18:36 | |
*** ubijtsa has joined #tripleo | 18:36 | |
*** nyechiel has joined #tripleo | 18:36 | |
openstackgerrit | Ben Nemec proposed openstack/tripleo-common: Install yum-plugin-priorities on centos images https://review.openstack.org/397330 | 18:38 |
*** amoralej is now known as amoralej|off | 18:38 | |
openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Add configurable mtu to --mulitnode-setup https://review.openstack.org/385162 | 18:38 |
openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Remove yum-plugin-priorities hack https://review.openstack.org/397331 | 18:38 |
bnemec | hax-- | 18:38 |
openstackgerrit | Harry Rybacki proposed openstack/tripleo-quickstart: [WIP] DO NOT MERGE -- Add tuned check to OOOQ https://review.openstack.org/396362 | 18:39 |
*** ebarrera has quit IRC | 18:39 | |
*** lmiccini has quit IRC | 18:48 | |
artom_ | Would the RFE to configure Nova migrations out of the box require a spec? (context: https://bugzilla.redhat.com/show_bug.cgi?id=1271058) | 18:48 |
openstack | bugzilla.redhat.com bug 1271058 in rhel-osp-director "[RFE] OpenStack director should enable instance live migration for overcloud" [High,New] - Assigned to hbrock | 18:48 |
*** cwolferh has quit IRC | 18:49 | |
*** cwolferh has joined #tripleo | 18:50 | |
*** achadha has joined #tripleo | 18:50 | |
*** ubijtsa has quit IRC | 18:53 | |
*** florianf has quit IRC | 18:54 | |
slagle | artom_: we don't have specific rules around that. and the spec deadline for ocata is this week | 18:54 |
slagle | artom_: if it could be done via environment files that could be optionally included, i doubt that people would ask for a spec | 18:55 |
openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: WIP: DONT MERGE TESTING https://review.openstack.org/316436 | 18:58 |
*** ubijtsa has joined #tripleo | 18:58 | |
*** ubijtsa has joined #tripleo | 18:58 | |
*** trown|lunch is now known as trown | 18:59 | |
artom_ | slagle, I'm not sure about your second question about environment files | 19:00 |
artom_ | slagle, but in your first sentence it sounds like you're saying that there's no chance of making it for ocata, spec or no spec | 19:00 |
slagle | let me try and rephrase | 19:04 |
slagle | if this work requires a spec, the spec deadline for ocata is this week | 19:04 |
slagle | so it is unlikely for ocata | 19:04 |
*** tiswanso has quit IRC | 19:04 | |
slagle | the second comment was a way to implement it that I feel would not require a spec, fwiw | 19:05 |
openstackgerrit | Harry Rybacki proposed openstack/tripleo-quickstart: [WIP] DO NOT MERGE -- Add tuned check to OOOQ https://review.openstack.org/396362 | 19:05 |
slagle | in which case, it has a chance at ocata | 19:05 |
artom_ | slagle, aha! Many thanks | 19:08 |
artom_ | I'll try and find out what environment files are :D | 19:08 |
openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: Full quickstart gate run on OVB https://review.openstack.org/381094 | 19:08 |
*** tiswanso has joined #tripleo | 19:13 | |
*** fragatin_ has quit IRC | 19:15 | |
*** _milan_ has quit IRC | 19:15 | |
*** fragatina has joined #tripleo | 19:15 | |
*** tiswanso has quit IRC | 19:16 | |
*** jkilpatr_ has joined #tripleo | 19:18 | |
*** jkilpatr has quit IRC | 19:18 | |
openstackgerrit | ayoung proposed openstack/python-tripleoclient: Fernet Key management https://review.openstack.org/397342 | 19:19 |
*** milan has joined #tripleo | 19:19 | |
ayoung | EmilienM, that's one! | 19:20 |
*** chandankumar has quit IRC | 19:20 | |
*** tiswanso has joined #tripleo | 19:20 | |
*** bfournie has left #tripleo | 19:21 | |
*** akrivoka has quit IRC | 19:22 | |
*** dprince has quit IRC | 19:22 | |
*** myoung|afk is now known as myoung | 19:23 | |
*** jkilpatr_ has quit IRC | 19:25 | |
*** bfournie has joined #tripleo | 19:25 | |
*** rbrady-afk is now known as rbrady | 19:27 | |
*** milan has quit IRC | 19:30 | |
openstackgerrit | Merged openstack/tripleo-common: Replace ucs_hostname with ucs_address https://review.openstack.org/396086 | 19:30 |
*** bana_k has quit IRC | 19:32 | |
openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Always use cached images in updates job https://review.openstack.org/386713 | 19:33 |
*** milan has joined #tripleo | 19:34 | |
openstackgerrit | Pradeep Kilambi proposed openstack/instack-undercloud: Add gnocchi support on undercloud https://review.openstack.org/392992 | 19:35 |
*** abregman has joined #tripleo | 19:38 | |
*** jkilpatr_ has joined #tripleo | 19:39 | |
openstackgerrit | Merged openstack/os-collect-config: Treat ec2 collector data as immutable https://review.openstack.org/384712 | 19:39 |
openstackgerrit | Merged openstack/python-tripleoclient: Change wording of help for --update-plan-only https://review.openstack.org/395726 | 19:39 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-tripleoclient: Updated from global requirements https://review.openstack.org/395379 | 19:42 |
openstackgerrit | ayoung proposed openstack/tripleo-heat-templates: Configure Keystone Fernet Keys https://review.openstack.org/397350 | 19:55 |
*** thrash is now known as thrash|biab | 19:57 | |
*** arxcruz has quit IRC | 20:00 | |
*** tiswanso has quit IRC | 20:02 | |
*** cwolferh has quit IRC | 20:03 | |
*** tiswanso has joined #tripleo | 20:06 | |
ayoung | SOME REWROTE THE TRIPLEOCLIENT! | 20:06 |
ayoung | Where did everything I need go? | 20:06 |
ayoung | Where is all the Credentials stuff? | 20:06 |
ayoung | You Fool! You Blew up the Machine! | 20:10 |
*** cwolferh has joined #tripleo | 20:10 | |
*** nyechiel has quit IRC | 20:10 | |
mwhahaha | :o | 20:11 |
trozet | dsneddon: could you please +2 and workflow this cherry pick? https://review.openstack.org/#/c/397228/ | 20:16 |
*** bana_k has joined #tripleo | 20:21 | |
trown | lol | 20:23 |
*** panda|bbl is now known as panda | 20:25 | |
mwhahaha | ayoung: it got moved into tripleo-common, check the constants.py (mistral generates them now) | 20:27 |
mwhahaha | also we probably should document this :o | 20:27 |
mwhahaha | in something other than a bug,https://bugs.launchpad.net/tripleo/+bug/1621097 | 20:28 |
openstack | Launchpad bug 1621097 in tripleo "Password generation should be moved from tripleoclient to mistral workflows" [High,Fix released] - Assigned to Dougal Matthews (d0ugal) | 20:28 |
*** bfournie has left #tripleo | 20:28 | |
*** bana_k has quit IRC | 20:31 | |
ayoung | mwhahaha, I don't see the Credentials in there | 20:31 |
dsneddon | trozet, Done | 20:32 |
trozet | dsneddon: ty | 20:32 |
*** ctayal has quit IRC | 20:34 | |
*** rcernin has quit IRC | 20:35 | |
*** thrash|biab is now known as thrash | 20:38 | |
*** dprince has joined #tripleo | 20:41 | |
*** pcaruana has quit IRC | 20:45 | |
*** bfournie has joined #tripleo | 20:46 | |
*** yamahata has joined #tripleo | 20:50 | |
*** yamahata has joined #tripleo | 20:53 | |
*** abregman has quit IRC | 20:55 | |
openstackgerrit | Merged openstack-infra/tripleo-ci: Use yaml files for building images for CI https://review.openstack.org/336312 | 20:58 |
*** penick has joined #tripleo | 20:58 | |
*** aufi has quit IRC | 20:58 | |
*** rcernin has joined #tripleo | 21:01 | |
*** abehl has quit IRC | 21:09 | |
openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Add configurable mtu to --mulitnode-setup https://review.openstack.org/385162 | 21:12 |
stevebaker | weshay: hey, I have this in my overcloud_containers.yaml | 21:12 |
stevebaker | weshay: push_destination: 192.0.2.3:8787 | 21:12 |
weshay | ah | 21:13 |
weshay | stevebaker, I'll try that.. thanks | 21:13 |
openstackgerrit | Bob Fournier proposed openstack/os-net-config: Stop dhclient in os-net-config if interface not set for DHCP https://review.openstack.org/396446 | 21:13 |
stevebaker | weshay: so never use localhost, always 192.168.24.3:8787 | 21:13 |
stevebaker | weshay: also in your custom.yaml environment file DockerNamespace: 192.168.24.3:8787/tripleoupstream | 21:14 |
stevebaker | weshay: and DockerNamespaceIsRegistry: true | 21:14 |
weshay | stevebaker, here's what I currently have https://github.com/redhat-openstack/ansible-role-tripleo-overcloud-prep-containers/blob/master/templates/container-default-parameters.yaml.j2 | 21:15 |
*** jayg is now known as jayg|g0n3 | 21:15 | |
dsneddon | bnemec, Regarding https://review.openstack.org/#/c/396285/, I wonder if we would be better off modifying os-net-config so that it automatically added fail-mode=standalone to OVS bridges, then making that a flag. If it's vital, we could make it default=True. | 21:16 |
d0ugal | ayoung: you can blame me probably :) | 21:17 |
stevebaker | weshay: I'd recomend DockerNamespace: 192.168.24.3... instead of .1 | 21:17 |
dsneddon | bnemec, The reason I initially rejected that idea was that it would guarantee a brief network outage on OVS bridges, but if the alternative is a more complete outage, I'm willing to consider it. | 21:17 |
stevebaker | weshay: where does your heat-docker-agents image come from? | 21:17 |
bnemec | dsneddon: I would prefer something like that. I think there's going to be a brief outage either way, right? We need to set the option at some point. | 21:18 |
weshay | stevebaker, https://github.com/redhat-openstack/ansible-role-tripleo-overcloud-prep-containers/blob/master/templates/overcloud-prep-containers.sh.j2#L64 | 21:18 |
dsneddon | bnemec, Right, and changing every template (including all custom ones) seems much worse. | 21:19 |
*** tiswanso has quit IRC | 21:22 | |
trozet | hi all. For mariadb, we see sometimes sql fails because it hits max open files limit. Looking at the limit, it is set to 1024 due to the system's ulimit default. If we set it to something huge like 65535 it works fine. Is there anything done in OOO today to configure the security limits of the system, or the mariadb service? | 21:24 |
*** tiswanso has joined #tripleo | 21:24 | |
trozet | dsneddon:^ any idea there? | 21:25 |
weshay | stevebaker, which template is push_destination a variable for? | 21:26 |
stevebaker | weshay: its all of these https://github.com/redhat-openstack/ansible-role-tripleo-overcloud-prep-containers/blob/master/templates/overcloud-prep-containers.sh.j2#L70 | 21:26 |
stevebaker | weshay: used by your /upload_images_to_local_registry.py | 21:26 |
*** jkilpatr_ has quit IRC | 21:27 | |
weshay | stevebaker, ya.. upload_script is here.. https://github.com/redhat-openstack/ansible-role-tripleo-overcloud-prep-containers/blob/master/templates/upload_images_to_local_registry.py.j2 | 21:27 |
stevebaker | weshay: ok... we really need to write an openstack command for that | 21:29 |
mwhahaha | ayoung: which credentials? | 21:30 |
bnemec | trozet: I think we fixed that by creating a limits.conf file for the mariadb service: https://github.com/openstack/instack-undercloud/blob/fc637ab99b83c73009f9aac81179cd61012ce0e4/elements/puppet-stack-config/puppet-stack-config.pp#L91 | 21:34 |
bnemec | Although I can't find where that is done on the overcloud for some reason. | 21:34 |
ayoung | d0ugal, I blame you anyway. What are we talking about again? | 21:35 |
ayoung | mwhahaha, ok, so credentials is a Keystone concept that they decided to encrypt by default | 21:35 |
ayoung | and that, of course, broken Tripleo | 21:35 |
ayoung | so EmilienM fixed it | 21:35 |
ayoung | and then they realized they were breaking the world | 21:36 |
ayoung | they being Keystone | 21:36 |
mwhahaha | https://github.com/openstack/tripleo-common/blob/master/tripleo_common/constants.py#L76-L77 | 21:36 |
ayoung | and by realized I really mean I hit them over the head with it | 21:36 |
mwhahaha | those? | 21:36 |
ayoung | yes | 21:36 |
ayoung | I just posted a review | 21:36 |
ayoung | not ificartions for tripleo-common don't show up here? | 21:36 |
ayoung | oH...one sec | 21:37 |
openstackgerrit | ayoung proposed openstack/tripleo-common: Fernet Key management https://review.openstack.org/397381 | 21:37 |
ayoung | There we go | 21:37 |
ayoung | mwhahaha, https://review.openstack.org/#/c/397381/ | 21:38 |
ayoung | Heh | 21:38 |
ayoung | faster than the bot~! | 21:38 |
*** Goneri has quit IRC | 21:38 | |
trozet | bnemec: yeah I don't see it set on the overcloud either. I see this bug https://bugs.launchpad.net/tripleo/+bug/1524809 - I just added a comment at hte bottom | 21:38 |
openstack | Launchpad bug 1524809 in tripleo "We set a value of -1 for the open_files_limit in galera.conf" [Low,In progress] - Assigned to Marios Andreou (marios-b) | 21:38 |
*** bana_k has joined #tripleo | 21:39 | |
mwhahaha | ayoung: so we should probably make the token provider properly configurable as well, is that something you're working on as well? | 21:40 |
ayoung | mwhahaha, lets make it possible before we make it the default | 21:40 |
mwhahaha | ayoung: well right now we inherit the defaults from puppet-keystone so we should probably define the default of uuid for now until fernet is ready | 21:40 |
mwhahaha | we had to back out the puppet-keystone change because of that | 21:40 |
ayoung | mwhahaha, EmilienM already did that | 21:41 |
mwhahaha | i don't think he did | 21:41 |
mwhahaha | maybe i missed it | 21:41 |
ayoung | Yeah, it broken stuff | 21:41 |
ayoung | Upstream Keystone changed the default | 21:41 |
ayoung | and borked OOO | 21:41 |
ayoung | bork bork bork | 21:41 |
mwhahaha | no puppet-keystone borked ooo | 21:41 |
mwhahaha | :D | 21:41 |
mwhahaha | https://review.openstack.org/#/c/392332/ | 21:42 |
mwhahaha | specifically because we don't define the provider type in THT | 21:42 |
openstackgerrit | Merged openstack/tripleo-heat-templates: Fixes missing OVS Firewall config with OpenDaylight https://review.openstack.org/397228 | 21:45 |
*** dougbtv has quit IRC | 21:47 | |
*** trown is now known as trown|outtypewww | 21:53 | |
*** kjw3 has joined #tripleo | 21:56 | |
*** bana_k has quit IRC | 21:59 | |
openstackgerrit | Alex Schultz proposed openstack/tripleo-heat-templates: Define keystone token provider https://review.openstack.org/397391 | 21:59 |
*** dsavineau has quit IRC | 22:03 | |
beagles | dsneddon: already working on a patch that does that | 22:03 |
beagles | ie standalone by default | 22:07 |
*** lblanchard has quit IRC | 22:07 | |
*** bvandenh has quit IRC | 22:08 | |
*** achadha has quit IRC | 22:11 | |
*** milan has quit IRC | 22:14 | |
*** ecerquei__ has quit IRC | 22:14 | |
*** tiswanso has quit IRC | 22:17 | |
*** kjw3 has quit IRC | 22:18 | |
openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Add line break between review link and duration https://review.openstack.org/397396 | 22:19 |
openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Add overall stats to ci-status page https://review.openstack.org/397397 | 22:19 |
dsneddon | beagles, Great, less work for me. ;-) Thanks. | 22:21 |
dsneddon | beagles, Was thinking of a flag for OVS bridges, something like "ovs_fail_mode: <foo>", which defaulted to "standalone". | 22:22 |
dsneddon | beagles, Which gets prepended to the OVS_EXTRA | 22:23 |
dsneddon | beagles, But you're probably already one step ahead. | 22:23 |
beagles | dsneddon: basically that | 22:23 |
beagles | :) | 22:23 |
dsneddon | beagles, If you have a quick moment to give a review on this patch, we are trying to merge it ASAP: https://review.openstack.org/#/c/396446/ | 22:24 |
beagles | dsneddon: will d | 22:25 |
dsneddon | beagles, Should fix this bug: https://bugs.launchpad.net/tripleo/+bug/1640598 | 22:25 |
openstack | Launchpad bug 1640598 in tripleo "dhcp-all-interfaces script continues to run after os-net-config runs" [High,In progress] - Assigned to Dan Sneddon (dsneddon) | 22:25 |
ggillies | does anyone know who/how I would go about getting a newer version of ansible included in RDO | 22:29 |
ggillies | we've only got 2.1.0 which is old and has known issues with the openstack modules | 22:29 |
ggillies | not sure if this is a tripleo request, or an RDO request | 22:29 |
*** pkovar has quit IRC | 22:33 | |
*** jcoufal has quit IRC | 22:33 | |
thrash | ggillies: pretty sure that would be rdo | 22:33 |
ggillies | thrash: ta | 22:34 |
*** ayoung has quit IRC | 22:39 | |
*** noslzzp has quit IRC | 22:42 | |
*** jkilpatr has joined #tripleo | 22:46 | |
*** ccamacho has quit IRC | 22:53 | |
openstackgerrit | Alex Schultz proposed openstack/tripleo-heat-templates: Define keystone token provider https://review.openstack.org/397391 | 22:56 |
openstackgerrit | Brent Eagles proposed openstack/os-net-config: Add ovs_fail_mode option for OVS bridges https://review.openstack.org/397405 | 22:59 |
*** penick has quit IRC | 23:00 | |
openstackgerrit | Brent Eagles proposed openstack/os-net-config: Add ovs_fail_mode option for OVS bridges https://review.openstack.org/397405 | 23:01 |
*** penick has joined #tripleo | 23:03 | |
*** jprovazn has quit IRC | 23:04 | |
*** dprince has quit IRC | 23:06 | |
*** noslzzp has joined #tripleo | 23:07 | |
openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: Full quickstart gate run on OVB https://review.openstack.org/381094 | 23:11 |
openstackgerrit | Ian Wienand proposed openstack/diskimage-builder: simplify ARCH param for rhel/centos param can be x86_64 and amd64 https://review.openstack.org/392176 | 23:18 |
openstackgerrit | Ian Wienand proposed openstack/diskimage-builder: add support for SUSE in dhcp-all-interfaces https://review.openstack.org/391772 | 23:20 |
*** absubram has quit IRC | 23:21 | |
*** pradk has quit IRC | 23:25 | |
*** cdearborn has quit IRC | 23:28 | |
*** dtrainor has quit IRC | 23:34 | |
*** dtrainor has joined #tripleo | 23:37 | |
*** penick has quit IRC | 23:38 | |
*** penick has joined #tripleo | 23:41 | |
openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Add newton branch to delorean-build branches https://review.openstack.org/397439 | 23:44 |
*** noslzzp has quit IRC | 23:44 | |
*** morazi has quit IRC | 23:44 | |
*** noslzzp has joined #tripleo | 23:45 | |
*** dtrainor has quit IRC | 23:46 | |
*** penick has quit IRC | 23:48 | |
openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Add 3 node multinode job https://review.openstack.org/397441 | 23:49 |
openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Add 3 node multinode job https://review.openstack.org/397441 | 23:49 |
openstackgerrit | Alex Schultz proposed openstack/tripleo-heat-templates: Define keystone token provider https://review.openstack.org/397391 | 23:50 |
*** ayoung has joined #tripleo | 23:51 | |
openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Always use cached images in updates job https://review.openstack.org/386713 | 23:52 |
openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Temporarily disable ceph in the updates job https://review.openstack.org/397288 | 23:52 |
*** dtrainor has joined #tripleo | 23:55 | |
openstackgerrit | Alex Schultz proposed openstack/tripleo-heat-templates: Define keystone token provider https://review.openstack.org/397391 | 23:55 |
*** panda is now known as panda|Zz | 23:59 | |
openstackgerrit | Bob Fournier proposed openstack/os-net-config: Stop dhclient in os-net-config if interface not set for DHCP https://review.openstack.org/396446 | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!