Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-05-30
19:03:33 sean-k-mooney not remove im in train but if not ci by U then remove them
19:04:09 mriedem the vz ci is busted b/c https://review.opendev.org/#/c/506686/ won't merge
19:04:15 mriedem http://openstack-3rd-party-virtuozzo-ci-logs.virtuozzo.com/24/638324/30/check/check-dsvm-tempest-vz7-exe-minimal/620b3db/console.html.gz
19:04:22 mriedem 2019-05-24 02:17:01.109 | + :pre_test_hook:L14: git cherry-pick FETCH_HEAD 2019-05-24 02:17:01.311 | error: could not apply 789a1f2... don't add device address if there is no any units 2019-05-24 02:17:01.313 | hint: after resolving the conflicts, mark the corrected paths 2019-05-24 02:17:01.315 | hint: with 'git add <paths>' or 'git rm <paths>' 2019-05-24 02:17:01.318 | hint: and commit the result with 'git commit'
19:04:47 sean-k-mooney mriedem: oh ok
19:05:13 sean-k-mooney well in anycase we could wait or we could add warning for all those that are not tested in train and see if anyone shouts
19:05:58 sean-k-mooney i still vaguly plan to fix the mount issue for libvirt lxc but its a supper low priority at the moment
19:06:59 mriedem i don't really care if we remove libvirt+lxc, it hasn't worked in forever as far as i know
19:07:02 mriedem i couldn't get ci working for it either
19:07:57 mriedem https://docs.openstack.org/nova/latest/configuration/config.html#libvirt.virt_type
19:08:03 sean-k-mooney ill try and find time between now and m2 to either fix the mount issue so it can actully be used or submit a patch to remove it since its been broken for 2 years
19:08:05 mriedem lxc, uml and xen could probably all be deprecated from that list
19:08:12 sean-k-mooney yep
19:08:23 mriedem we can't remove those without deprecating them first
19:08:48 sean-k-mooney your right
19:09:06 sean-k-mooney ill either fix it or add a deprecation warning
19:10:11 sean-k-mooney i always liked lxc but if noone uses it and its broken then it would be nice to clean it up
19:11:16 sean-k-mooney that said with stephen deleteing nova network, nova-consoleauth and cells v1 this cycle this cycle that is likely enought for one release :)
19:15:32 lpetrut mriedem: thanks for letting me know, I'll look into it
19:15:59 mriedem lpetrut: thanks
19:19:45 mriedem speaking of deleting stuff, we should delete this old online data migration https://review.opendev.org/#/c/651001/
19:20:10 mriedem and start working on dropping the pike-era data migrations
19:28:12 efried thanks mriedem (re xen)
20:29:59 lyarwood mriedem: https://review.opendev.org/#/c/572790/ - Do you have bandwidth to backport this to stable/queens? Happy to do it if you don't, just wanted to asked first.
20:31:52 lyarwood and assuming you even think that's a good idea.
20:35:50 mriedem i can...
20:36:18 mriedem i think we can, i originally had the bug marked as backport to queens and then a year went by
20:36:24 mriedem will do that in a bit
20:37:13 lyarwood mriedem: cool thanks! I'll review stable/stein in the morning to get things moving.
20:39:34 openstackgerrit Merged openstack/nova master: Add get_usages_counts_for_quota to SchedulerReportClient https://review.opendev.org/653145
20:39:48 openstackgerrit Merged openstack/nova master: Count instances from mappings and cores/ram from placement https://review.opendev.org/638073
20:39:59 openstackgerrit Merged openstack/nova master: Set [quota]count_usage_from_placement = True in nova-next https://review.opendev.org/653146
20:53:31 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Optimize creating servers with tags https://review.opendev.org/484170
20:59:56 openstackgerrit Matt Riedemann proposed openstack/nova stable/stein: Block swap volume on volumes with >1 rw attachment https://review.opendev.org/662331
21:14:22 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Block swap volume on volumes with >1 rw attachment https://review.opendev.org/662333
21:25:30 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Block swap volume on volumes with >1 rw attachment https://review.opendev.org/662333
21:30:45 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Block swap volume on volumes with >1 rw attachment https://review.opendev.org/662340
21:45:43 openstackgerrit Eric Fried proposed openstack/os-traits master: Create traits for NUMA & device subtree affinity https://review.opendev.org/657898
21:48:34 aram1s hello! has anyone seen this error before: CantRemoveHost: Can’t remove host(s) [u'313c9b24-16f0-4b06-b862-62727288f358'] from Aggregate <Aggregate: 1>"
21:48:48 aram1s the hosts won't come out of our freepool
21:49:18 openstackgerrit Matt Riedemann proposed openstack/nova master: Add functional test for two-cell scheduler behaviors https://review.opendev.org/452006
21:49:52 mriedem aram1s: that looks like a blazar specific error, not nova http://codesearch.openstack.org/?q=CantRemoveHost&i=nope&files=&repos=
21:50:16 mriedem so you'd have to ask the blazar devs, or check the nova api logs to see if there is a related failure there
21:50:42 aram1s thank you
21:51:29 aram1s (i did not see anything else in nova-api)
21:59:13 openstackgerrit Merged openstack/nova master: [ironic] Don't remove instance info twice in destroy https://review.opendev.org/659809
22:00:27 openstackgerrit Eric Fried proposed openstack/os-traits master: Create traits for NUMA & device subtree affinity https://review.opendev.org/657898
22:10:39 mriedem stephenfin: i'm not sure why you don't just remove the exception in https://review.opendev.org/#/c/651307/ since nothing else is using it, and i don't see any change later in the series that actually removes the exception, but hopefully you'll remember to do that
22:23:02 mriedem jaypipes: DROP COLUMN on a big ass instances table could take "awhile" yeah? https://review.opendev.org/#/c/651309/9/nova/db/sqlalchemy/models.py@343
22:26:22 mriedem zzzeek: ^
23:06:52 openstackgerrit Eric Fried proposed openstack/nova master: Bump openstackdocstheme to 1.30.0 https://review.opendev.org/662353
23:07:50 openstackgerrit Eric Fried proposed openstack/nova-specs master: Bump openstackdocstheme to 1.30.0 https://review.opendev.org/662354
23:25:10 openstackgerrit Merged openstack/nova master: Remove unused param from _fill_provider_mapping https://review.opendev.org/655107
23:42:16 openstackgerrit Merged openstack/nova master: Remove zeromq from getting started with compute docs https://review.opendev.org/581926
#openstack-nova - 2019-05-31
00:25:56 zzzeek mriedem_away: yep still seems like DROP is slow for millions of rows
00:56:15 openstackgerrit Merged openstack/nova master: Keep attach_mode as top-level field in _translate_attachment_ref https://review.opendev.org/574413
01:19:23 openstackgerrit Yongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology" https://review.opendev.org/658716
01:54:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (13) https://review.opendev.org/576020
01:54:39 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (14) https://review.opendev.org/576027
01:54:50 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15) https://review.opendev.org/576031
01:55:58 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16) https://review.opendev.org/576299
01:56:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17) https://review.opendev.org/576344
01:56:31 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18) https://review.opendev.org/576673
01:56:54 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19) https://review.opendev.org/576676
01:57:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20) https://review.opendev.org/576689
01:57:45 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21) https://review.opendev.org/576709
01:58:20 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22) https://review.opendev.org/576712
05:06:49 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix default values in update_cell command https://review.opendev.org/662383
05:48:02 openstackgerrit Guo Jingyu proposed openstack/nova master: Define and catch QemuGuestAgentNotRunning for set-password failure https://review.opendev.org/661466
07:42:10 openstackgerrit Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.opendev.org/621476
08:16:06 kashyap efried: Yeah, saw it; responded. Will address. (And yes, that "soft -1" is totally justified. :-))
08:40:46 openstackgerrit zhaixiaojun proposed openstack/nova master: Exclude broken ironicclient versions 2.7.1 https://review.opendev.org/662404
08:43:35 openstackgerrit Boxiang Zhu proposed openstack/nova master: Add valid compute nodes with host and/or node in api layer https://review.opendev.org/661237
08:43:36 openstackgerrit Boxiang Zhu proposed openstack/nova master: Add compute_nodes_uuid field to Destination object https://review.opendev.org/661188
08:43:36 openstackgerrit Boxiang Zhu proposed openstack/nova master: Add host and hypervisor_hostname flag to create server https://review.opendev.org/645520
08:48:49 openstackgerrit Dakshina Ilangovan proposed openstack/nova-specs master: Spec: Provider config YAML file https://review.opendev.org/612497
09:00:20 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'InstanceUnknownCell' exception https://review.opendev.org/662411
09:21:47 openstackgerrit Balazs Gibizer proposed openstack/os-traits master: Add COMPUTE_NET_VF_PARENT_NAME_REPORTING trait https://review.opendev.org/658852
11:02:10 openstackgerrit John Garbutt proposed openstack/nova-specs master: Add Unified Limits Spec https://review.opendev.org/602201
11:26:32 openstackgerrit Merged openstack/nova-specs master: Cap sphinx for py2 to match global requirements https://review.opendev.org/659205
11:26:33 openstackgerrit Merged openstack/nova-specs master: Bump openstackdocstheme to 1.30.0 https://review.opendev.org/662354
11:44:00 openstackgerrit Balazs Gibizer proposed openstack/nova master: Simplfy test setup for TestNovaMigrations* tests https://review.opendev.org/662434
12:05:45 openstackgerrit Merged openstack/nova master: Remove fake_libvirt_utils from snapshot tests. https://review.opendev.org/642558
12:29:01 kashyap stephenfin: or anyone: Can you put this through its misery? Already has +2 from Eric: https://review.opendev.org/#/c/639091/
12:29:10 kashyap ("libvirt: Rework 'EBUSY' (SIGKILL) error handling code path")
12:33:24 kashyap mdbooth: When you can, can I have your your +1 back, too?
12:34:01 mdbooth kashyap: Well I'm not sure I trust you. I need to check you haven't tried to slip anything by me.
12:34:05 aspiers ditto for https://review.opendev.org/#/c/661782/ :)
12:34:12 aspiers haha
12:34:20 kashyap mdbooth: Haha, sure. Distrust away
12:34:26 aspiers mdbooth: He's a slippery one, that kashyap ;-)
12:34:32 mdbooth aspiers: I know!
12:34:41 mdbooth kashyap: You've rephrased your comment!
12:34:43 mdbooth -1
12:34:59 kashyap mdbooth: Hehe, that was after an elaborate discussion with aspiers
12:35:00 aspiers I mean, he once tried to slip some Perl 4 into nova
12:35:03 kashyap For better clarity :-)
12:35:21 mdbooth kashyap: Yeah, that reads a bit better.
12:35:42 kashyap mdbooth: Thanks. *Clear* phrasing is bloody hard work, it never ceases to amaze me.
12:36:10 kashyap aspiers: Hah, now, tell the truth

Earlier   Later