Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-03-07
22:38:55 mriedem yw
22:38:56 inc0 no, not the same:)
22:38:59 inc0 but close enough:)
22:39:32 mriedem i asked a russian at the ptg if they vacation in crimea, which apparently was a faux pas...so par for the course
22:40:18 inc0 don't worry, it's getting confusing, in Poland for example time difference to UK changes from 1hr to 2hrs every 6 months
22:40:43 mriedem that's the same in the US
22:40:46 mriedem except for arizona
22:40:51 inc0 daylight savings \o/
22:40:59 inc0 saves no daylight since 1800s
22:41:00 mriedem yeah which is here in another week or so
22:41:15 mriedem it saves me from being able to sleep properly
22:42:38 sdague that reminds me, I really need to find another venue for my "when will then be now" talk about time in software :)
22:43:45 openstackgerrit Eric Brown proposed openstack/python-novaclient master: Remove py34 tox env and pypi classifier https://review.openstack.org/442847
22:43:57 mriedem sdague: as in not AZ?
22:44:34 sdague it included all kinds of fun about dates, timezones, dst, the leapsecond
22:45:51 inc0 duh running qemu with nova user fixes it
22:46:00 inc0 but it's no bueno security
22:47:04 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: List instances using Searchlight (spec) https://review.openstack.org/441692
22:47:10 inc0 anyway guys, please check out libvirt 2.5.0, it potentially breaks lots of stuff
22:48:14 inc0 I mean it's not horrible if only we could make vm files being group-writable
22:50:35 inc0 I think when nova creates dirs for instance it uses default umask which would be 722, if we could make it 772, that would fix it
23:11:07 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Fix paths when moving specs https://review.openstack.org/432470
23:11:07 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Fix a couple of spec names https://review.openstack.org/432469
23:11:08 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Move implemented specs for Ocata https://review.openstack.org/432471
23:17:11 dansmith mriedem: did you see my question about the sort order thing?
23:18:06 dansmith mriedem: I think the sort order is just there to ensure a stable marker search within a single query, not for the overall sort of the results
23:20:41 mriedem looking
23:22:29 dansmith ah yeah diana just confirmed
23:23:43 mriedem me too
23:23:47 mriedem the marker is the uuid
23:23:52 mriedem and i sort of remember talking about this in the spec,
23:24:09 mriedem that uuid being a marker is silly because you can create a new instance and it's random uuid could mess up your marker/sort order
23:24:31 dansmith meh, pagination is lossy anyway
23:24:39 dansmith that doesn't bother me
23:24:59 dansmith so I'm good to push right?
23:25:07 mriedem umm
23:25:12 mriedem so isn't she saying we should be sorting?
23:25:17 mriedem at least by project_id?
23:25:25 dansmith no?
23:25:37 mriedem i'm going to read https://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/paginate-simple-tenant-usage.html quick
23:25:47 mriedem https://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/paginate-simple-tenant-usage.html#data-model-impact
23:26:13 dansmith the only reason we'd need to sort the results is if you care about the sort order as a user right?
23:26:47 mriedem then why was it added to https://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/paginate-simple-tenant-usage.html#data-model-impact?
23:27:05 mriedem https://review.openstack.org/#/c/386771/
23:27:18 dansmith what I have doesn't affect the stability of the sequence across cells
23:27:46 dansmith all that matters is that the last uuid of the end of your limit query lands you in the same place when you go back digging through the cells for the next page
23:29:06 dansmith oh you think she means sorted results to the user _was_ a requirement
23:29:25 mriedem yes
23:29:31 dansmith I read that as "fell out" like, it wasn't in the spec and the code was never updated
23:29:39 dansmith but I guess that's not what she means
23:29:41 mriedem there is a bunch of discussion between alex_xu and diana and jaypipes and johnthetubaguy in https://review.openstack.org/#/c/386771/ about sorting
23:30:04 dansmith so this is even larger of a merge sort than list instances right?
23:30:20 dansmith I mean, this is potentially an order of magnitude or more larger than just a list of live instances
23:31:34 mriedem depends on the window i suppose
23:32:16 mriedem i'm wondering if we only need to sort if detailed=True
23:32:29 dansmith okay then I'm going to move this to later in the set so we can progress on some of the other ones while we argue about this forever
23:32:32 mriedem but, it might also be dependent on the limit/marker
23:33:08 mriedem if you get a tenant_id then all it does is sort on the uuids
23:33:14 openstackgerrit Dan Smith proposed openstack/nova master: Enlighten server tags API about cells https://review.openstack.org/440856
23:33:15 openstackgerrit Dan Smith proposed openstack/nova master: Change MQ targeting to honor only what is in the context https://review.openstack.org/438023
23:33:15 openstackgerrit Dan Smith proposed openstack/nova master: Target cell in super conductor operations https://review.openstack.org/438022
23:33:16 openstackgerrit Dan Smith proposed openstack/nova master: Make consoleauth target the proper cell https://review.openstack.org/441461
23:33:16 openstackgerrit Dan Smith proposed openstack/nova master: Teach os-migrations about cells https://review.openstack.org/442685
23:33:17 openstackgerrit Dan Smith proposed openstack/nova master: WIP: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025
23:33:17 openstackgerrit Dan Smith proposed openstack/nova master: Add cell field to Destination object https://review.openstack.org/438024
23:33:18 openstackgerrit Dan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db https://review.openstack.org/439754
23:33:18 openstackgerrit Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736
23:33:19 openstackgerrit Dan Smith proposed openstack/nova master: Make os-availability-zones know about cells https://review.openstack.org/442163
23:33:19 openstackgerrit Dan Smith proposed openstack/nova master: Teach HostAPI about cells https://review.openstack.org/442162
23:33:20 openstackgerrit Dan Smith proposed openstack/nova master: Teach simple_tenant_usage about cells https://review.openstack.org/441388
23:33:20 openstackgerrit Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891
23:33:21 openstackgerrit Dan Smith proposed openstack/nova master: Remove Mitaka-era service version check https://review.openstack.org/442861
23:33:21 openstackgerrit Dan Smith proposed openstack/nova master: Teach os-aggregates about cells https://review.openstack.org/442200
23:34:19 dansmith I really don't understand the sorting discussion on that spec
23:34:38 mriedem need diana and alex_xu to explain
23:34:47 mriedem i can't read through all of that and "sort" it out
23:34:51 mriedem get it
23:34:52 dansmith yeah
23:35:09 mriedem i'm literally laughing here to myself and my own joke
23:35:18 mriedem i think it's time to head home
23:51:28 openstackgerrit Jin Li proposed openstack/nova master: Transform instance.soft_delete notifications https://review.openstack.org/383970
#openstack-nova - 2017-03-08
00:31:43 openstackgerrit Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891
00:31:43 openstackgerrit Dan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db https://review.openstack.org/439754
00:31:44 openstackgerrit Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736
00:31:44 openstackgerrit Dan Smith proposed openstack/nova master: Remove Mitaka-era service version check https://review.openstack.org/442861
00:59:31 openstackgerrit Li Yingjun proposed openstack/nova-specs master: Spec for support set vgamem for video https://review.openstack.org/438787
01:14:29 openstackgerrit Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387
01:14:29 openstackgerrit Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622
01:14:31 openstackgerrit Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged volume attachment https://review.openstack.org/418100
01:14:32 openstackgerrit Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082
01:14:34 openstackgerrit Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941
01:14:36 openstackgerrit Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021
01:14:38 openstackgerrit Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment https://review.openstack.org/415520
02:11:40 openstackgerrit Maho Koshiya proposed openstack/nova master: Add interfaces functional negative tests https://review.openstack.org/442892
02:24:01 alex_xu dansmith: sort by 'uuid' is for stable sort result, since the 'project_id' isn't unique value. at least we should keep the sort result stable.
02:24:28 alex_xu dansmith: looks like CellMappingList.get_all have problem, it haven't any specific sort key
02:24:33 alex_xu so the result can be unstable
02:24:53 dansmith alex_xu: it'll be stable, but not guaranteed
02:25:17 alex_xu dansmith: yes, but better to have a specific sort key?
02:25:24 dansmith alex_xu: but sure we can make cellmapping sort explicitly

Earlier   Later