Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-04-29
16:29:24 mriedem larsks: not in the api or osc
16:29:28 sean-k-mooney and there likely should not be one at teh api
16:29:33 mriedem there is a nova cli command that does that
16:29:44 sean-k-mooney larsks: the nova host-evecuate-live command is impleeted in the nova client
16:29:50 mriedem larsks: http://www.danplanet.com/blog/2016/03/03/evacuate-in-nova-one-command-to-confuse-us-all/
16:30:16 larsks Is there an osc command? Because the nova cli doesn't support clouds.yaml, which is a pain.
16:30:38 larsks Meh, I can work around that. In any case, thank you for the pointer.
16:30:38 mriedem no
16:30:44 sean-k-mooney larsks: by the way the reason i said its not a good idea to have this at the api level is starting multple live migration on the same host concurrently increase the risk that they will fail or not complete
16:31:00 mriedem if you do live migrate with osc, i'd specify --os-compute-api-version 2.30 to avoid that force behavior https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id27
16:31:16 mriedem sean-k-mooney: by default the compute throws those into a blocking queue
16:31:19 sean-k-mooney larsks: the best way to work around it is to serialise the migrations
16:31:19 mriedem one at a time
16:31:37 sean-k-mooney really i was not aware of that
16:31:44 sean-k-mooney is that contoled via a config?
16:36:04 mriedem sean-k-mooney: yes
16:36:26 mriedem sean-k-mooney: https://docs.openstack.org/nova/latest/configuration/config.html#DEFAULT.max_concurrent_live_migrations
16:36:46 mriedem dansmith added that i believe because of these evacuate all clis
16:36:52 mriedem which was about the time he wrote that blog post
16:37:54 dansmith yes, on advice from libvirt/qemu people that they basically should be serialized anyway
16:38:01 dansmith like, that default should really be =1
16:38:04 sean-k-mooney ah ok im not sure if that is the default we ship in OSP. but ya from talking to danpb about this in the past he strongly recommended serailising it
16:38:16 dansmith but we made it =10 to kinda keep existing behavior, but in reality it should be 1
16:38:20 dansmith yeah
16:38:39 sean-k-mooney ah ok so it is an osp config change
16:38:43 sean-k-mooney cool
16:40:07 mriedem the default is 1
16:40:15 sean-k-mooney yes upstream
16:40:23 sean-k-mooney which is what it should be donwstream but its not
16:40:44 sean-k-mooney that was what was confusing me as this came up internally about a month ago
16:50:07 openstackgerrit Matt Riedemann proposed openstack/nova master: Add functional test for AggregateMultiTenancyIsolation + migrate https://review.opendev.org/571265
16:55:08 artom For those of us not (yet) in Denver, is there a way to recheck a single job in the gate?
16:55:54 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'instance_info_cache_update_at_top' https://review.opendev.org/651299
16:55:54 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'instance_fault_create_at_top' https://review.opendev.org/651298
16:55:55 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'instance_update_at_top', 'instance_destroy_at_top' https://review.opendev.org/651301
16:55:55 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'get_keypair_at_top' https://review.opendev.org/651300
16:55:56 openstackgerrit Stephen Finucane proposed openstack/nova master: Stop handling 'update_cells' on 'BandwidthUsage.create' https://review.opendev.org/651303
16:55:56 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'instance_update_from_api' https://review.opendev.org/651302
16:55:57 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove cells code https://review.opendev.org/651306
16:55:57 openstackgerrit Stephen Finucane proposed openstack/nova master: Stop handling cells v1 for instance naming https://review.opendev.org/651304
16:55:58 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove unnecessary wrapper https://review.opendev.org/651308
16:55:58 openstackgerrit Stephen Finucane proposed openstack/nova master: Stop handling 'InstanceUnknownCell' exception https://review.opendev.org/651307
16:55:59 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Remove cells v1 options, group https://review.opendev.org/651310
16:55:59 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Remove cell APIs https://review.opendev.org/651309
17:15:34 aspiers[m] artom: I couldn't find one
17:16:07 artom aspiers[m], thanks for looking!
17:16:12 artom I think I have other problems, anyways
17:16:25 artom Downstream CI Neutron and upstream CI Neutron are apparently doing different things :/
19:01:22 mriedem dansmith: you might want to take a look at this bug fix https://review.opendev.org/#/c/654584/
19:05:06 openstackgerrit Matt Riedemann proposed openstack/nova stable/stein: AZ list performance optimization: avoid double service list DB fetch https://review.opendev.org/656382
19:08:15 mriedem need a stable core to hit this https://review.opendev.org/#/c/656176/
19:58:54 openstackgerrit Merged openstack/nova master: Log notifications if assertion in _test_live_migration_force_complete fails https://review.opendev.org/650155
19:59:03 openstackgerrit Merged openstack/nova master: Add functional regression recreate test for bug 1825020 https://review.opendev.org/653268
19:59:04 openstack bug 1825020 in OpenStack Compute (nova) "resize of instance, that was booted from volume, trigger image size check then fails resize" [High,In progress] https://launchpad.net/bugs/1825020 - Assigned to Matt Riedemann (mriedem)
20:02:35 HW_Peter hey quick question
20:02:56 HW_Peter are network events
20:03:02 HW_Peter such as vif plugging and unplugging
20:03:09 HW_Peter supported in a multi cellsv2 environmnet
20:24:39 artom HW_Peter, Summit/PTG are happening, so you're unlikely to get too much attention here (sorry!), but yeah, since events are necessary for basic things like booting instances (with libvirt at least), I imagine we made sure they still worked in multicell
21:03:58 mriedem mtreinish: failed again so i guess the tempest docs are busted or i'm busted, but i'm going to drop the new lines and comments
21:04:38 mriedem gmann: fyi i pulled the tempest_test_regex and tempest_black_regex examples from the tempest readme and they don't seem to be working https://review.opendev.org/#/c/655222/7/.zuul.yaml
21:07:44 mtreinish mriedem: yeah I'm not sure if the comments will work (the new lines should I think, not 100% sure). I think with '|' the yaml parser passes comments as part of the string which probably is breaking the regex
21:08:30 mriedem i'm just going to use tempest_test_blacklist
21:08:33 mtreinish mriedem: this type of use case is why stestr has the whitelist and blacklist files so you don't need to try and use a giant complicated regex as a cli arg
21:09:39 openstackgerrit Matt Riedemann proposed openstack/nova master: Enable n-novnc in nova-multi-cell job https://review.opendev.org/655711
21:09:39 openstackgerrit Matt Riedemann proposed openstack/nova master: Add nova-multi-cell job https://review.opendev.org/655222
21:11:05 openstackgerrit Matt Riedemann proposed openstack/nova master: Enable n-novnc in nova-multi-cell job https://review.opendev.org/655711
21:11:05 openstackgerrit Matt Riedemann proposed openstack/nova master: Add nova-multi-cell job https://review.opendev.org/655222
21:12:47 mriedem jaypipes: there is no batch option for discover_hosts cli
21:19:14 gmann mriedem: mtreinish it worked as far as i know but yeah, may be comment on same line is issue? let me check
21:27:08 mriedem dansmith: https://review.opendev.org/#/c/654596/
21:28:23 mriedem otherwise https://review.opendev.org/#/c/647396
21:47:16 mriedem ERROR [nova.scheduler.client.report] Failed to retrieve allocation candidates from placement API for filters: RequestGroup(aggregates=[],forbidden_traits=set(['CUSTOM_COMPUTE_STATUS_DISABLED']),provider_uuids=[],requester_id=None,required_traits=set([]),resources={DISK_GB=1,MEMORY_MB=512,VCPU=1},use_same_provider=False) Got 400: {"errors": [{"request_id": "req-ee195267-ca87-4f1d-8f53-5390cc8c8538", "title": "Bad Request", "code":
21:47:16 mriedem so uh, we have to have custom traits defined before we can use them with GET /allocation_candidates?
21:47:17 mriedem "placement.undefined_code", "status": 400, "detail": "The server could not comply with the request since it is either malformed or otherwise incorrect.\n\n No such trait(s): CUSTOM_COMPUTE_STATUS_DISABLED. "}]}.
21:47:32 mriedem cdent: edleafe: ^?
21:48:23 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for node.list https://review.opendev.org/656027
21:48:23 openstackgerrit Dustin Cowles proposed openstack/nova master: Introduces SDK to IronicDriver and uses for node.get https://review.opendev.org/642899
21:48:36 cdent mriedem: yes
21:50:39 efried mriedem: Look for methods like ensure_traits. IIRC, the API returns success with different codes when the specified trait already exists vs when it gets created
21:51:39 mriedem i think it's actually trait.ids_from_names that hits it
21:51:42 mriedem but anyway, i didn't realize
21:52:56 openstackgerrit Dustin Cowles proposed openstack/nova master: WIP/PoC: Use SDK instead of ironicclient for node.list https://review.opendev.org/656027
21:52:57 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for checking instance exists https://review.opendev.org/656028
21:54:27 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for node.list https://review.opendev.org/656027
21:58:49 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for checking instance exists https://review.opendev.org/656028
22:03:49 openstackgerrit Merged openstack/nova master: libvirt: Avoid using os-brick encryptors when device_path isn't provided https://review.opendev.org/649951
22:10:20 openstackgerrit Dan Smith proposed openstack/nova master: Add image type request filter https://review.opendev.org/656413
22:37:32 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add placement request filter for disabled computes https://review.opendev.org/654596
23:02:00 openstackgerrit Merged openstack/nova master: Remove unused context parameter from RT._get_instance_type https://review.opendev.org/641792
23:02:12 openstackgerrit Merged openstack/nova master: Always pass HostAPI to get_availability_zones https://review.opendev.org/655583
23:11:36 openstackgerrit Balazs Gibizer proposed openstack/nova master: allow getting resource request of every bound ports of an instance https://review.opendev.org/655110
23:11:37 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add request_spec to server move RPC calls https://review.opendev.org/655721
23:11:37 openstackgerrit Balazs Gibizer proposed openstack/nova master: Pass network API to the conducor's MigrationTask https://review.opendev.org/655111
23:11:38 openstackgerrit Balazs Gibizer proposed openstack/nova master: Extend NeutronFixture to handle migrations https://review.opendev.org/655114
23:11:38 openstackgerrit Balazs Gibizer proposed openstack/nova master: re-calculate provider mapping during migration https://review.opendev.org/655112
23:11:39 openstackgerrit Balazs Gibizer proposed openstack/nova master: update allocation in binding profile during migrate https://review.opendev.org/656422
23:11:39 openstackgerrit Balazs Gibizer proposed openstack/nova master: func test for migrate server with ports having resource request https://review.opendev.org/655113
#openstack-nova - 2019-04-30
00:10:10 mriedem dansmith: https://review.opendev.org/#/c/655222/ for the nova-multi-cell job should finally be ready to go - the regex stuff is all sorted out
00:56:47 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Specifying az when restore shelved server https://review.opendev.org/624689

Earlier   Later