| opendevreview | Merged openstack/cinder master: db: Migrate "volume", "volume attachment" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/830095 | 05:30 |
|---|---|---|
| dc_arunkv | rosmaita: I have updated the Wiki page, sorry for the delay. Thank you. | 10:15 |
| *** whoami-rajat__ is now known as whoami-rajat | 10:36 | |
| *** dviroel_ is now known as dviroel | 11:19 | |
| *** sfinucan is now known as stephenfin | 12:04 | |
| opendevreview | Leonardo Campelo proposed openstack/cinder master: NetApp SolidFire: Fix manage_existing method https://review.opendev.org/c/openstack/cinder/+/755101 | 13:15 |
| whoami-rajat | mnaser, ricolin hey, rosmaita has added a topic discussing your patch at L#116 https://etherpad.opendev.org/p/cinder-zed-meetings | 13:47 |
| mnaser | ++ | 13:47 |
| whoami-rajat | the meeting is in 13 miutes in #openstack-meeting-alt channel (if you wish to attend the whole meeting) else i will ping when the topic is being discussed | 13:48 |
| ricolin | whoami-rajat: thanks for the notify:) | 13:48 |
| whoami-rajat | np | 13:48 |
| *** abishop_ is now known as abishop | 13:56 | |
| whoami-rajat | Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:58 |
| whoami-rajat | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval tobias-urdin | 13:58 |
| rosmaita | whoami-rajat: ty | 13:58 |
| whoami-rajat | np | 13:59 |
| rosmaita | oh, man, looking at the ping list i just realized that i never reviewed tobias-urdin's patch | 14:00 |
| whoami-rajat | :D better late than never | 14:00 |
| opendevreview | Eric Harney proposed openstack/cinder master: Use modern type annotation format for collections https://review.opendev.org/c/openstack/cinder/+/839987 | 14:01 |
| hemna | mep | 15:32 |
| hemna | made a simple tool to show the live view of volume states in a deployment. | 15:34 |
| hemna | https://asciinema.org/a/LRUINSLjPo0Xs6Hy0rVcjPxqu | 15:34 |
| *** dviroel is now known as dviroel|lunch | 15:39 | |
| whoami-rajat | nice | 15:47 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: api-ref: Add docs for clusters https://review.opendev.org/c/openstack/cinder/+/795785 | 15:53 |
| *** dviroel|lunch is now known as dviroel | 16:31 | |
| opendevreview | Merged openstack/cinder master: [IBM DS8000] Fixed Detach for multi-attach volumes https://review.opendev.org/c/openstack/cinder/+/837699 | 17:17 |
| hemna | ok looks like my suspicion about allocated_capacity_gb not being updated by the scheduler during volume create is true. | 18:26 |
| hemna | so you can currently create a bunch of volumes between the manager's call to get_volume_stats() and overprovision even thick provisioned pools | 18:27 |
| hemna | because volume creates don't trigger a get_volume_stats from the manager, and the scheduler doesn't update the allocated_capacity_gb. | 18:28 |
| hemna | bah consume_from_volume is broken | 18:45 |
| hemna | doesn't take into account pools!!! | 18:46 |
| hemna | https://github.com/openstack/cinder/blob/master/cinder/scheduler/host_manager.py#L335-L352 | 18:47 |
| hemna | no pools support at all | 18:47 |
| hemna | the revert call does though....https://github.com/openstack/cinder/blob/master/cinder/scheduler/host_manager.py#L335-L747 | 18:47 |
| hemna | man this has been broken forever | 18:47 |
| opendevreview | Walt proposed openstack/cinder master: Fix tracking allocated_capacity_gb https://review.opendev.org/c/openstack/cinder/+/836083 | 18:57 |
| hemna | ok filed a new bug for that one | 19:01 |
| opendevreview | Merged openstack/cinder master: pylint: ignore __original_module_threading errors https://review.opendev.org/c/openstack/cinder/+/842128 | 19:11 |
| *** dviroel is now known as dviroel|out | 20:28 | |
| opendevreview | Caique Mello proposed openstack/cinder master: [DNM] Testing tempest for storage_protocols https://review.opendev.org/c/openstack/cinder/+/842433 | 20:38 |
| opendevreview | Andrew Bogott proposed openstack/cinder master: backups: Use the most recent available backup for incrementals https://review.opendev.org/c/openstack/cinder/+/838009 | 21:22 |
| opendevreview | Andrew Bogott proposed openstack/cinder master: backups: Use the most recent available backup for incrementals https://review.opendev.org/c/openstack/cinder/+/838009 | 21:26 |
| opendevreview | Andrew Bogott proposed openstack/cinder master: backups: Use the most recent available backup for incrementals https://review.opendev.org/c/openstack/cinder/+/838009 | 21:30 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!