Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-05-04
20:36:38 superdan mriedem: looks weird to index by migration uuid instead of instance, is that because of how the cancel migration call works?
20:37:56 mriedem i think it could go either way, live_migration_abort gets the instance and migration
20:38:22 mriedem kevin just doesn't have that patch up yet
20:39:35 superdan okay I'll make a half-assed comment about it
20:40:50 mriedem quarter cheek friday please
20:44:03 mriedem arvindn05: that's not correct,
20:44:17 mriedem for rebuild, if the image doesn't change, we don't run through the scheduler, so we don't run the ImagePropertiesFilter
20:44:34 mriedem see https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L898
20:44:48 mriedem if host != instance.host: will be False in that block for rebuild
20:45:55 mriedem so, idk, that's new validation in the rebuild + same image case, i don't know if we need to revalidate the image in that case, we didn't before, but i also don't really care all that much
20:46:29 arvindn05 ok...i guess it can be handled as part of CR
20:50:33 mriedem arvindn05: +2 on the spec amendment
20:50:39 mriedem time for you to pop a bottle
20:50:45 mriedem of pills
20:50:48 arvindn05 awesome...thanks! :)
20:54:07 arvindn05 mriedem: see here on melwitt comments http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2018-05-03.log.html#t2018-05-03T20:48:55
20:55:57 arvindn05 i think if we are not running imageproperties filter now when image does not change we might want to be consistent...i can raise a bug to state that we dont run image properties filter if image hasnt changed(even though image properties on that image could have changed)
20:57:40 arvindn05 and when its fixed...they can fix to handle traits as well....another way is if i just need to call schedule instances for running the image properties filter...i can do that as part of this change
21:04:53 mriedem please no
21:05:14 mriedem i think if we don't call the scheduler (ImagePropertiesFilter) today, we should just continue to not do the validation
21:05:24 mriedem i.e. image doesn't change
21:07:03 arvindn05 sounds like a plan....although a bug might be warranted at that point....
21:07:26 mriedem i don't know that it's a bug
21:07:35 mriedem but it's 4pm on a friday, so i also don't care to think about this right now
21:07:59 arvindn05 in case of rebuild with a trait like Trusted host...if the host is no longer trusted we should prevent rebuild...might be a security issue
21:08:34 arvindn05 but like you said...its friday...we can figure it out in the CR :)
21:08:39 mriedem is trusted host a standard trait?
21:08:40 arvindn05 any plans for the weekend?
21:08:54 mriedem yardwork, mexican food and avoiding little girls
21:09:12 mriedem i.e. my daughters annoying friends, not pervy stuff
21:09:23 arvindn05 nope....its a custom trait...but traits could be used for security things as well
21:09:45 arvindn05 lol....good job with the quick clarification :P
21:09:59 openstackgerrit Jay Pipes proposed openstack/nova master: rework how we pass candidate request information https://review.openstack.org/566166
21:10:00 mriedem if an external service says this host is trusted or not and there are instances using an image that requires a trusted host, then i'd expect that external service to migrate those instances before saying it's no longer trusted
21:10:06 mriedem not waiting for the user to rebuild and find out
21:10:19 mriedem the instance user is not an admin so they can't see what the traits are on the node RP anyway
21:11:38 arvindn05 i guess there are 2 models...there is just a trust attestation piece which does not do any active managment...but more for reporting...another could be with active management like you mentioned
21:12:20 arvindn05 anyway...will leave you to enjoy your friday and the weekend :)
21:12:44 mriedem i would personally prefer that nova doesn't grow a bunch of side effect code to handle random traits-related what-if kind of conditionals because of the ability of external services to monkey with traits on compute node providers
21:13:02 mriedem and with that, i'm out also, ttyl
21:13:18 arvindn05_away ttyl
#openstack-nova - 2018-05-05
00:38:29 openstackgerrit Merged openstack/nova stable/queens: Handle @safe_connect returns None side effect in _ensure_resource_provider https://review.openstack.org/566161
00:38:37 openstackgerrit Merged openstack/nova master: Use test_base symbols directly https://review.openstack.org/564592
01:11:30 openstackgerrit Mohammed Naser proposed openstack/nova master: Added ability to configure default architecture for ImagePropertiesFilter https://review.openstack.org/566425
01:58:01 openstackgerrit Merged openstack/nova master: Use helpers in test_resource_provider (func) https://review.openstack.org/564638
14:45:31 openstackgerrit Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604
#openstack-nova - 2018-05-06
07:22:23 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Consider nested RPs in get_all_with_shared https://review.openstack.org/556450
07:22:24 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support shared and nested allocation candidates https://review.openstack.org/556514
07:22:25 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands without shared RPs https://review.openstack.org/561842
07:22:26 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045
07:22:27 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree https://review.openstack.org/559480
07:22:28 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return resource providers without inventories https://review.openstack.org/559554
07:22:29 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support nested allocation candidates in placement https://review.openstack.org/565487
08:49:59 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support nested allocation candidates in placement https://review.openstack.org/565487
18:32:57 openstackgerrit Theodoros Tsioutsias proposed openstack/nova master: Add scheduling notification https://review.openstack.org/566470
19:48:15 openstackgerrit Theodoros Tsioutsias proposed openstack/nova master: Introduce the PENDING instance state https://review.openstack.org/566473
22:23:31 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Fix the policy argument in server-group-create https://review.openstack.org/564699
22:30:13 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_compute_api.py (1) https://review.openstack.org/564415
22:30:30 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_conductor.py (1) https://review.openstack.org/564658
22:30:42 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_conductor.py (2) https://review.openstack.org/564659
22:30:54 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_live_migrate.py https://review.openstack.org/564656
22:31:06 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/virt/xenapi/test_vmops.py https://review.openstack.org/561846
22:31:22 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (1) https://review.openstack.org/564639
22:31:32 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (2) https://review.openstack.org/564643
22:31:43 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (3) https://review.openstack.org/564645
22:32:04 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in tests/unit/api/*/test_volumes.py https://review.openstack.org/564655
22:32:27 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc https://review.openstack.org/529520
#openstack-nova - 2018-05-07
00:30:20 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153
01:34:11 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201
01:54:17 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
01:54:26 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler https://review.openstack.org/563077
01:56:39 openstackgerrit Merged openstack/nova-specs master: Handle rebuild of instance with new image https://review.openstack.org/560718
03:01:56 openstackgerrit fupingxie proposed openstack/nova master: Remove stale pip-missing-reqs tox test https://review.openstack.org/566498
05:25:48 openstackgerrit Nguyen Hai proposed openstack/nova-specs master: Follow the new PTI for document build https://review.openstack.org/551802
08:16:50 openstackgerrit Surya Seetharaman proposed openstack/nova stable/queens: Make association_refresh configurable https://review.openstack.org/566288
08:44:59 openstackgerrit zhangyangyang proposed openstack/nova master: Remove the function get_backdoor_port() https://review.openstack.org/566219
08:49:29 bauzas so in case you want me reviewing things, ping me directly
09:03:46 openstackgerrit Vishakha Agarwal proposed openstack/nova master: Re-using the code of os brick cinder https://review.openstack.org/565703
09:05:43 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in tests/unit/api/openstack/compute https://review.openstack.org/566528
09:14:12 mrf Hi
09:14:30 mrf we reinstalled a compute node queens
09:14:41 mrf and we got error like:
09:14:47 mrf <mrf> 2018-05-07 07:25:31.268 16682 ERROR nova.compute.manager [req-5c63e054-a5a7-4d53-84b8-6ae24805f83a - - - - -] Error updating resources for node compute002.openstackxxxxxxxx.: ResourceProviderCreationFailed: Failed to create resource provider compute002.openstackxxxxxxxx
09:14:47 mrf conflict when trying to complete your request.\n\n Conflicting resource provider name: compute002.openstackxxxxx already exists. ", "title": "Conflict"}]}.
09:14:47 mrf 2018-05-07 07:25:31.266 16682 ERROR nova.scheduler.client.report [req-5c63e054-a5a7-4d53-84b8-6ae24805f83a - - - - -] [req-2cabe222-19a1-46f3-8427-5e297caacec3] Failed to create resource provider record in placement API for UUID 57741047-fb10-4c55-a284-e8d39b84c9f5. Got 409: {"errors": [{"status": 409, "request_id": "req-2cabe222-19a1-46f3-8427-5e297caacec3", "detail": "There was a
09:48:34 swamireddy hi
09:49:28 swamireddy quick Q on libvirt section of nova conf file - can I mention 2 libvirt section in nova conf file..basically one for block storage-1 and next one for 2nd block storage-2
10:36:07 artom sean-k-mooney[m], sean-k-m1, whichever is the real one, you around? Have some quick questions about the Intel CI
11:50:51 openstackgerrit MultipleCrashes proposed openstack/nova master: NCP-2367-deallocate-network-retry https://review.openstack.org/563418
11:51:03 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (1) https://review.openstack.org/566555
12:26:03 openstackgerrit Artom Lifshitz proposed openstack/nova master: Add InstanceNUMATopology to LibvirtLiveMigrateData https://review.openstack.org/566398
12:51:17 openstackgerrit Artom Lifshitz proposed openstack/nova master: Add InstanceNUMATopology to LibvirtLiveMigrateData https://review.openstack.org/566398
12:57:30 mnaser morning everyone
12:57:33 openstackgerrit Jake Yip proposed openstack/nova master: Add --before to nova-manage db archive_deleted_rows https://review.openstack.org/556751
12:57:39 mnaser it looks liek nova-next job is broken
12:58:05 mnaser http://zuul.openstack.org/builds.html?job_name=nova-next
12:58:20 mnaser oh i guess this is the fix https://review.openstack.org/#/c/566446/
13:26:14 lei-zh ping efried
13:26:22 efried lei-zh: Howdy.
13:26:32 efried finucannot: Didn't you have a patch for https://review.openstack.org/#/c/566498/ ?

Earlier   Later