Thursday, 2019-04-04

*** Liang__ has joined #openstack-cinder00:49
*** Liang__ is now known as LiangFang00:50
*** luksky has quit IRC00:52
*** mriedem has quit IRC01:09
*** enriquetaso has quit IRC01:23
*** enriquetaso has joined #openstack-cinder01:23
*** whoami-rajat has joined #openstack-cinder02:03
*** mriedem has joined #openstack-cinder02:23
openstackgerritYong Huang proposed openstack/cinder master: Convert float type value of QoS specs to interger
*** markvoelker has joined #openstack-cinder02:29
*** mriedem has quit IRC02:29
*** zhubx has quit IRC02:36
*** psachin has joined #openstack-cinder02:43
*** markvoelker has quit IRC02:59
*** zhubx has joined #openstack-cinder03:16
*** anks2k has joined #openstack-cinder03:26
*** enriquetaso has quit IRC03:32
anks2ksmcginnis: jungleboyj: Greetings. Request you please review with your feedback. OSNEXUS Quantastor CI has been responding to gerrit event stream cinder patches. Would like to really make sure it is in good state to be merged after train opens. Thanks in advance03:32
*** markvoelker has joined #openstack-cinder03:57
*** zhubx has quit IRC03:57
*** anks2k has quit IRC04:02
*** zhubx has joined #openstack-cinder04:03
*** udesale has joined #openstack-cinder04:09
*** markvoelker has quit IRC04:29
*** erlon has joined #openstack-cinder04:44
*** pcaruana has joined #openstack-cinder04:58
openstackgerritRajat Dhasmana proposed openstack/cinder stable/queens: Check Volume Status on attachment create/update
*** adriant has quit IRC05:09
*** adriant has joined #openstack-cinder05:09
*** e0ne has joined #openstack-cinder05:12
*** e0ne has quit IRC05:18
*** bhagyashris_ has joined #openstack-cinder05:23
*** markvoelker has joined #openstack-cinder05:27
*** ruffian_sheep has joined #openstack-cinder05:39
ruffian_sheepHi,does anyone know whether I should run all of the CI if I want to upload my driver of cinder05:40
*** markvoelker has quit IRC06:00
mszwedHi ruffian_sheep, I you asking if you should run all tests, then the answer is yes06:01
*** ruffian_sheep has quit IRC06:05
*** zhubx has quit IRC06:09
*** zhubx has joined #openstack-cinder06:09
whoami-rajatruffian_sheep mszwed  If the query is regarding checking the correctness of driver then only zuul and driver specific CI results matter.06:14
*** ruffian_sheep has joined #openstack-cinder06:15
ruffian_sheepwhoami-rajat|mszwed:I means if I should pass all the company's CI?I have passed the Unit Tests.06:17
*** Luzi has joined #openstack-cinder06:18
ruffian_sheepwhoami-rajat|mszwed:I take the missions from my master.It seems a little different06:19
ruffian_sheepwhoami-rajat|mszwed:This is the code I upload.If I should add more CI to test my driver?06:22
*** dpawlik has joined #openstack-cinder06:26
*** erlon has quit IRC06:36
*** sahid has joined #openstack-cinder06:37
*** slaweq has joined #openstack-cinder06:41
slaweqrecently I reported bug
openstackLaunchpad bug 1822958 in Cinder "Backup operations fails causing tempest failures" [Undecided,New]06:41
slaweqI see in last few days that we hit similar issues quite often recently, e.g.
slaweqcan maybe someone from cinder team take a look on this issue? Thx in advance06:42
*** e0ne has joined #openstack-cinder06:47
*** markvoelker has joined #openstack-cinder06:57
mszwedruffian_sheep: that's enough, but keep in mind that at least Zuul CI should pass the tests07:04
mszwedruffian_sheep: and as whoami-rajat mentioned also driver specific CI should pass the tests, which means that you have to add your own CI if you are creating new driver07:04
whoami-rajatruffian_sheep: is your CI currently running on gate? what's the name of CI?07:05
mszwedI'm sorry for my first comment - I didn't get correctly the meaning of your question07:05
openstackgerritMerged openstack/cinder master: Add upgrade checker for backup driver path
ruffian_sheepwhoami-rajat|mszwed:Now I am building the CI of my company to add.And I am trying to pass the tempest,but there is something confused me that I think I should use the CI to run the tempest so thath I can use my device.07:19
ruffian_sheepwhoami-rajat|mszwed:But in this case, how to let other drivers run on my device. Is my cognition wrong?07:21
ruffian_sheep <mszwed>:Is my statement not accurate enough :P07:21
*** rosmaita has quit IRC07:21
*** dims has quit IRC07:22
*** dims has joined #openstack-cinder07:24
ruffian_sheepwhoami-rajat|mszwed:Or tempest is used to test my own drivers and devices. Don't you have to test the implementation of other people's drivers on my device?07:24
mszwedruffian_sheep: only your CI should run on your devices. Other CI test their own devices.07:24
mszwedand Zuul runs generic tests that should pass no matter what driver and device you are developing - it won't so anything driver/device specific07:25
*** rosmaita has joined #openstack-cinder07:26
ruffian_sheep<mszwed>:Other people's drivers don't need to go to my device to run ,is it?07:26
mszwedruffian_sheep: yes07:26
mszwedtha's correct07:27
*** markvoelker has quit IRC07:31
ruffian_sheep<mszwed>: :)07:31
*** dims has quit IRC07:31
ruffian_sheepwhoami-rajat|mszwed:Let me summarize, that is, unit testing is used to make the driver conform to the specification. In this way, all manufacturers' drivers can be used after they meet the specifications. Tempest is for my own use, in order to verify that the function is passed, other manufacturers do not need to pass my tempest. And the job I need to set to my CI should be tempest.07:31
*** helenafm has joined #openstack-cinder07:32
*** dims has joined #openstack-cinder07:34
mszwedruffian_sheep: in general yes, but just to be sure that we are on the same page some clarification: unit tests basically will test your functions in your code. The more the better. As for Tempest: you don't have to create new tests for tempest, just need to run current set of Tempest tests ('all') in your CI on you driver and device.07:35
*** jistr is now known as jistr|afk07:35
mszwedruffian_sheep: but if you want to add new tests to Tempest please keep in mind that they have to be generic so that they will pass no matter what driver or device you are using07:36
mszwedwhoami-rajat: please correct me if I'm wrong :)07:36
mszwedruffian_sheep: one more comment - keep in mind that some of the Tempest tests (from 'all set) assume you are using scsi interface, but if you are using different type of interface (like NVMe-oF) you will have to exclude them07:38
mszwedsame story for manage_existing* functionality - it's being tested by Tempest, but it is not mandatory driver functionality07:41
mszwedso if you won't implement it you will also have to exclude tests for that also07:42
mszwedruffian_sheep: I hope I've explained this clear enough :)07:42
ruffian_sheep<mszwed> :Thank you so much!sir !07:46
ruffian_sheep<mszwed> :I finally cleared up a lot of relationships. I have never known what job should be set on the CI. I ask a little question more. Normally my driver should not be running on other devices (that is, the CI environment of other vendors). Because my driver code can't run their devices, but my driver runs through their CI environment. Does it mean that they are using the unit test jobs to do it?07:47
*** brinzhang has quit IRC07:50
*** brinzhang has joined #openstack-cinder07:50
mszwedruffian_sheep: unfortunately I don't know answer for this question. But you don't have to worry about other CIs - Zuul and your own CI are the only ones that you should worry about :)07:50
*** spsurya has joined #openstack-cinder07:55
ruffian_sheep<mszwed> :Get!Thank you !!!!07:57
mszwedruffian_sheep: you're welcome :)07:58
ruffian_sheep<whoami-rajat>:The name of my CI is MacroSAN Volume CI.07:59
ruffian_sheep<mszwed> :Mr,I suddenly remembered, if I should get a permission to add my CI environment? In order to listen and feedback the cinder project08:01
mszwedruffian_sheep: that I don't know unfortunately, because my CI was just extension of existing one, so I didn't need to go through that.08:03
ruffian_sheep<mszwed> :Get it.thank you so much.:P08:04
openstackgerritVadym Markov proposed openstack/cinder stable/ocata: cinder incremental backup with ceph fails if last one deleted
openstackgerritsahid proposed openstack/cinderlib master: add python 3.7 unit test job
*** tkajinam has quit IRC08:22
*** gkadam has joined #openstack-cinder08:27
*** markvoelker has joined #openstack-cinder08:28
*** e0ne has quit IRC08:30
*** e0ne has joined #openstack-cinder08:40
*** e0ne has quit IRC08:42
*** e0ne has joined #openstack-cinder08:50
*** markvoelker has quit IRC09:01
*** sdin has quit IRC09:19
*** sdinescu has quit IRC09:22
*** mvkr has joined #openstack-cinder09:24
*** LiangFang has quit IRC09:30
*** bhagyashris_ has quit IRC09:43
*** e0ne has quit IRC09:44
*** luksky has joined #openstack-cinder09:51
*** e0ne has joined #openstack-cinder09:54
*** sdinescu has joined #openstack-cinder09:57
*** sdin has joined #openstack-cinder09:57
*** markvoelker has joined #openstack-cinder09:58
*** brinzhang has quit IRC10:04
*** jistr|afk is now known as jistr10:23
*** markvoelker has quit IRC10:31
geguileowhoami-rajat: Could you +A the patch since you were the 2nd +2 please?10:33
*** peeejayz has joined #openstack-cinder10:47
*** sapd1_x has joined #openstack-cinder10:50
peeejayzHi All, We have been looking into some IOPs issues on our ceph cluster, we have found it, its cinder causing about 90% of the iops and they mainly seem to be cinder checking certain images - is this a known problem and is there any way to tune this down? as currently this is causing large problems for our cluster.10:50
*** udesale has quit IRC10:54
*** carloss has joined #openstack-cinder11:06
*** erlon has joined #openstack-cinder11:14
*** helenafm has quit IRC11:22
*** markvoelker has joined #openstack-cinder11:28
*** mvkr has quit IRC11:30
openstackgerritLee Yarwood proposed openstack/cinder master: Update volume status during volume migration.
*** thgcorrea has joined #openstack-cinder11:40
*** erlon_ has joined #openstack-cinder11:45
*** e0ne has quit IRC11:50
*** abishop has joined #openstack-cinder11:54
*** ruffian_sheep has quit IRC11:59
*** mvkr has joined #openstack-cinder12:00
*** markvoelker has quit IRC12:02
*** anks2k has joined #openstack-cinder12:16
*** anks2k has quit IRC12:17
*** brinzhang has joined #openstack-cinder12:21
rosmaitasmcginnis: yt?12:33
whoami-rajatgeguileo: can you provide the link ?12:40
openstackgerritYong Huang proposed openstack/cinder master: Supports volume attach, detach in nova api
*** Luzi has quit IRC12:48
*** zul has joined #openstack-cinder12:51
*** mriedem has joined #openstack-cinder12:54
openstackgerritBrian Rosmaita proposed openstack/cinder master: Add upgrade check for presence of policy.json file
*** lseki has joined #openstack-cinder13:00
*** dklyle has quit IRC13:02
rosmaitasmcginnis: ^^ (so we don't duplicate efforts)13:05
*** udesale has joined #openstack-cinder13:05
*** enriquetaso has joined #openstack-cinder13:06
*** Luzi has joined #openstack-cinder13:07
*** davidsha has joined #openstack-cinder13:08
*** gaoyan has joined #openstack-cinder13:12
*** itlinux has quit IRC13:12
smcginnisrosmaita: Thanks rosmaita. Odd I didn't see that locally.13:18
rosmaitasmcginnis: hmmmm ... let'13:20
rosmaitas see what happens in zuul13:20
*** raghavendrat has joined #openstack-cinder13:20
*** e0ne has joined #openstack-cinder13:22
geguileowhoami-rajat: oooops, I though I did, my bad:
openstackgerritRaghavendra Tilay proposed openstack/cinder master: 3PAR: Provide an option duing creation of volume from snapshot
whoami-rajatgeguileo: done13:27
geguileowhoami-rajat: thanks!13:27
whoami-rajatgeguileo: np :)13:27
openstackgerritJay Bryant proposed openstack/cinder master: Add upgrade check for Windows iSCSI driver.
openstackgerritJay Bryant proposed openstack/cinder master: Add Upgrade Check for Drivers Removed in Stein
jungleboyjsmcginnis: rosmaita  ^^ Those are hopefully my final upgrade checker changes.13:28
smcginnispeeejayz: Cinder doesn't "check images". Can you explain a little more what you are seeing?13:28
rosmaitajungleboyj: cool, maybe we can actually get these done today13:29
smcginnisjungleboyj: Cool, will try to look soon.13:29
smcginnisJust need to make a sacrifice to the zuul god.13:29
jungleboyj*Fingers crossed*13:29
peeejayzsmcginnis: so running logs on ceph try to work out where we are seeing IOPS and where they are comming from during the tests 140k requests from a single cinder node, compared to next highest thing that isn't cinder is a hv that did 600.13:31
peeejayzthe main requests are: 13093 [call rbd.metadata_list]13:31
peeejayz13093 [call rbd.image_get_group]13:31
peeejayz13093 [call rbd.get_stripe_unit_count]13:31
peeejayz13093 [call rbd.get_size,call rbd.get_object_prefix]13:31
peeejayz13093 [call rbd.get_size,call rbd.get_features,call13:31
peeejayzrbd.get_snapcontext,call rbd.get_parent,call lock.get_info]13:31
smcginnispeeejayz: Anything in the cinder logs? Cinder itself shouldn't be doing anything unless it's to service a request.13:31
peeejayz13093 [call rbd.get_data_pool]13:31
peeejayz13093 [call rbd.get_create_timestamp]13:31
eharneypeeejayz: what release are you running?13:32
geguileopeeejayz: most requests are probably coming from the stats reports13:32
openstackgerritJay Bryant proposed openstack/cinder stable/stein: Add NetApp E-Series Driver to Removal List
geguileopeeejayz: one for each existing volume13:32
smcginnisAh, was just thinking status updates.13:32
jdillamanpeeejayz: those are the standard calls to the OSD when opening an image13:32
geguileopeeejayz: what release?13:32
openstackgerritJay Bryant proposed openstack/cinder stable/stein: Remove scanning upgrade checker from opts generation
smcginniseharney: We merged something to make that less overhead, right?13:32
jungleboyjsmcginnis: In case we do an RC3 have those two cherry-picks that should go in.13:33
peeejayzgeguileo: Queens13:33
geguileosmcginnis: yes 'i wrote that13:33
smcginnisgeguileo: OK, great.13:33
geguileopeeejayz: let me check if the feature is there...13:33
*** Luzi has quit IRC13:34
geguileopeeejayz: set rbd_exclusive_cinder_pool = true13:34
geguileopeeejayz: in the RBD backend config section13:34
geguileopeeejayz: that makes cinder assume it is the only one using the Ceph pool13:35
geguileopeeejayz: so it doesn't have to query the backend13:35
peeejayzgeguileo: okay will this have any adverse affects13:35
geguileopeeejayz: not if you only use that RBD pool for Cinder13:36
peeejayzah we dont13:36
geguileopeeejayz: then you will not have the right stats in the scheduler if you enable that  :-(13:36
geguileothat's precisely why it's disabled by default13:36
peeejayzgeguileo: so best thing to actually do is to split our pools?13:36
geguileopeeejayz: that would be the best13:37
*** Luzi has joined #openstack-cinder13:37
peeejayzOkay, I will discuss as this is a big change with over 600 running vms currently.13:37
geguileopeeejayz: the other possibility is to enable the exclusive pool in Cinder13:37
geguileopeeejayz: and assume the scheduler will not have accurate data13:37
geguileopeeejayz: as it won't be able to account for all the storage used outside of Cinder13:38
geguileoso over provisioning and reserved space will not work as expected13:38
geguileoas far as I can tell those are your choices13:38
peeejayzgeguileo: exactly what I was going to ask, many thanks.13:38
geguileopeeejayz: no problem13:39
*** raghavendrat has quit IRC13:40
peeejayzgeguileo: is there any hints on how to migrate cinder pools while in production?13:46
geguileopeeejayz: I think the best way will be to a migration13:47
geguileopeeejayz: but it's going to be a pita, because you have to migrate ALL the volumes...13:48
geguileosome will do live migration and others will do offline migration13:48
*** spsurya has quit IRC13:51
*** altlogbot_0 has quit IRC13:53
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Fix get_connector_properties
*** e0ne has quit IRC13:59
*** imacdonn has quit IRC14:03
*** e0ne has joined #openstack-cinder14:05
*** udesale has quit IRC14:06
*** udesale has joined #openstack-cinder14:07
*** udesale has quit IRC14:11
*** udesale has joined #openstack-cinder14:11
*** gaoyan has quit IRC14:14
*** dpawlik has quit IRC14:22
openstackgerritBrian Rosmaita proposed openstack/cinder master: Document behavior of message.create
*** itlinux has joined #openstack-cinder14:32
*** itlinux has quit IRC14:32
*** Luzi has quit IRC14:34
*** e0ne has quit IRC14:46
*** itlinux has joined #openstack-cinder14:48
*** mvkr has quit IRC14:48
*** itlinux has quit IRC14:49
*** itlinux has joined #openstack-cinder14:50
*** e0ne has joined #openstack-cinder14:52
*** e0ne has quit IRC14:57
*** enriquetaso has quit IRC15:08
*** dviroel has joined #openstack-cinder15:12
*** dklyle has joined #openstack-cinder15:27
*** erlon has quit IRC15:29
*** udesale has quit IRC15:32
*** brinzhang has quit IRC15:45
*** e0ne has joined #openstack-cinder15:46
*** e0ne has quit IRC15:48
*** e0ne has joined #openstack-cinder15:48
*** e0ne has quit IRC15:48
*** e0ne has joined #openstack-cinder16:05
*** enriquetaso has joined #openstack-cinder16:07
*** lemko has quit IRC16:09
*** e0ne has quit IRC16:12
openstackgerritSean McGinnis proposed openstack/cinder master: Add upgrade check for presence of policy.json file
*** sapd1_x has quit IRC16:24
*** sahid has quit IRC16:24
*** dpawlik has joined #openstack-cinder16:25
jungleboyjrosmaita: How strong are your feelings about not failing on the first removed driver found?16:26
rosmaitanot very strong, i guess it depends on what our user expectations are16:27
rosmaitathe alternative is to keep a list of bad drivers and then you have to deal with "this driver" vs "these drivers", pretty big PITA to code up in a hurry16:27
jungleboyjSo, I think that this is the safer approach if you are ok with it.16:31
jungleboyjI am fixing the whitespace issue.  Sorry about that.16:32
whoami-rajatsmcginnis:  jungleboyj can we run a single upgrade check, like we do with unit tests?16:32
rosmaitayeah, the -1 was really about the test file & whitespace16:32
jungleboyjrosmaita:  I have figured.16:33
jungleboyjwhoami-rajat:  Not that I am aware of.16:33
rosmaitajungleboyj: maybe change the error message to indicate re-run to verify?16:34
whoami-rajatfrom rosmaita 's comment on jungleboyj  patch, i thought it would be better to have this functionality sometime in the future, running all checks again and again to check the fix for one seems tedious.16:35
rosmaitawhoami-rajat: i think i see what you mean -- it would be cool if we could use ddt to turn one check into several16:36
rosmaitai wonder if we could use multiple inheritance to get the upgrade check + unit tester functionality16:38
jungleboyjwhoami-rajat:  The documentation on how to run the upgrade check indicates re-running but I can add it to the error message.16:38
*** dpawlik has quit IRC16:38
whoami-rajatrosmaita:  not exactly. i mean to run the "removed drivers check" without running the "policy check".16:38
rosmaitawhoami-rajat: gotcha16:39
rosmaitajungleboyj: if the docs already say an operator should run multiple times, i am ok with your current check code16:39
jungleboyjOk.  Cool.  Thank you.16:42
whoami-rajatrosmaita:  not sure if it's possible but would be good to reuse the unit tester code to implement same functionality for these checks. again just for future reference.16:42
whoami-rajatjungleboyj:  ++ with the current approach.16:43
*** itlinux has quit IRC16:45
*** itlinux has joined #openstack-cinder16:47
*** itlinux has quit IRC16:54
*** itlinux has joined #openstack-cinder16:55
*** itlinux has quit IRC16:59
*** itlinux has joined #openstack-cinder17:00
*** davidsha has quit IRC17:03
*** gkadam has quit IRC17:11
*** v12aml has quit IRC17:15
*** v12aml has joined #openstack-cinder17:19
openstackgerritSean McGinnis proposed openstack/cinder master: Fix deprecation escape sequence errors under 3.6
*** imacdonn has joined #openstack-cinder17:27
openstackgerritJay Bryant proposed openstack/cinder master: Add Upgrade Check for Drivers Removed in Stein
* jungleboyj makes a sacrifice to the zuul gods.17:30
*** spsurya has joined #openstack-cinder17:37
openstackgerritJay Bryant proposed openstack/cinder master: Add upgrade check for Windows iSCSI driver.
*** gmann is now known as gmann_afk17:44
*** enriquetaso has quit IRC18:03
*** erlon_ has quit IRC18:07
*** psachin has quit IRC18:28
rosmaitasmcginnis: is the final rc deadline 23:59 UTC today?18:34
*** itlinux has quit IRC18:35
smcginnisWe just declare it is "today". So we can probably go to 12:59 PDT if we really had to.18:35
smcginnisBut I don't plan on staying up that late. That's way past my bedtime. :)18:36
rosmaitawell, it only needs to be submitted by then, right?18:36
smcginnisHeh, I suppose.18:36
smcginnisI'm seeing unit test failures on that policy check in queue, but even completely recreating my env I am not able to reproduce the failures.18:37
smcginnisOnly other difference I see is I'm currently on Fedora and that's running on Bionic.18:37
smcginnisI suppose I could spin up a machine.18:38
openstackgerritRajat Dhasmana proposed openstack/cinder master: Allow reset-state to detach volume as per policy
*** jmlowe has quit IRC18:44
rosmaitai'm on fedora too.  looks like maybe the call to CONF.find_file('cinder.conf') at line 72 in is returning None18:46
smcginnisHmm, I suppose for the unit test I can mock that to return the expected value.18:47
rosmaitai think that would be the way to go18:47
smcginnisWish I could get it to fail locally to make sure I'm actually fixing it though.18:47
*** itlinux has joined #openstack-cinder18:48
rosmaitai just did18:50
rosmaitai had an /etc/cinder hanging around from manual testing18:50
rosmaitawhen i deleted it, i get those same failures18:51
smcginnisAhhhh, that could be my case.18:51
rosmaitayeah, i am kicking myself there18:51
*** eharney has quit IRC18:55
smcginnisrosmaita: Thanks, that did it (both be able to reproduce and verify the fix).18:56
*** enriquetaso has joined #openstack-cinder18:58
*** eharney has joined #openstack-cinder18:58
*** jmlowe has joined #openstack-cinder18:59
openstackgerritSean McGinnis proposed openstack/cinder master: Add upgrade check for presence of policy.json file
*** mmethot has joined #openstack-cinder19:03
*** enriquetaso has quit IRC19:05
*** enriquetaso has joined #openstack-cinder19:05
smcginnisLooks like those jobs are all passing now.19:21
*** itlinux has quit IRC19:33
*** enriquetaso has quit IRC19:43
*** gmann_afk is now known as gmann19:49
*** enriquetaso has joined #openstack-cinder19:51
*** _erlon_ has joined #openstack-cinder19:56
_erlon_jungleboyj: hey20:00
_erlon_jungleboyj: do you know anything about the status of the reloadable options for cinder?20:00
*** thgcorrea has quit IRC20:01
jungleboyjThose were in discussion for a long time and we had no one to work on them so they didn't really progressed any further.20:01
enriquetasoHello cinder team o/ I need some reviews on backports &  ... Please if you have a minute, check them out :P20:12
openstackgerritSofia Enriquez proposed openstack/cinder master: Support Incremental Backup Completion In RBD
_erlon_jungleboyj: hmm, we might pick that up again20:17
_erlon_next weak Ill know more about20:17
smcginnis_erlon_, jungleboyj: That was a cycle goal.20:18
smcginnisThe base framework for it is all in place and IIRC it works for changing log levels.20:18
smcginnisBut we could/should review to see if there are any other config options that make sense to also be reloaded.20:19
_erlon_smcginnis: for that cicle or Train?20:19
smcginnis_erlon_: What for what?20:19
_erlon_smcginnis: yes, logging level was the main case, and I think is already supported via API right?20:19
smcginnisVia API and also via this reloadable config method.20:20
_erlon_smcginnis: :) reloadable options is the goal for this release?20:20
smcginnis_erlon_: No, it was for one of the past ones. Pike I think.20:21
_erlon_smcginnis: how does that reloadable config method work?20:21
smcginnis_erlon_: You send SIGHUP or some signal to the process. Can't remember the details, but it should have been documented.20:22
openstackgerritBrian Rosmaita proposed openstack/cinder master: Update gate-irrelevant-files
smcginnisAnd we never miss documentation, right. ;)20:22
rosmaitajungleboyj: smcginnis: wish i had thought of this a week ago: Update gate-irrelevant-files
_erlon_smcginnis: hmm, sending the SIGHUP is the approach mentioned on the spec20:23
_erlon_Ill dig some code to see what we have now20:23
smcginnisAh, it was rocky -
_erlon_mutable, reloadable, reconfiguration, mutant, we should have a spec to define how we call it :p20:25
jungleboyjsmcginnis:  I feel like we might have had an issue with that working right.  I had debugged a little but can't remember where I ended up.20:25
smcginnisI've purged those brain cells. :)20:26
jungleboyj_erlon_:  The log level should be dynamic but we were having issues reaching agreement as to what other things should be dynamic.  I am all for continuing the discussion though.20:28
_erlon_I wonder why we just don't set a new flag on oslo.config so, if, reloadable is set, the config will be read directly from the config file, not from memory20:28
_erlon_I didn't see this option in the spec20:28
_erlon_jungleboyj: yes, I remember that part too20:28
jungleboyjAnother thing we swept under the table due to lack of agreement.20:29
*** pcaruana has quit IRC20:29
jungleboyjAnd no one wanting to push it.20:29
_erlon_some drivers could have problems on initialization or keeping state with  the storage20:29
*** pcaruana has joined #openstack-cinder20:30
*** dpawlik has joined #openstack-cinder20:31
*** dpawlik has quit IRC20:35
*** itlinux has joined #openstack-cinder20:37
*** itlinux has quit IRC20:38
*** itlinux has joined #openstack-cinder20:39
jungleboyjHey, look at that.  My Upgrade Checker patches finally passed Zuul.20:41
jungleboyjsmcginnis: I assume at this point we are just going with RC2 and then backporting changes later?20:41
smcginnisjungleboyj: Yeah, I think that makes sense.20:42
smcginnisProbably not time to get anything else backported.20:42
jungleboyjOk.  I won't stress further over those patches then.20:42
jungleboyjAt least not today.20:42
openstackgerritSean McGinnis proposed openstack/cinder stable/stein: Add upgrade checker for backup driver path
*** spsurya has quit IRC20:54
*** slaweq has quit IRC20:58
*** abishop has quit IRC20:58
*** sapd1 has quit IRC21:03
*** dpawlik has joined #openstack-cinder21:04
*** sapd1 has joined #openstack-cinder21:08
*** slaweq has joined #openstack-cinder21:26
*** slaweq has quit IRC21:30
*** enriquetaso has quit IRC21:35
*** enriquetaso has joined #openstack-cinder21:39
*** enriquetaso has quit IRC21:41
*** enriquetaso has joined #openstack-cinder21:42
*** pcaruana has quit IRC21:51
*** slaweq has joined #openstack-cinder21:52
*** dpawlik has quit IRC21:53
*** enriquetaso has quit IRC21:53
*** slaweq has quit IRC21:57
*** dpawlik has joined #openstack-cinder22:05
*** dpawlik has quit IRC22:09
*** rcernin has joined #openstack-cinder22:23
*** whoami-rajat has quit IRC22:32
*** abishop has joined #openstack-cinder22:35
*** luksky has quit IRC22:57
*** tkajinam has joined #openstack-cinder22:59
*** carloss has quit IRC23:15
*** dklyle has quit IRC23:19
*** enriquetaso has joined #openstack-cinder23:19
*** enriquetaso has quit IRC23:19
*** _erlon_ has quit IRC23:25
*** itlinux has quit IRC23:31
*** eharney has quit IRC23:40

Generated by 2.15.3 by Marius Gedminas - find it at!