| opendevreview | DataCore Software proposed openstack/cinder master: Reintroduce DataCore driver https://review.opendev.org/c/openstack/cinder/+/836996 | 05:42 |
|---|---|---|
| *** bhagyashris is now known as bhagyashris|out | 06:26 | |
| opendevreview | Jan Hartkopf proposed openstack/cinder master: Ceph: add option to keep only last n snapshots per backup https://review.opendev.org/c/openstack/cinder/+/810457 | 08:48 |
| *** dviroel|out is now known as dviroel | 11:15 | |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [SVf]:Fix retype failure for replication volume-type https://review.opendev.org/c/openstack/cinder/+/836944 | 11:21 |
| opendevreview | Walt proposed openstack/cinder master: Fix tracking allocated_capacity_gb https://review.opendev.org/c/openstack/cinder/+/836083 | 12:12 |
| opendevreview | Alexander Malashenko proposed openstack/cinder master: Add check if size exist in model_update and update size in model_update https://review.opendev.org/c/openstack/cinder/+/836973 | 12:16 |
| opendevreview | Brian Rosmaita proposed openstack/os-brick master: Add encryption and decryption for files for Image Encryption https://review.opendev.org/c/openstack/os-brick/+/709432 | 12:20 |
| opendevreview | Mounika Sreeram proposed openstack/cinder master: [SVf] Resize of GMCV volumes in group https://review.opendev.org/c/openstack/cinder/+/830787 | 13:08 |
| *** hemna4 is now known as hemna | 14:22 | |
| opendevreview | Walt proposed openstack/cinder master: Fix tracking allocated_capacity_gb https://review.opendev.org/c/openstack/cinder/+/836083 | 14:26 |
| opendevreview | Eric Harney proposed openstack/cinder master: Bump mypy version to 0.942 https://review.opendev.org/c/openstack/cinder/+/835483 | 14:36 |
| hemna | mep | 14:37 |
| opendevreview | Eric Harney proposed openstack/os-brick master: Bump mypy version to 0.942 https://review.opendev.org/c/openstack/os-brick/+/837350 | 14:40 |
| opendevreview | Merged openstack/cinder master: Fix PowerFlex connector HTTPS certificate validation https://review.opendev.org/c/openstack/cinder/+/810415 | 14:57 |
| opendevreview | Merged openstack/cinder stable/ussuri: Reject bad img formats for uploaded encrypted vols https://review.opendev.org/c/openstack/cinder/+/828890 | 14:59 |
| *** dviroel is now known as dviroel|lunch|afk | 15:32 | |
| opendevreview | Stephen Finucane proposed openstack/cinder master: tests: Enable SQLAlchemy 2.0 deprecation warnings https://review.opendev.org/c/openstack/cinder/+/837163 | 15:33 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: models: Remove implicit coercion of SELECT to scalar subquery https://review.opendev.org/c/openstack/cinder/+/837164 | 15:33 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Pass case.whens as positionals, not a list https://review.opendev.org/c/openstack/cinder/+/837165 | 15:33 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Don't pass strings to Connection.execute https://review.opendev.org/c/openstack/cinder/+/837166 | 15:33 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Don't use strings to indicate column, relationship paths https://review.opendev.org/c/openstack/cinder/+/837167 | 15:33 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: Resolve invalid UUID warning https://review.opendev.org/c/openstack/cinder/+/837353 | 15:33 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Don't use legacy calling style of select() https://review.opendev.org/c/openstack/cinder/+/837354 | 15:33 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Don't use strings to indicate relationship names https://review.opendev.org/c/openstack/cinder/+/837355 | 15:33 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Remove use of 'as_scalar()' https://review.opendev.org/c/openstack/cinder/+/837356 | 15:33 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: privsep: Move luks utils to privsep https://review.opendev.org/c/openstack/os-brick/+/791274 | 15:37 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: privsep: Convert 'os_brick.caches.opencas' https://review.opendev.org/c/openstack/os-brick/+/791275 | 15:37 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: Remove unused pylintrc file https://review.opendev.org/c/openstack/os-brick/+/837357 | 15:37 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: WIP: privsep: Move linuxscsi initiator to privsep https://review.opendev.org/c/openstack/os-brick/+/837358 | 15:37 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: WIP: privsep: Add a privsep-compatible base connector https://review.opendev.org/c/openstack/os-brick/+/837359 | 15:37 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: WIP: privsep: Convert 'os_brick.initiator.connectors.rbd' https://review.opendev.org/c/openstack/os-brick/+/837360 | 15:37 |
| opendevreview | Ade Lee proposed openstack/cinder master: WIP/DNM: Add fips check jobs https://review.opendev.org/c/openstack/cinder/+/790535 | 15:38 |
| opendevreview | Brian Rosmaita proposed openstack/cinder stable/train: Reject bad img formats for uploaded encrypted vols https://review.opendev.org/c/openstack/cinder/+/837311 | 15:41 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: annotate image/glance.py https://review.opendev.org/c/openstack/cinder/+/766064 | 15:42 |
| opendevreview | Eric Harney proposed openstack/cinder master: Glance: remove _extract_attributes method https://review.opendev.org/c/openstack/cinder/+/837362 | 15:42 |
| opendevreview | Merged openstack/cinderlib master: Continue yoga development https://review.opendev.org/c/openstack/cinderlib/+/833781 | 15:47 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: encryptors: Remove 'CryptsetupEncryptor' https://review.opendev.org/c/openstack/os-brick/+/791272 | 15:50 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: encryptors: Drop support for out-of-tree encryptors https://review.opendev.org/c/openstack/os-brick/+/791273 | 15:50 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: encryptors: Drop support for out-of-tree encryptors https://review.opendev.org/c/openstack/os-brick/+/791273 | 15:55 |
| enriquetaso | woah | 15:59 |
| opendevreview | Ade Lee proposed openstack/cinder master: Add fips check jobs https://review.opendev.org/c/openstack/cinder/+/790535 | 16:06 |
| ade_lee | tosky, eharney geguileo ^^ relevant changes to fips job as discussed last week. Let me know if I missed anything. | 16:07 |
| tosky | ade_lee: the changes to the jobs, apart from the move of the definitions ot the other repositories, seems fine. Do we still need the changes to tools/test-setup.sh and bindep.txt now that the functional job is removed? | 16:15 |
| opendevreview | Mounika Sreeram proposed openstack/cinder master: [SVf] Resize of GMCV volumes in group https://review.opendev.org/c/openstack/cinder/+/830787 | 16:16 |
| ade_lee | tosky, I don't know tbh. I can try to remove them in the next iteration - although the changes that are there are still relevant to actually running the functional tests on centos | 16:17 |
| ade_lee | tosky, so they should get in somewhere - so as not to lose the hard earned knowledge on how to do this | 16:17 |
| ade_lee | tosky, maybe what makes sense is to separate those changes and add them as a separate patch with an experimental centos 9 functional job? | 16:19 |
| tosky | ade_lee: but the point iirc is that we don't gain any relevant information from adding FIPS-enabled functional jobs, if I remember the discussion | 16:22 |
| ade_lee | tosky, correct -- the changes there are not just for fips though, but to be able to run the functional jobs at all on centos | 16:23 |
| ade_lee | (fips or not) | 16:23 |
| tosky | ade_lee: oh, I see | 16:32 |
| tosky | maybe they should go into their own review which adds functional tests on py39 then | 16:33 |
| opendevreview | Merged openstack/cinder master: Remove privsep/hscli https://review.opendev.org/c/openstack/cinder/+/836859 | 16:37 |
| shubjero | Hey all, I've got a lab cluster running Wallaby that I'm trying to upgrade to Xena and when I go to upgrade the DB I get this error: https://paste.openstack.org/show/btiJGPQsP75RbUHPhRuL/ | 19:43 |
| shubjero | I did read that there was some specific notes about database migrations at https://docs.openstack.org/releasenotes/cinder/xena.html but I need some clarification | 19:44 |
| shubjero | I'm running mariadb v. 10.3.34 on ubuntu 20.04 if that matters | 19:48 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: privsep: Move luks utils to privsep https://review.opendev.org/c/openstack/os-brick/+/791274 | 20:17 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: privsep: Convert 'os_brick.caches.opencas' https://review.opendev.org/c/openstack/os-brick/+/791275 | 20:17 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: WIP: privsep: Move linuxscsi initiator to privsep https://review.opendev.org/c/openstack/os-brick/+/837358 | 20:17 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: WIP: privsep: Add a privsep-compatible base connector https://review.opendev.org/c/openstack/os-brick/+/837359 | 20:17 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: WIP: privsep: Convert 'os_brick.initiator.connectors.rbd' https://review.opendev.org/c/openstack/os-brick/+/837360 | 20:17 |
| rosmaita | shubjero: have not seen that error before ... the statement causing the error is in a loop where it's modifying 2 tables ... can you check your mariadb error log? Should be a message like 'InnoDB: Cannot add field use_quota in table X | 20:34 |
| rosmaita | would be good to know what table X is | 20:34 |
| rosmaita | shubjero: actually, forget that, it's the volumes table | 20:36 |
| shubjero | rosmaita: ok let me check my logs | 21:19 |
| shubjero | rosmaita: 2022-04-11 17:20:50 4737 [ERROR] InnoDB: Cannot add field `provider_auth` in table `cinder`.`volumes` because after adding it, the row size is 8628 which is greater than maximum allowed size (8126 bytes) for a record on index leaf page. | 21:21 |
| rosmaita | shubjero: ty, that is helpful | 21:27 |
| rosmaita | and by 'helpful' i mean it isn't what i expected to see and I am not sure what to make of it yet | 21:28 |
| shubjero | rosmaita: yeah thats fair, im looking at this error from a mariadb point of view and what others have done to resolve it | 21:32 |
| shubjero | rosmaita: I believe this limit of 8126 is derived from a configuration setting (default is 16k for the innodb_page_size variable) so I tried setting this to 32k but then mariadb wont start so I assume mariadb 10.3 only supports up to 16k | 21:33 |
| shubjero | rosmaita: I interpreted that from https://blog.gahooa.com/2019/10/10/error-innodb-cannot-add-field-in-table-because-after-adding-it-the-row-size-is-8572-which-is-greater-than-maximum-allowed-size-8126-for-a-record-on-index-leaf-page/ | 21:34 |
| rosmaita | that's how i read that too | 21:34 |
| rosmaita | shubjero: are you in a locale where you use a lot of chars that would take 2-3 bytes to represent in UTF8? | 21:56 |
| *** dviroel is now known as dviroel|out | 22:02 | |
| opendevreview | Oleg proposed openstack/cinder stable/yoga: Fix PowerFlex connector HTTPS certificate validation https://review.opendev.org/c/openstack/cinder/+/837432 | 22:33 |
| opendevreview | Oleg proposed openstack/cinder stable/yoga: Fix PowerFlex connector HTTPS certificate validation https://review.opendev.org/c/openstack/cinder/+/837432 | 22:49 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Deprecate [DEFAULT] use_forwarded_for https://review.opendev.org/c/openstack/cinder/+/836252 | 23:16 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!