Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-04-05
20:31:34 openstackgerrit Matt Riedemann proposed openstack/nova master: Set size/status during image create with FakeImageService https://review.openstack.org/449702
20:31:34 openstackgerrit Matt Riedemann proposed openstack/nova master: Perform old-style local delete for shelved offloaded instances https://review.openstack.org/453859
20:31:44 mriedem thar she blar
20:32:56 dtp mriedem how do you guys do rollbacks w/o downgrade on the migrations?
20:33:03 mriedem dtp: we dont
20:33:22 mriedem roll forward to the maxxx
20:33:33 mriedem i usually do the dew while rolling forward
20:34:07 dtp nova-manage db sync supports downgrade
20:34:31 mriedem it might say it does
20:34:35 mriedem which would be a bug
20:34:57 dtp i think it actually does....or else my upgrade is already idempotent
20:35:18 dtp because 358 > 359 > 358 > 359 works
20:35:40 mriedem well it works for your migration b/c you implemented downgrade
20:36:09 mriedem as i said in the review, i'm surprised we didn't have a test that would fail saying we don't support downgrade
20:36:18 dtp ok ok
20:36:38 dtp thanks for the review, btw
20:37:12 mriedem np
20:37:57 mriedem dtp: so i guess we weren't idempotent when adding the compute_nodes.uuid column either https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/migrate_repo/versions/314_add_resource_provider_tables.py#L78
20:38:43 mriedem but we should be, e.g. https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/migrate_repo/versions/358_bdm_attachment_id.py#L27
20:39:24 dtp makes sense
20:40:04 mriedem dtp: btw, when you said nova-manage db sync supports downgrade, you mean by passing in the --version right?
20:40:37 dtp yep
20:40:44 mriedem yeah i think if you tried going further, it'd explode
20:42:01 dtp surely
20:43:39 mriedem huh clearly test_no_migrations_have_downgrade failed it's job
20:43:42 edleafe OK, what am I doing wrong? I'm making a commit to the os-traits library. I have git-review installed, and a .gitreview file in the directory, just as in my nova project. But when I commit, I'm not getting a Change-Id added to the commit message.
20:43:58 mriedem edleafe: git review -s
20:44:02 mriedem in the repo
20:44:12 mriedem you probably don't have the gerrit remote
20:44:31 edleafe mriedem: that worked, but I did have the remote
20:44:45 mriedem maybe git review -s installs a hook or something, idk
20:45:23 edleafe dunno, but thanks for setting me straight
20:46:44 melwitt edleafe: you have to do git review -s anytime you clone a new repo, fwiw
20:47:58 edleafe melwitt: yeah, I thought I had, but I guess not...
20:48:02 edleafe thanks
20:51:36 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/451049
20:57:51 cdent edleafe: that os-traits stuff you did is very satisfying
21:00:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix test_no_migrations_have_downgrade https://review.openstack.org/453872
21:07:39 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/453896
21:08:58 edleafe cdent: a lot easier on the eyes, no?
21:09:05 cdent quite
21:20:27 openstackgerrit Dan Peschman proposed openstack/nova master: Add uuid to Service model https://review.openstack.org/453025
21:21:04 mriedem oh dansmith that reminds me, did the functional tests for https://review.openstack.org/#/c/438025/ work?
21:21:10 mriedem oh https://review.openstack.org/#/c/452006/2
21:21:53 dansmith yeah I think I got it working
21:22:26 mriedem you should take that off the end of the series
21:22:33 mriedem since it's testing something that's already merged
21:22:37 mriedem we can just work that in parallel
21:56:40 dansmith mriedem: sure
21:58:32 mriedem dansmith: there are some unnecessary changes in https://review.openstack.org/#/c/439891/ i think, so i'll let you check the comments, but otherwise i'm +2
22:02:43 dansmith mriedem: yeah they're all rebase noise, I'll fix
22:02:53 mriedem cool
22:06:28 mriedem where does the compute call the scheduler on init if not _update_scheduler_instance_info?
22:06:50 dansmith there,
22:07:04 dansmith which is a synchronous call all the way into scheduler,
22:07:07 dansmith which will look up hostmapping
22:07:10 dansmith and fail
22:07:14 dansmith if we haven't created it yet
22:07:32 mriedem that's an rcp cast
22:07:35 mriedem *rpc
22:07:54 dansmith not if we have CastAsCall
22:08:00 mriedem oh
22:08:01 mriedem heh
22:08:04 dansmith which is the devil
22:10:39 mriedem alright that's fun
22:10:42 mriedem left a comment to remind myself
22:11:07 mriedem also, i see that none of that compute -> scheduler stuff is necessary if you're not using the filter scheduler, but meh
22:12:59 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/453896
22:16:27 openstackgerrit Matt Riedemann proposed openstack/nova master: Mention track_instance_changes in scheduler.driver conf help https://review.openstack.org/453916
22:35:29 openstackgerrit Dan Smith proposed openstack/nova master: Add functional test for two-cell scheduler behaviors https://review.openstack.org/452006
22:38:42 openstackgerrit Dan Smith proposed openstack/nova master: Teach HostAPI about cells https://review.openstack.org/442162
22:38:44 openstackgerrit Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891
22:38:44 openstackgerrit Dan Smith proposed openstack/nova master: Sort CellMappingList.get_all() for safety https://review.openstack.org/443174
22:38:45 openstackgerrit Dan Smith proposed openstack/nova master: Add get_by_instance_uuids() to InstanceMappingList https://review.openstack.org/443292
22:38:45 openstackgerrit Dan Smith proposed openstack/nova master: Clean up ClientRouter debt https://review.openstack.org/444487
22:38:46 openstackgerrit Dan Smith proposed openstack/nova master: Make server_groups determine deleted-ness from InstanceMappingList https://review.openstack.org/443293
22:38:47 openstackgerrit Dan Smith proposed openstack/nova master: Remove Mitaka-era service version check https://review.openstack.org/442861
22:38:48 openstackgerrit Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736
22:47:36 openstackgerrit Matt Riedemann proposed openstack/nova master: Don't send instance updates from compute if not using filter scheduler https://review.openstack.org/453916
22:49:20 openstackgerrit Matt Riedemann proposed openstack/nova master: Don't send instance updates from compute if not using filter scheduler https://review.openstack.org/453916
23:05:30 openstackgerrit Octave Orgeron proposed openstack/nova master: Enables MySQL Cluster Support for Nova https://review.openstack.org/446643
23:22:08 openstackgerrit Matt Riedemann proposed openstack/nova master: Cleanup wording on compute service version checks in API https://review.openstack.org/453923
23:43:07 openstackgerrit Alex Szarka proposed openstack/nova master: Transform instance-exists notification https://review.openstack.org/403660
#openstack-nova - 2017-04-06
00:16:52 openstackgerrit Sujitha proposed openstack/nova master: Add BDM to InstancePayload https://review.openstack.org/448779
00:39:23 dtp mriedem where's the [who's going to boston] page?
00:47:34 mriedem dansmith: remind me which services can be in cell0
00:51:00 dansmith mriedem: anything but compute
00:55:21 mriedem hmm
00:55:21 mriedem weird
01:00:28 openstackgerrit Steven Webster proposed openstack/nova master: Update nova network info when doing rebuild for evacuate operation https://review.openstack.org/382853
01:01:30 openstackgerrit Steven Webster proposed openstack/nova master: Fix port update exception when unshelving an instance with PCI devices https://review.openstack.org/453938
01:03:07 mriedem dansmith: alright i've done all i could https://review.openstack.org/#/c/442162/
01:03:32 mriedem +2s up and down before and after that one though
01:03:40 mriedem it should be easy to resolve, just missing some error handling in other apis
01:35:27 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot https://review.openstack.org/394321
04:21:44 openstackgerrit Maciej Szankin proposed openstack/nova-specs master: Add deprecate-nova-consoleauth spec https://review.openstack.org/450063
04:57:59 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types https://review.openstack.org/430608
05:29:30 openstackgerrit jichenjc proposed openstack/python-novaclient master: Remove version 1.1 support https://review.openstack.org/453980
06:23:56 openstackgerrit Lingyong Xu proposed openstack/nova master: Use HostAddressOpt for opts that accept IP and hostnames https://review.openstack.org/453991
06:41:58 openstackgerrit Silvan Kaiser proposed openstack/nova master: Removes potentially bad exit value from accepted list in Quobyte volume driver https://review.openstack.org/453537

Earlier   Later