Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-08-12
20:23:56 mriedem *setdefault
20:25:24 bnemec Something like that, but you just call it once and the config object remembers that you overrode the default for that opt.
20:26:22 bnemec I guess dict inserts the key with the default so that is actually the same.
20:26:37 mriedem this is what i've got right now http://paste.openstack.org/show/756357/
20:28:22 mriedem this is probably better to avoid calling set_default when i don't need to
20:28:22 mriedem http://paste.openstack.org/show/756358/
20:28:34 mriedem otherwise i have to do this somewhere in wsgi init after parsing options and starting the service
20:29:03 mriedem which takes it quite far away from where it's used, which is ok but kind of hard to remember all of this
20:29:40 bnemec Heh, if you're still checking for None then I'm not sure you gain much from using set_default.
20:29:46 bnemec But either should work.
20:33:10 mriedem yeah true :/
20:35:34 bnemec It's probably not worth satisfying my pet peeve about the BoolOpt default by moving the setdefault call to some weird location though.
20:39:56 sean-k-mooney dict not only set the defult i return the value in the dict. so if it was already set it returns the value that was found in the dict and if not it returns you default after setting it in the dict
20:40:22 sean-k-mooney you can discard the return if you dont care but its a nice behavior to have
20:40:38 sean-k-mooney so if we had a CONF.set_default it would be nice to keep it the same
20:41:48 sean-k-mooney that would change your example to http://paste.openstack.org/show/756359/
20:43:01 sean-k-mooney its a minor change but its basicly free to return the value and it makes the useage terser
21:31:42 openstackgerrit Matt Riedemann proposed openstack/nova master: Allow disabling cold migration to the same host https://review.opendev.org/676022
22:02:14 openstackgerrit Matt Riedemann proposed openstack/nova master: Add nova-lxc job to the experimental queue https://review.opendev.org/676024
22:02:16 mriedem donnyd: sean-k-mooney: ^ and now i'm out o/
22:05:57 donnyd Yesssss
22:16:25 openstackgerrit Merged openstack/nova stable/queens: libvirt: move checking CONF.my_ip to init_host() https://review.opendev.org/672161
22:16:54 openstackgerrit Merged openstack/nova master: Avoid timeout from service update notification tests https://review.opendev.org/675935
22:22:58 openstackgerrit Eric Fried proposed openstack/nova master: Use SDK for node.list https://review.opendev.org/656027
22:52:15 openstackgerrit Eric Fried proposed openstack/nova master: Add nova-lxc job to the experimental queue https://review.opendev.org/676024
23:09:11 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Limit get_sdk_adapter to requested service type https://review.opendev.org/674678
23:40:22 openstackgerrit Takashi NATSUME proposed openstack/nova stable/stein: Retrun 400 if invalid query parameters are specified https://review.opendev.org/676026
23:53:51 openstackgerrit Takashi NATSUME proposed openstack/nova stable/rocky: Retrun 400 if invalid query parameters are specified https://review.opendev.org/676027
23:59:41 openstackgerrit Takashi NATSUME proposed openstack/nova stable/queens: Retrun 400 if invalid query parameters are specified https://review.opendev.org/676028
#openstack-nova - 2019-08-13
00:00:13 openstackgerrit Dustin Cowles proposed openstack/nova master: Provider config file schema and loader https://review.opendev.org/673341
00:00:14 openstackgerrit Dustin Cowles proposed openstack/nova master: WIP: Load custom providers to Resource Tracker https://review.opendev.org/676029
00:10:38 openstackgerrit Dustin Cowles proposed openstack/nova master: Provider config file schema and loader https://review.opendev.org/673341
00:10:38 openstackgerrit Dustin Cowles proposed openstack/nova master: WIP: Load custom providers to Resource Tracker https://review.opendev.org/676029
00:40:14 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix misuse of nova.objects.base.obj_equal_prims https://review.opendev.org/676030
00:45:10 openstackgerrit Filippo Inzaghi proposed openstack/nova-specs master: Add Python 3 Train unit tests https://review.opendev.org/675908
00:52:08 openstackgerrit Merged openstack/nova stable/ocata: Implement power_off/power_on for the FakeDriver https://review.opendev.org/674906
01:30:06 openstackgerrit zhufl proposed openstack/nova master: [Trivial]Remove unused helper _get_instances_by_filters https://review.opendev.org/676036
01:49:20 openstackgerrit zhufl proposed openstack/nova master: [Trivial]Remove unused helper filter_and_format_resource_metadata https://review.opendev.org/676038
02:51:33 openstackgerrit Merged openstack/nova master: Multiple API cleanup changes https://review.opendev.org/666889
02:51:42 openstackgerrit Merged openstack/nova master: DRY get_flavor in flavor manage tests https://review.opendev.org/668281
02:51:51 openstackgerrit Merged openstack/nova master: Add InstanceAction/Event create() method https://review.opendev.org/614036
02:52:04 openstackgerrit Merged openstack/nova master: Add Instance.hidden field https://review.opendev.org/631123
03:25:59 openstackgerrit Filippo Inzaghi proposed openstack/nova-specs master: Add Python 3 Train unit tests https://review.opendev.org/675908
03:31:32 openstackgerrit Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851
03:43:48 openstackgerrit Brin Zhang proposed openstack/nova master: Add delete_on_termination to volume-attach API https://review.opendev.org/673133
04:14:45 openstackgerrit Brin Zhang proposed openstack/python-novaclient master: Microversion 2.76: Support Specifying AZ to unshelve https://review.opendev.org/665136
04:28:08 openstackgerrit Dustin Cowles proposed openstack/nova master: Introduces SDK to IronicDriver and uses for node.get https://review.opendev.org/642899
04:28:09 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK for node.list https://review.opendev.org/656027
04:28:09 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK for validating instance and node https://review.opendev.org/656028
04:28:10 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK for setting instance id https://review.opendev.org/659690
04:28:10 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK for add/remove instance info from node https://review.opendev.org/659691
04:28:11 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK for getting network metadata from node https://review.opendev.org/670213
05:54:09 openstackgerrit Brin Zhang proposed openstack/nova master: Add user_id and project_id colume to Migration https://review.opendev.org/673990
06:09:40 openstackgerrit Lenny Verkhovsky proposed openstack/nova master: Revert "vif: Resolve a TODO and update another" https://review.opendev.org/675776
06:11:20 openstackgerrit Lenny Verkhovsky proposed openstack/nova master: Revert "vif: Resolve a TODO and update another" https://review.opendev.org/675776
06:16:34 openstackgerrit Brin Zhang proposed openstack/nova master: Filter migrations by user_id/project_id https://review.opendev.org/674243
07:28:24 gibi efried: I felt the same about importing the real fixtures in https://review.opendev.org/#/c/675935 ;)
08:14:03 tssurya good morning gmann, did you already propose a client side change for microversion 2.75 ? just asking so that I can rebase on client side change on that
08:46:05 zhouyao bauzas Can you tell me if nvram file can be stored in shared storage?
08:46:35 gmann tssurya: not yet, I am planning for today or tomorrow.
08:46:48 gmann i will ping the link.
08:46:57 tssurya gmann: thanks ! :)
09:01:04 openstackgerrit Balazs Gibizer proposed openstack/nova master: allow getting resource request of every bound ports of an instance https://review.opendev.org/655110
09:01:04 openstackgerrit Balazs Gibizer proposed openstack/nova master: Pass network API to the conducor's MigrationTask https://review.opendev.org/655111
09:01:05 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add request_spec to server move RPC calls https://review.opendev.org/655721
09:07:00 openstackgerrit Balazs Gibizer proposed openstack/nova master: re-calculate provider mapping during migration https://review.opendev.org/655112
09:09:03 openstackgerrit Balazs Gibizer proposed openstack/nova master: update allocation in binding profile during migrate https://review.opendev.org/656422
09:11:01 openstackgerrit Balazs Gibizer proposed openstack/nova master: Extend NeutronFixture to handle migrations https://review.opendev.org/655114
09:12:58 openstackgerrit Balazs Gibizer proposed openstack/nova master: prepare func test env for moving servers with bandwidth https://review.opendev.org/655109
09:12:59 openstackgerrit Balazs Gibizer proposed openstack/nova master: func test for migrate server with ports having resource request https://review.opendev.org/655113
09:17:00 openstackgerrit Balazs Gibizer proposed openstack/nova master: Make _rever_allocation nested allocation aware https://review.opendev.org/676138
09:19:00 openstackgerrit Balazs Gibizer proposed openstack/nova master: Func test for revert migration with bandwidth https://review.opendev.org/676140
09:29:37 openstackgerrit Stephen Finucane proposed openstack/nova master: Libvirt: add support for vPMU configuration. https://review.opendev.org/671338
09:56:43 lyarwood redkrieg: Apologies for missing your ping yesterday re stable device rescue, it's dropped down to the the bottom of the pile for me recently.
09:57:27 lyarwood redkrieg: there appears to be growing interest again so I might kick it back to life for the U cycle but I can't really guarantee anything at this point.
09:57:37 slaweq efried: hi
09:58:04 slaweq efried: I just opened bug https://bugs.launchpad.net/nova/+bug/1839961 as I see that this test if failing quite often
09:58:05 openstack Launchpad bug 1839961 in OpenStack Compute (nova) "Test tempest.api.compute.servers.test_novnc.NoVNCConsoleTestJSON.test_novnc failing often" [Undecided,New]
09:58:49 slaweq efried: please take a look at it if You will have few minutes, I opened it for nova but maybe it's e.g. tempest issue - I'm not 100% sure
10:30:08 shilpasd stephenfin: thanks for review 'isolate_aggregate' pre-request filter patches
10:30:34 shilpasd stephenfin: just small query on https://review.opendev.org/#/c/671075/6/nova/scheduler/request_filter.py@68 and @72
10:30:46 shilpasd if we have only one ResourceRequest, then we can omit union, right?
10:31:37 stephenfin shilpasd: Yeah, I think so. That's what we do in nova/scheduler/utils.py https://github.com/openstack/nova/blob/master/nova/scheduler/utils.py#L461-L483
10:32:05 stephenfin Ignore the numbered_groups_from_flavor piece and I think you have exactly what you want
10:33:01 stephenfin Also, there's a check there to see if 'Flavor.extra_specs' exists. However, I assumed it always would so I don't know why that's there. Maybe just tests?
10:35:55 shilpasd thanks for reference, will address accordingly, and in my changes will check do we need this check 'if 'Flavor.extra_specs' exists', then only add
10:36:29 shilpasd stephenfin: and for https://review.opendev.org/#/c/671074/5/nova/objects/aggregate.py@541
10:36:34 stephenfin Makes sense. To say everything is passing so far, I imagine you don't, but it would be no harm to trace the paths to see if the field is ever _not_ set
10:36:37 shilpasd will edit doc message
10:37:55 shilpasd sure, will check if field is ever _not_ set, is any causing issue
11:01:19 openstackgerrit Boxiang Zhu proposed openstack/nova master: Make evacuation respects anti-affinity rule https://review.opendev.org/649963
11:03:09 openstackgerrit Boxiang Zhu proposed openstack/nova master: Make evacuation respects anti-affinity rule https://review.opendev.org/649963
11:08:27 openstackgerrit Boxiang Zhu proposed openstack/nova master: Fix live migration break group policy simultaneously https://review.opendev.org/651969
11:44:52 openstackgerrit ya.wang proposed openstack/nova master: Support report driver migrate feature https://review.opendev.org/674522
11:44:52 openstackgerrit ya.wang proposed openstack/nova master: Add property check when create/rebuild/resize. https://review.opendev.org/674523
11:44:53 openstackgerrit ya.wang proposed openstack/nova master: Scheduler translate properties to traits https://review.opendev.org/674524
11:44:53 openstackgerrit ya.wang proposed openstack/nova master: Change method: _live_migration_flags https://review.opendev.org/674525
11:44:54 openstackgerrit ya.wang proposed openstack/nova master: Add releasenote and update docs. https://review.opendev.org/674526
11:59:05 mriedem stephenfin: replied in https://review.opendev.org/#/c/627892/
12:02:22 openstackgerrit Peter Penchev proposed openstack/nova master: libvirt: use native AIO mode for StorPool Cinder volumes. https://review.opendev.org/676172

Earlier   Later