Monday, 2017-12-18

*** yangyapeng has quit IRC00:25
*** yangyapeng has joined #openstack-ironic01:10
*** phuongnh has joined #openstack-ironic01:12
*** trungnv has joined #openstack-ironic01:22
*** namnh has joined #openstack-ironic01:27
*** TuanLA has joined #openstack-ironic01:31
*** zshi has quit IRC01:31
*** trungnv has quit IRC01:34
*** trungnv has joined #openstack-ironic01:34
*** namnh has quit IRC01:36
*** hshiina has joined #openstack-ironic01:39
*** TuanLA has quit IRC01:40
*** hoonetorg has quit IRC01:41
*** yuanying has joined #openstack-ironic01:42
-openstackstatus- NOTICE: The freenode network is currently the target of automated spam attacks, we have enabled temporary restrictions on targetted OpenStack channels which requires users to be logged on to NickServ. If you see spam in your channel, please report it in #openstack-infra. Thanks.01:47
*** TuanLA has joined #openstack-ironic01:48
*** snasir has quit IRC01:49
*** snasir has joined #openstack-ironic01:49
*** hoonetorg has joined #openstack-ironic01:50
*** baoli has joined #openstack-ironic01:50
*** zhangfei has joined #openstack-ironic01:51
*** zhangfei_ has joined #openstack-ironic01:53
*** zhangfei has quit IRC01:56
*** TuanLA has quit IRC02:02
*** Grenth has quit IRC02:04
*** Grenth has joined #openstack-ironic02:05
*** liuzz has joined #openstack-ironic02:18
*** baoli has quit IRC02:28
*** gcb has joined #openstack-ironic02:49
*** ChanServ sets mode: +r 02:53
*** Grenth has quit IRC02:57
openstackgerritHironori Shiina proposed openstack/python-ironicclient master: Accept None as a result of node validation in functional test  https://review.openstack.org/52860603:02
openstackgerritDao Cong Tien proposed openstack/python-ironicclient master: Add support for RESCUE and UNRESCUE provision states  https://review.openstack.org/40834103:12
openstackgerritzenghui.shi proposed openstack/ironic master: WIP: Add BIOSInterface to base driver class  https://review.openstack.org/50779303:19
openstackgerritzenghui.shi proposed openstack/ironic master: Add db migration and model for node bios table  https://review.openstack.org/51116203:19
openstackgerritzenghui.shi proposed openstack/ironic master: Add db api layer for node bios operations  https://review.openstack.org/51140203:19
openstackgerritzenghui.shi proposed openstack/ironic master: Add node BIOS support - RPC object  https://review.openstack.org/51171403:19
openstackgerritzenghui.shi proposed openstack/ironic master: Add Node BIOS support - REST API  https://review.openstack.org/51257903:19
openstackgerritzenghui.shi proposed openstack/ironic master: Add bios_interface DB field, database migration, and object entry for node bios functionality.  https://review.openstack.org/52860903:19
*** hshiina has quit IRC03:35
*** zhangfei_ has quit IRC03:40
*** rbudden has quit IRC04:05
*** udesale has joined #openstack-ironic04:12
*** MattMan has quit IRC04:28
*** MattMan has joined #openstack-ironic04:29
*** yangyapeng has quit IRC05:31
openstackgerritMerged openstack/ironic master: Use adapters for neutronclient  https://review.openstack.org/47617005:51
*** ijw has joined #openstack-ironic06:05
*** hjensas has quit IRC06:25
*** hjensas has joined #openstack-ironic06:26
*** namnh has joined #openstack-ironic06:37
*** namnh has quit IRC06:37
*** namnh has joined #openstack-ironic06:38
*** mcarden has quit IRC06:41
*** trungnv has quit IRC06:44
*** hoonetorg has quit IRC06:45
*** rcernin has quit IRC06:46
*** ijw has quit IRC06:47
*** phuongnh has quit IRC06:47
*** ijw has joined #openstack-ironic06:49
*** yolanda has quit IRC06:55
openstackgerritRushil Chugh proposed openstack/ironic master: Add XClarity Driver  https://review.openstack.org/51942706:57
*** hoonetorg has joined #openstack-ironic07:02
*** ijw has quit IRC07:05
*** phuongnh has joined #openstack-ironic07:08
*** ijw has joined #openstack-ironic07:36
*** ijw has quit IRC07:42
*** ijw has joined #openstack-ironic07:50
*** tuanla____ has joined #openstack-ironic07:51
*** ijw has quit IRC08:00
*** ijw has joined #openstack-ironic08:02
*** gcb has quit IRC08:04
*** rcernin has joined #openstack-ironic08:06
*** ijw has quit IRC08:07
*** ijw has joined #openstack-ironic08:07
*** ijw has quit IRC08:12
*** charcol has joined #openstack-ironic08:20
*** e0ne has joined #openstack-ironic08:24
etingofhappy Monday morning, Ironicers! ;)08:50
*** stendulker has joined #openstack-ironic08:51
*** ccamacho has joined #openstack-ironic08:51
*** ijw has joined #openstack-ironic08:57
*** ijw has quit IRC09:01
*** ramineni has joined #openstack-ironic09:05
*** lucas-afk is now known as lucasagomes09:23
*** e0ne_ has joined #openstack-ironic09:48
*** e0ne_ has quit IRC09:50
*** e0ne has quit IRC09:51
*** e0ne has joined #openstack-ironic09:51
*** e0ne has quit IRC09:52
*** e0ne has joined #openstack-ironic09:53
*** ijw has joined #openstack-ironic09:58
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: roles: bifrost-create-vm-nodes: Set CPU model to 'host-model'  https://review.openstack.org/52867709:58
*** ijw has quit IRC10:03
*** liuzz has quit IRC10:05
*** amarao has joined #openstack-ironic10:16
*** phuongnh has quit IRC10:16
*** udesale has quit IRC10:27
*** d0ugal_ has quit IRC10:36
*** udesale has joined #openstack-ironic10:38
*** makowals has joined #openstack-ironic10:50
makowalsHello ironic10:51
makowalsI have noticed quite a lot of "WARNING requests.packages.urllib3.connectionpool [-] Connection pool is full, discarding connection: keystone.cern.ch: Full" in my ironic-api.log and  wonder where do they come from ...10:51
*** namnh has quit IRC10:52
makowalsI have set the limits for nofile and noproc to 65536, so I doubt that's it10:52
*** sambetts_ is now known as sambetts10:54
sambettsMorning all10:54
*** rcernin has quit IRC10:58
*** charcol has quit IRC10:58
*** ijw has joined #openstack-ironic11:00
pas-hamorning ironic :)11:00
*** udesale has quit IRC11:01
*** tuanla____ has quit IRC11:02
*** ijw has quit IRC11:05
*** ijw has joined #openstack-ironic11:09
*** ijw has quit IRC11:14
openstackgerritSam Betts proposed openstack/ironic-inspector master: [WIP][DNM] Experiment with batch through iptables-restore  https://review.openstack.org/52794311:15
*** d0ugal has joined #openstack-ironic11:16
sambettso/ pas-ha11:18
*** derekh has joined #openstack-ironic11:29
*** milan_ has joined #openstack-ironic11:36
*** stendulker has quit IRC11:45
raminenisambetts: hi11:53
raminenisambetts: one question, does ironic-inspector identifies sr-iov, dpdk capabilities11:54
raminenimilan_: dtantsur|afk .. ^^11:55
milan_hi ramineni11:56
milan_let me check11:56
sambettsramineni: we added a generic PCI devices plugin to the ironic-inspector https://docs.openstack.org/ironic-inspector/latest/user/usage.html#plugins11:56
sambettsramineni: which should allow you to detect sr-iov PCI devices if they can be detected by the kernel11:57
milan_sambetts ++12:00
*** ijw has joined #openstack-ironic12:10
*** lucasagomes is now known as lucas-hungry12:13
*** ijw has quit IRC12:15
openstackgerritDao Cong Tien proposed openstack/ironic-tempest-plugin master: WIP Rescue mode tempest tests  https://review.openstack.org/52869912:16
*** etingof has quit IRC12:17
*** hw_wutianwei has joined #openstack-ironic12:28
openstackgerritDao Cong Tien proposed openstack/ironic-tempest-plugin master: WIP Rescue mode tempest tests  https://review.openstack.org/52869912:33
*** ricardoas-afk has quit IRC12:35
*** ricardoas has joined #openstack-ironic12:42
*** hw_wutianwei has quit IRC12:42
*** udesale has joined #openstack-ironic12:46
openstackgerritDao Cong Tien proposed openstack/ironic master: WIP CI job for rescue mode  https://review.openstack.org/52870412:53
*** nicodemos has joined #openstack-ironic12:53
nicodemosgood morning. :)12:57
*** igordc has quit IRC13:08
*** igordc has joined #openstack-ironic13:08
*** lucas-hungry is now known as lucasagomes13:08
*** ijw has joined #openstack-ironic13:12
*** ijw has quit IRC13:17
*** charcol has joined #openstack-ironic13:17
*** ijw has joined #openstack-ironic13:17
*** ijw has quit IRC13:22
*** trown|outtypewww is now known as trown13:24
*** trown is now known as trown|ruck13:29
*** namnh has joined #openstack-ironic13:32
*** jrist has quit IRC13:39
openstackgerritPavlo Shchelokovskyy proposed openstack/networking-generic-switch master: DNM testing gates  https://review.openstack.org/52872313:46
*** jrist has joined #openstack-ironic13:54
jrollmorning ironic13:57
sambettshey jroll, early morning?13:57
jrollsambetts: dunno, it's nearly 9am :)13:57
sambettsoh wow... just realised its like 14:00 my time... /me needs to get lunch!13:58
jrolllol, indeed13:58
*** e0ne has quit IRC13:58
*** namnh has quit IRC14:00
*** namnh has joined #openstack-ironic14:01
*** ijw has joined #openstack-ironic14:01
*** namnh has quit IRC14:02
*** namnh has joined #openstack-ironic14:02
*** ChanServ sets mode: -r 14:07
*** dprince has joined #openstack-ironic14:07
*** rloo has joined #openstack-ironic14:07
*** e0ne has joined #openstack-ironic14:07
*** ijw has quit IRC14:08
*** jcoufal has joined #openstack-ironic14:13
openstackgerritSam Betts proposed openstack/ironic-inspector master: [WIP][DNM] Experiment with batch through iptables-restore  https://review.openstack.org/52794314:14
*** zhangfei has joined #openstack-ironic14:18
TheJuliaGood morning everyone14:21
*** rbudden has joined #openstack-ironic14:23
jrollmorning TheJulia :)14:24
*** racedo has joined #openstack-ironic14:27
*** jcoufal_ has joined #openstack-ironic14:29
*** jcoufal has quit IRC14:29
*** rloo has quit IRC14:32
*** rloo has joined #openstack-ironic14:33
*** trown|ruck is now known as trown|afk14:40
rloohi and Happy-Monday-Morning-before-winter-solstice everyone, including jroll & TheJulia14:46
sambettso/ ramineni14:47
sambettso/ rloo14:47
rloohiya sambetts!14:47
jrollhappy monday rloo!14:48
*** zhangfei has quit IRC14:53
*** bfournie_ has joined #openstack-ironic14:55
*** d0ugal has quit IRC14:56
*** bfournie has joined #openstack-ironic14:57
rloopas-ha: howdy, if you have some time, would be good to get this updated. i feel it in my bones, it will land this week... https://review.openstack.org/#/c/478825/14:57
patchbotpatch 478825 - ironic - Finalize migration to keystoneauth adapters14:57
*** bfournie_ has quit IRC14:59
pas-harloo: :) will hopefully do it today14:59
rloopas-ha: thx!15:00
rloopas-ha: qq about that work, do we need to get the inspector patches landed too? https://review.openstack.org/#/q/topic:bug/1699547+(status:open+OR+status:merged)15:02
*** hw_wutianwei has joined #openstack-ironic15:04
rloosambetts: wrt routed network support -- the subteam status report in our whiteboard needs to be updated... (L236)15:04
pas-hayep, I will take the over - aarefiev left Mirantis :( and AFAIK won't work on OpenStack, at least in previous capacity15:04
*** dprince has quit IRC15:04
*** dprince has joined #openstack-ironic15:05
rloopas-ha: OH, I didn't know that about aarefiev :-(15:08
sambetts:(15:08
jrolldo I remember correctly that someone was working on passing a node UUID to the agent to skip lookup recently? maybe sambetts? this was outside of penick's work to totally remove IPA->ironic communication15:08
*** d0ugal has joined #openstack-ironic15:08
sambettsnot me, but I also remember seeing it15:09
sambettsmaybe lucasagomes ?15:09
rloodoesn't ring a bell :-(15:09
* jroll will dig through reviews, it isn't on master15:09
jrollthanks15:09
lucasagomeshmm not sure if it was me, but, it does ring me a bell15:10
*** d0ugal has quit IRC15:11
*** d0ugal has joined #openstack-ironic15:13
lucasagomesjroll, https://review.openstack.org/#/c/206067/ ?15:14
patchbotpatch 206067 - ironic - Allow agent lookup to directly accept node UUID (MERGED)15:14
openstackgerritSam Betts proposed openstack/ironic master: [devstack] Allow setting TFTP max blocksize  https://review.openstack.org/52873715:16
* rloo wonders why 206067 was not associated with a bug15:17
jrolllucasagomes: yeah, aware of that one, I thought there was something about the other end, passing the uuid to the agent15:18
lucasagomesoh right, yeah no idea... that one was the only related patch that I could find15:19
jrollno worries, thanks for looking :)15:19
*** hshiina has joined #openstack-ironic15:20
*** mcarden has joined #openstack-ironic15:20
*** ijw has joined #openstack-ironic15:20
lucasagomesnp :-)15:21
*** ijw has quit IRC15:26
TheJuliagah meetings right when you wake up :(15:29
rlooTheJulia: :-( ban all meetings!15:30
TheJuliajroll: re node uuid, what nobodycam does is his dhcp for discovery has a post hook that helps feed that into node creation so the actual uuid in ironic is the uuid of the node15:30
jrollTheJulia: neat, but I'm hoping to just kill the agent lookup process at the moment15:31
jrollmore concretely I'm reviving https://review.openstack.org/#/c/21220615:32
patchbotpatch 212206 - ironic-specs - No IPA to conductor communication (ABANDONED)15:32
TheJuliajroll: yeah, that would have to be code to be writen15:32
jrolland wondering if part of it is already being worked on :)15:32
TheJulianot from ironic's internals side, but we will have to continue to support lookup because otherwise it is a major workflow/behavior change and expectation change to add hardware to ironic15:32
TheJuliaa nice on/off switch is always nice ;)15:33
* TheJulia remembers discussing that at some point15:33
*** aarefiev has joined #openstack-ironic15:35
*** crushil_ has joined #openstack-ironic15:40
rloojroll: i don't recall the details of that, but the ipa version stuff that just merged might be useful for distinguishing whether ipa supports that or not15:41
rloojroll: err or not, i think it is via heartbeat...15:42
jrollrloo: oh yes, I'm going to need to make that work, thanks!15:42
*** crushil__ has joined #openstack-ironic15:42
jrollwe'll need to be able to pull that from the agent15:42
sambettsTheJulia: not sure it'll change that much of the workflow, because ironic can just inject the UUID using the PXE kernal flag15:42
pas-hajroll: also, at least the initial security implication behind that spec is IMO solved with multi-tenancy support, as ther'd be no access from tenant network to the internal ironic API which is in the provisioning network15:43
jrollTheJulia: and yes, that'll always be supported since non-neutron deployments won't have a way to know the IP address of the agent :)15:44
jrollpas-ha: yes, that's called out in the spec15:44
*** crushil_ has quit IRC15:46
*** jlabarre has joined #openstack-ironic15:47
*** amarao has quit IRC15:47
*** amarao has joined #openstack-ironic15:48
*** nicodemos has quit IRC15:49
*** fellypefca has quit IRC15:50
*** amarao has quit IRC15:51
*** amarao has joined #openstack-ironic15:51
*** ricardoas has quit IRC15:52
jrolloh, can a core please un-abandon that change for me when you have a moment? :)15:53
jrollhttps://review.openstack.org/#/c/21220615:53
patchbotpatch 212206 - ironic-specs - No IPA to conductor communication (ABANDONED)15:53
*** amarao has quit IRC15:55
*** r-daneel has joined #openstack-ironic15:55
sambettsjroll: on it15:56
sambettsjroll: done15:56
TheJuliasambetts: you beat me!15:56
TheJuliaI got a funky error when I clicked restore....15:57
*** makowals has quit IRC15:57
sambetts:/15:57
jrollthanks sambetts15:58
openstackgerritJim Rollenhagen proposed openstack/ironic-specs master: No IPA to conductor communication  https://review.openstack.org/21220615:58
*** dtantsur|afk has quit IRC16:03
*** dtantsur has joined #openstack-ironic16:04
leakypipesjroll: changed https://review.openstack.org/#/c/526753 to look up via compute node hypervisor_hostname as a fallback to when service.host can't be found (this is instead of switching on whether the hostname was a UUID or not)16:09
patchbotpatch 526753 - nova - allow compute nodes to be associated with host agg16:09
leakypipesjroll: I'm gonna start on the followup patch which adds aggregate associations to the server instance group calculations (which will allow us to use the existing server group functionality in Nova to group Ironic hosts together and do affinity/anti-affinity without any further hacking16:10
sambetts:O !!!16:10
sambetts\o/16:10
sambettsthanks leakypipes !!!16:10
leakypipessambetts: no problemo. :)16:10
*** charcol has quit IRC16:10
leakypipessambetts: I take payment in cookies. fyi. :P16:11
*** r-daneel_ has joined #openstack-ironic16:11
*** r-daneel has quit IRC16:11
*** r-daneel_ is now known as r-daneel16:11
sambettsyou'll have to remind me I owe you some if I see you at the next OS event  :-P16:12
*** etingof has joined #openstack-ironic16:13
*** leakypipes is now known as jaypipes16:15
* TheJulia is sure leakypipes has a spreadsheet tracking everyone's cookie debt :)16:19
*** hjensas has quit IRC16:19
*** hjensas has joined #openstack-ironic16:20
*** hjensas has quit IRC16:20
*** hjensas has joined #openstack-ironic16:20
jaypipesTheJulia: :)16:20
* rloo wonders when jay becomes leaky. is it raining in florida?16:21
jaypipesrloo: that's my casual Friday IRC nick :) just forgot to change it back this morning.16:21
TheJuliajaypipes: relevent question: does biscotti count as cookies?16:22
rloojaypipes: ah, mystery solved! :)16:22
jaypipesTheJulia: absolutely!16:22
jaypipesrloo: for instance, you might change yours on Fridays to "waterloo"16:22
jaypipes:P16:22
*** ijw has joined #openstack-ironic16:22
rloojaypipes: i'll keep that in mind if i decide to go that route; right now it is "" (no rloo)!16:24
jaypipeshehe16:24
NobodyCamGood Morning Ironic'ers16:26
*** datex has joined #openstack-ironic16:26
NobodyCamHappy Holidays!16:26
milan_folks, anyone into a DB issue?16:27
milan_morning NobodyCam! :)16:27
NobodyCam:) morning milan_ :)16:27
datex@vdrok: Hi, are you online?16:27
*** ijw has quit IRC16:27
*** ijw has joined #openstack-ironic16:27
milan_http://paste.openstack.org/show/629221/16:27
milan_I'm using a version_id to detect changes between two nodes (well, once there's an active-active deployment)16:28
milan_for now, I've added a new state transition https://review.openstack.org/#/c/510928/8/ironic_inspector/introspect.py@14016:28
patchbotpatch 510928 - ironic-inspector - Terminal state transitions in transactions16:28
milan_I'm updating version_id when I change stuff16:29
milan_but here it somehow doesn't work, with this new state transition16:29
* milan_ suspect it never actually worked16:29
*** jcoufal has joined #openstack-ironic16:30
milan_issue seems to be version_id is updated in the session but once I update the state within the same session (and a transaction) I get this error16:30
milan_having checked DB the version_id is not updated, dunno why the session tries to reload the data from the DB tho16:31
*** jcoufal_ has quit IRC16:32
*** ijw has quit IRC16:32
*** fxpester has quit IRC16:34
*** Goneri has joined #openstack-ironic16:35
openstackgerritMerged openstack/ironic-lib master: Updated from global requirements  https://review.openstack.org/52840316:36
*** namnh has quit IRC16:36
jrolljaypipes: <316:37
*** trown|afk has quit IRC16:37
*** catintheroof has joined #openstack-ironic16:38
jlvillalGood morning Ironic16:39
jlvillalrloo I sent an email to say that we will NOT backport the removal of the tempest plugin code to the stable branches16:39
rloojlvillal: thx, i just saw it :)16:39
jlvillal:)16:40
NobodyCamMorning jlvillal :)16:40
rloojlvillal: i must be wrong -- I thought that the stable branches used the tempest plugin from master.16:40
jlvillalHey NobodyCam :)16:40
rloojlvillal: and a good monday morning to you!16:40
NobodyCam:)16:40
jlvillalrloo, They do.16:40
jrollhey, morning jlvillal and NobodyCam \o16:40
*** penick has joined #openstack-ironic16:40
jlvillalrloo, And they did before all this16:40
NobodyCamMorning jroll :)16:40
jlvillaljroll, :)16:40
NobodyCamo/16:41
jlvillalrloo, But we believe that distros went ahead and used what was in the release to test.16:41
rloojlvillal: so stable/ will continue to use tempest plugin from master ironic? i thought we changed it so they took it from the new git repo/master.16:41
jlvillalrloo, No.16:42
jlvillalrloo, So before stable used the tempest plugin from master of openstack/ironic16:42
rloojlvillal: AHHHH. might be worth mentioning that in the email. in case they are testing stable using tempest/ironic/master, cuz they'll have to change to use tempest-from-new-repo-master16:42
jlvillalrloo, Now stable uses the tempest plugin from master of openstack/ironic-tempest-plugin16:42
jlvillalrloo, If that makes sense16:42
rloojlvillal: yes, that makes sense, and that's what i thought.16:42
*** dgautam has joined #openstack-ironic16:43
jlvillalrloo, I thought my email said that...16:43
rloojlvillal: which is why i got confused about mentioning that you were going to remove tempest stuff from stable16:43
*** crushil__ has quit IRC16:43
* jlvillal goes and re-reads his email16:43
rloojlvillal: i misintepreted your email16:43
jlvillalrloo, Okay. Let me know if I should clarify something in the email.16:44
rloojlvillal:  the 'And then after that doing backports of those changes to the stable branches'16:44
rloojlvillal: was too broad for me I guess :)16:44
jlvillalrloo, Ah. Probably better clarity on my part would help.16:44
rloojlvillal: your email made it sound like you had done all the work for the master branches only. don't know if it is just myself, but worth clarifying.16:45
jlvillalrloo, In reality I think the actual audience for this is stuff is teeny tiny.16:45
*** catintheroof has quit IRC16:45
jlvillalrloo, In reality I think the actual audience for this stuff is teeny tiny.16:45
jlvillal:)16:45
jlvillalSure16:45
rloojlvillal: so basically, we've migrated to using the new tempest repo, for master + stable. If third-party CI tests stable with the tempest plugin in that stable repo, they are good. if they have out of tree stuff using tempest -from-ironic, they will need to change to use new repo, right?16:46
rlootempest-from-ironic-master :)16:46
*** makowals has joined #openstack-ironic16:48
*** andreaf has joined #openstack-ironic16:49
dgautamhow can I unlock a openstack node.16:49
dgautamironic node*16:50
*** crushil_ has joined #openstack-ironic16:51
dgautamone of my node is stuck and now I am not able to even delete the node.16:51
jlvillalrloo, Proposed email follow-up: http://paste.openstack.org/show/629224/   Comments?16:51
jlvillalrloo, Also I hope no 3rd Party CI do test stable with the stable version. But if they did, we will not break them.16:52
andreafjlvillal: great progress on that :)16:52
jlvillalandreaf, Thanks. It was a giant team effort. A LOT of patches (20+) go merged to various projects to get it done.16:52
jlvillals/go merged/got merged/16:52
andreafyeah great work!16:53
*** AlexeyAbashkin has joined #openstack-ironic16:53
jlvillalrloo, Feel free to make a new pastebin with suggested changes if you wish :)16:53
*** makowals has quit IRC16:53
jlvillalandreaf, :)16:53
rloojlvillal: that looks great. maybe just add the date that we're planning on removing the tempest code from ironic/inspector16:54
andreafjlvillal: related to CI, but on the zuulv3 side - I've been working on zuul v3 native dsvm jobs in devstack and tempest16:54
jlvillalrloo, Will do16:54
andreafjlvillal: the single node base job is almost done, I'm only missing a series of patches on devstack side to pull in more logs and configs16:54
jlvillalandreaf, Cool. We want to do that at some point. Migrate the 'legacy' jobs16:55
jlvillalandreaf, Yeah I did notice the default devstack job said 'multinode'. But I didn't investigate if it really was.16:55
andreafjlvillal: the thing I wanted to mention is that a lot of the logic that used be in d-g is now going into the specific jobs and roles - so some of it is in zuul-jobs, some in devstack, some in tempest16:56
andreafjlvillal: but the d-g code also includes ironic specific bits, and my expectation is that those will end up in jobs / role owned by ironic in some ironic repo16:56
*** AlexeyAbashkin has quit IRC16:57
jlvillalandreaf, Okay. Cool.16:58
* TheJulia hopes people are updating the whiteboard :)16:58
andreafjlvillal: just to make an example https://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate.sh#L17116:58
*** rpioso|afk is now known as rpioso16:59
TheJulia#startmeeting ironic17:00
openstackMeeting started Mon Dec 18 17:00:19 2017 UTC and is due to finish in 60 minutes.  The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot.17:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.17:00
*** openstack changes topic to " (Meeting topic: ironic)"17:00
openstackThe meeting name has been set to 'ironic'17:00
sambettso/17:00
jroll\o17:00
TheJuliaEveryone ready for a meeting of irony?17:00
rpiosoo/17:00
crushil_\o17:00
TheJuliaerr, ironic :)17:00
pas-hao/17:00
hshiinao/17:00
datex!17:00
jlvillalo/17:00
rlooo/17:01
TheJuliaOur agenda is on the wiki!17:01
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting17:01
*** rwsu has joined #openstack-ironic17:01
TheJulia#topic Announcements / Reminder17:01
*** openstack changes topic to "Announcements / Reminder (Meeting topic: ironic)"17:01
TheJuliajlvillal: The first annoucement is yours, would you like to take the floor?17:01
jlvillalTheJulia, Thanks17:02
jlvillalHopefully every 3rd Party CI representative is here :)17:02
*** Mallik has joined #openstack-ironic17:02
jlvillalBasically tomorrow we will be removing the tempest-plugin code that is in openstack/ironic and openstack/ironic-inspector tomorrow17:02
*** lucasagomes is now known as lucas-afk17:03
jlvillalSo if your CI is pulling in the tempest code, then please update to use openstack/ironic-tempest-plugin instead.17:03
jlvillalIf you have any questions please ask me or on the channel.17:03
rlooirmc looks like the only one represented here17:03
jlvillalThat is about all I got.17:03
TheJulia#info Third Party CI - In tree tempest plugin code anticipated to be removed December 19th.17:03
TheJuliajlvillal: Thanks17:04
rajinirTheJulia. ok thanks17:04
TheJuliaNext up, deadlines!17:04
TheJulia#info Non-client library release deadline is January 18th17:04
TheJulia#info Client library release deadline is January 25th17:05
*** ricardoas has joined #openstack-ironic17:05
rlooTheJulia: remind me, that includes ironic-lib, sushy, right?17:05
TheJulia#info Feature Freeze is January 25th17:05
rlooTheJulia: what about virtualbmc, networing-generic-switch?17:05
*** makowals has joined #openstack-ironic17:05
*** srart has joined #openstack-ironic17:06
rpiosorloo: idrac is, too.17:06
*** hw_wutianwei has quit IRC17:06
TheJuliarloo: I believe we agreed at one point virtualbmc it does not apply... but I'd have to go back and look at prior discussions to be sure. For n-g-s, that is a plugin.17:06
rlooTheJulia: thx.17:06
TheJuliaIt does include ironic-lib, sushy, and other libraries that we utilize.17:06
TheJuliaAny questions?17:06
datexhi everyone.17:06
TheJuliaAnd does anyone have any other announcements today?17:07
* TheJulia senses the silence as a no....17:07
*** corey_ has joined #openstack-ironic17:07
datexIs depolyment ironic instance via BT still in progress?17:07
jlvillalNo meeting next week? and the next?17:08
jlvillalAs a reminder17:08
TheJuliadatex: Lets take that to open discussion in a little bit17:08
TheJuliajlvillal: I was just going to note that as a reminder :)17:08
datexok, thank you17:08
jlvillalI'll miss you all! :)17:08
TheJuliaCorrect, we will not have meetings the next two weeks. I sense some cores are going to be around, but please as in channel if you need priority reviews or eyeballs on patches.17:08
TheJuliaI guess we should move on then!17:09
TheJulia#topic Review action items from previous meeting17:09
*** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)"17:09
rlooso next meeting will be Jan 8, 2018 :)17:09
TheJulia#undo17:09
openstackRemoving item from minutes: #topic Review action items from previous meeting17:09
TheJulia#info Next meeting Jan 8, 201817:09
TheJulia#topic Review action items from previous meeting17:09
*** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)"17:09
TheJuliaI double checked the bugs last week and we had very few bugs come in which minimized the need for triaging.17:10
TheJuliaAnd I don't know if Dmitry created the nova bug.17:10
rlooTheJulia: maybe add the same action item for dmitry, so we can follow up next year17:11
jlvillalWe can wait until dtantsur gets back. But I did propose a patch to update the priority of the traits work to "essential"17:11
TheJulia#action dtantsur to follow-up on client version negotiation bug in nova17:11
jlvillalrloo, Made a comment that she wasn't sure if we update those or not.17:12
TheJuliajlvillal: Thanks!17:12
*** stendulker has joined #openstack-ironic17:12
*** trown has joined #openstack-ironic17:12
jlvillalThe patch: https://review.openstack.org/52838617:12
patchbotpatch 528386 - ironic-specs - Move traits from "high" to "essential" priority17:12
TheJuliajlvillal: your reading my mind :)17:12
jlvillal:)17:12
*** trown has quit IRC17:12
TheJuliaI'll put that on the todo list for this week, anyway we should move on if there are no objections.17:12
TheJulia#topic Review subteam status reports17:13
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)"17:13
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard17:13
*** nicodemos has joined #openstack-ironic17:13
TheJuliaOn our whiteboard, subteam status reports start at line 144.17:13
rlooTheJulia: do you know how to update the bug count?17:14
*** trown has joined #openstack-ironic17:14
TheJuliarloo: I do not17:14
*** trown has quit IRC17:14
TheJuliaWell, I guess it could be done by hand17:14
*** stendulker_ has joined #openstack-ironic17:15
jrollI believe it's at http://dashboard-ironic.7e14.starter-us-west-2.openshiftapps.com/17:15
jroll(there's a more friendly url, but I don't know it)17:16
TheJuliapas-ha: regarding the client reauth, do you know if anyone has volunteered to going to update/rebase the inspector patches?17:16
* jroll updates the count17:16
pas-haTheJulia: me17:16
TheJuliajroll: thanks!17:16
*** trown has joined #openstack-ironic17:16
*** trown is now known as trown|ruck17:17
TheJuliaHmm... I guess mgoddard has started taking over the trait work, is that correct?17:17
rloovdrok: will you be able to work on the clean up of deploy interfaces?17:17
rlooTheJulia: yup, he has a patch up17:17
TheJuliarloo: thanks!17:17
jlvillalTheJulia, correct on mgoddard17:17
*** stendulker has quit IRC17:18
*** stendulker_ has quit IRC17:18
*** stendulker_ has joined #openstack-ironic17:18
rlooi'm concerned about the bios config work. the spec hasn't been approved yet. maybe this week, I'll touch base with you TheJulia and/or Dmitry, and see if we can get it done.17:19
vdrokhey folks, good morning. sorry, I got caught up in things unrelated to upstream recently. this week I hope to finish it and get back here17:19
rloothx vdrok!17:19
TheJuliarloo: Also concerned, I can dedicate time today after lunch to looking at it in depth.17:19
rlooTheJulia: ok, we can discuss later, thx!17:19
TheJuliaEveryone done updating statuses on the whiteboard?17:19
*** crushil_ has quit IRC17:20
rloojlvillal: good work on the tempest plugin work17:20
*** crushil__ has joined #openstack-ironic17:20
rloojlvillal: that seems like the only thing that went well/fast this cycle17:20
jlvillalrloo, Thanks to everyone for their help in getting things reviewed and fixed up :)17:20
datexvdrok: hi, do you have some time? I have a question.17:21
TheJuliaOut of curiosity, is anyone picking up the uwsgi work?17:21
vdrokhey datex , yeah, ask after the meeting or in private17:21
*** makowals has quit IRC17:21
TheJuliaMoving on to priorities!17:21
pas-haI could take uqsgi too17:21
pas-hauwsgi17:21
rloothx pas-ha!17:22
TheJuliapas-ha: Thanks!17:22
TheJulia#topic Deciding on priorities for the coming week17:22
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"17:22
*** r-daneel_ has joined #openstack-ironic17:22
datexvdrok: I have send a mail to your inbox (vdrok@mirantis.com), thanks17:22
TheJuliaBack to the whiteboard, line 107!17:22
rlooi don't think the tempest plugin is a priority any more? just needs 2? patches to be merged?17:23
jlvillalEasy one is to keep #1. We should finish it tomorrow I hope.17:23
vdrokdatex: will take a look in a bit17:23
TheJuliajlvillal: what about the follow-up items?  I'm thinking we need to just move the priority down at this point17:23
datexvdrok: thanks17:23
*** r-daneel has quit IRC17:23
*** r-daneel_ is now known as r-daneel17:23
*** catintheroof has joined #openstack-ironic17:23
jlvillalTheJulia, Sure on moving it down17:23
rloobecause traits is now essential, maybe put the first patch down, although it needs an update: https://review.openstack.org/#/c/528238/17:24
patchbotpatch 528238 - ironic - Node traits: Add DB API & model17:24
jroll++, pretty close though17:24
TheJuliaI think: Auth refactor, bios spec, traits, rescue, routed networks, and tempest related cleanup?17:24
*** baha has joined #openstack-ironic17:25
TheJuliaIn order of course :)17:25
rloowhat state are the routed network patches in? they were WIP at some point.17:25
*** datex has quit IRC17:25
TheJuliaI think they were all updated last week17:25
rloocan anyone vouch that they're ready to be reviewed?17:26
jlvillalrloo, Removed WIP but I asked for some DSVM tests to be added...17:26
TheJuliaHmm, they do need revisions now17:26
rloojlvillal: do you think they're ready to be reviewed? (besides the tests) or do you want tests to show that they're ready to be reviewed?17:26
*** datex has joined #openstack-ironic17:26
jlvillalrloo, They can be reviewed. I would prefer an update be done, but the basic concept is there.17:26
TheJuliarloo: since it will be a few weeks, I think we might as well keep them on the list, but at the bottom so hopefully they get some review traffic over the next few weeks17:26
jlvillalsambetts, May have more input as he has also been reviewing17:27
rlooTheJulia, jlvillal: ok with me then17:27
TheJuliaany objections?17:27
*** ijw has joined #openstack-ironic17:28
jlvillalNo! :)17:28
sambettsI think the routed networks patches can be reviewed, but until we have a working test env proving it works with nova/neutron, its pretty much blocked merging17:29
rloosambetts: is someone working on the testing?17:29
sambettsI think hjensas is looking into it17:29
jlvillal+1 on what sambetts said.17:29
*** catintheroof has quit IRC17:30
rloodoes that work require changes to nova or neutron?17:30
jlvillalHopefully not. But testing of course will help to find out.17:30
TheJuliarloo: does not appear to at this point17:30
jlvillalI am unsure if it has even been tested in a local devstack environment or not.17:31
rloook good. if it does, someone needs to point it out. please.17:31
sambettsit shouldn't, its just a complicated devstack environment, and hopfully the other projects already have a foundation to setup an L3 network simulation but if not we'll be the first to try to similate an L3 network on infra17:31
sambettssimulate*17:31
TheJuliaI added a note to the list saying that the routed networks was review for input only.17:32
TheJuliaThat way we make it clear what the expectation is17:32
TheJuliaAny objections? Anyone have anything they should think that should be on the list? (Since, it will be a few weeks before we update the list again)17:32
* TheJulia hears crickets17:33
rlooi think that's good. it'll be a few weeks, but i am guessing there won't be that many folks around either17:33
TheJuliaWorks for me!17:34
*** stendulker_ has quit IRC17:34
*** ijw has quit IRC17:34
jroll+117:34
*** stendulker_ has joined #openstack-ironic17:34
TheJuliaI guess it is time to move on then :)17:34
TheJulia#topic Appointing a bug triaging lead for the coming week17:35
*** openstack changes topic to "Appointing a bug triaging lead for the coming week (Meeting topic: ironic)"17:35
*** penick has quit IRC17:35
TheJuliaDo we have any victims... err, I mean volunteers?17:35
rloofor the next 2? weeks :)17:35
TheJuliaReally, 317:35
rloowow, bonus 2 weeks! :D17:36
rlooi mean, bonus EXTRA 2 weeks :)17:36
TheJulialol17:36
TheJuliaOkay, I guess I'll put it on my plate17:36
TheJuliaSince I'll be around17:36
*** milan_ has quit IRC17:36
rloothx TheJulia!17:36
*** racedo has quit IRC17:36
TheJulia#action TheJulia to take on bug triaging until our next meeting.17:36
TheJuliaTopic RFE review17:37
TheJuliaerr17:37
TheJulia#topic RFE review17:37
*** openstack changes topic to "RFE review (Meeting topic: ironic)"17:37
TheJuliarloo: do we have any RFE's that need a look over today?17:37
rlooTheJulia: sorry, I don't know, I didn't feel like doing it. Happy Holidays! we'll resume next year :)17:37
TheJuliaAwesome!17:38
TheJuliaSince we have no set discussion topics, we can go directly to Open Discussion!17:38
* rloo just wants to wrap things up since it is that season...17:38
TheJulia#topic Open Discussion17:38
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"17:38
TheJulia:)17:38
rlooso what do folks think about a yearly release?17:38
rloono, let's not discuss it!17:39
TheJuliadatex: You mentioned interest in bit torrent based deploys?17:39
TheJuliarloo: lol17:39
*** racedo has joined #openstack-ironic17:39
TheJuliarloo: I have very very mixed feelings about the proposal and results, and I'll leave it at that for the time being :)17:39
rlooi thought there had been a spec, maybe code patches for bit torrent, but the author left? all vague...17:39
rlooTheJulia: me too, but nice that it is being discussed17:40
* rloo thinks we need to figure out how to work effectively with fewer people17:40
rloomaybe train crickets...17:40
TheJuliarloo: Yeah, I think BT'izing deploys kind of goes hand-in-hand with pre-deployed machines, and that code never completely landed... that is unless it just becomes the machine can grok image_source being a bittorrent tracker url....17:40
rlooTheJulia: well, datex hasn't responded so...17:41
TheJuliarloo: That is a very good topic for the PTG, but something we should ponder....17:41
rlooAre there only crickets now?17:41
TheJuliaI think there are only crickets....17:41
TheJuliaI must run a boring meeting :(17:41
* rloo thinks TheJulia is really effective :)17:41
TheJuliaLast call for any open discussion topics, or I'll close our meeting out for this week in a couple minutes17:42
rloothe last meeting of the year17:42
rloowe need an action item : everyone have FUN17:43
rpiosorloo: lol17:43
rpiosoI hope you have a wonderful holiday season, ironicers. And Happy 2018!17:43
rloo++17:43
TheJulia#action Everyone have a wonderful quiet holiday! :)17:43
pas-ha++17:43
jroll+++17:43
TheJuliaThanks everyone!17:44
jlvillal:)17:44
datexsorry i'm back17:44
*** aarefiev has quit IRC17:44
TheJuliadatex: Does the discussion from a few minutes ago give you an idea? We can dig up some links and discus further if your interested in carrying forward such a feature?17:44
datexyes, i'm interested in bt deploy 'cause i saw this post https://www.mirantis.com/blog/cut-ironic-provisioning-time-using-torrents/17:45
TheJuliaOkay, well in that case, I'll end the meeting.17:46
TheJuliaThanks again everyone!17:46
TheJulia#endmeeting17:46
*** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic"17:46
openstackMeeting ended Mon Dec 18 17:46:42 2017 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)17:46
openstackMinutes:        http://eavesdrop01.openstack.org/meetings/ironic/2017/ironic.2017-12-18-17.00.html17:46
openstackMinutes (text): http://eavesdrop01.openstack.org/meetings/ironic/2017/ironic.2017-12-18-17.00.txt17:46
openstackLog:            http://eavesdrop01.openstack.org/meetings/ironic/2017/ironic.2017-12-18-17.00.log.html17:46
*** sambetts is now known as sambetts|afk17:47
sambetts|afkNight all!17:47
sambetts|afkcya tomorrow17:47
*** corey_ has left #openstack-ironic17:47
rloogood night sambetts|afk17:47
TheJuliadatex: Interesting, I feel like I'm missing some of the mechanics of how they did that, but I think it could be equally interesting to just support download/use of files from trackers which could then possibly fire up a client...17:48
* TheJulia may also need lots of coffee this morning17:49
datexMy thesis is a BT client to distrub the distribute the Filesystem directly from/to disk17:49
TheJuliaHmm, that is an interesting twist on the idea17:50
datexMaybe there is any room to improve the performance with using my solution rather than transfer the whole disk to nodes17:50
*** Mallik has quit IRC17:51
datexbecause filesystem has "hole" which means that contains nothing important data17:51
TheJuliadatex: Well, it is typically not the entire disk, but a small-ish image that gets expanded or one that is aware of whitespace and how we write the image out can account for that.17:51
TheJuliaLargely we've always focused on image based deployment as it is fairly easy to checksum a single file and compare checksums, ideally checksums would need to be verified or able to be trusted on a per file level.  The other conundrum is bootloader17:52
*** derekh has quit IRC17:52
datexI know that if the image is fresh installation OS. But if we have a snapshot17:52
TheJuliaAlthough, we have code that can make a boot partition and put the appropriate data on disk to make a partition bootable17:53
TheJuliadatex: could you elaborate, I'm not quite sure if I understand your last comment17:53
datexIf we want to boot a image such as a Ubuntu Minimal, we can use as small as possible disk to build the image, and boot image to node17:55
datexthe disk won't have too much "hole" in disk.17:55
datexbut if we take a snapshot from a running server, and try to dup it.17:56
datexa snapshot image might have too many "hole" in the disk that we don't need to transfer to the new node17:57
TheJuliaHeh, Summer just hijacked my other laptop to read the mirantis blog post because she is intrigued as well17:57
datexCool XDDD17:57
TheJuliadatex: Ahh, yeah, that is a REALLY good point, unless you intentionally go rip out white space by doing something like balooning.... your going to get a lot of junk in the image17:58
datexIn mirantis post, they transfer whole disk to nodes17:58
datexyes17:58
*** udesale has quit IRC17:58
datexSo in my thesis, I tried to NOT transfer that holes to other node to save the bandwidth17:59
vdroklooking at the patches (needed to refresh my memory as well) we could also just download qcow2 in-memory and then unpack it to disk17:59
TheJuliadatex: would you be interested in writing a specification document up?17:59
*** etingof has quit IRC17:59
TheJuliavdrok: I think we already support that17:59
vdrokTheJulia: yeah, but via a torrent17:59
datexvdrok, I think thw way need RAM > image size, right?17:59
vdrokyes17:59
TheJuliadatex: That is kind of a hard operating requirement right now18:00
* TheJulia goes and makes a latte to hopefully reclaim her other laptop next to the espresso machine18:00
vdrokdatex: so basically, jsut propose a spec describing the details, we'll be happy to review18:01
*** ijw has joined #openstack-ironic18:01
*** ijw has quit IRC18:01
datexand I have done with transfer the disk(filesystem) with BT directly without a temp storage to restore back to the disk of node18:01
datexDid you know a software named "CloneZilla" for dup the one computer to all?18:02
*** etingof has joined #openstack-ironic18:03
datexCloneZilla used udpcast (multicast) to distribute the data and restore it.18:03
vdrokI have not heard of it18:03
datexoh, hmm18:03
vdrokoh yeah, there was some spec on multicast image writing as well18:03
*** harlowja has joined #openstack-ironic18:04
vdroktho I think glance does not support it18:04
TheJuliadatex: what about file level permissions?18:04
*** ijw has joined #openstack-ironic18:04
TheJuliadatex: For specs, we have a git repository where we raise questions that would be pertinant (in most cases), https://git.openstack.org/cgit/openstack/ironic-specs18:04
datexWe transfer the filesystem that means "inode" and "data" will be transfer to keep all attributes18:05
*** hshiina has quit IRC18:05
TheJuliaYeah, I'd love to read a specification. I'm sure I could even get summer to review it....18:06
datexTheJulia, thanks, but i want to have some discussion first to make sure my idea will work :)18:06
TheJuliadatex: oh, it should work, of course, the fine details are key :)18:06
datexok, I will try my best to explain more and more details in ironic-specs18:07
TheJuliadatex: feel free to ask questions along the way18:07
datexmaybe i need some time to read the ironic. I'm unfamiliar with ironic :)18:08
datexTheJulia and vdrok, Thanks for your help!18:10
vdrokyoure welcome!18:11
TheJuliadatex: happy to help!18:11
*** ijw has quit IRC18:13
*** AlexeyAbashkin has joined #openstack-ironic18:14
*** ijw_ has joined #openstack-ironic18:18
*** AlexeyAbashkin has quit IRC18:18
*** ijw_ has quit IRC18:22
*** e0ne has quit IRC18:26
*** ijw has joined #openstack-ironic18:26
*** ijw has quit IRC18:26
*** stendulker_ has quit IRC18:30
*** penick has joined #openstack-ironic18:31
snasirHi, I have a question, am trying to understand how nova creates a record for the ironic node. Everytime ironic node is created, i see a log in nova-compute logs that "Compute node record created for controller: with uuid of ironic node"...18:33
snasirSo as part of ironic node-create, does ironic send some sort of a notification to nova too ?18:34
*** catintheroof has joined #openstack-ironic18:36
*** catintheroof has quit IRC18:36
*** catintheroof has joined #openstack-ironic18:36
*** catintheroof has quit IRC18:36
*** namnh has joined #openstack-ironic18:37
*** fragatina has joined #openstack-ironic18:39
rloohi snasir. no, ironic doesn't send notification. the nova-ironic-virt-driver periodically asks ironic for the list of nodes18:39
*** namnh has quit IRC18:42
snasirrloo: can you kindly point me to the area/ function in nova-repo, where that happens ?18:43
*** makowals has joined #openstack-ironic18:46
rloosnasir: ah, not totally sure, has been awhile since i looked at that code. it could be this: https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L72718:47
*** makowals has quit IRC18:48
*** catintheroof has joined #openstack-ironic18:56
snasirrloo: Thanks... Just to clarify, so when an ironic node is created, an entry is created in the ironic db, and then when nova ironic driver audits for available resources, it just retreives that data from the ironic db, or does it make a rest api call to ironic ? Thanks18:57
rloosnasir: API call!18:58
*** fragatina has quit IRC18:59
*** ijw has joined #openstack-ironic19:01
*** e0ne has joined #openstack-ironic19:07
*** ijw has quit IRC19:08
TheJuliarloo: digesting bios spec19:11
rlooTheJulia: digesting latest opinions on longer cycle :)19:11
TheJulialol19:12
*** crushil__ has quit IRC19:13
TheJuliarloo: so we're expecting bios management interfaces that will appear that hare completely independent from the management of the hardware? It seems like they too would be hardware dependent...19:13
rlooTheJulia: yeah, that's what I don't really think can happen. Seems like you'd have one bios interface per hw type.19:14
rlooTheJulia: so how to handle changing bios interfaces may be a corner case19:14
rlooTheJulia: in that regard, it is similar to other Interface types.19:15
TheJuliaYeah, I don't think one really would ever.... unless thy pulled all the drives and went from soemthing like an ilo4 to an ilo519:15
rlooTheJulia: i think we've been loosy-goosy wrt when we allow the interface values (or other node values) to be changed, but I honestly don't recall when we allow what...19:15
TheJuliaI really see these as all hardware management actions19:16
rlooTheJulia: i believe mark goddard was the one that brought up the 'how do you handle when the bios interface changes'. it is a good question. now i am wondering, how do we handle when any iterface is changed...19:17
TheJuliarloo: we just create the task upon the next command with the new interface, we do nothing else.19:17
* TheJulia feels that we're way over using the word "interface"19:17
rlooyup, interface and driver :-(19:18
*** trown|ruck has quit IRC19:18
TheJuliaIf we just grafted this stuff on management though, that would cut out.... ?3? patches19:18
rlooTheJulia: presumably, regardless of what bios interface you use, the BIOS config settings on that hw won't change just cuz you changed the BIOSInterface19:18
*** trown has joined #openstack-ironic19:18
rlooTheJulia: oh. didn't we discuss that at the PTG?19:19
*** trown is now known as trown|ruck19:19
* rloo looks for ptg etherpad...19:19
TheJuliarloo: dunno, I missed ?thursday? morning19:19
TheJuliaraid I can see being different because I can buy any raid card and install it in any server19:19
*** racedo has quit IRC19:19
TheJulia(well, architecture compatible)19:20
TheJuliabut bios... hmmm19:20
rlooTheJulia: https://etherpad.openstack.org/p/ironic-queens-ptg-feature-proposals19:21
rlooTheJulia: L6019:21
rlooTheJulia: i guess the 'composibility' reason is why it isn't in mgt19:22
TheJuliaThat seems like one hell of an edge case19:22
rlooTheJulia: maybe we should discuss with dmitry when he is back tomorrow. I don't feel strongly one way or the other, but the BIOSInterface is the direction we've taken/suggested.19:24
TheJuliaI guess, the only case that _truly_ works is for bios settings that may be settable from with-in a running OS.19:25
openstackgerritJohn L. Villalovos proposed openstack/ironic master: Detect when log messages are mis-formatted  https://review.openstack.org/23225719:25
TheJuliawhere as we could have a redfish one that could make changes via the rest api19:26
rloojlvillal: any thoughts about putting some of that ^^ code in an oslo library?19:26
jlvillalrloo: Not really... I mean sure if someone wants to do it. I'm not sure I have the bandwidth to try to do the whole process on that.19:27
rloojlvillal: once upon a time, we blocked copies of code from going into ironic; they had to go into an oslo library first.19:28
rloojlvillal: i suspect those days are gone and we'll copy/paste until it gets 'bad' again :-(19:28
jlvillalrloo: In theory it is a good idea. I just wonder how long it would take to do it and how much effort would be involved.19:28
rloojlvillal: it depends; it didn't take long with lucas and i got some stuff added to one of the libraries. but i don't know how much involvement/people are around wrt oslo to help with reviews etc19:29
*** penick has quit IRC19:30
jlvillalrloo: Sadly at this time I have less time to devote towards OpenStack. Luckily I still have time, but not as much as before.19:30
rloojlvillal: yeah, i know :-(19:30
jlvillalrloo: Thus my hesitancy to volunteer to do that.19:30
*** penick has joined #openstack-ironic19:31
rloojlvillal: i just saw arkady's request not to pull the plug on tempest-plugin-in-master-ironic-n-inspector this week. sounds like a reasonable request...19:33
rloojlvillal: we DO need to make sure, for however long we keep that directory around, that it doesn't get updated...19:34
rloojlvillal: wrt the fixture stuff, maybe the thing to do is send email to devlist, oslo, asking them what they suggest. that it'll probably get merged to ironic and that it is copied in other repos right now.19:36
rloojlvillal: well, 'probably' if it is ok with dmitry; i want to get his opinion...19:36
rlooTheJulia: so I think it makes sense to have a BIOSInterface.19:36
jlvillalrloo: I could work on a 'pep8' check thingy to prevent new patches to the plugin directories. I'm not quite sure how to do that, but I could investigate.19:37
rloojlvillal: i don't know if we need to do that, unless it is quick. i think dmitry -2'd all patches last week, that touched that directory.19:38
rloojlvillal: might be enough to remind all the cores. dunno.19:38
jlvillalrloo: I am replying to arkady's email. With the holiday as a counterpoint this might be the best time to break things since very few people will care...19:39
*** penick has quit IRC19:39
jlvillalrloo: Though I wonder how well their (any 3rd Party) CI works in general if nobody is around to fix issues for two weeks. Well it still be working when they get back?19:39
jlvillalrloo: And work is allowed to go forward on openstack/ironic-tempest-plugin changes...19:40
rloojlvillal: i don't see it as urgent to delete that tempest code, and I'd rather not break 3rd party CI because of that19:42
TheJuliarloo: re bios, is it envisioned to be entirely out of band?19:43
jlvillalrloo: Okay. But it gives a false sense of positiveness for 3rd Party CI to keep using it.19:43
jlvillalrloo: As in they may really be broken but don't know it.19:43
rloojlvillal: apart from someone accidentally updating the tempest code, i don't think it affects us much. besides the 3rd party ci using 'older' tempest code19:43
rloojlvillal: realistically, i doubt much/any tempest changes will land in the next two weeks?19:44
rlooTheJulia: good question. if we don't explicitly say 'only out of band' then no. hard to predict the future...19:44
jlvillalrloo: Okay. I'm in the somewhat opposed to delaying the changeover camp at this time. Not vehemently opposed, but opposed.19:44
rloojlvillal: can discuss with dmitry tomorrow. you two were the most involved in this. i am just being devil's (err, 3rd party CI) advocate19:45
jlvillalrloo: We need that :) Thanks.19:45
rloojlvillal: doesn't seem like we gave them much notice/heads up about it.19:45
rloogonna take a break and get some fresh cold air. back soon...19:46
*** e0ne has quit IRC19:46
openstackgerritJohn L. Villalovos proposed openstack/ironic master: Remove ironic_tempest_plugin/ directory  https://review.openstack.org/52773319:49
*** ijw has joined #openstack-ironic20:01
*** e0ne has joined #openstack-ironic20:01
*** ijw has quit IRC20:07
TheJuliarloo: In that case, I feel like I just found an issue with bios stuffs :(20:07
TheJuliarloo: please review my comments, hopefully they all make sense.20:12
*** jtomasek has joined #openstack-ironic20:19
*** ccamacho has quit IRC20:26
*** penick has joined #openstack-ironic20:28
*** d0ugal has quit IRC20:36
*** d0ugal has joined #openstack-ironic20:38
*** namnh has joined #openstack-ironic20:38
*** namnh has quit IRC20:42
*** e0ne has quit IRC20:51
*** fragatina has joined #openstack-ironic20:51
*** dprince has quit IRC20:58
*** linkmark has joined #openstack-ironic20:59
rloothx TheJulia, I commented on your comments :)21:07
rlooTheJulia:  this reminds me so much of other features, eg cleaning. The spec hangs around for awhile, then we/someone decides we need to just do it. And then during code reviews, we try to address issues that we/reviewers see.21:08
rlooTheJulia: i have no idea whether it is better to spend a lot of time during the spec process, documenting details, or leave it for the coding phase.21:08
TheJuliaThere is definitely a happy medium, but I think in this case we've also hit some of the pitfalls before21:09
*** ijw has joined #openstack-ironic21:09
rlooTheJulia: part of it I think, depends on the author/people that are proposing the spec and writing the code. How 'capable' they are...21:09
rlooTheJulia: so you raise some good points. now how do we (or the author) deal with it, to speed things up...?21:10
rlooTheJulia: it is not too late to get rid of BIOSInterface. I always believe in better late than never, if it seems like BIOSInterface is a bad idea.21:11
rlooTheJulia: I don't mind adding more interfaces as long as they make sense. The more we add, the more used to it we'll be wrt how to add a new one etc.21:11
*** r-daneel has quit IRC21:11
*** r-daneel has joined #openstack-ironic21:11
*** e0ne has joined #openstack-ironic21:11
*** jcoufal_ has joined #openstack-ironic21:13
*** jcoufal has quit IRC21:15
TheJuliaI would say it is contextual knowledge more then capability, on all aspects21:16
TheJuliaI can totally see the point behind it being separate, but my questions are based out of a lack of context. Once I have that context, reconciliation is quick :)21:17
*** jcoufal_ has quit IRC21:17
*** jcoufal has joined #openstack-ironic21:20
*** catintheroof has quit IRC21:20
*** jtomasek has quit IRC21:24
*** amarao has joined #openstack-ironic21:24
*** jcoufal has quit IRC21:31
*** ijw has quit IRC21:32
*** jcoufal has joined #openstack-ironic21:33
*** openstackgerrit has quit IRC21:33
rlooTheJulia: right, and the spec ought to include that context too.21:37
*** AlexeyAbashkin has joined #openstack-ironic21:38
*** AlexeyAbashkin has quit IRC21:42
*** dprince has joined #openstack-ironic21:43
*** amarao has quit IRC21:44
rloojlvillal: dumb question. wrt your patch https://review.openstack.org/#/c/232257/, should or shouldn't the DEBUG logs be printed out?21:52
patchbotpatch 232257 - ironic - Detect when log messages are mis-formatted21:52
rloojlvillal: cuz I see them logged: http://logs.openstack.org/57/232257/17/check/openstack-tox-py27/fcb78d7/job-output.txt.gz21:52
rloojlvillal: but the commit msg seems to indicate they'd only be logged if OS_DEBUG was set?21:52
*** rwsu has quit IRC21:54
jlvillalrloo: Hmmm. Yeah, it isn't supposed to...21:55
jlvillalrloo: Sounds like I need to figure that out :(21:55
rloojlvillal: and did you see the sphinx job failed?21:56
jlvillalrloo: I had not21:56
jlvillalrloo: But I do now21:56
rloojlvillal:  i am just about to ask infra about that... :-( wondering if they turned on some flag...21:56
jlvillalrloo: I didn't make any changes in regards to that. Thanks for letting me know.21:57
rloojlvillal: i've seen it in 3 patches so far. was hoping to ignore but ...21:57
jlvillalrloo: Oh :(21:58
*** trown|ruck is now known as trown|outtypewww22:01
rlooso folks, our gate is broken cuz of sphinx changes. monty sent out email, i'm going to read and see if i can figure out what we need to do...22:01
* rloo would have liked a heads up...22:02
*** baha has quit IRC22:10
TheJuliaugh22:11
jrolllooks like just a warning treated as an error22:14
jrollshould be an easy fix, I can try to do it real quick :)22:14
jrollhm, this might be related to the autodoc thing he mentioned, though22:16
jrollwhich is merged now https://review.openstack.org/#/c/52879622:16
patchbotpatch 528796 - openstack-infra/zuul-jobs - Handle projects still using pbr autodoc_index_modules (MERGED)22:16
* jroll rechecks https://review.openstack.org/#/c/232257/22:16
patchbotpatch 232257 - ironic - Detect when log messages are mis-formatted22:16
* jroll sees rloo has already realized all this and rechecked22:16
rloojroll: well, a little angel M* told me that it should have been fixed :-)22:17
jroll:)22:18
*** e0ne has quit IRC22:37
*** rcernin has joined #openstack-ironic22:37
*** AlexeyAbashkin has joined #openstack-ironic22:38
*** e0ne has joined #openstack-ironic22:38
*** e0ne has quit IRC22:39
*** namnh has joined #openstack-ironic22:39
*** e0ne has joined #openstack-ironic22:40
*** e0ne has quit IRC22:40
*** e0ne has joined #openstack-ironic22:40
*** e0ne has quit IRC22:41
*** AlexeyAbashkin has quit IRC22:42
*** namnh has quit IRC22:43
*** e0ne has joined #openstack-ironic22:59
*** dprince has quit IRC23:00
*** e0ne has quit IRC23:00
*** e0ne has joined #openstack-ironic23:01
*** e0ne has quit IRC23:01
*** e0ne has joined #openstack-ironic23:02
*** e0ne has quit IRC23:02
*** e0ne has joined #openstack-ironic23:02
*** e0ne has quit IRC23:03
*** e0ne has joined #openstack-ironic23:03
*** e0ne has quit IRC23:04
*** e0ne has joined #openstack-ironic23:04
*** e0ne has quit IRC23:04
*** catinthe_ has joined #openstack-ironic23:08
*** catinthe_ has quit IRC23:13
*** tonyb has quit IRC23:26
*** tonyb has joined #openstack-ironic23:36
*** harlowja has quit IRC23:38
*** Goneri has quit IRC23:51
*** d0ugal has quit IRC23:51
* TheJulia wishes all network cards just spoke ipxe to begin with23:53
*** jcoufal has quit IRC23:59

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