Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-04-16
20:03:18 sean-k-mooney dansmith: ill try and take a look at the libvirt source code tomorrow and see whent there parse has defiend
20:03:56 dansmith sean-k-mooney: okay
20:11:17 openstackgerrit Artom Lifshitz proposed openstack/nova master: Add (Same|Different)HostFilter to defaults https://review.openstack.org/561651
20:13:32 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
20:13:33 openstackgerrit Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158
20:18:03 openstackgerrit Lance Bragstad proposed openstack/nova master: Set scope_types for admin password policies https://review.openstack.org/553613
20:18:05 openstackgerrit Lance Bragstad proposed openstack/nova master: Add scope_types to server policies https://review.openstack.org/525772
20:19:54 lbragstad johnthetubaguy_: jaypipes ^ look another shot at working a rough test structure into those
20:20:36 lbragstad they will fail, but i'm curious about the approach you'd like me to take with the testing
20:20:52 lbragstad or if what's proposed even makes sense
20:22:04 jaypipes lbragstad: cool, thanks
20:22:12 jaypipes lbragstad: will re-review shortly.
20:22:30 lbragstad awesome - ty
20:54:35 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: List instances performace optimization https://review.openstack.org/561715
20:54:49 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
20:54:50 openstackgerrit Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158
20:56:23 openstackgerrit Eric Fried proposed openstack/nova-specs master: Support forced separation of providers in granular https://review.openstack.org/561717
21:03:19 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove vestigial system_metadata param from info_from_instance() https://review.openstack.org/561724
21:27:00 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
21:27:01 openstackgerrit Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158
21:32:42 jackie-truong mriedem dansmith jaypipes melwitt johnthetubaguy_: Thanks for reviewing the certificate validation patches. 479949 and 561262 should just need one last look. The API patch is going through Zuul now.
21:33:00 jackie-truong https://review.openstack.org/#/c/479949/
21:33:05 jackie-truong https://review.openstack.org/#/c/561262/
21:39:39 melwitt thanks jackie-truong. I've rechecked the tempest tests so we can look at that too for the latest revision of the series https://review.openstack.org/#/c/515210
21:42:50 jackie-truong melwitt: Awesome, thanks
22:15:02 mriedem artom: gotta say, i'm not a fan of this https://review.openstack.org/#/c/561651/
22:15:31 cfriesen mriedem: I reproduced the Service UUID issue with vanilla stable/pike devstack. Writing up a bug now.
22:15:31 mriedem if there is a problem with tempest, let's fix tempest
22:15:50 mriedem artom: as noted inline, those tempest tests for these filters probably shouldn't even live in tempest
22:18:57 mriedem jackie-truong: besides dumping on artom's patch i'm done for the day, sorry
22:19:04 mriedem cfriesen: ok
22:19:34 jackie-truong mriedem: np, thanks for all of the comments!
22:21:16 melwitt jackie-truong: looks like there's a problem :( http://logs.openstack.org/10/515210/31/check/barbican-simple-crypto-devstack-tempest/0ff6bcc/logs/testr_results.html.gz
22:21:23 melwitt Additional properties are not allowed (u'trusted_image_certificates' was unexpected)", u'code': 400
22:21:31 melwitt looks like a schema validation fail?
22:24:08 jackie-truong melwitt: Saw that. Looking into it
22:47:56 openstack Launchpad bug 1764556 in OpenStack Compute (nova) ""nova list" fails with exception.ServiceNotFound if service is deleted and has no UUID" [Undecided,New]
22:47:56 cfriesen melwitt: mriedem: here's the Service-without-UUID exception.ServiceNotFound issue: https://bugs.launchpad.net/nova/+bug/1764556
22:48:13 melwitt thanks
22:48:30 cfriesen reproducing is straightforward, just needs a fake DB entry in the services table
23:01:44 cfriesen why does "openstack server --live" require a hostname?
23:02:00 cfriesen make that "openstack server migrate --live"
23:09:50 artom mriedem, yeah, I get it
23:10:32 artom We were having downstream discussions about it, and the argument that sort of got me was "if every CI has to change it, shouldn't it be in defaults?"
23:11:10 artom Devstack enables it, downstream OSP CI needs to enable it, so...
23:11:28 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_serversV21.py (1) https://review.openstack.org/560821
23:11:33 artom But yeah, "CI needs it" isn't a massively compelling argument
23:12:15 artom So I threw it up, just to see what discussion would come out of it, and to be able to say "upstream said" as opposed to "I said no" :)
23:12:34 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_serversV21.py (2) https://review.openstack.org/560829
23:15:53 melwitt artom: I think it makes sense that CIs would enable most, if not all of the filters that exist, to cover the features and make sure they work with their products
23:16:56 melwitt for deciding whether to include certain filters in default_filters, I think that would be mostly about whether the filters are used by "most users" or are useful in a typical deployment
23:17:20 artom melwitt, yeah, apparently it's really complicated for them to enabled it in their thousand+ jobs?
23:17:33 artom melwitt, but I agree - CI and users have different needs
23:19:26 melwitt I didn't think it's complicated in that I thought you just set enabled_filters to the filters you want enabled in nova.conf
23:19:57 artom melwitt, yeah, but apparently they need to do that for *every* job? As in, there's no common deployment that covers all jobs?
23:20:31 melwitt that said, I don't really have an opinion about whether SameHost and DifferentHost belong in default filters because TBH I don't know if those are really commonly used by end users or not
23:20:34 artom https://review.gerrithub.io/#/c/407794/ is the infrared change
23:20:53 artom Which apparently got -2 by Arie? I have no idea what's going on
23:21:23 artom melwitt, heh, don't we have those sosreports we can use for insights? At least for OSP users?
23:21:36 melwitt yeah, we could I would think
23:21:44 artom Man, the effort:complexity ratio of this thing is through the roof
23:22:00 artom There's, like, 5 of us, just to enable a filter
23:22:22 melwitt well, enabling a filter is supposed to be all pluggable, like you can do it in your nova.conf
23:23:41 melwitt this is starting a discussion about "what belongs in defaults?" "should everything be in defaults?" "what's the downside of putting everything in defaults?" and I don't really know the answers to those questions off the top of my head
23:24:29 artom Right, but the original problem is still "a filter that we need isn't enabled"
23:26:06 melwitt by default, no. there are a lot of filters we could say that about. so to me, it's more like, should we just add all filters to defaults and is there any reason to not do that? I'd be asking scheduler subteam folk about it
23:36:05 artom melwitt, yeah, it might be better to rephrase this as 2 questions: 1. do we have data that justifies adding same/different hostfilter to the defaults?
23:36:23 artom 2. Should we just enable all filters by default?
23:36:45 artom I can poke around our sosreports for 1 tomorrow, for 2... I guess tomorrow as well
23:37:48 cfriesen melwitt: https://review.openstack.org/#/c/560690/ has been updated to address your final comment
23:37:50 melwitt artom: yeah. the point of the default filters is to give someone a config for a typical cloud. if same/different are typically used, then they should be in defaults I think. CI are a different beast IMHO because they're aiming to run and verify as many features and capabilities as possible
23:38:36 melwitt so you would expect them to enable everything they possibly could. that said, IIUC there's some mismatch between what tempest defaults to and what nova defaults to and that is what's actually causing this problem
23:39:49 melwitt cfriesen: thanks, will take another look
23:40:25 cfriesen melwitt: the default set of enabled filters is actually pretty small
23:40:42 melwitt cfriesen: yes. I have learned that recently
23:41:38 melwitt or re-learned it. I've seen default filter list in the past and eventually forgot it
23:55:27 openstackgerrit Merged openstack/nova master: Remove RequestContext.instance_lock_checked https://review.openstack.org/554378
#openstack-nova - 2018-04-17
01:06:32 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.61 - support extra_specs in flavor API https://review.openstack.org/561520
01:12:57 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.61 - support extra_specs in flavor API https://review.openstack.org/561520
01:16:47 arvindn051 how do we get host uuid from hostname?
01:17:07 arvindn051 i am trying to get the host uuid for host specified in the request_spec
01:40:11 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045
01:40:12 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree https://review.openstack.org/559480
01:40:13 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return resource providers without inventories https://review.openstack.org/559554
01:54:33 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.62 - List/Show all server migration types https://review.openstack.org/430839
02:29:42 gmann alex_xu: ping
02:35:24 openstackgerrit Dinesh Bhor proposed openstack/nova master: WIP: PCPU: Define numa dedicated CPU resource class https://review.openstack.org/561770
02:37:03 openstackgerrit Dinesh Bhor proposed openstack/nova master: WIP: PCPU, VCPU: Add respective conf options https://review.openstack.org/561771
02:40:31 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Fix comments in novaclient/tests/unit/fakes.py https://review.openstack.org/561772
02:44:50 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add host/hostId to instance action events API https://review.openstack.org/557145
02:52:02 openstackgerrit Tetsuro Nakamura proposed openstack/nova-specs master: Return resources of entire trees in Placement https://review.openstack.org/559466
03:11:35 openstackgerrit jichenjc proposed openstack/nova master: Enhance api-guide general info https://review.openstack.org/561773
03:30:12 openstackgerrit Nguyen Hai proposed openstack/nova-specs master: Follow the new PTI for document build https://review.openstack.org/551802
03:30:54 openstackgerrit Nguyen Hai proposed openstack/nova-specs master: Follow the new PTI for document build https://review.openstack.org/551802
03:57:54 jichen melwitt: hi regarding your comments in the ML about z/VM driver
03:59:21 jichen melwitt: the z/VM driver behaves unlike any other in-tree driver [0-5] ---> we will make our code exactly as you pointed , we will NOT do anything special
04:14:03 openstackgerrit Nguyen Hai proposed openstack/nova-specs master: Follow the new PTI for document build https://review.openstack.org/551802
04:24:50 openstackgerrit Nguyen Hai proposed openstack/nova-specs master: Follow the new PTI for document build https://review.openstack.org/551802
06:20:24 openstackgerrit Merged openstack/nova master: remove ec2 in service and cmd https://review.openstack.org/556778
06:53:04 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi: Use XAPI pool instead of aggregate pool for shared SR migration https://review.openstack.org/554154

Earlier   Later