Thursday, 2017-03-02

*** jkilpatr has joined #openstack-ironic00:08
*** krtaylor has quit IRC00:08
*** rama__ has quit IRC00:10
*** rama__ has joined #openstack-ironic00:11
*** Nisha_Agarwal has joined #openstack-ironic00:13
*** rama__ has quit IRC00:17
*** rama__ has joined #openstack-ironic00:18
*** Nisha_Agarwal has quit IRC00:29
*** sdake has quit IRC00:36
openstackgerritMarkos Chandras proposed openstack/bifrost master: playbooks: roles: bifrost-ironic-install: Verify checksum for IPA files  https://review.openstack.org/43993800:37
*** tovin07 has joined #openstack-ironic00:38
*** sdake has joined #openstack-ironic00:42
*** ppiela has quit IRC00:45
openstackgerritMarkos Chandras proposed openstack/bifrost master: scripts: test-bifrost.sh: Initialize the ZUUL_BRANCH variable  https://review.openstack.org/43994000:46
*** rpioso has quit IRC00:51
*** krtaylor has joined #openstack-ironic00:53
*** Syed__ has quit IRC00:55
openstackgerritMarkos Chandras proposed openstack/bifrost master: playbooks: roles: bifrost-ironic-install: Verify checksum for IPA files  https://review.openstack.org/43993800:59
*** crushil has joined #openstack-ironic01:03
*** phuongnh has joined #openstack-ironic01:04
*** rama_y has quit IRC01:15
*** zhangjl has joined #openstack-ironic01:23
*** rloo has quit IRC01:31
*** mtanino has quit IRC01:37
openstackgerritBruno Cornec proposed openstack/ironic-specs master: Propose Redfish support into Ironic  https://review.openstack.org/18465301:48
openstackgerritBruno Cornec proposed openstack/ironic-specs master: Propose Redfish support into Ironic  https://review.openstack.org/18465301:57
*** ijw has quit IRC01:58
*** sayanta__ has quit IRC02:01
openstackgerritMarkos Chandras proposed openstack/bifrost master: playbooks: roles: bifrost-create-vm-nodes: Reboot VM if boot fails  https://review.openstack.org/43994602:08
*** sdake_ has joined #openstack-ironic02:10
*** sdake has quit IRC02:13
*** sdake_ is now known as sdake02:28
openstackgerritJohn L. Villalovos proposed openstack/ironic master: WIP/DNM: Debug info for grenade settings  https://review.openstack.org/43990302:43
openstackgerritJohn L. Villalovos proposed openstack/ironic master: WIP/DNM: Debug info for grenade settings  https://review.openstack.org/43990302:48
*** yee379 has quit IRC02:49
*** yee379 has joined #openstack-ironic02:50
openstackgerritJohn L. Villalovos proposed openstack/ironic master: WIP/DNM: Testing multi-tenant multi-node grenade job  https://review.openstack.org/42214902:52
*** Sukhdev has joined #openstack-ironic02:54
*** Sukhdev has quit IRC02:56
openstackgerritAnup Navare proposed openstack/ironic master: Allow logical name in port creation  https://review.openstack.org/40558602:57
*** Nisha_Agarwal has joined #openstack-ironic02:59
openstackgerritzack chen proposed openstack/ironic-python-agent master: Collected logs is 'bytes' type.  https://review.openstack.org/43890103:02
*** links has joined #openstack-ironic03:08
*** aparnav has joined #openstack-ironic03:24
*** vikrant has joined #openstack-ironic03:29
*** jaganathan_afk has joined #openstack-ironic03:31
*** jaganathan_afk is now known as jaganathan03:35
*** korekhov has quit IRC03:43
*** Nisha_ has joined #openstack-ironic03:46
*** Nisha_brb has joined #openstack-ironic03:47
*** Nisha_ has quit IRC03:47
*** Nisha_Agarwal has quit IRC03:48
*** tuanluong has joined #openstack-ironic03:50
*** rcernin has quit IRC03:50
*** nmathew has joined #openstack-ironic03:58
*** Nisha_brb has quit IRC04:04
*** Nisha_Agarwal has joined #openstack-ironic04:06
*** udesale has joined #openstack-ironic04:07
*** Nisha_Agarwal has quit IRC04:11
*** Nisha_Agarwal has joined #openstack-ironic04:12
*** fragatina has quit IRC04:13
*** sayantan_ has joined #openstack-ironic04:18
openstackgerritzack chen proposed openstack/ironic-python-agent master: Collected logs is 'bytes' type.  https://review.openstack.org/43890104:24
*** fragatina has joined #openstack-ironic04:28
*** fragatina has quit IRC04:32
*** jaganathan is now known as jaganathan_afk04:38
openstackgerritTony Breeds proposed openstack/ironic master: [DNM] Testing WebOb 1.7.1  https://review.openstack.org/43999504:40
*** jaganathan_afk has quit IRC04:43
*** crushil has quit IRC04:57
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent master: Updated from global requirements  https://review.openstack.org/44005204:58
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui master: Updated from global requirements  https://review.openstack.org/44005304:58
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironic-inspector-client master: Updated from global requirements  https://review.openstack.org/44009705:06
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient master: Updated from global requirements  https://review.openstack.org/44009805:06
openstackgerritOpenStack Proposal Bot proposed openstack/python-oneviewclient master: Updated from global requirements  https://review.openstack.org/38994205:06
*** sdake has quit IRC05:08
openstackgerritOpenStack Proposal Bot proposed openstack/virtualbmc master: Updated from global requirements  https://review.openstack.org/44012505:09
openstackgerritOpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements  https://review.openstack.org/44004905:13
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/44005005:13
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent master: Updated from global requirements  https://review.openstack.org/44005205:13
*** sasha21 has quit IRC05:16
*** sayantan_ has quit IRC05:19
*** fragatina has joined #openstack-ironic05:22
*** fragatina has quit IRC05:26
openstackgerritOpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements  https://review.openstack.org/44004905:27
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/44005005:27
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent master: Updated from global requirements  https://review.openstack.org/44005205:27
*** yolanda has quit IRC05:35
*** jcoufal has joined #openstack-ironic05:36
*** Sukhdev has joined #openstack-ironic05:52
*** jcoufal_ has joined #openstack-ironic05:59
*** jcoufal has quit IRC06:00
*** Sukhdev has quit IRC06:05
*** jcoufal has joined #openstack-ironic06:12
*** jcoufal_ has quit IRC06:15
openstackgerritMadhuri Kumari proposed openstack/ironic master: Fix directories permission for tftpboot  https://review.openstack.org/42707706:16
*** jaganathan_afk has joined #openstack-ironic06:19
*** sayantan_ has joined #openstack-ironic06:19
*** jcoufal_ has joined #openstack-ironic06:29
*** jcoufal has quit IRC06:31
*** e0ne has joined #openstack-ironic06:31
openstackgerritzack chen proposed openstack/ironic-python-agent master: Collected logs is 'bytes' type.  https://review.openstack.org/43890106:42
*** mjura has joined #openstack-ironic06:43
*** jcoufal has joined #openstack-ironic06:43
*** jcoufal_ has quit IRC06:46
*** zhurong has joined #openstack-ironic06:46
*** vsaienko has joined #openstack-ironic06:51
*** vsaienko has quit IRC06:51
*** zhurong has quit IRC06:53
*** jcoufal has quit IRC06:59
*** ChubYann has quit IRC07:04
*** rcernin has joined #openstack-ironic07:18
*** tesseract has joined #openstack-ironic07:18
*** rbartal has joined #openstack-ironic07:26
openstackgerritOksana Voshchana proposed openstack/ironic master: Adds wsgi to ironic-api in devstack  https://review.openstack.org/43085107:27
*** hwoarang has quit IRC07:28
*** hwoarang has joined #openstack-ironic07:29
*** zhugaoxiao has joined #openstack-ironic07:34
*** sayantan_ has quit IRC07:36
openstackgerritM V P Nitesh proposed openstack/ironic master: Now displays maintenance and maintenance_reason  https://review.openstack.org/43302907:40
*** dsneddon has quit IRC07:40
*** e0ne has quit IRC07:47
*** amarao has joined #openstack-ironic07:50
openstackgerritzack chen proposed openstack/ironic-python-agent master: Collected logs is 'bytes' type.  https://review.openstack.org/43890107:57
openstackgerritPavlo Shchelokovskyy proposed openstack/python-ironicclient master: Do not depend on python-openstackclient  https://review.openstack.org/29773008:04
pas-hamorning ironic :)08:05
zhangjlpas-ha:morning08:07
*** athomas has joined #openstack-ironic08:07
zhangjlpas-ha:could ironic work with zmq as rpc_backend?08:08
pas-hazhangjl: we at mirantis were intending to test this for long time already but still did not got our hands on it :( so I do not have any definite answer08:09
zhangjlpas-ha:ok. i reported a bug of ironic work with zmq #link https://bugs.launchpad.net/ironic/+bug/166924308:10
openstackLaunchpad bug 1669243 in Ironic "Ironic cannot work with zmq using oslo.messaing" [Undecided,New] - Assigned to zhangjialong (zhangjl)08:10
zhangjlpas-ha:pls give me some advice08:10
zhangjlpls give me some advices08:11
pas-haunfortunately I do not have any beside asking you to attach error messages from ir-api and ir-cond logs to the bug so we could see what's actually happening08:13
*** pcaruana has joined #openstack-ironic08:13
Nisha_Agarwallucas-afk, hi08:14
Nisha_Agarwallucas-afk, i see following error when i install the redfish driver patch https://review.openstack.org/#/c/438982/208:14
patchbotpatch 438982 - ironic - Add redfish driver08:14
Nisha_Agarwal2017-03-02 00:33:54.882 ERROR ironic.conductor.base_manager [req-f2a5c766-d14a-4df3-ab54-51cd0af3755d None None] Failed to register hardware types. For hardware type 'redfish', no default value found for management interface.08:15
*** zhangjl has quit IRC08:18
*** zhangjl has joined #openstack-ironic08:18
*** Fdaisuke has quit IRC08:24
*** ccamacho has joined #openstack-ironic08:33
*** nmathew has quit IRC08:41
*** nmathew- has joined #openstack-ironic08:41
*** dnikisho1 has quit IRC08:41
*** dnikishov has joined #openstack-ironic08:42
*** dnikishov is now known as Guest6638508:42
*** ralonsoh has joined #openstack-ironic08:46
*** zackf has quit IRC08:50
*** sdake_ has joined #openstack-ironic08:52
*** zhangjl1 has joined #openstack-ironic08:54
*** jaosorior has joined #openstack-ironic08:56
*** zhangjl has quit IRC08:56
*** vmud213 has joined #openstack-ironic08:56
openstackgerritRico Lin proposed openstack/ironic master: Update test requirement  https://review.openstack.org/44022608:58
*** zzzeek has quit IRC09:00
openstackgerritRico Lin proposed openstack/python-ironicclient master: Update test requirement  https://review.openstack.org/44022709:00
*** zzzeek has joined #openstack-ironic09:00
*** priteau has joined #openstack-ironic09:01
openstackgerritRico Lin proposed openstack/ironic-lib master: Update test requirement  https://review.openstack.org/44022909:01
openstackgerritRico Lin proposed openstack/ironic-ui master: Update test requirement  https://review.openstack.org/44023009:03
*** amoralej|off is now known as amoralej09:13
openstackgerritPavlo Shchelokovskyy proposed openstack/python-ironicclient master: Do not depend on python-openstackclient  https://review.openstack.org/29773009:20
*** Nisha_Agarwal has quit IRC09:23
*** udesale__ has joined #openstack-ironic09:26
*** udesale has quit IRC09:28
*** racedo has joined #openstack-ironic09:31
*** aparnav has quit IRC09:31
*** appu103 has joined #openstack-ironic09:31
*** nmathew- has quit IRC09:34
*** lucas-afk is now known as lucasagomes09:35
lucasagomesmorning all09:35
*** aarefiev_afk is now known as aarefiev09:39
aarefievmorning all!09:39
zhangjl1lucasagomes, aarefiev: morning09:40
lucasagomeso/09:40
aarefievo/09:40
zhangjl1lucasagomes, aarefiev: i have a questions about ironic and zmq09:40
*** sdake_ is now known as sdake09:41
zhangjl1lucasagomes, aarefiev: now, ironic could not work with zmq as expected09:41
zhangjl1does ironic need add support for zmq?09:42
lucasagomes:-/09:42
aarefievwe are using oslo.messaging abstraction what did you see in logs09:43
lucasagomeszhangjl1, yeah, what aarefiev said we just use oslo.messgaging so whatever backend supports should be supported by ironic09:43
aarefievactually I don't know if somebody even try ironic with zmq09:45
aarefievmost uses rabbitmq09:45
zhangjl1lucasagomes, aarefiev: yes. While, when i using zmq as oslo.messaging driver, ironic cannot work09:47
*** udesale__ has quit IRC09:47
zhangjl1lucasagomes, aarefiev:  https://bugs.launchpad.net/ironic/+bug/166924309:47
openstackLaunchpad bug 1669243 in Ironic "Ironic cannot work with zmq using oslo.messaing" [Undecided,New] - Assigned to zhangjialong (zhangjl)09:47
*** udesale has joined #openstack-ironic09:48
lucasagomeszhangjl1, other services are running fine ?09:48
zhangjl1lucasagomes, aarefiev: yes09:48
zhangjl1lucasagomes, aarefiev: is this a bug?09:49
zhangjl1lucasagomes, aarefiev: zmq as rpc_backend can work well in nova,cinder09:49
zhangjl1lucasagomes, aarefiev: and neutron ...09:49
lucasagomesit's possible yeah, I mean, you probably are the first one trying09:50
zhangjl1lucasagomes, aarefiev: and i checked the usage of rpc in nova and ironic09:50
aarefievdid you see something in conductor logs? I see "Matchmaker contains no records for specified target"09:51
zhangjl1lucasagomes, aarefiev: in nova : https://github.com/openstack/nova/blob/master/nova/compute/rpcapi.py#L42709:51
zhangjl1aarefiev: cannot see anything in conductor09:51
zhangjl1and in ironic, we use rpc like : https://github.com/openstack/ironic/blob/master/ironic/conductor/rpcapi.py#L16409:52
*** zhangjl1 has quit IRC09:52
*** zhangjl has joined #openstack-ironic09:53
aarefievI don't thinks the problem in how we use rpc api ,my guess it's some misconfiguration in config or in client/server object useages09:54
aarefievhard to say09:54
zhangjllucasagomes, aarefiev:  the topic of ironic registed in zmq like "ROUTER/ironic.conductor_manager/devstack"09:54
zhangjllucasagomes, aarefiev:  while , in https://github.com/openstack/ironic/blob/master/ironic/conductor/rpcapi.py#L12609:55
zhangjlwe got topic as ironic.conductor_manager.devstack09:56
zhangjllucasagomes, aarefiev:  then, the error occured09:56
zhangjllucasagomes, aarefiev: i need your advices and help09:58
*** MattMan has quit IRC10:00
lucasagomeszhangjl, right, but I mean you need to investigate the problem and if your really stuck ask around here cause I have some other priorities than looking at zmq at the moment10:00
*** MattMan has joined #openstack-ironic10:00
aarefievsorry, I don't know much about zmq, but looks like zmq driver added ROUTER prefix to target topic, so uses proxy router, maybe you missing some configuration, but hard to say w/o digging10:02
zhangjllucasagomes, aarefiev: thanks.10:04
*** ralonsoh_ has joined #openstack-ironic10:05
*** ralonsoh has quit IRC10:05
*** ralonsoh_ is now known as ralonsoh10:06
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic stable/mitaka: DNM test gates with disabled *_ssh drivers  https://review.openstack.org/43978110:08
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic stable/mitaka: Pin version of virtualbmc in devstack plugin  https://review.openstack.org/44028810:08
zhangjllucasagomes, aarefiev:  maybe i made a mistake. i means, if i change self.client.prepare(topic=topic or self.topic, version='1.36') to self.client.prepare(server=server, version='1.36') like nova , ironic worked well, both rabbitmq and zmq.10:09
openstackgerritPavlo Shchelokovskyy proposed openstack/python-ironicclient stable/mitaka: DNM test gates with disabled *_ssh drivers  https://review.openstack.org/43967310:11
zhangjllucasagomes, aarefiev:  do we need to change it ?10:11
openstackgerritYuriy Zveryanskyy proposed openstack/ironic master: Move eventlet monkey patch code  https://review.openstack.org/44029210:11
*** aparnav has joined #openstack-ironic10:12
*** appu103 has quit IRC10:12
lucasagomeszhangjl, hmm so ironic make heavy use of topic so it can route the messages from the API to the right conductor that is managing the node (see the hash ring we have)10:12
lucasagomesso I think you should not change and maybe, we need some extra code to deal with zmq10:12
lucasagomesI haven't looked so I don't know10:12
*** tovin07 has quit IRC10:12
zhangjllucasagomes: thanks for your advice10:13
lucasagomesnp10:13
*** nmathew has joined #openstack-ironic10:16
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic stable/newton: DNM test gates with disabled *_ssh drivers  https://review.openstack.org/44030010:17
*** yufei has joined #openstack-ironic10:17
*** sdake has quit IRC10:18
*** dtantsur|afk is now known as dtantsur10:19
dtantsurmorning ironic10:19
openstackgerritVasyl Saienko proposed openstack/ironic master: WIP/DNM: Testing multi-tenant multi-node grenade job  https://review.openstack.org/43695710:20
lucasagomesdtantsur, o/10:20
aarefievmorning dtantsur10:22
*** sdake has joined #openstack-ironic10:22
dtantsuro/10:23
*** udesale__ has joined #openstack-ironic10:26
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-inspector master: DNM test gates with disabled *_ssh drivers  https://review.openstack.org/44030810:26
openstackgerritMarkos Chandras proposed openstack/bifrost master: playbooks: roles: bifrost-create-vm-nodes: Reboot VM if boot fails  https://review.openstack.org/43994610:26
*** udesale has quit IRC10:28
*** udesale__ has quit IRC10:30
*** yufei has quit IRC10:30
*** links has quit IRC10:31
*** zhangjl has quit IRC10:36
openstackgerritMerged openstack/ironic-lib master: Update UPPER_CONSTRAINTS_FILE for stable/ocata  https://review.openstack.org/42211210:37
dtantsurlucasagomes, do you remember the reason of not having virtualbmc in global requirements?10:40
*** sdake has quit IRC10:41
lucasagomesdtantsur, I don't remember that discussion to be honest10:42
lucasagomesit's not necessary a library, maybe that's why ?10:42
vsaienk0morning Ironic!10:42
dtantsurlucasagomes, now we have a problem with mitaka CI, which has pyghmi capped, but not virtualbmc...10:43
lucasagomesdtantsur, oh right, yeah I saw pas-ha email10:45
lucasagomeshmm10:45
*** mgoddard has joined #openstack-ironic10:45
lucasagomesso, I don't think it will do any harm to have it in the global-requirements10:45
pas-hadirty hack for mitaka https://review.openstack.org/#/c/440288/10:45
patchbotpatch 440288 - ironic (stable/mitaka) - Pin version of virtualbmc in devstack plugin10:45
lucasagomesor we could also change our devstack plugin in mitaka to install the right version ?10:45
lucasagomesright, as pas-ha did10:45
openstackgerritDmitry Tantsur proposed openstack/ironic master: doc: update FAQ for release notes  https://review.openstack.org/43905410:46
pas-hawell, there is no problem with that on CI as is, since we had no voting job gating on ipmitool drivers in mitaka10:46
dtantsurpas-ha, if that passes CI, I'd prefer we introduce virtualbmc in global-requirements instead..10:46
dtantsurpas-ha, ... except for ironic-lib10:47
dtantsurwhere it's voting :(10:47
pas-haoops10:47
pas-hathe g-r + u-c is the right way10:47
pas-has/the/then10:47
*** links has joined #openstack-ironic10:47
pas-habut a change to devstack plugin might also be needed to use 'pip_install_gr' devstack function instead of simple 'pip_install'10:48
dtantsurcorrect10:49
openstackgerritMerged openstack/ironic-specs master: Specs implemented in Ocata  https://review.openstack.org/43981810:54
*** gcb has quit IRC10:57
*** gcb has joined #openstack-ironic10:58
*** sambetts|afk is now known as sambetts10:59
sambettsMorning all10:59
*** derekh has joined #openstack-ironic10:59
dtantsurpas-ha, wanna submit the requirements proposal or should I?11:01
dtantsurmorning sambetts11:01
*** vsaienko has joined #openstack-ironic11:01
pas-hadtantsur: I'll do it11:01
pas-hato all branches11:02
dtantsurpas-ha, thanks! just make it clear in your commit messages that virtualbmc is already used, we should screwed up ourselves by not putting it in g-r..11:02
pas-hayep :(11:02
*** gcb has quit IRC11:04
openstackgerritMerged openstack/python-ironicclient stable/mitaka: Handle upper constraints in tox  https://review.openstack.org/43974211:06
*** nmathew has quit IRC11:08
vdrokmorning all!11:09
pas-hadtantsur: for master g-r https://review.openstack.org/44034811:10
patchbotpatch 440348 - requirements - Add virtualbmc to requirements and constraints11:10
pas-haoops, local requirements were too old, rebasing...11:11
*** sdake has joined #openstack-ironic11:12
*** phuongnh has quit IRC11:13
*** vsaienko has quit IRC11:17
openstackgerritVasyl Saienko proposed openstack/ironic master: Fix updating node.driver to classic  https://review.openstack.org/43902611:20
pas-halucasagomes: what's your github handle? I'm looking for original virtualbmc code as of 0.0.4 release to check requirements11:22
lucasagomespas-ha, umago11:23
pas-hathanks11:23
lucasagomesgithub.com/umago11:23
pas-hafound it, was there any particular commit for 0.0.4? (there's no tag :( )11:24
*** hshiina has joined #openstack-ironic11:24
lucasagomespas-ha, hmm don't remember, you need to know which version of pyghmi was used at the time ?11:28
pas-hakind of. latest commit in your repo says >=0.9.8, and in g-r/mitaka we have >=0.9.9, u-c==0.9.9, and this combination passes our mitaka gate with pxe_impitool driver enforced and virtualbmc pinned to ==0.0.4, so I presume it is working11:30
*** yufei has joined #openstack-ironic11:31
openstackgerritMerged openstack/ironic-specs master: Remove APIImpact flag instruction  https://review.openstack.org/43985111:32
*** jkilpatr has quit IRC11:34
openstackgerritGalyna Zholtkevych proposed openstack/ironic master: TEST ipa log messages when failure  https://review.openstack.org/44037211:39
*** vmud213 has quit IRC11:42
openstackgerritOpenStack Proposal Bot proposed openstack/bifrost master: Updated from global requirements  https://review.openstack.org/44003111:42
openstackgerritOpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements  https://review.openstack.org/44004911:45
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/44005011:45
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-lib master: Updated from global requirements  https://review.openstack.org/44005111:45
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent master: Updated from global requirements  https://review.openstack.org/44005211:45
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui master: Updated from global requirements  https://review.openstack.org/44005311:45
*** yufei has quit IRC11:47
*** sdake has quit IRC11:49
*** sdake has joined #openstack-ironic11:50
*** jkilpatr has joined #openstack-ironic11:51
*** links has quit IRC11:51
*** lucasagomes is now known as lucas-brb11:52
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient master: Updated from global requirements  https://review.openstack.org/44009811:54
openstackgerritOpenStack Proposal Bot proposed openstack/python-oneviewclient master: Updated from global requirements  https://review.openstack.org/38994211:55
pas-hadtantsur: so, the vbmc not in g-r also affects stable/newton voting grenade job, as it first installs mitaka, where again vbmc 1.0.0 + pyghmi 0.9.9 are installed, and those are incompatible11:55
*** sdake_ has joined #openstack-ironic11:55
openstackgerritGalyna Zholtkevych proposed openstack/ironic master: TEST ipa log messages when failure  https://review.openstack.org/44037211:55
*** sdake has quit IRC11:56
*** jmccarthy has joined #openstack-ironic11:57
pas-hastrike my last comment, this is only when pxe_impitool is enforced, which is the case only for my test patches11:57
openstackgerritOpenStack Proposal Bot proposed openstack/virtualbmc master: Updated from global requirements  https://review.openstack.org/44012511:57
*** jkilpatr has quit IRC11:58
dtantsurpas-ha, ok, but we have to fix it, right?12:00
pas-habut looks we have to fix it anyway for mitaka's ironic-lib12:00
pas-haexactly12:00
*** links has joined #openstack-ironic12:04
*** ralonsoh_ has joined #openstack-ironic12:06
*** e0ne has joined #openstack-ironic12:06
*** sdake_ is now known as sdake12:09
*** ralonsoh has quit IRC12:09
*** jkilpatr has joined #openstack-ironic12:12
openstackgerritAnton Arefiev proposed openstack/ironic-inspector master: Fix valid provision states for introspection  https://review.openstack.org/43865112:13
*** awiddersheim has quit IRC12:17
openstackgerritVasyl Saienko proposed openstack/ironic master: Fix updating node.driver to classic  https://review.openstack.org/43902612:17
*** athomas has quit IRC12:18
*** dtantsur is now known as dtantsur|bbl12:21
openstackgerritAparna proposed openstack/ironic-python-agent master: Support to collect logs using file name  https://review.openstack.org/44049512:22
*** athomas has joined #openstack-ironic12:29
openstackgerritAparna proposed openstack/ironic master: Support to collect logs using file name  https://review.openstack.org/44050212:29
*** sdake_ has joined #openstack-ironic12:29
*** sdake has quit IRC12:30
openstackgerritKyrylo Romanenko proposed openstack/ironic master: Add functional api tests for node resource class  https://review.openstack.org/43796312:32
openstackgerritJaganathan Palanisamy proposed openstack/ironic-python-agent master: NUMA-topology collector  https://review.openstack.org/42472912:36
*** Nisha_Agarwal has joined #openstack-ironic12:37
*** tuanluong has quit IRC12:40
*** sdake has joined #openstack-ironic12:59
*** bfournie has quit IRC13:01
*** sdake_ has quit IRC13:01
*** aparnav has quit IRC13:02
openstackgerritMarkos Chandras proposed openstack/bifrost master: playbooks: roles: bifrost-ironic-install: Verify checksum for IPA files  https://review.openstack.org/43993813:04
TheJuliagood morning13:05
TheJuliahmmm13:06
Nisha_AgarwalTheJulia, good morning13:07
TheJuliaGood morning13:07
openstackgerritSam Betts proposed openstack/ironic-inspector master: [WIP] Put logic in place for using new Ironic network simulation  https://review.openstack.org/42128813:10
openstackgerritMerged openstack/ironic-ui master: Update test requirement  https://review.openstack.org/44023013:12
openstackgerritMarkos Chandras proposed openstack/bifrost master: playbooks: roles: bifrost-ironic-install: Verify checksum for IPA files  https://review.openstack.org/43993813:15
*** sdake_ has joined #openstack-ironic13:16
*** sdake has quit IRC13:19
openstackgerritGalyna Zholtkevych proposed openstack/ironic master: TEST ipa log messages when failure  https://review.openstack.org/44037213:21
openstackgerritBruno Cornec proposed openstack/ironic-specs master: Propose Redfish support into Ironic  https://review.openstack.org/18465313:22
*** dprince has joined #openstack-ironic13:26
*** trown|outtypewww is now known as trown13:26
*** bfournie has joined #openstack-ironic13:31
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic stable/mitaka: WiP Mind global requirements for virtualbmc  https://review.openstack.org/44055913:34
jrollmorning y'all13:37
vdrokmorning TheJulia and jroll13:39
*** chlong_ has joined #openstack-ironic13:40
jrollvsaienk0: jlvillal: just a heads up: I'm trying to set up a devstack where ironic doesn't run on the primary, probably some WIP changes coming today13:40
vsaienk0jroll o/13:42
openstackgerritVasyl Saienko proposed openstack/ironic-specs master: Dynamic port groups support  https://review.openstack.org/41500313:43
*** lucas-brb is now known as lucasagomes13:44
*** ralonsoh_ has quit IRC13:44
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-lib stable/mitaka: DNM test the virtualbmc from g-r fix  https://review.openstack.org/44056213:45
*** milan has joined #openstack-ironic13:45
milanmorning Ironic! :)13:45
openstackgerritVasyl Saienko proposed openstack/ironic-specs master: Dynamic port groups support  https://review.openstack.org/41500313:46
*** sayantan_ has joined #openstack-ironic13:48
jmccarthyI have a question, in env-setup.sh, ansible stable-2.1 is in use, ANSIBLE_PIP_VERSION=${ANSIBLE_PIP_VERSION:-${ANSIBLE_GIT_BRANCH:-stable-2.1}}13:49
jmccarthyIs there a reason or could this be bumped up to stable-2.2 ?13:49
*** vikrant has quit IRC13:49
*** aolwas has quit IRC13:51
jmccarthy(I mean env-setup.sh in scripts, bifrost repo)13:51
*** jaganathan_afk has quit IRC13:54
jrolljmccarthy: historically ansible minor versions love to break bifrost, so that's likely why it's pinned that way13:55
jrollsurely patches are welcome if tests pass and it otherwise appears to work on 2.2 :)13:56
mat128I confirm the above ^13:56
mat128ansible minors are more like majors in other projects13:56
*** sasha2 has joined #openstack-ironic14:00
TheJuliajmccarthy: I've had reports that stable-2.2 works, but I've never personally tested it14:01
*** jjohnson2 has joined #openstack-ironic14:01
*** causten has joined #openstack-ironic14:01
*** causten has quit IRC14:02
*** causten has joined #openstack-ironic14:02
*** Guest66385 has quit IRC14:03
*** rloo has joined #openstack-ironic14:06
*** ralonsoh has joined #openstack-ironic14:06
*** jaypipes has quit IRC14:08
*** cdearborn has joined #openstack-ironic14:08
*** jcoufal has joined #openstack-ironic14:09
*** trown is now known as trown|brb14:12
*** links has quit IRC14:12
jmccarthyOk, I'm trying to test out some stuff myself to see how I go - shall I fire up a review anyway and see what happens ?14:13
jrolljmccarthy: can't hurt :)14:14
jmccarthy:) Sounds like a plan14:14
*** crushil has joined #openstack-ironic14:17
TheJuliaThat is actually a fairly good way to do it :)14:18
TheJuliaHey, I noticed for tiny ipa we now have sha256 files.  Are we planing on doing the same with coreos builds?14:18
*** trown|brb is now known as trown14:22
*** dtantsur|bbl is now known as dtantsur14:22
dtantsurmorning TheJulia, jroll, vdrok, mat12814:22
dtantsurand milan14:23
mat128hi14:23
milanmorning dtantsur, jroll, mat128, TheJulia, vdrok! :)14:23
milanand everyone else!14:23
jmccarthyOk went ahead with this here to get this ball rolling: https://review.openstack.org/440588 Update ansible version in env-setup.sh14:24
patchbotpatch 440588 - bifrost - Update ansible version in env-setup.sh14:24
*** baoli has joined #openstack-ironic14:24
jmccarthy(back asap have to run errand)14:24
*** ppiela has joined #openstack-ironic14:26
hwoarangTheJulia: i can submit a similar patchset for coreos builds too14:31
TheJuliahwoarang: that would be totally awesome if you would14:32
pas-hajmccarthy: please don't do it right now..14:32
* pas-ha should have removed this 'ansible_branch' thing after ocata release..14:33
pas-haor should we wait for 3 months...14:33
*** e0ne has quit IRC14:33
*** [1]cdearborn has joined #openstack-ironic14:33
*** cdearborn has quit IRC14:34
vdrokmorning dtantsur and milan :)14:35
pas-hajmccarthy: it won't install actual stable-2.2 branch, instead it will pip-install  <2.3, which currently is 2.2.1 which might have some compat issues with ironic due to mismatch in Jinja2 versions required14:35
*** dnikishov has joined #openstack-ironic14:35
*** dnikishov is now known as Guest2599114:35
jrollO_o14:35
*** e0ne has joined #openstack-ironic14:36
pas-hajroll: why? :)14:36
mariojvgood morning ironic14:37
pas-hamorning mariojv14:37
rloohi and good morning pas-ha, mariojv, jroll, vdrok, TheJulia, hwoarang, jmccarthy, mat128, jroll14:37
jrollpas-ha: "hi, use 2.2 stable please" doesn't mean that?14:37
vdrokmorning rloo14:38
*** jaypipes has joined #openstack-ironic14:38
dtantsurmorning rloo14:38
openstackgerritLucas Alvares Gomes proposed openstack/ironic-specs master: Propose Redfish support into Ironic  https://review.openstack.org/18465314:39
rloodtantsur: when i tried to write a spec for an RFE in python-ironicclient launchpad, the unit test fails cuz it wants the bug url to start with https://bugs.launchpad.net/ironic/+bug/. i think that's why i thought all rfes had to be in 'ironic'.14:39
rloodtantsur: do we allow rfe's in all the ironic (sub)projects?14:39
milanmorning rloo! :)14:40
pas-hadtantsur: about ironic-lib, has anyone made a patch to project-config to disable doc job for mitaka?14:40
dtantsurpas-ha, no, not yet. I started diving into it, then figured out that we use virtualbmc, and dropped it14:40
milanmorning mariojv and pas-ha! :D lot's of good morning today! ;)14:40
dtantsurrloo, ironicclient-only RFE are rare things, but we can fix the test to accept anything with "ironic" in its name14:41
rloodtantsur: that's what i was wondering. that'll exclude bifrost but i think that is fine?14:41
dtantsurdunno if bifrost does specs at all :) TheJulia?14:41
dtantsurmilan, these goodmornings are a big part of our culture :)14:41
TheJuliaBifrost does not do specs14:42
jrollcould just test for bugs.launchpad.net14:42
*** [1]cdearborn has quit IRC14:42
pas-haI have a patch fixing the ironic-lib/mitaka coverage job too (might also consider applying the use-u-c cherrypick)14:42
jrollthe project name doesn't mean much anyway14:42
dtantsuralso true14:42
milandtantsur, and I like that! I'm not complaining :D14:42
rlooTheJulia: ok, good. if it ever does and the specs are in ironic-specs, we can update the test.14:42
pas-hathis going to be a spaghetti  of dependent patches :)14:42
rloojroll: i could do that too...14:42
TheJuliarloo: wfm14:42
jrollfor example, this link works, even though it isn't an ironic bug https://bugs.launchpad.net/ironic/+bug/1234514:42
openstackLaunchpad bug 12345 in isdnutils (Ubuntu) "isdn does not work, fritz avm (pnp?)" [Medium,Fix released] - Assigned to Matthias Klose (doko)14:42
dtantsurpas-ha, stable branch stuff is never easy.. thanks for tracking it!14:42
dtantsurjroll, OUCH14:43
*** [1]cdearborn has joined #openstack-ironic14:43
jrolldtantsur: it's a feature (because multiple projects on a bug)14:43
jrolla weird feature, but a feature anyway14:43
rloojroll: heh, gotta love those features :). ok, i'll go with your suggestion.14:43
milanoh, and morning aarefiev whom I've just -1 ed :P14:47
*** jaypipes has quit IRC14:48
openstackgerritLucas Alvares Gomes proposed openstack/ironic-specs master: Pin the python version for tox -edocs  https://review.openstack.org/44060414:49
aarefievmilan: hey, I'm pretty sure it's there :)14:50
openstackgerritJohn L. Villalovos proposed openstack/ironic master: WIP/DNM: Testing multi-tenant multi-node grenade job  https://review.openstack.org/42214914:50
milanaarefiev, /me blind maybe :D14:50
jristlucasagomes: do you have core on heat?14:50
jristnvermind14:51
lucasagomesjrist, hi there, no14:51
jrist!14:51
lucasagomesheh14:51
jristlucasagomes: saw your beard14:51
jristit's awesome14:51
jrist10/10 would beard14:51
lucasagomeslol thanks man14:51
jristfrom the PTG pics14:51
jristhaha14:51
dtantsurLOL14:52
lucasagomeshah yeah, I don't even know how I would look like without it to be honest14:52
* lucasagomes think it's easier to have a beard than having to shave every two days14:53
milanaarefiev, right you are! :D +w14:53
aarefievmilan: I've found, line 131 in svg :)14:53
*** jaypipes has joined #openstack-ironic14:54
milanlol aarefiev, /me not patient enough to read svg :D14:54
aarefiev:)14:54
jroll+1 for lucasagomes beard14:54
openstackgerritLucas Alvares Gomes proposed openstack/ironic master: Add redfish driver  https://review.openstack.org/43898214:54
lucasagomes:D14:54
jristlucasagomes: +10014:55
openstackgerritKyrylo Romanenko proposed openstack/ironic master: Add functional API tests for portgroups  https://review.openstack.org/43687814:55
milanlucasagomes, bwt, mind mailing pixie over to me? I'd take him to Iceland  next week ;)14:56
lucasagomesmilan, next week ? Sure I can post to you tomorrow or saturday14:56
lucasagomesmilan, send me ur addr in PM14:56
milanlucasagomes, thx! :)14:56
lucasagomeswe need to create the instagram thingy14:56
milandtantsur, ^ :P14:57
* milan mails pixie to vdrok after, that was the plan, correct?14:58
dtantsurlol, a bit later14:58
dtantsurI need to finish boring stuff first14:58
openstackgerritRuby Loo proposed openstack/ironic-specs master: Allow spec to have RFE from any ironic project  https://review.openstack.org/44061014:58
*** baoli has quit IRC14:59
*** dyasny has joined #openstack-ironic15:00
openstackgerritKyrylo Romanenko proposed openstack/ironic master: Add functional API tests for portgroups  https://review.openstack.org/43687815:01
rloomilan, lucasagomes: i think it would be more challenging if pixie was not mailed, but handed over to people :)15:01
rloolucasagomes: wrt https://review.openstack.org/#/c/440604/, that's a stop gap for now. is there a bug to fix it for py 3?15:02
patchbotpatch 440604 - ironic-specs - Pin the python version for tox -edocs15:02
lucasagomesrloo, heh yeah definitely more challenging, we will do both :-)15:02
lucasagomesrloo, I think it's one of the dependencies it installs that doesn't work with python315:02
lucasagomesthat's why I just thought of pinning it15:02
rloolucasagomes: i suspect it will need to be fixed at some point. we should open a bug15:03
lucasagomesrloo, feedformatter (/me googles about it's python3 suppport)15:03
rloolucasagomes: we == you :)15:03
* milan 'd love to see rloo traveling over to my place to pick up pixie :D15:03
lucasagomesrloo, could be, lemme do it15:03
lucasagomesrloo, https://pypi.python.org/pypi/feedformatter/ = Programming Language :: Python :: 2 :: Only15:03
lucasagomes:-(15:03
* lucasagomes opens a bug anyway15:04
openstackgerritMarkos Chandras proposed openstack/ironic-python-agent master: imagebuild: coreos: Build sha256 checksum files  https://review.openstack.org/44061415:04
hwoarangTheJulia: ^15:04
vdrokmilan: yah, tho I'll need it in mid-april :)15:04
*** mtanino has joined #openstack-ironic15:04
openstackgerritKyrylo Romanenko proposed openstack/ironic master: Add negative functional API tests for portgroups  https://review.openstack.org/43753515:05
openstackgerritJulia Kreger proposed openstack/bifrost master: Make ironic database use more configurable  https://review.openstack.org/44061615:06
TheJuliaNobodyCam: ^^^ I know you've wanted that for a while :)  cross your fingers! :)15:07
*** zackf has joined #openstack-ironic15:07
openstackgerritRuby Loo proposed openstack/ironic-specs master: Allow spec to have RFE from any ironic project  https://review.openstack.org/44061015:07
milanvdrok, ack, actually I've got another traveling plan for the first week of April (back on 8th) so Pixie could have more fun if he manages to get to your place in one week ;)15:07
openstackgerritLucas Alvares Gomes proposed openstack/ironic-specs master: Pin the python version for tox -edocs  https://review.openstack.org/44060415:08
lucasagomesrloo, ^ (the bug is linked in the ocmmit message)15:08
rloomilan: no plans to go to brno in the near or far future. see you in canada :)15:08
rloothx lucasagomes!15:09
TheJuliahwoarang: For some reason, your IRC handle never matches to your first name in my head. :)15:09
* rloo wonders what hwoarang's first name is...15:09
hwoarangit's on purpose so i can hide when i break stuff :)15:09
milanrloo, right :D15:09
*** baoli has joined #openstack-ironic15:10
openstackgerritMarkos Chandras proposed openstack/bifrost master: playbooks: roles: bifrost-ironic-install: Verify checksum for IPA files  https://review.openstack.org/43993815:10
TheJuliahwoarang: crafty!15:11
rloosambetts: you around? wanted to ask what you meant by L134 in https://etherpad.openstack.org/p/ironic-pike-ptg-operations15:13
*** crushil has quit IRC15:15
openstackgerritMarkos Chandras proposed openstack/bifrost master: playbooks: roles: bifrost-ironic-install: Verify checksum for IPA files  https://review.openstack.org/43993815:15
*** crushil has joined #openstack-ironic15:15
sambettsrloo: https://bugs.launchpad.net/ironic/+bug/165563315:16
openstackLaunchpad bug 1655633 in Ironic "[RFE] Enhance /drivers/properties API to have machine readable detail view " [Wishlist,Confirmed] - Assigned to Madhuri Kumari (madhuri-rai07)15:16
*** sayantan_ has quit IRC15:17
*** sayantan_ has joined #openstack-ironic15:17
rloosambetts: yes, we had discussed that a bit.15:18
*** rbartal has quit IRC15:18
*** baoli has quit IRC15:18
rloosambetts: how does that relate to the OSC commands... driver property list <driver> is preferred by you?15:19
rloosambetts: so that it just lists the names of the properties?15:19
jmccarthypas-ha: Hmm ok I see your comment re:stable-2.2, I tried myself a bit locally but got some strange errors .. not sure how to debug the errors OPNFV CI flagged against the review15:19
rloosambetts: and driver property show <driver> <option> is what?15:20
*** sayantan_ has quit IRC15:20
*** sayantan_ has joined #openstack-ironic15:20
sambettsrloo: driver property show <driver> <name> would return { description: <description>,15:20
TheJuliajmccarthy: hwoarang might have insight regarding opnfv ci failures15:21
sambetts                     required: <true|false>,15:21
sambetts                     depends-on: [<another property name>]15:21
hwoaranglets see15:21
rloosambetts: so <name> would be the name of a property?15:21
sambettsrloo: yup15:21
*** amarao has quit IRC15:22
rloosambetts: then we'd probably need a --detailed or whatever for the list15:22
sambettsrloo: was lkiterally just typing that :-P15:23
*** hafe has joined #openstack-ironic15:23
jmccarthyhwoarang: Are you taking a peek at this one here ? https://review.openstack.org/#/c/440588/15:23
patchbotpatch 440588 - bifrost - Update ansible version in env-setup.sh15:23
rloosambetts: and we'd have 'driver property', and 'driver raid property (or driver raid-logical-disk-property)' as objects. ugh.15:23
hwoarangjmccarthy: yep but i believe that the new ansible just exposed an underlaying issue or something15:23
hwoarangbecause everything works with 2.115:23
rloosambetts:  i had been wondering whether there might be value in filtering the properties via interface-type.15:24
rloosambetts: i started https://review.openstack.org/#/c/439907/ and will incorporate the above...15:24
patchbotpatch 439907 - ironic-specs - OSC commands for ironic driver-related commands15:24
sambettsrloo: as in driver property list --interface=<inter>15:24
rloosambetts: yeah, something like that. was thinking it might help with the raid-logical-disk-properties but i don't htink it will.15:25
rloosambetts: i don't like having raid-logical-disk-properties as its own object. but...15:25
jmccarthyhwoarang: Hrm :/ Related to what pas-ha mentioned is it ? (Issue of jinja template or ?)15:25
hwoarangnot quite sure i need to investigate a bit15:27
jmccarthyOk .. maybe 2.2 just isn't quite there yet .. ah well to get green lights all around was too much too hope for :D15:27
sambettsrloo: when you say its own object what do you mean ?15:27
*** sayantan_ has quit IRC15:27
*** sayantan_ has joined #openstack-ironic15:28
rloosambetts: sorry, i mean 'an object'. in osc language. 'baremetal driver property' would be an object, as would 'baremetal driver raid logical disk property'.15:28
pas-hajmccarthy: there was a bug in Jinja2 2.9[0-4], while it was being fixed, ansible have put "jinja2<2.9" into latest stable releases (2.1.4 and 2.2.1), meanwhile the bug was fixed in jinja2 2.9.5, and now openstack's global reqs have upper constraint Jinja2===2.9.5 which is being installed for ironic and is incompatible with ansible 2.1.4 or 2.2.1...15:28
pas-habut honestly it shouldn't be the case, at least on gates15:29
rloosambetts: or something like that. i would have liked to get access to the raid stuff via whatever object (in this case baremetal driver property) but not sure it would make sense.15:29
pas-haor may be it is... not sure. we will have to investigate I guess15:29
openstackgerritSofiia Andriichenko proposed openstack/ironic master: Add tempest plugin API tests for node  https://review.openstack.org/41119715:31
*** hshiina has quit IRC15:33
hwoarangjmccarthy: the issue seems related to templating it seems. for whatever reason is trying to use virtualenv but the 'virtualenv' option in the pip module is empty at that point. so this shouldn't happen15:34
hwoarangvery strange15:34
*** amarao has joined #openstack-ironic15:35
*** mjturek has joined #openstack-ironic15:35
Nisha_AgarwalIf i get this error "DriverNotFoundInEntrypoint: Could not find the following items in the 'ironic.hardware.types' entrypoint: ipmi" while starting conductor, what is missing?15:36
jrollNisha_Agarwal: check if it's in setup.cfg, like so: https://github.com/openstack/ironic/blob/master/setup.cfg#L14115:37
jrollNisha_Agarwal: if it is, then make sure you've installed the code, not just done a git pull (e.g. pip install .)15:38
Nisha_Agarwaljroll, its there is setup,cfg15:38
Nisha_Agarwali have latest ironic15:38
Nisha_Agarwali did "sudo python setup.py install" many times15:38
Nisha_Agarwalbut i still get this error15:38
Nisha_Agarwali am not sure what is missing15:38
jrollwe use "sudo pip install ." in openstack, try that15:38
Nisha_Agarwalok let me try15:39
TheJuliaWhat jroll said, we're not supposed to interact with setup.py directly in openstack15:39
*** baoli has joined #openstack-ironic15:39
openstackgerritVasyl Saienko proposed openstack/ironic-specs master: Dynamic port groups support  https://review.openstack.org/41500315:40
Nisha_AgarwalTheJulia, ok. i am trying as jroll suggested15:40
openstackgerritVasyl Saienko proposed openstack/ironic-specs master: Dynamic port groups support  https://review.openstack.org/41500315:41
*** rama_y has joined #openstack-ironic15:41
lucasagomesNisha_Agarwal, did you update the ironic.conf ?15:43
Nisha_Agarwallucasagomes, i updated it to have ironic_hardware_types15:43
*** baoli has quit IRC15:44
openstackgerritMerged openstack/python-ironic-inspector-client master: Updated from global requirements  https://review.openstack.org/44009715:44
Nisha_Agarwaland other default/enabled interfaces15:44
lucasagomesNisha_Agarwal, is ipmi listed there ?15:44
Nisha_Agarwalyes15:44
lucasagomesstrange... ok15:44
openstackgerritMarkos Chandras proposed openstack/ironic-python-agent master: imagebuild: coreos: Build sha256 checksum files  https://review.openstack.org/44061415:44
rlooNisha_Agarwal: are you using master? lucasagomes, jroll: 'ipmi' hardware type should work w/o doing anything extra?15:46
Nisha_Agarwalrloo, yes15:46
rloojroll: don't we have any tests in gate that use 'ipmi' hardware type?15:47
Nisha_Agarwallucasagomes, yeah its strange but somehow its not working at all :(15:47
*** rpioso has joined #openstack-ironic15:47
rlooNisha_Agarwal: so the conductor doesn't start up?15:47
Nisha_Agarwalrloo, yeah the conductor fails to start15:47
jrollrloo: if installed properly, correct. and we load it by default, so every CI job tests this.15:49
openstackgerritDmitry Tantsur proposed openstack/ironic-specs master: Add priorities for the Pike cycle  https://review.openstack.org/43971015:49
dtantsur\o/15:49
*** vsaienko has joined #openstack-ironic15:50
jrollrloo: Nisha_Agarwal's error implies this isn't installed properly, there's almost no other way this could be broken (except maybe a "corrupted" setup.cfg)15:50
rloojroll: that's what i thought15:50
* jroll continues to wait for Nisha_Agarwal to pip install and run15:50
Nisha_Agarwaljroll, then it should work :(15:50
Nisha_Agarwaljroll, it still fails15:50
jrollNisha_Agarwal: same error?15:50
rlooNisha_Agarwal: is this a development environment?15:50
rlooNisha_Agarwal: did you set things up properly, eg: #2 of Step 1: https://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#exercising-the-services-locally15:51
Nisha_Agarwalrloo, yes this is development env....but i stacked it two days back only15:52
*** rama_y has quit IRC15:52
jrollNisha_Agarwal: also, are there any patches on top?15:52
jrollNisha_Agarwal: was it working two days ago? what changed when it stopped?15:52
*** rajinir_ has joined #openstack-ironic15:52
rlooNisha_Agarwal: maybe just start clean and redo your dev environment.15:52
lucasagomesrloo, I'm in my devstack setup (built on monday)15:52
Nisha_Agarwaljroll, no....initially i had added lucasagomes redfish driver patch...but when conductor didnt start i removed that patch and switched to master to bring up the conductor15:53
jrollwell, something went wrong in there15:53
*** athomas has quit IRC15:53
Nisha_Agarwalrloo, looks like i have to restack15:54
* Nisha_Agarwal restacks15:54
rlooNisha_Agarwal: always better to restack to see if there is a problem with your environment first, before asking for help15:55
lucasagomesNisha_Agarwal, but you changed the setup.cfg for redfish ? (I'ts not needed btw, you just need to update the ironic.conf to enable the hardware type)15:55
Nisha_Agarwalrloo, i had freshly restacked it this week only15:55
Nisha_Agarwallucasagomes, i just cherry-picked ur patch and did python install15:56
*** ccamacho has quit IRC15:56
rlooNisha_Agarwal: and it worked earlier this week and you mucked around with it and it doesn't work now. So...15:56
lucasagomesI see, yeah strange15:56
Nisha_Agarwalrloo, no i never tested with new stack15:56
Nisha_Agarwalrloo, but yes the conductor was up this week before i had the patch15:57
Nisha_Agarwalanyway i will restack as it shouldnt happen15:57
lucasagomescheck if the cherry-pick didn't conflict to, because I'm adding stuff to setup.cfg in the redfish patch15:57
Nisha_Agarwal:(15:57
rlooNisha_Agarwal: perhaps that might be a good thing to do in the future, to make sure things work before you start mucking with it15:57
* lucasagomes guess15:57
rlooNisha_Agarwal: otherwise, how do you know what broke?15:57
Nisha_Agarwalrloo, true....15:58
Nisha_Agarwalanyway i will update after restacking15:58
*** rcernin has quit IRC15:59
*** athomas has joined #openstack-ironic16:00
*** chlong_ has quit IRC16:01
*** baoli has joined #openstack-ironic16:01
*** chlong__ has joined #openstack-ironic16:01
*** rook is now known as rook-afk16:04
*** yolanda has joined #openstack-ironic16:05
*** baoli has quit IRC16:05
openstackgerritVasyl Saienko proposed openstack/ironic master: Allow to attach/detach VIFs to active ironic nodes  https://review.openstack.org/42472316:10
dtantsurmariojv, gerrit won't allow me to add to a review, so here: https://review.openstack.org/#/c/439710/16:11
patchbotpatch 439710 - ironic-specs - Add priorities for the Pike cycle16:11
mariojvthanks dtantsur16:11
mariojvi don't know why that happens, jlvillal pointed it out to me earlier, not sure how to fix16:11
openstackgerritVasyl Saienko proposed openstack/ironic master: Allow to attach/detach VIFs to active ironic nodes  https://review.openstack.org/42472316:11
*** sayantan_ has quit IRC16:13
*** sayantan_ has joined #openstack-ironic16:13
*** vsaienko has quit IRC16:14
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-lib stable/mitaka: Use constraints for all the things  https://review.openstack.org/44066316:15
*** Syed__ has joined #openstack-ironic16:16
jmccarthyOk maybe there is a different angle, really what I'd like to avoid in bifrost is having to fetch the os_keystone_service module from https://raw.githubusercontent.com/ansible/ansible-modules-extras/stable-2.2/cloud/openstack/os_keystone_service.py in the env-setup.sh script.16:16
jmccarthyPossibly these could be refactored ?16:16
jmccarthyplaybooks/roles/bifrost-ironic-install/tasks/keystone_setup.yml:  os_keystone_service:16:16
jmccarthy- name: "Create keystone service record for ironic"16:16
jmccarthyplaybooks/roles/bifrost-ironic-install/tasks/keystone_setup_inspector.yml:  os_keystone_service:16:16
jmccarthy- name: "Create keystone service record for ironic-inspector"16:16
*** mjura has quit IRC16:17
*** baoli has joined #openstack-ironic16:18
mariojvthis is so messed up - so i think there are 2 gerrit accounts under my name, but i can no longer sign into one of them16:19
mariojvwhen you go to add a reviewer, 2 entries for mario.villaplana at gmail dot com show up16:20
mariojvboth don't allow you to add me16:20
*** [1]cdearborn has quit IRC16:20
mariojvadding mario.villaplana at rackspace dot com works16:20
mariojvbut that's not my real account16:20
jrollinfra could fix if they have free time16:20
mariojvand sending a password reset to mario.villaplana at rackspace just makes me log into my real one (gmail)16:20
*** cdearborn has joined #openstack-ironic16:21
jlvillalmariojv: What did I point out? Oh adding you to a review. I remember. "Well played mariojv, well played" ;)16:23
mariojvi promise it wasn't on purpose lol16:23
jlvillalmariojv: Maybe infra folks can help you out?16:23
mariojvyeah, i'm going to ask in a bit16:24
*** hafe has quit IRC16:24
jlvillalvsaienk0: jroll: As a note, I have some some scripts that I have hacked together to create a pseudo devstack-gate environment and it can setup a multi-node one.16:24
jlvillalDepends on having an OpenStack cloud to run on.16:24
jlvillalAs it uses the openstack client to create the VMs and such.16:25
* jlvillal goes to get some libations16:25
*** karthiks has quit IRC16:25
*** baoli has quit IRC16:26
*** baoli has joined #openstack-ironic16:26
jrolljlvillal: yeah I'm doing some similar16:26
*** baoli has quit IRC16:27
*** baoli has joined #openstack-ironic16:28
vsaienk0jlvillal: I didn't found other way how to solve problem with second plugin enablement than https://review.openstack.org/#/c/44030316:31
patchbotpatch 440303 - openstack-dev/devstack - Ignore second enable_plugin request16:31
*** trown is now known as trown|lunch16:32
jlvillalvsaienk0: That was my first way of doing the change to devstack. As I added the die if 2nd request. Initially I did ignore but they asked me to make it die...16:32
*** bcornec2 has quit IRC16:33
*** baoli has quit IRC16:33
*** ElCoyote_ has left #openstack-ironic16:36
*** makowals has quit IRC16:36
jlvillalvsaienk0: FYI: https://review.openstack.org/#/c/396794/16:36
patchbotpatch 396794 - openstack-dev/devstack - If plugin is enabled multiple times fail (MERGED)16:36
TheJuliajmccarthy: The downside of not using the module if memory serves is the openstack cli just keeps creating, there has to be logic to check presence and then re-add  Any specific reason behind wanting to avoid that?16:37
*** cdearborn has quit IRC16:37
*** cdearborn has joined #openstack-ironic16:38
vsaienk0https://review.openstack.org/#/c/396794/ I've opened a bug might be devstack folks can help on that, because I have no clue why it is happening16:38
patchbotpatch 396794 - openstack-dev/devstack - If plugin is enabled multiple times fail (MERGED)16:38
vsaienk0jlvillal https://bugs.launchpad.net/devstack/+bug/1669509 ^16:38
openstackLaunchpad bug 1669509 in devstack "ironic grenade multinode try to enable_plugins second time during dumping databases" [Undecided,New]16:38
*** amarao has quit IRC16:41
jrolldo we set -e in devstack? O_o16:41
JayFyes, I believe so16:41
jrollhow would I do net_id=$(openstack network show foo); if [[ $net_id == "" ]]16:41
* jroll hrm16:41
JayFan old shell trick was like16:42
JayFnet_id=x16:42
jmccarthyTheJulia: Just that it means fetching a module from an external source that we don't mirror internally or anything like that16:42
JayFif [[$net_id -eq "x"]]16:42
*** pcaruana has quit IRC16:42
*** cdearborn has quit IRC16:42
TheJuliaahhhh16:42
jrollJayF: to be clear, it's the osc command that blows up16:42
JayFugh16:42
jrollbut I might need that too16:42
JayFnet_id=$(command || true)16:43
jrolloh, duh16:43
JayFjust add an || true to the end of commands you're "OK" with failing16:43
jrollya16:43
jrolltoo obvious16:43
jrollthanks JayF16:43
TheJuliajmccarthy: I feel like (but would need to dig to confirm) that we refectored OSC commands out with using the module, so potentially could be changed16:44
*** cdearborn has joined #openstack-ironic16:44
*** baoli has joined #openstack-ironic16:44
*** makowals has joined #openstack-ironic16:45
*** baoli has quit IRC16:47
mariojvalright, folks should be able to add me to reviews now16:47
mariojvinfra did some wizardry16:47
jmccarthyTheJulia: I was hoping the bump to 2.2 would be a quick fix :) Actually I just realised I may have another angle that will work for me - I will go and investigate that and see if that works16:47
dtantsur\o/16:47
* jroll lunches16:47
jrollgo devstack go!16:48
TheJuliajmccarthy: fwiw, it looks like bifrost's gate jobs are very broken as of this morning, so don't trust CI at the moment :)16:48
*** cdearborn has quit IRC16:48
* TheJulia noms lunch and will dig into that after lunch16:50
*** cdearborn has joined #openstack-ironic16:51
NobodyCamGood16:52
NobodyCamieek16:52
TheJuliaieek?16:52
NobodyCam(Take #2) Good Morning Ironic'ers16:52
NobodyCamlol16:52
TheJuliaThere will always be good mornings!16:52
NobodyCamtried for shift got enter16:52
joannamorning NobodyCam, TheJulia, dtantsur :)16:53
NobodyCammorning TheJulia, joanna :)16:53
NobodyCamhehehe16:53
*** cdearborn has quit IRC16:54
*** ChubYann has joined #openstack-ironic16:57
*** baoli has joined #openstack-ironic16:58
dtantsurmorning joanna16:59
joanna:)16:59
dtantsurmorning NobodyCam16:59
NobodyCammorning dtantsur :)17:00
*** e0ne has quit IRC17:00
mgouldmorning NobodyCam17:02
NobodyCamMorning mgould :)17:02
*** sdake has joined #openstack-ironic17:02
vsaienk0jlvillal could you please talk with devstack folks about our multinode issue? It looks it is the only one real issue17:03
jlvillalvsaienk0: Okay. I'll see if I can ask dtroyer about it.17:04
vsaienk0jlvillal thanks!17:04
openstackgerritJulia Kreger proposed openstack/bifrost master: Update hacking in test-requirements.txt  https://review.openstack.org/44070217:04
NobodyCammorning jlvillal and vsaienk0 :)17:05
jlvillalHey NobodyCam ! :)17:05
vsaienk0morning NobodyCam17:05
jmccarthyTheJulia: Oh that'd be great if CI decided it was all ok :) !17:05
NobodyCam:)17:05
*** sdake_ has quit IRC17:06
lucasagomesfolks I'm calling it a day17:07
lucasagomeshave a great evening all :-)17:07
jlvillallucasagomes: Ciao!17:07
*** lucasagomes is now known as lucas-afk17:07
lucas-afko/17:07
TheJuliajmccarthy: yeah, the virtualenv pip failure looks totally awesome :(17:07
TheJuliagoodnight lucas-afk17:08
NobodyCammorning and good night lucas-afk :)17:08
NobodyCampas-ha: TheJulia: great stuff on the roadmap ether pad !17:09
pas-haNobodyCam: thanks :)17:09
*** baha has joined #openstack-ironic17:09
milanmorning NobodyCam! :)17:10
NobodyCam:) and Good Morning / evening pas-ha and milan17:10
milan:D17:10
NobodyCam:)17:10
NobodyCamhehehe17:10
*** crushil has quit IRC17:10
*** korekhov has joined #openstack-ironic17:12
milanaarefiev,  dtantsur, sambetts here is the spec for the pxe-filtering backend for inspector: https://review.openstack.org/#/c/439862/ to describe the driver interface; thanks for reviews! ;)17:13
patchbotpatch 439862 - ironic-inspector-specs - Multiple PXE filtering backends17:13
*** rajinir_ is now known as rajinir17:13
rloojoanna: i think i missed something. what's your involvement with rolling upgrades?17:13
milanand with that news, good night Ironic! :P17:14
rloojoanna: wrt https://review.openstack.org/#/c/439710/17:14
patchbotpatch 439710 - ironic-specs - Add priorities for the Pike cycle17:14
openstackgerritJulia Kreger proposed openstack/bifrost master: Make ironic database use more configurable  https://review.openstack.org/44061617:14
*** milan has quit IRC17:14
aarefievmilan: thanks, I'll move my question there17:14
aarefievnight milan17:14
TheJuliaNobodyCam: it is 99.95% pas-ha, then again, it is also stuff that has been publicly discussed for a while, and we're all kind of on the same page which rocks :)17:14
* pas-ha blushes :)17:15
rloojoanna: that reminds me. weren't you going to do a followup patch to the db deadlocking patch? I don't recall seeing it?17:15
NobodyCamit looks like all good stuff! and morning rloo :)17:15
rloohey, morning NobodyCam!17:15
NobodyCam:)17:16
*** korekhov_ has joined #openstack-ironic17:16
*** korekhov has quit IRC17:16
openstackgerritAnnie Lezil proposed openstack/ironic-python-agent master: Collect NIC name given by BIOS  https://review.openstack.org/39640617:17
joannaI'm working on it, rloo :) don't worry, probably will submit <EOD :)17:18
joannarloo: thanks for the reminder! :)17:18
rloojoanna: the point of the followon patches, is to quickly allow the orig patch to be approved. if i have to remember to check that there is a followon, then it makes me want to not approve patches that need a bit more work.17:20
*** korekhov_ has quit IRC17:22
*** makowals has quit IRC17:22
*** korekhov has joined #openstack-ironic17:22
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Mind global requirements for virtualbmc  https://review.openstack.org/44071517:22
*** aolwas has joined #openstack-ironic17:24
openstackgerritSam Betts proposed openstack/ironic master: Grenade source correct Ironic devstack plugin version  https://review.openstack.org/44071717:31
openstackgerritSam Betts proposed openstack/ironic master: Handle multinode VXLAN tunnel inside ironic devstack plugin  https://review.openstack.org/44071817:31
openstackgerritSam Betts proposed openstack/ironic master: Remove extra bridge created for plumbing BM VMs  https://review.openstack.org/44071917:31
openstackgerritSam Betts proposed openstack/ironic master: Rework Ironic devstack baremetal network simulation  https://review.openstack.org/44072017:31
openstackgerritSam Betts proposed openstack/ironic master: Make Ironic devstack simulation less dependent on OVS  https://review.openstack.org/44072117:31
openstackgerritMerged openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/44005017:33
*** ralonsoh has quit IRC17:34
*** makowals has joined #openstack-ironic17:38
*** racedo has quit IRC17:39
JayFrloo: good catch about the dropped things from prio list, and I explicitly had both etherpad and spec up looking for those things and still missed it /o\17:39
rlooJayF: so we didn't mean to drop them? I wasn't sure...17:39
rlooJayF: i figured even if we did, we should note it down somewhere :)17:40
JayFIDK if it was intentional, but I figured if it was it should be called out on spec and commented in review why they were droppped if so17:40
openstackgerritAnnie Lezil proposed openstack/ironic-inspector master: Reapply doesn't update started_at time  https://review.openstack.org/43105317:41
openstackgerritSam Betts proposed openstack/ironic master: Make Ironic devstack simulation less dependent on OVS  https://review.openstack.org/44072117:42
openstackgerritSam Betts proposed openstack/ironic master: Rework Ironic devstack baremetal network simulation  https://review.openstack.org/39295917:42
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui master: Updated from global requirements  https://review.openstack.org/44005317:43
rlooJayF: ++17:44
*** mgould is now known as mgould|afk17:44
*** Nisha_ has joined #openstack-ironic17:45
*** zackf has quit IRC17:46
dtantsuro/17:46
*** dtantsur is now known as dtantsur|afk17:46
*** krtaylor has quit IRC17:46
*** Nisha_Agarwal has quit IRC17:47
*** fragatina has joined #openstack-ironic17:49
NobodyCamnight dtantsur|afk :)17:50
*** rook-afk is now known as rook17:50
*** fragatina has quit IRC17:50
*** vsaienko has joined #openstack-ironic17:51
*** tesseract has quit IRC17:52
*** fragatina has joined #openstack-ironic17:55
*** derekh has quit IRC17:55
*** vsaienko has quit IRC17:55
*** fragatina has quit IRC17:57
*** fragatina has joined #openstack-ironic17:58
*** mgoddard has quit IRC17:58
*** fragatina has quit IRC17:58
*** fragatina has joined #openstack-ironic17:59
*** fragatina has quit IRC17:59
openstackgerritSam Betts proposed openstack/ironic master: Make Ironic devstack simulation less dependent on OVS  https://review.openstack.org/44072118:00
openstackgerritSam Betts proposed openstack/ironic master: Rework Ironic devstack baremetal network simulation  https://review.openstack.org/39295918:00
sambettsNight all o/18:02
*** sambetts is now known as sambetts|afk18:02
*** jaypipes has quit IRC18:12
NobodyCamnight sambetts|afk18:12
jrollhm, why would this give me a 400? openstack network create --provider-network-type vlan --provider-physical-network mynetwork ironic-provision -f value -c id18:15
jrolljlvillal: ever seen something like that18:15
jrolloh maybe that physnet doesn't exist, hm18:15
*** mgoddard has joined #openstack-ironic18:16
jlvillaljroll: I have not seen that before.18:18
jrolljlvillal: yeah, I think that's the problem. thanks for answering :)18:19
*** jaosorior has quit IRC18:19
*** trown|lunch is now known as trown|mtg18:19
*** trown|mtg is now known as trown18:19
jlvillaljroll: heh. That's about all I did...18:19
jrolljlvillal: just saying I appreciate it, even if you couldn't help :P18:20
jlvillaljroll: Thanks! :)18:20
*** fragatina has joined #openstack-ironic18:21
* jlvillal didn't realize what an ordeal it was to add to update upper-constraints.txt ...18:21
jlvillalOr at least how time consuming it was as the program to do it takes a long time to run18:22
*** crushil has joined #openstack-ironic18:25
openstackgerritMerged openstack/python-ironicclient master: Update test requirement  https://review.openstack.org/44022718:29
*** vsaienko has joined #openstack-ironic18:31
pas-hag'night all18:33
*** racedo has joined #openstack-ironic18:35
*** racedo has quit IRC18:42
*** racedo has joined #openstack-ironic18:43
TheJuliagoodnight pas-ha18:43
openstackgerritJulia Kreger proposed openstack/bifrost master: Make ironic database use more configurable  https://review.openstack.org/44061618:43
*** amoralej is now known as amoralej|off18:44
openstackgerritMerged openstack/ironic-lib master: Update test requirement  https://review.openstack.org/44022918:46
*** fragatina has quit IRC18:48
openstackgerritMerged openstack/ironic-lib master: Updated from global requirements  https://review.openstack.org/44005118:48
*** fragatina has joined #openstack-ironic18:49
*** mgoddard has quit IRC18:51
*** mgoddard has joined #openstack-ironic18:52
jkilpatrso I have servers that support virtual media but seem to be using the tftp/ipxe driver, how would I configure them not to?18:52
*** crushil has quit IRC18:53
*** Nisha_ has quit IRC18:53
*** Nisha_ has joined #openstack-ironic18:53
*** crushil has joined #openstack-ironic18:53
*** sdake_ has joined #openstack-ironic18:54
*** baha has quit IRC18:54
*** vsaienko has quit IRC18:54
*** baha has joined #openstack-ironic18:54
*** sdake has quit IRC18:55
openstackgerritJulia Kreger proposed openstack/bifrost master: Documentation cleanup  https://review.openstack.org/43915118:56
openstackgerritMerged openstack/ironic-python-agent master: Updated from global requirements  https://review.openstack.org/44005218:56
rlooTheJulia: didn't realize you were going to update that patch :)18:56
*** Nisha_ has quit IRC18:57
*** Nisha_ has joined #openstack-ironic18:57
TheJuliarloo: *shrugs*  I have a few minutes before I need to jump on a planning call for my trip next week, so a quick fix is easy :)18:58
rlooTheJulia: have a great trip if I don't talk to you before then!18:58
TheJuliajkilpatr: what is the hardware and what driver are you using?18:58
TheJuliaIt will be a long trip. :(  I'm not a fan of flying cross country, but at least the conference is three days, just Monday/Friday end up being largely travel19:00
*** e0ne has joined #openstack-ironic19:00
jkilpatrTheJulia, dell r630's so DRAC and right now I seem  to be using the ipxe driver because it's the default.19:04
*** krtaylor has joined #openstack-ironic19:05
*** vsaienko has joined #openstack-ironic19:06
*** ntpttr_ has quit IRC19:07
TheJuliajkilpatr: so the drac driver actually uses pxe/ipxe (if enabled).  The one that is in tree only supports that path, and does not support media as far as I'm aware.19:07
*** e0ne has quit IRC19:08
*** racedo has quit IRC19:08
*** e0ne has joined #openstack-ironic19:09
*** e0ne has quit IRC19:11
jkilpatrTheJulia, the interface seems to support it, do you know of one out of tree that works? (just curious if it exists, don't think I could use it even if it did)19:11
*** e0ne has joined #openstack-ironic19:13
TheJuliarpioso: ^^^ ?19:14
*** Nisha_brb has joined #openstack-ironic19:15
*** Nisha_ has quit IRC19:15
*** sayantan_ has quit IRC19:19
*** sayantan_ has joined #openstack-ironic19:19
*** sayantan_ has quit IRC19:23
*** sayantan_ has joined #openstack-ironic19:23
*** sdake_ has quit IRC19:24
openstackgerritJim Rollenhagen proposed openstack/ironic master: WIP: hacks for multinode without ironic on primary  https://review.openstack.org/44077519:28
*** ijw has joined #openstack-ironic19:34
*** jkilpatr has quit IRC19:35
*** Nisha_brb has quit IRC19:35
*** jkilpatr has joined #openstack-ironic19:35
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Neutron notifications  https://review.openstack.org/44077819:35
openstackgerritMerged openstack/ironic master: Updated from global requirements  https://review.openstack.org/44004919:37
*** jcoufal has quit IRC19:37
vdrokgood night everyone!19:37
*** jcoufal has joined #openstack-ironic19:38
*** e0ne has quit IRC19:38
crushilgood night vdrok19:50
*** vsaienko has quit IRC19:51
*** jcoufal has quit IRC19:53
openstackgerritMerged openstack/bifrost master: Update hacking in test-requirements.txt  https://review.openstack.org/44070220:12
*** vsaienko has joined #openstack-ironic20:13
mrdaMorning Ironic20:18
*** athomas has quit IRC20:24
*** sayantan_ has quit IRC20:24
*** sayantan_ has joined #openstack-ironic20:24
*** vsaienko has quit IRC20:30
*** vsaienko has joined #openstack-ironic20:33
NobodyCammorning mrda :)20:34
*** sayantan_ has quit IRC20:34
*** sayantan_ has joined #openstack-ironic20:35
*** sayantan_ has quit IRC20:36
*** sayantan_ has joined #openstack-ironic20:36
*** korekhov has quit IRC20:37
mrdao/20:44
*** jkilpatr has quit IRC20:45
*** jkilpatr has joined #openstack-ironic20:45
*** sayantan_ has quit IRC20:49
*** sayantan_ has joined #openstack-ironic20:49
rloomariojv: i looked at the 'ironic-review-party' etherpad, nothing slated for today?20:54
rloohowdy mrda!20:54
rookhas anyone noted the CPU usage from ironic -- and that restarting ironic services "fixes" the usage?20:55
mariojvrloo: I put some stuff down for week of 27th20:56
mariojv(this week)20:56
mariojvi'll be in the hangouts room in a minute20:56
*** crushil has quit IRC20:56
rloomariojv: oh, right. i was thinking it is march now, but it started with end of feb.20:56
TheJuliarook: Was there anything in the logs? Was it the conductor service?  Anything in progress?20:57
mariojvmaybe i should just put the actual date of the review party :P20:57
*** jaypipes has joined #openstack-ironic20:57
rookTheJulia: Yeah, the conductor service. there was stuff ongoing - however some downtime20:57
*** crushil has joined #openstack-ironic20:57
rloomariojv: that would have helped me. i cannot prepare in 15 min to discuss any of that stuff in etherpad, so i'm not going to attend.20:57
rookI would expect CPU usage to dip down during the quiet time20:57
*** sayantan_ has quit IRC20:57
rookhttps://snapshot.raintank.io/dashboard/snapshot/G0ytd3lNuIe3hHIqCnr3Rk2ViHyhMXPU20:58
rookTheJulia: ^ what I am seeing20:58
rookthe huge drop, is when i restarted ironic20:58
*** sayantan_ has joined #openstack-ironic20:58
*** amarao has joined #openstack-ironic20:59
mrdahi rloo :)20:59
TheJuliarook: how many nodes is that conductor managing?20:59
rloomrda: :)20:59
mariojvhey all, just a heads up, doing an ironic review party in hangouts right now: https://etherpad.openstack.org/p/ironic-review-party21:00
mrdathanks mariojv for posting that here21:00
TheJuliamariojv: would love to, just still trying to recover some of the brain cells from last week first ;)21:00
*** vsaienko has quit IRC21:02
rookTheJulia: 107 nodes right now21:04
*** causten has quit IRC21:05
TheJuliarook: just one conductor for the entire deployment?21:06
*** causten has joined #openstack-ironic21:06
*** dprince has quit IRC21:06
rookTheJulia: Does the Conductor have its own owker configuration>?21:07
rookworker*21:07
*** causten has quit IRC21:07
*** vsaienko has joined #openstack-ironic21:07
rooki don't see it on my undercloud if so21:07
*** causten has joined #openstack-ironic21:07
rookI see the api_worker, which is set to 821:07
*** causten has quit IRC21:07
rooklet me see how many conductors i see running21:07
rookyup21:08
rooksingle ironic-conductor21:08
*** baha has quit IRC21:08
rookTheJulia: I am going to grow this to ~1000 nodes soon, so it would be good to know what I should tune to ease the pressure.21:08
rookmaybe bump the # of conductors to 1/2 # cores.21:08
TheJuliarook: off hand, perhaps turning off power sync might make that load drop quite a bit.21:08
*** sayantan_ has quit IRC21:09
TheJuliarook: so each conductor _should_ be on a unique hostname, there should not be multiple conductor processes trying to run on a single host with one hostname21:09
*** sayantan_ has joined #openstack-ironic21:09
rookok21:09
*** baha has joined #openstack-ironic21:09
rookgood to know TheJulia21:09
rookso it is setup right21:10
*** gcb has joined #openstack-ironic21:11
TheJuliarook: I think the first tunable people go for is https://git.openstack.org/cgit/openstack/ironic/tree/etc/ironic/ironic.conf.sample#n93421:12
rookbump that to 120?21:12
TheJuliaWell, right now, it is basically checking and if there is a discrepancy, assert what it believes the power state to be upon a node.  That can naturally take quite a bit of resources, so you may want to tun that to meet your operational needs, and if you don't need it, you could possibly even disable it entirely21:15
*** mgoddard has quit IRC21:17
*** kjw3 has joined #openstack-ironic21:19
*** spartacloud has joined #openstack-ironic21:20
*** trown is now known as trown|brb21:24
*** vsaienko has quit IRC21:28
*** vsaienko has joined #openstack-ironic21:33
*** jkilpatr has quit IRC21:34
*** amarao has quit IRC21:41
*** mtanino_ has joined #openstack-ironic21:53
*** jkilpatr has joined #openstack-ironic21:53
*** anteaya has quit IRC21:54
*** mtanino has quit IRC21:54
*** trown|brb is now known as trown|outtypewww21:56
*** baha has quit IRC21:59
*** causten has joined #openstack-ironic22:00
*** causten has quit IRC22:00
*** causten has joined #openstack-ironic22:01
*** _vsaienko has joined #openstack-ironic22:02
*** vsaienko has quit IRC22:02
*** _vs_ has joined #openstack-ironic22:03
*** _vsaienko has quit IRC22:03
*** causten has quit IRC22:05
*** sayanta__ has joined #openstack-ironic22:06
*** _vs_ has quit IRC22:06
*** _vs_ has joined #openstack-ironic22:06
*** anteaya has joined #openstack-ironic22:06
openstackgerritVasyl Saienko proposed openstack/ironic master: WIP/DNM: Testing multi-tenant multi-node grenade job  https://review.openstack.org/43695722:07
*** sayantan_ has quit IRC22:08
*** _vs_ has quit IRC22:09
*** vsaienko has joined #openstack-ironic22:11
*** srart has quit IRC22:14
*** srart has joined #openstack-ironic22:15
*** crushil has quit IRC22:17
*** srart has quit IRC22:17
*** krtaylor has quit IRC22:17
*** vsaienko has quit IRC22:27
*** jjohnson2 has quit IRC22:28
*** vsaienko has joined #openstack-ironic22:33
jlvillalCan someone help me understand how this works?  https://github.com/openstack/ironic-lib/blob/master/ironic_lib/metrics.py#L27522:35
jlvillalAnd: https://github.com/openstack/ironic-lib/blob/master/ironic_lib/metrics.py#L27822:35
jlvillalAnd: https://github.com/openstack/ironic-lib/blob/master/ironic_lib/metrics.py#L28122:36
jlvillal I was playing around with type checking and it threw up an error. And I think the error is right.22:37
jlvillalBut maybe I'm being dense.22:37
*** vsaienko has quit IRC22:39
jlvillalI think I figured it out. Never mind :)22:41
*** srart has joined #openstack-ironic22:42
*** bfournie has quit IRC22:43
openstackgerritJulia Kreger proposed openstack/bifrost master: Make ironic database use more configurable  https://review.openstack.org/44061622:44
openstackgerritJulia Kreger proposed openstack/bifrost master: Add the ability to leverage a separate rabbitmq server  https://review.openstack.org/44084322:44
rloojlvillal: glad you figured it out (I just looked up now).22:45
jlvillalrloo: Thanks22:45
rlooc'mon mariojv, i dare you to +A this :D https://review.openstack.org/#/c/439054/22:55
patchbotpatch 439054 - ironic - doc: update FAQ for release notes22:55
mariojvrloo: is there consensus from other cores about that ? ^22:57
openstackgerritMerged openstack/bifrost master: Documentation cleanup  https://review.openstack.org/43915122:57
mariojvi feel like this was talked about at ptg but i don't remember the conversation22:58
rloomariojv: no we didn't talk about it at the ptg. it came up cuz jlvillal mentioned in some patch, that there was a british spelling. i checked on irc earlier this week, but didn't put a link to that here. jroll was fine with it.22:58
mariojvenough for me22:59
rloomariojv: i don't know if jroll is fine with moving that info down into the next faq but i don't care :)22:59
rloomariojv: and it probably should have been done in a separate patch instead of sullying mine ;)22:59
* jlvillal wants the record to show that he said 'nit: <american spelling>' :)23:00
rloomariojv: ^^ :)23:00
* mariojv adds it to "The Record"23:00
aNuposicmariojv: for static IP configuration for driver rescue IPv4 will be used correct and not IPv6?23:00
mariojvthat's a really good question23:00
mariojvaNuposic: both if it's easy, but i'd be fine with doing them in separate patches if it's hard23:01
rloomariojv: btw, don't do something just cuz someone dares you, only cuz you want to :)23:01
mariojvaNuposic: i'm not extremely familiar with systemd-networkd + ipv623:01
aNuposicmariojv: np :) yup for now will stick to ipv4 may be23:02
*** baoli has quit IRC23:04
*** krtaylor has joined #openstack-ironic23:05
openstackgerritAnnie Lezil proposed openstack/ironic-inspector master: Reapply doesn't update started_at time  https://review.openstack.org/43105323:07
mariojvaNuposic: just be sure that docs reflect the limitations when updating23:07
mariojvaNuposic: i had to be pretty careful about the dhcp caveat in the current rescue docs up for review, for example23:07
openstackgerritMerged openstack/ironic master: doc: update FAQ for release notes  https://review.openstack.org/43905423:07
*** sayanta__ has quit IRC23:08
*** sayantan_ has joined #openstack-ironic23:08
*** priteau has quit IRC23:08
*** sdake has joined #openstack-ironic23:09
*** priteau has joined #openstack-ironic23:09
aNuposicmariojv: I see, sure. Updating docs is a far goal :) right now researching how static configuration can be done for tinyipa23:10
*** priteau has quit IRC23:14
*** jaypipes has quit IRC23:16
aNuposicmariojv: do you have your patch link for dhcp configuration handy?23:17
*** yuanying_ has joined #openstack-ironic23:18
*** sdake has quit IRC23:18
*** oomichi has quit IRC23:18
*** sdake has joined #openstack-ironic23:19
*** yuanying has quit IRC23:20
aNuposicmariojv: got it never mind23:20
*** oomichi has joined #openstack-ironic23:21
*** hw_wutianwei1 has joined #openstack-ironic23:21
*** hw_wutianwei has quit IRC23:23
*** hw_wutianwei1 is now known as hw_wutianwei23:23
*** Fdaisuke has joined #openstack-ironic23:27
*** baoli has joined #openstack-ironic23:33
*** Nisha_Agarwal has joined #openstack-ironic23:38
openstackgerritMerged openstack/python-ironicclient master: Updated from global requirements  https://review.openstack.org/44009823:38
*** sdake has quit IRC23:40
*** sdake_ has joined #openstack-ironic23:41
*** sayanta__ has joined #openstack-ironic23:42
*** sayantan_ has quit IRC23:42
openstackgerritJoanna Taryma proposed openstack/ironic master: Follow up patch of 3428cb74f0a6fe05c23cf00dc66da4c1d9766409  https://review.openstack.org/44086523:42
openstackgerritOpenStack Proposal Bot proposed openstack/bifrost master: Updated from global requirements  https://review.openstack.org/44003123:53
*** sdake_ has quit IRC23:53
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-lib master: Updated from global requirements  https://review.openstack.org/44087123:55
*** jmccarthy has quit IRC23:56

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