| opendevreview | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/837443 | 02:21 |
|---|---|---|
| *** Bartoszer1 is now known as Bartoszer | 06:47 | |
| opendevreview | ZhangFan proposed openstack/cinder master: Add REIMAGE_VOLUME https://review.opendev.org/c/openstack/cinder/+/837502 | 08:50 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Drop support for MySQL 5.5 https://review.opendev.org/c/openstack/cinder/+/830087 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Indicate functions that should not be converted yet https://review.opendev.org/c/openstack/cinder/+/830088 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "service" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/830089 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "cluster" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/830091 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "quota" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/830092 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "quota class" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/830093 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "quota usage", "quota reservation" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/830094 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "volume", "volume attachment" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/830095 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "volume metadata" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/830096 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "snapshot" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/830097 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "snapshot metadata" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/830098 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Remove unnecessary sqlalchemy abstraction https://review.opendev.org/c/openstack/cinder/+/813229 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "volume type", "group type" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/837522 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "volume type specs", "group type specs" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/837523 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "qos specs" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/837524 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "volume type encryption" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/837525 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "volume glance metadata" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/837526 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "backup", "backup metadata" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/837527 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "transfer" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/837528 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "consistency group" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/837529 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "group" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/837530 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "cg snapshot" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/837531 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "group snapshot" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/837532 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "message" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/837533 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "driver initiator data" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/837534 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "image volume cache" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/837535 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate "worker" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/837536 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate online upgrade helpers to enginefacade https://review.opendev.org/c/openstack/cinder/+/837537 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Migrate 'purge_deleted_rows' to enginefacade https://review.opendev.org/c/openstack/cinder/+/837538 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Final cleanup for context-based enginefacade https://review.opendev.org/c/openstack/cinder/+/837539 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Remove final users of 'get_session' https://review.opendev.org/c/openstack/cinder/+/837540 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Remove unused API https://review.opendev.org/c/openstack/cinder/+/837541 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Fix up some API signatures, other style issues https://review.opendev.org/c/openstack/cinder/+/837542 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Move helpers to cinder.db.sqlalchemy.api https://review.opendev.org/c/openstack/cinder/+/837543 | 10:11 |
| stephenfin | Jaysus | 10:12 |
| tosky | is this the record for the longest pile of patches in openstack? | 10:13 |
| stephenfin | I mean, it still better than munging them altogether in a GitHub Pull Request, amirite 😂 | 10:14 |
| stephenfin | Hold on while I submit my "fixme" commits * ∞ | 10:14 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: WIP: db: Remove resolved TODO https://review.opendev.org/c/openstack/cinder/+/837546 | 10:26 |
| xuan | Hello every one, I got an exception about iscsi initiator: iscsi logout error, reports that: session not found, but it exists in iscsiadm -m session list. Does anyone know about it? | 11:03 |
| hemna | yikes | 12:12 |
| shubjero | rosmaita: Canadian, so I don't think so. This is also just a small lab cluster that is only used to test openstack upgrade before i do them in prod | 12:28 |
| rosmaita | shubjero: thanks, i will keep thinking ... how long was your wallaby running before you tried the upgrade to xena? | 12:48 |
| enriquetaso | Hey, if you're interested in Enable volume revert to snapshot NFS tests please review : https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/812716 | 12:51 |
| shubjero | rosmaita: btw I talked with some other openstack operators and this is their suggestion (it worked) ALTER TABLE volumes ROW_FORMAT=DYNAMIC; | 13:11 |
| shubjero | rosmaita: Wallaby was only running for a couple days while I performed some operator type smoke tests (create instance, create network, create volume, etc).. then upgraded to Xena | 13:11 |
| rosmaita | shubjero: got it ... i was asking because i think the upgrade script tries to add a 'use_quota' boolean column to the wallaby DB, and the error message you found in the mariadb error log said a different column ('provider_auth') was causing the problem | 13:14 |
| rosmaita | i was naively thinking that innodb would just slap the new column on the end of each row, but maybe it has to shuffle columns around and keep the fixed length ones together or something | 13:16 |
| shubjero | rosmaita: ahh I see. Do you think I should submit a bug report with cinder ? | 13:17 |
| rosmaita | shubjero: yes, i think we should track this | 13:18 |
| rosmaita | what version of mariadb are you using? | 13:18 |
| rosmaita | looks like row_format=dynamic became the default in 10.2.2 | 13:18 |
| shubjero | rosmaita: OK I will submit a bug report. I am running mariadb 10.3 but this lab cluster (and my production) have been upgraded consistently since 2015 so although my deployment is maintained, it is very old | 13:19 |
| shubjero | I think we started with Juno or Kilo (I was hired when it was on libert) | 13:19 |
| shubjero | *liberty | 13:19 |
| rosmaita | shubjero: ok, that would explain it, i imagine the old tables from liberty days would have used the previous default of 'compact' | 13:21 |
| rosmaita | we need to recompute our projected row sizes, though, because we have a lot of VARCHAR(255) columns, which in some languages are going to be 3 x 255 + 2 bytes | 13:23 |
| rosmaita | shubjero: looks like you ran into this mariadb issue: https://mariadb.com/kb/en/innodb-row-formats-overview/#upgrading-causes-row-size-too-large-errors | 13:25 |
| shubjero | rosmaita: yeah, I guess one of the db migrations should have modified the volumes table ROW_FORMAT to dynamic right? | 13:25 |
| shubjero | rosmaita: yeah thats the link my operator friends sent me | 13:26 |
| rosmaita | shubjero: maybe ... it does explain why we never saw this in grenade, it uses a fairly new version of mariadb | 13:28 |
| whoami-rajat | enriquetaso, left a comment on the patch https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/812716 | 14:03 |
| enriquetaso | thanks! | 14:09 |
| ade_lee | tosky, eharney geguileo so it seems like geguileo patch fixed things in the cinder ci job -- https://review.opendev.org/c/openstack/cinder/+/790535 | 14:10 |
| ade_lee | tosky, whats broken now is the ceph job -- can you remind me what needed to be done to get that one working? | 14:11 |
| ade_lee | tosky, iirc - some config required and a depends-on .. | 14:11 |
| ade_lee | https://zuul.opendev.org/t/openstack/build/c992d1bd8ba64deab365e1a43196b1e7 | 14:11 |
| tosky | ade_lee: wasn't it the story about os-brick? uhm | 14:13 |
| geguileo | I haven't look at tempest jobs lately, but that ceph job hasn't actually run the tempest tests, right? | 14:15 |
| eharney | i think it's that we don't have the ceph job setup for el9 yet | 14:15 |
| tosky | ade_lee: oh, the cephadm story | 14:15 |
| *** dviroel is now known as dviroel|mtg | 14:15 | |
| geguileo | [ERROR] /opt/stack/devstack-plugin-ceph/devstack/lib/ceph:279 If you wish to install Ceph on this distribution anyway run with FORCE_CEPH_INSTALL=yes, this assumes that YOU will setup the proper repositories | 14:16 |
| geguileo | yup | 14:16 |
| tosky | you need to depend-on: https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/834223 | 14:17 |
| tosky | geguileo: ^^ | 14:17 |
| tosky | ade_lee: ^^ | 14:17 |
| geguileo | tosky: thanks :-) | 14:18 |
| tosky | but it's still WIP and a few tests may fail, as you can see devstack-plugin-ceph-tempest-py3 is in FAILURE state | 14:18 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: PEP8: Don't force name in TODO https://review.opendev.org/c/openstack/os-brick/+/836058 | 14:47 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: Support extending encrypted volumes https://review.opendev.org/c/openstack/os-brick/+/836059 | 14:47 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: NVMe-oF: Consolidate code paths https://review.opendev.org/c/openstack/os-brick/+/836060 | 14:47 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: NVMe-oF: Disconnect subsystems https://review.opendev.org/c/openstack/os-brick/+/836062 | 14:47 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: Support shared_targets tristate value https://review.opendev.org/c/openstack/os-brick/+/836063 | 14:47 |
| ade_lee | tosky, geguileo thanks -- I'll give it a shot and see what happens. maybe we'll end up putting it in experiemntal or non-voting | 15:01 |
| ade_lee | till it all works | 15:02 |
| *** dviroel|mtg is now known as dviroel|lunch | 16:02 | |
| opendevreview | Luigi Toscano proposed openstack/devstack-plugin-ceph master: [DNM][CI] Add CEPHADM_DEPLOY flag to py3 tests https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/834223 | 16:17 |
| *** dviroel|lunch is now known as dviroel | 16:58 | |
| opendevreview | Merged openstack/devstack-plugin-nfs master: Enable volume revert to snapshot NFS tests https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/812716 | 17:58 |
| opendevreview | Luigi Toscano proposed openstack/devstack-plugin-ceph master: [DNM][CI] Add CEPHADM_DEPLOY flag to py3 tests https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/834223 | 18:41 |
| opendevreview | Luigi Toscano proposed openstack/devstack-plugin-ceph master: [DNM][CI] Add CEPHADM_DEPLOY flag to py3 tests https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/834223 | 19:40 |
| hemna | man openstacksdk's calls to the cinder api return NOTHING. all of them | 20:07 |
| hemna | everything is success all the time! | 20:07 |
| hemna | https://bugs.launchpad.net/ubuntu/+source/python-openstacksdk/+bug/1968783 | 20:14 |
| hemna | pewp | 20:14 |
| hemna | no return codes from cinder calls | 20:15 |
| opendevreview | Francesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 20:34 |
| *** dviroel is now known as dviroel|out | 20:55 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!