| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-07 | |||
| 23:08:42 | melwitt | okay, I think that's why it wouldn't show up | |
| 23:08:46 | mriedem | well, one does | |
| 23:08:49 | mriedem | the cells refresh one | |
| 23:08:55 | mriedem | the other doesn't, presumably b/c it's in a greenthread | |
| 23:09:06 | mriedem | or maybe b/c we have'nt discovered any hosts yet | |
| 23:15:47 | melwitt | yeah, looks like the greenthread is spawned during startup so it's effectively during startup too | |
| 23:15:59 | openstackgerrit | Merged openstack/nova master: [placement] Add version directives in the history doc https://review.openstack.org/589392 | |
| 23:16:08 | openstackgerrit | Merged openstack/nova master: Avoid joins in _server_group_count_members_by_user https://review.openstack.org/580764 | |
| 23:16:16 | openstackgerrit | Merged openstack/nova master: Use common functions in granular fixture https://review.openstack.org/588113 | |
| 23:17:11 | melwitt | but I see now, it doesn't run at all, don't find it in the log | |
| 23:17:51 | melwitt | oh, because logging in the greenthread is expected not to work? | |
| 23:19:00 | melwitt | oh, it's because [filter_scheduler]/track_instance_changes = False | |
| 23:19:04 | mriedem | melwitt: no i think it's because we disable CONF.filter_scheduler.track_instance_changes in superconductor mode in devstack | |
| 23:19:53 | mriedem | http://logs.openstack.org/32/582332/5/check/neutron-grenade/b319aa4/logs/screen-n-sch.txt.gz#_Jul_31_18_48_24_316837 | |
| 23:20:06 | melwitt | ahh | |
| 23:20:07 | mriedem | grenade runs in singleconductor mode so it's logged there | |
| 23:20:18 | melwitt | nice | |
| 23:20:50 | melwitt | so let's see if there's a difference in that job before the patch | |
| 23:22:20 | melwitt | looks like it, new request-id as of async_init_instance_info http://logs.openstack.org/58/540258/10/check/neutron-grenade/54f96c0/logs/screen-n-sch.txt.gz#_Jul_24_06_24_31_353068 | |
| 23:22:43 | melwitt | wait, but I still see the request-id prior to that being logged too | |
| 23:22:52 | melwitt | hm | |
| 23:28:07 | melwitt | oh, bc it has its own greenthread. duh. yeah so that one wouldn't show anything anyway | |
| 23:29:29 | melwitt | it can't cause a change in any other greenthread's local context | |
| 23:38:51 | jaypipes | sean-k-mooney: sure, just add a patch on top. | |
| 23:47:22 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: fix min_version for parent_provider_uuid in responses https://review.openstack.org/579577 | |
| 23:57:25 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: Add descriptions for rebuild https://review.openstack.org/588931 | |
| #openstack-nova - 2018-08-08 | |||
| 00:12:37 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix bad links for admin-guide https://review.openstack.org/580259 | |
| 00:24:32 | openstackgerrit | Merged openstack/os-vif master: Update reno for stable/rocky https://review.openstack.org/586106 | |
| 00:26:02 | mriedem | i have no idea how this can be happening https://review.openstack.org/#/c/588455/ | |
| 00:26:10 | mriedem | if only we had some multi-node pci ci | |
| 00:32:24 | openstack | bug 1784705 in OpenStack Compute (nova) queens "ResourceTracker.stats can leak across multiple ironic nodes" [High,In progress] https://launchpad.net/bugs/1784705 - Assigned to Matt Riedemann (mriedem) | |
| 00:32:24 | openstackgerrit | Merged openstack/nova stable/queens: Add recreate test for RT.stats bug 1784705 https://review.openstack.org/587921 | |
| 01:23:42 | openstackgerrit | zhufl proposed openstack/nova master: xx_instance_type_id in list_migrations should be integer https://review.openstack.org/588481 | |
| 02:08:59 | alex_xu | efried: yea, I'm working on it https://review.openstack.org/569011, we will discuss this in the PTG | |
| 03:18:50 | openstackgerrit | Merged openstack/nova stable/queens: Make ResourceTracker.stats node-specific https://review.openstack.org/587976 | |
| 03:18:57 | openstackgerrit | Merged openstack/nova master: Fix none-ascii char in doc https://review.openstack.org/588422 | |
| 03:19:06 | openstackgerrit | Merged openstack/nova master: Add microversion info in the os-server-groups API samples https://review.openstack.org/589006 | |
| 03:19:12 | openstackgerrit | Merged openstack/nova master: api-ref: Add descriptions for rebuild https://review.openstack.org/588931 | |
| 05:18:01 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] api-ref: add description for 1.29 https://review.openstack.org/589407 | |
| 06:01:54 | gmann | API office hour time | |
| 06:02:14 | gmann | #startmeeting nova api | |
| 06:02:15 | openstack | Meeting started Wed Aug 8 06:02:14 2018 UTC and is due to finish in 60 minutes. The chair is gmann. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 06:02:16 | openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 06:02:19 | openstack | The meeting name has been set to 'nova_api' | |
| 06:02:29 | gmann | who all here today? | |
| 06:02:37 | gmann | PING List: gmann, alex_xu | |
| 06:03:34 | alex_xu | o/ | |
| 06:04:12 | gmann | alex_xu: let's start | |
| 06:04:24 | gmann | #topic Priorities | |
| 06:04:34 | gmann | #link https://etherpad.openstack.org/p/rocky-nova-priorities-tracking | |
| 06:04:54 | gmann | it will be quick update as Rocky FF already done | |
| 06:05:29 | gmann | there are 4 pending item from this which we need to continue in Stein | |
| 06:05:45 | gmann | and after PTG we will update the priority for stein | |
| 06:06:27 | gmann | i will continue the policy granular things as default roles are done in keystone now | |
| 06:06:58 | gmann | #link this one https://review.openstack.org/#/c/547850/ | |
| 06:07:19 | gmann | alex_xu: anything you have on this topic ? | |
| 06:08:00 | alex_xu | gmann: why that depends on default roles in keystone? | |
| 06:08:42 | gmann | that is do to the policy granular along with some meaningful defaults roles | |
| 06:09:32 | alex_xu | ok, i see | |
| 06:09:35 | gmann | for example, in this spec i am proposing to make GET policy separate than POST/PUT so new GET policy should go with new default like reader, auditor etc | |
| 06:09:46 | gmann | which make sense | |
| 06:09:56 | gmann | otherwsie we end up doing policy change twice | |
| 06:11:04 | alex_xu | ok | |
| 06:11:43 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Use oslo_utils.uuidutils https://review.openstack.org/589717 | |
| 06:11:55 | gmann | let's move next? | |
| 06:12:07 | alex_xu | yea, nothing from me | |
| 06:12:11 | gmann | #topic Bug Triage/Discussion | |
| 06:12:13 | gmann | #link https://etherpad.openstack.org/p/nova-api-weekly-bug-report | |
| 06:12:23 | gmann | updated this week report | |
| 06:13:10 | gmann | there was 1 new bug which we can discuss here and keypair in-use bug | |
| 06:13:11 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Use uuidutils of oslo.utils https://review.openstack.org/589717 | |
| 06:13:14 | openstack | Launchpad bug 1644457 in OpenStack Compute (nova) "keypair quota error" [Medium,Confirmed] - Assigned to Vishakha Agarwal (vishakha.agarwal) | |
| 06:13:14 | gmann | #link https://bugs.launchpad.net/nova/+bug/1644457 | |
| 06:14:06 | gmann | alex_xu: i agree with doc only update as of now on keypair things, and later if we need we should go with barbican integration as matt mentioned via sdague mail | |
| 06:14:13 | gmann | i just updated the comment there. | |
| 06:14:50 | alex_xu | gmann: yea, that the thing we can do now | |
| 06:15:06 | gmann | alex_xu: +1 | |
| 06:15:09 | gmann | vishakha: ^^ | |
| 06:15:50 | gmann | next bug i want to discuss is new one show up - | |
| 06:15:53 | openstack | Launchpad bug 1783338 in OpenStack Compute (nova) "Unexpected exception in API method: ValueError: year is out of range" [Medium,Confirmed] - Assigned to Ghanshyam Mann (ghanshyammann) | |
| 06:15:53 | gmann | #link https://bugs.launchpad.net/nova/+bug/1783338 | |
| 06:15:55 | vishakha | gmann yes I will update the doc | |
| 06:16:06 | gmann | i commented and confirmed that. | |
| 06:16:09 | gmann | vishakha: thanks | |
| 06:16:32 | vishakha | gmann: thanks for the help | |
| 06:16:34 | gmann | vishakha: you can just make clear in api-ref that in-use field will always be zero for keypairs. | |
| 06:16:59 | vishakha | gmann: ok sure | |
| 06:17:01 | gmann | and some more clarification like keypair is per user only not per projects | |
| 06:17:45 | gmann | alex_xu: on second bug, error is that start or end dates in GET /simple-tenant-usage API is not iso8601 format. | |
| 06:18:11 | gmann | it says year is out of range which i do not know why as attached screenshot in bug show year as '2018' | |
| 06:18:28 | vishakha | gmann : ok | |
| 06:18:39 | gmann | one thing we are surly missing is to handle the value error which leads to 500 in this API | |
| 06:21:10 | alex_xu | gmann: yea, we can update the query schema | |
| 06:21:48 | gmann | as of now we just check string there | |
| 06:21:50 | gmann | #link https://github.com/openstack/nova/blob/e8a73c400846b5be0f12a2eea32bd2c5c5aeb370/nova/api/openstack/compute/schemas/simple_tenant_usage.py#L22 | |
| 06:22:24 | alex_xu | yea | |
| 06:22:31 | gmann | need to check if iso8601 format there in json schema or we can do via custom format_checker also | |
| 06:22:46 | alex_xu | I remember we have somewhere use it | |
| 06:23:00 | alex_xu | change-since in server list API i guess | |
| 06:24:29 | alex_xu | gmann: https://github.com/openstack/nova/blob/e8a73c400846b5be0f12a2eea32bd2c5c5aeb370/nova/api/openstack/compute/schemas/servers.py#L592 | |