Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-09-25
20:02:54 imacdonn but yeah
20:02:59 mriedem you said you hacked something
20:03:19 imacdonn Oh, I only hacked out the check that max-count has to be >0 .. but I'm not applying that here
20:03:22 mriedem however,
20:03:23 mriedem http://logs.openstack.org/87/604687/4/check/neutron-grenade/6c10d43/logs/grenade.sh.txt.gz
20:03:30 mriedem shows that the table output is also busted in our grenade run
20:03:41 mriedem http://logs.openstack.org/87/604687/4/check/neutron-grenade/6c10d43/logs/grenade.sh.txt.gz#_2018-09-25_09_44_39_645
20:03:56 mriedem well...
20:04:02 mriedem maybe not, if we just don't have anything to migrate
20:04:02 imacdonn I think you're right .. I think that each call to _run_migration clobbers the previous result, instead of adding to it
20:05:39 mriedem that populate_uuids migration was added in queens, and looking at a grenade run from queens, we should see bdms getting migrated to have a uuid, and i'm not seeing any results in the table for a grenade run on queens
20:05:40 mriedem http://logs.openstack.org/48/604448/1/check/neutron-grenade/19d8166/logs/grenade.sh.txt.gz
20:05:44 mriedem so yeah, likely busted
20:05:47 mriedem imacdonn: please report a bug
20:05:59 imacdonn I think the clobberation is happening at https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L720
20:06:00 imacdonn will bug
20:06:32 mriedem yeah because of https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L689
20:06:36 mriedem resets everything to 0
20:07:03 imacdonn it makes a new "migrations" dict, with all of the members, for each pass
20:18:33 openstack Launchpad bug 1794364 in OpenStack Compute (nova) "'nova-manage db online_data_migrations' count fail" [Undecided,New]
20:18:33 imacdonn https://bugs.launchpad.net/nova/+bug/1794364
20:44:06 mriedem melwitt: can you this this queens change? https://review.openstack.org/#/c/600101/
20:44:15 mriedem *hit
20:44:25 melwitt sure
20:58:27 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Cross-cell resize https://review.openstack.org/603930
21:07:41 openstackgerrit Matt Riedemann proposed openstack/nova master: api-ref: add 'migrations' param to GET /os-migrations https://review.openstack.org/605232
21:18:47 mriedem https://www.openstack.org/summit/berlin-2018/vote-for-speakers#/22793
21:18:58 mriedem dansmith: ^ lemme know if you want to moderate that instead of me
21:19:02 mriedem i'm pretty sure you do
21:20:19 melwitt when is dan coming back?
21:23:54 mriedem next week
21:40:43 imacdonn Error attempting to run <function create_incomplete_consumers at 0x7f60e7498668>
21:40:57 imacdonn but it didn't log anything :/ This migrations stuff is finicky ..... ?
21:45:54 openstack Launchpad bug 1790701 in OpenStack Compute (nova) rocky "online_data_migrations fail in rocky+" [Critical,Fix committed] - Assigned to Matt Riedemann (mriedem)
21:47:44 openstackgerrit Jonte Watford proposed openstack/nova master: Add __str__ method to objects NUMACell objects https://review.openstack.org/600269
21:48:35 mriedem imacdonn: https://review.openstack.org/#/c/600085/
21:49:14 imacdonn mriedem: ack
21:49:32 mriedem just haven't updated that yet
22:00:54 melwitt mriedem: I made the mistake of thinking nova-consoleauth wasn't required anymore because the proxy code defaults to only using the db backend to verify a console. but the nova-compute code is still storing the token auth in both nova-consoleauth and the database backend, for reasons I don't know, when I took over the patch series
22:01:39 melwitt this cycle, I can remove the nova-compute code that stores the token auth in nova-consoleauth, at which point nova-consoleauth would not be needed on a fresh install
22:03:24 melwitt as for the deprecation, we could un-deprecate it
22:03:28 mriedem well,
22:03:34 mriedem if nova-compute in stein doesn't need it,
22:03:38 mriedem that would be the time to deprecate it,
22:03:45 mriedem but rocky wasn't really the right time to deprecate it sounds like
22:03:50 mriedem since something was still using it
22:04:04 mriedem i do'nt think i recall a retroactive undeprecate on stable
22:05:35 melwitt yeah, I think that's accurate, that rocky was not the right time. the only benefit to it that I can think of is, a heads up the cycle before it's no longer needed. but that's a stretch, I guess
22:08:01 mriedem i don't think there would be anything wrong with undeprecating it on stable/rocky,
22:08:12 mriedem if it's wrong, we shouldn't report it as deprecated b/c people will be getting warnings
22:08:42 mriedem we have done something like this in the past when we removed the original [placement_database] options b/c they weren't used
22:08:52 mriedem and them being in config made people think they could use them
22:08:59 imacdonn I discovered this issue because I was trying to be a good boy, and not to deprecated things, when looking at rocky upgrade, so I turned off the service, and things broke
22:09:15 imacdonn not do* deprecated things
22:09:49 melwitt yeah. I have caused a fair bit of operator pain with this mistake. sorry for that
22:09:59 mriedem yeah if there is no way to *not* use it in rocky, then saying it's deprecated (but still required) is weird
22:10:09 imacdonn not complaining .. just sharing my experience ;)
22:10:14 mriedem if ^ is true, then we likely want to undeprecate in stable/rocky
22:10:25 melwitt I'm sure it was not pleasant :P
22:10:35 mriedem gotta run
22:10:57 melwitt yeah, there's no way to not use it in rocky. I'll propose some reverts, because we need to undo the deprecation
22:16:48 melwitt s/nova-compute/nova-api/ earlier
22:20:35 openstackgerrit melanie witt proposed openstack/nova master: Revert "Deprecate the nova-consoleauth service" https://review.openstack.org/605250
22:21:30 openstackgerrit sean mooney proposed openstack/nova master: Add Result wrapper class to nova common https://review.openstack.org/605251
22:21:57 sean-k-mooney melwitt: ^ i think this is what we want for the cell gather thing
22:22:35 melwitt thanks
22:25:07 openstackgerrit melanie witt proposed openstack/nova master: Revert "Deprecate the nova-consoleauth service" https://review.openstack.org/605250
22:26:45 sean-k-mooney melwitt: no worries it was fun. i like creating special pourpose data structure like this. i rarely have a need to do it however :)
22:28:10 melwitt :)
22:45:03 openstackgerrit melanie witt proposed openstack/nova master: Un-deprecate the nova-consoleauth service https://review.openstack.org/605255
23:00:53 openstackgerrit Matt Riedemann proposed openstack/nova master: Enforce case-sensitive hostnames in aggregate host add https://review.openstack.org/604906
23:05:40 openstackgerrit Matt Riedemann proposed openstack/nova master: Enforce case-sensitive hostnames in aggregate host add https://review.openstack.org/604906
23:05:40 openstackgerrit Matt Riedemann proposed openstack/nova master: Revert "Make host_aggregate_map dictionary case-insensitive" https://review.openstack.org/604898
23:09:28 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Enforce case-sensitive hostnames in aggregate host add https://review.openstack.org/605261
23:09:28 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Revert "Make host_aggregate_map dictionary case-insensitive" https://review.openstack.org/605260
23:21:18 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Enforce case-sensitive hostnames in aggregate host add https://review.openstack.org/605267
23:21:18 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Revert "Make host_aggregate_map dictionary case-insensitive" https://review.openstack.org/605266
23:25:07 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Revert "Make host_aggregate_map dictionary case-insensitive" https://review.openstack.org/605268
23:25:08 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Enforce case-sensitive hostnames in aggregate host add https://review.openstack.org/605269
23:27:59 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Skip ServerShowV247Test.test_update_rebuild_list_server in nova-cells-v1 job https://review.openstack.org/605271
23:27:59 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Skip ServerShowV254Test.test_rebuild_server in cells v1 job https://review.openstack.org/605270
#openstack-nova - 2018-09-26
01:04:44 openstackgerrit Tao Li proposed openstack/nova master: Rollback instance vm_state to original where instance claims failed https://review.openstack.org/592252
01:14:10 openstackgerrit Tao Li proposed openstack/nova master: Don't persist retry information into database https://review.openstack.org/605011
01:23:41 openstackgerrit Merged openstack/nova stable/queens: Fix nova-status "_check_resource_providers" check https://review.openstack.org/600101
02:03:21 openstackgerrit Brin Zhang proposed openstack/nova master: Specifies the storage backend to boot instance https://review.openstack.org/579360
02:07:58 openstackgerrit Brin Zhang proposed openstack/nova master: Add volume_type field to BlockDeviceMapping object https://review.openstack.org/604687
02:42:52 openstackgerrit tianhui proposed openstack/nova master: doc: update availability zone doc https://review.openstack.org/605306
02:46:21 openstackgerrit Brin Zhang proposed openstack/nova master: Specifies the storage backend to boot instance https://review.openstack.org/579360
03:17:44 naichuans Hi, melwitt: has stein runways in use? https://etherpad.openstack.org/p/nova-runways-stein
03:30:24 openstackgerrit Merged openstack/nova master: Option "scheduler_default_filters" is deprecated. https://review.openstack.org/604148
03:30:32 openstackgerrit Merged openstack/nova master: doc:update virtual gpu doc https://review.openstack.org/605012
04:33:43 openstackgerrit iain MacDonnell proposed openstack/nova master: nova-manage - fix online_data_migrations counts https://review.openstack.org/605329
05:25:33 openstackgerrit Merged openstack/nova master: Consumer gen support for put allocations https://review.openstack.org/591647
06:03:12 gmann api office hour time
06:04:33 gmann #startmeeting nova api
06:04:34 openstack Meeting started Wed Sep 26 06:04:33 2018 UTC and is due to finish in 60 minutes. The chair is gmann. Information about MeetBot at http://wiki.debian.org/MeetBot.
06:04:35 openstack Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
06:04:37 openstack The meeting name has been set to 'nova_api'
06:04:46 gmann PING List: gmann, alex_xu
06:12:43 gmann i will be around here for some time.

Earlier   Later