Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-03-06
02:57:48 oomichi_ as the previous irc meeting, we are trying to test newer API versions(CURRENT) as possible instead of deprecated versions
02:58:17 oomichi_ v3 API is current on keystone now.
02:59:09 oomichi_ you can see the API version status of Keystone with https://review.openstack.org/#/c/441523/ btw
02:59:49 masayukig oomichi_: sure, I'll take a look later. BTW, why did you bring this on nova channel?
03:00:11 oomichi_ masayukig: because I was wrong ;)
03:00:58 masayukig oomichi_: heh
04:28:55 openstackgerrit John L. Villalovos proposed openstack/nova master: Use flake8-import-order https://review.openstack.org/435575
04:58:37 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/407237
04:58:57 openstackgerrit Takashi NATSUME proposed openstack/nova master: Refactor a test method including 7 test cases https://review.openstack.org/395436
04:59:21 openstackgerrit Takashi NATSUME proposed openstack/nova master: Refactor a test method including 3 test cases https://review.openstack.org/394738
05:00:05 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types https://review.openstack.org/430608
05:00:45 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444
05:01:11 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Fix formatting errors and warnings https://review.openstack.org/407364
05:02:49 openstackgerrit Takashi NATSUME proposed openstack/nova master: Avoid forcing translation on logging calls https://review.openstack.org/413876
05:03:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters and examples in aggregate API https://review.openstack.org/425512
05:54:54 openstackgerrit Dinesh Bhor proposed openstack/python-novaclient master: Fix ValueError when incorrect metadata passed https://review.openstack.org/441750
05:58:10 openstackgerrit Alex Xu proposed openstack/nova-specs master: The ResourceProviderTraits API - Manage traits with ResourceProviders https://review.openstack.org/345138
06:23:25 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix keypair API parameters https://review.openstack.org/420626
06:51:52 openstackgerrit Gao Yuan proposed openstack/nova master: Remove mox from nova.tests.unit.api.openstack.compute.test_block_device_mapping https://review.openstack.org/440475
07:44:22 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732
08:06:03 openstackgerrit Gao Yuan proposed openstack/nova master: Remove mox from nova.tests.unit.objects.test_instance.py https://review.openstack.org/441784
08:35:48 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Enable SR-IOV NIC offload feature discovery https://review.openstack.org/435954
09:01:14 lyarwood morning all, would anyone with stable +2 mind hitting https://review.openstack.org/#/c/431530/ & https://review.openstack.org/#/c/431540/ if they have time this morning
09:04:59 openstackgerrit Gao Yuan proposed openstack/nova master: Remove mox from nova.tests.unit.objects.test_instance.py https://review.openstack.org/441784
09:19:26 openstackgerrit Alex Xu proposed openstack/nova master: placement: Move the common code out of ResourceClassCache https://review.openstack.org/441828
09:19:26 openstackgerrit Alex Xu proposed openstack/nova master: placement: Add Trait and TraitList objects https://review.openstack.org/376199
09:19:27 openstackgerrit Alex Xu proposed openstack/nova master: placement: add cache for traits https://review.openstack.org/441829
09:21:09 bauzas oops, I did that in a wrong chan (-ptg) but
09:21:13 bauzas good morning Nova :)
09:21:26 bauzas lyarwood: looking
09:22:04 lyarwood bauzas: morning and thanks!
09:26:07 openstackgerrit Alex Xu proposed openstack/nova master: placement: Move the common code out of ResourceClassCache https://review.openstack.org/441828
09:26:07 openstackgerrit Alex Xu proposed openstack/nova master: placement: Add Trait and TraitList objects https://review.openstack.org/376199
09:26:08 openstackgerrit Alex Xu proposed openstack/nova master: placement: Add simple cache object for traits https://review.openstack.org/441829
09:29:31 openstackgerrit Alex Xu proposed openstack/nova master: placement: Move the common code out of ResourceClassCache https://review.openstack.org/416007
09:29:32 openstackgerrit Alex Xu proposed openstack/nova master: placement: Add simple cache object for traits https://review.openstack.org/441829
09:29:32 openstackgerrit Alex Xu proposed openstack/nova master: placement: Add Trait and TraitList objects https://review.openstack.org/376199
09:53:08 johnthetubaguy alex_xu: I think that restriction on Name is really recent, I am remembering that wrongly?
09:56:34 alex_xu johnthetubaguy: yes, we just didn't that in Ocata by the patch of server query parameter validation
09:56:54 alex_xu s/didn't/did/
09:58:07 johnthetubaguy alex_xu: so we do have better behaviour now, that when the bug was reported at least, but it seems really easy to "fix" the old behaviour, as proposed
09:58:18 johnthetubaguy s/that when/than when/
09:59:04 openstackgerrit Alin Balutoiu proposed openstack/os-vif master: vif_plug_ovs: Skip setting MTU on Windows when plugging devices https://review.openstack.org/441855
09:59:28 alex_xu johnthetubaguy: is that old behaviour? as my understand, the API always return 500 with invalid regex, so it should be never works in the before
10:00:03 alex_xu let me check the bug report, whether I missed something
10:00:35 johnthetubaguy alex_xu: what I mean is, the current behaviour is better than the old 500
10:00:46 johnthetubaguy alex_xu: its just odd from a user perspective
10:00:52 alex_xu johnthetubaguy: ah, yea
10:01:34 johnthetubaguy alex_xu: they have a list of servers, they are using filtering by name to show some server details, but some of the queries return 400 (or 500) instead of the server
10:02:29 johnthetubaguy defaulting to exact match when it was an invalid regex seemed quite nice, although I guess its hard to debug your regex that way
10:03:15 alex_xu johnthetubaguy: yea, and that sounds like API behaviour change, doesn't sounds like we can change API without microversion
10:04:52 johnthetubaguy if it were 500 -> results like when this was first proposed, it wouldn't be, which is the odd thing I guess
10:05:50 johnthetubaguy if it were name_regex= I wouldn't worry about this either I guess
10:06:29 bauzas lyarwood: AFAIR, nova unittests can run behind a mysql driver backend instead of sqlite
10:06:50 bauzas given the Database fixture does use the default oslo.db values for creating the DB
10:07:07 mdbooth_ bauzas: I did some work on this a couple of years ago and there were a few wrinkles but mostly it just worked
10:07:15 mdbooth_ For unit tests, that is
10:07:16 bauzas for functional tests, we need to make sure it does use the same path
10:07:38 lyarwood bauzas: yeah I think you need to actually configure mysql/mariadb to use the default creds for it to work right?
10:07:56 bauzas yeah
10:07:57 mdbooth_ IIRC, the biggest wrinkle at the time was that sqlite (used by default) allowed millisecond accuracy in timestamps, whereas mysql only does 1 second
10:08:15 mdbooth_ This broke a bunch of unit tests which wrote non-rounded seconds into the db and expected to read the same value back.
10:08:42 bauzas here, I suppose lyarwood wanna test some very specific ones
10:08:51 bauzas not running the whole job
10:09:09 lyarwood bauzas: well downstream we hit a func test failure caused by the sqlite version we are shipping in RHEL
10:09:15 alex_xu johnthetubaguy: yes, agree that is odd
10:09:24 mdbooth Although running the whole job isn't a bad idea. There are a bunch of very interesting semantic differences.
10:09:26 lyarwood bauzas: as we don't support sqlite we need to switch these jobs over to mariadb
10:09:28 alex_xu johnthetubaguy: just check the history, we fix the 500 at https://review.openstack.org/#/c/344945/
10:09:49 bauzas lyarwood: yeah I heard that from migi
10:09:58 johnthetubaguy alex_xu: yeah, I remember worrying about that bug fix we had pending when approving that one
10:10:34 bauzas lyarwood: so you would run a specific functional tests job backed by mariadb, right?
10:10:40 bauzas not all the UTs ?
10:11:54 lyarwood bauzas: well, all of the func tests that we can using mariadb
10:12:04 lyarwood bauzas: and any UTs that are using the DB as well tbh
10:12:31 bauzas okay, so fixing upstream the wrinkles that mdbooth said would then a good idea
10:12:53 bauzas at least, the first step is to configure the jobs and see what are the results
10:13:08 bauzas I'd only consider functional tests at a first step tho
10:13:15 alex_xu johnthetubaguy: so probably we can introduce "/servers?name=in:VM1,VM2" in the future
10:13:24 alex_xu for the exact match
10:13:36 bauzas alex_xu: I had a patch up for the in: tag a while ago
10:13:49 alex_xu bauzas: yea, I learned from that
10:14:07 johnthetubaguy alex_xu: oh... the in operator, thats a good way around that one
10:14:16 bauzas alex_xu: disclaimar, I haven't read the above discussion, so no context here yet
10:14:25 bauzas disclaimer* even
10:14:59 johnthetubaguy alex_xu: so I -2ed that patch, I can catch up with sneti on why I did that later on
10:15:38 alex_xu johnthetubaguy: ok, if sneti happy with that, we better to have a patch to tell the user how to match exact name in the current API behaviour
10:15:57 alex_xu in api-ref
10:16:22 johnthetubaguy alex_xu: yeah, I can't remember what we say in there right now
10:16:53 johnthetubaguy "Filters the response by a server name, as a string. You can use regular expressions in the query. For example, the ?name=bob regular expression returns both bob and bobb. If you must match on only bob, you can use a regular expression that matches the syntax of the underlying database server that is implemented for Compute, such as MySQL or PostgreSQL."
10:16:57 johnthetubaguy thats probably OK
10:17:55 alex_xu oops, that depend on the underlying database, that is bad :(
10:20:10 alex_xu and we validate the regex with python first...https://github.com/openstack/nova/blob/master/nova/api/validation/validators.py#L40
10:35:59 openstackgerrit sahid proposed openstack/nova master: numa: add numa constraints for emulator threads policy https://review.openstack.org/385023
10:36:01 openstackgerrit sahid proposed openstack/nova master: libvirt: return a CPU overhead if isolate emulator threads requested https://review.openstack.org/385364
10:36:02 openstackgerrit sahid proposed openstack/nova master: numa: fit instance topology with cpus reserved onto cores https://review.openstack.org/400875
10:36:04 openstackgerrit sahid proposed openstack/nova master: virt: update overhead to take into account vCPUs https://review.openstack.org/385024
10:36:07 openstackgerrit sahid proposed openstack/nova master: libvirt: make emulator threads to run on the reserved pCPU https://review.openstack.org/400878
10:36:08 openstackgerrit sahid proposed openstack/nova master: numa: update numa usage incording the cpu reserved https://review.openstack.org/400877
10:36:08 openstackgerrit sahid proposed openstack/nova master: numa: take into account cpus reserved https://review.openstack.org/400876

Earlier   Later