| *** ayoung has quit IRC | 00:04 | |
| *** chlong has joined #tripleo | 00:12 | |
| *** dmacpher has quit IRC | 00:21 | |
| *** cdearborn has quit IRC | 00:25 | |
| openstackgerrit | Bob Fournier proposed openstack/os-net-config: Stop dhclient in os-net-config if interface is not configured for DHCP https://review.openstack.org/396446 | 00:25 |
|---|---|---|
| dsneddon | bfournie++ | 00:26 |
| *** colonwq has joined #tripleo | 00:31 | |
| *** jerrygb has joined #tripleo | 00:34 | |
| *** dmacpher has joined #tripleo | 00:34 | |
| *** jerrygb has quit IRC | 00:38 | |
| *** limao has joined #tripleo | 00:39 | |
| *** ayoung has joined #tripleo | 00:45 | |
| openstackgerrit | Gabriele Cerami proposed openstack-infra/tripleo-ci: POC: Initial transition to quickstart setup https://review.openstack.org/396451 | 00:45 |
| 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 | 00:54 |
| openstackgerrit | Dan Sneddon proposed openstack/os-net-config: Stop dhclient in os-net-config if interface is not configured for DHCP https://review.openstack.org/396446 | 01:28 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: TEST: DONT RECHECK: periodic jobs https://review.openstack.org/359215 | 01:29 |
| *** jkilpatr has quit IRC | 01:35 | |
| *** tiswanso has joined #tripleo | 01:38 | |
| *** tiswanso has quit IRC | 01:38 | |
| *** tiswanso has joined #tripleo | 01:39 | |
| 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 | 01:43 |
| openstackgerrit | Steve Baker proposed openstack/instack-undercloud: Configure undercloud docker for insecure registry https://review.openstack.org/396460 | 01:46 |
| *** bfournie has quit IRC | 01:49 | |
| *** fzdarsky|afk has quit IRC | 02:08 | |
| *** limao has quit IRC | 02:08 | |
| *** limao has joined #tripleo | 02:09 | |
| *** chlong has quit IRC | 02:11 | |
| *** chandankumar has joined #tripleo | 02:17 | |
| *** chandankumar has quit IRC | 02:18 | |
| *** tiswanso has quit IRC | 02:22 | |
| *** tiswanso has joined #tripleo | 02:22 | |
| *** anton has quit IRC | 02:30 | |
| *** jerrygb has joined #tripleo | 02:34 | |
| *** jerrygb_ has joined #tripleo | 02:37 | |
| *** jerrygb has quit IRC | 02:39 | |
| *** bfournie has joined #tripleo | 02:49 | |
| 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 | 02:53 |
| *** jerrygb has joined #tripleo | 02:54 | |
| *** jerrygb_ has quit IRC | 02:56 | |
| *** jerrygb has quit IRC | 03:00 | |
| *** tzumainn has quit IRC | 03:17 | |
| *** mhenkel has quit IRC | 03:17 | |
| *** ebalduf has joined #tripleo | 03:21 | |
| openstackgerrit | RedHat RDO CI proposed openstack/tripleo-heat-templates: GATE TEST, please ignore https://review.openstack.org/365449 | 03:30 |
| *** mhenkel has joined #tripleo | 03:48 | |
| *** sudipto has joined #tripleo | 04:10 | |
| *** sudipto_ has joined #tripleo | 04:10 | |
| *** fzdarsky|afk has joined #tripleo | 04:10 | |
| *** tiswanso has quit IRC | 04:11 | |
| *** mhenkel has quit IRC | 04:12 | |
| *** jerrygb has joined #tripleo | 04:20 | |
| *** mhenkel has joined #tripleo | 04:22 | |
| *** jerrygb has quit IRC | 04:23 | |
| *** jerrygb has joined #tripleo | 04:23 | |
| *** bana_k has joined #tripleo | 04:24 | |
| *** jerrygb has quit IRC | 04:28 | |
| *** mhenkel has quit IRC | 04:33 | |
| *** bana_k has quit IRC | 04:35 | |
| *** numans has joined #tripleo | 04:45 | |
| *** fragatin_ has joined #tripleo | 04:53 | |
| *** fragatina has quit IRC | 04:57 | |
| *** fragatin_ has quit IRC | 04:57 | |
| *** fragatina has joined #tripleo | 05:00 | |
| *** jerrygb has joined #tripleo | 05:01 | |
| *** fragatina has quit IRC | 05:01 | |
| *** jerrygb has quit IRC | 05:02 | |
| *** Vijayendra has joined #tripleo | 05:07 | |
| *** mhenkel has joined #tripleo | 05:10 | |
| *** rlandy has quit IRC | 05:11 | |
| *** yamahata has quit IRC | 05:21 | |
| *** thrash is now known as thrash|g0ne | 05:22 | |
| *** masco has joined #tripleo | 05:24 | |
| *** fragatina has joined #tripleo | 05:37 | |
| *** nyechiel has joined #tripleo | 05:38 | |
| *** coolsvap has joined #tripleo | 05:45 | |
| *** rajinir has quit IRC | 05:46 | |
| *** florianf has joined #tripleo | 05:52 | |
| *** mhenkel has quit IRC | 05:54 | |
| *** nyechiel has quit IRC | 05:59 | |
| *** chandankumar has joined #tripleo | 06:21 | |
| *** redhatkj has quit IRC | 06:22 | |
| openstackgerrit | Aparna proposed openstack/diskimage-builder: element: proliant-tools: Update hpssacli to ssacli https://review.openstack.org/396504 | 06:30 |
| *** jaosorior has joined #tripleo | 06:32 | |
| *** redhatkj has joined #tripleo | 06:33 | |
| *** hewbrocca_afk is now known as hewbrocca | 06:37 | |
| *** lmiccini has joined #tripleo | 06:38 | |
| *** hewbrocca is now known as hewbrocca_afk | 06:38 | |
| *** hewbrocca_afk is now known as hewbrocca | 06:39 | |
| *** abehl has joined #tripleo | 06:41 | |
| *** ealcaniz has joined #tripleo | 06:43 | |
| *** aufi has joined #tripleo | 06:44 | |
| *** pmannidi has quit IRC | 06:47 | |
| *** oshvartz has joined #tripleo | 06:48 | |
| *** iranzo has joined #tripleo | 06:48 | |
| *** iranzo has quit IRC | 06:48 | |
| *** iranzo has joined #tripleo | 06:48 | |
| jaosorior | therve: around yet? | 06:59 |
| *** yamahata has joined #tripleo | 06:59 | |
| *** udesale has joined #tripleo | 06:59 | |
| *** tesseract has joined #tripleo | 07:03 | |
| *** tesseract is now known as Guest7605 | 07:03 | |
| *** b00tcat` has joined #tripleo | 07:05 | |
| *** rasca has joined #tripleo | 07:06 | |
| *** Alex___1234 has quit IRC | 07:09 | |
| *** rcernin has joined #tripleo | 07:11 | |
| *** hewbrocca is now known as hewbrocca_afk | 07:12 | |
| *** rasca has quit IRC | 07:18 | |
| *** rasca has joined #tripleo | 07:18 | |
| *** ebalduf has quit IRC | 07:21 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/instack-undercloud: Configure keystone ec2 conf for heat behind haproxy https://review.openstack.org/396516 | 07:21 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/instack-undercloud: Move glance_api_servers conf for nova to hiera https://review.openstack.org/396517 | 07:21 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/instack-undercloud: Pass keystone configuration via hiera https://review.openstack.org/396518 | 07:21 |
| *** jprovazn has joined #tripleo | 07:21 | |
| *** oshvartz has quit IRC | 07:25 | |
| *** ealcaniz has quit IRC | 07:27 | |
| *** jaosorior has quit IRC | 07:31 | |
| openstackgerrit | Julie Pichon proposed openstack/instack-undercloud: Increase Mistral Task Size limit https://review.openstack.org/396523 | 07:34 |
| *** ealcaniz has joined #tripleo | 07:34 | |
| openstackgerrit | Julie Pichon proposed openstack/python-tripleoclient: Reset the parameter_defaults between deployments via the CLI https://review.openstack.org/396524 | 07:36 |
| *** anton has joined #tripleo | 07:40 | |
| *** oshvartz has joined #tripleo | 07:41 | |
| *** yolanda has quit IRC | 07:41 | |
| *** yolanda has joined #tripleo | 07:41 | |
| *** pmannidi has joined #tripleo | 07:50 | |
| *** ebarrera has joined #tripleo | 07:54 | |
| *** gbarros has quit IRC | 07:56 | |
| *** jlinkes has joined #tripleo | 08:00 | |
| *** tremble has joined #tripleo | 08:00 | |
| *** jaosorior has joined #tripleo | 08:02 | |
| *** zoli|gone is now known as zoli|wfh | 08:03 | |
| *** phpcodemonkey has joined #tripleo | 08:04 | |
| jaosorior | can anybody review this commit for CI? https://review.openstack.org/#/c/396056/ | 08:05 |
| jaosorior | marios: hey dude, could you check this one out? https://review.openstack.org/#/c/392473/ it's a quick one :D | 08:06 |
| *** ccamacho has joined #tripleo | 08:08 | |
| jaosorior | ccamacho: hey, if you have time today could you check these TLS patches? https://review.openstack.org/#/c/392090/ https://review.openstack.org/#/c/392095/ | 08:10 |
| ccamacho | hey jaosorior morning, sure let me add them to my list :) | 08:11 |
| marios | jaosorior: done | 08:12 |
| jaosorior | marios: thanks dude | 08:13 |
| *** shardy has joined #tripleo | 08:17 | |
| *** hjensas has quit IRC | 08:18 | |
| jaosorior | shardy: any idea what would cause this http://paste.openstack.org/show/588907/ to appear all over the cfn api? | 08:18 |
| jaosorior | therve: ^^ | 08:21 |
| *** athomas has joined #tripleo | 08:22 | |
| shardy | jaosorior: could be a number of things, but basically the heat ec2token middleware takes the request and signature, and asks the keystone ec2tokens extension to validate it | 08:24 |
| shardy | that is failing, either because we've mangled something in the URL so the signature is now wrong, or because the ec2tokens API is somehow broken | 08:25 |
| shardy | jaosorior: does the switch to the httpd API involve any rewriting of the URL we get in the API vs what the client requests? | 08:26 |
| shardy | the signature calculation requires the path etc to be *exactly* the same, or the signatures won't match | 08:26 |
| jaosorior | shardy: it doesn't | 08:26 |
| jaosorior | shardy: is the signature passed via a header or something of the sort? | 08:29 |
| shardy | jaosorior: I think we're still using the v2 CFN API format, so it's just part of the query string | 08:30 |
| jaosorior | shardy: how does it change in further versions? | 08:30 |
| *** ccamacho has quit IRC | 08:31 | |
| *** jpich has joined #tripleo | 08:31 | |
| shardy | jaosorior: I'd probably put some debug in here: | 08:31 |
| shardy | https://github.com/openstack/python-keystoneclient/blob/master/keystoneclient/contrib/ec2/utils.py#L80 | 08:31 |
| shardy | then compare the credentials dict during the create with that used during the validation | 08:32 |
| openstackgerrit | Julie Pichon proposed openstack/puppet-tripleo: Persist the UI config file https://review.openstack.org/388592 | 08:32 |
| *** amoralej|off is now known as amoralej | 08:34 | |
| shardy | (that library is used by both heat to calculate the signature, and keystone to validate it, so all else being equal they should match) | 08:34 |
| *** hjensas has joined #tripleo | 08:37 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/instack-undercloud: Move glance_api_servers conf for nova to hiera https://review.openstack.org/396517 | 08:38 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/instack-undercloud: Pass keystone configuration via hiera https://review.openstack.org/396518 | 08:38 |
| *** jpena|off is now known as jpena | 08:38 | |
| jaosorior | shardy: just to be clear, this generate function is called from heat in the undercloud? or where? | 08:43 |
| shardy | jaosorior: heat and keystone on the undercloud both call it | 08:43 |
| b00tcat` | what is the difference between `config_settings` and `service_config_settings` used in the service YAML files? I don't quite get it :-/ | 08:43 |
| shardy | heat calls it to calculate the signature | 08:43 |
| *** phpcodemonkey has quit IRC | 08:43 | |
| shardy | jaosorior: then later, when we signal back to heat, the ec2tokens middleware asks keystone's ec2tokens api to validate the signature | 08:44 |
| shardy | which happens to use the exact same keystoneclient code | 08:44 |
| shardy | in both cases we provide details about the request, and get a hash back | 08:44 |
| shardy | so something about the request we give to keystone when the signal happens must be wrong | 08:44 |
| *** ohamada has joined #tripleo | 08:45 | |
| shardy | jaosorior: you could also do a minimal reproducer with a heat stack that only contains one OS::Heat::SoftwareDeployment resource and nothing else | 08:45 |
| shardy | that will create a signature, and you can hit the URL it exposes via curl | 08:46 |
| shardy | which is essentially all TripleO is doing | 08:46 |
| shardy | b00tcat`: config_settings is data we deploy on the node where the service is running, e.g in services/heat-api.yaml it's deployed on the nodes running heat-api | 08:47 |
| *** abregman has joined #tripleo | 08:47 | |
| shardy | b00tcat`: service_config_settings is used when e.g heat-api needs the node running keystone to do something, e.g create a user or similar | 08:47 |
| shardy | b00tcat`: so in that case we wire service_config_settings: {keystone: foo ... | 08:48 |
| shardy | to the node running keystone | 08:48 |
| shardy | which isn't necessarily the same one that's running e.g heat | 08:48 |
| shardy | b00tcat`: we do need better docs on this, thanks for the reminder | 08:48 |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Format the nodes list in openestack overcloud delete node https://review.openstack.org/395083 | 08:52 |
| openstackgerrit | Jiri Tomasek proposed openstack/tripleo-specs: GUI Deployment Configuration improvements https://review.openstack.org/393365 | 08:53 |
| *** paramite has joined #tripleo | 08:56 | |
| jaosorior | shardy: ok, trying out the minimal reproducer | 08:56 |
| Ng | morning folks | 08:57 |
| *** abregman has quit IRC | 08:58 | |
| Ng | can we try and get bnemec's puppet-lint fix landed? It's blocking zuul from merging a few backports atm. It's https://review.openstack.org/#/c/396419/1 | 08:58 |
| Ng | (well, it's blocking anything from landing in puppet-tripleo, but I know of at least two backport patches that would be very welcome if they could land before an internal deadline today :) | 09:00 |
| Ng | jaosorior: :) | 09:00 |
| shardy | Ng: done! | 09:00 |
| Ng | awesome, thank you both! | 09:00 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add missing Barbican endpoint from tls-everywhere environment https://review.openstack.org/392473 | 09:00 |
| * shardy wonders why we didn't catch that in CI before it landed | 09:01 | |
| *** zoli|wfh is now known as zoli|doctor | 09:01 | |
| Ng | shardy: I think it may be that a package was upgraded, and the linter got slightly stricter | 09:01 |
| shardy | ah, I guess that would explain it | 09:01 |
| *** b00tcat` has quit IRC | 09:05 | |
| jaosorior | shardy: soo... how do I get that URL from the deployment? | 09:05 |
| *** mcornea has joined #tripleo | 09:07 | |
| *** b00tcat` has joined #tripleo | 09:07 | |
| shardy | jaosorior: ah, I forgot, you'll need either a server or a dummy resource acting as a server: | 09:07 |
| shardy | https://etherpad.openstack.org/p/noop-softwareconfig | 09:08 |
| shardy | you can cut/paste from that I think | 09:08 |
| b00tcat` | thanks for the answer shardy | 09:08 |
| shardy | see the deploy_signal_id in the resource metadata for the dummy server | 09:08 |
| jaosorior | I deployed it on a server | 09:08 |
| shardy | jaosorior: Ok then you can just do heat resource-metadata thestack theserver_resource_name | 09:08 |
| *** bnemec has quit IRC | 09:09 | |
| jaosorior | ok, deploy_signal_id looks like this http://172.29.172.1:8000/v1/signal/arn%3Aopenstack%3Aheat%3A%3A039a7d5c7f214d55b4fe90855dca7327%3Astacks%2Fwalrus%2Fc919935a-e241-4a6f-a58b-6323bd67d0fa%2Fresources%2Fdeploymentthing?Timestamp=2016-11-11T08%3A56%3A36Z&SignatureMethod=HmacSHA256&AWSAccessKeyId=f94159185bb14e189c8f882239a2c3a4&SignatureVersion=2&Signature=ulsDqSkgM%2FCudiIETkxNEB4374%2FdoJr2CMwaSNF4ROo%3D | 09:09 |
| jaosorior | wow | 09:10 |
| openstackgerrit | yolanda.robla proposed openstack/tripleo-quickstart: Create directories with root https://review.openstack.org/384892 | 09:10 |
| jaosorior | shardy: http://paste.openstack.org/show/588912/ | 09:10 |
| jaosorior | right so it's an issue with the URL apparently | 09:11 |
| shardy | jaosorior: did you put quotes around the entire URL? | 09:11 |
| jaosorior | I didn't | 09:11 |
| shardy | curl -X POST "http://172.29.172.1:8000/v1/s ..." | 09:11 |
| *** bnemec has joined #tripleo | 09:12 | |
| shardy | https://github.com/openstack/tripleo-image-elements/blob/master/elements/os-refresh-config/os-refresh-config/post-configure.d/99-refresh-completed#L32 | 09:12 |
| jaosorior | now that I did I got this: http://paste.openstack.org/show/588913/ | 09:12 |
| shardy | Ok, so looks like you reproduced your issue | 09:12 |
| shardy | so you can add details to the ec2signer stuff and compare the data used to calculate the signature on stack create | 09:13 |
| shardy | with that used when doing the curl call | 09:13 |
| jaosorior | not entirely sure what that means yet, if it's an actual authorization issue or that the signature didn't match | 09:13 |
| shardy | the heat logs and some keystone debug will tell you I think | 09:13 |
| shardy | I suspect the move to httpd has somehow subtly changed the data we pass to ec2Signer | 09:14 |
| jaosorior | crap | 09:18 |
| jaosorior | adding debug logs to keystone middleware does nothing | 09:18 |
| jaosorior | * keystoneclient | 09:18 |
| *** mcornea has quit IRC | 09:19 | |
| *** mcornea has joined #tripleo | 09:20 | |
| *** aufi_ has joined #tripleo | 09:21 | |
| *** aufi has quit IRC | 09:21 | |
| *** lucas-afk is now known as lucasagomes | 09:22 | |
| shardy | jaosorior: did you restart heat-engine and keystone? | 09:23 |
| jaosorior | I did | 09:24 |
| shardy | you can even kill heat-engine and run it on the CLI and grep for your debug | 09:24 |
| jaosorior | now I'm trying to create a logging object and write that log file to my home dir | 09:24 |
| jaosorior | lets see if that works | 09:24 |
| *** derekh has joined #tripleo | 09:25 | |
| shardy | https://github.com/openstack/heat/blob/master/heat/engine/resources/signal_responder.py#L173 | 09:25 |
| shardy | that's where heat calls keystoneclient FYI | 09:25 |
| *** derekh has quit IRC | 09:29 | |
| *** gfidente has joined #tripleo | 09:31 | |
| jaosorior | shardy: ok, I think I got it | 09:37 |
| jaosorior | So this is the actual signature used q61gKkwVqhHxRd3dfl2FfaBPf3/8t2doLc1LeO/7I0s= | 09:37 |
| gfidente | mwhahaha, you around? | 09:38 |
| gfidente | any clue why this is failing the tests only for puppet 4.5? | 09:38 |
| gfidente | https://review.openstack.org/#/c/395785/ | 09:38 |
| Ng | gfidente: https://review.openstack.org/#/c/396419/1 :) | 09:40 |
| Ng | +A, just waiting for zuul to grant it access to the underworld, and then we can land puppet stuff again | 09:40 |
| jaosorior | shardy: looks like this http://paste.openstack.org/show/588918/ | 09:40 |
| gfidente | Ng, that fixes the lint issue indeed | 09:40 |
| gfidente | but I was looking at the unit tests instead | 09:41 |
| jaosorior | gfidente: it'll be fixed in around ~20 min | 09:41 |
| jaosorior | oh, if that's another issue I don't know | 09:41 |
| Ng | gfidente: oh, doh, sorry | 09:41 |
| *** rbrady is now known as rbrady-afk | 09:41 | |
| gfidente | yeah it looks like the unit tests are passing for puppet < 4 | 09:42 |
| gfidente | at least for that submission | 09:42 |
| shardy | jaosorior: ack, so yeah now you need to compare the inputs to the signature calculation, one of them must be different during the API request | 09:43 |
| shardy | the "/" in the signature looks bogus to me? | 09:43 |
| openstackgerrit | yolanda.robla proposed openstack/tripleo-quickstart: Create directories with root https://review.openstack.org/384892 | 09:43 |
| jaosorior | shardy: it's what appeared in the logs :/ | 09:44 |
| *** ccamacho has joined #tripleo | 09:45 | |
| *** derekh has joined #tripleo | 09:47 | |
| *** openstackgerrit has quit IRC | 09:48 | |
| *** openstackgerrit has joined #tripleo | 09:48 | |
| shardy | Anyone feel like approving https://review.openstack.org/#/c/396317/ ? | 09:49 |
| shardy | Also https://review.openstack.org/#/q/Id240709e256b057bbdff038aefc7fcacb0d339b8,n,z | 09:49 |
| *** abregman has joined #tripleo | 09:50 | |
| shardy | marios, matbu: Hey, spotted a possible issue re https://review.openstack.org/#/c/395234/4 | 09:52 |
| openstackgerrit | Merged openstack/puppet-tripleo: Fix puppet lint failure https://review.openstack.org/396419 | 09:53 |
| shardy | I'm not sure we can unconditionally enable external ceph there? | 09:53 |
| b00tcat` | shardy: this blogpost of yours where you explain composable services points to the default ControllerServices list at https://github.com/openstack/tripleo-heat-templates/blob/master/overcloud.yaml#L356 however this link is a dead end and grepping by "ControllerServices" didn't give any result back | 09:53 |
| b00tcat` | where can I find the default services list? | 09:53 |
| jaosorior | any +As for this patch adding scenario004 to our tests? https://review.openstack.org/#/c/396056/ | 09:53 |
| shardy | https://github.com/openstack/tripleo-heat-templates/blob/master/roles_data.yaml#L23 | 09:54 |
| shardy | b00tcat`: ah, it moved after I wrote the post to roles_data.yaml | 09:54 |
| shardy | we now template in the defaults with jinja2, which was needed for the custom-roles implementation (explained in another more recent blog post) | 09:54 |
| d0ugal | What does "workload cloud stack" mean? https://github.com/openstack/tripleo-common/blob/master/workbooks/deployment.yaml#L101 | 09:54 |
| b00tcat` | shardy: ok I will check it out | 09:55 |
| shardy | d0ugal: AFAICS it means "overcloud" | 09:55 |
| d0ugal | shardy: aha, thanks. I might change it to that :) | 09:56 |
| shardy | yeah, or just remove it | 09:56 |
| openstackgerrit | yolanda.robla proposed openstack/tripleo-quickstart: Create directories with root https://review.openstack.org/384892 | 09:57 |
| panda | master undercloud is borked http://logs.openstack.org/periodic/periodic-tripleo-ci-centos-7-ovb-nonha/9521bfb/logs/undercloud/var/log/undercloud_install.txt.gz#_2016-11-11_06_22_11_000 | 10:02 |
| panda | opening bug | 10:02 |
| openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Only start the deploy if the Heat stack isn't already in progress https://review.openstack.org/396567 | 10:02 |
| yolanda | hi, good morning. Can I get reviews on https://review.openstack.org/394426 ? | 10:02 |
| ccamacho | shardy jaosorior.. Just to add some more info about the lint snag, we have upgraded to puppet lint 2.0.2. Here are the tests (https://review.openstack.org/#/q/branch:master+topic:puppet-lint_check) but not sure how I missed puppet-tripleo | 10:03 |
| ccamacho | Landed yesterday night.. | 10:03 |
| ccamacho | But missed that fix.. :( sorry for that | 10:03 |
| marios | shardy: thanks unset +2 as well and added comment, should we go back to v1 approach then matbu https://review.openstack.org/#/c/395234/4 | 10:04 |
| shardy | marios: I don't think the addition to the hierarchy works, so we need to document including environments/puppet-ceph-external.yaml on upgrade perhaps? | 10:05 |
| *** nijaba has quit IRC | 10:06 | |
| matbu | shardy: marios hey guys, the addition of the hiera file works, but it's a wrong fix | 10:06 |
| marios | shardy: but the error (about Could not find data item ceph::profile::params::client_keys in any Hiera data ) happens on compute nodes https://bugs.launchpad.net/tripleo/+bug/1640328 and as you said perhaps they don't have ceph or even external ceph at that | 10:06 |
| openstack | Launchpad bug 1640328 in tripleo "[Upgrade] Could not find data item ceph::profile::params::client_keys in any Hiera data" [High,In progress] - Assigned to mbu (mat-bultel) | 10:06 |
| *** abregman is now known as abregman|nb | 10:07 | |
| shardy | marios: Yeah on compute it should be provided by OS::TripleO::Services::CephClient | 10:07 |
| *** nijaba has joined #tripleo | 10:07 | |
| *** nijaba has quit IRC | 10:07 | |
| *** nijaba has joined #tripleo | 10:07 | |
| shardy | in the non-external case | 10:07 |
| *** pmannidi has quit IRC | 10:08 | |
| shardy | matbu: yeah I think it works because it's reading old hieradata, but that would mean any future config changes are ignored | 10:09 |
| matbu | actually i got another issue result of fixing this bug. I'm off today but i'll try to update this review during the week end | 10:09 |
| marios | matbu: ok thanks matbu forgot you are off today enjoy | 10:09 |
| shardy | matbu: ack Ok, we can revisit on Monday, thanks! | 10:09 |
| matbu | shardy: yes, btw, i was wondering if it's possible to remove the old hiera files ? | 10:09 |
| shardy | matbu: Yeah we should, but currently don't | 10:10 |
| matbu | shardy: i guess so, but | 10:10 |
| shardy | matbu: as part of the move to using dprince's hiera hook, I'd like to be able to purge all hieradata, and write it freshly each deploy | 10:10 |
| matbu | shardy: k, it would be better because, in the upgrade case it's a bit dangerous i think | 10:10 |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Normalize civetweb binding address if IPv6 https://review.openstack.org/395785 | 10:10 |
| shardy | matbu: yeah - it probably will work because of the hierarchy ordering, but I agree it's at best confusing, and potentially a source of bugs | 10:11 |
| matbu | shardy: yep | 10:11 |
| shardy | I'll raise a bug so we can track it | 10:12 |
| matbu | shardy: cool thanks | 10:12 |
| jaosorior | shardy: well... still don't figure out the issue... the signatures look fine and stuff doesn't seem to be changing http://paste.openstack.org/show/588927/ any idea where the AWSAccessKeyId comes from? | 10:15 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Correct opencontrail hiera interpolation key https://review.openstack.org/393292 | 10:17 |
| shardy | jaosorior: yes it's also provided from the keystone ec2tokens API | 10:17 |
| shardy | heat creates an ec2 keypair, then uses the key/secret for signing | 10:17 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Correct opencontrail hiera interpolation key https://review.openstack.org/393340 | 10:17 |
| shardy | same data is used by keystone as credentials for validating the signature | 10:17 |
| shardy | if the signatures match, I don't see why keystone would nack the validation of the signature? | 10:18 |
| shardy | all it does is compare the two values | 10:18 |
| openstackgerrit | Martin Mágr proposed openstack/tripleo-heat-templates: Use default Sensu redact https://review.openstack.org/394847 | 10:18 |
| jaosorior | shardy: cause at some point I see some other AWSAccessKeyId being used for that same resource | 10:19 |
| shardy | https://github.com/openstack/heat/blob/master/heat/api/aws/ec2token.py#L222 | 10:19 |
| shardy | you might want to comment out the except there and let the error blow up so you can inspect the backtrace | 10:19 |
| openstackgerrit | Tomas Sedovic proposed openstack/tripleo-docs: Add tripleo-validations user documentation https://review.openstack.org/396197 | 10:20 |
| shardy | jaosorior: the AWSAccessKeyId is associated with each deployment, not the server | 10:20 |
| shardy | so in a test stack with one deployment, the access key should always be the same | 10:21 |
| jaosorior | strange | 10:21 |
| jaosorior | there is onl;y one deployment | 10:21 |
| shardy | If the access key on validation is different that definitely explains it | 10:21 |
| shardy | but I don't understand how that happens, because heat calculates the signature, and pushes metadata right into the server | 10:21 |
| shardy | there's nowhere for the access key to get mangled there AFAICT | 10:21 |
| jaosorior | when the scripts to depoy over apache were written, was there any more documentation about it? | 10:23 |
| shardy | jaosorior: not sure tbh, you may want to git blame the wsgi stuff from the heat tree | 10:25 |
| shardy | seems like you're close to figuring it out tho - you just need to know exactly why it's hitting that execpt block | 10:25 |
| shardy | could even be we've got an unrelated Key/AttributeError tripping it | 10:25 |
| marios | shardy: newton backport please https://review.openstack.org/#/c/395460/ green ci run and master merged last night | 10:26 |
| marios | shardy: (if/when you get a chance) | 10:26 |
| shardy | marios: will do, wow quite a big patch | 10:29 |
| marios | shardy: yeah it is a scary looking patchh. there is discussion on master. it is just because it shifts everything over by one step to isolate the ng ha migration into step 1 | 10:29 |
| shardy | marios: just to clarify - the scripts are run in sequence right, so I can copy the logic out from those steps and into my composable PoC which runs per-step over all roles? | 10:30 |
| marios | shardy: so scary looking but actually not adding anything just shifting | 10:30 |
| marios | shardy: yeah we do it with depends on as usual like https://review.openstack.org/#/c/395460/3/extraconfig/tasks/major_upgrade_pacemaker.yaml | 10:31 |
| marios | shardy: i mean the steps | 10:31 |
| shardy | marios: Ack, thanks, so it will translate directly into the steps I defined | 10:31 |
| shardy | marios: jistr was asking for names instead of step numbers, what are your thoughts on that? | 10:31 |
| shardy | I'm leaning towards leaving the steps, as it's more consistent with the deployment steps | 10:31 |
| shardy | and closer to this existing implementation | 10:31 |
| marios | shardy: so bandini made a good point on one of the reviews (maybe this one... sec) | 10:31 |
| shardy | we could make it configurable, e.g have a list of steps for each role in roles_data.yaml | 10:32 |
| shardy | so adding a new step wouldn't be a big deal | 10:32 |
| shardy | perhaps do 10, 20, 30 to allow for that | 10:32 |
| shardy | Or, we could use names and define the ordering in a similar way | 10:32 |
| openstackgerrit | Raoul Scarazzini proposed openstack/tripleo-quickstart: Add swift and block to default node flavors https://review.openstack.org/396577 | 10:33 |
| marios | shardy: so names alone don't work.. we need the numbers too so we can enforce sequencing | 10:34 |
| shardy | marios: yeah, but we could use names combined with a list that determines ordering | 10:34 |
| marios | shardy: but then if we have name_number we would have to keep renaming things (this was the gist of bandini comments https://review.openstack.org/#/c/388649/ ) | 10:34 |
| shardy | amounts to the same thing, I just wanted to clarify the consensus re which folks prefer | 10:34 |
| shardy | before doing a second pass on my PoC patch | 10:35 |
| marios | shardy: right if we do names plus sequence we may ave to keep renaming things. leaving gaps like 10/20/30 is probably going to take care of that concern though | 10:35 |
| marios | shardy: so we don't have to rename 22KeystoneThing we just add 23AFterKeystoneThing | 10:35 |
| marios | shardy: (sorry bandini comments at Oct 25 1:03 PM ) | 10:36 |
| *** tosky has joined #tripleo | 10:36 | |
| shardy | marios: ack Ok we can go with 10/20/30 for now then | 10:37 |
| *** limao has quit IRC | 10:37 | |
| shardy | I'm thinking this will be statically defined per role in roles_data.yaml tho, do you think we need some way to allow services to plug in at arbitrary steps? | 10:38 |
| *** yamahata has quit IRC | 10:38 | |
| shardy | making it dynamically decide the steps at runtime will be more work I think, although maybe possible | 10:38 |
| jistr | what's the advantage of 10/20/30 over names though? | 10:38 |
| shardy | jistr: Just that it makes the ordering a bit more obvious/explicit | 10:39 |
| marios | shardy: 'plug in' to an overall upgrades workflow you mean? not clear what 'this' is either at tis point :) | 10:39 |
| jistr | ok | 10:39 |
| marios | jistr: the idea was to have names_number like INIT_10 or w/e | 10:39 |
| marios | jistr: so both | 10:39 |
| marios | jistr: the name bit is only for the humans though | 10:39 |
| openstackgerrit | Tomas Sedovic proposed openstack/tripleo-docs: Add tripleo-validations user documentation https://review.openstack.org/396197 | 10:40 |
| shardy | marios: by plug in I mean, if someone wants to add INIT_11 - is it reasonable to make them edit roles_data.yaml to add the new step? | 10:40 |
| shardy | or do we have to make it dynamic, e.g you just add INIT_11: foo to services/foo.yaml | 10:41 |
| shardy | then the step is automagically wired in | 10:41 |
| shardy | the former is what I've prototyped, the latter may be possible but will mean more work, and probably more expensive heat resources | 10:41 |
| shardy | like a ResourceChain of deployments or something | 10:41 |
| shardy | the other option is to roll up all the steps and drive the entire workflow via either mistral or ansible | 10:42 |
| marios | shardy: so it depends what the steps are that we are defining in roles data. is it the overall upgrades workflow ? I thought we would be doing the latter, i.e. tagging upgrade config/steps in the servies directly that would then get combined | 10:42 |
| shardy | heat basically becomes an ansible playbook generator in that case | 10:42 |
| marios | shardy: defining an overall start bit , including the stuff in order from all the services, and then any end bits. | 10:43 |
| marios | shardy: yeah so i thought that's what we were describing in the spec so far anyway | 10:43 |
| jistr | hmm yea one potential advantage of having the numbers is that we don't have to list the steps upfront, and have them automatically generated. (Maybe we could go 11_init instead of init_11 to make the ordering super easy...) | 10:43 |
| jistr | hopefully if we go 00 - 99 we won't run out of numbers :D | 10:43 |
| shardy | 000 - 999 ;) | 10:44 |
| jaosorior | shardy: where is that stacktrace supposed to appear? I see no sign of it in any of the logs :/ | 10:44 |
| marios | shardy: i mean that the core idea was to decentralise the config so it lives in each service and then 'the tripleo tooling' would combine those into the script or scripts that the operator would use to drive the upgrade | 10:44 |
| shardy | jaosorior: in the heat-api-cfn logs | 10:44 |
| marios | shardy: i mean https://review.openstack.org/#/c/392116/ (i'll update that to point to your protoypes in the spec too) | 10:44 |
| jaosorior | nope | 10:45 |
| b00tcat` | why are parameters passed twice to THT services (from puppet-tripleo and THT)? i.e. https://github.com/openstack/puppet-tripleo/blob/master/manifests/profile/base/snmp.pp#L36 and https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/services/snmp.yaml#L38 | 10:45 |
| marios | shardy: now having said all of that, i think at this point is 'whatever works' so if it is easier to define steps in roles_data for now do that | 10:45 |
| shardy | jaosorior: when we looked at a CI run for this recently, the move to httpd had somehow disabled all the cfn api logging | 10:45 |
| shardy | did that get fixed? | 10:45 |
| jaosorior | shardy: it hadn't been disabled, it just didn't get any calls | 10:46 |
| jaosorior | shardy: httpd does lazy initialization, so it hadn't been loaded yet | 10:46 |
| jaosorior | that was all | 10:46 |
| jaosorior | so yeah, it's now getting logs | 10:46 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: WIP prototyping composable upgrades with Heat+Ansible https://review.openstack.org/393448 | 10:47 |
| shardy | marios: ack, yeah my question is here (line 22) | 10:47 |
| shardy | https://review.openstack.org/#/c/393448/4/upgrade/major_upgrade_steps.j2.yaml | 10:47 |
| shardy | it's easier to use a fixed range of steps | 10:47 |
| shardy | but if we have to, we can allow it to be dynamic | 10:48 |
| shardy | I'll probably start with it fixed anyway | 10:48 |
| arxcruz | Is ooo support barbican ? | 10:48 |
| marios | shardy: makes sense thanks shary will try review that some more today and update the spec later | 10:48 |
| marios | shardy: we said we'd land the specs for O1 so should be doable by next week (spec i mean) | 10:49 |
| shardy | marios: ack, thanks - I'll re-review the spec later, and hopefully get another pass of the prototype up in the next few days | 10:49 |
| shardy | yeah hopefully we can agree the approach enough to land the spec on Monday | 10:49 |
| *** athomas has quit IRC | 10:50 | |
| panda | any idea why this is not alerting ? https://bugs.launchpad.net/tripleo/+bug/1641076 | 10:50 |
| openstack | Launchpad bug 1641076 in tripleo "CI: master job fail on undercloud install because of missing ceilometer-dbsync command" [Critical,Confirmed] | 10:50 |
| jaosorior | shardy: nope, it doesn't crash | 10:56 |
| *** athomas has joined #tripleo | 10:56 | |
| jaosorior | shardy: this is even the token it gets http://paste.openstack.org/raw/588932/ | 10:57 |
| shardy | jaosorior: so why is it logging from https://github.com/openstack/heat/blob/master/heat/api/aws/ec2token.py#L223 ? | 10:58 |
| openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Only start the deploy if the Heat stack isn't already in progress https://review.openstack.org/396567 | 10:59 |
| Ng | what the heck, the puppet-lint fix landed, but my reverify failed with the old error?! | 11:00 |
| *** owalsh has joined #tripleo | 11:00 | |
| jaosorior | makes no sense to me | 11:00 |
| jaosorior | then again, my knowledge of the heat code-base is no good either | 11:00 |
| shardy | jaosorior: well you seem to be really close now - it's going down the except (AttributeError, KeyError): path, why? | 11:01 |
| shardy | https://github.com/openstack/heat/blob/master/heat/api/aws/ec2token.py#L237 | 11:01 |
| shardy | if it goes down that path, the API request will work | 11:02 |
| *** dtantsur|afk is now known as dtantsur | 11:05 | |
| openstackgerrit | Arx Cruz proposed openstack/tripleo-quickstart: Enable cinder-backup service on minimal pacemaker https://review.openstack.org/396587 | 11:06 |
| *** d0ugal has quit IRC | 11:07 | |
| *** akrivoka has joined #tripleo | 11:09 | |
| *** ooolpbot has joined #tripleo | 11:10 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 11:10 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1641076 | 11:10 |
| *** ooolpbot has quit IRC | 11:10 | |
| openstack | Launchpad bug 1641076 in tripleo "CI: master job fail on undercloud install because of missing ceilometer-dbsync command" [Critical,Confirmed] | 11:10 |
| jaosorior | shardy: by the way, noticed the ec2 endpoint was being configured for heat using the controller IP instead of the actual endpoint https://review.openstack.org/#/c/396516/1 not a big deal, just thought it would be good to change it | 11:11 |
| jaosorior | shardy: aaand... it's weird, seems that for a tripleo deployment I'm seeing those authentication errors, but for the local deployment, that actually succeeds | 11:14 |
| jaosorior | s/local/minimal | 11:14 |
| jaosorior | however, the deployment is still stuck | 11:16 |
| openstackgerrit | Arx Cruz proposed openstack/tripleo-quickstart: Configuring vcpus for undercloud, controller and compute nodes https://review.openstack.org/396593 | 11:17 |
| openstackgerrit | Dan Sneddon proposed openstack/os-net-config: Stop dhclient in os-net-config if interface not set for DHCP https://review.openstack.org/396446 | 11:18 |
| shardy | jaosorior: you can do the same debug by copying the URL from the server metadata for a failed TripleO deployment | 11:22 |
| shardy | e.g curl the deploy URL | 11:22 |
| jaosorior | shardy: well, the minimal deployment is still stuck :/ so either way it's an issue | 11:23 |
| jaosorior | all I see from os-collect-config there is Nov 11 11:22:18 identity-base os-collect-config[2064]: /var/lib/os-collect-config/local-data not found. Skipping | 11:23 |
| jaosorior | Nov 11 11:22:18 identity-base os-collect-config[2064]: No local metadata found (['/var/lib/os-collect-config/local-data']) | 11:23 |
| shardy | jaosorior: did you set user_data_format: SOFTWARE_CONFIG on the server? | 11:24 |
| *** hewbrocca_afk is now known as hewbrocca | 11:25 | |
| jaosorior | yes | 11:25 |
| shardy | I'd focus on the curl call that didn't work | 11:25 |
| shardy | as by default os-collect-config will poll via the same API | 11:25 |
| shardy | so you would expect the server polling to be broken too | 11:25 |
| jaosorior | yeah, that's still not working | 11:26 |
| shardy | http://paste.openstack.org/show/588907/ | 11:26 |
| shardy | it's going down that "AWS authentication failure." path, so we just need to know why (printing the exception would probably help give some clues) | 11:27 |
| *** jprovazn has quit IRC | 11:28 | |
| jaosorior | shardy: yeah, the issue is that now that I went to using a more minimal install, I see authentication success in the logs | 11:28 |
| *** abregman|nb is now known as abregman|afk | 11:28 | |
| jaosorior | yes, using that URL, it still gets denied | 11:28 |
| *** coolsvap has quit IRC | 11:30 | |
| * shardy is confused | 11:30 | |
| * jaosorior is confused and frustrated by this | 11:31 | |
| b00tcat` | re. custom services: is there a thing like _get_nodes_by_role_ that I can use in tht? I mean, I would need some nodes with a certain role to be aware of themselves | 11:31 |
| openstackgerrit | Dan Sneddon proposed openstack/os-net-config: Stop dhclient in os-net-config if interface not set for DHCP https://review.openstack.org/396446 | 11:31 |
| shardy | jaosorior: In http://paste.openstack.org/show/588913/ you showed the curl request failing | 11:33 |
| shardy | so I'm confused when you now say the minimal example works | 11:33 |
| *** Baggypants12000 has quit IRC | 11:33 | |
| jaosorior | shardy: that's what's messed up. In the minimal example, the heat-cfn logs show "authorization success", however, the deployment is still stuck, and when I try to do the curl, that keeps failing like before | 11:34 |
| jaosorior | makes no sense | 11:34 |
| jaosorior | and I have no idea what else to tryu | 11:34 |
| shardy | so, the curl call fails, but you don't see "AWS authentication failure." in the logs? | 11:34 |
| jaosorior | yeah | 11:35 |
| shardy | Ok, so that's not consistent with your earlier data | 11:35 |
| jaosorior | which is why I'm so confused | 11:35 |
| jaosorior | all I added was debug logs | 11:36 |
| *** zoli|doctor is now known as zoli|wfh | 11:36 | |
| *** masco has quit IRC | 11:36 | |
| jaosorior | Anyway, I think my environment is messed up | 11:40 |
| jaosorior | shardy: this is how it's failing in the gate, which is what I had initially been able to reproduce http://logs.openstack.org/37/394837/4/check/gate-tripleo-ci-centos-7-nonha-multinode/8539cdf/logs/var/log/heat/heat-api_cfn.txt.gz#_2016-11-10_15_19_58_114 | 11:40 |
| shardy | jaosorior: I'd push a WIP patch to keystoneclient adding debug and recheck with a Depends-On perhaps | 11:41 |
| shardy | http://logs.openstack.org/37/394837/4/check/gate-tripleo-ci-centos-7-nonha-multinode/8539cdf/logs/var/log/keystone/keystone.txt.gz#_2016-11-10_14_48_20_576 | 11:42 |
| shardy | could that be related? | 11:42 |
| shardy | Doesn't seem to be getting as far as actually calculating the signature? | 11:42 |
| jaosorior | qnot sure | 11:43 |
| jaosorior | that error is quite common as far as I've seen' | 11:43 |
| *** fzdarsky|afk is now known as fzdarsky|lunch | 11:48 | |
| openstackgerrit | Julie Pichon proposed openstack/puppet-tripleo: Persist the UI config file https://review.openstack.org/388592 | 11:51 |
| openstackgerrit | vinay kumar muddu proposed openstack/diskimage-builder: Build wholedisk images capable of UEFI secure boot https://review.openstack.org/396602 | 11:51 |
| *** rhallisey has joined #tripleo | 11:55 | |
| *** hjensas has quit IRC | 12:00 | |
| -openstackstatus- NOTICE: Our OpenStack CI systems are stuck and no new jobs are submitted. Please do not recheck - and do not approve changes until this is fixed. | 12:02 | |
| *** spredzy has quit IRC | 12:03 | |
| *** maticue has joined #tripleo | 12:07 | |
| *** ooolpbot has joined #tripleo | 12:10 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 12:10 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1641076 | 12:10 |
| openstack | Launchpad bug 1641076 in tripleo "CI: master job fail on undercloud install because of missing ceilometer-dbsync command" [Critical,Confirmed] | 12:10 |
| *** ooolpbot has quit IRC | 12:10 | |
| *** lucasagomes is now known as lucas-hungry | 12:10 | |
| *** hjensas has joined #tripleo | 12:13 | |
| *** shardy is now known as shardy_lunch | 12:13 | |
| openstackgerrit | Oliver Walsh proposed openstack/diskimage-builder: Include lsb_release on redhat/centos7 images https://review.openstack.org/396607 | 12:16 |
| *** hjensas has quit IRC | 12:17 | |
| *** hjensas has joined #tripleo | 12:18 | |
| *** jkilpatr has joined #tripleo | 12:22 | |
| *** hjensas has quit IRC | 12:23 | |
| *** hjensas has joined #tripleo | 12:25 | |
| *** dprince has joined #tripleo | 12:26 | |
| *** dougbtv has quit IRC | 12:26 | |
| *** adarazs is now known as adarazs_lunch | 12:27 | |
| *** thrash|g0ne is now known as thrash | 12:30 | |
| *** milan has joined #tripleo | 12:31 | |
| *** jprovazn has joined #tripleo | 12:32 | |
| *** ccamacho has quit IRC | 12:37 | |
| *** hjensas has quit IRC | 12:38 | |
| *** hjensas has joined #tripleo | 12:39 | |
| *** hjensas has quit IRC | 12:39 | |
| *** hjensas has joined #tripleo | 12:39 | |
| *** lblanchard has joined #tripleo | 12:39 | |
| jaosorior | shardy_lunch: so the "reason" it' | 12:41 |
| jaosorior | it's spitting out in the logs is | 12:41 |
| jaosorior | auth fail reason: The request you have made requires authentication.. | 12:41 |
| jaosorior | shardy_lunch: so you might be right, that it doesn't contain tyhe token and it doesn't even get to the signature calculation | 12:42 |
| *** fzdarsky|lunch is now known as fzdarsky | 12:44 | |
| *** hjensas has quit IRC | 12:45 | |
| *** jaosorior has quit IRC | 12:47 | |
| *** pkovar has joined #tripleo | 12:50 | |
| *** hjensas has joined #tripleo | 12:51 | |
| *** hjensas has quit IRC | 12:51 | |
| *** hjensas has joined #tripleo | 12:51 | |
| *** arxcruz has quit IRC | 12:51 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/puppet-tripleo: Add panko service support https://review.openstack.org/396435 | 12:56 |
| openstackgerrit | Pradeep Kilambi proposed openstack/puppet-tripleo: Fix barbican server name to not use aodh hiera https://review.openstack.org/396434 | 12:57 |
| *** pabelanger has quit IRC | 12:57 | |
| *** pabelanger has joined #tripleo | 12:57 | |
| -openstackstatus- NOTICE: Our OpenStack CI system is coming back online again. Thanks for your patience. | 13:01 | |
| *** trown|outtypewww is now known as trown | 13:01 | |
| *** jayg|g0n3 is now known as jayg | 13:07 | |
| openstackgerrit | Merged openstack/tripleo-quickstart: Force the use of python version 2.7 https://review.openstack.org/368013 | 13:07 |
| *** ooolpbot has joined #tripleo | 13:10 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 13:10 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1641076 | 13:10 |
| openstack | Launchpad bug 1641076 in tripleo "CI: master job fail on undercloud install because of missing ceilometer-dbsync command" [Critical,Confirmed] | 13:10 |
| *** ooolpbot has quit IRC | 13:10 | |
| panda | will a ssl + ipv6 deployment work ? | 13:14 |
| *** adarazs_lunch is now known as adarazs | 13:19 | |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates: Add fail_mode workaround for connectivity issues on reboot https://review.openstack.org/396285 | 13:20 |
| *** arxcruz has joined #tripleo | 13:22 | |
| *** arxcruz has quit IRC | 13:27 | |
| *** ecerquei has joined #tripleo | 13:27 | |
| *** arxcruz has joined #tripleo | 13:27 | |
| *** lucas-hungry is now known as lucasagomes | 13:29 | |
| *** phpcodemonkey has joined #tripleo | 13:31 | |
| florianf | My current deployment seems to have literally fallen asleep at AllNodesDeploySteps. It's calling heat.engine.scheduler._sleep every couple of seconds. | 13:31 |
| *** hrybacki is now known as hrybacki|VDayRit | 13:31 | |
| florianf | I wonder if it's ever going to wake up from that. | 13:31 |
| openstackgerrit | Brent Eagles proposed openstack/puppet-tripleo: Call VF configuration from udev rules https://review.openstack.org/394604 | 13:32 |
| *** rlandy has joined #tripleo | 13:34 | |
| openstackgerrit | Brent Eagles proposed openstack/os-net-config: Add support for enabling hotplug on interfaces https://review.openstack.org/394660 | 13:35 |
| *** phpcodemonkey has quit IRC | 13:35 | |
| panda | beagles: it's friday for the deployments too. | 13:36 |
| *** ecerquei has quit IRC | 13:36 | |
| *** ecerquei has joined #tripleo | 13:36 | |
| *** jpena is now known as jpena|lunch | 13:36 | |
| *** zaneb has joined #tripleo | 13:37 | |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates: Increase reserved memory for computes when enabling DVR https://review.openstack.org/382594 | 13:40 |
| openstackgerrit | Chris Jones proposed openstack/puppet-tripleo: Fix puppet lint failure https://review.openstack.org/396631 | 13:42 |
| *** ecerquei has quit IRC | 13:42 | |
| *** zaneb has quit IRC | 13:42 | |
| Ng | ^^ will get puppet-tripleo CI moving again on newton. It's a straight backport of bnemec's fix from master | 13:43 |
| Ng | I (and some others I suspect :) would very much appreciate it if a couple of cores could help land it :) | 13:43 |
| *** rbowen has quit IRC | 13:44 | |
| Ng | slagle: :) | 13:45 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Fix race during major-upgrade-pacemaker step https://review.openstack.org/395460 | 13:51 |
| *** cdearborn has joined #tripleo | 13:52 | |
| *** tzumainn has joined #tripleo | 13:53 | |
| *** tiswanso has joined #tripleo | 13:53 | |
| *** ecerquei has joined #tripleo | 13:53 | |
| *** ecerquei has quit IRC | 13:54 | |
| *** ecerquei has joined #tripleo | 13:54 | |
| *** amoralej is now known as amoralej|lunch | 13:54 | |
| *** ecerquei has quit IRC | 13:57 | |
| *** ealcaniz has quit IRC | 13:57 | |
| *** fultonj has joined #tripleo | 13:58 | |
| *** dprince has quit IRC | 14:00 | |
| *** dprince has joined #tripleo | 14:01 | |
| *** dciabrin has quit IRC | 14:02 | |
| jpich | Here's a couple of patches under 3 lines that'd be helpful, if folks have a bit of time to review: https://review.openstack.org/#/c/393719/ https://review.openstack.org/#/c/388592/ (puppet-tripleo, instack-undercloud with one +2 already) | 14:03 |
| *** zoli|wfh is now known as zoli|afk | 14:05 | |
| jtomasek | florianf: do we already have a bug or blueprint for using heat zaqar messages instead of polling? | 14:07 |
| *** abregman|afk is now known as abregman | 14:07 | |
| florianf | jtomasek: https://blueprints.launchpad.net/tripleo/+spec/ui-deployment-progress-websockets | 14:08 |
| jtomasek | florianf: nice, thanks | 14:08 |
| florianf | jtomasek: not very detailed, but it's there... | 14:08 |
| *** ooolpbot has joined #tripleo | 14:10 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 14:10 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1641076 | 14:10 |
| *** ooolpbot has quit IRC | 14:10 | |
| openstack | Launchpad bug 1641076 in tripleo "CI: master job fail on undercloud install because of missing ceilometer-dbsync command" [Critical,Confirmed] | 14:10 |
| *** abregman is now known as abregman|afk | 14:12 | |
| openstackgerrit | Ana Krivokapic proposed openstack/tripleo-ui: Replace direct reference of Mistral workflow with constant https://review.openstack.org/396643 | 14:12 |
| *** rhefner has joined #tripleo | 14:13 | |
| *** jkilpatr_ has joined #tripleo | 14:14 | |
| *** jkilpatr has quit IRC | 14:16 | |
| *** shardy_lunch is now known as shardy | 14:18 | |
| *** amoralej|lunch is now known as amoralej | 14:21 | |
| *** ccamacho has joined #tripleo | 14:21 | |
| ccamacho | Hey Ng, around? | 14:21 |
| ccamacho | quick question, from the backported patch | 14:21 |
| ccamacho | checking http://logs.openstack.org/92/396092/2/check/gate-puppet-tripleo-puppet-lint/324ea78/console.html | 14:21 |
| Ng | ccamacho: hey | 14:22 |
| ccamacho | the job its fetching 2016-11-11 13:25:18.923640 | Fetching https://git.openstack.org/openstack/puppet-openstack_spec_helper | 14:22 |
| ccamacho | without the branch | 14:22 |
| ccamacho | so its getting master instead of newton, right? | 14:22 |
| Ng | ccamacho: looks like it, yeah | 14:23 |
| ccamacho | 2016-11-11 13:26:59.320683 | Using puppet-openstack_spec_helper 10.0.0 from https://git.openstack.org/openstack/puppet-openstack_spec_helper (at master@6b7aa95) | 14:23 |
| ccamacho | :P | 14:23 |
| ccamacho | thats the actual issue then, not the backport | 14:23 |
| *** Vijayendra has quit IRC | 14:24 | |
| Ng | ccamacho: interesting, so presumably I need to track down whatever owns the gate-puppet-tripleo-puppet-lint jenkins job, and fix that? | 14:27 |
| ccamacho | Ng yeahp, Im trying to get it also but had no luck yet :) | 14:28 |
| *** jcoufal has joined #tripleo | 14:28 | |
| openstackgerrit | Tomas Sedovic proposed openstack/tripleo-docs: Add tripleo-validations user documentation https://review.openstack.org/396197 | 14:29 |
| *** hrybacki|VDayRit is now known as hrybacki | 14:29 | |
| *** gbarros has joined #tripleo | 14:30 | |
| *** abehl has quit IRC | 14:31 | |
| Ng | ccamacho: would it be the Gemfile in puppet-tripleo? That specifies to grab puppet-openstack_spec_helper from git, with no specific branch | 14:31 |
| ccamacho | I think yes :) | 14:31 |
| ccamacho | Ng but Im checking also other projects.. | 14:32 |
| ccamacho | and they are the same.. | 14:32 |
| Ng | puppet-nova has :branch => 'stable/newton', in its newton branch | 14:32 |
| Ng | if nova does it, I'm comforatble pushing up a patch for tripleo to do it ;) | 14:32 |
| trozet | shardy: hi, for OOO when is the next design summit? | 14:33 |
| ccamacho | Ng yeah! but I think they are also the same :( | 14:34 |
| Ng | ccamacho: the same in what way? | 14:34 |
| ccamacho | they are not configuring the branch name | 14:34 |
| shardy | trozet: I believe after some debates we were granted space at the PTG in Atlanta | 14:35 |
| openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart-extras: Add essential repo files https://review.openstack.org/396652 | 14:35 |
| Ng | ccamacho: sorry, I don't understand. Their gemfile in newton is specifying the newton spec_helper. or do you mean something else? | 14:35 |
| trozet | shardy: ok do you have any links to that? or dates? | 14:35 |
| shardy | trozet: when Emilien gets back from PTO I expect more details will be circulated, I've not got any communication about it since handing over PTL responsibilities | 14:35 |
| ccamacho | https://github.com/openstack/puppet-nova/blob/master/Gemfile and https://github.com/openstack/puppet-tripleo/blob/master/Gemfile i.e. | 14:36 |
| trozet | shardy: so Emilien is PTL now? | 14:36 |
| shardy | trozet: https://www.openstack.org/ptg/ | 14:36 |
| ccamacho | Ng^ | 14:36 |
| shardy | trozet: yes, for ocata he is | 14:36 |
| Ng | ccamacho: in master, yes, that seems right, since master is the default, but look at https://github.com/openstack/puppet-nova/blob/stable/newton/Gemfile | 14:36 |
| ccamacho | Ng hey! your right, need more coffee | 14:37 |
| ccamacho | :) | 14:37 |
| openstackgerrit | Tomas Sedovic proposed openstack/tripleo-validations: Fix the check-network-gateway validation https://review.openstack.org/396654 | 14:37 |
| ccamacho | that will be the fix then.. | 14:37 |
| openstackgerrit | Chris Jones proposed openstack/puppet-tripleo: Use stable/newton for puppet-openstack_spec_helper. https://review.openstack.org/396655 | 14:37 |
| trozet | shardy: ok cool. I'm not going to make it to Brno, but ATL is 1 hour plane ride for me...so will probably see you at that one if you go | 14:37 |
| ccamacho | Ng that would be the real bug :) | 14:37 |
| ccamacho | awesome | 14:37 |
| openstackgerrit | Tomas Sedovic proposed openstack/tripleo-validations: Fix the rabbitmq-limits validations https://review.openstack.org/396656 | 14:37 |
| openstackgerrit | Tomas Sedovic proposed openstack/tripleo-validations: Fix the pacemaker-status validation https://review.openstack.org/396657 | 14:38 |
| shadower | would appreciate reviews on the 3 patches above. They're just cherrypicks to tripleo-validations (all one-liners) | 14:40 |
| *** d0ugal has joined #tripleo | 14:41 | |
| *** dsariel has quit IRC | 14:42 | |
| *** jpena|lunch is now known as jpena | 14:43 | |
| ccamacho | Ng im manually checking all puppet repos here https://etherpad.openstack.org/p/gemfile_check just in case you want to check them also :) | 14:44 |
| Ng | ccamacho: I'll start at the bottom of the list :) | 14:45 |
| ccamacho | Ng awesome :) | 14:45 |
| openstackgerrit | Merged openstack/python-tripleoclient: Use correct region value https://review.openstack.org/394515 | 14:45 |
| *** udesale has quit IRC | 14:46 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Fix inconsistent Manila service naming https://review.openstack.org/396317 | 14:47 |
| *** dsariel has joined #tripleo | 14:50 | |
| *** dciabrin has joined #tripleo | 14:51 | |
| mwhahaha | gfidente: whats up? | 14:53 |
| bnemec | shadower: This should be backported, right? https://review.openstack.org/#/c/396317/ | 14:54 |
| bnemec | shardy: Same question^ Sorry shadower. | 14:55 |
| * bnemec needs more caffeine | 14:56 | |
| Ng | ccamacho: so now the question is, do we just blindly submit patches to each of those projects? :) | 14:57 |
| openstackgerrit | Brent Eagles proposed openstack/os-net-config: Add support for enabling hotplug on interfaces https://review.openstack.org/394660 | 14:57 |
| Ng | ccamacho: if so, I'm happy to rattle through them | 14:57 |
| ccamacho | Ng, the ones you tagged them as missing? | 14:58 |
| Ng | ccamacho: yeah | 14:58 |
| ccamacho | They are missing the branch tag? | 14:58 |
| Ng | yup | 14:58 |
| ccamacho | and they are using multiple branches? | 14:58 |
| *** zoli|afk is now known as zoli|wfh | 14:58 | |
| ccamacho | I know there are some only with master | 14:58 |
| shardy | bnemec: yes I think so, I was just waiting for it to land | 14:58 |
| Ng | ccamacho: oh that's a good point, I quickly scripted the check and didn't test for that. bah! | 14:59 |
| shardy | bnemec: there's a small risk users could be impacted, but I think it's OK since we're changing all the environment files too | 14:59 |
| shardy | and the current syntax could be really confusing | 14:59 |
| * shardy cherry-picks | 14:59 | |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Fix inconsistent Manila service naming https://review.openstack.org/396662 | 14:59 |
| ccamacho | Ng, I think they should be fine if only using master, but in case using multiple branches the branch tag should be there | 15:00 |
| bnemec | shardy: Yeah, definitely agree. | 15:00 |
| Ng | ccamacho: yeah, none of the MISSING ones have multiple branches, so looks like it's just puppet-tripleo :) | 15:00 |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Normalize civetweb binding address if IPv6 https://review.openstack.org/395785 | 15:00 |
| ccamacho | Ng awesome, maybe a snag when starting to use puppet-openstack_spec_helper :) | 15:01 |
| Ng | :) | 15:01 |
| ccamacho | missed puppet-tripleo | 15:01 |
| openstackgerrit | Arx Cruz proposed openstack/tripleo-heat-templates: Increasing neutron timeout for low memory usage https://review.openstack.org/390361 | 15:01 |
| ccamacho | good then | 15:01 |
| ccamacho | I8ll add https://etherpad.openstack.org/p/gemfile_check to the bug ticket | 15:01 |
| gfidente | mwhahaha, was trying to understand why previous patchset of https://review.openstack.org/395785 did not pass puppet 4 unit tests | 15:01 |
| Ng | ccamacho: thank you very much for your insights and help! | 15:01 |
| gfidente | social, helped looking into it and it's passing now, if you could review that would be nice | 15:01 |
| ccamacho | Ng no way! thanks to you for the help there :) | 15:01 |
| mwhahaha | gfidente: sure | 15:02 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Configure civetweb bind socket via puppet-tripleo https://review.openstack.org/390569 | 15:03 |
| *** dprince has quit IRC | 15:03 | |
| *** florianf has quit IRC | 15:03 | |
| hrybacki | hey all -- having issues with mistral while trying to delete a node on the overcloud: I've hit the end of my debugging abilities and posted https://bugs.launchpad.net/mistral/+bug/1640933 can anyone give me some additional troubleshooting pointers? #openstack-mistral believes it is likely a #tripleo issue | 15:03 |
| openstack | Launchpad bug 1640933 in Mistral "mistral raises 'Enviornment not found' for id of an existing stack while attempting to delete a node" [Undecided,New] | 15:03 |
| *** dprince has joined #tripleo | 15:04 | |
| d0ugal | hrybacki: Can you show me `mistral environment-list` ? | 15:07 |
| *** florianf has joined #tripleo | 15:07 | |
| hrybacki | d0ugal: https://paste.fedoraproject.org/477940/47887687/ | 15:07 |
| d0ugal | hrybacki: interesting - the environment seems to be there. | 15:08 |
| *** ebalduf has joined #tripleo | 15:09 | |
| *** dmacpher has quit IRC | 15:09 | |
| hrybacki | indeed strange. env details for the overcloud: https://paste.fedoraproject.org/477942/76947147/ | 15:09 |
| d0ugal | hrybacki: Can you show me the delete command you used? I don't see that in the bug | 15:09 |
| d0ugal | hrybacki: did you pass "--stack overcloud" ? | 15:09 |
| hrybacki | you know I didn't. I passed the stack id of the overcloud | 15:09 |
| * hrybacki fetches commadn | 15:09 | |
| hrybacki | openstack overcloud node delete --debug --stack 06b0022a-7819-41c3-a5f9-f097a880e905 --templates -e /usr/share/openstack-tripleo-heat-templates/environments/network-isolation.yaml -e /usr/share/openstack-tripleo-heat-templates/environments/net-single-nic-with-vlans.yaml -e /home/stack/network-environment.yaml c8167eae-3695-439c-953f-42cd3df89d93 | 15:10 |
| *** ooolpbot has joined #tripleo | 15:10 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 15:10 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1641076 | 15:10 |
| openstack | Launchpad bug 1641076 in tripleo "CI: master job fail on undercloud install because of missing ceilometer-dbsync command" [Critical,Confirmed] | 15:10 |
| *** ooolpbot has quit IRC | 15:10 | |
| hrybacki | it's in the bug, just sort of hidden by the other stuff | 15:10 |
| d0ugal | hrybacki: aha, I think it assumes the stack name. I don't know if the ID would work. | 15:10 |
| hrybacki | hrm, interesting | 15:10 |
| d0ugal | hrybacki: I have only tested this command with the name. | 15:10 |
| hrybacki | d0ugal: testing now w/ 'overcloud' | 15:12 |
| *** milan has quit IRC | 15:12 | |
| d0ugal | hrybacki: cool, thanks. let me know how you get on | 15:13 |
| d0ugal | hrybacki: FWIW, it will actually default to 'overcloud' soon. https://review.openstack.org/#/c/395522/ | 15:13 |
| d0ugal | (well, soon = in Ocata) | 15:13 |
| hrybacki | d0ugal: that's it. | 15:13 |
| d0ugal | hrybacki: Great. So I guess we need to update your bug to give people a more sane error. | 15:14 |
| hrybacki | ack, updating now | 15:14 |
| rook-out | hey shardy not sure the proper order of operations here but I added boot_option to ironic-inspector for puppet to clear out this bug https://bugs.launchpad.net/tripleo/+bug/1639858, here is my patch https://review.openstack.org/#/c/396667/ -- should i open a new bug against puppet-ironic to expose this option? | 15:16 |
| openstack | Launchpad bug 1639858 in tripleo "[RFE] Enable by default boot_mode in inspector.conf" [Undecided,New] | 15:16 |
| rook-out | Once my patch merges, we can close out the bug i reference ^ | 15:17 |
| d0ugal | hrybacki: Thanks. Should be fairly easy to fix, I'll try on Monday. | 15:17 |
| panda | derekh: (tripleo-ci) what does this call do ? curl http://$MIRRORSERVER/cgi-bin/upload.cgi -F "repohash=$TRUNKREPOUSED" -F "$JOB_NAME=SUCCESS" | 15:18 |
| hrybacki | d0ugal: added a comment to the bug. Would you like me to adjust the title or is it fine? | 15:19 |
| panda | derekh: updates status page ? | 15:19 |
| *** yamahata has joined #tripleo | 15:19 | |
| derekh | panda: At the end of the successfull periodic job it tells the mirror server that the jab names $JOB_NAME passed | 15:19 |
| d0ugal | hrybacki: I'll do a quick update | 15:20 |
| panda | derekh: and what does the mirror server do in response ? | 15:20 |
| derekh | panda: then on the mirror server there is a script that promotes the current repository if all the jobs passed | 15:20 |
| dtantsur | rook-out, reviewed. bug is only needed if you want it backported for some reason | 15:20 |
| derekh | panda: here is the scipt its calling http://git.openstack.org/cgit/openstack-infra/tripleo-ci/tree/scripts/mirror-server/upload.cgi | 15:21 |
| hrybacki | d0ugal: okay, thanks! /me is happy to have found the root cause | 15:21 |
| rook-out | dtantsur: ok | 15:21 |
| derekh | panda: and here is the script that us run by the cron job http://git.openstack.org/cgit/openstack-infra/tripleo-ci/tree/scripts/mirror-server/promote.sh | 15:21 |
| *** abregman|afk has quit IRC | 15:22 | |
| dtantsur | rook-out, as to https://bugs.launchpad.net/tripleo/+bug/1639858, you'll also need an instack-undercloud patch to change the value | 15:22 |
| openstack | Launchpad bug 1639858 in tripleo "[RFE] Enable by default boot_mode in inspector.conf" [Undecided,New] | 15:22 |
| derekh | panda: the cron job is defined at the bottom of this file http://git.openstack.org/cgit/openstack-infra/tripleo-ci/tree/scripts/mirror-server/mirror-server.pp#n46 | 15:22 |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Normalize civetweb binding address if IPv6 https://review.openstack.org/395785 | 15:22 |
| *** dougbtv has joined #tripleo | 15:23 | |
| panda | derekh: I understand, thanks. | 15:24 |
| gfidente | mwhahaha, updated, there is this guy which depends on https://review.openstack.org/#/c/390569/ | 15:26 |
| gfidente | *depends on it | 15:26 |
| mwhahaha | gfidente: ok | 15:26 |
| *** liverpooler has joined #tripleo | 15:27 | |
| *** dprince has quit IRC | 15:27 | |
| gfidente | mwhahaha, thanks | 15:27 |
| ccamacho | gfidente mwhahaha mind to check when having some cycles https://review.openstack.org/#/c/396655/ ?? Its related to add stable/newton to the gemfile in puppet-tripleo (stable/newton) currently blocking backports. | 15:34 |
| ccamacho | A 1 liner fix | 15:34 |
| ccamacho | thanks! | 15:35 |
| hrybacki | adarazs: jpena bkero dmsimard panda trown weshay please take a few minutes and add your thoughts to the rdo-infra retrospective board before the meeting: https://trello.com/b/0VFswmht/rdo-infra-retrospective thanks! | 15:35 |
| dmsimard | hrybacki: when's that meeting ? | 15:35 |
| hrybacki | dmsimard: in ~55 minutes | 15:36 |
| dmsimard | hrybacki: holiday in Canada today :( | 15:36 |
| trown | hrybacki: wrong channel? | 15:37 |
| trown | hrybacki: fine here, but rdo-infra seems like a #rdo topic :) | 15:37 |
| hrybacki | trown: should have hit up #rdo yes, wrong channel :P | 15:37 |
| hrybacki | sorry for spam #tripleo ! | 15:38 |
| *** liverpooler has quit IRC | 15:40 | |
| hrybacki | dmsimard: no worries! | 15:40 |
| openstackgerrit | Merged openstack/puppet-tripleo: Ensure keepalived is restarted when necessary. https://review.openstack.org/395053 | 15:43 |
| *** numans has quit IRC | 15:45 | |
| *** chandankumar has quit IRC | 15:45 | |
| Ng | ccamacho: jenkins is back and green for https://review.openstack.org/#/c/396655/1 :) | 15:45 |
| *** fragatina has quit IRC | 15:45 | |
| ccamacho | Ng done :) | 15:45 |
| Ng | \o/ | 15:45 |
| *** fragatina has joined #tripleo | 15:46 | |
| slagle | pradk: hi, could you work on a patch for tripleo-docs about the enable_telemetry flip flop we had to do for newton? | 15:50 |
| slagle | pradk: and the impact, etc | 15:50 |
| pradk | slagle, sure | 15:50 |
| slagle | shardy: pradk : fyi, there was some concern from the stable release team about these patches, and it's holding up our newton release | 15:50 |
| slagle | https://review.openstack.org/#/c/391799/ | 15:50 |
| slagle | see the comments on the patch from tony | 15:50 |
| *** jprovazn has quit IRC | 15:51 | |
| pradk | slagle, yea reading now.. so what we need is this behavior documented on what the default is and how to switch if needed? | 15:52 |
| *** iranzo has quit IRC | 15:52 | |
| slagle | pradk: yes, and particularly if they happened to upgrade to the first newton release, they got the new default (false), and now we're switching that back to true | 15:53 |
| beagles | slagle: dprince: dsneddon: etc. I've added you as reviewers to a couple of patches aimed at https://bugs.launchpad.net/tripleo/+bug/1640812 ... | 15:53 |
| openstack | Launchpad bug 1640812 in tripleo "Network connectivity lost on node reboot" [Critical,In progress] - Assigned to Brent Eagles (beagles) | 15:53 |
| pradk | slagle, understood, i'll get a docs patch out today | 15:53 |
| slagle | thanks! | 15:54 |
| beagles | it's kind of a weird (yet monstrous) issue and the workaround may seem a bit odd... I'm concerned about corner cases, unexpected consequences etc. | 15:54 |
| shardy | slagle: ack, thanks for the update | 15:55 |
| shardy | hopefully some docs will prove sufficient to unblock things | 15:56 |
| beagles | I'm enumerating different configuration types, looking for problems with this approach, but I know I'll miss some | 15:56 |
| *** dsariel has quit IRC | 16:01 | |
| *** ebarrera has quit IRC | 16:01 | |
| *** rcernin has quit IRC | 16:05 | |
| *** zaneb has joined #tripleo | 16:05 | |
| yolanda | hi, can i get some reviews on https://review.openstack.org/394426 ? to support whole disk images in tripleo client | 16:06 |
| ansiwen | mwhahaha: what do you mean with release notes? a comment which says which release is required for that option? | 16:07 |
| trown | yolanda: added myself to that review, I will take a look this afternoon | 16:07 |
| mwhahaha | ansiwen: a release note using reno so people who read our release notes will see the added option, http://docs.openstack.org/developer/reno/ | 16:08 |
| yolanda | thanks | 16:08 |
| *** ooolpbot has joined #tripleo | 16:10 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 16:10 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1641076 | 16:10 |
| openstack | Launchpad bug 1641076 in tripleo "CI: master job fail on undercloud install because of missing ceilometer-dbsync command" [Critical,Confirmed] | 16:10 |
| *** ooolpbot has quit IRC | 16:10 | |
| shardy | jistr: Hey can we close https://bugs.launchpad.net/tripleo/+bug/1635606 or are more patches needed? | 16:10 |
| openstack | Launchpad bug 1635606 in tripleo "Lost ability to deploy Glance backed by NFS" [High,In progress] - Assigned to Jiřà Stránský (jistr) | 16:10 |
| shardy | I started setting the topic for patches associated with bugs targetted at ocata-1 | 16:11 |
| jistr | shardy: ah yes, thanks for the reminder. I set it to fix-commited | 16:11 |
| shardy | https://review.openstack.org/#/q/topic:tripleo/ocata-1 | 16:11 |
| shardy | over the next few days we can land as many of the bugfixes as we can, then I'll defer any non-critical issues on Tuesday next week | 16:12 |
| shardy | jistr: thanks! | 16:12 |
| *** paramite has quit IRC | 16:13 | |
| *** fragatina has quit IRC | 16:16 | |
| shardy | slagle: hey can we declare https://bugs.launchpad.net/tripleo/+bug/1637204 complete for ocata-1 ? | 16:16 |
| openstack | Launchpad bug 1637204 in tripleo "Parameter 'CephStorageCount' was not found in existing stack when using composable roles" [High,In progress] - Assigned to James Slagle (james-slagle) | 16:16 |
| openstackgerrit | Arx Cruz proposed openstack/tripleo-quickstart: Configuring vcpus for undercloud, controller and compute nodes https://review.openstack.org/396593 | 16:17 |
| *** tremble has quit IRC | 16:18 | |
| *** ramishra has quit IRC | 16:18 | |
| *** ramishra has joined #tripleo | 16:20 | |
| *** bnemec is now known as beekneemech | 16:22 | |
| *** ealcaniz has joined #tripleo | 16:22 | |
| *** ohamada has quit IRC | 16:25 | |
| slagle | shardy: i declare it | 16:27 |
| openstackgerrit | Merged openstack/tripleo-validations: Fix the rabbitmq-limits validations https://review.openstack.org/396656 | 16:27 |
| shardy | slagle: ack, thanks! | 16:27 |
| *** mcornea has quit IRC | 16:27 | |
| openstackgerrit | Merged openstack/tripleo-validations: Fix the pacemaker-status validation https://review.openstack.org/396657 | 16:28 |
| openstackgerrit | Merged openstack/tripleo-validations: Fix the check-network-gateway validation https://review.openstack.org/396654 | 16:28 |
| *** ealcaniz has quit IRC | 16:28 | |
| *** abregman has joined #tripleo | 16:29 | |
| openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart: Revert "Force the use of python version 2.7" https://review.openstack.org/396689 | 16:30 |
| openstackgerrit | Pradeep Kilambi proposed openstack/tripleo-docs: Explicitly document enable_telemetry defaults in upgrade docs https://review.openstack.org/396690 | 16:30 |
| openstackgerrit | Merged openstack/tripleo-validations: undercloud-tokenflush: changed group to pre-introspection https://review.openstack.org/396212 | 16:31 |
| *** arxcruz has quit IRC | 16:31 | |
| openstackgerrit | Merged openstack/tripleo-quickstart: Revert "Force the use of python version 2.7" https://review.openstack.org/396689 | 16:34 |
| openstackgerrit | Pradeep Kilambi proposed openstack/tripleo-heat-templates: Add panko api support to service templates https://review.openstack.org/396439 | 16:34 |
| *** abregman is now known as abregman|nb | 16:35 | |
| openstackgerrit | Tomas Sedovic proposed openstack/tripleo-docs: Add tripleo-validations user documentation https://review.openstack.org/396197 | 16:36 |
| *** pkovar has quit IRC | 16:37 | |
| openstackgerrit | Merged openstack/puppet-tripleo: Use stable/newton for puppet-openstack_spec_helper. https://review.openstack.org/396655 | 16:38 |
| *** jlinkes has quit IRC | 16:42 | |
| *** abregman|nb is now known as abregman | 16:43 | |
| openstackgerrit | Alejandro Andreu proposed openstack/tripleo-heat-templates: WIP: Create MidoNet services https://review.openstack.org/396692 | 16:43 |
| *** Guest7605 has quit IRC | 16:43 | |
| openstackgerrit | Dougal Matthews proposed openstack/python-tripleoclient: Format the nodes list in openstack overcloud delete node https://review.openstack.org/395083 | 16:45 |
| openstackgerrit | Attila Darazs proposed openstack/tripleo-quickstart: WIP: shell script to merge roles into extras https://review.openstack.org/396694 | 16:51 |
| *** hewbrocca is now known as hewbrocca_afk | 16:53 | |
| openstackgerrit | Merged openstack/instack-undercloud: Remove instack-create-overcloudrc https://review.openstack.org/388823 | 16:55 |
| *** pkovar has joined #tripleo | 16:56 | |
| *** david-lyle has quit IRC | 16:58 | |
| *** lmiccini has quit IRC | 16:58 | |
| *** lblanchard has quit IRC | 16:59 | |
| *** lmiccini has joined #tripleo | 16:59 | |
| *** panda is now known as panda|bbl | 17:00 | |
| *** rajinir has joined #tripleo | 17:01 | |
| *** abregman is now known as abregman|afk | 17:03 | |
| *** jprovazn has joined #tripleo | 17:07 | |
| *** gbarros has quit IRC | 17:08 | |
| openstackgerrit | Oliver Walsh proposed openstack/diskimage-builder: In disk-image-create, append to INSTALL_PACKAGES instead of clobbering. This allows -p <packagelist> to be used multiple times. https://review.openstack.org/396702 | 17:09 |
| *** ooolpbot has joined #tripleo | 17:10 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 17:10 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1641076 | 17:10 |
| *** ooolpbot has quit IRC | 17:10 | |
| openstack | Launchpad bug 1641076 in tripleo "CI: master job fail on undercloud install because of missing ceilometer-dbsync command" [Critical,Confirmed] | 17:10 |
| openstackgerrit | Oliver Walsh proposed openstack/diskimage-builder: In disk-image-create, append to INSTALL_PACKAGES instead of clobbering. https://review.openstack.org/396702 | 17:12 |
| *** dbecker has quit IRC | 17:12 | |
| *** jkilpatr_ has quit IRC | 17:12 | |
| openstackgerrit | Merged openstack/instack-undercloud: Open firewall port for the TripleO UI https://review.openstack.org/393719 | 17:13 |
| *** abregman|afk is now known as abregman|nb | 17:14 | |
| *** gbarros has joined #tripleo | 17:15 | |
| *** jpich has quit IRC | 17:16 | |
| openstackgerrit | Merged openstack/os-net-config: Add route_options parameter https://review.openstack.org/388916 | 17:16 |
| marios | shardy: hey around? | 17:18 |
| shardy | marios: hey, yup! | 17:18 |
| *** dprince has joined #tripleo | 17:19 | |
| *** ecerquei has joined #tripleo | 17:19 | |
| marios | shardy: hi... are we still backporting things for liberty? we have an ovs workaround which we backported to mitaka already but been told we need it in liberty too (and also kilo but ignoring that for now) | 17:19 |
| marios | shardy: https://bugs.launchpad.net/tripleo/+bug/1636748 and https://bugs.launchpad.net/tripleo/+bug/1635205 together (2 patches) make the fix | 17:19 |
| openstack | Launchpad bug 1636748 in tripleo "Mitaka to Newton upgrade ovs upgrade check failed on ovs 2.5" [Undecided,Fix released] - Assigned to Marios Andreou (marios-b) | 17:19 |
| openstack | Launchpad bug 1635205 in tripleo "Mitaka to Newton upgrade - ovs 2.4 to 2.5 causes nodes to lose ovs interfaces" [Critical,Fix released] - Assigned to Marios Andreou (marios-b) | 17:20 |
| marios | shardy: i have added mitaka-backport-potential to those and done the backport and it landed. is it just liberty-backport-potential? I ask because the tag wasn't found when i just tried to add it | 17:20 |
| openstackgerrit | Julie Pichon proposed openstack/instack-undercloud: Open firewall port for the TripleO UI https://review.openstack.org/396706 | 17:20 |
| shardy | marios: we can definitely backport them to newton/mitaka, but liberty goes EOL next week | 17:21 |
| shardy | So you may decide not to backport there | 17:22 |
| shardy | https://releases.openstack.org/ | 17:22 |
| marios | shardy: ok thanks | 17:22 |
| *** michapma_alt has quit IRC | 17:23 | |
| *** dtantsur is now known as dtantsur|afk | 17:24 | |
| openstackgerrit | Jiri Tomasek proposed openstack/tripleo-specs: GUI Deployment Configuration improvements https://review.openstack.org/393365 | 17:24 |
| dprince | a quick review here would help me: https://review.openstack.org/#/c/396175/ | 17:26 |
| dprince | passing CI ^^^ | 17:26 |
| *** jkilpatr_ has joined #tripleo | 17:26 | |
| *** b00tcat` has quit IRC | 17:30 | |
| trown | dprince: so currently if role_data is null, we get data=null and the expression breaks somehow, but your updated wrapping in an extra dict we get data={role_data: null} and it works? | 17:32 |
| trown | oh or does it just break immediately when trying to pass null as "data" but passing a dict with the null as a value works? | 17:34 |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-common: Only use parameter_defaults for scaling down https://review.openstack.org/396712 | 17:36 |
| akrivoka | jtomasek: still here? | 17:39 |
| *** derekh has quit IRC | 17:42 | |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-common: Only use parameter_defaults for scaling down https://review.openstack.org/396712 | 17:43 |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: WIP prototyping composable upgrades with Heat+Ansible https://review.openstack.org/393448 | 17:43 |
| *** shardy has quit IRC | 17:43 | |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: WIP prototyping composable upgrades with Heat+Ansible https://review.openstack.org/393448 | 17:44 |
| *** zaneb has quit IRC | 17:47 | |
| openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: POC: Use tripleo-repos in tripleo.sh https://review.openstack.org/395813 | 17:48 |
| *** lucasagomes is now known as lucas-afk | 17:48 | |
| *** zaneb has joined #tripleo | 17:49 | |
| *** sudipto has quit IRC | 17:51 | |
| *** sudipto_ has quit IRC | 17:51 | |
| *** yamahata has quit IRC | 17:55 | |
| *** Baggypants12000 has joined #tripleo | 17:55 | |
| *** d0ugal has quit IRC | 17:57 | |
| jtomasek | akrivoka: now | 17:58 |
| *** david-lyle has joined #tripleo | 17:58 | |
| akrivoka | jtomasek: so I'm trying to understand how the new plan import feature would differ from simple plan creation (what we have now). with the new plan import, the configuration of environments and parameters would be stored so the user does not have to manually set them? correct? is that the point? | 18:00 |
| *** Goneri has joined #tripleo | 18:01 | |
| *** pkovar has quit IRC | 18:01 | |
| *** tiswanso has quit IRC | 18:01 | |
| *** tosky has quit IRC | 18:02 | |
| *** david-lyle has quit IRC | 18:03 | |
| jtomasek | akrivoka: yes, correct. Import is a create, we just need to enhance plan creation so it consumes the mistral-environment.yaml (json) | 18:03 |
| jtomasek | (plan update needs to support this too) | 18:03 |
| akrivoka | jtomasek: cool, thanks, just wanted to check my understanding was correct | 18:04 |
| *** mhenkel has joined #tripleo | 18:04 | |
| jtomasek | akrivoka: np | 18:04 |
| akrivoka | jtomasek: now switch the computer off start pto-ing :) | 18:05 |
| *** chandankumar has joined #tripleo | 18:05 | |
| *** athomas has quit IRC | 18:06 | |
| *** trown is now known as trown|lunch | 18:06 | |
| *** bana_k has joined #tripleo | 18:07 | |
| openstackgerrit | Pierre Blanc proposed openstack/os-net-config: Add support for ethtool_opts option https://review.openstack.org/392697 | 18:08 |
| *** fragatina has joined #tripleo | 18:10 | |
| *** ooolpbot has joined #tripleo | 18:10 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 18:10 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1641076 | 18:10 |
| *** ooolpbot has quit IRC | 18:10 | |
| openstack | Launchpad bug 1641076 in tripleo "CI: master job fail on undercloud install because of missing ceilometer-dbsync command" [Critical,Confirmed] | 18:10 |
| *** rcernin has joined #tripleo | 18:11 | |
| *** rcernin has quit IRC | 18:12 | |
| *** d0ugal has joined #tripleo | 18:12 | |
| jtomasek | akrivoka: one more review reply is required:) | 18:13 |
| *** aufi_ has quit IRC | 18:15 | |
| *** tiswanso has joined #tripleo | 18:21 | |
| *** tiswanso has quit IRC | 18:22 | |
| *** tiswanso has joined #tripleo | 18:23 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/tripleo-docs: Explicitly document enable_telemetry defaults in upgrade docs https://review.openstack.org/396690 | 18:24 |
| *** tiswanso has quit IRC | 18:24 | |
| *** tiswanso has joined #tripleo | 18:25 | |
| *** fragatina has quit IRC | 18:27 | |
| *** fragatina has joined #tripleo | 18:28 | |
| *** rhallisey has quit IRC | 18:30 | |
| *** rhallisey has joined #tripleo | 18:31 | |
| *** mgould is now known as mgould|afk | 18:32 | |
| *** jpena is now known as jpena|off | 18:33 | |
| *** rasca has quit IRC | 18:34 | |
| *** yamahata has joined #tripleo | 18:34 | |
| *** bana_k has quit IRC | 18:35 | |
| *** david-lyle has joined #tripleo | 18:36 | |
| *** pblaho has quit IRC | 18:38 | |
| 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 | 18:38 |
| *** lmiccini has quit IRC | 18:42 | |
| *** yamahata has quit IRC | 18:44 | |
| *** yamahata has joined #tripleo | 18:45 | |
| slagle | beekneemech: wasn't telemetry disabled in the 5.0.0 release of i-u? | 18:47 |
| slagle | that's what the docs are meant to address, people who are on 5.0.0 | 18:47 |
| beekneemech | slagle: But it's not possible to install that anymore. The final newton release (presumably 5.0.something) flipped the default back. | 18:48 |
| openstackgerrit | Marios Andreou proposed openstack/puppet-tripleo: Ensure keepalived is restarted when necessary. https://review.openstack.org/396731 | 18:49 |
| beekneemech | We're basically documenting a fixed bug. | 18:49 |
| slagle | beekneemech: oh so you're saying we don't need the last part of the comment? | 18:49 |
| jrist | would love one more core eye on this if possible https://review.openstack.org/#/c/391259/1 | 18:49 |
| beekneemech | slagle: I'm not sure we need any of that comment at all. The only way you could run into it is if you upgraded from Mitaka to Newton rc2, which is not a supported release. | 18:51 |
| *** bana_k has joined #tripleo | 18:51 | |
| beekneemech | Unless I'm misunderstanding the intent of release candidates. | 18:51 |
| slagle | beekneemech: we did a real release for newton already, 5.0.0 | 18:52 |
| beekneemech | I guess there's a CD argument here, but if you're CD'ing you've already hit the bug and gotten passed it. | 18:52 |
| slagle | enable_telemetry=false was in that release | 18:53 |
| beekneemech | slagle: And how would you install that today? | 18:53 |
| beekneemech | Pick a random old dlrn repo? | 18:53 |
| slagle | not random, use rdo newton | 18:53 |
| beekneemech | Is RDO Newton not using the actual released version of TripleO/ | 18:53 |
| slagle | or use the release tarball, although i agree that seems unlikely to have been done | 18:53 |
| beekneemech | I guess maybe the problem here is we released instack-undercloud as part of an RC, but didn't tag the instack-undercloud version as RC? | 18:54 |
| beekneemech | So there's a mismatch between the package version and the "TripleO" version. | 18:55 |
| slagle | wasn't 5.0.0 a non-rc release of instack-undercloud? | 18:55 |
| beekneemech | slagle: I don't know. We released newton rc3 on Oct. 18: https://github.com/openstack/releases/commit/674605c37c495fd8f9090ba6f0a800e1eee8c270 | 18:57 |
| beekneemech | We merged the patch to change the default on Oct. 14: https://github.com/openstack/instack-undercloud/commit/8a738272c7f63ad7e6e00f0836c2d1793ea4f125 | 18:57 |
| beekneemech | Oh wait, but we had branched. | 18:57 |
| beekneemech | Crap, let me check stable/newton. | 18:58 |
| *** Goneri has quit IRC | 18:58 | |
| slagle | https://review.openstack.org/#/c/388799/ | 18:59 |
| *** jkilpatr_ has quit IRC | 18:59 | |
| beekneemech | Ah hell, that didn't merge until Oct. 31, which is after the final release was cut. | 18:59 |
| openstackgerrit | John Trowbridge proposed openstack/tripleo-quickstart: add doc regarding how to console into the libvirt guests https://review.openstack.org/392983 | 18:59 |
| *** trown|lunch is now known as trown | 19:00 | |
| *** jkilpatr has joined #tripleo | 19:01 | |
| beekneemech | slagle: Okay, I +2'd the docs patch. It looks like we did release with the changed default. :-/ | 19:01 |
| slagle | ok | 19:01 |
| slagle | i guess we should maybe hold off on approving the docs patch until the new release is actually done | 19:02 |
| *** d0ugal has quit IRC | 19:02 | |
| beekneemech | slagle: Did we not do another release last week? | 19:03 |
| openstackgerrit | Tim Rozet proposed openstack/tripleo-heat-templates: Fixes missing OVS Firewall config with OpenDaylight https://review.openstack.org/396734 | 19:03 |
| slagle | beekneemech: no. i requested a new one this week | 19:03 |
| beekneemech | slagle: Ah, okay. | 19:03 |
| * beekneemech should just stay out of release-related stuff | 19:03 | |
| slagle | this is one of the patches the release team wanted us to address | 19:03 |
| slagle | or would like us to anyway | 19:04 |
| slagle | we dont have the stable-follows-policy tag, but we should be aiming for it | 19:04 |
| beekneemech | +1 | 19:04 |
| openstackgerrit | Merged openstack/tripleo-quickstart: add doc regarding how to console into the libvirt guests https://review.openstack.org/392983 | 19:05 |
| *** eggmaster has quit IRC | 19:05 | |
| openstackgerrit | Merged openstack/tripleo-ui: Replace direct reference of Mistral workflow with constant https://review.openstack.org/396643 | 19:06 |
| *** pcaruana has quit IRC | 19:07 | |
| *** lblanchard has joined #tripleo | 19:07 | |
| openstackgerrit | Ben Nemec proposed openstack/tripleo-docs: Add Newton repo details https://review.openstack.org/396736 | 19:08 |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: [WIP] config for containerized-compute https://review.openstack.org/393348 | 19:09 |
| *** ooolpbot has joined #tripleo | 19:10 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 19:10 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1641076 | 19:10 |
| openstack | Launchpad bug 1641076 in tripleo "CI: master job fail on undercloud install because of missing ceilometer-dbsync command" [Critical,Confirmed] | 19:10 |
| *** ooolpbot has quit IRC | 19:10 | |
| trown | should we remove alert tag from ^ | 19:11 |
| trown | patch to fix should merge soon | 19:11 |
| trown | or do we wait until we see promote pass? | 19:12 |
| openstackgerrit | Andy Smith proposed openstack/tripleo-specs: Enable Oslo.Messaging Dual Backends https://review.openstack.org/396740 | 19:12 |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: [WIP] config for containerized-compute https://review.openstack.org/393348 | 19:18 |
| openstackgerrit | Merged openstack/puppet-tripleo: Enable TLS in the internal network for Cinder API https://review.openstack.org/392090 | 19:19 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Handle null role_data in services https://review.openstack.org/396175 | 19:20 |
| *** zoli|wfh is now known as zoli|gone | 19:21 | |
| openstackgerrit | Merged openstack/puppet-tripleo: Improve failed mysql node removal time in HA deploys. https://review.openstack.org/396092 | 19:23 |
| openstackgerrit | Merged openstack/puppet-tripleo: Normalize civetweb binding address if IPv6 https://review.openstack.org/395785 | 19:23 |
| *** zaneb has quit IRC | 19:24 | |
| *** fzdarsky is now known as fzdarsky|afk | 19:25 | |
| *** zaneb has joined #tripleo | 19:26 | |
| *** hjensas has quit IRC | 19:28 | |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates: Add fail_mode workaround for connectivity issues on reboot https://review.openstack.org/396285 | 19:30 |
| dtrainor | anyone got a sec to talk to me about NeutronNetworkType, NeutronTunnelTypes, how they relate, if they need to be of the same type/value, etc? | 19:33 |
| *** akrivoka has quit IRC | 19:34 | |
| *** gbarros has quit IRC | 19:37 | |
| *** hjensas has joined #tripleo | 19:37 | |
| *** gfidente has quit IRC | 19:41 | |
| beagles | dtrainor: the NeutronNetworkType (could be better named I admit) specifies types of tenant networks you want to allow neutron to create | 19:43 |
| *** rhallisey has quit IRC | 19:43 | |
| panda|bbl | trown: I believe the alert is needed also so people know they may have problems with patches, not only to attract fixers. | 19:43 |
| beagles | dtrainor: the tunnel types are basically gre and vxlan, and they are also tenant network types | 19:44 |
| dtrainor | understood. can they mix and match or do they need to be the same for both Network and Tunnel types? | 19:44 |
| dtrainor | (does it matter?) | 19:44 |
| trown | panda|bbl: k, in this case though it only affects promote... I just worry if the alert just becomes spam that is ignored | 19:45 |
| beagles | dtrainor: if you specify vxlan and/or gre, they must also appear in the NeutronNetworkType - tunnel types that aren't in the tenant networks is kind of meaningless and should probably error out | 19:45 |
| dtrainor | ok thanks. figured as much but wanted confirmation. sorry if i'm not too knowledgeable on the subject. | 19:45 |
| *** rajinir has quit IRC | 19:46 | |
| dtrainor | i saw an opportunity for a validation | 19:49 |
| *** rajinir has joined #tripleo | 19:49 | |
| beagles | dtrainor: tbh, considering how we are using it, I'm not sure it makes sense to have both parameters | 19:50 |
| *** gbarros has joined #tripleo | 19:50 | |
| dtrainor | oh yeah? | 19:51 |
| beagles | dtrainor: in any case all of those command line arguments are deprecated so that statement is kind of moot :) | 19:52 |
| dtrainor | actually you're right :) I remember seeing that. | 19:53 |
| *** Goneri has joined #tripleo | 19:53 | |
| *** jprovazn has quit IRC | 19:53 | |
| *** jprovazn has joined #tripleo | 19:55 | |
| *** tbonds has quit IRC | 20:06 | |
| *** bana_k has quit IRC | 20:07 | |
| beekneemech | panda|bbl: But people aren't going to have a problem with patches because it only affects an un-promoted version of ceilometer. | 20:09 |
| beekneemech | Since the patch is approved, I think we should drop the alert. | 20:09 |
| *** chandankumar has quit IRC | 20:10 | |
| beekneemech | I'm not entirely sure we should alert on master issues anyway. Critical is fine, but those aren't blockers. | 20:10 |
| *** ooolpbot has joined #tripleo | 20:10 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 20:10 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1641076 | 20:10 |
| openstack | Launchpad bug 1641076 in tripleo "CI: master job fail on undercloud install because of missing ceilometer-dbsync command" [Critical,Confirmed] - Assigned to Alfredo Moralejo (amoralej) | 20:10 |
| *** ooolpbot has quit IRC | 20:10 | |
| openstackgerrit | Mike Burns proposed openstack/tripleo-image-elements: Include lsb_release in the hosts element since it's used there. https://review.openstack.org/396759 | 20:11 |
| openstackgerrit | wes hayutin proposed openstack/tripleo-quickstart: [WIP] config for containerized-compute https://review.openstack.org/393348 | 20:12 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-puppet-elements: Disable old hiera.yaml o-a-c script https://review.openstack.org/392615 | 20:13 |
| *** amoralej is now known as amoralej|off | 20:13 | |
| *** panda|bbl is now known as panda | 20:20 | |
| *** bana_k has joined #tripleo | 20:20 | |
| openstackgerrit | Mike Burns proposed openstack/tripleo-image-elements: Include lsb_release in the hosts element since it's used there. https://review.openstack.org/396759 | 20:21 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Increasing neutron timeout for low memory usage https://review.openstack.org/390361 | 20:21 |
| *** bana_k has quit IRC | 20:25 | |
| openstackgerrit | Dan Sneddon proposed openstack/os-net-config: Stop dhclient in os-net-config if interface not set for DHCP https://review.openstack.org/396446 | 20:25 |
| panda | I need to take a look at the promotion diagram again ? why this package was only blocking promotion and e.g. redis was blocking the gates too ? | 20:25 |
| mburned | beekneemech: thanks for the quick review... | 20:26 |
| beekneemech | mburned: So I just realized that the yum-minimal example I linked to has a bug. | 20:28 |
| beekneemech | It's not referring to the pkg-map name correctly. | 20:29 |
| mburned | no worries... | 20:29 |
| mburned | not sure if this is the end fix or if it's https://review.openstack.org/#/c/396607/1 (though that will have the same issues, i expect...) | 20:29 |
| *** zaneb has quit IRC | 20:30 | |
| openstackgerrit | Ben Nemec proposed openstack/diskimage-builder: Install lsb package by map name instead of package name https://review.openstack.org/396762 | 20:31 |
| openstackgerrit | Mike Burns proposed openstack/tripleo-image-elements: Include lsb_release in the hosts element since it's used there. https://review.openstack.org/396759 | 20:31 |
| beekneemech | mburned: I prefer yours. The element that uses the package should list it as a dep. | 20:32 |
| mburned | beekneemech: yeah, i sorta agree too... | 20:32 |
| beekneemech | The funny thing is I just noticed this today because I ran a CI job that accidentally didn't include ceph and it failed on an lsb-release call. :-) | 20:32 |
| mburned | yeah, that's what caused this too | 20:34 |
| mburned | ci escalations... | 20:34 |
| *** abregman|nb is now known as abregman|afk | 20:37 | |
| *** zaneb has joined #tripleo | 20:39 | |
| trown | panda: it is because ceilometer removed the deprecated ceilometer-dbsync command... so even though fix is in puppet-ceilometer (which we follow master of in all CI), the break is in ceilometer itself which we have pinned | 20:39 |
| *** akrzos has quit IRC | 20:40 | |
| trown | panda: I am not familiar with the redis issue, so I dont have a reverse explanation there | 20:40 |
| trown | panda: was the issue in puppet-redis? | 20:40 |
| panda | trown: no | 20:41 |
| trown | oh right redis is in deps repo | 20:41 |
| trown | we can't pin deps repo, it is not a static repo like dlrn repos | 20:41 |
| *** dsariel has joined #tripleo | 20:41 | |
| *** akrzos has joined #tripleo | 20:41 | |
| openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Always use cached images in updates job https://review.openstack.org/386713 | 20:42 |
| openstackgerrit | John Trowbridge proposed openstack/tripleo-heat-templates: Increasing neutron timeout for low memory usage https://review.openstack.org/396768 | 20:42 |
| *** ccamacho has quit IRC | 20:44 | |
| panda | trown: why was ceilometer pinned ? or is pinning part of the promotion ? the command was deprecated a long time ago, do we need some tool that gathers all deprecation warnings so they can be addressed before it's too late ? | 20:44 |
| trown | panda: everything in dlrn is pinned that does not have a tripleo gate (all the things in current-tripleo repo list are "unpinned") | 20:45 |
| *** akrzos has quit IRC | 20:47 | |
| panda | trown: https://media.tenor.co/images/5fc0bd75396977ab93e0b3405edb5c4e/raw | 20:54 |
| panda | trown: ok, thanks. | 20:55 |
| trown | panda: is that zach galafanakis? what is that frome :) | 20:55 |
| *** florianf has quit IRC | 20:55 | |
| trown | oh hangover ;) | 20:56 |
| trown | hit me on my beeper | 20:56 |
| panda | trown: https://www.youtube.com/watch?v=l2qZY5WFOYg&t=1m22s | 20:57 |
| trown | those equations have integrals, which make no sense in discrete math used for blackjack calculations... just saying | 21:00 |
| *** ccamacho has joined #tripleo | 21:00 | |
| *** maticue has quit IRC | 21:03 | |
| *** akrzos has joined #tripleo | 21:04 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Enable internal TLS for Cinder API https://review.openstack.org/392095 | 21:04 |
| panda | trown: Fermat's theorem was proved with elliptic curves. | 21:04 |
| openstackgerrit | Brent Eagles proposed openstack/os-net-config: Add support for enabling hotplug on interfaces https://review.openstack.org/394660 | 21:05 |
| trown | panda: blackjack math is pretty simple counting though :), but props on Fermat's last theorem knowledge | 21:06 |
| *** dprince has quit IRC | 21:08 | |
| panda | now if only I could get this netiso ipv6 right.... :) | 21:09 |
| *** akrzos has quit IRC | 21:09 | |
| *** florianf has joined #tripleo | 21:10 | |
| panda | I'm getting this error Could not fetch contents for file:///tmp/tripleoclient-WqEngI/network/ports/storage_v6.yaml. The netiso template uses relative paths, but I'm not sure if I should move it or specify a different base dir for the template (if possible) | 21:17 |
| panda | Maybe I should read how "deploy command" code works | 21:17 |
| *** akrzos has joined #tripleo | 21:18 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Neutron L3 service cleanups for hiera json hook https://review.openstack.org/393262 | 21:19 |
| panda | also, this is really difficult to debug since if I do something wrong is just hangs. No errors. only timeouts ... | 21:20 |
| mburned | beekneemech: anything else needed on 396759 ? | 21:29 |
| mburned | or can i cherry-pick to newton? | 21:29 |
| trown | mburned: just have to wait for it to pass CI, then we can merge and cherry-pick | 21:30 |
| mburned | or is there another pipeline that runs... | 21:30 |
| *** jprovazn has quit IRC | 21:30 | |
| trown | ah, ya there are more jobs that run | 21:30 |
| mburned | ack | 21:31 |
| *** jayg is now known as jayg|g0n3 | 21:34 | |
| *** jkilpatr has quit IRC | 21:39 | |
| openstackgerrit | Merged openstack/puppet-tripleo: Fix puppet lint failure https://review.openstack.org/396631 | 21:42 |
| openstackgerrit | Merged openstack/os-net-config: Add support for ethtool_opts option https://review.openstack.org/392697 | 21:42 |
| *** florianf has quit IRC | 21:43 | |
| *** akrzos has quit IRC | 21:43 | |
| *** akrzos has joined #tripleo | 21:47 | |
| *** fultonj has quit IRC | 22:01 | |
| *** jcoufal has quit IRC | 22:07 | |
| *** bana_k has joined #tripleo | 22:08 | |
| *** cdearborn has quit IRC | 22:16 | |
| *** jkilpatr has joined #tripleo | 22:21 | |
| *** trown is now known as trown|outtypewww | 22:22 | |
| *** lblanchard has quit IRC | 22:30 | |
| *** akrzos has quit IRC | 22:35 | |
| *** akrzos has joined #tripleo | 22:36 | |
| *** dougbtv has quit IRC | 22:36 | |
| *** myoung is now known as myoung|afk | 22:38 | |
| *** mhenkel has quit IRC | 22:39 | |
| *** mhenkel has joined #tripleo | 22:49 | |
| gbarros | #tripleo, looking for the actual executed cmdline of a command puppet is running during a deploy on a compute, where would I find that? | 23:06 |
| beekneemech | gbarros: You might want to try puppet-openstack for that question. It's not really specific to tripleo and there are more puppet experts there. | 23:10 |
| gbarros | ty | 23:10 |
| *** zaneb has quit IRC | 23:15 | |
| *** zaneb has joined #tripleo | 23:19 | |
| *** pradk has quit IRC | 23:19 | |
| openstackgerrit | Ben Nemec proposed openstack-infra/tripleo-ci: Always use cached images in updates job https://review.openstack.org/386713 | 23:20 |
| *** limao has joined #tripleo | 23:29 | |
| *** limao_ has joined #tripleo | 23:30 | |
| *** limao has quit IRC | 23:33 | |
| *** ccamacho has quit IRC | 23:40 | |
| *** panda is now known as panda|Zz | 23:42 | |
| *** zaneb has quit IRC | 23:52 | |
| *** pmannidi has joined #tripleo | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!