Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-02-15
10:13:30 gibi efried, bauzas: NUMA spec is now approved
10:13:34 gibi nice work!
10:25:13 openstackgerrit Merged openstack/nova-specs master: Proposes NUMA topology with RPs https://review.opendev.org/552924
10:33:10 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Support re-configure deleted_on_termination in server https://review.opendev.org/580336
10:34:26 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Support re-configure deleted_on_termination in server https://review.opendev.org/580336
#openstack-nova - 2020-02-16
20:16:09 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP libvirt: Support attaching RBD volumes as host block devices https://review.opendev.org/708029
20:17:19 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP workarounds: Add option to disable native LUKSv1 decryption by QEMU https://review.opendev.org/708030
20:32:24 openstackgerrit Lee Yarwood proposed openstack/nova master: DNM: Test RBD block device support https://review.opendev.org/708033
20:53:18 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP workarounds: Add option to disable native LUKSv1 decryption by QEMU https://review.opendev.org/708030
20:53:19 openstackgerrit Lee Yarwood proposed openstack/nova master: DNM: Test RBD block devices with dm-crypt encryptor workaround https://review.opendev.org/708036
20:54:35 openstackgerrit Lee Yarwood proposed openstack/nova master: DNM: Test RBD block device support https://review.opendev.org/708033
#openstack-nova - 2020-02-17
05:41:44 openstackgerrit Abhishek Kekane proposed openstack/nova-specs master: Fixed minor suggestions https://review.opendev.org/708058
07:37:21 openstackgerrit Kobi Samoray proposed openstack/nova master: Support large network queries towards neutron https://review.opendev.org/704520
07:39:04 openstackgerrit Kobi Samoray proposed openstack/nova master: Support large network queries towards neutron https://review.opendev.org/704520
08:26:10 bauzas gibi: thanks for +Wing https://review.opendev.org/#/c/552924/ I'll create a FUP change
09:23:08 gibi bauzas: I was happy to push it through
09:23:54 bauzas cool thanks
10:23:49 openstackgerrit Brin Zhang proposed openstack/nova master: Add new default roles in os-instance-actions policies https://review.opendev.org/706470
10:30:39 openstackgerrit Brin Zhang proposed openstack/nova master: Add new default roles in os-instance-actions policies https://review.opendev.org/706470
10:40:26 openstackgerrit Sylvain Bauza proposed openstack/nova-specs master: FUP for NUMA Topology spec https://review.opendev.org/708088
10:40:46 bauzas gibi: ^
10:40:53 gibi bauzas: ack
10:42:13 gibi bauzas: thanks, +2
10:42:20 bauzas kk thanks
11:32:29 openstackgerrit Kevin Zhao proposed openstack/nova master: [WIP] Add arm64 CI https://review.opendev.org/708102
11:36:46 openstackgerrit Kevin Zhao proposed openstack/nova master: [WIP] Add arm64 CI https://review.opendev.org/708102
11:42:14 openstackgerrit Kevin Zhao proposed openstack/nova master: [WIP] Add arm64 CI https://review.opendev.org/708102
12:11:12 openstackgerrit Luyao Zhong proposed openstack/nova master: bug-fix: Reject live migration with vpmem https://review.opendev.org/708110
12:14:07 openstackgerrit Kevin Zhao proposed openstack/nova master: [WIP] Add arm64 CI https://review.opendev.org/708102
12:25:09 openstackgerrit Kevin Zhao proposed openstack/nova master: [WIP] Add arm64 CI https://review.opendev.org/708102
12:35:37 openstackgerrit Kevin Zhao proposed openstack/nova master: [WIP] Add arm64 CI https://review.opendev.org/708102
12:47:00 openstackgerrit Kevin Zhao proposed openstack/nova master: [WIP] Add arm64 CI https://review.opendev.org/708102
12:49:40 openstackgerrit Kevin Zhao proposed openstack/nova master: [WIP] Add arm64 CI https://review.opendev.org/708102
12:55:35 sean-k-mooney bauzas: gibi by the way rather then adapt the exitisting filter to work on allocation candiates we might actully want to a seperate filtering laywer that works on allocatgion candiates before the existing filters
12:56:10 gibi sean-k-mooney: that could be an alternative
12:56:30 sean-k-mooney that would avoid needing to pass an addtional paramater to all the filters so we wont break out of tree filters by changing the interface
12:56:37 gibi sean-k-mooney: it has a trade off to change the order of the filters as then NTF would run before any other filter
12:57:10 gibi but the such order just affects scheduling performance not scheduling result
12:57:32 sean-k-mooney im suggesting we leave the NTF unchanged and add a new one but yes the oder does affect performance
12:57:38 sean-k-mooney *order
12:58:16 sean-k-mooney you could put the new filter after the exstiing onece i guess
12:58:37 sean-k-mooney the NTF normally should go at the end of the existing list anyway
12:58:45 sean-k-mooney so maybe that would make more sense
12:59:42 sean-k-mooney so prefilters->placemnt->currnent filters->allocation candiate filters->weighers?
13:01:49 gibi sean-k-mooney: yeah, NTF tend to be at the end of the list so having the ac filter at the end would be closer to todays behavior
13:02:31 sean-k-mooney from an information gain point of view the host based fileter have the potentail to eliminate multipel allocation candiates by removing the host
13:02:40 sean-k-mooney so i think it makes sesne for them to come after too
13:02:52 bauzas sean-k-mooney: well, let's discuss this during the implementation
13:03:00 sean-k-mooney ya we can
13:03:17 sean-k-mooney i was just wondering if we could avoid breaking out of tree filters
13:03:27 sean-k-mooney which i think this would allow
13:07:18 openstackgerrit Brian Rosmaita proposed openstack/nova master: Reject boot request for unsupported images https://review.opendev.org/708126
13:20:09 gibi stephenfin: do you have some ideas why jsonutils fails while json does not in https://review.opendev.org/#/c/703133 ?
13:20:55 stephenfin have you got a traceback?
13:21:28 gibi stephenfin: sure, give me a sec
13:21:51 gibi stephenfin: http://paste.openstack.org/show/789642/
13:22:21 gibi stephenfin: are nova uses old oslo.serialization that is not py37 compatible?
13:23:11 sean-k-mooney it would seam that we are mixing byte and text strings somehwere
13:23:22 sean-k-mooney but this would have broken on py3.5
13:23:33 sean-k-mooney so its not a py 3.7 thing
13:26:56 gibi sean-k-mooney: right. but the combination for byte and str seems to happen in codecs which is stdlib
13:27:23 sean-k-mooney right so we are expecting to pass in a string
13:27:36 stephenfin gibi: looking at the source, it seems we attempt to decode whatever file we get to utf-8
13:27:37 sean-k-mooney and its expecting bytes?
13:27:46 stephenfin but I don't know why that's no okay
13:28:06 sean-k-mooney you mean https://github.com/openstack/oslo.serialization/blob/master/oslo_serialization/jsonutils.py#L255-L276
13:28:20 stephenfin at meetings for the day but I'll take a look over the week
13:37:16 stephenfin bauzas: You think you could take a look at this today? https://review.opendev.org/#/c/662522/
13:41:14 bauzas stephenfin: ok, will look today
13:57:06 gibi hm. If I change the test to load the content of the file first and then call jsonutils.loads() then it works
13:57:09 gibi with open(schema_file) as f:
13:57:11 gibi data = f.read()
13:57:14 gibi schema = jsonutils.loads(data)
14:07:39 openstackgerrit Brian Rosmaita proposed openstack/nova master: Reject boot request for unsupported images https://review.opendev.org/707738
14:28:13 openstackgerrit Brian Rosmaita proposed openstack/nova master: Absolutely-non-inheritable image properties https://review.opendev.org/708126
15:04:15 openstackgerrit Artom Lifshitz proposed openstack/nova master: func tests: move _run_periodics() into own mixin https://review.opendev.org/705545
15:04:15 openstackgerrit Artom Lifshitz proposed openstack/nova master: tests: work around malformed serial XML https://review.opendev.org/705546
15:04:16 openstackgerrit Artom Lifshitz proposed openstack/nova master: Functional tests for NUMA live migration https://review.opendev.org/672595
15:04:17 openstackgerrit Artom Lifshitz proposed openstack/nova master: NUMA LM: Add func test for bug 1845146 https://review.opendev.org/687404
15:04:17 openstack bug 1845146 in OpenStack Compute (nova) train "NUMA aware live migration failed when vCPU pin set" [High,Fix committed] https://launchpad.net/bugs/1845146 - Assigned to Dan Smith (danms)
15:14:10 openstackgerrit Brian Rosmaita proposed openstack/nova master: Absolutely-non-inheritable image properties https://review.opendev.org/708126
15:21:10 openstackgerrit Brian Rosmaita proposed openstack/nova master: Reject boot request for unsupported images https://review.opendev.org/707738
15:23:35 artom So I guess something Fedora-specific is wrong with the stable/queens func tests?
15:24:01 artom Because locally I don't get the same failure as https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_c05/707887/1/check/nova-tox-functional-py35/c05109a/testr_results.html
15:34:53 rosmaita alex_xu: gmann: need your opinion on whether https://review.opendev.org/#/c/707738/ will require a microversion bump
15:38:46 sean-k-mooney artom: the functional tests should not be os specific nor should any of the other tests really
15:40:10 artom sean-k-mooney, I know, which is why it's so weird
15:40:14 artom In any case, it's a moot point
15:40:28 artom Because turns out my backport isn't applicable, and I got the root cause completely wrong
15:40:44 artom So back to square 1 for me, and I really don't feel like log diving
15:40:48 artom But no choice :(
15:42:56 openstackgerrit Eric Fried proposed openstack/nova-specs master: Fixed minor suggestions https://review.opendev.org/708058
15:43:35 openstackgerrit Eric Fried proposed openstack/nova-specs master: Fixed minor suggestions https://review.opendev.org/708058
16:04:38 openstackgerrit Merged openstack/nova master: Support large network queries towards neutron https://review.opendev.org/704520
16:06:00 bauzas stephenfin: FWIW, have a comment https://review.opendev.org/#/c/662522/17
16:06:17 bauzas stephenfin: just tell me when you see it and then I'll +W it
16:07:08 stephenfin bauzas: Do you mean something more than I provided? https://review.opendev.org/#/c/662522/17/nova/tests/functional/libvirt/test_numa_servers.py
16:07:31 sean-k-mooney efried: regarding https://review.opendev.org/#/c/706276/ i can remove the changes related to the one ussuri spec that got pulled in but for the removal of the template, looking at the other releases we keep it in the root of the specs folder an remove it in the approve/implemented folder
16:07:32 bauzas stephenfin: oh my bad
16:07:43 bauzas stephenfin: lemme look at it

Earlier   Later