Wednesday, 2017-12-20

*** bfournie has quit IRC00:00
*** bfournie has joined #openstack-ironic00:01
zshiTheJulia, hi00:14
TheJuliao/00:15
zshianything I could help, I noticed that you pinged me yesterday00:17
zshiI think it's about bios interface ?00:18
*** crushil__ has joined #openstack-ironic00:18
*** crushil has joined #openstack-ironic00:18
TheJuliazshi: It was, I left some comments and questions on the spec00:18
zshiI saw that, didn't yet reply00:18
zshiwill do today00:19
TheJuliaOkay, if you do in the next few hours, ping me and I’ll happy to take a look tonight00:19
TheJuliaAwesome00:19
zshisure, thanks !00:19
TheJuliaErr, I’ll be happy to. Apparently I need more coffee even though the day star is out of the sky00:20
*** crushil_ has joined #openstack-ironic00:22
*** crushil__ has quit IRC00:26
*** crushil has quit IRC00:26
*** crushil has joined #openstack-ironic00:26
*** threestrands has joined #openstack-ironic00:26
*** threestrands has quit IRC00:26
*** threestrands has joined #openstack-ironic00:26
*** yangyapeng has quit IRC00:27
*** yangyapeng has joined #openstack-ironic00:28
*** yangyapeng has quit IRC00:32
*** sasha2 has quit IRC00:38
zshiTheJulia, just added few comments there, please have a look at it if the star is still out in the sky :-)00:52
*** tuanla____ has joined #openstack-ironic00:54
*** dgautam has quit IRC00:54
*** phuongnh has joined #openstack-ironic01:06
*** hw_wutianwei has joined #openstack-ironic01:09
*** yangyapeng has joined #openstack-ironic01:12
*** asettle has quit IRC01:19
*** asettle has joined #openstack-ironic01:22
*** nitishk has quit IRC01:31
*** makowals has quit IRC01:43
*** makowals has joined #openstack-ironic01:44
*** rmcall has quit IRC01:47
*** snasir has quit IRC01:49
*** snasir has joined #openstack-ironic01:49
*** dgautam has joined #openstack-ironic01:50
*** dgautam has quit IRC01:55
*** edleafe- has joined #openstack-ironic01:57
*** edleafe has quit IRC01:57
*** dgautam has joined #openstack-ironic02:03
*** dprince has quit IRC02:06
*** crushil has quit IRC02:10
*** crushil_ has quit IRC02:10
*** karthiks has joined #openstack-ironic02:16
*** dgautam has quit IRC02:18
*** r-daneel has quit IRC02:20
openstackgerritMerged openstack/ironic master: Finalize migration to keystoneauth adapters  https://review.openstack.org/47882502:21
*** sasha2 has joined #openstack-ironic02:25
*** jlabarre has quit IRC02:26
*** Goneri has quit IRC02:30
*** rloo has quit IRC02:32
*** catintheroof has joined #openstack-ironic02:38
openstackgerritDao Cong Tien proposed openstack/ironic master: CI job for rescue mode  https://review.openstack.org/52870402:38
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Rescue extension for TinyIPA with DHCP network  https://review.openstack.org/44569202:40
*** catintheroof has quit IRC02:53
*** ijw has quit IRC02:58
*** rmcall has joined #openstack-ironic02:59
*** hshiina has joined #openstack-ironic03:02
TheJuliazshi: Thanks for the replies. I've gone ahead and replied. I still think we need another revision of the specification, for the purpose of clarity.03:02
zshiTheJulia, thanks, will update another version based on the comment03:02
*** threestrands has quit IRC03:03
zshiTheJulia, btw, can you give some guidance on how to fix this error : http://logs.openstack.org/62/511162/12/check/ironic-grenade-dsvm/8d93ea1/logs/grenade.sh.txt.gz03:04
*** threestrands has joined #openstack-ironic03:04
*** threestrands has quit IRC03:04
*** threestrands has joined #openstack-ironic03:04
zshiI'm working on the code for bios in parallel with the spec, seems it breaks the upgrade after adding NodeBIOS table03:05
*** threestrands has quit IRC03:05
zshinot sure what is missing to pass the ironic-grenade-dsvm ci job03:06
*** threestrands has joined #openstack-ironic03:06
*** threestrands has quit IRC03:06
*** threestrands has joined #openstack-ironic03:06
*** threestrands has quit IRC03:07
TheJulialoading the log03:07
*** threestrands has joined #openstack-ironic03:07
*** zhangfei has joined #openstack-ironic03:20
*** karthiks has quit IRC03:23
TheJuliazshi: your downrevision is wrong03:24
TheJuliaso it is not able to order things correctly03:24
TheJuliawhich is likely why the upgrade is failing03:25
zshiok, so it's not sth that I need to add in devstack , just changing the downrevision03:26
zshiTheJulia, thanks, going to check the revision and try03:26
zshiI used to hit the downrevision when doing py27 tox testing and fixed them03:27
TheJuliaare you developing off master?03:28
zshiI did rebase recently , there is a patch the changed the db03:28
TheJuliayeah, added a column to nodes03:28
TheJuliayou would have to change the downrevision to that change03:28
TheJulia(typically, this is the hard part, it goes quick once you get past such issues :)03:30
TheJuliabtw, i commented on the db model, storing the node uuid instead of the id matching out to the nodes table03:30
zshiI rebased on the rescue interface at : https://review.openstack.org/#/c/528609/103:30
patchbotpatch 528609 - ironic - Add bios_interface DB field, database migration, a...03:30
TheJuliazshi: one other thing, you may want to rename the table node_bios, since it is one entry per node03:31
zshigotcha, will check and fix them03:32
TheJuliazshi: I feel like you might be slightly out of order, database first, then objects/class updates at a high level, then node interface settings, then new drive rinterfaces, then rpc, then rest api03:34
TheJuliabut, it is also getting late here03:34
zshiyes, I was trying to fix some unit tests error which makes me mix the order03:35
zshiwill fix them in the later update03:38
TheJuliazshi: if you look at the addition of volume targets, it might provide some additional context03:52
TheJuliaat least, structural context into where you'll need to put a couple items, and you'll have to update a few tests as you go most likely, just depending on to the exact order you implement03:52
TheJuliaanyway, I'm going to go get some sleep, have a wonderful day!03:53
zshiI was following the Nodetag and recuse interface when adding bios, but will look at the volume taget03:53
zshithanks Julia, have a good sleep?!03:53
zshis/sleep?!/sleep!/g03:53
TheJuliazshi: ahh, node tags is very old and has not landed completely yet :(03:53
zshigood talk with you, appreciate all the suggestions!03:55
TheJulia:)03:55
*** udesale has joined #openstack-ironic03:57
*** threestrands_ has joined #openstack-ironic03:57
*** threestrands has quit IRC03:57
*** IgorYozhikov has quit IRC03:58
*** threestrands_ has quit IRC03:58
*** threestrands_ has joined #openstack-ironic03:59
*** IgorYozhikov has joined #openstack-ironic04:00
*** crushil has joined #openstack-ironic04:02
*** penick has joined #openstack-ironic04:02
*** penick has quit IRC04:07
*** penick_ has joined #openstack-ironic04:07
*** namnh has joined #openstack-ironic04:14
*** dgautam has joined #openstack-ironic04:19
*** dgautam has quit IRC04:23
*** vsaienk0 has joined #openstack-ironic04:23
*** MattMan has quit IRC04:29
*** MattMan has joined #openstack-ironic04:29
*** dgautam has joined #openstack-ironic04:32
*** vsaienk0 has quit IRC04:32
*** dgautam has quit IRC04:43
*** liuzz has joined #openstack-ironic04:44
*** dpawar has joined #openstack-ironic04:47
*** dgautam has joined #openstack-ironic04:47
*** ianw is now known as ianw_pto04:57
*** links has joined #openstack-ironic04:57
*** penick has joined #openstack-ironic05:18
*** namnh has quit IRC05:19
*** penick_ has quit IRC05:19
*** zhangfei has quit IRC05:26
*** AlexeyAbashkin has joined #openstack-ironic05:37
*** penick_ has joined #openstack-ironic05:38
*** zhangfei has joined #openstack-ironic05:38
*** yangyapeng has quit IRC05:40
*** yangyapeng has joined #openstack-ironic05:41
*** penick has quit IRC05:41
*** AlexeyAbashkin has quit IRC05:42
*** zhangfei has quit IRC05:44
*** yangyapeng has quit IRC05:45
*** crushil has quit IRC05:47
*** zhangfei has joined #openstack-ironic06:01
*** yangyapeng has joined #openstack-ironic06:03
*** penick_ has quit IRC06:04
*** penick has joined #openstack-ironic06:05
*** dgautam has quit IRC06:15
*** sasha2 has quit IRC06:15
*** marios has joined #openstack-ironic06:16
*** d0ugal has quit IRC06:17
*** jaganathan has joined #openstack-ironic06:19
*** rmcall has quit IRC06:21
*** d0ugal has joined #openstack-ironic06:23
*** trungnv has joined #openstack-ironic06:39
*** ianychoi has quit IRC06:44
*** e0ne has joined #openstack-ironic06:47
*** karthiks has joined #openstack-ironic06:49
*** karthiks has quit IRC06:54
*** threestrands_ has quit IRC06:57
*** karthiks has joined #openstack-ironic07:06
*** rcernin has quit IRC07:08
*** mjura has joined #openstack-ironic07:09
*** e0ne has quit IRC07:12
*** dsneddon has quit IRC07:12
*** penick has quit IRC07:14
*** penick has joined #openstack-ironic07:16
*** e0ne has joined #openstack-ironic07:19
*** liuzz has quit IRC07:23
*** e0ne has quit IRC07:25
*** vsaienk0 has joined #openstack-ironic07:28
*** zhangfei has quit IRC07:31
*** zhangfei has joined #openstack-ironic07:43
pas-hamorning ironic :)07:44
openstackgerritguanzuoyu proposed openstack/ironic-specs master: Fix an splling error of driver  https://review.openstack.org/52925507:55
*** stendulker has joined #openstack-ironic08:06
openstackgerritguanzuoyu proposed openstack/ironic-specs master: Fix an spelling error of driver  https://review.openstack.org/52925508:06
*** yangyapeng has quit IRC08:09
*** yangyapeng has joined #openstack-ironic08:11
*** tuanla____ has quit IRC08:18
*** tuanla____ has joined #openstack-ironic08:19
*** AlexeyAbashkin has joined #openstack-ironic08:19
*** hoonetorg has quit IRC08:19
openstackgerritThomas Herve proposed openstack/ironic master: Split pxe drivers  https://review.openstack.org/52917008:19
*** ccamacho has joined #openstack-ironic08:27
*** jtomasek has joined #openstack-ironic08:27
*** jtomasek has quit IRC08:31
*** jtomasek has joined #openstack-ironic08:32
*** hoonetorg has joined #openstack-ironic08:33
*** racedo has joined #openstack-ironic08:44
openstackgerritThomas Herve proposed openstack/ironic master: Split pxe drivers  https://review.openstack.org/52917008:47
*** pprokop has joined #openstack-ironic08:49
*** hjensas has quit IRC09:01
*** hshiina has quit IRC09:11
*** dougsz has joined #openstack-ironic09:11
*** mgoddard has joined #openstack-ironic09:11
*** dougsz has quit IRC09:15
*** e0ne has joined #openstack-ironic09:26
*** fragatina has joined #openstack-ironic09:30
*** fragatina has quit IRC09:31
*** AJaeger has joined #openstack-ironic09:31
AJaegerironic-inspector team, could you review https://review.openstack.org/528577 to improve your jobs, please?09:31
*** fragatina has joined #openstack-ironic09:31
patchbotpatch 528577 - ironic-inspector - Use native v3 tox jobs09:32
*** lucas-afk is now known as lucasagomes09:34
*** derekh has joined #openstack-ironic09:44
*** pprokop has quit IRC09:54
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: [ansible] add defaults to config  https://review.openstack.org/52571010:03
*** amarao has joined #openstack-ironic10:04
*** e0ne_ has joined #openstack-ironic10:08
*** MattMan_1 has joined #openstack-ironic10:11
*** e0ne has quit IRC10:11
*** MattMan has quit IRC10:12
*** vsaienk0 has quit IRC10:22
*** sambetts|afk is now known as sambetts10:27
*** aarefiev has joined #openstack-ironic10:30
*** vsaienk0 has joined #openstack-ironic10:31
*** jaianshu has joined #openstack-ironic10:37
*** hw_wutianwei has quit IRC10:37
*** anupn has quit IRC10:38
*** zhangfei has quit IRC10:42
*** hjensas has joined #openstack-ironic10:43
*** hjensas has joined #openstack-ironic10:43
*** anupn has joined #openstack-ironic10:46
*** krtaylor has quit IRC10:53
*** milan_ has joined #openstack-ironic11:01
*** dtantsur|afk is now known as dtantsur11:02
dtantsurmorning ironic11:02
dtantsurAJaeger: thanks, will do11:02
milan_morning dtantsur! :)11:02
milan_and morning Ironic! :)11:02
AJaegerthanks, dtantsur - and good morning11:03
*** yolanda__ has joined #openstack-ironic11:05
dtantsuris gerrit slow for everyone or is it just me?11:05
*** krtaylor has joined #openstack-ironic11:06
dtantsurjlvillal: I left -2 on patches that only touch ironic_tempest_plugin and nothing else11:07
*** AlexeyAbashkin has quit IRC11:08
*** yolanda has quit IRC11:08
*** gcb has quit IRC11:12
*** tuanla____ has quit IRC11:14
*** priteau has joined #openstack-ironic11:24
*** stendulker has quit IRC11:25
sambettsdtantsur: its not just you, its being a nightmare for me and keeps timing out11:30
*** vsaienk0 has quit IRC11:45
openstackgerritDmitry Tantsur proposed openstack/ironic master: ansible: handle mount of /sys the same way IPA does it  https://review.openstack.org/52640211:48
*** AlexeyAbashkin has joined #openstack-ironic11:52
*** phuongnh has quit IRC11:56
* milan_ relocates11:58
*** milan_ has quit IRC11:58
*** priteau has quit IRC12:03
*** priteau has joined #openstack-ironic12:03
*** priteau has quit IRC12:08
openstackgerritDao Cong Tien proposed openstack/ironic master: devstack: add support for rescue mode  https://review.openstack.org/52411812:09
openstackgerritDao Cong Tien proposed openstack/ironic master: CI job for rescue mode  https://review.openstack.org/52870412:10
dtantsursambetts, TheJulia, mind reviewing https://review.openstack.org/#/c/526402/ please? without this, the ansible deploy is not usable for me12:12
patchbotpatch 526402 - ironic - ansible: handle mount of /sys the same way IPA doe...12:12
sambettsdtantsur: su12:12
dtantsurpas-ha: updated ^^^ with vsaienk0's comments12:12
sambettssure*12:12
*** vsaienk0 has joined #openstack-ironic12:19
*** bfournie has quit IRC12:23
*** bfournie has joined #openstack-ironic12:23
*** dpawar has quit IRC12:26
*** lucasagomes is now known as lucas-hungry12:26
ricardoasmorning, ironic!12:28
*** bfournie has quit IRC12:28
*** jlabarre has joined #openstack-ironic12:29
*** aarefiev has quit IRC12:33
*** crushil has joined #openstack-ironic12:35
*** crushil_ has joined #openstack-ironic12:37
*** crushil__ has joined #openstack-ironic12:37
*** gcb has joined #openstack-ironic12:39
*** crushil has quit IRC12:42
*** gcb has quit IRC12:47
*** crushil__ has quit IRC12:51
*** dpawar has joined #openstack-ironic12:51
*** crushil_ has quit IRC12:51
*** yolanda__ is now known as yolanda12:53
openstackgerritShivanand Tendulker proposed openstack/ironic master: Adds rescue_interface to base driver class  https://review.openstack.org/50933512:56
openstackgerritShivanand Tendulker proposed openstack/ironic master: Adds RPC calls for rescue interface  https://review.openstack.org/50933612:56
*** crushil_ has joined #openstack-ironic13:03
*** crushil__ has joined #openstack-ironic13:03
*** dmellado has quit IRC13:05
*** dprince has joined #openstack-ironic13:08
*** yangyapeng has quit IRC13:08
*** yangyapeng has joined #openstack-ironic13:09
*** e0ne has joined #openstack-ironic13:10
*** e0ne_ has quit IRC13:13
jrollmorning ironic13:13
*** yangyapeng has quit IRC13:13
*** openstackgerrit has quit IRC13:13
-openstackstatus- NOTICE: gerrit is being restarted due to extreme slowness13:14
jrollhttps://content.pivotal.io/blog/what-is-a-unit-test-the-answer-might-surprise-you13:14
jrollgood stuff13:14
*** dmellado has joined #openstack-ironic13:15
*** openstackgerrit has joined #openstack-ironic13:17
openstackgerritMerged openstack/ironic master: Remove deprecated ironic.common.policy.enforce()  https://review.openstack.org/52783113:17
openstackgerritMerged openstack/ironic-inspector master: Use native v3 tox jobs  https://review.openstack.org/52857713:18
*** dmellado has quit IRC13:19
openstackgerritHarald Jensås proposed openstack/ironic master: Use openstack port create instead of neutron port-create  https://review.openstack.org/52931113:20
*** dmellado has joined #openstack-ironic13:21
openstackgerritMerged openstack/ironic-inspector master: Imported Translations from Zanata  https://review.openstack.org/52845313:21
*** jaganathan has quit IRC13:23
dtantsurmorning jroll13:24
jroll\o :)13:24
jrolldtantsur: you may be interested in this spec, btw https://review.openstack.org/#/c/529135/13:24
patchbotpatch 529135 - nova-specs - Support aggregate affinity scheduler filters13:24
jrolltl;dr this accomplishes affinity for ironic13:24
dtantsursounds promising13:24
dtantsurthanks13:24
jrollnp13:24
sambettsjroll: node to node affinity but not node to conductor affinity right?13:25
*** milan_ has joined #openstack-ironic13:26
jrollsambetts: affinity (or anti-affinity) for instances based on arbitrary groups of nodes13:26
jrolle.g. rack affinity, power domain affinity13:26
jroll"I want three instances all in different racks"13:26
*** hw_wutianwei has joined #openstack-ironic13:26
dtantsurjroll: looks awesome. the only thing is that we're way past nova spec freeze for queens..13:27
jrollheh, yeah, just a heads up13:27
jroll(I forgot about that detail)13:28
sambettsjroll: cool, thats great for nova users, /me has been thinking about node to conductor affinity which is required in larger ironic deploys to force node onto conductor that are nearest them13:28
jrollsambetts: yep, that's on my list in the near future as well13:28
*** lucas-hungry is now known as lucasagomes13:28
jrollor put another way: how do I manage a small remote DC without dumping an entire control plane there13:29
sambettsjroll: yup, or the problem space that I was targetting, in a large spine + leaf L3 network, how do I make sure the node's in a leaf end up on a conductor also in the leaf13:31
jrollyep, very similar but different :)13:31
*** AJaeger has left #openstack-ironic13:31
jrollsuspect that remote DC I mentioned is just a leaf from the L3 perspective13:32
sambettsas long as there is a L3 route between your main DC and the remote DC13:32
openstackgerritShivanand Tendulker proposed openstack/ironic master: Adds rescue_interface to base driver class  https://review.openstack.org/50933513:32
openstackgerritShivanand Tendulker proposed openstack/ironic master: Adds RPC calls for rescue interface  https://review.openstack.org/50933613:32
jrollright13:32
sambettsif the networks are different it might get weird although in theory it could work13:33
jrollI'm sure it's more complicated than "just another leaf", but you get my point :)13:34
sambettsyeah13:34
dtantsuris anyone doing anything around ironic-conductor location awareness?13:34
*** jaianshu has quit IRC13:34
sambettsthat's what we were just discussing13:34
jrollI am not yet, but will be, hopefully sooner than later13:34
jrollhave some other things to tackle first13:34
jrollunless sam is going to do it all for me :D13:35
sambettsI have an RFE up with one proposal tied to network specifically, but I've also been thinking about a different one which is more generic13:35
* sambetts needs to write that one up 13:35
dtantsursambetts: link handy?13:35
sambettshttps://bugs.launchpad.net/ironic/+bug/173487613:36
openstackLaunchpad bug 1734876 in Ironic "[RFE] Add physical network awareness to hash ring placement" [Wishlist,Confirmed]13:36
dtantsuryeah, I was thinking of just a random field called "location" on nodes, and a matching field in conductor's ironic.conf13:36
dtantsurI suspect that will cover the case of physical networks as well13:37
sambettsdtantsur: thats basically what my second proposal is, except I was thinking more tag style, where it find the best match13:37
dtantsurmmm, this will work, though it may be an overcomplication (esp. thinking how it plays with the hash ring)13:37
sambettse.g. a conductor might have "network1, rack10, rack11" and a node might have "rack10"13:38
jaypipessambetts, dtantsur, jroll, TheJulia: https://review.openstack.org/#/c/529201/13:38
patchbotpatch 529201 - nova - WIP Support aggregate affinity filters13:38
jrollfeels like ironic should be able to be aware of multiple groups (rack/power/network/etc) that a node is in, I'm just struggling to think through if the conductor cares13:38
jrolljaypipes: working my way there this morning :)13:39
jaypipesw00t.13:39
sambettsjroll: personally the more I thought about it making the node to conductor placement arbetrity not directly ties to any physical propertys made the most sense13:40
jrollsambetts: yeah, I agree with that. just a question of multiple groupings or a single one13:41
jrollof course, if we agree on just allowing one grouping per node, we'll land the code about the same time someone has a use case for more13:41
dtantsurheh13:42
dtantsurthere'll be a use case for every craziness in the world :)13:42
jaypipesdtantsur: you can say that again.13:42
sambettsso the way I was thinking about it was a conductor would have a list of tags, a node would have a list of tags, the hashring would score conductors based on how many of the tags matched between the node and conductor, and then put the node on the conductor with the highest score13:42
dtantsurwill we have priorities of tags? :) like, what is better, to match network1 or rack11?13:43
sambettsall equal13:43
* jroll loves specs being written in irc13:43
*** jcoufal has joined #openstack-ironic13:43
dtantsurwe can schedule a call in January :) if we scope some problems to go over13:44
jaypipesdtantsur: sounds like you guys are writing a .... scheduler :)13:44
dtantsurjaypipes: NO WAY :D13:44
jrolljaypipes: don't tell them13:44
jaypipes:P13:44
sambettsjaypipes: or a .... placement engine...13:44
jaypipesgasp13:44
sambetts:O13:44
dtantsurI wish openstack had a scheduler...13:44
jaypipes:)13:44
sambetts#cantwejustuseplacements13:44
dtantsurbut, hmm.. there is a point to that.13:45
*** edleafe- is now known as edleafe13:45
dtantsurdunno what to do about standalone deployments13:45
jrollit's worth exploring, yeah13:45
jaypipesjroll: I see you saw I named you as a co-conspirator on the aggregate-affinity work ;)13:45
jroll:)13:45
dtantsurjaypipes: good job, make this slacker work!13:45
*** SerenaFeng has joined #openstack-ironic13:45
dtantsur:D13:45
jaypipeshehe13:45
*** yangyapeng has joined #openstack-ironic13:45
sambettsif we can speed up splitting placements out from nova, then standalone could just use it right?13:45
jrolldtantsur: at the point where you need to manage locality of conductors, I feel like you might be past the point where standalone ironic works, dunno13:46
jaypipessambetts: you can use it as-is now.13:46
jroll(though I'd love it to work for any use case)13:46
dtantsurjroll: if you use ironic as a provisioner for your data center, locality is quite important, I suspect13:46
dtantsurbut we may wait for TheJulia to weigh in13:46
sambettsjroll: NobodyCam needs this locatlity in standlone ASAP, he's hacking it into his deployment using custom driver names13:46
jrolldtantsur: yeah maybe13:46
jrollwell then13:47
dtantsurtotally unrelated: jroll are you in easter time now?13:47
jrolldtantsur: eastern yes13:47
jrolleaster time is a few months yet13:47
dtantsurLOL13:47
TheJuliagood morning13:48
dtantsurokay, I just remember you showing up on IRC much later13:48
dtantsurmorning TheJulia13:48
*** dpawar has quit IRC13:48
sambettsjaypipes: will it run happily without anything else of nova running?13:48
jrollmorning TheJulia :)13:48
TheJuliaI see I should go drink very strong coffee13:48
*** trown|outtypewww is now known as trown13:48
jrollI'd just detach :P13:48
jrolldtantsur: yeah, was in california for a week. but also, my wife begins work around 3pm now, so still trying to decide the best schedule :)13:49
jaypipessambetts: yes... if you create the API database separately it should run happy as a clam..13:49
sambettsjaypipes: awesome!13:49
dtantsurjroll: that's tough. 3pm your time is, I suspect, evening my time13:49
jaypipessambetts: which reminds me... I should work with cdent to get a fully separate package for placement done.13:50
jrolldtantsur: indeed, I'm utc-513:50
dtantsurTheJulia: when you get enough coffee, could you please help move https://review.openstack.org/526402 forward?13:51
patchbotpatch 526402 - ironic - ansible: handle mount of /sys the same way IPA doe...13:51
*** catintheroof has joined #openstack-ironic13:52
*** trown is now known as trown|ruck13:52
dtantsurthat's the last bit that makes ansible deploy work on tripleo quite well13:53
* dtantsur has managed to apply custom kernel params ^_^13:53
TheJuliahas anyone pointed out that l2 networks can be present in different physical locations?13:54
jrollTheJulia: not this morning, no13:54
TheJuliaokay13:55
sambettsTheJulia: as in the same L2 network, because of a tunnel?13:55
TheJuliasambetts: yes, or via a good old fashioned connection13:55
TheJuliaor private fiber, etc13:55
sambettsTheJulia: yeah, this is why I don't think my original proposal based on physical network will work very well, because the definition of a physical network we are stealing from neutron is the same L2 infrastructure where vlan X is the same vlan X13:57
sambettsto everytrhing on physical network 113:58
sambettsbut vlan X is different vlan X on physical network 213:58
*** dpawar has joined #openstack-ironic13:58
TheJuliaor they could be the same on physical network 313:58
TheJuliabecause they have been bridged someplace13:58
TheJuliabut I'm not sure that is something we need to care about13:59
* TheJulia needs just a IV of coffee13:59
*** bfournie has joined #openstack-ironic14:00
*** catintheroof has quit IRC14:02
dtantsursambetts: mind +A https://review.openstack.org/#/c/526402/ it just passed the CI?14:04
patchbotpatch 526402 - ironic - ansible: handle mount of /sys the same way IPA doe...14:04
*** catintheroof has joined #openstack-ironic14:04
sambettsdtantsur: done14:04
openstackgerritNikolay Fedotov proposed openstack/ironic-lib master: Add wait_for_disk to destroy_disk_metadata function  https://review.openstack.org/52932514:04
jrolljaypipes: that filter patch lgtm overall14:05
*** rloo has joined #openstack-ironic14:07
*** logan- has quit IRC14:08
*** yangyapeng has quit IRC14:09
*** yangyapeng has joined #openstack-ironic14:09
TheJuliaIf anyone wants to look at a rescue related tinyipa patch, https://review.openstack.org/#/c/445692 I made some changes last night that should be better behavior overall14:09
patchbotpatch 445692 - ironic-python-agent - Rescue extension for TinyIPA with DHCP network14:09
*** logan- has joined #openstack-ironic14:10
*** sasha2 has joined #openstack-ironic14:10
* TheJulia goes back to read from discussion earlier today14:12
openstackgerritRuby Loo proposed openstack/ironic-inspector stable/pike: Use native v3 tox jobs  https://review.openstack.org/52933114:14
sambettsjroll, dtantsur: https://bugs.launchpad.net/ironic/+bug/1739426 tried to put my inital thoughts regarding placement into an RFE instead of just on IRC logs, no implementation details (using the placements API might be a solution) but just a high level concept of the matching I was thinking about14:15
openstackLaunchpad bug 1739426 in Ironic "[RFE] ironic conductor - node affinity placement" [Wishlist,New] - Assigned to Sam Betts (sambetts)14:15
*** vsaienk0 has quit IRC14:16
jrollcool, thanks sambetts14:17
TheJuliajroll: I think the conductor needs to care, because If I want a single pane of glass for my baremetal, and place conductors in logical fault domains or in places where they need to do specific work that is logical to that grouping of hardware. The case I like to roll around in my brain is "Customer Y owns racks 9 and 10. We manage them, but we need a dedicated conductor to talk to that environment". In a14:18
TheJuliastandalone case, they could just be using it to deploy OS images, but it is still a valid use case for ironic even if the customer or management (which could be the admins of the cloud or not) manages the hardware.14:18
*** zhangfei has joined #openstack-ironic14:20
jrollTheJulia: right, what I meant was does the conductor ever need to care about racks AND power domains, for instance. or just racks.14:20
jrollor just power domains.14:21
TheJuliasambetts: I was thinking have the ability to configure conductors to be in their own distinct grouping, and then nodes could be configured ownership, logical grouping, as well to rack/power/etc.14:21
jrollunless you're solely talking about the standalone case, in which case I do agree it's important14:21
*** zhangfei has quit IRC14:22
TheJuliajroll:  I absolutely believe so, but I might change my mind.14:22
* TheJulia poners14:22
*** catinthe_ has joined #openstack-ironic14:22
jrollTheJulia: yeah, I'm just struggling to find the use case. probably just needs a coffee application14:23
TheJuliaI've built racks with their own power failure domains. I'e had racks conneted between buildings that were also in their own power/networking failure domains, that were all not in the same rack, but were the same customer. In both facilities I had other customers that could be on shared physical networks14:23
TheJulias/poners/ponders14:23
*** catintheroof has quit IRC14:26
*** vsaienk0 has joined #openstack-ironic14:27
TheJuliasambetts: I'm not convinced on tagging and scoring to impact the hash ring, I was thinking more explicit nature because if an operator is going down the road of his level of isolation while still wanting a single pane of glass, then they are going to need to be very explicit configuraiton wise. I guess maybe some sort of tagging might work, but we likely need to provide visibility to nodes that can't be14:27
TheJuliamanaged because they fall out of the scope of the presently running conductors14:27
sambettsTheJulia: with my current proposal node can't ever fall out of scape14:28
sambettsTheJulia: with my current proposal node can't ever fall out of scope14:28
sambettsat the point where a node doesn't match any conductors tags all conductors score 0 and basically how the hashring works today will apply14:29
TheJuliajroll: Perhaps, PII cases where only specific systems may talk to the PII machines, and those conductors would naturally have a far higher logging level for auditing/accounting14:29
sambettsit ensures our HA model remains intact14:29
TheJuliasambetts: I don't think that is reasonable, at some point we need to introduce the fact that a node is unreachable14:29
sambettsTheJulia: I think if we do affity and antiaffinty of node to conductors correctly it would acheive that, because the anti affinity would let to block node's from being placed onto certain conductors, (although I'm not sure how to express that logic using my tag system) I was thinking anti-tags, e.g. if a node matches any of these tags do not place it on me14:30
TheJuliait is absolutely impossible to maintain the concept that any conductor can manage any machine configured in ironic, especially in highly regulated environments14:31
sambettsoh absolutly14:31
sambettslike I said if we do affinity and antiaffinity correctly it *should* provide the controls we need I think (my proposal is just how far I've thought about it so far, and it only really acheives best conductor affinity part)14:32
*** openstackgerrit has quit IRC14:33
TheJuliaso there is one wrinkle I think, we would still need to make tenant ownership a reality, and likely at some point need to offer restricting the view and possibilities, at the same time, I'm not sure such tags should be visible to such a "power user"14:33
TheJuliacould be separate, but it would start touching the same code in the end so it might just be easy to add the starting points in at the same time and finish up api level stuff affinity/anti-affinity14:34
sambettsperhaps, although I could think that we can provide tenent onwership without having to do differnent conductor placment :14:35
sambetts:/14:35
*** rbudden has joined #openstack-ironic14:35
TheJuliasambetts: did I tell you about my experiences working for three highly regulated competing companies, that had reason to work together on one area? :)14:36
dtantsursounds like fun14:37
TheJulia"fun"14:37
sambettsI believe you mentioned it at the summit, when me and NobodyCam were chatting14:37
sambettsabout placement14:38
*** trown|ruck is now known as trown|brb14:39
TheJuliawe had specific cross customer security requirements, which went beyond rack level isolation, except we also had particular cases where multiple customers could be in the same rack, but still need isolated management for their own island.14:40
TheJuliaAt that time, I got to spend lots of time with auditors and the ticketing system records :(14:40
TheJuliaUgh, those memories. Too early to apply whiskey :(14:40
*** trown|brb is now known as trown14:47
sambettsTheJulia, jroll: added an edit at the bottom of the RFE with anti-affinity idea's see what you think14:49
sambettsunlike the main part I've not fully scoped out that idea14:50
*** openstackgerrit has joined #openstack-ironic14:50
openstackgerritMichael Turek proposed openstack/ironic-python-agent master: Use lshw in place of dmidecode for the default hardware manager  https://review.openstack.org/50179914:50
*** trown is now known as trown|ruck14:52
*** crushil__ has quit IRC14:56
*** crushil_ has quit IRC14:56
*** catinthe_ has quit IRC15:02
*** catintheroof has joined #openstack-ironic15:03
*** SerenaFeng has quit IRC15:05
*** SerenaFeng has joined #openstack-ironic15:07
*** makowals has quit IRC15:08
*** catintheroof has quit IRC15:08
dtantsurTheJulia: FYI https://bugs.launchpad.net/nova/+bug/173944015:24
openstackLaunchpad bug 1739440 in OpenStack Compute (nova) "Ironic virt driver is hardcoding Ironic API microversion" [Undecided,New]15:24
dtantsurnow waiting for mriedem to show up15:24
openstackgerritMerged openstack/ironic-python-agent master: Updated from global requirements  https://review.openstack.org/52840415:24
*** crushil_ has joined #openstack-ironic15:30
*** crushil__ has joined #openstack-ironic15:30
*** karthiks has quit IRC15:32
*** dpawar has quit IRC15:36
*** fragatina has quit IRC15:37
*** fragatina has joined #openstack-ironic15:37
*** hjensas has quit IRC15:47
*** pcaruana has joined #openstack-ironic15:51
*** jcoufal has quit IRC16:04
*** SerenaFeng has quit IRC16:09
NobodyCamGood Morning Ironic'ers :)16:10
dtantsurmorning NobodyCam16:11
NobodyCam:) hey hey dtantsur Good Morning :)16:11
*** jcoufal has joined #openstack-ironic16:24
*** mjura has quit IRC16:25
*** vsaienk0 has quit IRC16:25
openstackgerritMerged openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/52886316:31
openstackgerritMerged openstack/ironic-inspector master: zuul: Remove duplicated TEMPEST_PLUGINS entry  https://review.openstack.org/52834616:31
*** rmcall has joined #openstack-ironic16:34
*** Goneri has joined #openstack-ironic16:37
*** AlexeyAbashkin has quit IRC16:37
*** rmcall has quit IRC16:38
*** links has quit IRC16:44
*** lucasagomes is now known as lucas-afk16:48
*** e0ne has quit IRC16:49
TheJuliagod morning NobodyCam16:51
TheJuliagood16:51
NobodyCam:) Go[o]d morning TheJulia :) :p16:53
*** jtomasek has quit IRC16:54
NobodyCamrunning out to the doctors, back in a while :P16:54
rloogood morning NobodyCam, TheJulia, dtantsur and all the wonderful ironic folks that are out there!16:54
NobodyCamGood Morning rloo :)16:54
dtantsurmorning rloo16:57
dtantsurjlvillal: morning, do you plan on proposing stable jobs to ironic-tempest-plugin?16:57
*** marios has quit IRC16:57
rloodtantsur, TheJulia: easy cherry-pick'd patch: https://review.openstack.org/#/c/529331/16:57
patchbotpatch 529331 - ironic-inspector (stable/pike) - Use native v3 tox jobs16:57
dtantsur+A16:58
rloodtantsur, TheJulia: this is an equiv for ipa that i started (and forgot) awhile ago: https://review.openstack.org/#/c/522025/16:59
patchbotpatch 522025 - ironic-python-agent - Use 'functional' tox environment16:59
*** ijw has joined #openstack-ironic16:59
openstackgerritSam Betts proposed openstack/ironic-inspector master: [WIP][DNM] Experiment with batch through iptables-restore  https://review.openstack.org/52794316:59
dtantsurrloo: I wonder if we actually need to maintain compatibility with older 'func' env.. we don't even raise a warning, maybe just remove it (the same as in the inspector patch)?16:59
rloodtantsur: i did that cuz someone wanted that done in another project. personally, i don't care.16:59
rloodtantsur: thinking it was pas-ha17:00
dtantsurI don't care much either :)17:00
rloodtantsur: if we don't care, then better to just get rid of that now, vs having to do it later17:01
rloodtantsur: although TheJulia just +2'd it. TheJulia what do you think? leave the 'func' shim or remove?17:02
*** priteau has joined #openstack-ironic17:04
*** priteau has quit IRC17:08
TheJuliaI'm kind of in the not care category17:11
openstackgerritSam Betts proposed openstack/ironic master: [devstack] Allow setting TFTP max blocksize  https://review.openstack.org/52873717:14
*** udesale has quit IRC17:18
openstackgerritRuby Loo proposed openstack/ironic-python-agent master: Use 'functional' tox environment  https://review.openstack.org/52202517:20
rlooTheJulia: I removed the shim since you don't care either. No sense creating extra work and removing later if we can just remove now17:21
jlvillaldtantsur, Uh. I had forgot about stable jobs.17:21
jlvillalAlso I haven't investigated how hard/easy it is to do17:21
*** crushil_ has quit IRC17:22
*** crushil__ has quit IRC17:22
*** dprince has quit IRC17:23
*** d0ugal has quit IRC17:23
*** trown|ruck is now known as trown|lunch17:25
*** hw_wutianwei has quit IRC17:26
dtantsurjlvillal: I suspect it's better to ask infra17:27
jlvillaldtantsur, agreed17:27
dtantsurjlvillal: https://github.com/openstack-infra/tripleo-ci/blob/master/zuul.d/ovb-jobs.yaml#L18917:28
jlvillaltriple-o does it all ;)17:29
dtantsurI wonder if this is enough17:29
dtantsurheh, yeah :)17:29
dtantsurit should be easy to just try it and see17:29
jlvillaldtantsur, I wonder. Does that mean it tries to use stable for everything including tempest?17:29
dtantsurjlvillal: IIRC it falls back to master if a branch is not found17:29
jlvillalyeah, trying will be the test17:29
sambettsdtantsur: correct, if branch isn't found it falls back to master17:30
jlvillalsambetts, Were you able to successfully migrate to using the openstack/ironic-tempest-plugin repo?17:30
sambettsso you can safely specify globally checkout stable/pike, but it'll always use master ironic-tempest-plugin because there are no stable branches17:31
jlvillalJust wondering if any hiccups17:31
sambettsjlvillal: yup, seemed to work flawlessly once I killed DEVSTACK_GATE_TEMPEST_ALL_PLUGIN17:31
jlvillalsambetts, Awesome! :)17:31
sambettsjlvillal: testing it out here: https://review.openstack.org/#/c/514312/17:32
patchbotpatch 514312 - ironic - [DNM] Testing Cisco Third Party CI17:32
jlvillalthanks17:32
dtantsurtime to go, see you tomorrow17:34
*** dtantsur is now known as dtantsur|afk17:34
sambettso/ dtantsur|afk17:34
*** d0ugal has joined #openstack-ironic17:35
jlvillalrloo, Last night after work (yes I'm crazy) I looked into that DEBUG message issue. Ugh. Seems to be something with multiple tests running.17:42
jlvillalrloo, If I run just single tests one at a time. I never see the issue. If I get a batch of 8 or so, then it starts showing up.17:42
jlvillalrloo, Anyway did a bunch of debugging. Not sure how to fix it yet. Will need more time to troubleshoot. But not tonight since I'm going to go see Jumanji :)17:43
*** milan_ has quit IRC17:43
jlvillalrloo, And I notice nova has similar issue in their log files17:43
*** etingof has quit IRC17:43
rloojlvillal: yeah, i wondered, since you copied it, thought it was doing that elsewhere too17:47
rloojlvillal: if we are ok with having those debug, then you can mention it in your patch/commit msg :)17:47
jlvillalrloo, Maybe over Christmas I'll get bored and look more :)17:47
rloojlvillal: or ping sean? i saw his name in a comment. maybe he wrote that code.17:48
jlvillalrloo, Yeah I think sdague did write it. I'll flail about a bit more and see what I figure out. I'll likely ping him17:48
rloojlvillal: flailing is good for the soul :D17:49
jlvillalrloo, Sometimes. Usually I learn something :)17:49
rlooTheJulia: want to +A this? https://review.openstack.org/#/c/522025/17:50
patchbotpatch 522025 - ironic-python-agent - Use 'functional' tox environment17:50
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: [ansible] add defaults to config  https://review.openstack.org/52571017:50
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: WiP gate job for ansible interface  https://review.openstack.org/52938317:50
*** rmcall has joined #openstack-ironic17:50
TheJuliarloo: but the job is non-voting17:51
TheJuliaso.... we should ideally wait until it passes17:51
* TheJulia is sure it will, but still17:51
rlooTheJulia: sorry, what am i thinking. sorry about that. too many patches in diff states...17:51
TheJuliarloo: no worries :)17:52
*** derekh has quit IRC17:52
* sambetts ---> home 17:53
sambettsNight all o/17:53
sambettscya'll tomrrow17:53
*** sambetts is now known as sambetts|afk17:53
openstackgerritJohn L. Villalovos proposed openstack/ironic-tempest-plugin master: WIP/DNM: Testing a stable/pike job...  https://review.openstack.org/52938617:54
*** crushil_ has joined #openstack-ironic17:56
*** crushil__ has joined #openstack-ironic17:57
rloobye sambetts|afk17:57
openstackgerritJohn L. Villalovos proposed openstack/ironic-tempest-plugin master: WIP/DNM: Testing a stable/pike job...  https://review.openstack.org/52938617:59
*** dprince has joined #openstack-ironic18:03
*** dsneddon has joined #openstack-ironic18:04
*** mgoddard has quit IRC18:06
*** AlexeyAbashkin has joined #openstack-ironic18:08
*** makowals has joined #openstack-ironic18:11
*** AlexeyAbashkin has quit IRC18:13
*** etingof has joined #openstack-ironic18:14
*** e0ne has joined #openstack-ironic18:16
*** etingof has quit IRC18:22
*** etingof has joined #openstack-ironic18:35
*** jtomasek has joined #openstack-ironic18:40
*** ijw has quit IRC18:52
openstackgerritMerged openstack/ironic-inspector stable/pike: Use native v3 tox jobs  https://review.openstack.org/52933118:57
*** mcevik has joined #openstack-ironic18:59
*** mcevik has quit IRC19:00
*** trown|lunch is now known as trown|brb19:08
*** trown|brb is now known as trown19:08
*** trown is now known as trown|ruck19:08
*** makowals has quit IRC19:15
*** ElCoyote_ has quit IRC19:19
openstackgerritJohn L. Villalovos proposed openstack/ironic master: WIP/DNM: See if can add a variable to the playbook...  https://review.openstack.org/52940119:28
openstackgerritJohn L. Villalovos proposed openstack/ironic-tempest-plugin master: WIP/DNM: Add a stable/pike and stable/ocata test job  https://review.openstack.org/52938619:29
*** pcaruana has quit IRC19:29
*** ijw has joined #openstack-ironic19:32
*** ijw has quit IRC19:43
*** ijw has joined #openstack-ironic19:44
*** pcaruana has joined #openstack-ironic19:48
*** jtomasek has quit IRC19:50
*** pcaruana has quit IRC19:54
*** ijw has quit IRC19:59
*** dprince has quit IRC20:01
*** ijw has joined #openstack-ironic20:01
*** ijw has quit IRC20:05
*** pcaruana has joined #openstack-ironic20:06
*** pcaruana has quit IRC20:13
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: WiP gate job for ansible interface  https://review.openstack.org/52938320:15
*** yangyape_ has joined #openstack-ironic20:25
*** yangyapeng has quit IRC20:26
openstackgerritMerged openstack/ironic-python-agent master: Use 'functional' tox environment  https://review.openstack.org/52202520:37
*** links has joined #openstack-ironic20:41
*** links has quit IRC20:47
openstackgerritMerged openstack/ironic master: ansible: handle mount of /sys the same way IPA does it  https://review.openstack.org/52640220:51
*** jcoufal has quit IRC20:53
*** catintheroof has joined #openstack-ironic20:54
*** bfournie has quit IRC20:58
*** dprince has joined #openstack-ironic21:00
*** ijw has joined #openstack-ironic21:01
*** ijw has quit IRC21:02
*** ijw has joined #openstack-ironic21:02
*** priteau has joined #openstack-ironic21:04
*** crushil__ has quit IRC21:05
*** crushil_ has quit IRC21:05
*** priteau has quit IRC21:09
openstackgerritJohn L. Villalovos proposed openstack/ironic-tempest-plugin master: Add a stable/pike and stable/ocata test job  https://review.openstack.org/52938621:13
openstackgerritJohn L. Villalovos proposed openstack/ironic-tempest-plugin master: Add a stable/pike and stable/ocata test job  https://review.openstack.org/52938621:15
*** catinthe_ has joined #openstack-ironic21:23
*** catintheroof has quit IRC21:25
*** catinthe_ has quit IRC21:35
*** catintheroof has joined #openstack-ironic21:35
*** bnemec has quit IRC21:36
*** threestrands_ has joined #openstack-ironic21:36
*** rmcall has quit IRC21:38
*** catintheroof has quit IRC21:40
*** ijw has quit IRC21:50
*** trown|ruck is now known as trown|outtypewww21:50
*** priteau has joined #openstack-ironic21:51
*** priteau has quit IRC21:56
*** Goneri has quit IRC22:07
openstackgerritJohn L. Villalovos proposed openstack/ironic master: zuul: Remove some redundancy by consolidating the 'post.yaml' files  https://review.openstack.org/52940122:15
*** e0ne has quit IRC22:23
*** Goneri has joined #openstack-ironic22:23
*** e0ne has joined #openstack-ironic22:26
*** e0ne has quit IRC22:26
*** e0ne has joined #openstack-ironic22:27
*** e0ne has quit IRC22:28
*** e0ne has joined #openstack-ironic22:28
*** e0ne has quit IRC22:28
openstackgerritJohn L. Villalovos proposed openstack/ironic master: zuul: Add ability to specify a 'branch_override' value  https://review.openstack.org/52942122:29
*** rcernin has joined #openstack-ironic22:32
openstackgerritJohn L. Villalovos proposed openstack/ironic-tempest-plugin master: Add a stable/pike and stable/ocata test job  https://review.openstack.org/52938622:32
*** jappleii__ has joined #openstack-ironic22:35
*** threestrands_ has quit IRC22:36
*** exodusftw has quit IRC22:36
*** ijw has joined #openstack-ironic22:40
*** exodusftw has joined #openstack-ironic22:44
openstackgerritMerged openstack/ironic-specs master: Fix an spelling error of driver  https://review.openstack.org/52925522:51
*** ijw has quit IRC22:53
*** etingof has quit IRC22:56
openstackgerritRuby Loo proposed openstack/ironic master: Document the check done in "ironic-dbsync upgrade"  https://review.openstack.org/52911722:57
*** rmcall has joined #openstack-ironic23:09
*** rmcall has quit IRC23:10
*** dprince has quit IRC23:10
*** rmcall has joined #openstack-ironic23:10
*** etingof has joined #openstack-ironic23:11
openstackgerritJohn L. Villalovos proposed openstack/ironic-tempest-plugin master: Add a stable/pike and stable/ocata test job  https://review.openstack.org/52938623:27
openstackgerritJohn L. Villalovos proposed openstack/ironic-tempest-plugin master: Add a stable/pike and stable/ocata test job  https://review.openstack.org/52938623:29
*** ijw has joined #openstack-ironic23:53
*** ijw has quit IRC23:59

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