Monday, 2020-12-14

*** tosky has quit IRC00:36
*** eharney has quit IRC01:00
*** eharney has joined #openstack-manila01:18
*** vishalmanchanda has joined #openstack-manila04:23
*** carloss has quit IRC08:14
*** carloss has joined #openstack-manila08:14
*** tbarron has quit IRC08:16
*** johanssone has quit IRC08:19
*** tbarron has joined #openstack-manila08:22
*** johanssone has joined #openstack-manila08:23
*** tosky has joined #openstack-manila08:38
*** vhari has quit IRC08:43
openstackgerritwushiming proposed openstack/manila-tempest-plugin master: Update TOX_CONSTRAINTS_FILE  https://review.opendev.org/c/openstack/manila-tempest-plugin/+/76691710:00
*** lseki has joined #openstack-manila10:10
*** vhari has joined #openstack-manila13:25
*** _erlon_ has joined #openstack-manila14:01
*** baojg has quit IRC15:34
*** lbragstad_ has joined #openstack-manila16:38
*** lbragstad has quit IRC16:41
*** lbragstad__ has joined #openstack-manila17:13
*** lbragstad_ has quit IRC17:16
*** johanssone has quit IRC17:45
*** johanssone has joined #openstack-manila17:47
*** eharney has quit IRC18:19
*** lbragstad__ has quit IRC18:24
*** eharney has joined #openstack-manila18:31
*** eharney has quit IRC18:41
*** eharney has joined #openstack-manila18:41
*** lbragstad__ has joined #openstack-manila18:42
*** openstackgerrit has quit IRC18:50
*** gmann is now known as gmann_lunch18:53
*** baojg has joined #openstack-manila19:19
*** hoonetorg has quit IRC19:41
*** hoonetorg has joined #openstack-manila19:54
*** arne_wiebalck has quit IRC20:03
*** arne_wiebalck has joined #openstack-manila20:03
gouthamrvkmc: o/ can i have your review on: https://review.opendev.org/76407520:05
gouthamrwhen you get a chance :)20:05
*** gmann_lunch is now known as gmann20:08
*** vishalmanchanda has quit IRC20:12
*** raildo has quit IRC20:18
*** lbragstad__ has quit IRC20:19
*** raildo has joined #openstack-manila20:21
*** lbragstad has joined #openstack-manila20:22
*** rajinir has joined #openstack-manila20:25
vkmcgouthamr, of course20:33
gouthamrty vkmc20:38
*** openstackgerrit has joined #openstack-manila20:41
openstackgerritDina Saparbaeva proposed openstack/manila-ui master: Addition of share shrinking feature to Manila-UI  https://review.opendev.org/c/openstack/manila-ui/+/76701720:41
openstackgerritDina Saparbaeva proposed openstack/manila-ui master: Addition of share shrinking feature to Manila-UI  https://review.opendev.org/c/openstack/manila-ui/+/76701720:52
*** baojg has quit IRC20:54
*** baojg has joined #openstack-manila20:55
*** raissa has joined #openstack-manila20:59
*** raissa has quit IRC20:59
*** raissa has joined #openstack-manila21:00
openstackgerritDina Saparbaeva proposed openstack/manila-ui master: Addition of share shrinking feature to Manila-UI  https://review.opendev.org/c/openstack/manila-ui/+/76701721:00
*** raissa has quit IRC21:00
*** raissa has joined #openstack-manila21:03
*** raissa has quit IRC21:03
*** raissa has joined #openstack-manila21:04
*** raissa has joined #openstack-manila21:05
*** raissa has quit IRC21:05
*** raissa has joined #openstack-manila21:11
*** raissa has quit IRC21:11
*** raildo has quit IRC21:29
*** jmlowe has quit IRC21:43
*** jmlowe has joined #openstack-manila21:45
gouthamrcarloss dviroel tbarron: o/ pinging for reviews on the fix for broken lower-constraints jobs on stable/victoria and stable/ussuri: https://review.opendev.org/766670 and https://review.opendev.org/766674 :)23:04
dviroelo/23:05
*** baojg has quit IRC23:06
*** baojg has joined #openstack-manila23:07
vkmcgouthamr, one question, was this fixed in the driver already? https://review.opendev.org/c/openstack/manila/+/76407523:27
vkmcI don't see it merged23:28
gouthamrvkmc: a recent fix reinitialized the list, which makes our CI fail: https://review.opendev.org/q/I14835f6c54376737b41cbf78c94908ea1befde1523:28
vkmcyes, I see that23:29
gouthamrvkmc: we unfortunately ended up backporting the bug when the CI was broken23:29
vkmcbut now you are removing the reinit... and we cannot the list of used volumes to continue growing23:29
vkmcor what I am missing?23:30
vkmcI read your suggestion to Lin to make the driver smarter by using the controller database to store state23:30
vkmcbut I don't see they added that fix yet23:30
vkmcwe cannot prevent*23:33
vkmcso I assume that we are fixing the gate and then fixing that issue in a follow up?23:34
gouthamrvkmc: sorry, stepped afk for a few23:41
gouthamrvkmc: no, its a set, so additions are idempotent23:41
vkmcgouthamr, oh sweet23:42
vkmcyes, I got confused23:42
vkmccool23:42
gouthamrvkmc: maintaining a set of volumes like that is definitely bad if you have multiple readers/writers that can't operate in a thread safe manner on the same data23:43
vkmcgouthamr, so that's why you are suggesting to use the control database instead of having it in memory23:43
gouthamrso a better fix is necessary, and hopefully Lin's able to provide it23:43
gouthamryep23:43
vkmcnice23:43
vkmcok23:43
vkmcthanks for the clarification23:43
gouthamryou're welcome, thanks for reviewing vkmc23:43

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!