| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-03 | |||
| 14:29:33 | mriedem | sahid: after going over the re-proposed spec yeah | |
| 14:29:37 | mriedem | i can do that shortly | |
| 14:32:50 | sahid | mriedem: cool, thank you | |
| 14:34:07 | sdague | mriedem: honestly, I expect just a lot of people approved stuff, and our normal races are all in there | |
| 14:34:19 | mriedem | sdague: the job isn't running | |
| 14:34:23 | mriedem | it's something with zuul | |
| 14:34:31 | sdague | mriedem: on what job? | |
| 14:34:39 | mriedem | http://status.openstack.org/zuul/ | |
| 14:34:45 | mriedem | gate-grenade-dsvm-neutron-multinode-ubuntu-trusty: queued | |
| 14:34:53 | mriedem | top job in the queue has been queued for nearly 19 hours | |
| 14:35:12 | AJaeger | sdague: I just pushed some notes on #openstack-infra about my findings | |
| 14:35:22 | mriedem | http://graphite.openstack.org/render/?from=-24hours&height=400&until=now&width=800&bgcolor=ffffff&fgcolor=000000&target=alias%28smartSummarize%28sumSeries%28stats_counts.zuul.pipeline.*.all_jobs%29,%271h%27%29,%27All%20Jobs%27%29&title=Zuul%20Jobs%20Launched%20%28per%20Hour%29&_t=0.5040648836791604#1488551678795 | |
| 14:35:43 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: handle uninited fields in notification payload https://review.openstack.org/415857 | |
| 14:35:54 | sdague | oh | |
| 14:35:57 | sdague | neat | |
| 14:38:05 | mdbooth | Using only the cli, how do I determine which security group a particular instance is using? | |
| 14:38:30 | mdbooth | server show foo tells me name='default', but there are 4 of those | |
| 14:40:18 | mriedem | there is a secgroup-list command | |
| 14:40:21 | mriedem | or something like that | |
| 14:40:27 | mriedem | although we're removing that from novaclient | |
| 14:40:58 | mdbooth | Ah, I think I can get it from the port | |
| 14:41:01 | superdan | mriedem: hah, your create cell early thing is the same reason my fleetify patch is failing | |
| 14:41:08 | superdan | Found 1 cells: 00000000-0000-0000-0000-000000000000 cells | |
| 14:41:29 | mriedem | superdan: \o/ | |
| 14:41:34 | mriedem | and it's been in the gate 18 hours | |
| 14:41:42 | superdan | yeah :/ | |
| 14:41:47 | superdan | I guess I can go ahead and rebase on it | |
| 14:41:51 | mriedem | mdbooth: yeah otherwise the instance uuid is the device_id on the port | |
| 14:42:58 | mdbooth | For the curious, this seems to have done it: | |
| 14:43:01 | mdbooth | $ openstack port list --server foo --long | |
| 14:44:14 | openstackgerrit | Arnaud Morin proposed openstack/nova master: Add warning on setting secure_proxy_ssl_header https://review.openstack.org/423375 | |
| 14:44:46 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Refactor InstancePayload creation https://review.openstack.org/418488 | |
| 14:44:58 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove **kwargs passing in payload __init__ https://review.openstack.org/418489 | |
| 14:48:14 | mdbooth | Next question: is it possible to restart a devstack after the host goes down? | |
| 14:48:35 | mdbooth | i.e. simply restart everything running in screen without re-initialising everything | |
| 14:50:13 | migi | figleaf: re that funct test, with help by lyarwood it's now clear there is issue with older sqlite version, updating to newer one fixes issue, so probably it's broken at oslo.db level | |
| 14:50:37 | figleaf | migi: thanks for digging into that | |
| 14:53:13 | lyarwood | migi / figleaf ; yeah oslo_db just passes back an empty list because of this IndexError with the older sqlite version we have in RHEL - https://github.com/openstack/oslo.db/blob/c61d2d47bd39c31f1cd06faf497ffa85ae66cdd4/oslo_db/sqlalchemy/exc_filters.py#L185 | |
| 14:54:35 | figleaf | lyarwood: got it. That jibes with what was causing the test to fail. | |
| 14:57:56 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Improve assertJsonEqual error reporting https://review.openstack.org/428199 | |
| 14:59:17 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Libvirt driver emulator threads placement policy https://review.openstack.org/427066 | |
| 15:18:26 | jeblair | mriedem: /win 11 | |
| 15:18:34 | jeblair | gah | |
| 15:18:46 | jeblair | mriedem: no we're not running zuulv3 yet. :) | |
| 15:18:58 | mriedem | i saw, osic is down | |
| 15:19:32 | jeblair | mriedem: the gist of those emails is that we have made enough progress to run a test instance, zuulv3 is coming but is a little ways off yet. and we'll make much more noise when it gets closer. | |
| 15:20:32 | mriedem | oh so osic changed a flavor? | |
| 15:20:37 | mriedem | sdague: ^? | |
| 15:20:48 | sdague | mriedem: they changed all their flavors | |
| 15:21:05 | mriedem | and nodepool was hardcoded to use the flavor by id or something? | |
| 15:21:09 | sdague | nope | |
| 15:21:13 | sdague | but it caches flavorid | |
| 15:21:19 | mriedem | oh | |
| 15:21:47 | sdague | so it searches for flavors to lookup, but it caches the flavorid with no expiration | |
| 15:22:04 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622 | |
| 15:22:04 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387 | |
| 15:22:06 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged volume attachment https://review.openstack.org/418100 | |
| 15:22:06 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082 | |
| 15:22:09 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941 | |
| 15:22:09 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021 | |
| 15:22:11 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment https://review.openstack.org/415520 | |
| 15:22:28 | sdague | given that until osic this was working against public for pay clouds, it had never experienced "we deleted the flavor" | |
| 15:22:41 | sdague | because if you do that to customers... it all 'splodes | |
| 15:22:56 | mriedem | cdent: figleaf: are you able to answer my question here? https://review.openstack.org/#/c/440656/5 | |
| 15:24:05 | cdent | mriedem: short answer: no | |
| 15:24:23 | mriedem | lyarwood: see the issue in https://review.openstack.org/#/c/242602/ ? | |
| 15:24:54 | cdent | mriedem: but guessing I think that the length check was duplicated from somewhere else and this is just removing that redundancy while adding another confirmation of contents | |
| 15:24:57 | figleaf | mriedem: It's just checking that it's there. Why do you think it's random? | |
| 15:25:14 | mriedem | figleaf: because it could be anything right? | |
| 15:25:22 | mriedem | i'm wondering about the pattern going forward | |
| 15:25:33 | mriedem | if not a length check, should all new namespaces be checked that way in the tests? | |
| 15:25:37 | mriedem | or only the most recently added one? | |
| 15:25:53 | openstackgerrit | Michael Henkel proposed openstack/nova master: Implements: blueprint opencontrail-dpdk-support https://review.openstack.org/441183 | |
| 15:25:54 | figleaf | mriedem: wait - are you asking about the STORAGE_DISK_SSD? | |
| 15:25:58 | mriedem | yeah | |
| 15:26:03 | cdent | mriedem: that test isn't namespace | |
| 15:26:15 | figleaf | Ah - I was reading your comment a few lines up | |
| 15:26:15 | cdent | it's just asserting it's been defined | |
| 15:26:17 | lyarwood | mriedem: yup, I'll sort it out shortly | |
| 15:26:22 | mriedem | yeah sorry symbol name | |
| 15:26:30 | figleaf | mriedem: FWIW, I'm not a fan of tests like that | |
| 15:26:44 | figleaf | but it seems that Jay is | |
| 15:26:44 | mriedem | i mean i get that it's totally duplicating a constant | |
| 15:27:07 | mriedem | i guess i was trying to see the value | |
| 15:27:12 | mriedem | anyway, need to move rooms | |
| 15:32:29 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622 | |
| 15:32:29 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387 | |
| 15:32:30 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged volume attachment https://review.openstack.org/418100 | |
| 15:32:31 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082 | |
| 15:32:33 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941 | |
| 15:32:33 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021 | |
| 15:32:35 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment https://review.openstack.org/415520 | |
| 15:53:16 | mriedem | cdent: figleaf: i'm sure there has been an api wg discussion before about specific error codes in the api, do you happen to remember if anything was ever proposed | |
| 15:53:18 | mriedem | like a spec | |
| 15:53:25 | mriedem | talking about it in -infra | |
| 15:53:40 | cdent | mriedem: yeah, one sec | |
| 15:55:43 | leakypipes | superdan: won't be until this afternoon that I can get to splitting that Ironic inventory patch. I'd still like to do it, though. OK with ya? | |
| 15:57:12 | leakypipes | superdan: also, wallaby-bait: https://review.openstack.org/#/c/439187/ | |
| 15:58:36 | superdan | leakypipes: yar | |
| 15:58:47 | superdan | leakypipes: I could even cater that activity, depending | |