Tuesday, 2017-12-05

*** namnh has quit IRC00:03
*** jesusaur has quit IRC00:04
*** IgorYozhikov has quit IRC00:16
*** IgorYozhikov has joined #openstack-ironic00:17
*** rbartal has quit IRC00:17
*** etingof has quit IRC00:18
*** etingof has joined #openstack-ironic00:19
*** yangyapeng has quit IRC00:21
*** sasha2 has joined #openstack-ironic00:22
*** sasha2 has quit IRC00:22
*** sasha2 has joined #openstack-ironic00:22
*** athomas has quit IRC00:27
*** hjensas has joined #openstack-ironic00:28
*** rbartal has joined #openstack-ironic00:29
*** masber has quit IRC00:33
*** rbudden has quit IRC00:36
*** masber has joined #openstack-ironic00:41
*** hjensas has quit IRC00:44
*** masber has quit IRC00:47
*** affie has quit IRC00:49
*** bfournie has joined #openstack-ironic00:52
*** bfournie has quit IRC00:53
*** bfournie has joined #openstack-ironic00:59
*** hw_wutianwei has joined #openstack-ironic01:04
*** akhilaki has joined #openstack-ironic01:07
*** phuongnh has joined #openstack-ironic01:12
*** fragatina has quit IRC01:14
*** fragatina has joined #openstack-ironic01:15
*** fragatina has quit IRC01:16
*** yangyapeng has joined #openstack-ironic01:16
*** SerenaFeng has quit IRC01:19
*** mat128 has joined #openstack-ironic01:21
*** jesusaur has joined #openstack-ironic01:23
*** akhilaki has quit IRC01:33
*** phuongnh has quit IRC01:35
*** namnh has joined #openstack-ironic01:40
*** namnh has quit IRC01:41
*** fragatina has joined #openstack-ironic01:42
*** fragatina has quit IRC01:44
*** phuongnh has joined #openstack-ironic01:50
*** rloo has quit IRC02:00
*** gcb has joined #openstack-ironic02:06
*** Grenth has quit IRC02:12
*** Grenth has joined #openstack-ironic02:13
*** jesusaur has quit IRC02:30
*** jesusaur has joined #openstack-ironic02:31
*** zhangfei has joined #openstack-ironic02:35
*** masber has joined #openstack-ironic02:39
openstackgerritOpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements  https://review.openstack.org/52537903:05
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/52538003:06
*** links has joined #openstack-ironic03:07
*** MattMan has quit IRC03:09
*** MattMan has joined #openstack-ironic03:10
*** threestrands has joined #openstack-ironic03:27
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironic-inspector-client master: Updated from global requirements  https://review.openstack.org/52539503:31
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient master: Updated from global requirements  https://review.openstack.org/52539603:31
*** fragatina has joined #openstack-ironic03:56
*** fragatina has quit IRC04:01
*** fragatina has joined #openstack-ironic04:17
*** Nisha_Agarwal has joined #openstack-ironic04:18
*** dpawar has joined #openstack-ironic04:43
*** jaganathan has joined #openstack-ironic04:44
*** udesale has joined #openstack-ironic04:53
*** ianychoi has joined #openstack-ironic05:07
*** sasha2 has quit IRC05:11
*** tzumainn has quit IRC05:11
*** threestrands has quit IRC05:25
*** hshiina has joined #openstack-ironic05:33
*** vishwana_ has joined #openstack-ironic05:41
*** fragatina has quit IRC05:44
*** vishwanathj has quit IRC05:45
*** mat128 has quit IRC05:47
*** rbartal has quit IRC05:57
*** oanson has quit IRC06:20
*** oanson has joined #openstack-ironic06:20
*** SerenaFeng has joined #openstack-ironic06:37
*** zhangfei has quit IRC06:38
*** karthiks has joined #openstack-ironic06:39
*** zhangfei has joined #openstack-ironic06:50
*** marios has joined #openstack-ironic06:53
*** udesale__ has joined #openstack-ironic06:58
*** udesale has quit IRC06:58
*** udesale has joined #openstack-ironic07:00
*** udesale__ has quit IRC07:02
*** jaianshu has joined #openstack-ironic07:06
*** SerenaFeng has quit IRC07:29
*** pmannidi has quit IRC07:31
*** pcaruana has joined #openstack-ironic07:32
*** Nisha_Agarwal has quit IRC07:32
*** Nisha_Agarwal has joined #openstack-ironic07:33
*** amarao has joined #openstack-ironic07:41
*** amarao1 has joined #openstack-ironic07:41
*** hjensas has joined #openstack-ironic07:41
*** pmannidi has joined #openstack-ironic07:44
*** pmannidi has quit IRC07:45
*** pmannidi has joined #openstack-ironic07:45
*** pmannidi has quit IRC07:48
*** rcernin has quit IRC07:48
*** Nisha_Agarwal has quit IRC07:52
*** athomas has joined #openstack-ironic07:54
*** amarao1 has quit IRC07:56
*** vishwana_ has quit IRC08:04
*** vishwanathj has joined #openstack-ironic08:04
*** v12aml has quit IRC08:08
*** jtomasek has joined #openstack-ironic08:16
*** dpawar has quit IRC08:21
*** tesseract has joined #openstack-ironic08:24
*** priteau has joined #openstack-ironic08:29
*** SerenaFeng has joined #openstack-ironic08:30
*** dpawar has joined #openstack-ironic08:31
*** ccamacho has joined #openstack-ironic08:32
*** rcernin has joined #openstack-ironic08:33
*** v12aml has joined #openstack-ironic08:36
*** Nisha_Agarwal has joined #openstack-ironic08:38
*** pcaruana has quit IRC08:40
*** mrch has joined #openstack-ironic08:43
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Add docs for ansible deploy interface  https://review.openstack.org/52550108:51
*** rbartal has joined #openstack-ironic08:57
*** hshiina has quit IRC09:07
*** SerenaFeng has quit IRC09:07
*** etingof has quit IRC09:09
*** zhangfei has quit IRC09:12
pas-hamorning ironic :)09:14
*** makowals has quit IRC09:25
*** makowals has joined #openstack-ironic09:25
*** PixieBoots has quit IRC09:27
*** affie has joined #openstack-ironic09:30
*** zhangfei has joined #openstack-ironic09:30
*** dtantsur|afk has quit IRC09:30
*** lucas-afk is now known as lucasagomes09:31
*** affie_ has joined #openstack-ironic09:32
*** dtantsur has joined #openstack-ironic09:33
*** dtantsur has quit IRC09:33
*** dtantsur has joined #openstack-ironic09:33
*** dtantsur has quit IRC09:33
*** PixieBoots has joined #openstack-ironic09:35
*** etingof has joined #openstack-ironic09:35
*** affie has quit IRC09:35
*** dtantsur has joined #openstack-ironic09:38
*** e0ne has joined #openstack-ironic09:39
*** derekh has joined #openstack-ironic09:39
hjensasGood Morning. :)09:40
dtantsurmorning hjensas09:40
dtantsur#pixiesay -mflexing I AM BACK09:41
PixieBootsᕙʕ⇀ᴥ⇀ʔᕗ: I AM BACK09:41
*** etingof has quit IRC09:42
*** Nisha_Agarwal has quit IRC09:44
*** namnh has joined #openstack-ironic09:45
*** stendulker has joined #openstack-ironic09:48
*** etingof has joined #openstack-ironic09:55
*** makowals_ has joined #openstack-ironic09:56
*** makowals has quit IRC09:58
*** priteau has quit IRC10:03
*** priteau has joined #openstack-ironic10:04
*** Nisha_Agarwal has joined #openstack-ironic10:05
*** fxpester has quit IRC10:06
*** priteau has quit IRC10:09
*** fxpester has joined #openstack-ironic10:10
openstackgerritShivanand Tendulker proposed openstack/ironic master: Adds rescue_interface to base driver class  https://review.openstack.org/50933510:12
openstackgerritShivanand Tendulker proposed openstack/ironic master: Adds RPC calls for rescue interface  https://review.openstack.org/50933610:12
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add rescue related methods to network interface  https://review.openstack.org/50934210:12
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add rescuewait timeout periodic task  https://review.openstack.org/35315610:12
openstackgerritShivanand Tendulker proposed openstack/ironic master: Agent rescue implementation  https://review.openstack.org/40043710:12
*** sambetts|afk is now known as sambetts10:14
*** priteau has joined #openstack-ironic10:16
sambettsMorning all10:21
dtantsurhey sambetts10:21
dtantsursambetts: mind starting your day with an easy review? https://review.openstack.org/#/c/524560/10:21
patchbotpatch 524560 - sushy-tools - add dvsm CI job10:21
openstackgerritMerged openstack/python-ironicclient master: Avoid tox_install.sh for constraints support  https://review.openstack.org/52482310:21
sambettsdtantsur: sure :)10:21
sambettsdtantsur: although its Depends-On: patch needs a recheck if it hasn't been given one yet10:22
dtantsurah! then https://review.openstack.org/#/c/524808/ and https://review.openstack.org/#/c/522020/ also technical patches10:23
patchbotpatch 524808 - virtualbmc - Avoid tox_install.sh for constraints support10:23
patchbotpatch 522020 - virtualbmc - Clean up zuul files10:23
dtantsurtrying a recheck, though IIRC there are problems with multinode10:24
*** namnh has quit IRC10:29
sambettsdtantsur: its really intermitent, I expect its the same race condition that we've reported to Nova though right?>10:29
*** rcernin has quit IRC10:30
dtantsuryeah, probably10:30
stendulkerdtantsur: Hi10:46
dtantsurhi stendulker10:47
stendulkerdtantsur: We wanted to make rescue interface having a 'None' as the default value10:47
dtantsuryep, similar to other optional interfaces10:48
stendulkerBut it seems does not work for me.10:48
stendulkerother interfaces have the value in enabled_*_interfaces as some values10:48
stendulkerwhich would get used when any node is created. We wanted to have the rescue_interface as 'None'10:49
dtantsurstendulker: only for classic drivers10:50
dtantsurwhich are not affected by enabled_*_interfaces10:50
stendulkerSo that any nodes created after patch https://review.openstack.org/#/c/509335 would have None as rescue interface10:50
patchbotpatch 509335 - ironic - Adds rescue_interface to base driver class10:50
stendulkerYes for classic its being taken care.10:50
*** trungnv has quit IRC10:50
*** trungnv has joined #openstack-ironic10:51
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: Test OPNFV CI verify job. Please ignore.  https://review.openstack.org/36760810:51
dtantsurstendulker: well, for hardware types it should be no-rescue indeed10:51
stendulkerFor dynamic drivers also if we make it None then migration becomes easier, else they would be having no-rescue10:51
dtantsurwhich may be a bit funnu around migration10:51
dtantsurright10:52
*** phuongnh has quit IRC10:52
stendulkerAlso current users would need to set-up the rescue network config or we would have to hardcode not to have rescue network if its is no-rescue10:52
dtantsurwell, if we update all nodes to 'no-rescue', it should not break classic drivers, as they should not rely on the rescue_interface field at all10:52
*** phuongnh has joined #openstack-ironic10:52
hwoaranggood morning10:53
hwoarangwe are seeing some issues with bifrost: 'Endpoint of type baremetal-introspection was not found in the service catalog and was not provided explicitly' i was wondering if you have any pointers10:53
hwoarangpas-ha: ^ :)10:53
* dtantsur wonder if python-ironic-inspector-client 3.0.0 breaks bifrost10:54
stendulkerHad posted patch with the 'None' as value of enabled_rescue_interface to show the thing, but it breaks in config10:54
dtantsuryep, this is incorrect, it should be no-rescue10:54
*** ianychoi has quit IRC10:55
stendulkerThen I will revert it no-rescue. Other day when we were discussing with rloo and TheJulia I thought we wanted to have 'None' for this. But seems its does not work.10:55
*** ianychoi has joined #openstack-ironic10:55
dtantsuryeah10:56
dtantsurhwoarang: where exactly do you see this failure?10:56
hwoarangdtantsur: we are seeing that in our 3rd party CI in OPNFV but now I am also trying on the openstack CI too10:56
dtantsurhwoarang: I mean, what exactly are you trying to do?10:56
stendulkerand then for migration we would need both migration script and live upgrade related method.10:56
stendulkerWith None, no migration script was required and only  _convert_to_version() would have been enough.10:57
hwoarangdtantsur: we provision 3 VMs using bifrost and enroll them to ironic. So we use a variant of the upstream ./scripts/test-bifrost.sh script10:57
dtantsurhwoarang: okay, I guess we'll have to wait for TheJulia or pas-ha, but it's probably because python-ironic-inspector-client 3.0.0 no longer uses 127.0.0.1:5050 as an implicit default10:59
*** serlex has joined #openstack-ironic11:00
dtantsurhwoarang: try export INSPECTOR_URL=http://127.0.0.1:5050 as a quick-n-dirty workaround11:00
hwoarangok let me try that. Thanks!11:00
pas-hahwoarang: in the [inspector] section set 'auth_type=none' and 'endpoint_override=<your inspector endpoint URL>"11:01
*** milan_ has joined #openstack-ironic11:01
milan_morning Ironic! :)11:01
*** makowals_ has quit IRC11:01
dtantsurmorning milan_11:02
hwoarangpas-ha: ok I will. Is this something that needs to be changed in upstream bifrost stuff?11:02
milan_morning dtantsur! :)11:02
dtantsurI suspect bifrost has to be fixed, yes11:02
pas-hahwoarang: it should've been backward compatible...11:03
hwoarangwe are not modifying any of these. we take what bifrost gives us so i guess not :)11:03
* pas-ha will double check11:03
stendulkerdtantsur: But in migration, I think we cannot update all to 'no-rescue'. As in manager.py we would be just performing getattr() on driver to know if the rescue interface is present and call the method. https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L2431-L243311:04
pas-hait is a bug of my implementation then11:04
stendulkerdtantsur: Though it would be throw exception as part of  'no-rescue', would it be ok? to have value for migrated nodes as 'no-rescue'?11:05
*** zhangfei has quit IRC11:06
dtantsurstendulker: for classic drivers it will still be None, I guess11:06
dtantsura driver.rescue is a constant for classic drivers11:06
hwoarangpas-ha: http://logs.openstack.org/08/367608/20/check/bifrost-integration-tinyipa-opensuse-423/0bcb3b0/job-output.txt.gz#_2017-12-05_11_04_26_67702411:07
stendulkerdtantsur: In migration time we would have to check the driver and fill as None11:07
hwoarangthat's from an upstream bifrost job11:07
stendulkerPost patch newly created nodes would have None.11:08
dtantsurstendulker: no, you don't. just put no-rescue there11:09
dtantsurfor all nodes, I mean11:09
stendulkerdtantsur: I am trying to understand your comment 'if we update all nodes to 'no-rescue', it should not break classic drivers, as they should not rely on the rescue_interface field at all'11:09
dtantsurstendulker: well, literelly. if the driver is classic, driver.rescue == None always11:10
dtantsurwhatever you put in rescue_interface11:10
stendulkerdtantsur: Ok11:10
stendulkerdtantsur: Thank you. Will re-work on this patch and post new one shortly.11:11
dtantsurstendulker: np. just to clarify, this is the code bit for classic drivers: https://github.com/openstack/ironic/blob/master/ironic/common/driver_factory.py#L9911:11
*** Nisha_Agarwal has quit IRC11:11
*** etingof has quit IRC11:12
dtantsurand as you see here https://github.com/openstack/ironic/blob/master/ironic/common/driver_factory.py#L104-L109 the rescue interface is not copied from node.rescue_interface for classic drivers11:12
dtantsurso it will stay None11:12
stendulkerdtantsur: ok. Thank you.11:12
pas-hadtantsur: milan_: aarefiev: re inspectorclient - how does it handles inspector_url arg? is there some builtin default? because even before my patches to ironic it seems it was working with CONF.inspector.service_url=None...11:14
dtantsurpas-ha: there used to be a default, it was removed in the yesterday's release11:14
pas-hahehe, now we have it :)11:14
dtantsurwell, the default is to try from the catalog now. if there is no catalog - ooops11:14
*** stendulker_ has joined #openstack-ironic11:14
dtantsurso, yes, CONF.inspector.service_url=None would work, if there is keystone11:15
dtantsurif there is no keystone, previous 127.0.0.1:5050 was tried; but it is no longer11:15
pas-haso we do need to modify bifrost then, and make it using the new opt already (with endpoint_override)11:15
pas-haI'll make a patch11:15
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-python-agent master: Installl OpenSSH by default  https://review.openstack.org/52552911:16
*** marios has quit IRC11:16
*** mgoddard has joined #openstack-ironic11:18
*** stendulker has quit IRC11:18
*** stendulker_ has quit IRC11:21
openstackgerritMerged openstack/virtualbmc master: Avoid tox_install.sh for constraints support  https://review.openstack.org/52480811:26
openstackgerritMerged openstack/virtualbmc master: Clean up zuul files  https://review.openstack.org/52202011:26
*** etingof has joined #openstack-ironic11:26
*** udesale has quit IRC11:29
*** phuongnh has quit IRC11:35
*** jkilpatr has quit IRC11:36
*** jkilpatr has joined #openstack-ironic11:55
*** mtsv has joined #openstack-ironic11:58
openstackgerritPavlo Shchelokovskyy proposed openstack/bifrost master: Fix inspector not found when noauth  https://review.openstack.org/52557512:01
pas-hahwoarang: ^12:02
*** racedo has joined #openstack-ironic12:02
openstackgerritMerged openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/52538012:03
hwoarangthank you pas-ha12:03
pas-hanow let's see if it passes :)12:04
*** lucasagomes is now known as lucas-hungry12:13
*** serlex has quit IRC12:14
*** marios has joined #openstack-ironic12:17
openstackgerritNikolay Fedotov proposed openstack/ironic-python-agent master: Try to find partition 3 times  https://review.openstack.org/52557712:21
*** bfournie has quit IRC12:24
*** bfournie has joined #openstack-ironic12:24
*** bfournie has quit IRC12:28
pas-hahwoarang: it seems the bifrost-integration-tinyipa-opensuse is passing http://logs.openstack.org/75/525575/1/check/bifrost-integration-tinyipa-opensuse-423/149ec1f/12:35
pas-haand overall only dib-based job has failed12:35
*** zhurong has joined #openstack-ironic12:45
*** Goneri has quit IRC12:45
*** mrch has quit IRC12:48
*** dprince has joined #openstack-ironic12:58
*** mrch has joined #openstack-ironic12:59
*** zhurong has quit IRC13:02
*** zhurong has joined #openstack-ironic13:03
*** jcoufal has joined #openstack-ironic13:05
*** bfournie has joined #openstack-ironic13:10
*** lucas-hungry is now known as lucasagomes13:14
*** links has quit IRC13:22
dtantsurpas-ha: another thoughts re ansible: what about accepting SSH key file and user in ironic.conf as well?13:25
dtantsure.g. in case of TripleO these are going to be constants (per deploy)13:25
*** makowals has joined #openstack-ironic13:26
*** zhurong has quit IRC13:27
dtantsuractually, it make sense to me to allow all its driver_info options in ironic.conf13:27
*** tzumainn has joined #openstack-ironic13:28
*** jaianshu has quit IRC13:30
dtantsurpas-ha: https://bugs.launchpad.net/ironic/+bug/173640913:30
openstackLaunchpad bug 1736409 in Ironic "[RFE] Allow setting defaults for ansible deploy driver_info parameters in ironic.conf" [Wishlist,Confirmed]13:30
*** trown|outtypewww is now known as trown13:30
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Make the agent distributed using hashring and notifications  https://review.openstack.org/52470913:30
*** udesale has joined #openstack-ironic13:32
*** yangyapeng has quit IRC13:33
*** yangyapeng has joined #openstack-ironic13:34
*** yangyapeng has quit IRC13:38
*** e0ne_ has joined #openstack-ironic13:40
*** jaganathan has quit IRC13:42
*** e0ne has quit IRC13:43
sambettshey hjensas, whats your plans around the networking-baremetal agent stuff? it seems like your pretty up to speed with the concept and the code, so do you want to take lead on it, and I'll review and chip in if required  (it'll be easier for me that way as I am working on getting my CI back up and running so don't have a lot of time dedicated to it)13:44
*** jkilpatr has quit IRC13:47
*** jkilpatr has joined #openstack-ironic13:48
pas-hadtantsur: sounds reasonable, will gladly do. should that be an RFE now? or, since there was no tagged release of ironic with ansible-deploy we could just improve it until release w/o requiring an RFE?13:48
hjensassambetts: Well you where on holidays and I got a bit inpatient. :) Sure, I can take the lead, but please review.13:48
dtantsurpas-ha: I guess it should be an RFE, as it was not in the spec13:49
sambettshjensas: no problem :) it something that needs doing and I have about 20 different things going on, so having someone take that over means I have 19 things to focus on instead :D13:50
pas-haok, will make a quick RFE and a small patch with implementation (I guess the former will be more lines that the latter :) )13:51
hjensassambetts: the last one where I use notifications to manage the group. I am not sure, it might be better to use tooz and rely on one of the backends regis,etcd,memcache instead? I decided to try the notification approach since that means one less dependency.13:51
sambettshjensas: yeah, its a tough one, the OpenStack TC have stated that etcd is now an OpenStack common so we should be good to use that if we want, otherwise I like your approach too (I had worried that we might need a DB to track the hashring like Ironic does) but keeping it in memory works too, although I expect there might be problems with a new agent coming up knowing that its heard from all the13:55
sambettsexisting agents, and probelms with segmentation, basically all the normal distributed system/leader election pitfalls13:55
*** sasha2 has joined #openstack-ironic13:56
hwoarangpas-ha: yeah thanks for the quick fix13:59
*** mat128 has joined #openstack-ironic14:03
*** rloo has joined #openstack-ironic14:03
*** crushil has joined #openstack-ironic14:07
hjensassambetts: Indeed, the timings need to be tuned, but with an initialdelay on the looping call that does _report_state we should be able to wait 2-4 iterations of the notifications so that new nodes have a chance to learn the ring.14:12
openstackgerritRuby Loo proposed openstack/python-ironic-inspector-client stable/pike: Centralize list of irrelevant files  https://review.openstack.org/52561714:13
hjensassambetts: As long as we use the same messaging service to control the group members that we use to call neutron, I hoped we could kind of leave the segmentation problem to a rabbit cluster ... And eventually, if two agents where to report the same ironic nodes it should'nt be a major issue.14:18
sambettshjensas: yeah it might not be an issue, just something to keep in mind, as they are talking to central message service instead of to each other segmentation isn't such a big issue14:20
*** rloo_ has joined #openstack-ironic14:24
*** rloo has quit IRC14:28
*** Goneri has joined #openstack-ironic14:28
*** jcoufal has quit IRC14:29
rloo_good morning everyone, including sambetts, hjensas !14:32
*** rbudden has joined #openstack-ironic14:33
sambettshey rloo_14:33
*** dpawar has quit IRC14:33
rloo_good morning pas-ha, dtantsur. wrt the ansible deploy interface changes -- i am also good with updating the original spec (vs creating a new one). the 'feature' isn't quite done, cuz docs + testing :)14:33
*** rloo_ is now known as rloo14:33
dtantsurI was hoping to not even have a spec :) morning rloo14:34
rloodtantsur: that too -- i was just skimming discussions earlier today :)14:34
rloodtantsur: whatever is easiest i think.14:34
rloosambetts: wrt the multinode CI failures -- do we have a bug # for the nova issue?14:34
sambettsrloo: that is a good question14:35
pas-hadtantsur: since there was no release yet we indeed might just update the (merged) spec14:35
*** mjturek has joined #openstack-ironic14:35
rloosambetts: i wanted to update our whiteboard with the CI situation14:35
rloopas-ha: having it all in one spec makes it easier too, for the reader to know what that feature includes :)14:37
sambettsrloo: asking in the nova channel to see if they are tracking it anywhere14:39
*** Nisha_Agarwal has joined #openstack-ironic14:39
*** mjturek1 has joined #openstack-ironic14:39
rloothx sambetts14:39
*** mjturek has quit IRC14:39
sambettsrloo: I can't see a obvious bug in the nova bug list but it might be worded differently to what I'm looking fo14:39
sambettsfor14:39
rloosambetts: for now, i put down that we think it is a race condition in nova. if you find a bug, please update. as long as nova is aware/going to fix (cross fingers...)14:40
*** udesale has quit IRC14:40
sambettsrloo: will do14:44
*** udesale has joined #openstack-ironic14:50
*** gcb has quit IRC14:53
*** gcb has joined #openstack-ironic14:54
*** udesale has quit IRC14:59
*** rbartal has quit IRC15:10
*** mrch has quit IRC15:14
openstackgerritRuby Loo proposed openstack/ironic master: 9.2.0 is the ironic version with rebuild configdrive  https://review.openstack.org/52565115:29
*** udesale has joined #openstack-ironic15:36
*** dgautam has joined #openstack-ironic15:40
*** udesale has quit IRC15:41
*** udesale has joined #openstack-ironic15:42
*** openstackgerrit has quit IRC15:48
*** Nisha_ has joined #openstack-ironic15:52
*** Nisha_Agarwal has quit IRC15:52
*** Nisha_ has quit IRC15:58
dgautamIs there a guide available about various ironic drivers that could help to choose a driver while create a node. e.g. pxe_ipmitool and agent_ipmitool15:59
*** hamzy has quit IRC16:02
*** crushil has quit IRC16:05
*** ccamacho has quit IRC16:06
rloodgautam: would this help? https://docs.openstack.org/ironic/latest/install/refarch/index.html16:08
*** charcol has quit IRC16:09
rloodgautam: also, not to confuse you, but we're moving away from 'classic drivers' and encourage you to use a hardware type: https://docs.openstack.org/ironic/latest/admin/upgrade-to-hardware-types.html16:09
*** crushil has joined #openstack-ironic16:09
*** athomas has quit IRC16:12
*** dgautam has quit IRC16:13
*** dgautam has joined #openstack-ironic16:16
dgautamrloo: Thanks. Hardware types are available pike onwards I believe.16:16
rloodgautam: i don't recall when; would need to look it up. I live in the present :)16:17
*** nicodemos has quit IRC16:19
*** ricardoas1 has quit IRC16:23
*** mrtenio-afk has quit IRC16:23
*** udesale has quit IRC16:23
*** openstackgerrit has joined #openstack-ironic16:26
openstackgerritOpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements  https://review.openstack.org/52537916:26
*** e0ne_ has quit IRC16:33
*** marios has quit IRC16:33
*** hamzy_ has joined #openstack-ironic16:37
*** dgautam has quit IRC16:38
*** hamzy has joined #openstack-ironic16:42
*** hamzy_ has quit IRC16:46
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: fix dvsm config deprecations  https://review.openstack.org/52319616:47
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Make discovery use dnsmasq dhcp filter  https://review.openstack.org/52568516:47
*** hamzy has quit IRC16:54
*** fragatina has joined #openstack-ironic17:01
*** fragatin_ has joined #openstack-ironic17:02
*** derekh has quit IRC17:03
*** fragatina has quit IRC17:06
NobodyCamGood Morning Ironic'ers17:10
openstackgerritMerged openstack/python-ironicclient master: Updated from global requirements  https://review.openstack.org/52539617:10
dtantsurmorning NobodyCam17:11
NobodyCam:) Morning dtantsur :)17:20
*** pcaruana has joined #openstack-ironic17:22
*** mtsv has quit IRC17:24
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-specs master: Amend ansible deploy interface spec  https://review.openstack.org/52570817:25
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: [ansible] add defaults to config  https://review.openstack.org/52571017:25
openstackgerritIlya Etingof proposed openstack/sushy-tools master: add dvsm CI job  https://review.openstack.org/52456017:26
*** pcaruana has quit IRC17:26
*** lucasagomes is now known as lucas-afk17:27
*** pcaruana has joined #openstack-ironic17:27
dtantsuro/17:27
*** dtantsur is now known as dtantsur|afk17:27
pas-hadtantsur|afk: ^^^17:28
pas-ha(would actually simplify devstack changes for ansible deploy setup and testing)17:29
pas-hadtantsur|afk: thanks for the idea, glad to see someone is actively using it :) the more usage, the more bugs and ideas, the better the stuff becomes :)17:29
*** SerenaFeng has joined #openstack-ironic17:31
*** racedo has quit IRC17:32
*** dgautam has joined #openstack-ironic17:33
NobodyCam:)17:34
*** amarao has quit IRC17:34
openstackgerritRuby Loo proposed openstack/ironic stable/ocata: DNM Want to see the CI results for stable/ocata  https://review.openstack.org/52571317:35
*** affie_ has quit IRC17:37
*** mrtenio has joined #openstack-ironic17:43
*** fragatin_ has quit IRC17:44
* sambetts --> home 17:45
sambettsnight all o/17:45
*** sambetts is now known as sambetts|afk17:45
rloobye sambetts|afk17:55
*** pcaruana has quit IRC17:58
*** trown is now known as trown|lunch18:02
*** mgoddard has quit IRC18:04
*** SerenaFeng has quit IRC18:10
openstackgerritIlya Etingof proposed openstack/ironic-python-agent master: Report /dev/disk/by-path on inspection followup  https://review.openstack.org/50785818:10
*** makowals has quit IRC18:11
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Terminal state transitions in transactions  https://review.openstack.org/51092818:11
*** makowals has joined #openstack-ironic18:11
*** SerenaFeng has joined #openstack-ironic18:11
*** makowals has quit IRC18:12
*** makowals has joined #openstack-ironic18:12
*** makowals has quit IRC18:13
*** makowals has joined #openstack-ironic18:13
*** makowals has quit IRC18:13
*** makowals has joined #openstack-ironic18:14
*** makowals has quit IRC18:14
*** makowals has joined #openstack-ironic18:15
*** makowals has quit IRC18:15
openstackgerritIlya Etingof proposed openstack/sushy-tools master: documentation on simulators added  https://review.openstack.org/52344518:15
*** tesseract has quit IRC18:21
openstackgerritMerged openstack/python-ironic-inspector-client stable/pike: Centralize list of irrelevant files  https://review.openstack.org/52561718:24
*** crushil has quit IRC18:28
*** crushil has joined #openstack-ironic18:31
*** amarao has joined #openstack-ironic18:36
rlooi think we have a problem with stable/ocata. so far, all the zuul-related patches on stable/ocata have tempest failures: https://etherpad.openstack.org/p/ironic-zuulv3-intree-tracking18:37
jlvillalrloo: :(18:43
jlvillalrloo: Have you tried an empty patch?18:43
rloojlvillal: yeah. i have a test patch to try on ironic -- the others are non-ironic repos but they use the same tests so...18:43
*** mgoddard has joined #openstack-ironic18:43
rloojlvillal: https://review.openstack.org/#/c/525713/18:44
patchbotpatch 525713 - ironic (stable/ocata) - DNM Want to see the CI results for stable/ocata18:44
rloojlvillal: i also had some test patches (or maybe just one) on some other ironic project. i can't keep track of them and when I might have started to look and then stopped :-(18:44
jlvillalrloo: thanks18:44
rloojlvillal: i was wondering if your retry-patches-script could be used to eg. submit test patches to the stable branches, maybe once a week. might be useful if there is a breakage...18:45
jlvillalrloo: Well we can have something that does something like that. Would be possible for sure. Just takes some time to write the code of course18:47
rloojlvillal: yes, everything is possible except world peace! :D18:48
*** dgautam has quit IRC18:54
*** dgautam has joined #openstack-ironic18:55
*** dgautam has quit IRC19:01
*** dgautam has joined #openstack-ironic19:02
*** milan_ has quit IRC19:06
*** fragatina has joined #openstack-ironic19:06
*** Goneri has quit IRC19:10
openstackgerritMerged openstack/ironic-python-agent master: Updated from global requirements  https://review.openstack.org/52373319:10
*** hamzy has joined #openstack-ironic19:10
*** pester has joined #openstack-ironic19:12
*** trown|lunch is now known as trown19:14
*** fxpester has quit IRC19:16
openstackgerritMerged openstack/ironic-python-agent master: [LLDP] Skip NICs that say they are ready but are unreadable.  https://review.openstack.org/43438519:17
jlvillalDoes anyone here know how to make an EFI based image using DIB?19:23
jlvillalrloo: I was looking at https://review.openstack.org/#/c/525713/ in the Zuul status page. Every DSVM job is failing. Is that what you were seeing?19:25
patchbotpatch 525713 - ironic (stable/ocata) - DNM Want to see the CI results for stable/ocata19:25
rloojlvillal: yes, one of the tempest tests was always failing in the 4? I had looked at.19:25
jlvillalrloo: Thanks.19:26
rloojlvillal: ironic_tempest_plugin.tests.scenario.test_baremetal_basic_ops.BaremetalBasicOps19:27
rloojlvillal: and then.. i think i got distracted and started working on something else :-(19:27
*** amarao has quit IRC19:43
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Abstract away the virtualization management interface  https://review.openstack.org/52332119:49
*** SerenaFeng has quit IRC19:56
jlvillalrloo: This looks like it might be the problem: http://logs.openstack.org/13/525713/1/check/ironic-tempest-dsvm-ipa-partition-uefi-pxe_ipmitool-tinyipa/b98dcf8/logs/screen-ir-api.txt.gz?level=ERROR19:59
jlvillalI wonder if pas-ha knows about why for ^^^19:59
rloooh, critical.19:59
jlvillalrloo: I think that's bad ;)20:01
rloodidn't things change to use v3 keystone recently?20:01
jlvillalrloo: Before we didn't see CRITICAL errors. I'll humble-brag and say that I added the code to show CRITICAL errors on the web log files20:01
jlvillalrloo: Maybe. I'm not sure. Hopefully pas-ha knows.20:01
rloothx humble jlvillal :)20:02
jlvillalrloo: heh20:02
jlvillalrloo: Maybe I should s/humble-// ;)20:02
*** jrist has quit IRC20:05
*** SerenaFeng has joined #openstack-ironic20:08
rloojlvillal: could you do me a favour and update the whiteboard to include your findings?20:09
*** SerenaFeng has quit IRC20:10
*** mgoddard has quit IRC20:13
*** SerenaFeng has joined #openstack-ironic20:16
*** SerenaFe_ has joined #openstack-ironic20:20
*** yangyapeng has joined #openstack-ironic20:20
*** SerenaFeng has quit IRC20:21
*** SerenaFe_ has quit IRC20:22
*** yangyapeng has quit IRC20:24
*** e0ne has joined #openstack-ironic20:25
openstackgerritIlya Etingof proposed openstack/sushy-tools master: OpenStack cloud driver added  https://review.openstack.org/52387920:28
*** rwsu has quit IRC20:31
openstackgerritIlya Etingof proposed openstack/sushy-tools master: OpenStack cloud driver added  https://review.openstack.org/52387920:31
*** rwsu has joined #openstack-ironic20:31
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Abstract away the virtualization management interface  https://review.openstack.org/52332120:32
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Abstract away the virtualization management interface  https://review.openstack.org/52332120:37
openstackgerritIlya Etingof proposed openstack/sushy-tools master: OpenStack cloud driver added  https://review.openstack.org/52387920:37
openstackgerritIlya Etingof proposed openstack/sushy-tools master: documentation on simulators added  https://review.openstack.org/52344520:37
*** Goneri has joined #openstack-ironic20:59
*** SerenaFeng has joined #openstack-ironic21:11
*** hamzy has quit IRC21:12
*** jtomasek has quit IRC21:18
*** penick has joined #openstack-ironic21:23
*** e0ne has quit IRC21:25
jlvillalrloo: Sure21:29
*** mat128 has quit IRC21:34
*** jkilpatr has quit IRC21:35
jlvillalrloo: I updated the whiteboard: https://etherpad.openstack.org/p/IronicWhiteBoard21:39
rloothx jlvillal!21:40
jlvillalrloo: I don't think keystone is the issue. (pas-ha it is not keystone)21:40
jlvillalrloo: Maybe this: http://logs.openstack.org/13/525713/1/check/ironic-tempest-dsvm-ipa-wholedisk-bios-pxe_ipmitool-tinyipa/1b15f6c/logs/screen-q-agt.txt.gz?level=ERROR21:40
jlvillalrloo: I am seeing that in all the jobs I looked at.21:40
rloojlvillal: that critical is odd, but it seems like if it was keystone, nothing would have happened.21:40
rloojlvillal: now, that, i can believe. most of our failures are due to neutron :-(21:41
jlvillalrloo: Hmmm. But now I am looking at more jobs and they don't have that error. Argh :(21:41
jlvillalrloo: Oh wait. Yes they do :)21:41
rloojlvillal: so that's the problem. w/o knowing which errors were meant to happen (cuz, when you test, you also want to test to check that it fails when you think it should fail)...21:41
jlvillalrloo: I was looking in screen-g-* instead of screen-q-* for 'agt'21:42
rloojlvillal: heh, bit of dyslexia there!21:42
jlvillalg vs q21:42
jlvillalrloo: yes!21:42
jlvillalrloo: So a working build on master does not have that error.21:43
rloojlvillal: stable/pike seems ok too21:43
rloojlvillal: just ocata21:43
jlvillalrloo: So maybe our networking expert vdrok would know :)21:43
rloojlvillal: makes me wonder if neutron pushed out something for ocata...21:43
* jlvillal doesn't know21:43
rloojlvillal: how does it work, our stable/ocata will use stable/ocata versions of other projects? not released ocata versions?21:45
jlvillalrloo: I think it uses what is on the stable/ocata branch. For the most part.21:45
rloojlvillal: http://git.openstack.org/cgit/openstack/neutron/log/?h=stable/ocata21:45
jlvillalrloo: For the stuff that is 'git cloned'21:45
rloojlvillal: take. your. pick...21:46
vdrokjlvillal: this is the first time in my life my nick was used in the same sentence with networking expert :D21:46
*** jrist has joined #openstack-ironic21:46
vdrokgood morning jlvillal rloo21:46
rloogood morning vdrok!21:46
jlvillalvdrok: heh :)21:46
rloovdrok: our stable/ocata CI seem to be broken, jlvillal thinks it might be due to neutron ^^21:47
vdroklooking21:47
*** penick has quit IRC21:47
vdrokjlvillal: hrm, that happens right on the agent start, ironic should have nothing to do with that21:49
jlvillalvdrok: Yeah I don't think is error is because of ironic21:49
*** trown is now known as trown|outtypewww21:50
*** racedo has joined #openstack-ironic21:51
vdrokjlvillal: http://logs.openstack.org/13/525713/1/check/ironic-tempest-dsvm-ipa-partition-uefi-pxe_ipmitool-tinyipa/b98dcf8/logs/screen-ir-cond.txt.gz?level=WARNING21:52
jlvillalvdrok: What do you think that means?21:53
vdroknevermind, I see the node is being deployed fine afterwards21:54
vdrokso, something has aborted the build of an instance21:54
vdrokhttp://logs.openstack.org/13/525713/1/check/ironic-tempest-dsvm-ipa-partition-uefi-pxe_ipmitool-tinyipa/b98dcf8/logs/screen-n-cpu.txt.gz#_2017-12-05_18_18_32_59621:55
jlvillalrloo: vdrok: Not sure the neutron error might be not an error. Seems like it logs it but it is ignored.21:59
jlvillalI have to get back to working on something else. So I can't dig into it :(21:59
rloojlvillal: thx, i've been working on something else, so I understand.22:00
openstackgerritIlya Etingof proposed openstack/sushy-tools master: OpenStack cloud driver added  https://review.openstack.org/52387922:01
*** zshi has quit IRC22:03
*** threestrands has joined #openstack-ironic22:05
*** threestrands has quit IRC22:05
*** threestrands has joined #openstack-ironic22:05
*** rcernin has joined #openstack-ironic22:05
*** priteau has quit IRC22:07
*** crushil has quit IRC22:07
*** crushil has joined #openstack-ironic22:07
*** racedo has quit IRC22:10
*** crushil has quit IRC22:12
*** fragatina has quit IRC22:13
*** fragatina has joined #openstack-ironic22:17
*** fragatin_ has joined #openstack-ironic22:17
*** makowals has joined #openstack-ironic22:18
*** dprince has quit IRC22:18
*** fragatina has quit IRC22:21
*** SerenaFeng has quit IRC22:32
*** makowals has quit IRC22:40
*** affie_ has joined #openstack-ironic22:56
*** SerenaFeng has joined #openstack-ironic23:04
*** bfournie has quit IRC23:12
*** srart has quit IRC23:44
*** akhilaki has joined #openstack-ironic23:46
*** yangyapeng has joined #openstack-ironic23:51
*** yangyapeng has quit IRC23:55
*** srart has joined #openstack-ironic23:59

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