| rosmaita | I could use some feedback on cycle highlights (proofreading and/or suggestions for stuff that should be included): https://review.opendev.org/c/openstack/releases/+/830917 | 00:03 |
|---|---|---|
| rosmaita | e0ne eharney geguileo hemna jungleboyj smcginnis whoami-rajat enriquetaso anyone-else-whos-interested ^^ | 00:03 |
| enriquetaso | sure | 00:14 |
| opendevreview | Merged openstack/cinder master: Followup: Address review comments on re-image patch https://review.opendev.org/c/openstack/cinder/+/830879 | 03:50 |
| *** bhagyashris is now known as bhagyashris|ruck | 04:38 | |
| 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 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: Support AIX as host OS type https://review.opendev.org/c/openstack/cinder/+/828061 | 07:43 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: Support AIX as host OS type https://review.opendev.org/c/openstack/cinder/+/828061 | 07:59 |
| hgy | Hello, Could I ask a question? | 08:23 |
| hgy | Could cinder can assume a host when create volumes | 08:23 |
| hgy | I want to create volumes in assumed hosts | 08:24 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: Add Target Port Assignment for VSP Driver https://review.opendev.org/c/openstack/cinder/+/828060 | 09:32 |
| *** bhagyashris_ is now known as bhagyashris|ruck | 09:55 | |
| opendevreview | OpenStack Release Bot proposed openstack/python-brick-cinderclient-ext stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/830953 | 09:56 |
| opendevreview | OpenStack Release Bot proposed openstack/python-brick-cinderclient-ext stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/830954 | 09:56 |
| opendevreview | OpenStack Release Bot proposed openstack/python-brick-cinderclient-ext master: Update master for stable/yoga https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/830955 | 09:56 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: add GAD volume support https://review.opendev.org/c/openstack/cinder/+/796170 | 10:06 |
| *** bhagyashris_ is now known as bhagyashris|ruck | 10:44 | |
| opendevreview | Merged openstack/python-brick-cinderclient-ext stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/830953 | 11:02 |
| opendevreview | Francesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 11:27 |
| opendevreview | OpenStack Release Bot proposed openstack/python-cinderclient stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/python-cinderclient/+/830979 | 12:43 |
| opendevreview | OpenStack Release Bot proposed openstack/python-cinderclient stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/python-cinderclient/+/830980 | 12:43 |
| opendevreview | OpenStack Release Bot proposed openstack/python-cinderclient master: Update master for stable/yoga https://review.opendev.org/c/openstack/python-cinderclient/+/830981 | 12:43 |
| opendevreview | Walt proposed openstack/cinder master: Bugfix: Account for consumed space better https://review.opendev.org/c/openstack/cinder/+/826510 | 12:58 |
| TusharTgite_ | rosmaita: done the changes https://review.opendev.org/c/openstack/cinder/+/773985 | 13:08 |
| rosmaita | TusharTgite_: ack | 13:13 |
| raghavendrat | hi rosmaita: addressed comment in https://review.opendev.org/c/openstack/cinder/+/786054 | 13:15 |
| raghavendrat | whenever you get time, please have a look. Appreciate your feedback | 13:15 |
| rosmaita | raghavendrat: ack | 13:16 |
| raghavendrat | thanks | 13:16 |
| rosmaita | could use some software engineering advice: https://review.opendev.org/c/openstack/cinder/+/773985/38/cinder/api/contrib/admin_actions.py#134 | 13:18 |
| rosmaita | e0ne eharney geguileo hemna jungleboyj smcginnis whoami-rajat enriquetaso ^^ | 13:18 |
| rosmaita | TusharTgite_: patch looks good, I want to get some feedback on whether I gave you good advice there or not | 13:20 |
| rosmaita | raghavendrat: please work on getting your individual logs displayable in a web browser, this downloading a tarball stuff is a big time waster on the reviewing side | 13:21 |
| rosmaita | raghavendrat: looks good, left a comment about a followup, but the current patch is ready to go | 13:30 |
| opendevreview | Merged openstack/python-brick-cinderclient-ext stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/830954 | 13:46 |
| opendevreview | Merged openstack/python-cinderclient stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/python-cinderclient/+/830979 | 13:49 |
| opendevreview | Merged openstack/python-cinderclient stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/python-cinderclient/+/830980 | 13:52 |
| raghavendrat | Thanks a lot rosmaita: | 14:12 |
| * enriquetaso looking | 14:12 | |
| raghavendrat | Hi reviewers, ... regarding https://review.opendev.org/c/openstack/cinder/+/786054 | 14:47 |
| raghavendrat | Received +2 from Brian. It would be great if someone else can also have a look. | 14:47 |
| raghavendrat | Thank you. | 14:47 |
| enriquetaso | something happened with gerrit display.. it doesn't show slip view with the two files anymore :/ | 14:53 |
| *** dviroel is now known as dviroel|lunch | 14:57 | |
| opendevreview | Hironori Shiina proposed openstack/cinder master: Set attach status of backup volume to avoid deletion https://review.opendev.org/c/openstack/cinder/+/830901 | 15:31 |
| *** dviroel|lunch is now known as dviroel | 16:04 | |
| *** dviroel is now known as dviroel|afk | 20:21 | |
| hemna | ok rosmaita and geguileo I've come up with this gist to describe the factors of capacity for cinder that should be adopted based upon the definition of terms in that queens document. | 20:26 |
| hemna | https://gist.github.com/hemna/b85d6a7c7b5193e526ff45c088b596f4 | 20:26 |
| rosmaita | thanks, will take a look | 20:29 |
| hemna | my thoughts are to put that in place in cinder/utils.py and have utils. calculate_virtual_free_capacity() use that internally to get it's calculations. | 20:36 |
| hemna | and then eventually add that dict as part of the pool stats API url. | 20:37 |
| hemna | so there is a definitive answer from cinder the reported capacity utilization at any time, so external tools can do things.....such as automatically add new pool storage when the overall backend gets low. | 20:37 |
| hemna | which is what I'm facing daily | 20:37 |
| hemna | Part of the problem, is that I have to write external tooling gleaning the capacity functions from cinder internals to decide if my overall backend is getting low | 20:38 |
| hemna | as only the scheduler knows if it can or can't provision on any given pool. | 20:38 |
| hemna | based on it's algorithm. | 20:39 |
| hemna | that algorithm should be based off of those calculated stats in 1 place in cinder. Then the capacity filter and capacity weigher can all use the same capacity calculations. | 20:40 |
| hemna | which is not the case today. | 20:40 |
| rosmaita | yeah, it is crazy to be computing the "same" values in different places in the code | 20:42 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!