Friday, 2021-08-20

opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Improve base compute and utils helper organization  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80529500:19
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Simplify test_multiqueue  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80531400:19
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Simplify test_live_migration  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80531500:19
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Simplify test_pointer_device_type  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80529600:19
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Simplify test_rx_tx_queue_size  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80529700:19
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Remove reboot_server() helper  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80529800:19
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Remove resize_server() helper  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80529900:19
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Stop silently overriding clients with admin clients  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80478000:19
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Remove create_test_server() helper override  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80530000:19
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Improve base compute and utils helper organization  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80529501:46
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Simplify test_multiqueue  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80531401:46
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Simplify test_live_migration  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80531501:46
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Simplify test_pointer_device_type  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80529601:46
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Simplify test_rx_tx_queue_size  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80529701:46
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Remove reboot_server() helper  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80529801:46
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Remove resize_server() helper  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80529901:46
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Stop silently overriding clients with admin clients  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80478001:46
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Remove create_test_server() helper override  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80530001:46
*** pojadhav- is now known as pojadhav04:09
opendevreviewOpenStack Proposal Bot proposed openstack/devstack master: Updated from generate-devstack-plugins-list  https://review.opendev.org/c/openstack/devstack/+/80533606:13
*** rpittau|afk is now known as rpittau07:38
*** jpena|off is now known as jpena07:38
rpittauhey good morning! We just noticed an interesting issue when resolving dependencies installing cinver with pip: https://zuul.opendev.org/t/openstack/build/1ec156a09a9e43c2bafe0bc7a05212f408:02
rpittauseems like the dependency resolver in pip doesn't really work very well ?08:02
rpittauis it a known issue maybe ?08:02
iurygregoryyay!08:05
opendevreviewMerged openstack/devstack master: Updated from generate-devstack-plugins-list  https://review.opendev.org/c/openstack/devstack/+/80533608:10
ykarelseems the issue is in pypi infra mirrors, as installation is fine when not using mirrors08:37
yoctozeptorpittau, iurygregory: yeah, there are some random hiccups from time to time; they should be rare; I have no idea why that happens; the error is obviously misleading; I would suspect pip failing on weird answer from pypi mirror or some internal race condition; as it is rare, I did not bother escalating to pip team - please help if you think it gets08:37
yoctozeptooverly annoying ;-)08:37
yoctozeptoykarel: yeah, it usually "just works" after a recheck08:37
ykarelyoctozepto, it not fixed atleast yet08:38
rpittauyoctozepto: it's annoying because it's friday :D08:38
ykarelyoctozepto, small reproducer: pip install --index-url=https://mirror.mtl01.inap.opendev.org/pypi/simple -c https://raw.githubusercontent.com/openstack/requirements/master/upper-constraints.txt boto308:38
ykarelif not using infra mirrors it installs fine08:38
rpittaujokes aside, I'm glad it looks like not pip related08:39
yoctozeptoykarel: ah, so you managed to reproduce locally08:41
yoctozeptoso it is pypi mirrors precisely08:41
yoctozeptothe only issue is pip is not telling the truth08:41
yoctozeptocould you raise this at #opendev ?08:42
ykarelyes sure08:43
iurygregorytks yoctozepto ykarel :D08:44
*** dviroel|ruck|out is now known as dviroel|ruck11:26
*** bhagyashris_ is now known as bhagyashris11:31
*** jpena is now known as jpena|lunch11:36
artomrpittau, huh, I saw that last night on https://review.opendev.org/c/openstack/tempest/+/805264 - not just me then12:25
rpittauartom: if it's due to mirrors, it can hit anyone :)12:34
artomrpittau, yeah, saw the conversation #opendev afterwards12:34
artomI guess it went away on its own, /me rechecks patches12:34
*** jpena|lunch is now known as jpena12:36
*** rpittau is now known as rpittau|afk13:55
clarkbit is important to clarify a few things about those mirrors. They don't actually mirror pypi, they proxy cache it. They do mirror distro packages though.14:57
clarkbWhat this means is that we're just showing you what pypi's CDN is showing us14:57
clarkbindexes have a 10 minute ttl and packages have a 24 hour ttl iirc14:57
clarkbWhen pypi's CDN has an error retrieving from the primary backend it falls back to a secondary backend that we've seen isn't always up to date14:58
clarkblong story short the issue is likely in pypi's CDN layer14:58
*** jpena is now known as jpena|off15:51
*** pojadhav- is now known as pojadhav16:00
opendevreviewyatin proposed openstack/devstack master: [DNM] Test c8-stream  https://review.opendev.org/c/openstack/devstack/+/80314417:13
lbragstadgmann you had some patches to tempest that attempted to run various server operations with system-scoped tokens, yeah?19:07
gmannlbragstad: hypervisor  test actually not server19:08
gmannlbragstad: https://review.opendev.org/c/openstack/tempest/+/74012219:09
lbragstadgmann aha - ok 19:15
lbragstadgmann i'm looking for an appropriate place to put a test that uses a system-scoped token to create a server so that i can test the KSM and KSA project ID passthrough bits19:16
lbragstadi was thinking about putting it in tempest/api/compute/servers/test_servers.py but maybe that's not the right place?19:18
gmannlbragstad: as it is admin test you can add in tempest/api/compute/admin/test_servers.py19:22
lbragstadok - that makes sense19:22
gmannlbragstad: but for that we need to open nova create server policy for 'system' scope also https://github.com/openstack/nova/blob/6edf06dbfd902f7af413d2d7a93760d05efab4fa/nova/policies/servers.py#L17619:23
lbragstadgmann ack19:24
gmanni have not looked it into your KSM patch but may be it is needed to pass the very first policy check in POST server API19:24
lbragstadi fully expect a mess of depends on patches between tempest, nova, ksm, and ksa 19:24
gmannhumm. I am hoping it will be devstack too ?19:25
lbragstadthe main thing i'm trying to do write now is get a single test up that would exercise the project ID pass-through functionality with KSM and KSA19:25
gmannk19:25
lbragstadfor example, using a system-scoped token that would create a server would exercise that KSM is allowing the pass-through and KSA is implementing the pass-through to neutron (for example) 19:26
gmannlbragstad: and add project_id in context for system scope token too?19:27
lbragstadyeah - so that's what KSM is responsible for 19:27
gmannany required project_id right not dummy only?19:27
lbragstadthat depends if we want to use this feature to cleanup orphaned resources19:28
gmannbecause in nova case, it has to be a valid project id for which system scope will create server request for19:29
lbragstadoh - sure19:29
lbragstadi was thinking for the delete case19:29
gmannok, got it. +119:30
opendevreviewLance Bragstad proposed openstack/tempest master: Add test coverages for system-admins creating instances  https://review.opendev.org/c/openstack/tempest/+/80545219:55
opendevreviewLance Bragstad proposed openstack/tempest master: Add test coverage for system-admins creating instances  https://review.opendev.org/c/openstack/tempest/+/80545219:56
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Stop silently overriding clients with admin clients  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80478020:20
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Remove create_test_server() helper override  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80530020:20
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Clean up .zuul.yaml  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80545320:20
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Create NUMA helper mixin class  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80545420:20
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Start transition to declarative CPU pinning configs  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80545520:20
artomjparker, ^^ there's a first attemp20:21
artomt20:21
jparkerartom: ack thanks20:21
artomI'm sure CI will explode20:21
opendevreviewLance Bragstad proposed openstack/tempest master: Add test coverage for system-admins creating instances  https://review.opendev.org/c/openstack/tempest/+/80545220:33
opendevreviewLance Bragstad proposed openstack/tempest master: Add test coverage for system-admins creating instances  https://review.opendev.org/c/openstack/tempest/+/80545220:35
lbragstadgmann ok - with ^ i was able to confirm that server create fails for system-admins when enforce_scope=True because it hangs getting the network from neutron20:35
lbragstadusing devstack with enforce_scope=True and the dependent changes in the commit message to keystonemiddleware and nova20:36
*** dviroel|ruck is now known as dviroel|out22:04
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Clean up .zuul.yaml  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80545322:30
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Create NUMA helper mixin class  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80545422:30
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Start transition to declarative CPU pinning configs  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80545522:30
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Stop silently overriding clients with admin clients  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80478022:30
opendevreviewArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: Remove create_test_server() helper override  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/80530022:30
opendevreviewGhanshyam proposed openstack/devstack stable/wallaby: Fix Usage of rdo-release rpm  https://review.opendev.org/c/openstack/devstack/+/80307123:05
gmannlbragstad: ack23:06

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!