openstackgerrit | Tom Barron proposed openstack/cinder: Fix invalid uuid warnings in backup unit tests https://review.openstack.org/286118 | 00:04 |
---|---|---|
*** andymaier_ has quit IRC | 00:06 | |
*** yangyapeng has quit IRC | 00:08 | |
*** ntpttr has quit IRC | 00:08 | |
openstackgerrit | Kurt Martin proposed openstack/cinder: Don't fail on clearing 3PAR object volume key https://review.openstack.org/287502 | 00:10 |
openstackgerrit | Tom Barron proposed openstack/cinder: Fix retype failure when original has no volume type https://review.openstack.org/282579 | 00:12 |
openstackgerrit | Patrick East proposed openstack/cinder: Add backend id to Pure Volume Driver trace logs https://review.openstack.org/288190 | 00:17 |
*** laughterwym has joined #openstack-cinder | 00:27 | |
*** diogogmt has quit IRC | 00:28 | |
*** hemna is now known as hemnafk | 00:30 | |
*** daneyon_ has quit IRC | 00:31 | |
*** andymaier_ has joined #openstack-cinder | 00:32 | |
*** Rodrigo_BR has joined #openstack-cinder | 00:34 | |
Rodrigo_BR | Live Migration failure: Unsafe migration: Migration may lead to data corruption if disks use cache != none | 00:34 |
Rodrigo_BR | Can I perform live migration using NFS backend ? | 00:34 |
tbarron | Rodrigo_BR: hmm, can you tell us more about this? looks like there might be a launchpad bug, but I don't see the link. | 00:40 |
patrickeast | Rodrigo_BR: i think you should be able to, assuming you understand the risks you can get around that error with VIR_MIGRATE_UNSAFE in your migrations flags in nova.conf | 00:40 |
tbarron | k, patrickeast knows more about this, I'll listen. | 00:42 |
patrickeast | tbarron: haha, not sure about that... just quick to googling :p | 00:43 |
*** laughterwym has quit IRC | 00:43 | |
patrickeast | tbarron: Rodrigo_BR: found this https://bugs.launchpad.net/nova/+bug/1222682 and did a quick look at https://www.redhat.com/archives/libvir-list/2012-February/msg00883.html | 00:43 |
openstack | Launchpad bug 1222682 in openstack-manuals "Live Migration does not work unless cache=none" [High,Won't fix] | 00:43 |
*** laughterwym has joined #openstack-cinder | 00:44 | |
tbarron | patrickeast: Rodrigo_BR : so is there something special about NFS that would imply that "it's configured | 00:45 |
tbarron | in a way that does not ensure data integrity once it is migrated"? | 00:45 |
* tbarron has a improbable thought that Rodrigo_BR might be on vacation from manila, where he knows a whole lot about such stuff, slumming over here after FF | 00:46 | |
*** laughterwym has quit IRC | 00:48 | |
patrickeast | tbarron: from the docs it seems like having the cache enabled might mean that the files are not in sync for both systems, which could cause the unsafe migration | 00:49 |
Rodrigo_BR | I am using the netapp driver in cinder | 00:50 |
Rodrigo_BR | a post the configuration in https://ask.openstack.org/en/question/89302/migration-operation-has-aborted/ | 00:50 |
*** gouthamr has quit IRC | 00:51 | |
tbarron | patrickeast: k, that makes sense. | 00:51 |
tbarron | Rodrigo_BR: looking ... | 00:51 |
tbarron | Rodrigo_BR: I think the safest thing to do is ask on #openstack-netapp | 00:52 |
tbarron | Rodrigo_BR: I used to work there and I don't know about the cacheiing. | 00:53 |
tbarron | Rodrigo_BR: offhand I would think it's a client-side thing, not NetApp or any server-side specific. | 00:53 |
tbarron | Rodrigo_BR: but I haven't checked that answer in detail. | 00:53 |
tbarron | Rodrigo_BR: you should be aware that it's Feature Freeze crunch right now, so people's attentions are on that. | 00:54 |
tbarron | Rodrigo_BR: unless it's absolutely critical, you'll probably get better response on that channel in a couple days. | 00:54 |
tbarron | patrickeast: Rodrigo_BR: thanks for pointing me to this NFS caching issue. Learned a bit. | 00:56 |
Rodrigo_BR | <tbarron> Thank You | 00:56 |
*** ntpttr has joined #openstack-cinder | 00:56 | |
Rodrigo_BR | <tbarron> I will try in ocenstack-netapp | 00:56 |
*** laughterwym has joined #openstack-cinder | 00:57 | |
Rodrigo_BR | * Sorry | 00:57 |
Rodrigo_BR | openstack-netapp | 00:57 |
*** akshai has quit IRC | 00:58 | |
*** akshai has joined #openstack-cinder | 01:02 | |
*** laughter_ has joined #openstack-cinder | 01:04 | |
*** laughter_ has quit IRC | 01:04 | |
*** jaypipes has quit IRC | 01:04 | |
*** laughter_ has joined #openstack-cinder | 01:04 | |
*** laughter_ has quit IRC | 01:05 | |
*** laughter_ has joined #openstack-cinder | 01:05 | |
*** laughter_ has quit IRC | 01:06 | |
*** laughterwym has quit IRC | 01:06 | |
*** laughterwym has joined #openstack-cinder | 01:06 | |
*** akshai has quit IRC | 01:08 | |
*** laughter_ has joined #openstack-cinder | 01:10 | |
*** laughterwym has quit IRC | 01:10 | |
*** laughter_ has quit IRC | 01:11 | |
*** yangyapeng has joined #openstack-cinder | 01:11 | |
*** laughterwym has joined #openstack-cinder | 01:11 | |
*** sasukeh has joined #openstack-cinder | 01:15 | |
*** ntpttr has quit IRC | 01:16 | |
*** yangyapeng has quit IRC | 01:17 | |
*** Suyash has quit IRC | 01:17 | |
*** garthb has quit IRC | 01:19 | |
*** gouthamr has joined #openstack-cinder | 01:25 | |
*** EinstCrazy has joined #openstack-cinder | 01:26 | |
*** gouthamr_ has joined #openstack-cinder | 01:28 | |
*** akshai has joined #openstack-cinder | 01:31 | |
*** gouthamr has quit IRC | 01:31 | |
*** bardia has quit IRC | 01:34 | |
*** diogogmt has joined #openstack-cinder | 01:35 | |
openstackgerrit | Merged openstack/cinder: Tintri image direct clone https://review.openstack.org/261126 | 01:35 |
*** laughterwym has quit IRC | 01:36 | |
*** laughterwym has joined #openstack-cinder | 01:37 | |
*** cheneydc has joined #openstack-cinder | 01:37 | |
*** mtanino has quit IRC | 01:37 | |
*** akshai_ has joined #openstack-cinder | 01:39 | |
*** laughte__ has joined #openstack-cinder | 01:39 | |
*** laught___ has joined #openstack-cinder | 01:40 | |
*** laughterwym has quit IRC | 01:41 | |
*** akshai has quit IRC | 01:42 | |
*** laughte__ has quit IRC | 01:44 | |
*** laught___ has quit IRC | 01:45 | |
*** laughterwym has joined #openstack-cinder | 01:48 | |
openstackgerrit | Anthony Lee proposed openstack/python-brick-cinderclient-ext: Query volume paths implementation https://review.openstack.org/268405 | 01:50 |
openstackgerrit | Merged openstack/cinder: Update unittest for Storwize pool-aware-cinder-scheduler https://review.openstack.org/287120 | 01:53 |
*** apoorvad has quit IRC | 01:53 | |
*** bjornar has quit IRC | 01:55 | |
*** esker has joined #openstack-cinder | 02:01 | |
*** rhagarty_ has quit IRC | 02:04 | |
*** esker has quit IRC | 02:07 | |
*** vilobhmm11 has quit IRC | 02:09 | |
*** rhagarty has joined #openstack-cinder | 02:12 | |
openstackgerrit | Peter Wang proposed openstack/cinder: VNX: Update replication for v2.1 https://review.openstack.org/285139 | 02:12 |
*** bardia has joined #openstack-cinder | 02:16 | |
*** baojg has joined #openstack-cinder | 02:17 | |
*** ntpttr has joined #openstack-cinder | 02:22 | |
*** andymaier_ has quit IRC | 02:26 | |
*** ntpttr has quit IRC | 02:28 | |
*** Lee1092 has joined #openstack-cinder | 02:31 | |
*** Rodrigo_BR has quit IRC | 02:32 | |
*** gcb has quit IRC | 02:34 | |
*** apoorvad has joined #openstack-cinder | 02:36 | |
*** rhagarty has quit IRC | 02:36 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Record and check LUN wwn https://review.openstack.org/271592 | 02:38 |
*** haomaiwang has joined #openstack-cinder | 02:39 | |
*** esker has joined #openstack-cinder | 02:43 | |
*** houming has joined #openstack-cinder | 02:44 | |
*** haomaiwang has quit IRC | 02:45 | |
*** esker has quit IRC | 02:48 | |
*** akshai_ has quit IRC | 02:51 | |
*** gcb has joined #openstack-cinder | 02:53 | |
*** akshai has joined #openstack-cinder | 02:56 | |
*** haomaiwang has joined #openstack-cinder | 02:56 | |
*** cdelatte has joined #openstack-cinder | 02:59 | |
*** akshai has quit IRC | 03:00 | |
*** haomaiwang has quit IRC | 03:01 | |
*** links has joined #openstack-cinder | 03:01 | |
*** mmandel has joined #openstack-cinder | 03:08 | |
*** haomaiwang has joined #openstack-cinder | 03:12 | |
*** vilobhmm11 has joined #openstack-cinder | 03:15 | |
*** haomaiwang has quit IRC | 03:17 | |
*** esker has joined #openstack-cinder | 03:18 | |
*** vilobhmm111 has joined #openstack-cinder | 03:22 | |
*** vilobhmm11 has quit IRC | 03:25 | |
openstackgerrit | Merged openstack/cinder: 3PAR use same LUN id for each export path https://review.openstack.org/286889 | 03:37 |
openstackgerrit | Merged openstack/cinder: EMC VMAX - Limit SG and MV to 64 characters https://review.openstack.org/272777 | 03:38 |
*** vilobhmm111 has quit IRC | 03:39 | |
*** dims has quit IRC | 03:40 | |
*** dims has joined #openstack-cinder | 03:43 | |
*** dims has quit IRC | 03:44 | |
*** links has quit IRC | 03:44 | |
*** apoorvad has quit IRC | 03:45 | |
*** gouthamr_ has quit IRC | 03:48 | |
*** esker has quit IRC | 03:49 | |
*** yangyapeng has joined #openstack-cinder | 03:50 | |
*** Poornima has joined #openstack-cinder | 03:51 | |
*** hemna has joined #openstack-cinder | 03:55 | |
*** asselin__ has joined #openstack-cinder | 04:02 | |
*** mmandel has quit IRC | 04:02 | |
*** jdurgin1 has joined #openstack-cinder | 04:04 | |
*** laughterwym has quit IRC | 04:13 | |
*** laughterwym has joined #openstack-cinder | 04:15 | |
*** hemna has quit IRC | 04:16 | |
*** garthb has joined #openstack-cinder | 04:17 | |
*** laughterwym has quit IRC | 04:27 | |
*** esker has joined #openstack-cinder | 04:27 | |
*** sborkows has quit IRC | 04:27 | |
*** links has joined #openstack-cinder | 04:27 | |
*** yangyape_ has joined #openstack-cinder | 04:27 | |
*** openstack has joined #openstack-cinder | 14:08 | |
openstackgerrit | Philipp Marek proposed openstack/cinder: DRBD: Policy-based waiting for completion https://review.openstack.org/287807 | 14:09 |
*** haomaiwang has joined #openstack-cinder | 14:13 | |
*** mriedem has joined #openstack-cinder | 14:15 | |
*** knikolla has joined #openstack-cinder | 14:21 | |
*** edtubill has joined #openstack-cinder | 14:22 | |
*** ildikov has joined #openstack-cinder | 14:24 | |
*** dustins has joined #openstack-cinder | 14:25 | |
*** Julien-zte has joined #openstack-cinder | 14:26 | |
*** dansmith is now known as superdan | 14:29 | |
smcginnis | flip214: You commented "new version coming up Monday", but is that it? ^^ | 14:31 |
*** merooney has quit IRC | 14:32 | |
e0ne | smcginnis: hi. are we going to get this https://review.openstack.org/#/c/268405/ in Mitaka or it's too late? | 14:33 |
smcginnis | e0ne: It's too late. I had to cut the release yesterday for that. | 14:34 |
smcginnis | e0ne: So we have a very limited 0.1.0 release now. | 14:34 |
e0ne | smcginnis: ok, thanks | 14:34 |
smcginnis | e0ne: Wish we could have gotten more in there, but it's a good start. | 14:34 |
e0ne | smcginnis: do you know when will we got stable branches for clients? | 14:35 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/python-brick-cinderclient-ext: Require root permissions for CLI operations https://review.openstack.org/287938 | 14:35 |
smcginnis | e0ne: If I remember right they get branched at RC time. | 14:35 |
*** merooney has joined #openstack-cinder | 14:35 | |
*** omolchanov1 has quit IRC | 14:35 | |
e0ne | smcginnis: ok, I was not sure about clients | 14:36 |
*** omolchanov has joined #openstack-cinder | 14:36 | |
*** openstackgerrit_ has quit IRC | 14:36 | |
smcginnis | e0ne: Yeah, they are a little different. | 14:36 |
*** dave-mccowan has joined #openstack-cinder | 14:37 | |
*** mmandel has joined #openstack-cinder | 14:37 | |
*** openstackgerrit_ has joined #openstack-cinder | 14:38 | |
*** eharney has joined #openstack-cinder | 14:39 | |
mriedem | can one of the cinder drivers accept the liberty nomination for this? https://bugs.launchpad.net/cinder/+bug/1523646 | 14:41 |
openstack | Launchpad bug 1523646 in OpenStack Compute (nova) liberty "Nova/Cinder Key Manager for Barbican Uses Stale Cache" [High,In progress] - Assigned to Matt Riedemann (mriedem) | 14:41 |
mriedem | the fix is released on liberty, i'm just doing LP paperwokr | 14:41 |
mriedem | *paperwork | 14:41 |
*** links has quit IRC | 14:41 | |
smcginnis | mriedem: Don | 14:42 |
smcginnis | e | 14:42 |
mriedem | thanks | 14:42 |
*** diogogmt has quit IRC | 14:42 | |
*** rhagarty has joined #openstack-cinder | 14:45 | |
*** omolchanov has quit IRC | 14:46 | |
openstackgerrit | Merged openstack/cinder: Imported Translations from Zanata https://review.openstack.org/288264 | 14:46 |
*** rhagarty_ has joined #openstack-cinder | 14:46 | |
*** omolchanov has joined #openstack-cinder | 14:47 | |
*** jgregor has joined #openstack-cinder | 14:48 | |
*** rhagarty has quit IRC | 14:48 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/288344 | 14:54 |
*** cknight has joined #openstack-cinder | 14:55 | |
*** jwcroppe has joined #openstack-cinder | 14:55 | |
EmilienM | Hi Cinder folks, I've been trying (and failed) to deploy Cinder API with SSL enabled (without apache) - is it something that is supposed to work? | 14:56 |
eharney | thingee: the dependency for this patch has landed in openstack/requirements now: https://review.openstack.org/#/c/284929/ | 14:56 |
*** martyturner has quit IRC | 14:57 | |
*** diogogmt has joined #openstack-cinder | 14:57 | |
*** merooney has quit IRC | 14:58 | |
smcginnis | EmilienM: I haven't tried. | 14:58 |
*** baumann has joined #openstack-cinder | 14:58 | |
smcginnis | EmilienM: I know we recently had a patch to switch some wsgi stuff over to oslo. Hopefully we didn't miss something there. | 14:58 |
EmilienM | smcginnis: and when I deploy with apache, it does not work either, I've got 500 errors | 14:58 |
EmilienM | in cinder.conf; I give all ssl options but cinder keeps starting in http | 14:59 |
smcginnis | EmilienM: Does it work for you with non-SSL and Apache? | 14:59 |
EmilienM | no ! | 14:59 |
EmilienM | I have some logs, etc I can show you, a sec | 14:59 |
smcginnis | EmilienM: Maybe you're apache setup isn't right? | 14:59 |
EmilienM | I copied devstack ;è) | 14:59 |
smcginnis | EmilienM: If you could file a bug that would be great. | 15:00 |
EmilienM | http://logs.openstack.org/91/288091/4/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-trusty/2010cc5/logs/apache_config/10-cinder_wsgi.conf.txt.gz | 15:00 |
EmilienM | and I've got http://logs.openstack.org/91/288091/4/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-trusty/2010cc5/logs/apache/cinder_wsgi_error.txt.gz | 15:00 |
*** haomaiwang has quit IRC | 15:01 | |
smcginnis | EmilienM: This is off of master? | 15:01 |
*** tonyb has quit IRC | 15:01 | |
EmilienM | smcginnis: running mitaka b2 | 15:01 |
*** tonyb has joined #openstack-cinder | 15:01 | |
*** haomaiwang has joined #openstack-cinder | 15:01 | |
e0ne | smcginnis: to be sure: we're accepting only bugfixes now, aren't we? E.g. https://review.openstack.org/#/c/286600/5 | 15:01 |
smcginnis | e0ne: There were some of those that were going to make upgrades easier that we might still want to get in. | 15:02 |
eharney | e0ne: xyang1: could you look at my comment here before this patch merges? https://review.openstack.org/#/c/275408/ | 15:02 |
smcginnis | e0ne: Bug fix only, but at the discretion of the core team if we want to accept anything else based on risk of introducing changes that might break things. | 15:02 |
*** timcl has quit IRC | 15:02 | |
smcginnis | e0ne: And replication support. | 15:02 |
*** yuriy_n17 has joined #openstack-cinder | 15:03 | |
e0ne | smcginnis: yes. I remember about replication support | 15:03 |
*** timcl has joined #openstack-cinder | 15:03 | |
smcginnis | EmilienM: This came in after b2: https://review.openstack.org/#/c/258541/ | 15:03 |
smcginnis | EmilienM: I wonder if that might have an impact. | 15:03 |
*** martyturner has joined #openstack-cinder | 15:04 | |
e0ne | eharney: I'm sorry, I thought it was addressed | 15:04 |
*** merooney has joined #openstack-cinder | 15:04 | |
e0ne | eharney, xyang1: can we -2 or remove +A on it? | 15:04 |
smcginnis | e0ne: We don't need to block the lib patches either. Those are released, so anything that goes in now will just be in the next release we do post-Mitaka. | 15:05 |
*** kenhui has quit IRC | 15:05 | |
openstackgerrit | Eric Harney proposed openstack/cinder: Correcting thin provisioning behavior https://review.openstack.org/275408 | 15:05 |
eharney | have to hit rebase to kick it out of the queue ^ | 15:05 |
*** arecknag has quit IRC | 15:06 | |
e0ne | smcginnis: it's bacame too complicated. is it documented anywhere? | 15:07 |
smcginnis | e0ne: Is what documented? | 15:07 |
e0ne | smcginnis: what do we accept now before RC1 and what should be -2'd | 15:07 |
xyang1 | e0ne: eharney removed +A | 15:09 |
e0ne | smcginnis: to get released library for Mitaka without a stable branch is confusing, IMO | 15:09 |
smcginnis | e0ne: Really cinder is the only one that we need to worry about. The rest of the projects are libraries that are released on demand. | 15:09 |
smcginnis | e0ne: We don't _really_ do anything with stable branches on libs. | 15:09 |
xyang1 | eharney: are you suggesting to give warning now and throw exception in Newton? | 15:09 |
smcginnis | bbl | 15:10 |
e0ne | smcginnis: but we've got stable branch for cinderclient too | 15:10 |
eharney | xyang1: yes, that's how we normally do this | 15:10 |
xyang1 | eharney: ok | 15:10 |
*** markvoelker has quit IRC | 15:10 | |
*** markvoelker has joined #openstack-cinder | 15:10 | |
*** omolchanov has quit IRC | 15:10 | |
*** omolchanov has joined #openstack-cinder | 15:11 | |
eharney | could someone drop by this patch to get some missing requirements added? https://review.openstack.org/#/c/284929/ | 15:11 |
*** Julien-zte has quit IRC | 15:13 | |
e0ne | eharney: if it's only a driver requirements, do we really need it in requirements.txt? | 15:13 |
eharney | e0ne: we do if the driver doesn't do any handling around it not being present | 15:14 |
*** esker has joined #openstack-cinder | 15:14 | |
openstackgerrit | Merged openstack/cinder: Fix invalid uuid warnings in backup unit tests https://review.openstack.org/286118 | 15:14 |
*** esker has quit IRC | 15:15 | |
*** esker has joined #openstack-cinder | 15:15 | |
*** timcl has quit IRC | 15:17 | |
*** mvk has joined #openstack-cinder | 15:22 | |
*** david_lyle__ has joined #openstack-cinder | 15:34 | |
*** rhagarty_ has quit IRC | 15:36 | |
*** david-lyle has quit IRC | 15:37 | |
*** yangyapeng has joined #openstack-cinder | 15:38 | |
*** yangyapeng has quit IRC | 15:39 | |
*** lpetrut has quit IRC | 15:39 | |
*** lpetrut has joined #openstack-cinder | 15:39 | |
*** crose has joined #openstack-cinder | 15:41 | |
*** laughterwym has quit IRC | 15:44 | |
*** laughterwym has joined #openstack-cinder | 15:45 | |
*** lpetrut has quit IRC | 15:47 | |
*** kenhui has joined #openstack-cinder | 15:48 | |
*** mtanino has joined #openstack-cinder | 15:49 | |
*** jungleboyj has joined #openstack-cinder | 15:50 | |
*** rhagarty has joined #openstack-cinder | 15:51 | |
*** ntpttr has joined #openstack-cinder | 15:52 | |
*** rhagarty_ has joined #openstack-cinder | 15:52 | |
*** dims is now known as dimsum_ | 15:53 | |
*** merooney has quit IRC | 15:54 | |
*** martyturner has quit IRC | 15:54 | |
*** mriedem is now known as mriedem_meeting | 15:55 | |
*** rhagarty has quit IRC | 15:56 | |
*** timcl has joined #openstack-cinder | 15:56 | |
*** martyturner has joined #openstack-cinder | 15:58 | |
*** omolchanov has quit IRC | 15:58 | |
*** omolchanov1 has joined #openstack-cinder | 15:58 | |
*** merooney has joined #openstack-cinder | 15:58 | |
*** haomaiwang has quit IRC | 16:01 | |
*** hemnafk is now known as hemna | 16:04 | |
openstackgerrit | Scott DAngelo proposed openstack/cinder: support new HTTP microversion header https://review.openstack.org/286901 | 16:04 |
*** daneyon has joined #openstack-cinder | 16:07 | |
*** laughterwym has quit IRC | 16:07 | |
*** belmoreira has quit IRC | 16:07 | |
*** laughterwym has joined #openstack-cinder | 16:08 | |
*** e0ne has quit IRC | 16:12 | |
*** merooney has quit IRC | 16:13 | |
*** daneyon_ has joined #openstack-cinder | 16:13 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Refactor sqlalchemy service methods https://review.openstack.org/286598 | 16:15 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add service_nodes table and related methods https://review.openstack.org/286599 | 16:15 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add ServiceNode Versioned Object https://review.openstack.org/286600 | 16:15 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Change Job Distribution for HA A-A https://review.openstack.org/286601 | 16:15 |
*** daneyon has quit IRC | 16:16 | |
*** ivase has joined #openstack-cinder | 16:17 | |
*** david_lyle__ is now known as david_lyle | 16:17 | |
*** kmartin__ has quit IRC | 16:17 | |
*** laughter_ has joined #openstack-cinder | 16:18 | |
*** timcl has quit IRC | 16:22 | |
*** laughterwym has quit IRC | 16:22 | |
mriedem_meeting | e0ne must have left? | 16:23 |
mriedem_meeting | jungleboyj: https://review.openstack.org/#/c/286677/ since you're stable CPL | 16:23 |
jungleboyj | mriedem_meeting: Looking. | 16:25 |
*** laughter_ has quit IRC | 16:25 | |
*** laughterwym has joined #openstack-cinder | 16:25 | |
*** tongli has joined #openstack-cinder | 16:26 | |
jungleboyj | mriedem_meeting: Since it is a test impact I am fine with it. What is up with the py34 failure though? | 16:26 |
*** andymaier_ has quit IRC | 16:27 | |
mriedem_meeting | looking | 16:27 |
jungleboyj | mriedem_meeting: I looked at this yesterday and debated. Thought I ahd voted. | 16:27 |
mriedem_meeting | oh the py34 thing is the hyperv fialure that we fixed the other day | 16:27 |
mriedem_meeting | i'll recheck | 16:27 |
jungleboyj | mriedem_meeting: Ok. Cool. | 16:27 |
mriedem_meeting | i also replied to ivan in there | 16:28 |
jungleboyj | Yeah, I saw that. | 16:28 |
mriedem_meeting | the point is the 65K debug logging messages in a console log per unit test run | 16:28 |
mriedem_meeting | which kills our ELK stack in infra | 16:28 |
*** rhagarty has joined #openstack-cinder | 16:28 | |
*** rhagarty_ has quit IRC | 16:30 | |
jungleboyj | mriedem_meeting: +2 from me. smcginnis can you take a look when you have a chance. It is rechecking right now. | 16:30 |
smcginnis | jungleboyj, mriedem_meeting: Done | 16:33 |
*** yuriy_n17 has quit IRC | 16:35 | |
*** mriedem_meeting is now known as mriedem | 16:40 | |
*** jaypipes has joined #openstack-cinder | 16:41 | |
*** jaypipes is now known as leakypipes | 16:42 | |
*** leeantho has joined #openstack-cinder | 16:42 | |
*** rhagarty_ has joined #openstack-cinder | 16:45 | |
*** rhagarty has quit IRC | 16:47 | |
*** fthiagogv__ is now known as fthiagogv | 16:47 | |
*** apoorvad has joined #openstack-cinder | 16:49 | |
*** rcernin has quit IRC | 16:51 | |
*** mmandel has quit IRC | 16:54 | |
*** leeantho_ has joined #openstack-cinder | 16:59 | |
*** omolchanov1 has quit IRC | 17:02 | |
*** omolchanov has joined #openstack-cinder | 17:02 | |
*** leeantho has quit IRC | 17:02 | |
*** daneyon_ has quit IRC | 17:02 | |
*** kmartin has joined #openstack-cinder | 17:02 | |
*** daneyon has joined #openstack-cinder | 17:03 | |
*** laughterwym has quit IRC | 17:09 | |
*** laughterwym has joined #openstack-cinder | 17:10 | |
*** mmandel has joined #openstack-cinder | 17:10 | |
diablo_rojo | Does anyone know if there are volume attach_statuses besides attached, attaching, invalid and detached? Or is that it? | 17:11 |
*** bjornar has joined #openstack-cinder | 17:13 | |
*** daneyon has quit IRC | 17:13 | |
*** laughterwym has quit IRC | 17:14 | |
mc_nair | diablo_rojo: I'm seeing error_attaching and error_detaching also | 17:16 |
mc_nair | diablo_rojo: in the manager. Trying to see if anything else. Seems like maybe a detaching should be in there too? Haven't seen it yet | 17:16 |
diablo_rojo | mc_nair: Yeah I thought there would be a detaching too, but I haven't seen one | 17:17 |
openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Does not reset volume status when resetting migration status https://review.openstack.org/288598 | 17:19 |
openstackgerrit | Merged openstack/cinder: Convert huawei ISCSIDriver unit tests to RFC5737 addrs https://review.openstack.org/287165 | 17:21 |
*** martyturner has quit IRC | 17:22 | |
scottda | attach_status does not completely map to volume status | 17:24 |
scottda | It's really clunky that we need both, IMO | 17:24 |
*** martyturner has joined #openstack-cinder | 17:24 | |
diablo_rojo | scottda: Yeah I noticed not everything carried over, I just wanted to be sure I wasn't missing any valid statuses. | 17:24 |
*** haplo37 has joined #openstack-cinder | 17:28 | |
*** Suyash has joined #openstack-cinder | 17:29 | |
openstackgerrit | Merged openstack/cinder: EMC VMAX - Recreating SG when it has been deleted https://review.openstack.org/279900 | 17:36 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/288344 | 17:41 |
*** ndipanov has quit IRC | 17:43 | |
*** merooney has joined #openstack-cinder | 17:46 | |
*** jistr has quit IRC | 17:46 | |
*** vilobhmm11 has joined #openstack-cinder | 17:46 | |
*** jwcroppe has quit IRC | 17:47 | |
*** jwcroppe has joined #openstack-cinder | 17:48 | |
*** vilobhmm111 has joined #openstack-cinder | 17:49 | |
*** Thelo has quit IRC | 17:49 | |
*** Thelo has joined #openstack-cinder | 17:50 | |
*** vilobhmm11 has quit IRC | 17:51 | |
*** salv-orlando has joined #openstack-cinder | 17:52 | |
diablo_rojo | scottda: Is availble a valid attach status? I just saw an instance of it in test_dothill.py | 17:53 |
diablo_rojo | *available | 17:53 |
*** akerr_ has quit IRC | 17:54 | |
*** timcl has joined #openstack-cinder | 17:55 | |
*** salv-orl_ has quit IRC | 17:55 | |
*** leakypipes has quit IRC | 17:56 | |
*** kenhui has quit IRC | 17:56 | |
*** timcl has quit IRC | 18:02 | |
scottda | diablo_rojo: I'm not positive. I can look in a little while... | 18:02 |
*** timcl1 has joined #openstack-cinder | 18:02 | |
*** esker has quit IRC | 18:02 | |
*** chris_morrell has joined #openstack-cinder | 18:04 | |
*** ntpttr_ has joined #openstack-cinder | 18:07 | |
*** ntpttr has left #openstack-cinder | 18:10 | |
*** permalac has quit IRC | 18:12 | |
*** merooney has quit IRC | 18:12 | |
*** kenhui has joined #openstack-cinder | 18:14 | |
*** ntpttr_ has quit IRC | 18:15 | |
*** vilobhmm111 has quit IRC | 18:16 | |
*** ntpttr_ has joined #openstack-cinder | 18:16 | |
*** ntpttr_ has quit IRC | 18:16 | |
*** jdurgin1 has quit IRC | 18:17 | |
*** ntpttr_ has joined #openstack-cinder | 18:19 | |
*** tongli has quit IRC | 18:21 | |
*** merooney has joined #openstack-cinder | 18:21 | |
*** ntpttr_ has quit IRC | 18:25 | |
*** jordanP has quit IRC | 18:25 | |
*** shakamunyi has joined #openstack-cinder | 18:29 | |
*** ntpttr_ has joined #openstack-cinder | 18:30 | |
*** ntpttr__ has joined #openstack-cinder | 18:31 | |
*** ntpttr__ has quit IRC | 18:32 | |
*** mrrowden has joined #openstack-cinder | 18:32 | |
*** JoseMello has joined #openstack-cinder | 18:34 | |
*** akerr has joined #openstack-cinder | 18:35 | |
*** eharney has quit IRC | 18:35 | |
*** ntpttr has joined #openstack-cinder | 18:37 | |
*** ntpttr_ is now known as ntpttr | 18:37 | |
*** yangyapeng has joined #openstack-cinder | 18:40 | |
*** anshul has quit IRC | 18:44 | |
*** yangyapeng has quit IRC | 18:45 | |
*** merooney has quit IRC | 18:46 | |
*** merooney has joined #openstack-cinder | 18:47 | |
*** mriedem has quit IRC | 18:47 | |
*** lprice1 has quit IRC | 18:48 | |
*** mriedem has joined #openstack-cinder | 18:48 | |
*** vilobhmm11 has joined #openstack-cinder | 18:48 | |
*** lprice has joined #openstack-cinder | 18:48 | |
*** lprice1 has joined #openstack-cinder | 18:49 | |
*** lprice has quit IRC | 18:53 | |
*** kenhui has quit IRC | 18:55 | |
*** omolchanov has quit IRC | 18:56 | |
*** omolchanov has joined #openstack-cinder | 18:56 | |
*** akerr has quit IRC | 18:57 | |
*** akerr has joined #openstack-cinder | 18:58 | |
xyang1 | eharney: hi | 18:58 |
*** rlrossit has quit IRC | 19:00 | |
*** omolchanov has quit IRC | 19:05 | |
*** omolchanov1 has joined #openstack-cinder | 19:05 | |
*** crose has quit IRC | 19:07 | |
*** rlrossit has joined #openstack-cinder | 19:08 | |
*** rlrossit_ has joined #openstack-cinder | 19:09 | |
jgriffith | diablo_rojo: BTW, the detaching is in the "begin_detaching" method of volume.api as part of the conditional update | 19:09 |
*** martyturner has quit IRC | 19:09 | |
jgriffith | mc_nair: ^^ | 19:09 |
*** rlrossit has quit IRC | 19:10 | |
jgriffith | suppose it would be good to continue dulek 's enum work on the objects early in Newton | 19:10 |
mc_nair | jgriffith: gotcha. Interesting, so it looks like "detaching" is only part of the status, whereas "attaching" also gets pushed to attach_status and status | 19:12 |
jgriffith | mc_nair: soemtimes we get sloppy :) | 19:13 |
*** ociuhandu has quit IRC | 19:13 | |
*** eharney has joined #openstack-cinder | 19:13 | |
jgriffith | mc_nair: the attach/detach cod has gotten a bit wonky over the years I think. But easy enough to fix up | 19:14 |
mc_nair | jgriffith: we'll call it "code personality" :) | 19:14 |
jgriffith | mc_nair: ohhh... I like that | 19:14 |
mc_nair | jgriffith: sure, I don't think it's a big deal. Was just curious cause grepping was getting attaching but no detaching | 19:14 |
jgriffith | mc_nair: yeah, it does add confusion though :( | 19:14 |
jgriffith | mc_nair: I hope to finish my attach/detach refactor when N opens up. I"ll make a note to clean up the status stuff then as well | 19:15 |
smcginnis | Seems kind of odd that we don't keep that in an enum. | 19:15 |
mc_nair | smcginnis: yea, would be nice. Guessing that's why diablo_rojo was asking perhaps? | 19:16 |
smcginnis | It does make it pretty darn difficult to figure out what the valid states are as it is at the moment. | 19:17 |
*** timcl1 has quit IRC | 19:17 | |
kmartin | I guess this missing a few as well then http://developer.openstack.org/api-ref-blockstorage-v2.html#volumes-v2-volumes | 19:20 |
diablo_rojo | jgriffith: smcginnis Yeah I am working on the volume attach_status enum | 19:21 |
jgriffith | diablo_rojo: Ohh.. NICE!! | 19:21 |
diablo_rojo | jgriffith: I already have the snapshot status patch out there | 19:22 |
*** fthiagogv has quit IRC | 19:22 | |
*** apoorvad has quit IRC | 19:22 | |
jgriffith | diablo_rojo: Yeah, I think I already reviewed that | 19:22 |
jgriffith | diablo_rojo: prgress... progress :) | 19:22 |
diablo_rojo | jgriffith: I see that you reran solidfire on it and its happy | 19:23 |
jgriffith | diablo_rojo: oh.. stupid *&)( job failed again | 19:23 |
xyang1 | eharney: in this patch https://review.openstack.org/#/c/275408/ , check_for_setup_error is abstract method | 19:23 |
xyang1 | eharney: are you suggesting moving the check to lvm.py? | 19:24 |
jgriffith | diablo_rojo: ahh.. yes, I remember now | 19:24 |
jgriffith | diablo_rojo: I'm +2, just waiting to get ceph to run now | 19:24 |
diablo_rojo | jgriffith: Not sure why gate-temptest-dsvm-full-ceph keeps failing though. I tried a recheck a few days ago and it never happened. So I rechecked again today. | 19:24 |
jgriffith | diablo_rojo: we'll see if your recheck does the trick | 19:24 |
diablo_rojo | jgriffith: yep :) | 19:24 |
* diablo_rojo crosses fingers | 19:24 | |
eharney | xyang1: no... | 19:24 |
jgriffith | diablo_rojo: it's test_list_server_filters that fails, which is odd | 19:25 |
jgriffith | diablo_rojo: what i mean is it's in the server setup routines | 19:25 |
diablo_rojo | jgriffith: Yeah I thought so too. I know there was an issue a few days back with Ceph, but I dont recall the details | 19:25 |
eharney | xyang1: that verification could also be done in the manager | 19:26 |
*** apoorvad has joined #openstack-cinder | 19:26 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Clean up image tmp file if c-vol gets restarted https://review.openstack.org/225081 | 19:26 |
smcginnis | diablo_rojo: The ceph job was switched to non-voting now. | 19:26 |
xyang1 | eharney: where in manager? | 19:27 |
diablo_rojo | smcginnis: Cool. Hopefully it rechecks soon then. | 19:27 |
smcginnis | diablo_rojo: There is a bug in there and the version available on Ubuntu 14.04 is so old that it won't be patched. | 19:27 |
eharney | xyang1: around where the driver is loaded | 19:28 |
diablo_rojo | smcginnis: So it will never be voting now? | 19:28 |
xyang1 | eharney: _init_? | 19:29 |
smcginnis | diablo_rojo: I'm actually not sure what the plan is there. | 19:29 |
smcginnis | Hopefully it's resolved somehow. | 19:29 |
smcginnis | Ceph is so widely used. | 19:29 |
smcginnis | I don't really like that it's non-voting. | 19:29 |
smcginnis | Other than that it now keeps us from having to constantly recheck. | 19:29 |
diablo_rojo | smcginnis: Yeah I thought so. Makes me a tad nervous too. | 19:29 |
xyang1 | eharney: in manager init_host, it calls driver.check_for_setup_error | 19:30 |
*** ociuhandu has joined #openstack-cinder | 19:31 | |
*** sdague has quit IRC | 19:32 | |
*** Lee1092 has quit IRC | 19:32 | |
*** merooney has quit IRC | 19:32 | |
eharney | xyang1: not sure i understand the question... the point was it shouldn't be done in the driver's __init__ | 19:33 |
openstackgerrit | Merged openstack/cinder: Fix issue with Pure drivers delete_snapshot exception handling https://review.openstack.org/288255 | 19:33 |
xyang1 | eharney: I agree | 19:33 |
*** timcl has joined #openstack-cinder | 19:33 | |
xyang1 | eharney: right now check for setup error is an abstract method in BaseVD | 19:33 |
openstackgerrit | Tom Barron proposed openstack/cinder: Run py34 tests with plain 'tox' command https://review.openstack.org/288696 | 19:34 |
xyang1 | eharney: so the question is which check for setup error shoukd this check be moved to | 19:34 |
*** merooney has joined #openstack-cinder | 19:34 | |
eharney | xyang1: i don't think it's possible to move it to check_for_setup_error because it looks like most driver's don't call a super's version of that method anyway | 19:35 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/288344 | 19:35 |
xyang1 | eharney: then where should it go? | 19:35 |
*** omolchanov1 has quit IRC | 19:35 | |
*** omolchanov has joined #openstack-cinder | 19:35 | |
eharney | xyang1: i don't know. | 19:36 |
eharney | xyang1: we're trying to add error checking of a sort we didn't have before, it seems | 19:36 |
xyang1 | eharney: get rid of the check? is that better? | 19:37 |
eharney | xyang1: IMO this check shouldn't be necessary anyway, because <1.0 should work anyway and just reserve space | 19:39 |
eharney | xyang1: but i dunno | 19:39 |
eharney | i just didn't want it in driver __init__ | 19:40 |
xyang1 | eharney: I am fine removing the check. it is handled by the scheduler already | 19:40 |
mrrowden | eharney, I'm working on the patch that xyang1is talking about - we're already doing a warning in capacity_filter.py, is there a need for another warning in driver.py? | 19:40 |
jgriffith | jungleboyj: please don't +A this patch https://review.openstack.org/#/c/225081/9 | 19:40 |
jgriffith | jungleboyj: well, I guess you wouldn't actually... never mind | 19:41 |
*** martyturner has joined #openstack-cinder | 19:41 | |
jungleboyj | jgriffith: Wouldn't ... | 19:41 |
jgriffith | jungleboyj: yeah, sorry... wasn't thinking when I asked that :) | 19:42 |
*** vilobhmm111 has joined #openstack-cinder | 19:42 | |
openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Incorrect values in REST API Login call https://review.openstack.org/283257 | 19:42 |
jungleboyj | jgriffith: I removed my +2 to be safe. :-) | 19:42 |
jgriffith | jungleboyj: hehe :) | 19:42 |
jgriffith | jungleboyj: wasn't sure if you'd agree with my comments there or not | 19:43 |
xyang1 | mrrowden: so remove the change in _init_ in driver.py. keep the help string change. keep the scheduler change | 19:43 |
mrrowden | xyang1, and remove second paragraph from commit message? | 19:43 |
xyang1 | mrrowden: yes | 19:44 |
eharney | mrrowden: what does the check in the filter do? | 19:44 |
eharney | mrrowden: just ignores it right? | 19:45 |
xyang1 | mrrowden: add a line in the commit msg saying you fixed the help string | 19:45 |
*** vilobhmm11 has quit IRC | 19:45 | |
jungleboyj | jgriffith: They look like good questions. | 19:45 |
xyang1 | eharney: if thin is true but ratio is invalid, the host will be skipped | 19:46 |
mc_nair | I guess a question is - should a ratio < 1 be valid? | 19:47 |
mc_nair | think eharney and patrickeast have brought up | 19:47 |
xyang1 | mc_nair: no | 19:47 |
mrrowden | eharney, it gives a warning message about >= 1 then returns false | 19:47 |
eharney | ultimately i think >0 should be valid, but we don't have to fix that right now | 19:47 |
patrickeast | eharney: +1 | 19:48 |
eharney | there is no real reason to disallow setting 0.8 etc | 19:48 |
xyang1 | eharney: there is no real reason to support it either | 19:48 |
xyang1 | eharney: I have not heard a real use case for that | 19:48 |
eharney | xyang1: sure there is, you get more functionality via the same calculations | 19:48 |
eharney | but anyway | 19:49 |
xyang1 | eharney: I have not heard a real customer wanting that, unless if it is sfor some testing | 19:49 |
eharney | xyang1: then why do we have stuff like "reserved_percentage"? | 19:50 |
mc_nair | eharney: am I missing something or do we just need to remove the "host_state.max_over_subscription_ratio >= 1" part of the check to fix this for < 1 also? | 19:50 |
xyang1 | eharney: anyway, we don't support that earlier, so we are not losing anything | 19:50 |
xyang1 | eharney: reserved percentage has a specific parameter to achieve it | 19:51 |
eharney | mc_nair: that should be the case ideally, but i'd need to look. but then you need a check for >0 anyway, so that's probably for later | 19:52 |
mc_nair | eharney: Good point. Doesn't help with the question of *should* we, was just curious on that | 19:53 |
*** martyturner has quit IRC | 19:54 | |
xyang1 | mrrowden: I added my comments in the patch | 19:54 |
patrickeast | jungleboyj: hey, if you get a chance can you peek at https://review.openstack.org/#/c/281625/ again? lost your +2 with a rebase/merge conflict | 19:54 |
jungleboyj | patrickeast: Sure. | 19:55 |
patrickeast | jungleboyj: sweet, thanks! | 19:55 |
eharney | i think just removing those pieces from driver.py is sensible | 19:55 |
mrrowden | xyang1, thanks, new patch coming up soon | 19:55 |
xyang1 | mrrowden: thanks | 19:55 |
openstackgerrit | Michael Rowden proposed openstack/cinder: Correcting thin provisioning behavior https://review.openstack.org/275408 | 19:57 |
*** msbrogli has joined #openstack-cinder | 19:59 | |
mrrowden | thanks everyone for your input | 20:00 |
*** mrrowden has quit IRC | 20:00 | |
msbrogli | Hey! I have one node running the VMs and one storage. But my storage has 2 IP addresses. The volumes are mounted using “the wrong IP”. I tried settings the config iscsi_ip_address but it has not worked. Can anyone help me? | 20:00 |
msbrogli | I’ve set the iscsi_ip_address, restarted cinder-api cinder-scheduler cinder-volume. Then, I restarted the VM using the nova client. Is it enough? | 20:02 |
*** sheel has quit IRC | 20:07 | |
*** timcl has quit IRC | 20:11 | |
*** vilobhmm111 has quit IRC | 20:13 | |
jungleboyj | patrickeast: Done. | 20:13 |
*** vilobhmm11 has joined #openstack-cinder | 20:13 | |
*** merooney has quit IRC | 20:19 | |
*** merooney has joined #openstack-cinder | 20:23 | |
*** Suyash has quit IRC | 20:23 | |
*** Suyash has joined #openstack-cinder | 20:24 | |
*** diablo_rojo has quit IRC | 20:27 | |
*** lpetrut has joined #openstack-cinder | 20:27 | |
*** salv-orlando has quit IRC | 20:33 | |
*** salv-orlando has joined #openstack-cinder | 20:33 | |
*** timcl has joined #openstack-cinder | 20:35 | |
*** mmandel has quit IRC | 20:38 | |
*** ericksonsantos has quit IRC | 20:45 | |
*** bardia has joined #openstack-cinder | 20:49 | |
*** ivase has quit IRC | 20:50 | |
openstackgerrit | Mitsuhiro Tanino proposed openstack/python-cinderclient: Don't reset volume status when resetting migration status https://review.openstack.org/288729 | 20:53 |
openstackgerrit | Jay Bryant proposed openstack/cinder: Add CGSnapshotStatus enum field https://review.openstack.org/261068 | 20:55 |
*** merooney has quit IRC | 20:55 | |
openstackgerrit | Mitsuhiro Tanino proposed openstack/python-cinderclient: Don't reset volume status when resetting migration status https://review.openstack.org/288732 | 20:57 |
*** delattec has quit IRC | 20:58 | |
Swanson | jgriffith, Any chance this https://review.openstack.org/#/c/286256/ can get a shove through the gate? | 20:58 |
mriedem | are we gauranteed to have target_lun in connection_info['data'] for scsi attachments? | 20:59 |
smcginnis | mriedem: I'm not aware of any SCSI backends that do not set that. | 21:01 |
smcginnis | Pretty sure it's required. | 21:01 |
*** kenhui has joined #openstack-cinder | 21:02 | |
*** kenhui1 has joined #openstack-cinder | 21:02 | |
*** kenhui has quit IRC | 21:02 | |
*** timcl has quit IRC | 21:02 | |
hemna | mriedem, I'm pretty sure that's required afaik | 21:02 |
hemna | mriedem, some backends may always have a LUN of 0 | 21:03 |
mriedem | ok | 21:03 |
hemna | mriedem, but their IQN typically changes in that case | 21:03 |
mriedem | was just looking at https://review.openstack.org/#/c/287055/5/nova/virt/vmwareapi/volumeops.py | 21:03 |
hemna | yah target_lun should be set afaik | 21:04 |
*** rhagarty_ has quit IRC | 21:07 | |
*** rhagarty_ has joined #openstack-cinder | 21:07 | |
mriedem | ok | 21:07 |
*** jgregor has quit IRC | 21:07 | |
*** ivase_ has joined #openstack-cinder | 21:07 | |
*** JoseMello has quit IRC | 21:08 | |
*** laughterwym has joined #openstack-cinder | 21:09 | |
*** apoorvad has quit IRC | 21:10 | |
mriedem | jwcroppe: do you have a fix for this else i'll push one up quick https://bugs.launchpad.net/nova/+bug/1553319 | 21:14 |
openstack | Launchpad bug 1553319 in OpenStack Compute (nova) "When CPU metric collection fails, stack trace not in nova logs" [Undecided,New] - Assigned to Joe Cropper (jwcroppe) | 21:14 |
mriedem | oops | 21:14 |
*** laughterwym has quit IRC | 21:15 | |
*** ivase_ is now known as ivase | 21:16 | |
*** raildo is now known as raildo-afk | 21:17 | |
*** edtubill has quit IRC | 21:20 | |
*** baumann has left #openstack-cinder | 21:33 | |
*** akerr has quit IRC | 21:34 | |
*** apoorvad has joined #openstack-cinder | 21:36 | |
*** porrua has quit IRC | 21:39 | |
*** jgregor has joined #openstack-cinder | 21:42 | |
*** gouthamr has quit IRC | 21:43 | |
msbrogli | I changed the provider_location field in the volumes table. How to do I propagate the changes to the VMs? | 21:45 |
*** salv-orlando has quit IRC | 21:47 | |
*** knikolla has quit IRC | 21:48 | |
*** salv-orlando has joined #openstack-cinder | 21:48 | |
*** mrrowden has joined #openstack-cinder | 21:50 | |
*** salv-orlando has quit IRC | 21:51 | |
*** kfarr has joined #openstack-cinder | 21:51 | |
*** salv-orlando has joined #openstack-cinder | 21:52 | |
mrrowden | xyang1 eharney can please I get your input on patrickeast comments for https://review.openstack.org/#/c/275408/12 to see if another patch is in order? | 21:56 |
xyang1 | mrrowden: I am about to add my comments | 21:59 |
jgriffith | hemna: ping | 22:05 |
hemna | hey | 22:05 |
jgriffith | hemna: hey... your requests patch doesn't seem to be working for some reason | 22:06 |
hemna | crud | 22:06 |
jgriffith | hemna: but it's weird... I can't get the config setting to work which I don't get | 22:06 |
hemna | are you getting errors reading the conf entry ? | 22:08 |
jgriffith | lemme look | 22:08 |
hemna | I think the conf entry needs to be in the driver section ? | 22:08 |
jgriffith | hemna: oh, because it's in the manager :) | 22:09 |
jgriffith | hemna: that makes sense :( | 22:09 |
hemna | that way you can disable it for 1 driver, and not for others. | 22:09 |
hemna | I thought that was safer, than globally disabling it for all | 22:09 |
hemna | dunno | 22:09 |
jgriffith | hemna: yeah, that's it :) | 22:09 |
hemna | :) | 22:09 |
ntpttr | sorry if I'm late to the party, but is something up with the gate? Looks like only 5 patches are in the 'check' stage and my patches aren't getting run http://status.openstack.org/zuul/ | 22:09 |
openstackgerrit | Merged openstack/cinder: Add backend id to Pure Volume Driver trace logs https://review.openstack.org/288190 | 22:10 |
*** dustins has quit IRC | 22:10 | |
xyang1 | jgriffith: hemna when you two have time, can you take a look of this replication patch again? https://review.openstack.org/#/c/285139/ | 22:11 |
hemna | xyang1, sure | 22:11 |
xyang1 | thanks | 22:11 |
*** e0ne has joined #openstack-cinder | 22:11 | |
*** laughterwym has joined #openstack-cinder | 22:13 | |
hemna | xyang1, isn't backend_id a single entry ? emc_vnx_cli.py:3972 | 22:15 |
xyang1 | hemna: yes, it is fixed, right? | 22:15 |
*** jwcroppe has quit IRC | 22:16 | |
hemna | msg = (_('Invalid secondary_backend_id specified. ' | 22:16 |
hemna | 'Choose one entry from: %s.') % backend_id) | 22:16 |
hemna | I'm confused | 22:16 |
xyang1 | hemna: why? | 22:16 |
hemna | the wording says choose one entry from: backend_id | 22:16 |
jgriffith | hemna: what's that coming from? | 22:16 |
xyang1 | hemna: oh, the text | 22:16 |
*** jwcroppe has joined #openstack-cinder | 22:16 | |
hemna | if backend_id is a single entry..... | 22:16 |
xyang1 | hemna: got it | 22:16 |
hemna | jgriffith, https://review.openstack.org/#/c/285139/6/cinder/volume/drivers/emc/emc_vnx_cli.py | 22:16 |
jgriffith | hemna: it's a list | 22:16 |
patrickeast | hemna: each replication target has one, you specify one to failover too, so at times you are dealing with more than one | 22:17 |
hemna | self.configuration.replication_device[0]['backend_id']) | 22:17 |
hemna | that's a list ? | 22:17 |
hemna | that smells like a single entry to me...but maybe I'm confused. | 22:17 |
patrickeast | oo | 22:18 |
patrickeast | i see | 22:18 |
hemna | line 3968 -> 3972 | 22:18 |
*** jgregor has quit IRC | 22:18 | |
patrickeast | yea that code is just looking at the first replication target | 22:18 |
jgriffith | hemna: let me catch up here :) | 22:18 |
xyang1 | hemna: we only support 1 | 22:18 |
*** diablo_rojo has joined #openstack-cinder | 22:18 | |
hemna | my question is, what is backend_id in that msg = (_( ... line ? | 22:18 |
*** jungleboyj has quit IRC | 22:18 | |
hemna | I think it's a single entry, not a list | 22:18 |
hemna | which makes the wording in the message confusing. | 22:19 |
patrickeast | yep | 22:19 |
jgriffith | hemna: yeah | 22:19 |
hemna | ok, phew. | 22:19 |
jgriffith | hemna: i see what you're saying now | 22:19 |
xyang1 | hemna: the message is confusing | 22:19 |
xyang1 | hemna: he removed [ ], but did not change text | 22:19 |
xyang1 | hemna: are we on the same page? | 22:19 |
hemna | xyang1, ok thanks. I thought I was nuts there for a second hehe | 22:20 |
hemna | xyang1, yup, I think so now. :) | 22:20 |
xyang1 | hemna: sorry, I did not pay close attention to the wording | 22:20 |
hemna | nah it's cool :) | 22:20 |
xyang1 | good:) | 22:20 |
hemna | well, -1 cool, but cool | 22:20 |
hemna | :P | 22:20 |
jgriffith | hemna: xyang1 just don't forget that self.configuration.replication_device *can* be a list though | 22:21 |
xyang1 | jgriffith: sure, vnx only supports 1 device, is it ok | 22:21 |
hemna | jgriffith, so at worst, they are hard coding looking for the 1st entry always then | 22:21 |
hemna | in this case | 22:21 |
jgriffith | hemna: yeah, and based on xyang1 's statement that they only support '1' that's cool | 22:22 |
jgriffith | and I shouldn't say *can* I should say "it is" a list | 22:22 |
jgriffith | but never mind | 22:22 |
jgriffith | I'm just talking out loud here ;) | 22:22 |
xyang1 | jgriffith: add your comments there if you have concerns | 22:22 |
jgriffith | xyang1: no no, I don't have concerns | 22:23 |
*** gouthamr has joined #openstack-cinder | 22:23 | |
jgriffith | xyang1: I was just pointing out that even though your backend only supports one, that config option IS a list so using [0] is appropriate | 22:23 |
jgriffith | xyang1: but that's aside from the comment about the wording :) | 22:23 |
xyang1 | jgriffith: ok:) | 22:23 |
*** akshai has quit IRC | 22:25 | |
*** tpsilva has quit IRC | 22:27 | |
*** lascii is now known as alaski | 22:28 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/288344 | 22:29 |
*** [1]Thelo has joined #openstack-cinder | 22:29 | |
*** kfarr has quit IRC | 22:31 | |
*** Thelo has quit IRC | 22:31 | |
*** [1]Thelo is now known as Thelo | 22:31 | |
*** rlrossit_ has quit IRC | 22:34 | |
*** arch-nemesis has quit IRC | 22:34 | |
*** chris_morrell has quit IRC | 22:37 | |
*** ircuser-1 has quit IRC | 22:39 | |
openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Incorrect values in REST API Login call https://review.openstack.org/283257 | 22:39 |
*** ircuser-1 has joined #openstack-cinder | 22:43 | |
*** laughterwym has quit IRC | 22:43 | |
*** laughterwym has joined #openstack-cinder | 22:44 | |
*** sheel has joined #openstack-cinder | 22:45 | |
*** mriedem has quit IRC | 22:46 | |
*** erlon has quit IRC | 22:46 | |
*** laughterwym has quit IRC | 22:48 | |
*** arch-nemesis has joined #openstack-cinder | 22:51 | |
*** e0ne has quit IRC | 22:55 | |
*** msbrogli has quit IRC | 22:55 | |
patrickeast | hey so for the replication v2.1 implementations in drivers and release notes... how do we want to handle it? | 22:55 |
patrickeast | theres a mix of strategies right now, which might make for confusing release notes | 22:56 |
patrickeast | some drivers have ones that say implement v2 and implement v2.1 | 22:56 |
patrickeast | some (like mine) only have implement v2 | 22:56 |
patrickeast | and i think the dell one removed the v2 and only have a note for v2.1 | 22:56 |
jgriffith | patrickeast: update your to say "cheesecake" :) | 22:56 |
patrickeast | jgriffith: of course! | 22:56 |
jgriffith | patrickeast: my opinion is that it should be 2.1 | 22:56 |
jgriffith | not 2 | 22:56 |
*** haplo37 has quit IRC | 22:57 | |
patrickeast | jgriffith: yea im kinda thinking any of the leftover v2 release notes should be removed | 22:57 |
jgriffith | patrickeast: I would agree for sure | 22:57 |
patrickeast | Swanson: smcginnis: aorourke: ^^ | 22:57 |
patrickeast | i was going to put up a patch to fix the pure one, but can fixup the others all in one go | 22:58 |
Swanson | patrickeast, v2.1 | 22:58 |
Swanson | only | 22:58 |
patrickeast | any the in-flight ones can be fixed in review | 22:58 |
patrickeast | Swanson: cool, yea so far thats the favorite | 22:58 |
patrickeast | s/any/and/ | 22:58 |
Swanson | cause I don't support v2 anymore and I think it is important to make sure we make that distinction. | 22:58 |
aorourke | 0. | 23:00 |
*** ivase has quit IRC | 23:00 | |
aorourke | oops. I added a release note for supporting v2.1 but did not remove the old ones | 23:00 |
patrickeast | aorourke: yea thats what i did | 23:00 |
patrickeast | aorourke: i can remove it with mine if you want | 23:00 |
aorourke | patrickeast, that works for me | 23:01 |
Swanson | v2 didn't comeout until early liberty, right? So no release ever supported v2, right? | 23:01 |
aorourke | patrickeast, the LeftHand and 3PAR ones need to be removed | 23:01 |
patrickeast | Swanson: yep, it wasn't ever really supported officially by anyone afaik | 23:01 |
patrickeast | put a note on the reviews that are up now (I think I got them all..) | 23:03 |
*** lpetrut has quit IRC | 23:03 | |
*** edmondsw has quit IRC | 23:05 | |
openstackgerrit | Patrick East proposed openstack/cinder: Fixup release notes for v2 -> v2.1 replication impls https://review.openstack.org/288787 | 23:06 |
patrickeast | aorourke: ^ | 23:06 |
openstackgerrit | Patrick East proposed openstack/cinder: Fixup release notes for v2 -> v2.1 replication impls https://review.openstack.org/288787 | 23:12 |
patrickeast | whew, almost forgot to change it to say cheesecake | 23:12 |
*** kenhui1 has quit IRC | 23:13 | |
*** e0ne has joined #openstack-cinder | 23:15 | |
xyang1 | patrickeast: why remove release note? | 23:17 |
patrickeast | xyang1: the driver wont support v2 anymore | 23:17 |
xyang1 | patrickeast: driver has a new feature, still need a release note | 23:17 |
patrickeast | xyang1: so a release note saying support for v2 is kinda confusing, right? | 23:17 |
patrickeast | xyang1: it would be supporting v2.1 | 23:18 |
xyang1 | patrickeast: that is what it says | 23:18 |
xyang1 | patrickeast: ? | 23:18 |
patrickeast | xyang1: right so https://review.openstack.org/#/c/285139/6/releasenotes/notes/vnx-replication-v2.1-4d89935547183cc9.yaml says the driver added support for v2.1 in mitaka, and https://github.com/openstack/cinder/blob/master/releasenotes/notes/vnx-replication-v2-2afc4ac0c2ecfa60.yaml is going to say it also suports v2 | 23:19 |
patrickeast | xyang1: for the mitaka release notes | 23:19 |
patrickeast | xyang1: so keep the new one, or modify the original | 23:19 |
patrickeast | xyang1: does that make sense? | 23:19 |
xyang1 | patrickeast: oh, I see | 23:20 |
xyang1 | patrickeast: we should remove the old one | 23:20 |
xyang1 | patrickeast: the name is wrong too | 23:20 |
patrickeast | xyang1: cool, sounds good | 23:21 |
xyang1 | patrickeast: thanks | 23:21 |
patrickeast | np | 23:21 |
*** mrrowden has quit IRC | 23:23 | |
*** msbrogli has joined #openstack-cinder | 23:26 | |
*** xyang1 has quit IRC | 23:29 | |
*** Trident has quit IRC | 23:33 | |
*** msbrogli has quit IRC | 23:35 | |
openstackgerrit | Vasanthi Thirumalai proposed openstack/cinder: Violin Memory ISCSI storage for 7000 series AFA https://review.openstack.org/269796 | 23:35 |
*** msbrogli has joined #openstack-cinder | 23:38 | |
*** daneyon has joined #openstack-cinder | 23:49 | |
*** diablo_rojo has quit IRC | 23:51 | |
*** daneyon_ has joined #openstack-cinder | 23:52 | |
*** salv-orl_ has joined #openstack-cinder | 23:52 | |
*** daneyon has quit IRC | 23:55 | |
*** salv-orlando has quit IRC | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!