| opendevreview | wangzhiguang proposed openstack/tempest master: create volume type encryption nonexistent type id https://review.opendev.org/c/openstack/tempest/+/805123 | 02:02 |
|---|---|---|
| opendevreview | wangzhiguang proposed openstack/tempest master: add deactivate and reactivate image negative test https://review.opendev.org/c/openstack/tempest/+/741866 | 02:09 |
| *** abhishekk is now known as akekane|home | 06:00 | |
| *** akekane|home is now known as abhishekk | 06:00 | |
| *** rpittau|afk is now known as rpittau | 07:07 | |
| *** jpena|off is now known as jpena | 07:38 | |
| gibi | hi! it is just me or tempest-tox-plugin-sanity-check frequently fails with "Failed Plugins: x/vmware-nsx-tempest-plugin" | 10:30 |
| opendevreview | Balazs Gibizer proposed openstack/tempest master: Refactor existing QoS min bw tests https://review.opendev.org/c/openstack/tempest/+/806255 | 10:38 |
| opendevreview | Balazs Gibizer proposed openstack/tempest master: Add QoS minimum packet rate rule client https://review.opendev.org/c/openstack/tempest/+/806256 | 10:38 |
| opendevreview | Balazs Gibizer proposed openstack/tempest master: Test Network QoS minimum packet rate scenarios https://review.opendev.org/c/openstack/tempest/+/806257 | 10:39 |
| slaweq | frickler: yoctozepto: hi, if You would have a minute, please check https://review.opendev.org/c/openstack/devstack/+/806267 - it is needed for Manila team | 11:20 |
| slaweq | thx in advance | 11:20 |
| *** jpena is now known as jpena|lunch | 11:34 | |
| *** dviroel|out is now known as dviroel|ruck | 12:17 | |
| *** jpena|lunch is now known as jpena | 12:31 | |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Check request spec after resize https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/806239 | 13:14 |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Address subset of vgpu nits https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/806405 | 13:39 |
| *** rpittau is now known as rpittau|afk | 13:42 | |
| ykarel | gmann, https://review.opendev.org/c/openstack/devstack/+/803071 and https://review.opendev.org/c/openstack/devstack/+/806090 when you get chance | 14:04 |
| gmann | gibi: ack, we usually add failing plugins in blacklist until they are fixed. I will add this one too | 14:07 |
| gmann | ykarel: +2 on 2nd one and +1 on 1st as being co-author. | 14:09 |
| gmann | I would like to have other core to review it. | 14:09 |
| gmann | frickler: ianw please check this https://review.opendev.org/c/openstack/devstack/+/803071 | 14:10 |
| ykarel | gmann, Thanks | 14:11 |
| gibi | gmann: ack,thanks | 14:13 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Blacklist failing x/vmware-nsx-tempest-plugin https://review.opendev.org/c/openstack/tempest/+/806411 | 14:18 |
| artom | jparker_, I think you missed the bottom patch here: https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/805295/6 | 14:37 |
| artom | Also - are you confident enough to update your +1 to a +2 on any of them, and we can get them through the gate? | 14:37 |
| jparker_ | artom: I did yes, let me take a look at ps6 and I can start updating +2 as well | 14:40 |
| *** ykarel is now known as ykarel|away | 14:43 | |
| opendevreview | Ade Lee proposed openstack/tempest master: WIP - replace paramiko with libssh https://review.opendev.org/c/openstack/tempest/+/806274 | 14:48 |
| *** dviroel|ruck is now known as dviroel|ruck|lunch | 14:57 | |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Check request spec after resize https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/806239 | 14:58 |
| gmann | kopecmartin: if youa re around, to unblock tempest gate https://review.opendev.org/c/openstack/tempest/+/806411 | 15:18 |
| kopecmartin | gmann: thanks, i don't know why it started failing again :/ | 15:20 |
| gmann | i have not checked it but seems it is less maintained | 15:20 |
| *** jpena is now known as jpena|off | 15:39 | |
| *** dviroel|ruck|lunch is now known as dviroel|ruck | 16:06 | |
| opendevreview | Ghanshyam proposed openstack/tempest master: Avoid running tempest-full-ipv6 for unrelated tools files https://review.opendev.org/c/openstack/tempest/+/806435 | 17:10 |
| gmann | kopecmartin: ^^ this will avoid running etmpest-full-ipv6 job to run on sanity job fixes. | 17:10 |
| gmann | that is holding the 806411 fix taking so much time to merge | 17:11 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Avoid running tempest-ipv6-only for unrelated tools files https://review.opendev.org/c/openstack/tempest/+/806435 | 17:26 |
| opendevreview | Merged openstack/whitebox-tempest-plugin master: Improve base compute and utils helper organization https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/805295 | 17:42 |
| opendevreview | Merged openstack/whitebox-tempest-plugin master: Simplify test_multiqueue https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/805314 | 17:42 |
| opendevreview | Merged openstack/whitebox-tempest-plugin master: Simplify test_live_migration https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/805315 | 17:43 |
| opendevreview | Merged openstack/tempest master: Blacklist failing x/vmware-nsx-tempest-plugin https://review.opendev.org/c/openstack/tempest/+/806411 | 19:34 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Start transition to declarative CPU pinning configs https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/805455 | 20:00 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Stop silently overriding clients with admin clients https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/804780 | 20:00 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Remove create_test_server() helper override https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/805300 | 20:00 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Simplify test_pointer_device_type https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/805296 | 22:38 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Simplify test_rx_tx_queue_size https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/805297 | 22:38 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Remove reboot_server() helper https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/805298 | 22:38 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Remove resize_server() helper https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/805299 | 22:38 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Use new config_all_computes() helper in file backed memory tests https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/805921 | 22:38 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Clean up .zuul.yaml https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/805453 | 22:38 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Create NUMA helper mixin class https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/805454 | 22:38 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Start transition to declarative CPU pinning configs https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/805455 | 22:38 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Stop silently overriding clients with admin clients https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/804780 | 22:38 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Remove create_test_server() helper override https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/805300 | 22:38 |
| artom | What, nooo | 22:38 |
| artom | It was supposed to keep the other ones unchanged :( | 22:38 |
| clarkb | did you rebase or cherry pick? | 22:52 |
| clarkb | cherry picking creates a new commit which means a new timestamp which means a new sha | 22:52 |
| clarkb | and rebasing can change parents which will do similar but due to graph differences | 22:53 |
| artom | I think I just forgot the -R | 23:09 |
| artom | So `git review` rebased on top of master HEAD | 23:10 |
| clarkb | artom: are you running a very old git-review version? | 23:10 |
| artom | git-review version 1.28.0 | 23:10 |
| artom | Don't think it's that old, it's from Fedora 33 | 23:11 |
| clarkb | a more modern (at least a few yeras old git review) does a rebase to test if it can merge. If it fails it aborts and tells you you need to resolve conflicts. If it succeeds it undoes the reabse | 23:11 |
| clarkb | unless you've configured it to rebase always you shouldn't run into that problem | 23:11 |
| clarkb | the -R should only be necessary now if you're operating on a dirty tree and don't want to commit or stash | 23:12 |
| artom | Weird. Anyways, it's not a big deal, "just" wasted a bit of gate resources | 23:13 |
| clarkb | fridays are a good day for that | 23:38 |
| *** dviroel|ruck is now known as dviroel|out | 23:44 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!