| *** mhen_ is now known as mhen | 02:56 | |
| opendevreview | Johannes Beisiegel proposed openstack/cinder master: add oslo_db retry for volume_attach https://review.opendev.org/c/openstack/cinder/+/965362 | 11:29 |
|---|---|---|
| opendevreview | Johannes Beisiegel proposed openstack/cinder master: add oslo_db retry for volume_attach https://review.opendev.org/c/openstack/cinder/+/965362 | 11:29 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Fix perfromance stats collection error https://review.opendev.org/c/openstack/cinder/+/965365 | 11:37 |
| opendevreview | Johannes Beisiegel proposed openstack/cinder master: add oslo_db retry for volume_attach https://review.opendev.org/c/openstack/cinder/+/965362 | 11:44 |
| *** iurygregory_ is now known as iurygregory | 12:57 | |
| cardoe | jbernard: what time is the netapp | 13:34 |
| jbernard | cardoe: right at the start, 1400 (in 25 minutes) | 13:35 |
| opendevreview | Fernando Ferraz proposed openstack/devstack-plugin-nfs master: [WIP] zuul: add job for testing Glance with Cinder as backend https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/940482 | 14:21 |
| cardoe | Sorry I had to drop. | 14:39 |
| cardoe | Who were the NetApp folks? | 14:39 |
| jbernard | cardoe: nicks here are jayaanand, Anoop_Shukla, gireesh, and others | 14:44 |
| jbernard | jayaanand: do you have url for your markdown docs? | 14:47 |
| cardoe | jayaanand: Anoop_Shukla | 14:57 |
| cardoe | sorry for bad tagging there. | 14:57 |
| cardoe | So running into some issues with the driver when using NVMe. | 14:57 |
| cardoe | I discussed some of the oddities in the code the other day on a cinder call. But essentially the driver attempts to make the connection setup to be generic and loading the oslo_config section every time a connection is made by passing the section. | 14:58 |
| cardoe | NVMe only works over REST but due to the implementation it actually requires some SSH stuff because if you create a REST connection instance, internally it keeps a SSH connection as well. | 14:59 |
| cardoe | So that's a waste of a setup. | 14:59 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Fix perfromance stats collection error https://review.opendev.org/c/openstack/cinder/+/965365 | 15:10 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Fix performance stats collection error https://review.opendev.org/c/openstack/cinder/+/965365 | 15:11 |
| jbernard | sfernand: i underestimate the netapp size, we'll start with your topics once they've wrapped up | 15:15 |
| sfernand | jbernard: my tops have already been discussed yesterday | 15:20 |
| sfernand | we are all good, I think Erlons topics have been addressed too | 15:21 |
| jbernard | sfernand: ok, sounds good, just wanted to make sure | 15:24 |
| jbernard | noonedeadpunk: heya, after the glance team session, at 1700, okay for backup evolution topic? | 15:26 |
| gouthamr | eharney: we're going to start chatting about eventlet in the manila room in a few mins | 15:28 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Fix performance stats collection error https://review.opendev.org/c/openstack/cinder/+/965365 | 15:52 |
| cardoe | The REST interface also implements def __call__() which for any method calls the SSH connection. Which in get_volume_stats() happens every time cause something is queried that's not implemented over REST. | 16:03 |
| cardoe | From a maintainability standpoint it seems like you might want an abstract class for a "client" and then have the SSH, the REST (and now there's another REST client) all implement those interfaces. It just seems like it'll be much cleaner. | 16:05 |
| cardoe | It would also be good to pass the connection around rather than recreating the connection. We're seeing a LOT of warnings from Cinder that certain calls took a long time and its just doing costly setup each time. | 16:06 |
| cardoe | I've also been wanting to run a NetApp virtual appliance to test Cinder against (which I think could be useful for the generic Zuul case as well). | 16:07 |
| jayaanand | etherpad has encoding limitations... images spacings are gone https://etherpad.opendev.org/p/NetApp_cross-pool_cache and https://etherpad.opendev.org/p/NetApp_Replication_improvements | 16:14 |
| *** benj_8 is now known as benj_ | 16:17 | |
| opendevreview | Merged openstack/cinder master: mypy: Bump to 1.18.1 https://review.opendev.org/c/openstack/cinder/+/961985 | 16:17 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Configure check constraint https://review.opendev.org/c/openstack/cinder/+/896444 | 16:33 |
| opendevreview | Fernando Ferraz proposed openstack/devstack-plugin-nfs master: [WIP] zuul: add job for testing Glance with Cinder as backend https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/940482 | 16:53 |
| opendevreview | Fernando Ferraz proposed openstack/devstack-plugin-nfs master: [WIP] zuul: add job for testing Glance with Cinder as backend https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/940482 | 16:55 |
| opendevreview | Fernando Ferraz proposed openstack/devstack-plugin-nfs master: [DNM] Test Glance over Cinder/NFS with NFS driver fixes https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/965409 | 17:00 |
| jbernard | glance session running a bit long | 17:02 |
| jbernard | rosmaita: can https://review.opendev.org/c/openstack/cinder/+/846054 be abandoned? | 17:05 |
| rosmaita | i think so, given that cinderlib is no longer a thing ... we may want to revert the keyring change as no longer needed, that was https://review.opendev.org/c/openstack/cinder/+/738216 | 17:11 |
| jbernard | ok will add to my queue | 17:13 |
| jbernard | rosmaita: looking for reviews on https://review.opendev.org/c/openstack/cinder/+/959901 if you end up with some time | 17:13 |
| jbernard | rosmaita: there are a few related, but that particular one is the most important for them | 17:14 |
| jbernard | rosmaita: do i have permission to update the irc topic? | 17:14 |
| jbernard | rosmaita: who has that? | 17:14 |
| rosmaita | that's a good question, i have no idea | 17:15 |
| rosmaita | you may have to ask in #openstack-infra | 17:16 |
| jbernard | noonedeadpunk: are you around? | 17:20 |
| cardoe | So is there a cinderlib replacement? | 17:25 |
| jbernard | cardoe: no, it wasn't being maintained so we ended it | 17:26 |
| cardoe | ah okay cause I was gonna do those constants in os-brick but didn't know if there was a better place. | 17:32 |
| jbernard | it was really cool, just not enough interest to keep it going | 17:34 |
| cardoe | So just wondering if there'd be any appetite for having the ability to execute some driver backend code on a volume type create/update/delete? Since a volume type for many is associated with a pool it could be a way to dynamically create those as an admin. | 19:34 |
| cardoe | I'm also wondering if you guys have heard about any vendors using Swordfish? | 19:34 |
| cardoe | Along the lines of all that... given some storage vendors have partitioning (like NetApp and SVM/vserver) is there any appetite for some kind of "per tenant partition provisioning" API? | 19:40 |
| opendevreview | Merged openstack/cinder master: Integrate 'pre-commit' https://review.opendev.org/c/openstack/cinder/+/916789 | 20:16 |
| opendevreview | Merged openstack/cinder master: api: Move rate limiting middleware to its own module https://review.opendev.org/c/openstack/cinder/+/916434 | 20:16 |
| opendevreview | Merged openstack/cinder master: api: Remove v2 limits API controller https://review.opendev.org/c/openstack/cinder/+/916435 | 20:16 |
| opendevreview | Merged openstack/cinder master: api: Merge v2, v3 volume metadata controller https://review.opendev.org/c/openstack/cinder/+/916436 | 20:16 |
| opendevreview | Merged openstack/cinder master: tests: Don't use mocks as requests https://review.opendev.org/c/openstack/cinder/+/916437 | 20:17 |
| opendevreview | Merged openstack/cinder master: tests: Merge v2, v3 volume API tests https://review.opendev.org/c/openstack/cinder/+/916438 | 20:17 |
| opendevreview | Gireesh Awasthi proposed openstack/cinder master: NetApp - Enable configurable performance pooling. https://review.opendev.org/c/openstack/cinder/+/964191 | 21:17 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!