| opendevreview | Simon Dodsley proposed openstack/cinder master: PureStorage FlashArray: Add active/active replication https://review.opendev.org/c/openstack/cinder/+/829473 | 04:49 |
|---|---|---|
| opendevreview | Simon Dodsley proposed openstack/cinder master: PureStorage FlashArray: Add active/active replication https://review.opendev.org/c/openstack/cinder/+/829473 | 04:51 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status https://review.opendev.org/c/openstack/cinder/+/773985 | 07:37 |
| *** dwhite449 is now known as dwhite44 | 07:38 | |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for backup os-reset_status https://review.opendev.org/c/openstack/cinder/+/778193 | 07:38 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for group-snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804757 | 07:38 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for group os-reset_status https://review.opendev.org/c/openstack/cinder/+/804735 | 07:38 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804035 | 07:39 |
| yuval | xc I uploaded a question in this: https://review.opendev.org/c/openstack/os-brick/+/823654 if someone can advice that would be helpful | 09:53 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status https://review.opendev.org/c/openstack/cinder/+/773985 | 10:25 |
| opendevreview | yuval proposed openstack/os-brick master: Failure to generate hostnqn in case missing "show-hostnqn" sub-command https://review.opendev.org/c/openstack/os-brick/+/823654 | 10:43 |
| opendevreview | Oleg proposed openstack/cinder master: Add NVMe/TCP support to Dell EMC PowerStore driver https://review.opendev.org/c/openstack/cinder/+/819149 | 10:46 |
| *** dviroel|out is now known as dviroel | 11:21 | |
| raghavendrat | Hi, has anyone installed Ussuri devstack on Ubuntu 20 ? | 12:20 |
| raghavendrat | I am asking this because, we are facing some issue with Ussuri on Ubuntu 20. | 12:21 |
| raghavendrat | Note: Ussuri on Ubuntu 18 worked fine for us. | 12:21 |
| raghavendrat | Thanks. | 12:21 |
| opendevreview | Nahim Alves de Souza proposed openstack/cinder master: NetApp ONTAP: Fixed get_ontap_version https://review.opendev.org/c/openstack/cinder/+/829529 | 12:26 |
| *** josecastroleon__ is now known as josecastroleon | 12:36 | |
| yuval | Hey question: who is responsible to modprobe modules on the system -> does openstack count on the operator to do it beforehand? | 12:41 |
| yuval | for example modprobe nvme | 12:41 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: PureStorage FlashArray: Add active/active replication https://review.opendev.org/c/openstack/cinder/+/829473 | 12:50 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status https://review.opendev.org/c/openstack/cinder/+/773985 | 12:58 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status https://review.opendev.org/c/openstack/cinder/+/773985 | 13:17 |
| opendevreview | yuval proposed openstack/os-brick master: Failure to generate hostnqn in case missing "show-hostnqn" sub-command https://review.opendev.org/c/openstack/os-brick/+/823654 | 13:19 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status https://review.opendev.org/c/openstack/cinder/+/773985 | 13:20 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for group os-reset_status https://review.opendev.org/c/openstack/cinder/+/804735 | 13:21 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for backup os-reset_status https://review.opendev.org/c/openstack/cinder/+/778193 | 13:21 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for group-snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804757 | 13:21 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804035 | 13:21 |
| raghavendrat | Following patches were reviewed by me recently: | 13:40 |
| raghavendrat | https://review.opendev.org/c/openstack/os-brick/+/827496 | 13:40 |
| raghavendrat | https://review.opendev.org/c/openstack/cinder/+/768458 | 13:40 |
| raghavendrat | https://review.opendev.org/c/openstack/cinder/+/828060 | 13:40 |
| yuval | raghavendrat: which patches you want review on? | 13:46 |
| raghavendrat | oh. i think some copy paste error my side. i will paste again ... | 13:51 |
| raghavendrat | hi, it would be great if i can get review on below patch: | 13:51 |
| raghavendrat | https://review.opendev.org/c/openstack/cinder/+/824911 | 13:51 |
| opendevreview | Francesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 13:59 |
| rosmaita | Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:59 |
| rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval ^^ | 13:59 |
| *** whoami-rajat__ is now known as whoami-rajat | 14:01 | |
| opendevreview | Francesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 14:02 |
| geguileo | rosmaita: thanks | 14:02 |
| *** enriquetaso_ is now known as enriquetaso | 15:01 | |
| enriquetaso | #startmeeting cinder_bs | 15:01 |
| opendevmeet | Meeting started Wed Feb 16 15:01:54 2022 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:01 |
| enriquetaso | Hey, I just wanna say that there are a bunch of new bugs reported during my holiday, so I'm still processing them. Also, most of the 'incomplete' or 'waiting for more info bugs' were updated with info so I'm trying to check them as well.. | 15:02 |
| enriquetaso | I guess you'll receive the report email at the end of the day with all the bugs. I have one bug for today's meeting: | 15:02 |
| rosmaita | o/ | 15:03 |
| whoami-rajat | hi | 15:03 |
| enriquetaso | #topic [RBD] Amend the formula of retrieving total_capacity for a rbd pool | 15:03 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1960206 | 15:03 |
| enriquetaso | I think this is a bug report for quotas so geguileo might be interesting. | 15:03 |
| enriquetaso | Summary: The reporter points out that the value of the RBD total_capacity field is not being correctly calculated. Because it changes depending on the ceph version being used. | 15:03 |
| yuval | sorry, cinder_bs stand for? | 15:04 |
| rosmaita | cinder bug squad | 15:04 |
| yuval | ok... not what I had in mind | 15:04 |
| geguileo | lol | 15:04 |
| rosmaita | :) | 15:04 |
| yuval | lol | 15:04 |
| geguileo | you though it was BS ;-) | 15:04 |
| enriquetaso | It's a short irc meeting to discuss the bug reported this week | 15:04 |
| enriquetaso | bugs* | 15:04 |
| rosmaita | yes, i was in a sophomoric mood when i proposed that name for the meeting | 15:04 |
| yuval | :) | 15:05 |
| yuval | first I thought it was just a joke | 15:05 |
| yuval | sorry, please continue | 15:05 |
| geguileo | enriquetaso: that has nothing to do with quotas ;-) | 15:05 |
| enriquetaso | Cool, my bad | 15:06 |
| geguileo | that's how the RBD driver is calculating the stats of the backend to report to the scheduler | 15:06 |
| enriquetaso | is it a problem that the calculation changes between ceph versions? | 15:06 |
| enriquetaso | I should probably reproduce the bug first, i haven't | 15:07 |
| enriquetaso | More info: | 15:07 |
| enriquetaso | Currently, the value of free_capacity for a rbd pool is "max_avail" retrieved from the command 'ceph df -f json' when quota limit is not set for this pool. And in that scenario the value of total_capacity is the sum of free_capacity and 'bytes_used' also retrieved from the command 'ceph df -f json' in the case of dynamic total capacity. | 15:07 |
| enriquetaso | However, there is a considerable difference for the command 'ceph df -f json' for different ceph versions. For example, the output of that command of the version 12.2.11 of ceph lacks 'STORED' field and the formula of its '%USED' filed is relevant to 'USED' field while the output of that command of the version 12.2.13 possesses 'STORED' field and the formula of its '%USED' filed is relevant to 'STORED' field. | 15:07 |
| enriquetaso | Finally I think the LP needs a name change. | 15:07 |
| enriquetaso | OK.. feel free to check out the bug if you have time. Moving on.. | 15:11 |
| enriquetaso | #topic open discussion | 15:11 |
| rosmaita | ok, that bug doesn't look critical unless you are running close to capacity | 15:11 |
| rosmaita | enriquetaso: nice to see you back, hope you had a good vacation | 15:11 |
| enriquetaso | True.. I'll move it to 'low' and remove the quota tag | 15:11 |
| enriquetaso | thanks rosmaita! | 15:12 |
| enriquetaso | guess we don't have bugs for the open discussion | 15:13 |
| enriquetaso | Thanks for today's meeting! | 15:14 |
| rosmaita | thanks enriquetaso | 15:15 |
| enriquetaso | #endmeeting | 15:16 |
| opendevmeet | Meeting ended Wed Feb 16 15:16:18 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:16 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-02-16-15.01.html | 15:16 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-02-16-15.01.txt | 15:16 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-02-16-15.01.log.html | 15:16 |
| opendevreview | Jay Bryant proposed openstack/cinder master: Fix typo in release note. https://review.opendev.org/c/openstack/cinder/+/829560 | 15:21 |
| *** dviroel is now known as dviroel|lunch | 15:31 | |
| yuval | e0ne: yes, but t require NVMe SSD doesn't it? - if you are using a vm you can emulate nvme device for testing | 15:38 |
| yuval | didnt had time to write it in the meeting - so... | 15:38 |
| opendevreview | yuval proposed openstack/os-brick master: Failure to generate hostnqn in case missing "show-hostnqn" sub-command https://review.opendev.org/c/openstack/os-brick/+/823654 | 15:49 |
| yuval | had to rebase it to check it^ on our CI - I think the gui rebase might fail without warning | 15:50 |
| rosmaita | yuval: what time is it where you are? | 15:55 |
| e0ne | yuval: thanks for the clarification | 15:55 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: RBD: Fix total_capacity https://review.opendev.org/c/openstack/cinder/+/829565 | 16:09 |
| TusharTgite | rosmaita: will you take alook at my comment https://review.opendev.org/c/openstack/cinder/+/773985/27..32 | 16:11 |
| rosmaita | TusharTgite: ok | 16:11 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: RBD: Fix total_capacity https://review.opendev.org/c/openstack/cinder/+/829565 | 16:11 |
| TusharTgite | rosmaita: and the tox-py36 is passed in my env but fails in zuul | 16:15 |
| rosmaita | TusharTgite: ok, i will take a look | 16:16 |
| yuval | rosmaita I am at utc+2 | 16:35 |
| rosmaita | yuval: thanks ... left some comments on your patch | 16:36 |
| yuval | Thanks, I will handle them soon as possible | 16:37 |
| rosmaita | yuval: thanks, i think after you address them, i will have no more comments | 16:38 |
| *** dviroel|lunch is now known as dviroel | 16:39 | |
| rosmaita | TusharTgite: i think i see what happened there, left a comment on your patch | 16:45 |
| TusharTgite | rosmaita: see my comment https://review.opendev.org/c/openstack/cinder/+/773985 | 16:52 |
| dswebb | @here. would be good to get someone to do a once over on https://review.opendev.org/c/openstack/cinder/+/820027 to get some feedback. I believe it's at a state now where it's hopefully suitable for submission. The one part I'm not sure of is how the third part CI failures work, especially since I didn't touch anything that they should have been using? | 17:01 |
| rosmaita | dswebb: unfortunately, the third-party CIs can be unreliable ... the only one relevant to your change would be ceph, and since that's a community driver, the CI is done by zuul | 17:04 |
| rosmaita | so as long as you have +1 from zuul on your patch , you should be ok | 17:04 |
| dswebb | ah ok, thanks for that. | 17:07 |
| rosmaita | TusharTgite: left a comment ... i think the problem is with the tests, but i won't have time to look until later today | 17:09 |
| rosmaita | the tests are policy tests, and your patch changes when the policy check is done (we do it earlier now), which could be one reason; another is that now you are returning 400s whereas before the calls were successful; so the problem may have to do with what's being mocked in the policy tests | 17:11 |
| TusharTgite | rosmaita: see my comment https://review.opendev.org/c/openstack/cinder/+/773985 i'll restore it as per your suggestion and will take a look at test after i think " def test_reset_attached_status(self):" this is the test which going to cause problems | 17:15 |
| rosmaita | TusharTgite: thanks, i'll check back later | 17:16 |
| TusharTgite | rosmaita: thanks only rush i have is this is the base patch and have 4 child patch for the feature | 17:19 |
| rosmaita | TusharTgite: i think once we get the base patch figured out, the others will move quickly | 17:19 |
| TusharTgite | rosmaita: yes | 17:20 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status https://review.opendev.org/c/openstack/cinder/+/773985 | 17:23 |
| opendevreview | Brian Rosmaita proposed openstack/os-brick master: Add "known issues" note to yoga os-brick release https://review.opendev.org/c/openstack/os-brick/+/829588 | 17:51 |
| rosmaita | geguileo: ^^ when you have a few minutes | 17:52 |
| opendevreview | Merged openstack/cinder master: Fix typo in release note. https://review.opendev.org/c/openstack/cinder/+/829560 | 18:43 |
| geguileo | rosmaita: sorry, I got distracted | 18:55 |
| geguileo | rosmaita: I have finally created the LP bug: https://bugs.launchpad.net/os-brick/+bug/1961102 | 18:55 |
| rosmaita | geguileo: thanks, will update to include the bug | 19:09 |
| *** hemna0 is now known as hemna | 19:10 | |
| opendevreview | Brian Rosmaita proposed openstack/os-brick master: Add "known issues" note to yoga os-brick release https://review.opendev.org/c/openstack/os-brick/+/829588 | 19:32 |
| jungleboyj | rosmaita: Looks good to me. | 19:44 |
| rosmaita | jungleboyj: ty | 19:44 |
| *** _pewp__ is now known as _pewp_ | 19:47 | |
| *** hemna4 is now known as hemna | 19:49 | |
| *** dviroel is now known as dviroel|out | 21:26 | |
| opendevreview | Merged openstack/cinder master: Update 'cinder-specs-core' description https://review.opendev.org/c/openstack/cinder/+/829129 | 23:12 |
| opendevreview | yuval proposed openstack/os-brick master: Failure to generate hostnqn in case missing "show-hostnqn" sub-command https://review.opendev.org/c/openstack/os-brick/+/823654 | 23:30 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!