Tuesday, 2017-05-23

*** chas has joined #openstack-ironic00:04
*** crushil has joined #openstack-ironic00:08
*** crushil has quit IRC00:08
*** crushil has joined #openstack-ironic00:08
*** mtanino has joined #openstack-ironic00:09
*** chas has quit IRC00:09
*** crushil has quit IRC00:14
*** crushil has joined #openstack-ironic00:14
*** causten has quit IRC00:19
*** causten has joined #openstack-ironic00:20
*** causten_ has joined #openstack-ironic00:23
*** causten has quit IRC00:24
*** causten has joined #openstack-ironic00:25
*** chas has joined #openstack-ironic00:25
*** gcb has quit IRC00:26
openstackgerritHironori Shiina proposed openstack/ironic master: Add CRUD notification objects for volume connector and volume target  https://review.openstack.org/46393000:28
*** ijw has quit IRC00:28
*** causten_ has quit IRC00:29
*** chas has quit IRC00:30
*** crushil has quit IRC00:38
*** causten has quit IRC00:38
*** crushil has joined #openstack-ironic00:39
*** chas has joined #openstack-ironic00:46
*** chas has quit IRC00:51
openstackgerritHironori Shiina proposed openstack/ironic master: Add REST API for volume connector and volume target operation  https://review.openstack.org/21458600:51
openstackgerritHironori Shiina proposed openstack/ironic master: Add storage interface field to nodes API  https://review.openstack.org/46390800:53
openstackgerritHironori Shiina proposed openstack/ironic master: Add storage interface field to node-related notifications  https://review.openstack.org/46397200:53
openstackgerritHironori Shiina proposed openstack/ironic master: DevStack: Add configuration for boot-from-volume  https://review.openstack.org/46633300:53
openstackgerritHironori Shiina proposed openstack/ironic master: Add a guide for Devstack configuration for boot-from-volume  https://review.openstack.org/46656800:54
openstackgerritHironori Shiina proposed openstack/ironic master: Add a guide for Devstack configuration for boot-from-volume  https://review.openstack.org/46656800:58
*** nicodemos has quit IRC00:59
*** ijw has joined #openstack-ironic01:02
*** phuongnh has joined #openstack-ironic01:04
*** shuyingya has joined #openstack-ironic01:04
*** shuyingya has quit IRC01:07
*** shuyingya has joined #openstack-ironic01:07
*** chas has joined #openstack-ironic01:07
*** SerenaFeng has joined #openstack-ironic01:10
*** chas has quit IRC01:12
*** dprince has joined #openstack-ironic01:14
*** pmannidi has quit IRC01:28
*** chas has joined #openstack-ironic01:28
*** chas has quit IRC01:33
*** dprince has quit IRC01:39
*** ijw has quit IRC01:41
*** chas has joined #openstack-ironic01:49
*** chas has quit IRC01:53
*** pmannidi has joined #openstack-ironic01:55
*** links has joined #openstack-ironic01:58
*** pmannidi has quit IRC02:00
*** gcb has joined #openstack-ironic02:04
*** dprince has joined #openstack-ironic02:06
*** dprince has quit IRC02:09
*** chas has joined #openstack-ironic02:10
*** pmannidi has joined #openstack-ironic02:10
*** chas has quit IRC02:14
*** mtanino has quit IRC02:15
*** crushil has quit IRC02:16
*** chas has joined #openstack-ironic02:30
*** chas has quit IRC02:36
*** pmannidi has quit IRC02:39
*** crushil has joined #openstack-ironic02:41
*** chas has joined #openstack-ironic02:51
*** chas has quit IRC02:56
*** fragatina has joined #openstack-ironic02:58
*** litao__ has joined #openstack-ironic03:04
*** baoli has joined #openstack-ironic03:06
*** baoli has quit IRC03:11
*** ijw has joined #openstack-ironic03:12
*** chas has joined #openstack-ironic03:12
*** pmannidi has joined #openstack-ironic03:15
*** chas has quit IRC03:17
*** ijw has quit IRC03:17
*** stendulker has joined #openstack-ironic03:30
*** rbudden has quit IRC03:31
*** shuyingy_ has joined #openstack-ironic03:33
*** chas has joined #openstack-ironic03:33
*** vikrant has joined #openstack-ironic03:34
*** shuyingya has quit IRC03:36
*** chas has quit IRC03:38
*** pmannidi_ has joined #openstack-ironic03:42
*** pmannidi_ is now known as pmannidi|Trg03:42
*** pmannidi|Trg is now known as pmannidi__03:43
*** pmannidi has quit IRC03:44
*** chas has joined #openstack-ironic03:54
*** chas has quit IRC03:59
*** stendulker has quit IRC04:00
*** fragatina has quit IRC04:00
*** Nisha_Agarwal has joined #openstack-ironic04:10
*** SerenaFeng has quit IRC04:11
*** chas has joined #openstack-ironic04:15
*** chas has quit IRC04:19
openstackgerritHironori Shiina proposed openstack/ironic master: DNM/POC Workaround for boot from Cinder volume  https://review.openstack.org/44194404:26
*** chas has joined #openstack-ironic04:36
*** chas has quit IRC04:40
*** shuyingy_ has quit IRC04:53
*** shuyingya has joined #openstack-ironic04:53
*** chas has joined #openstack-ironic04:57
*** chas has quit IRC04:57
*** chas has joined #openstack-ironic04:57
*** shuyingy_ has joined #openstack-ironic05:01
*** shuyingya has quit IRC05:04
*** chas has quit IRC05:08
*** udesale has joined #openstack-ironic05:16
*** jaosorior_away is now known as jaosorior05:24
*** chas has joined #openstack-ironic05:28
*** chas has quit IRC05:33
*** mjura has joined #openstack-ironic05:35
*** adisky_ has joined #openstack-ironic05:36
*** mtreinish has quit IRC05:38
*** SerenaFeng has joined #openstack-ironic05:40
*** chas has joined #openstack-ironic05:50
*** Nisha_ has joined #openstack-ironic05:50
*** Nisha_Agarwal has quit IRC05:53
*** crushil has quit IRC05:54
*** chas has quit IRC05:54
*** rcernin has quit IRC06:01
*** shuyingy_ has quit IRC06:07
*** shuyingya has joined #openstack-ironic06:07
*** shuyingy_ has joined #openstack-ironic06:09
*** chas has joined #openstack-ironic06:11
*** shuyingya has quit IRC06:13
*** ChubYann has quit IRC06:15
*** chas has quit IRC06:15
*** rcernin has joined #openstack-ironic06:18
*** SerenaFeng has quit IRC06:18
*** mvpnitesh has joined #openstack-ironic06:19
*** shuyingy_ has quit IRC06:20
*** SerenaFeng has joined #openstack-ironic06:22
*** ijw has joined #openstack-ironic06:24
*** shuyingya has joined #openstack-ironic06:25
*** rbartal has joined #openstack-ironic06:26
*** mvpnitesh has quit IRC06:26
*** mvpnitesh has joined #openstack-ironic06:27
*** ijw has quit IRC06:28
*** shuyingy_ has joined #openstack-ironic06:29
*** chas has joined #openstack-ironic06:32
*** shuyingy_ has quit IRC06:33
*** mtreinish has joined #openstack-ironic06:33
*** shuyingy_ has joined #openstack-ironic06:34
*** shuying__ has joined #openstack-ironic06:35
*** chas has quit IRC06:36
*** shuying__ has quit IRC06:36
*** shuyingya has quit IRC06:37
*** shuyingya has joined #openstack-ironic06:37
*** shuyingy_ has quit IRC06:37
*** shuyingy_ has joined #openstack-ironic06:38
*** pcaruana has joined #openstack-ironic06:39
*** shuyingya has quit IRC06:41
*** chas has joined #openstack-ironic06:52
*** shuyingy_ has quit IRC06:53
*** shuyingya has joined #openstack-ironic06:56
*** chas has quit IRC06:56
*** Jagan has joined #openstack-ironic06:59
*** Jagan is now known as Guest3068306:59
*** Guest30683 has quit IRC07:01
*** jaganathan has joined #openstack-ironic07:02
*** chas has joined #openstack-ironic07:03
*** shuyingy_ has joined #openstack-ironic07:04
*** Serlex has joined #openstack-ironic07:05
*** Serlex has quit IRC07:05
*** Serlex has joined #openstack-ironic07:05
*** Nisha_ has quit IRC07:06
*** shuyingya has quit IRC07:07
*** shuyingy_ has quit IRC07:12
*** shuyingya has joined #openstack-ironic07:13
*** rcernin has quit IRC07:16
*** rcernin has joined #openstack-ironic07:17
*** hw_wutianwei1 has joined #openstack-ironic07:17
*** hw_wutianwei has quit IRC07:20
*** hw_wutianwei1 is now known as hw_wutianwei07:20
*** priteau has joined #openstack-ironic07:28
*** kromanenko has joined #openstack-ironic07:31
*** milan has joined #openstack-ironic07:35
*** ralonsoh has joined #openstack-ironic07:53
*** cuongnq has joined #openstack-ironic07:55
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
*** amarao has joined #openstack-ironic08:05
*** mvpnitesh has quit IRC08:14
*** lucas-afk is now known as lucasagomes08:16
lucasagomesmorning all08:17
*** openstackgerrit has quit IRC08:18
milanmorning Ironic! :)08:18
milanmorning anupn, lucasagomes, vdrok! :)08:18
*** athomas has joined #openstack-ironic08:21
*** racedo has joined #openstack-ironic08:21
lucasagomeso/08:26
*** mvpnitesh has joined #openstack-ironic08:27
pas-hamorning ironic :)08:31
milanmorning pas-ha! :)08:32
*** mvpnitesh has quit IRC08:32
*** mvpnitesh has joined #openstack-ironic08:33
*** derekh has joined #openstack-ironic08:36
*** baoli has joined #openstack-ironic08:37
*** baoli has quit IRC08:42
*** ccamacho|brb is now known as ccamacho08:45
*** dtantsur|afk is now known as dtantsur08:45
dtantsurmorning ironic08:45
*** retr0h has quit IRC08:46
*** mkrai has quit IRC08:46
*** sambetts|afk has quit IRC08:48
*** amotoki has quit IRC08:49
*** amotoki has joined #openstack-ironic08:51
*** mkrai has joined #openstack-ironic08:51
*** sambetts_ has joined #openstack-ironic08:52
milanmorning dtantsur! :)08:52
*** dtantsur is now known as dtantsur|brb08:54
*** shuyingy_ has joined #openstack-ironic08:55
*** mvpnitesh has quit IRC08:55
*** jistr is now known as jistr|call08:56
*** mvpnitesh has joined #openstack-ironic08:56
*** shuyingya has quit IRC08:58
vdrokmorning ironic, lucasagomes milan and dtantsur|brb !08:59
milandtantsur|brb, hmmm, maybe a coffee day today? ;)09:00
*** fragatina has joined #openstack-ironic09:01
jaganathanmvpnitesh, hi09:01
*** shuyingy_ has quit IRC09:02
*** shuyingya has joined #openstack-ironic09:02
*** aarefiev_afk is now known as aarefiev09:04
aarefievmorning all!09:04
lucasagomesvdrok, morning :D09:05
*** fragatina has quit IRC09:09
*** racedo has quit IRC09:18
*** jistr|call is now known as jistr09:19
*** sambetts_ is now known as sambetts09:24
sambettsMorning all09:24
*** e0ne has joined #openstack-ironic09:25
*** retr0h has joined #openstack-ironic09:32
vdrokgood morning sambetts09:43
vdrokand morning jaganathan and aarefiev :)09:43
*** SerenaFeng has quit IRC09:50
*** milan has quit IRC10:01
*** SerenaFeng has joined #openstack-ironic10:01
*** cuongnq has quit IRC10:04
*** openstackgerrit has joined #openstack-ironic10:12
openstackgerritM V P Nitesh proposed openstack/python-ironicclient master: Fixed wrap from taking negative values  https://review.openstack.org/37943610:12
*** dtantsur|brb is now known as dtantsur10:13
*** yee379 has quit IRC10:13
*** yee379 has joined #openstack-ironic10:14
*** e0ne has quit IRC10:14
*** SerenaFeng has quit IRC10:14
*** e0ne has joined #openstack-ironic10:14
*** racedo has joined #openstack-ironic10:19
dtantsurlucasagomes: hi! may I please bother you again with https://review.openstack.org/#/c/464617/ ? I'd like all the in-progress patch to start using it asap.10:24
patchbotpatch 464617 - sushy - Declarative approach to parsing JSON fields10:24
openstackgerritYuriy Zveryanskyy proposed openstack/ironic master: Add security tips  https://review.openstack.org/46672910:25
lucasagomesdtantsur, will take a look10:35
*** SerenaFeng has joined #openstack-ironic10:35
*** shuyingy_ has joined #openstack-ironic10:40
*** jkilpatr has quit IRC10:40
*** shuyingya has quit IRC10:43
lucasagomesdtantsur, neat!10:47
dtantsur:)10:50
*** fragatina has joined #openstack-ironic10:52
*** jkilpatr has joined #openstack-ironic10:57
openstackgerritAnton Arefiev proposed openstack/ironic-inspector master: Introduce oslo service for workers  https://review.openstack.org/44633610:58
*** xavierr has joined #openstack-ironic10:59
*** mvpnitesh has quit IRC11:05
*** mvpnitesh has joined #openstack-ironic11:08
openstackgerritDmitry Tantsur proposed openstack/ironic master: [install-guide] explain the defaults calculation for hardware types  https://review.openstack.org/46674111:10
*** rbudden has joined #openstack-ironic11:13
openstackgerritMerged openstack/sushy master: Declarative approach to parsing JSON fields  https://review.openstack.org/46461711:14
*** mvpnitesh has quit IRC11:15
*** udesale has quit IRC11:16
*** root____7 has joined #openstack-ironic11:24
*** phuongnh has quit IRC11:25
*** root____5 has quit IRC11:27
*** ralonsoh_ has joined #openstack-ironic11:32
*** ralonsoh_ has quit IRC11:32
*** fellypefca has joined #openstack-ironic11:35
*** lucasagomes is now known as lucas-afk11:38
*** lucas-afk is now known as lucas-hungry11:38
openstackgerritDmitry Tantsur proposed openstack/ironic master: [install-guide] explain the defaults calculation for hardware types  https://review.openstack.org/46674111:49
dtantsurvdrok: quick fix for the bracket ^^ mind re+2ing?11:49
openstackgerritOpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements  https://review.openstack.org/46714811:50
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/46714911:50
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-lib master: Updated from global requirements  https://review.openstack.org/46715011:50
vdrokSure thing :)11:51
dtantsurthanks!11:51
*** litao__ has quit IRC11:53
*** nicodemos has joined #openstack-ironic11:56
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironic-inspector-client master: Updated from global requirements  https://review.openstack.org/46719411:59
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient master: Updated from global requirements  https://review.openstack.org/46719511:59
fellypefcadtantsur Hi! Did you take some time to take a look at the backport I told you about? (https://review.openstack.org/#/c/459325/)12:03
patchbotpatch 459325 - ironic (stable/ocata) - Validation before perform node deallocation12:03
dtantsurno, not yet, sorry12:03
dtantsurchecking now12:03
*** jaganathan has quit IRC12:04
fellypefcaThanks! :)12:06
*** SerenaFeng has quit IRC12:08
*** mkrai has quit IRC12:09
*** retr0h has quit IRC12:09
*** ricardoas has quit IRC12:11
*** milan has joined #openstack-ironic12:12
milanmorning aarefiev, sambetts and all! :)12:12
aarefievg'morning milan :)12:12
*** aolwas has quit IRC12:12
milan:) aarefiev I see you've updated the patch for API--worker split!12:13
* milan gonna review12:13
aarefievyep, updated first one12:15
aarefievI didn't push others with related changes in case I've to update first patch once more time12:15
aarefievthx milan :)12:16
*** mkrai has joined #openstack-ironic12:16
milanI see, no prob! :)12:16
* milan still writes coverage for the dnsmasq-based filter (WIP)12:16
*** bfournie has quit IRC12:17
fellypefcadtantsur Thanks again for the review! :)12:19
*** SerenaFeng has joined #openstack-ironic12:19
aarefievmilan: btw as for dnsmasq filter, why are you managing dnsmasq12:20
aarefievI saw you start/stop it12:20
milanaarefiev, in case of outage I'd like it to stop so it doesn't interfere w/ ironic12:20
milanor while the blacklist isn't populated12:21
milanaarefiev, in this case the dnsmasq should be co-located w/ inspector otherwise it makes no sense to configure it either12:27
*** dprince has joined #openstack-ironic12:30
*** ipukha has joined #openstack-ironic12:30
aarefievmilan: in this case we need to make init system configurable12:32
aarefievI'm wondering is any python lib for this12:32
milanaarefiev, hmmm what exactly would need configure?12:33
*** crushil has joined #openstack-ironic12:34
*** jaganathan has joined #openstack-ironic12:34
openstackgerritKyrylo Romanenko proposed openstack/ironic master: Add api tests for portgroup update  https://review.openstack.org/45085512:36
*** lucas-hungry is now known as lucasagomes12:36
*** jcoufal has joined #openstack-ironic12:38
aarefievmilan: ahh, I see your comment "make service/systemctl configurable?"12:40
milanaarefiev, yeah :)12:41
milanI plan to have it as an opaque string that's gonna be split and executed through the rootwrap.subprocess12:41
milanthe patch update is ready but still working on the test cases ;)12:42
milanaarefiev, I can push if you want to take a look12:42
*** baoli has joined #openstack-ironic12:43
aarefievmilan: I'll when you are ready12:43
milanaarefiev, cool, thanks  for looking into it anyway! :)12:43
pas-hadtantsur: I've fixed first 4 patches according to vdrok comments in this chain for ansible-deploy https://review.openstack.org/#/q/topic:ansible+project:openstack/ironic-staging-drivers, so whenever you have spare minute or two please review (and hopefully merge :) )12:46
dtantsuron my radar, thanks12:46
*** retr0h has joined #openstack-ironic12:47
*** mvpnitesh has joined #openstack-ironic12:47
*** SerenaFeng has quit IRC12:51
*** ricardoas has joined #openstack-ironic12:51
*** bfournie has joined #openstack-ironic12:55
*** gcb has quit IRC12:55
*** crushil has quit IRC12:56
openstackgerritMerged openstack/ironic master: Remove unneeded lookup policy check  https://review.openstack.org/46673512:57
*** jaosorior has quit IRC12:57
*** Goneri has joined #openstack-ironic12:57
*** crushil has joined #openstack-ironic12:57
*** jaosorior has joined #openstack-ironic12:57
*** udesale has joined #openstack-ironic12:58
*** sasha2 has joined #openstack-ironic13:00
*** crushil has quit IRC13:04
pesterfluentd have UI ? how to access it ?13:04
*** mvpnitesh has quit IRC13:04
*** crushil has joined #openstack-ironic13:04
*** shuyingy_ has quit IRC13:08
*** shuyingya has joined #openstack-ironic13:09
*** fellypefca has quit IRC13:09
dtantsurlucasagomes: I think we have an issue in redfish driver set_power_state. we don't wait for the actual power state to change. I'm not sure why we don't do it on ironic level though..13:12
*** shuyingya has quit IRC13:13
dtantsurhttps://bugs.launchpad.net/ironic/+bug/169289513:15
openstackLaunchpad bug 1692895 in Ironic "Waiting for power state is repeated in all drivers, instead of the conductor" [Medium,Triaged] - Assigned to Dmitry Tantsur (divius)13:15
*** root____5 has joined #openstack-ironic13:16
openstackgerritMerged openstack/python-ironic-inspector-client master: Updated from global requirements  https://review.openstack.org/46719413:17
*** root____7 has quit IRC13:19
*** rloo has joined #openstack-ironic13:24
mjturekmorning ironic13:28
xavierrmorning ironic13:30
xavierrmorning mjturek o/13:30
*** chas has quit IRC13:30
*** chas has joined #openstack-ironic13:30
vdrokgood morning mjturek and xavierr13:31
*** udesale has quit IRC13:31
*** crushil has quit IRC13:32
rloohi vdrok, xavierr, mjturek!13:35
*** vikrant has quit IRC13:35
*** chas has quit IRC13:35
vdrokgood morning rloo !13:35
*** gcb has joined #openstack-ironic13:45
*** ralonsoh has quit IRC13:47
*** hamzy has quit IRC13:48
dtantsurmorning rloo, xavierr, mjturek13:49
rloohi dtantsur!13:49
NobodyCamGood Morning Ironic13:50
xavierrNobodyCam: hey hey :)13:50
rloogood morning NobodyCam!13:50
TheJuliaHey mjturek, saw your message from yesterday. I'm at the ICPC this week.  Do whatever you need to do :)13:51
dtantsurmorning TheJulia, NobodyCam13:51
NobodyCamhey hey morning wavier, rloo and TheJulia13:51
NobodyCamand dtantsur :)13:51
mjturekcool cool, thanks TheJulia :) enjoy ICPC!!13:51
lucasagomesdtantsur, sorry I was in my 1:113:52
lucasagomeswill check the bug13:52
dtantsurlucasagomes: I'm fixing it, just as a heads up.13:52
*** crushil has joined #openstack-ironic13:53
lucasagomesdtantsur, is it just a refactor ?13:53
lucasagomesdon't need to asnwer haha idk why I replied with a question13:53
lucasagomesyeah it's a refactor13:53
dtantsur:D13:53
*** chlong has joined #openstack-ironic13:55
lucasagomesdtantsur, one thing is... for redfish you probably need to make sure refresh() is called so get_power_state() doesn't return the cached power state13:56
*** links has quit IRC13:57
dtantsurgood point14:00
*** ralonsoh has joined #openstack-ironic14:01
*** shuyingya has joined #openstack-ironic14:01
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Move power state wait loop to utils and use it in Redfish and IPMI  https://review.openstack.org/46724714:01
dtantsurlucasagomes: draft ^^^14:01
* lucasagomes checks14:02
openstackgerritMerged openstack/ironic stable/ocata: Validation before perform node deallocation  https://review.openstack.org/45932514:02
*** mtanino__ has joined #openstack-ironic14:04
*** cdearborn has joined #openstack-ironic14:04
*** ricardoas has left #openstack-ironic14:06
*** Nisha_Agarwal has joined #openstack-ironic14:08
openstackgerritChris Krelle proposed openstack/ironic-inspector stable/mitaka: minor change to README to trigger build  https://review.openstack.org/46724914:09
NobodyCamhahaha14:09
NobodyCambad post...14:10
vdrokmorning TheJulia and NobodyCam !14:10
NobodyCamGood Morning vdrok :)14:11
*** chas has joined #openstack-ironic14:12
* TheJulia tries to wake up14:13
*** Nisha_Agarwal has quit IRC14:14
*** chas has quit IRC14:16
milanmorning mjturek, NobodyCam, rloo, TheJulia, xavierr and all that joined since my last good morning announce ;)14:19
rloohi milan :)14:20
NobodyCamGood Morning milan :)14:20
milan:)14:20
*** shuyingy_ has joined #openstack-ironic14:22
lucasagomesNobodyCam, TheJulia rloo morning :D14:22
rloohi lucasagomes!14:22
NobodyCamGood Morning lucasagomes :)14:22
*** shuyingya has quit IRC14:24
TheJuliaGood morning everyone!14:29
* TheJulia totally forgot to talk about universal greeting time in her talk yesterday14:29
*** shuyingya has joined #openstack-ironic14:30
sambettsmorning TheJulia14:30
*** chas has joined #openstack-ironic14:33
*** shuyingy_ has quit IRC14:33
*** rpioso has joined #openstack-ironic14:34
*** MattMan_1 has quit IRC14:37
*** MattMan_1 has joined #openstack-ironic14:37
*** chas has quit IRC14:37
*** hamzy has joined #openstack-ironic14:37
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Move power state wait loop to utils and use it in Redfish and IPMI  https://review.openstack.org/46724714:39
*** David_Orange has joined #openstack-ironic14:42
sambettsdtantsur: regarding ^^ please checkout https://review.openstack.org/#/c/449454/ it was a patch to clean up the way the loop works which might be worth taking a look at14:46
patchbotpatch 449454 - ironic - Don't retry power status if power action fails14:46
*** srobert has joined #openstack-ironic14:48
*** rbartal_ has joined #openstack-ironic14:49
*** rbartal_ has quit IRC14:49
dtantsursambetts: yeah, this is nice. wanna just merge my changes in? I'd prefer we stop copy-pasting this code to every driver14:50
*** David_Orange has left #openstack-ironic14:51
*** chas has joined #openstack-ironic14:54
milanTheJulia, universal greeting time is the best invention since hammer ;)14:56
vdrok++ to making it a common function14:56
sambettsdtantsur: also like you said above I wonder if we should move it into the conductor manager?14:56
*** chas has quit IRC14:58
*** kromanenko has quit IRC15:05
*** mat128 has joined #openstack-ironic15:06
sambettsdtantsur: somewhere here?? https://github.com/openstack/ironic/blob/master/ironic/conductor/utils.py#L17015:07
dtantsursambetts: I thought about that. But it's going to be tricky because of all drivers using different options for timeout15:07
dtantsurilo has number of retries + delay, ipmi has an overall timeout, etc15:08
sambettsdtantsur: my understanding is that the IPMI ones are just for retries on the commands if they fail, I think their meaning has been overridden and should use a standard wait timeout like you've added15:09
sambettsdtantsur: not sure if its the same on the ilo side15:09
dtantsurmaybe? I'm ok with any approach that allows us to delete the repeated code15:09
*** mat128 has quit IRC15:10
sambettsdtantsur: q15:11
sambetts-_- wrong window..15:11
sambettsdtantsur: yeah, I think that it makes sense to treat it the same for all the drivers and reuse the power status function, and the drivers should be able to internally handle not making the requests to frequently like the IPMItool driver does in the _exec function15:13
*** chas has joined #openstack-ironic15:14
*** Serlex has quit IRC15:16
*** shuyingy_ has joined #openstack-ironic15:19
*** chas has quit IRC15:19
*** shuyingya has quit IRC15:22
*** rcernin has quit IRC15:22
*** mgoddard has joined #openstack-ironic15:30
*** chlong has quit IRC15:31
*** shuyingya has joined #openstack-ironic15:34
*** mgoddard has quit IRC15:35
*** chas has joined #openstack-ironic15:35
*** shuyingy_ has quit IRC15:37
*** mjura has quit IRC15:40
*** chas has quit IRC15:40
*** chlong has joined #openstack-ironic15:43
*** rbudden has quit IRC15:46
*** pcaruana has quit IRC15:47
*** amarao has quit IRC15:49
openstackgerritSam Betts proposed openstack/ironic master: Don't retry power status if power action fails  https://review.openstack.org/44945415:49
*** e0ne has quit IRC15:52
*** chas has joined #openstack-ironic15:56
*** jcoufal has quit IRC15:57
openstackgerritRuby Loo proposed openstack/ironic master: [install-guide] Add 'ipmitool' vendor interface  https://review.openstack.org/46729515:58
*** chas has quit IRC16:01
*** jkilpatr has quit IRC16:04
vdrokoh,  pas-ha by the way the glance related stuff it seems was done by GheRivero , maybe he has some context16:06
GheRiverovdrok:  what? sorry, but it's been a while since my last contribution to ironic, so I don't know what are you talking about16:10
vdrokGheRivero: yeah, I know :) but that stuff was added pretty long time ago :) http://lists.openstack.org/pipermail/openstack-dev/2017-May/117297.html16:11
GheRiverothx for the info. Will take a look anyway in case I can help16:12
vdrokthank you!16:13
*** chas has joined #openstack-ironic16:17
GheRiveroyeah, most of the code was borrowed from the nova-baremetal driver. When implemented there was no keystone catalog, or a simple glance client code (at that time de glancev1 was "old" and glancev2 was supposed to be the way to go, but we all know what happened).16:18
GheRiverowow... git blame says most of the code is almost 4 years old!16:19
openstackgerritOpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements  https://review.openstack.org/46714816:20
*** jkilpatr has joined #openstack-ironic16:20
*** trown is now known as trown|lunch16:21
*** chas has quit IRC16:22
GheRiveroI'm sure nowadays huge improvement can be made to that code16:23
pas-haGheRivero: if you don't mind, I'll start proposing deprecating all this (especially points 1) and 2) in my email). Ideally I'd like to leave only glance_host or similar as fully qualified glance API endpoint and use it for standalone (noauth) case only16:24
*** Sukhdev has joined #openstack-ironic16:24
dtantsurpas-ha: keep in mind that we've deprecated v1 support, so it's no use touching it16:25
pas-haok16:25
GheRiveroI'm more than ok to kill my code. It's about time to do something better16:25
dtantsurhttps://docs.openstack.org/releasenotes/ironic/unreleased.html#id516:25
dtantsurtime to go, see you tomorrow and good night16:26
*** dtantsur is now known as dtantsur|afk16:26
pas-haI also plan to work on unifying our code for managing noauth and keystone modes under the same keystoneauth logic, with using identity.Token and token_endpoint16:27
*** chlong has quit IRC16:30
pas-haalso a small headsup - devstack has removed 'internal' endpoints for most of the services, but in our common.keystone.get_service_url we default exaclty to 'internal'16:33
*** shuyingya has quit IRC16:33
pas-hait's a miracle (due to still hardcoding <service>_url in configs) that we were not broken yet.16:34
GheRiverowow... that code is also ages old :)16:36
pas-haso AFAIK what we need to do is introduce 'endpoint_type' to every 'client' section in config, default that to 'internal', use these config options in the code and set devstack to populate those config sections with 'public', then switch defaults to 'public' in the next release.16:36
*** Nisha_Agarwal has joined #openstack-ironic16:37
pas-haGheRivero: keystone one? not all of it, I've reworked in about a year ago and now plan to continue :)16:37
GheRiverocool! Thx for the effort16:37
*** chas has joined #openstack-ironic16:38
TheJuliapas-ha: +1 to providing flexibility, although I think making everything public, might not work in all cases, but as long as it is tunable.16:39
*** chas has quit IRC16:42
*** aarefiev is now known as aarefiev_afk16:45
*** derekh has quit IRC16:48
*** kjw3 has quit IRC16:56
*** harlowja has joined #openstack-ironic16:56
*** lucasagomes is now known as lucas-afk16:57
*** chas has joined #openstack-ironic16:59
*** baoli has quit IRC17:00
*** baoli has joined #openstack-ironic17:01
*** ralonsoh has quit IRC17:01
*** fragatin_ has joined #openstack-ironic17:01
*** ijw has joined #openstack-ironic17:03
*** fragatina has quit IRC17:03
*** chas has quit IRC17:03
pas-haexactly the reason I'd like to keep it tunable, just to be sure we are not broken by devstack... https://github.com/openstack-dev/devstack/commit/11eb2017ef70a758c54cc984e90e445205886e7e17:05
*** baoli has quit IRC17:15
openstackgerritMerged openstack/ironic-specs master: Change CLI default API version  https://review.openstack.org/44866117:16
*** jaosorior is now known as jaosorior_away17:17
*** baoli has joined #openstack-ironic17:19
*** chas has joined #openstack-ironic17:20
*** causten has joined #openstack-ironic17:20
*** chas has quit IRC17:24
*** jaganathan has quit IRC17:33
*** trown|lunch is now known as trown17:33
*** ionutbalutoiu has quit IRC17:36
*** ionutbalutoiu has joined #openstack-ironic17:39
*** chas has joined #openstack-ironic17:41
*** bfournie has quit IRC17:41
*** causten_ has joined #openstack-ironic17:45
*** chas has quit IRC17:45
*** ChubYann has joined #openstack-ironic17:46
openstackgerritMerged openstack/python-ironicclient master: Replace assertRaisesRegexp with assertRaisesRegex  https://review.openstack.org/46617517:48
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient master: Updated from global requirements  https://review.openstack.org/46719517:48
*** causten has quit IRC17:49
TheJuliacrushil: ppiela: I have a sudden conflict for the meeting today.  Please proceed without me, I'll review on Thursday17:59
*** baoli has quit IRC17:59
crushilTheJulia, Ok, thanks for letting me know17:59
*** baoli has joined #openstack-ironic18:00
*** Nisha_ has joined #openstack-ironic18:00
*** chas has joined #openstack-ironic18:01
*** Nisha_Agarwal has quit IRC18:04
*** jkilpatr has quit IRC18:06
*** chas has quit IRC18:06
*** jkilpatr has joined #openstack-ironic18:07
*** racedo has quit IRC18:08
*** racedo has joined #openstack-ironic18:09
*** racedo has quit IRC18:09
*** bfournie has joined #openstack-ironic18:15
*** bfournie has quit IRC18:17
*** fragatin_ has quit IRC18:19
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: WIP Introducing a dnsmasq PXE filter driver  https://review.openstack.org/46644818:21
* milan gone 18:22
milanNight Ironic! :)18:22
*** milan has quit IRC18:22
*** chas has joined #openstack-ironic18:23
*** chas has quit IRC18:28
*** baoli has quit IRC18:30
*** jaganathan has joined #openstack-ironic18:31
*** baoli has joined #openstack-ironic18:32
*** ijw has quit IRC18:32
openstackgerritLeif Madsen proposed openstack/bifrost master: Fix when clause in pymysql check on CentOS  https://review.openstack.org/46735718:33
*** krtaylor has quit IRC18:35
*** chas has joined #openstack-ironic18:43
*** dyasny has quit IRC18:43
*** Sukhdev has quit IRC18:44
*** chas has quit IRC18:48
*** dyasny has joined #openstack-ironic18:59
*** jaganathan has quit IRC19:01
*** jcoufal has joined #openstack-ironic19:02
*** jcoufal has quit IRC19:03
*** jcoufal has joined #openstack-ironic19:04
*** chas has joined #openstack-ironic19:04
*** dyasny has quit IRC19:06
*** adisky_ has quit IRC19:07
*** dyasny has joined #openstack-ironic19:09
*** chas has quit IRC19:09
*** jaganathan has joined #openstack-ironic19:14
*** crushil has quit IRC19:20
*** jaganathan has quit IRC19:22
*** chas has joined #openstack-ironic19:25
*** krtaylor has joined #openstack-ironic19:28
*** Nisha_ has quit IRC19:28
*** chas has quit IRC19:29
*** priteau has quit IRC19:39
*** crushil has joined #openstack-ironic19:41
*** penick has joined #openstack-ironic19:43
*** jcoufal has quit IRC19:45
*** chas has joined #openstack-ironic19:46
*** clenimar has quit IRC19:47
*** sayantan_ has joined #openstack-ironic19:49
*** chas has quit IRC19:50
*** e0ne has joined #openstack-ironic19:51
*** crushil has quit IRC19:56
*** crushil has joined #openstack-ironic19:56
*** ijw has joined #openstack-ironic19:57
*** bfournie has joined #openstack-ironic19:58
openstackgerritMichael Turek proposed openstack/ironic master: Wire in storage interface attach/detach operations  https://review.openstack.org/40629020:00
*** chas has joined #openstack-ironic20:07
*** sambetts is now known as sambetts|afk20:08
*** boris-42_ has joined #openstack-ironic20:11
*** chas has quit IRC20:11
*** sayantan_ has quit IRC20:13
*** e0ne has quit IRC20:18
*** sayantan_ has joined #openstack-ironic20:20
*** jkilpatr has quit IRC20:22
*** sayantan_ has quit IRC20:24
*** rcernin has joined #openstack-ironic20:24
*** crushil has quit IRC20:26
*** sayantan_ has joined #openstack-ironic20:26
*** crushil has joined #openstack-ironic20:27
*** chas has joined #openstack-ironic20:27
*** crushil has quit IRC20:28
*** sayantan_ has quit IRC20:31
*** chas has quit IRC20:32
*** harlowja_ has joined #openstack-ironic20:32
*** harlowja has quit IRC20:33
*** chas has joined #openstack-ironic20:48
openstackgerritMichael Turek proposed openstack/ironic master: iPXE template support for iSCSI  https://review.openstack.org/41332420:49
*** fragatina has joined #openstack-ironic20:52
*** chas has quit IRC20:53
*** fragatina has quit IRC20:57
*** fragatina has joined #openstack-ironic21:03
*** trown is now known as trown|outtypewww21:04
*** dprince has quit IRC21:04
*** harlowja_ has quit IRC21:05
*** fragatina has quit IRC21:06
*** fragatina has joined #openstack-ironic21:07
*** harlowja has joined #openstack-ironic21:08
*** chas has joined #openstack-ironic21:09
*** Goneri has quit IRC21:12
*** chas has quit IRC21:14
*** xavierr has quit IRC21:17
-openstackstatus- NOTICE: The logserver has filled up, so jobs are currently aborting with POST_FAILURE results; remediation is underway.21:21
*** ChanServ changes topic to "The logserver has filled up, so jobs are currently aborting with POST_FAILURE results; remediation is underway."21:21
*** chas has joined #openstack-ironic21:30
*** srobert has quit IRC21:34
*** chas has quit IRC21:35
*** priteau has joined #openstack-ironic21:40
*** krtaylor has quit IRC21:41
*** jkilpatr has joined #openstack-ironic21:44
*** priteau has quit IRC21:45
*** jroll has quit IRC21:47
*** jroll has joined #openstack-ironic21:47
*** jroll has quit IRC21:49
*** chas has joined #openstack-ironic21:51
*** jroll has joined #openstack-ironic21:53
*** chas has quit IRC21:56
*** jralph has joined #openstack-ironic22:04
*** rbudden has joined #openstack-ironic22:11
*** chas has joined #openstack-ironic22:12
*** cdearborn has quit IRC22:12
*** baoli has quit IRC22:13
*** chas has quit IRC22:16
*** penick has quit IRC22:28
*** chas has joined #openstack-ironic22:33
*** BadCub has quit IRC22:34
*** BadCub has joined #openstack-ironic22:35
*** NobodyCam has quit IRC22:35
*** NobodyCam has joined #openstack-ironic22:35
*** baoli has joined #openstack-ironic22:36
*** chas has quit IRC22:38
*** baoli has quit IRC22:41
*** chas has joined #openstack-ironic22:54
*** chas has quit IRC22:59
*** athomas has quit IRC23:04
*** rpioso has quit IRC23:10
*** chas has joined #openstack-ironic23:15
*** chas has quit IRC23:20
*** KennethWilke has joined #openstack-ironic23:25
*** mtanino__ has quit IRC23:30
*** chas has joined #openstack-ironic23:36
*** chas has quit IRC23:40
*** priteau has joined #openstack-ironic23:41
*** shuyingya has joined #openstack-ironic23:43
*** priteau has quit IRC23:45
*** shuyingya has quit IRC23:47
*** KennethWilke has quit IRC23:49
*** chas has joined #openstack-ironic23:56
*** fragatina has quit IRC23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!