Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-12-11
22:42:53 openstackgerrit Merged openstack/nova master: nova-evacuate: Remove leftover playbook from standalone job https://review.opendev.org/c/openstack/nova/+/764618
23:51:48 openstackgerrit Merged openstack/nova master: nova-grenade-multinode: Skip test_live_block_migration_paused https://review.opendev.org/c/openstack/nova/+/766720
#openstack-nova - 2020-12-12
00:33:27 openstackgerrit melanie witt proposed openstack/nova stable/ussuri: Omit resource inventories from placement update if zero https://review.opendev.org/c/openstack/nova/+/766825
04:18:57 brinzhang_ gibi:ack
05:27:37 openstackgerrit Wenping Song proposed openstack/nova master: Remove redundant code in instance delete https://review.opendev.org/c/openstack/nova/+/759577
06:05:48 openstackgerrit Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from Rebuild Server API https://review.opendev.org/c/openstack/nova/+/766380
08:52:10 gibi if there are stable core around today then https://review.opendev.org/c/openstack/nova/+/765749 needs some low
08:52:13 gibi love
10:27:37 zer0def hey there, would there be any reason for a non-monkey-patched eventlet-based nova-conductor to receive RPC messages sent from nova-metadata, but not act on them? the database backend is on psycopg2 and I've disabled monkey patching, because it raised "greenlet.error: cannot switch to a different thread"
10:32:16 zer0def oh yeah, and the reason I'm running eventlet-based nova components is because they're spawned from openstack-helm without certificates
10:33:51 zer0def version is victoria
22:29:25 openstackgerrit Ghanshyam proposed openstack/placement master: [goal] Deprecate the JSON formatted policy file https://review.opendev.org/c/openstack/placement/+/764394
22:35:58 openstackgerrit Ghanshyam proposed openstack/nova master: Reuse code from oslo lib for JSON policy migration https://review.opendev.org/c/openstack/nova/+/763487
#openstack-nova - 2020-12-13
03:09:32 openstackgerrit Merged openstack/nova stable/victoria: lower-constraints: Bump packaging to 20.4 https://review.opendev.org/c/openstack/nova/+/765749
04:18:35 prometheanfire stephenfin: mind looking at https://review.opendev.org/c/openstack/nova/+/714676 for the mock fixes?
08:02:46 openstackgerrit Mamduh proposed openstack/os-vif stable/ussuri: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/c/openstack/os-vif/+/765419
08:07:42 openstackgerrit Mamduh proposed openstack/os-vif stable/ussuri: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/c/openstack/os-vif/+/765419
10:52:43 openstackgerrit Mamduh proposed openstack/os-vif stable/train: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/c/openstack/os-vif/+/765912
10:57:08 openstackgerrit Mamduh proposed openstack/os-vif stable/stein: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/c/openstack/os-vif/+/765914
11:25:19 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.binary_type/integer_types/string_types https://review.opendev.org/c/openstack/nova/+/728094
11:26:07 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.text_type (1/2) https://review.opendev.org/c/openstack/nova/+/728109
11:27:11 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.text_type (2/2) https://review.opendev.org/c/openstack/nova/+/728117
11:28:11 openstackgerrit Takashi Natsume proposed openstack/nova master: Add a description in the PTL guide https://review.opendev.org/c/openstack/nova/+/765682
11:28:36 openstackgerrit Takashi Natsume proposed openstack/nova master: Fix a deprecation warning of 'collections' https://review.opendev.org/c/openstack/nova/+/765646
18:34:59 openstackgerrit Merged openstack/nova master: Fix a deprecation warning of 'collections' https://review.opendev.org/c/openstack/nova/+/765646
19:09:06 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: Add 'socket' to NUMACell and PciDevice https://review.opendev.org/c/openstack/nova/+/766795
19:09:06 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: Start populating `socket_id` in host NUMA topology https://review.opendev.org/c/openstack/nova/+/766816
19:09:07 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: db: Add `socket` to pci_devices https://review.opendev.org/c/openstack/nova/+/766862
19:11:01 openstackgerrit Merged openstack/nova master: Add a description in the PTL guide https://review.opendev.org/c/openstack/nova/+/765682
19:38:34 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: db: Add `socket` to pci_devices https://review.opendev.org/c/openstack/nova/+/766862
19:38:35 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: objects: Add `socket` to NUMACell and PciDevice https://review.opendev.org/c/openstack/nova/+/766795
19:38:35 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: Start populating `socket_id` in host NUMA topology https://review.opendev.org/c/openstack/nova/+/766816
#openstack-nova - 2020-12-14
00:19:34 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: db: Add `socket` to pci_devices https://review.opendev.org/c/openstack/nova/+/766862
00:19:34 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: objects: Add `socket` to NUMACell and PciDevice https://review.opendev.org/c/openstack/nova/+/766795
00:19:35 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: Start populating `socket_id` in host NUMA topology https://review.opendev.org/c/openstack/nova/+/766816
02:52:28 openstackgerrit Merged openstack/nova-specs master: Smart-nic Support Specification https://review.opendev.org/c/openstack/nova-specs/+/742785
05:02:36 openstackgerrit Hemanth N proposed openstack/nova stable/ussuri: functional: Clean up PCI tests https://review.opendev.org/c/openstack/nova/+/766879
08:01:55 gibi good morning
08:41:25 openstackgerrit Balazs Gibizer proposed openstack/nova master: Reproduce bug 1907522 in functional test https://review.opendev.org/c/openstack/nova/+/766791
08:41:25 openstack bug 1907522 in OpenStack Compute (nova) "test_*_with_qos_min_bw_allocation fails in the nova-multi-cell job with: nova.exception.MigrationPreCheckError: Migration pre-check error: Failed to create port bindings for host " [High,Triaged] https://launchpad.net/bugs/1907522 - Assigned to Balazs Gibizer (balazs-gibizer)
08:42:48 openstackgerrit Merged openstack/nova stable/victoria: Reproduce bug 1896463 in func env https://review.opendev.org/c/openstack/nova/+/763152
08:42:48 openstack bug 1896463 in OpenStack Compute (nova) victoria "evacuation failed: Port update failed : Unable to correlate PCI slot " [Low,In progress] https://launchpad.net/bugs/1896463 - Assigned to Balazs Gibizer (balazs-gibizer)
09:56:12 lyarwood gibi: morning, just sent a reply to fungi on the ML without CC'ing you in reply. He raised a number of valid points about our tox envs and requirements files, wanted to ask if you wanted them tracked in LP or somewhere else
10:05:43 gibi lyarwood: hi! ack, replying
10:13:50 openstackgerrit Lee Yarwood proposed openstack/nova stable/ussuri: [stable-only] lower-contraints: Bump various packages https://review.opendev.org/c/openstack/nova/+/766738
10:19:17 gibi lyarwood: I'm not considering these as serious issues, more like improvements, but bugs are OK to track them
10:25:33 lyarwood right that's why I asked
10:25:36 lyarwood cool
10:31:52 lyarwood gibi: https://etherpad.opendev.org/p/nova-stable-pip-fallout btw I'm trying to capture things here re the broken stable branches etc
10:31:55 lyarwood elod: ^ also
10:32:32 gibi thanks lyarwood
10:37:55 openstackgerrit Lee Yarwood proposed openstack/nova stable/train: [stable-only] Cap bandit to 1.6.2 and raise hacking and flake8 in LC https://review.opendev.org/c/openstack/nova/+/766171
10:39:40 lyarwood hmm does Depends-On work across branches with grenade?
10:40:35 openstackgerrit Lee Yarwood proposed openstack/nova stable/train: [stable-only] Cap bandit to 1.6.2 and raise hacking and flake8 in LC https://review.opendev.org/c/openstack/nova/+/766171
10:41:18 openstackgerrit Balazs Gibizer proposed openstack/nova master: Reproduce bug 1907522 in functional test https://review.opendev.org/c/openstack/nova/+/766791
10:41:18 openstack bug 1907522 in OpenStack Compute (nova) "test_*_with_qos_min_bw_allocation fails in the nova-multi-cell job with: nova.exception.MigrationPreCheckError: Migration pre-check error: Failed to create port bindings for host " [High,Triaged] https://launchpad.net/bugs/1907522 - Assigned to Balazs Gibizer (balazs-gibizer)
10:42:50 openstackgerrit Balazs Gibizer proposed openstack/nova master: Fallback to same-cell resize with qos ports https://review.opendev.org/c/openstack/nova/+/766925
10:44:15 openstackgerrit Balazs Gibizer proposed openstack/nova master: DNM: test how multi-cell job behaves now https://review.opendev.org/c/openstack/nova/+/766926
11:09:54 elod lyarwood: fyi, bandit 1.6.3 is yanked, so hopefully that error is now history :) >>> https://pypi.org/project/bandit/#history
11:11:05 elod lyarwood: sorry, my bad. it seems 1.7.0 is still universal :'(
11:11:38 lyarwood /o\
11:13:24 elod quite frustrating :/
11:14:55 elod the new bug report: https://github.com/PyCQA/bandit/issues/665
11:17:03 lyarwood I'm going to close out my bandit changes either way as this should eventually get resolved there
11:17:29 lyarwood did we want to switch our LC jobs to non-voting btw while we wait for an agreement on dropping them out right or not?
11:18:07 elod lyarwood: maybe we could even remove them (at least from old branches)
11:18:52 lyarwood elod: personally I'd like to wait for distros to comment more, I wanted to chase up with the RDO folks for example
11:18:53 elod where it's clearly contradicts with constraints (i.e. LC fails)
11:19:19 lyarwood elod: switching to non-voting at least gives them a chance to respond and later fix these issues if they find it useful
11:19:25 elod lyarwood: hmmm, yes, you are right. Let's just set them non-voting
11:19:54 lyarwood kk
11:20:33 lyarwood oh it's a template
11:20:37 lyarwood my bad
11:20:44 lyarwood can we mark templates as non-voting?
11:21:17 elod we should be able to
11:22:18 elod worst case the job needs to be added in .zuul.yaml outside of the template
11:22:35 elod and then set non-voting
11:22:56 lyarwood yup true
11:26:35 openstackgerrit Lee Yarwood proposed openstack/nova master: zuul: Make lower-constraints jobs non-voting https://review.opendev.org/c/openstack/nova/+/766931
11:27:30 elod lyarwood: btw, if now the lower-constraints do work on master and victoria then maybe it's enough to set non-voting from ussuri. what do you think?
11:29:56 elod as the main problem is where the LC's are kinda irreparable
11:33:54 elod or time-consuming to fix and involves lots of LC changes (a real "butterfly effect" :P)
11:33:59 lyarwood elod: tbh I'd still like to mark them all as non-voting to signal that we don't think they are useful going forward
11:37:12 elod lyarwood: you are probably right. however it feels now almost useful - if the constraints are really kept with the new pip resolver. But I'm okay with setting it non-voting on master.
12:28:16 ftarasenko hi! Team, can someone help my colleague with review of tiny commit? https://review.opendev.org/c/openstack/nova/+/764927
12:38:52 sean-k-mooney ftarasenko: im not sure that makes sense
12:39:38 sean-k-mooney the constuction of the loop is such that we sleep for 1 seconed on every iteration
12:40:04 sean-k-mooney i understand the logic but just addint that while keeping the current looping seams odd
12:41:59 sean-k-mooney we shoudl proably drop the loop var and instead just do a while loop instead
12:43:19 sean-k-mooney ftarasenko: as written the current look will at most be off by the time it takes for the body to execute
12:43:52 sean-k-mooney so i agree it can be longer then you expect but not by a large interval
12:46:26 ftarasenko Thank you, your explanation seems good to me. Forwarded to my colleague. He says that he can rewrite it to while loop, if someone will review)
12:47:06 ftarasenko Is it ok to ask for review here? I'm new to nova project.
12:47:20 sean-k-mooney yes its ok
12:49:47 sean-k-mooney ftarasenko: we encorage people to ask question about developement here espcially if they are new
12:55:27 sean-k-mooney ftarasenko: by the way they should ingore the lower constraints failure.
12:55:45 sean-k-mooney ftarasenko: we have an ongoing issue with that and the new pip depency resolver
12:57:26 lyarwood gibi: https://zuul.opendev.org/t/openstack/build/02576a8cc541451bb8add7f1423d14bf - is that a known issue?
12:57:38 lyarwood openstack.exceptions.BadRequestException: BadRequestException: 400: Client Error for url: https://199.204.45.203/compute/v2.1/os-aggregates/openstack.tests.functional.cloud.test_aggregate.TestAggregate.test_aggregates-1, Invalid input received: id must be an integer'
12:57:59 lyarwood ah sorry I thought that was part of test_qos_minimum_bandwidth_rule_lifecycle

Earlier   Later