| *** dviroel|rover is now known as dviroel|out | 00:11 | |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: add an option for host group name format https://review.opendev.org/c/openstack/cinder/+/866526 | 06:18 |
|---|---|---|
| opendevreview | Jean-Sébastien Bevilacqua proposed openstack/cinder master: Add cinder Driver for Lustre https://review.opendev.org/c/openstack/cinder/+/853785 | 06:47 |
| opendevreview | Jean-Sébastien Bevilacqua proposed openstack/os-brick master: Add Lustre support in remote fs https://review.opendev.org/c/openstack/os-brick/+/853787 | 06:48 |
| LarsErikP | Hi! I'll try again, maybe tagging whoami-rajat , was anything done regarding this blueprint? https://specs.openstack.org/openstack/cinder-specs/specs/yoga/optimize-upload-volume-to-rbd-store.html | 07:25 |
| whoami-rajat | LarsErikP, hey | 07:26 |
| whoami-rajat | LarsErikP, so the problem is that effort depends on the service role in keystone | 07:27 |
| whoami-rajat | I've done the foundation work in cinder in last cycle but it can't be completed until we've the service role in place | 07:27 |
| LarsErikP | aha | 07:27 |
| LarsErikP | so, in general, people just accept that the upload-to-image process is slow, and gives their cinder hosts a lot of disk? :p | 07:28 |
| whoami-rajat | yeah, maybe not a lot of people use this operation at a large scale or at high concurrency | 07:33 |
| whoami-rajat | there are other problems associated with it like the dependency chain RBD has, so there will be deployment changes as well with this | 07:33 |
| opendevreview | Hao Wang proposed openstack/cinder master: Support to query snapshot filter by updated_at/created_at https://review.opendev.org/c/openstack/cinder/+/733062 | 08:38 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Configure a storage node in cinder https://review.opendev.org/c/openstack/cinder/+/820994 | 10:25 |
| opendevreview | Eric Harney proposed openstack/cinder master: RBD: Flattening of child volumes during deletion https://review.opendev.org/c/openstack/cinder/+/835384 | 10:54 |
| *** dviroel|out is now known as dviroel|rover | 11:37 | |
| opendevreview | YuehuiLei proposed openstack/cinder master: RemoteFs:Ensure cinder can create mount directory https://review.opendev.org/c/openstack/cinder/+/850142 | 12:15 |
| nahimsouza[m] | Hello, everyone, we are needing some help to review/merge the patch https://review.opendev.org/c/openstack/cinder/+/863711 - QA/CI is passing - please, take a look when you have a chance. | 12:17 |
| nahimsouza[m] | whoami-rajat - this patch is related to the bugfix on FCP volume attachment we discussed some time ago, it would be good to have your eyes on this change :) | 12:17 |
| whoami-rajat | nahimsouza[m], sure, will take a look | 12:19 |
| nahimsouza[m] | thank you whoami-rajat | 12:20 |
| opendevreview | Felix Huettner proposed openstack/cinder master: Limit number of file descriptors for image conversion https://review.opendev.org/c/openstack/cinder/+/871976 | 13:37 |
| ganso | rosmaita, whoami-rajat: hi! thanks for approving the patch! I don't remember if I asked this in the past, but do you think the patch could be backported? I suspect it cannot as it is adding a new config option, but I just wanted to confirm | 15:00 |
| ganso | the patch: Filter reserved image properties: https://review.opendev.org/c/openstack/cinder/+/812685 | 15:00 |
| whoami-rajat | ganso, interesting question, it does have a new config option which we don't usually backport but it's also a bugfix affecting branches that run glance multi store | 15:51 |
| whoami-rajat | rosmaita, what do you think about this? ^ | 15:51 |
| rosmaita | definitely backport | 15:52 |
| rosmaita | i think as long as the change doesn't break existing configs, it's ok to backport | 15:52 |
| opendevreview | Rodrigo Barbieri proposed openstack/cinder stable/zed: Filter reserved image properties https://review.opendev.org/c/openstack/cinder/+/872581 | 15:52 |
| rosmaita | and iirc, the opt is empty by default? | 15:52 |
| ganso | I pushed the trigger ^ xD | 15:53 |
| whoami-rajat | it's empty but we've internal values ["os_glance", "img_signature"] | 15:53 |
| whoami-rajat | the releasenote doesn't have a features section which is a good thing, we surely need the upgrade and fixes section even in backport https://review.opendev.org/c/openstack/cinder/+/812685/40/releasenotes/notes/fix-reserved-image-properties-9519ddc080e7ed1a.yaml | 15:54 |
| rosmaita | yeah, but i think the issue is that you don't need to modify the config to get the fix to work | 15:54 |
| whoami-rajat | yes, correct | 15:54 |
| whoami-rajat | so i see no issues apart from the new config option which i think should be fine with it's default value = [] | 15:55 |
| ganso | whoami-rajat, rosmaita: thank you! :D | 15:56 |
| whoami-rajat | np | 15:58 |
| drencrom | eharney: You asked a question yesterday about this patch https://review.opendev.org/c/openstack/cinder/+/868485 regarding if large matadata were previously allowed | 16:55 |
| drencrom | I checked and the 255 char limit seems to be in place since 2015 when the API for managing image metadata was introduced (https://github.com/openstack/cinder/commit/4196e5f2d56d68134f08563e7eb51164214d2075) | 16:56 |
| opendevreview | Jorge Merlino proposed openstack/cinder master: Increase size of volume image metadata values https://review.opendev.org/c/openstack/cinder/+/868485 | 17:26 |
| hemna | mep | 20:15 |
| *** dviroel|rover is now known as dviroel|rout | 22:38 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!