Friday, 2014-06-20

JoshNanghey, i'd love to get some more input on this (whether it's needed or not): https://review.openstack.org/#/c/98922/00:04
*** hemna has quit IRC00:09
*** takadayuiko has joined #openstack-ironic00:21
*** igordcard has quit IRC00:23
*** matsuhashi has joined #openstack-ironic00:28
*** mrda has quit IRC00:47
*** matsuhashi has quit IRC00:48
*** matsuhashi has joined #openstack-ironic00:49
*** mrda has joined #openstack-ironic00:51
*** matsuhashi has quit IRC00:54
*** matsuhashi has joined #openstack-ironic01:01
devanandaand my laptop returns from battery-death :)01:04
devanandaJoshNang: hi! I don't understand the use-case for that01:04
*** Abhishek has quit IRC01:07
*** matsuhashi has quit IRC01:09
*** matsuhashi has joined #openstack-ironic01:10
*** yjiang5 is now known as yjiang5_away01:11
*** keekz has joined #openstack-ironic01:15
*** theDavidAiken has quit IRC01:24
*** ellenh has quit IRC01:24
*** eguz has joined #openstack-ironic01:25
*** eghobo has quit IRC01:30
*** nosnos has joined #openstack-ironic01:30
devanandarloo: around? any chance you'll revive the fix to https://bugs.launchpad.net/ironic/+bug/1261915 ?01:42
rloodevananda: yeah. I wanted to ping you about it. I was actually wondering if it needed a BP/spec, cuz of the API.01:43
rloodevananda: I didn't want to start working on the patch again, only to be told later to put together a spec.01:44
rloodevananda: the question of implementing via rpc/conductor, or going to the DB seems to fit into a spec/design. Anyway, those were my thoughts on it this week ;)01:45
devanandarloo: yea. that thought has crossed my mind as well01:56
devanandarloo: since there is an API change, requiring a spec would be in keeping with everything else we're doing01:57
devanandarloo: i think this is a feature we need to land in Juno, tbh. because without it, clients can't discover our API fully01:58
*** ellenh has joined #openstack-ironic02:00
rloodevananda: yeah. Fair is fair. I should suffer like everyone else ;)02:00
devanandai'm not as concerned about the synchronous nature of it as I would be with other things02:00
devanandathis doesn't touch hardware02:00
devanandait's just an RPC trip to get data from a conductor02:01
devanandait'll suffer if the conductor is overloaded, but it's not significantly adding to that. there's no TaskManager lock on the other side02:01
rloodevananda: so you're OK with it doing RPC, vs saving the info in the DB? Well, I guess I have to write it up in a spec anyway. romcheg didn't like the rpc part.02:02
devanandarloo: off topic, part of what's in my mind as I think about this in particular is our need to support upgrades02:02
devanandarloo: eg, support someone running stable/icehouse today who will want to upgrade to stable/juno in a few months02:03
devanandaand the testing of that which we will need to provide (eg, via tempest and grenade)02:03
rloo*shudder*. but yeah.02:03
devanandawhich expect to be able to *discover* which version of the service they're talking to *and* what its capabilities are02:03
devanandaiow, i think drivers need to expose "here's what I can do" in some way. and we've chosen not to follow the extension model that nova uses. this is what we've got instead02:04
devanandarloo: actually, thanks for the reminder. stashing the properties' list on the `ironic`.`conductors` table is a better solution02:06
devanandaheh, see? that's why a spec would e helpful :)02:06
rlooha ha.02:06
rlooyeah, i knew there was a reason why my patch for that wasn't approved for so long. Cuz it needed a spec!02:07
devananda:p02:07
rloodevananda: I'll start on that next week.02:07
devanandarloo: cheers!02:08
rloodevananda: I just want to get one-more review done tonight, and that'll be it until Monday. (yay)02:08
* rloo wonders how long it'll take for the core-spec'ers to approve the spec-that-will-exist though ;) Will see...02:10
*** ellenh has quit IRC02:10
* devananda wonders who will becomethe spec-czar02:10
*** rwsu has quit IRC02:24
*** eguz has quit IRC02:33
*** harlowja is now known as harlowja_away02:56
*** vinbs has joined #openstack-ironic03:02
*** matsuhashi has quit IRC03:02
*** matsuhashi has joined #openstack-ironic03:03
*** rloo has quit IRC03:06
*** matsuhashi has quit IRC03:07
*** nosnos has quit IRC03:18
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements  https://review.openstack.org/9622803:33
*** ramineni has joined #openstack-ironic03:38
*** Abhishek has joined #openstack-ironic03:45
*** theDavidAiken has joined #openstack-ironic03:49
*** matsuhashi has joined #openstack-ironic03:53
*** Poornima has joined #openstack-ironic03:59
*** matsuhashi has quit IRC04:02
*** matsuhashi has joined #openstack-ironic04:02
*** matsuhashi has quit IRC04:06
*** nosnos has joined #openstack-ironic04:11
*** rameshg87 has joined #openstack-ironic04:26
*** eghobo has joined #openstack-ironic04:40
*** dwalleck has joined #openstack-ironic04:44
*** radsy has quit IRC04:45
*** Mikhail_D_ltp has joined #openstack-ironic04:49
*** openstackgerrit has quit IRC04:57
*** rakesh_hs2 has joined #openstack-ironic05:02
*** dwalleck_ has joined #openstack-ironic05:03
*** dwalleck has quit IRC05:04
*** Abhishek has quit IRC05:05
*** Abhishek has joined #openstack-ironic05:08
*** dwalleck_ has quit IRC05:12
*** coolsvap|afk is now known as coolsvap05:22
*** lazy_prince has joined #openstack-ironic05:27
*** sabah has joined #openstack-ironic05:34
*** k4n0 has joined #openstack-ironic05:43
mrdaHave a nice weekend Ironic!05:47
*** mrda is now known as mrda-away05:48
lifelessmrda-away: was that ... boom tish, ironic ? :P05:51
*** loki184 has joined #openstack-ironic05:53
*** eghobo has quit IRC06:11
*** Mikhail_D_ltp has quit IRC06:12
*** Abhishek has quit IRC06:14
*** theDavidAiken has quit IRC06:16
*** rameshg87 has left #openstack-ironic06:34
*** rameshg87 has joined #openstack-ironic06:50
*** max_lobur has joined #openstack-ironic07:00
*** rameshg87 has quit IRC07:04
*** pquerna has quit IRC07:08
*** pquerna has joined #openstack-ironic07:08
*** rameshg87 has joined #openstack-ironic07:09
dtantsur|afkmorning Ironic!07:13
*** dtantsur|afk is now known as dtantsur07:13
*** sysexit has joined #openstack-ironic07:14
rameshg87dtantsur, good morning07:15
rameshg87dtantsur, a quick question, do you use an irc bouncer to stay connected to the channel ?07:16
dtantsurrameshg87, yes, ZNC07:16
rameshg87dtantsur, have you installed on your own server or do you use elite bnc ?07:17
dtantsurrameshg87, I'm using server in Red Hat office07:17
rameshg87dtantsur, ah okay :-)07:17
dtantsurrameshg87, elite bnc looks promising, but I need to access also server behind VPN07:19
rameshg87dtantsur, okay. i am just trying to join elitebnc, that's why i just asked :-)07:19
*** bmahalakshmi has joined #openstack-ironic07:20
*** rameshg8- has joined #openstack-ironic07:25
*** max_lobur has quit IRC07:30
*** rameshg87 has quit IRC07:46
*** rameshg87 has joined #openstack-ironic07:47
*** r-mibu has joined #openstack-ironic07:51
*** athomas has joined #openstack-ironic07:52
*** jcoufal has joined #openstack-ironic08:00
*** martyntaylor has joined #openstack-ironic08:08
*** foexle has joined #openstack-ironic08:10
*** r-mibu has left #openstack-ironic08:13
*** max_lobur has joined #openstack-ironic08:14
*** yuriyz has quit IRC08:20
*** lucasagomes has joined #openstack-ironic08:23
*** igordcard has joined #openstack-ironic08:23
*** romcheg has joined #openstack-ironic08:37
*** yuriyz has joined #openstack-ironic08:39
*** eglynn has joined #openstack-ironic08:42
*** romcheg has quit IRC08:46
*** romcheg has joined #openstack-ironic08:46
*** Mikhail_D_ltp has joined #openstack-ironic09:04
*** takadayuiko has quit IRC09:17
*** davidlenwell has quit IRC09:27
*** romcheg has quit IRC09:27
*** rakesh_hs2 has quit IRC09:38
*** rakesh_hs2 has joined #openstack-ironic09:39
lucasagomesdevananda, #96136 rebased on master09:41
*** pelix has joined #openstack-ironic09:42
*** sabah has quit IRC09:47
*** romcheg has joined #openstack-ironic09:51
*** davidlenwell_ has joined #openstack-ironic09:52
*** davidlenwell_ is now known as davidlenwell09:52
*** davidlenwell is now known as davidlenwell_09:52
*** davidlenwell_ is now known as davidlenwell09:52
*** dguerri is now known as _dguerri09:53
*** _dguerri is now known as dguerri09:53
vinbsHello Ironic!09:59
vinbsmorning lucasagomes :)10:00
romchegMorning Ironic!10:00
lucasagomesmorning all10:00
vinbsmorning romcheg!10:00
romchegMorning vinbs!10:00
romchegand lucasagomes :)10:00
vinbslucasagomes, after uploading proper kernel and ramdisk images to glance, my OS deploy on baremetal went through :)10:09
lucasagomesvinbs, nice :D10:10
vinbslucasagomes, but I see this warning on my baremetal server while it boots up :10:10
vinbshttps://wiki.openstack.org/wiki/How_To_Contribute10:10
vinbssry wrong link10:10
vinbsi see this error:10:10
vinbsurl_helper.py[WARNING]: Calling 'http://169.254.169.254/2009-04-04/meta-data/instance-id' failed [120/120s]: request error [HTTPConnectionPool(host='169.254.169.254', port=80): Max retries exceeded with url: /2009-04-04/meta-data/instance-id (Caused by <class 'socket.error'>: [Errno 113] No route to host)]10:10
vinbs*warning10:10
vinbslucasagomes, why do I get this warning?10:11
lucasagomesvinbs, it's a problem accessing the meta-data server I suppose... gotta digg more into into tho10:13
*** rakesh_hs3 has joined #openstack-ironic10:13
lucasagomesvinbs, http://askubuntu.com/questions/165115/cloud-instances-in-openstack-cant-import-public-ssh-key10:14
*** rakesh_hs2 has quit IRC10:14
vinbslucasagomes, let me check it out :)10:17
*** coolsvap is now known as coolsvap|afk10:19
dtantsurmorning, lucasagomes, vinbs, romcheg!10:22
romchegMorning dtantsur!10:23
*** coolsvap|afk is now known as coolsvap10:31
dtantsurFolks, what do you think about PXE current cache limit of 1 GiB judging by logs in https://bugs.launchpad.net/ironic/+bug/1326556 it maybe unreasonably small10:39
yuriyzmorning Ironic10:39
yuriyzdtantsur, agree it is small10:40
dtantsurmorning, yuriyz10:40
dtantsurany ideas on making it more reasonable but still safe for most cases? 4GiB?10:41
yuriyzthis can be 8-10GiB10:43
romchegI agree with yuriyz10:44
*** rakesh_hs4 has joined #openstack-ironic10:46
*** rakesh_hs3 has quit IRC10:47
dtantsurok, so let me try to make a patch with 8 GiB...10:47
dtantsurNext question, folks. TTL is 1 hour now. Is it also enough? For me it looks ok10:48
dtantsuryuriyz, romcheg ^^^10:48
romchegProvisioning could take more time10:48
romchegIf it's some bulk provisioning10:49
yuriyzI think it is small also10:49
dtantsurIdeas?10:50
romcheg5?10:51
yuriyz4h minimum10:51
romchegI don't know why 5, but I like that number :)10:51
dtantsurCan we have some arguments for it?10:51
dtantsurOr maybe even write a spec explaining this change?10:51
romchegdtantsur: I'm trying to come up with one10:51
dtantsurI believe we need a written reasoning for why we chose them10:52
dtantsurlucasagomes, can we have some final agreement on what size hints to use in https://review.openstack.org/#/c/94371/ ?10:53
lucasagomesdtantsur, :( that's what I want10:54
dtantsurlucasagomes, should I update with checks for raw format? Should I leave some more space just in case?10:54
lucasagomesdtantsur, probably we can discuss it on the next meeting where everyone is online10:54
lucasagomesohhh the size hitns10:54
lucasagomessorry I was confused about the parameters for partitioning10:54
lucasagomesthe unit sizes10:54
lucasagomesdtantsur, sure, will take a look at the patch again10:54
lucasagomes1 sec10:54
*** openstackgerrit has joined #openstack-ironic10:56
*** ramineni has quit IRC11:00
lucasagomesdtantsur, so, alright I'm grand to give it a go then11:01
lucasagomesdtantsur, 2* might be fine... I'd love to see other ppl opnions on that hto11:01
lucasagomestho*11:01
dtantsurlucasagomes, ok, let us see, thank you :)11:02
*** nosnos has quit IRC11:03
lucasagomesdtantsur, I added devananda and ruby to the list of reviewers to have their opnion11:04
dtantsurack11:04
lucasagomesand changed my vote from +1 to +211:04
lucasagomesto show I'm fine with that :)11:05
*** dguerri is now known as _dguerri11:06
romchegSo if I understand it right an image after converting cannot be bigger than a raw one.11:07
romchegThan x2 seems to be good11:07
*** _dguerri is now known as dguerri11:07
openstackgerritA change was merged to openstack/ironic: Use temporary dir for image conversion  https://review.openstack.org/9833811:07
*** igordcard has quit IRC11:09
lucasagomesromcheg, the normal case is convert it to raw from a compacted format11:10
lucasagomesso it could be twice the size after converting to raw AFAIUI11:10
romcheglucasagomes: that's what I mean, sorry if confused :)11:11
romchegI will share my opinion in the review11:11
lucasagomesromcheg, :) aight, thank u11:12
*** keekz has quit IRC11:14
*** keekz has joined #openstack-ironic11:16
*** martyntaylor has quit IRC11:18
romcheggotta go for a lunch, will be back soon11:18
*** rakesh_hs4 has quit IRC11:28
*** rakesh_hs4 has joined #openstack-ironic11:30
*** coolsvap is now known as coolsvap|afk11:31
*** rakesh_hs4 has quit IRC11:33
*** martyntaylor has joined #openstack-ironic11:34
*** martyntaylor has quit IRC11:35
*** martyntaylor has joined #openstack-ironic11:35
*** rakesh_hs4 has joined #openstack-ironic11:36
*** igordcard has joined #openstack-ironic11:38
*** Poornima has quit IRC11:46
*** lucasagomes is now known as lucas-lunch11:50
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: Fix leaking DB details to API on error  https://review.openstack.org/7312111:52
*** vinbs has quit IRC12:04
*** jdob has joined #openstack-ironic12:07
*** athomas has quit IRC12:21
foexlehiho guys, is there any package installation needed for compute ?12:29
foexlebecause i get nova [-] ImportError: No module named ironic.nova.compute.manager12:29
foexlemaybe the common package ?!12:29
foexleicehouse => ubuntu 14.0412:30
*** k4n0 has quit IRC12:30
*** eglynn is now known as eglynn-call12:32
*** jcoufal has quit IRC12:35
foexleall right python-ironic package ... you should add this to your doc => http://docs.openstack.org/developer/ironic/deploy/install-guide.html12:41
foexle:)12:41
openstackgerritA change was merged to openstack/ironic: Test fake console driver  https://review.openstack.org/9762912:50
*** lucas-lunch is now known as lucasagomes12:54
*** jbjohnso has joined #openstack-ironic12:55
*** rameshg87 has left #openstack-ironic12:56
*** ifarkas has quit IRC13:00
*** loki184 has quit IRC13:04
dtantsurfoexle, hi! Thank you for raising this, we have a bug for installation guide improvements: https://bugs.launchpad.net/ironic/+bug/132358913:11
dtantsurfoexle, but! Change you mentioned will not be required after our driver is merged into Nova code, so I guess this won't be fixed :)13:11
*** bmahalakshmi has quit IRC13:14
*** praefect has joined #openstack-ironic13:15
dtantsurrelocating, brb13:21
foexledtantsur: ahhh nice, all right good to know :) ... thanks for that hint. Ill take a look into devstack :)13:24
*** linggao has joined #openstack-ironic13:24
*** jdob has quit IRC13:29
*** jdob has joined #openstack-ironic13:29
*** sysexit has quit IRC13:33
foexlea last comprehension question, should be the conductor service running on compute node/s ? or on controller node ?13:34
*** jbjohnso has quit IRC13:43
*** jdob_ has joined #openstack-ironic13:50
*** jdob has quit IRC13:53
*** jbjohnso has joined #openstack-ironic13:55
*** mkerrin has quit IRC13:56
*** mkerrin has joined #openstack-ironic13:59
dtantsurfoexle, maybe I'm not the best source of information on this topic, but it does not seem to really matter14:01
dtantsurconductor is quite an independent service (or independent services - you can have many)14:02
*** linggao_ has joined #openstack-ironic14:04
*** jrist has quit IRC14:04
*** comstud is now known as bearhands14:04
*** linggao_ has quit IRC14:05
*** linggao_ has joined #openstack-ironic14:06
*** rakesh_hs4 has quit IRC14:07
*** linggao has quit IRC14:07
NobodyCamgood morning ironic14:11
romchegMorning NobodyCam!14:12
dtantsurmorning, NobodyCam14:13
NobodyCammorning romcheg & dtantsur14:19
NobodyCamTGIF!14:19
dtantsuroh yes :)14:19
romchegTGIABOC!14:20
romchegABOC === almost beer o'clock :)14:20
*** jrist has joined #openstack-ironic14:20
*** jrist has quit IRC14:21
*** jrist has joined #openstack-ironic14:21
*** theDavidAiken has joined #openstack-ironic14:24
*** Abhishe__ has joined #openstack-ironic14:25
lucasagomesmorning NobodyCam14:26
*** Abhishe__ has quit IRC14:30
*** Mikhail_D_ltp has quit IRC14:31
*** rwsu has joined #openstack-ironic14:35
jrollmorning ironic14:45
dtantsurjroll, morning14:48
NobodyCamlol14:52
NobodyCammorning lucasagomes14:52
NobodyCamnow to figure out why neutron is failing to bind the port to boot from14:54
*** martyntaylor has quit IRC14:57
*** sysexit has joined #openstack-ironic15:00
lucasagomesNobodyCam, :(15:12
romchegNobodyCam: are you looking at the same problem I hit yesterday?15:12
*** max_lobur1 has joined #openstack-ironic15:13
*** ifarkas has joined #openstack-ironic15:15
*** mdorman has joined #openstack-ironic15:15
*** theDavidAiken has quit IRC15:17
*** max_lobur has quit IRC15:17
NobodyCam??? romcheg what did you hit15:39
romchegnode gets stuck in wait for callback15:39
romchegfails to pxeboot15:40
NobodyCamyes .. but the gets dhcp address15:40
NobodyCamI have been looking at tcpdumps15:40
NobodyCamI see dhcp request15:40
NobodyCambut nothing for tftp requests15:41
NobodyCamromcheg: do you have a fix?15:41
romchegNo, still debuging15:41
NobodyCamcheck the neutron port status.. I'll bet its down15:42
lucasagomeshave u guys checked the iptables rules?15:42
romchegI remember hitting the same a few month ago15:42
romchegThe port was down as well15:43
*** Nisha has joined #openstack-ironic15:44
*** yjiang5_away is now known as yjiang515:45
NobodyCamlucasagomes: iptables looks ok15:46
NobodyCamnothing waco15:46
lucasagomes:/15:47
NobodyCamhttp://paste.openstack.org/show/yNuSaFnUd0HaOmUIGLqe/15:47
*** theDavidAiken has joined #openstack-ironic15:48
NobodyCamtop is my local seed ... bottom is system with issue15:48
*** eghobo has joined #openstack-ironic15:52
devanandamorning, all15:53
NobodyCamgood morning devananda15:53
romchegMorning devananda!15:53
jroll\o devananda15:54
dtantsurdevananda, morning!15:54
Nishamorning devananda !15:56
devanandafoexle, dtantsur: ironic-conductor service is quite independent from where nova* services run. depending on your security model/requirements, it may need to be on a physically separate machine15:57
lucasagomesdevananda, morning15:57
*** foexle has quit IRC15:57
devanandafoexle, dtantsur: because ir-conductor is the only service which requires access to both data and IPMI networks15:57
Nishadtantsur: : did you had chance to further review Ilo power driver spec? https://review.openstack.org/#/c/97455/6 I know you had given comments on it, and after that a new patch has been posted.15:59
dtantsurNisha, looking. Also please find someone who does know a thing or two about iLO to review it.16:00
Nishadevananda: could you please too review the design spec?16:00
Nishai think devananda should know about iLO16:00
Nishadevananda: am i correct?16:01
devanandaNisha: Yes. I haven't had time lately, but it's on my list to do16:01
NishaThanks devananda16:01
dtantsurNisha, not all my comment were addressed. I asked for a list of new options to introduce into driver_info16:02
romchegGotta go guys, have a nice weekend16:02
lucasagomesromcheg, g'night16:02
dtantsurg'night, romcheg!16:02
NobodyCamhave a great weekend romcheg16:02
Nishadtantsur: : Ok apart from that if you you have any further comments could you log them too16:03
romchegNobodyCam: Oh, it will be really unusual :)16:03
devanandaNisha: regarding the "ilo client python library" which was part of the code proposed in Icehouse, is that now available at https://github.com/hpproliant/proliantutils ?16:03
NobodyCam:)16:03
NishaYes deva16:04
*** romcheg has quit IRC16:04
lucasagomesdevananda, can I pick ur brain a little bit? re https://review.openstack.org/#/c/95551 ... So I agree we might want to wait till K complete remove the deploy kernel and ramdisk from the flavor16:04
Nishadevananda: yes we seperated it and it is availabel under that link16:05
lucasagomesdevananda, but what you think about making that code check if the pxe_deploy_{kernel, ramdisk} are already set on the node, and if so, do not overwrite it with the ones in the flavor16:05
NobodyCamanyone know how to update binding:host_id via neutron cli16:05
devanandalucasagomes: the problem with that will be cleanup16:05
lucasagomesalso if they are not set we might even want to log a message saying that having the kernel and ramdisk on the flavor will be deprecated16:05
Nishadtantsur: i will let the owner know to update the patch with your comments.16:06
lucasagomesdevananda, hmm16:06
lucasagomesdevananda, yeah that's true :(16:06
devanandalucasagomes: if pxe_deploy* on flavor, nova driver will stash it on drver_info and delete from driver_info.16:06
dtantsurdevananda, I see you're already occupied :) but if you have time, please have a look at approach at https://review.openstack.org/#/c/73121/ this my attempt to revive patch for disclosing SQL errors16:06
devanandalucasagomes: if it's not on the flavor, the driver does neither of those (and either it's already on the node's driver_info,a nd things work, or the node fails validate)16:07
devanandalucasagomes: I think the most backwards-compatible thing we can do is ^16:07
devanandaNisha: thanks - good to see it has been refactored into a library. that'll make the ironic ilo driver code much easier to review16:08
lucasagomesdevananda, yeah sounds correct, thanks16:08
Nishadevananda: , thanks16:09
devanandaviktors: hi! you've been working on oslo.db lately, yes?16:10
Nishadevananda: : could you please take out some time to review it, as code is reviewed and is waiting for spec to be approved16:10
viktorsdevananda: Hi! Yes, you right16:11
devanandadtantsur: i'd like to know that this is in line with oslo.db as it seems like something which would affect many projects16:11
devanandaviktors: when you have time, could you comment on https://review.openstack.org/#/c/73121/1016:11
dtantsuryeah, would be nice16:12
devanandaviktors: tldr; sqla errors are bubbling up,a nd we need to catch them, log them, and convert to a user-friendly error message. does oslo.db provide somthing like this, or should we implement it locally?16:12
viktorsdevananda: ok, but it it's 7pm Friday in my country, o I'm not sure about today :)16:12
viktorsdevananda: in a firs look,  oslo.db should provide something like this16:13
Nishadevananda: about https://review.openstack.org/#/c/100951/5 , i will post the new patch with your yesterday's comments but i was concerned abt disk size parameter after yesterday's discussion.16:13
Nishashould i go ahead and populate the disk size in properties field?16:14
Nishadevananda: or i should not populate that16:15
devanandaviktors: fair enough :) I think dtantsur is also near your timezone, so perhaps you two can sync up next week?16:16
* dtantsur makes a note16:16
viktorsdevananda: ok, sure will do16:16
Nishadevananda: ?16:19
dtantsurdevananda, are you going to have a look at oslo.db patch https://review.openstack.org/#/c/42159 ? Otherwise, I'm approving it16:24
devanandadtantsur: wait16:28
dtantsurack16:28
devanandadtantsur: some things should not be approved so quickly...16:28
dtantsurthat's why I'm asking :)16:28
devanandai'm tempted to -2 it just to be safe now16:29
dtantsurheh16:30
openstackgerritAleksandr Gordeev proposed a change to openstack/ironic-python-agent: Improve GenericHardwareManager  https://review.openstack.org/9284716:30
openstackgerritAleksandr Gordeev proposed a change to openstack/ironic-python-agent: Add hardware_utils  https://review.openstack.org/9239916:30
dtantsurdevananda, and the last question for today: are we writing specs for client changes (e.g. https://review.openstack.org/#/c/101297/ )?16:31
devanandadtantsur: generally no. but that's an interesting change ....16:33
dtantsuryeah, I have a gut feeling that spec may be required here for people to review cli interface16:34
lucasagomesdtantsur, devananda I think that that kinda of change may not need a spec because the client is just implement something that Ironic API already have... But for other kinds of changes, for e.g auto completion, output with colors etc... It maybe need a spec16:40
lucasagomesdevananda, sorry, I saw you got a lot of ping's... but I'm kinda struggling with the REST API for the management interface (the sync vs async thing)16:43
lucasagomesdevananda, I was thinking about making it sync for things like get_boot_device, but drivers could implement a cache if needed (ipmi* drivers since the BMC is flaky)16:44
lucasagomesit could look something like... request came -> if the boot device was already checked within <time> just return what is in the db16:45
lucasagomesif not, ask the BMC; set the time to current time; return16:45
lucasagomesand update the db in case of it's inconsistent16:46
*** eglynn-call has quit IRC16:46
lucasagomesdevananda, for settting the boot device it could always be async16:46
devanandalucasagomes: i dont have a perfect solution now .. .the async api stuff has been stalled as every discussion there ends up turning into, essentially, a /v2/ API16:49
lucasagomesdevananda, right16:50
devanandalucasagomes: also, rloo has started thinking about how to make the driver-properties work async by stashing it in the db16:50
devanandalucasagomes: which sounds somewhat similar to your suggestiona bove16:50
lucasagomesdevananda, I thought about the cache because the last time we talked we agreed to avoid having periodic tasks to sync stuff by poking the BMC16:50
*** yjiang5 is now known as yjiang5_afk16:50
lucasagomesdevananda, right16:50
devanandalucasagomes: yah. i have to run, and just got out the oslo meeting, maybe capture your thoughts in an etherpad or a spec draft and let's review on monday?16:51
lucasagomesit feels like a resonable solution for me, no periodic tasks and cache to avoid poking the bmc16:51
lucasagomesdevananda, sure16:51
lucasagomesdevananda, thanks16:51
devanandasounds good at first glance - would like to see more detail / have timeto digest it16:52
lucasagomesdevananda, will update the spec to reflect the idea16:52
*** yjiang5_afk is now known as yjiang516:53
*** viktors is now known as viktors|afk16:56
*** harlowja_away is now known as harlowja17:00
*** eghobo has quit IRC17:00
*** eghobo has joined #openstack-ironic17:01
dtantsurhave to go, folks! g'night and have a nice weekend!17:02
*** dtantsur is now known as dtantsur|afk17:02
lucasagomesdtantsur, g'night17:02
lucasagomesI'm heading to, see u all17:02
lucasagomesg'night and enjoy the weekend17:02
*** lucasagomes is now known as lucas-dinner17:02
NobodyCamnight lucas-dinner17:07
NobodyCamnight dtantsur|afk17:07
*** dguerri is now known as _dguerri17:08
*** Abhishek has joined #openstack-ironic17:09
*** Abhishek has quit IRC17:19
*** romcheg has joined #openstack-ironic17:25
*** lazy_prince has quit IRC17:28
*** rameshg87 has joined #openstack-ironic17:32
*** ellenh has joined #openstack-ironic17:36
*** godp1301 has joined #openstack-ironic17:41
openstackgerritRamakrishnan G proposed a change to openstack/ironic-specs: iLO Power Driver for Ironic  https://review.openstack.org/9745517:47
openstackgerritRamakrishnan G proposed a change to openstack/ironic-specs: iLO Power Driver for Ironic  https://review.openstack.org/9745517:49
*** ellenh has joined #openstack-ironic17:52
openstackgerritMathieu Gagné proposed a change to openstack/ironic: Add genconfig tox job for sample config file generation  https://review.openstack.org/10161417:53
*** Abhishe__ has joined #openstack-ironic17:56
*** godp1301 has quit IRC18:02
*** ellenh has quit IRC18:04
*** ellenh has joined #openstack-ironic18:05
*** max_lobur1 has quit IRC18:06
*** ellenh has quit IRC18:07
*** ellenh has joined #openstack-ironic18:08
*** blamar has joined #openstack-ironic18:12
*** ellenh has quit IRC18:19
*** ellenh has joined #openstack-ironic18:20
*** pelix has quit IRC18:24
*** Abhishe__ is now known as abhi_18:34
*** abhi_ is now known as abhi_cs18:35
*** Nisha has quit IRC18:43
*** devananda has quit IRC18:47
NobodyCamshould the overcloud's gateway be the undercloud (192.0.2.2) or the seed's (192.0.2.1)?18:48
NobodyCamdoh ww18:50
* jroll hacks NobodyCam's undercloud18:50
NobodyCamlol18:50
NobodyCamhack away18:51
theDavidAikenI'm getting a "Could not find disk to use." during a baremetal install, a few network errors then it just sticks, any ideas on troubleshooting?18:52
NobodyCamjroll: while you at it could you fix :<insert very long list of things that are wrong>18:52
jroll:P18:53
NobodyCamtheDavidAiken: can you tell us a bit more about the setup?18:53
theDavidAikensure, what do you want to know? I'm running through a Helion baremetal install and this is the undercloud. I see the error on the server console.18:54
*** devananda has joined #openstack-ironic18:55
NobodyCamtftp timeout?18:55
theDavidAikenserver has 2 nics, and virtual/floppy enabled18:55
theDavidAikensome Network is unreachable errors too18:56
mgagneJayF: ping18:57
NobodyCamwhat is the network setup : simple flat, vlan?18:57
theDavidAikenwasn't sure if it was bug 118394018:57
theDavidAikenvlan18:57
*** harlowja is now known as harlowja_away18:57
NobodyCamhumm, any chance you could paste the logs from the nodes console?19:00
*** harlowja_away is now known as harlowja19:01
theDavidAikenlet me see if i can grab them19:02
*** sysexit has quit IRC19:03
devanandajroll: interesting conversation on the nova driver in -nova ya'll may want to listen / participate on19:05
* jroll reads19:05
*** rameshg87 has left #openstack-ironic19:06
openstackgerritJarrod Johnson proposed a change to stackforge/pyghmi: Tolerate more privilege degradation scenarios  https://review.openstack.org/10162819:10
openstackgerritA change was merged to stackforge/pyghmi: Tolerate more privilege degradation scenarios  https://review.openstack.org/10162819:13
*** praefect has quit IRC19:13
JayFmgagne: What's up? I'm here but heavily distracted19:17
mgagneJayF: right, quick question: why 3 or more, why not 4? https://github.com/rackerlabs/ironic-python-agent/blob/master/ironic_python_agent/shell/write_image.sh#L17-L1819:21
JayFmgagne: because I'm creating a primary partition.19:21
JayFmgagne: just an assumption made to get something working19:21
JayFmgagne: for any images with complex partitions, or GPT, I'd suggest already creating a partition for configdrive, with a dummy filesystem labelled config-219:21
mgagneJayF: sure, isn't the limit of primary partitions 4? if there is already 3, adding a new one for the configdrive would make it 4 which is still valid, no?19:22
JayFmgagne: it would, but you'd remove the ability to ever create extended partitions19:22
JayFmgagne: also that comment should probably be moved to copy_configdrive_to_disk.sh19:22
*** max_lobur has joined #openstack-ironic19:22
JayFsince that's what actually adds a partition now19:22
mgagneJayF: ok, so it's not a technical limitation where it wouldn't work but more like "this wouldn't be ideal for people wanting to create an extended parition"19:23
JayFyeah, exactly19:23
mgagneJayF: cool, thanks for the info =)19:23
JayFfor OnMetal, images that have GPT + complex partitions, we've just used a precreated configdrive partition19:23
JayFand that's working in our environments, it's what powers our coreos image right now19:23
mgagnehehe19:23
mgagneJayF: do you allow people to upload their own images yet?19:24
JayFNo.19:24
mgagneok19:24
mgagneJayF: if yes, I suppose those requirements would be mentioned somewhere19:24
JayFmgagne: almost certainly, but I don't think it's something we at Rackspace would choose to do for a while.19:24
mgagneJayF: alright, thanks for your time!19:25
JayFbut this isn't really relevant to -ironic in general anymore :)19:25
JayFno problem19:25
*** abhi_cs has quit IRC19:35
*** sysexit has joined #openstack-ironic19:37
*** blamar has quit IRC19:40
*** ellenh has quit IRC19:46
*** Penick has joined #openstack-ironic19:50
*** Mikhail_D_ltp has joined #openstack-ironic19:57
ryanpetrelloanybody know what “NOT_REGISTERED” in failing CI check job means?19:59
ryanpetrellohttps://review.openstack.org/#/c/94244/19:59
jrollweird, idk19:59
ryanpetrellonever seen this one before, and to my knowledge, I haven’t changed any of pecan’s infra configs re: ceilometer19:59
jrollit's just ceilometer, it's ok19:59
jroll:P19:59
ryanpetrelloblurgh19:59
jrollfor real though, maybe they dropped those jobs?19:59
ryanpetrelloposted this in the wrong room :)19:59
jrollheh19:59
*** jdob_ has quit IRC19:59
ryanpetrello*moves on over to openstack-i*nfra*19:59
*** ifarkas has quit IRC20:01
* devananda wanders off to find food20:04
*** romcheg has quit IRC20:05
*** abhi_cs has joined #openstack-ironic20:06
*** romcheg has joined #openstack-ironic20:08
NobodyCam:)20:09
*** Penick has quit IRC20:11
*** sysexit has quit IRC20:12
*** ellenh has joined #openstack-ironic20:12
*** abhi_cs has quit IRC20:14
*** ellenh has quit IRC20:14
*** abhi_cs has joined #openstack-ironic20:15
*** Penick has joined #openstack-ironic20:16
*** max_lobur has quit IRC20:22
*** ellenh has joined #openstack-ironic20:37
*** eghobo has quit IRC20:48
*** eghobo has joined #openstack-ironic20:48
*** ryanpetrello has left #openstack-ironic20:53
*** linggao_ has quit IRC20:57
*** mdorman has quit IRC21:03
*** jbjohnso has quit IRC21:03
*** sysexit has joined #openstack-ironic21:04
*** Mikhail_D_ltp has quit IRC21:07
*** abhi_cs has quit IRC21:19
*** abhi_cs has joined #openstack-ironic21:19
JayFRFR https://review.openstack.org/#/c/101654/ -- this is an IPA builder that JoshNang has worked on today21:27
*** romcheg has quit IRC21:27
*** romcheg has joined #openstack-ironic21:27
*** eguz has joined #openstack-ironic21:31
*** eghobo has quit IRC21:34
*** romcheg has quit IRC21:42
*** abhi_cs has quit IRC21:52
*** abhi_cs has joined #openstack-ironic21:54
*** abhi_cs_ has joined #openstack-ironic21:57
*** abhi_cs has quit IRC21:59
*** abhi_cs_ has quit IRC22:01
*** Abhishe__ has joined #openstack-ironic22:03
jrollJayF / JoshNang : sick22:23
JoshNang:D22:23
jrollJoshNang: one question, target: 'tarballs/ironic-python-agent/coreos/ipa-coreos.tar.gz'22:24
jrollwill this not be overwritten each time?22:24
jrolloh22:24
jrollthis is post-merge22:24
jrollignore me22:25
jroll :)22:25
JoshNangyup! we went back and forth on which was was better. i figure we'll build it locally in devstack for testing22:25
jroll+122:26
JayFjroll: baby steps22:31
JayFjroll: once we get this working, I'd like to do it more often, and maybe get some for each sha122:31
JayFwe can also add some stuff to the gate+check queues to ensure the build /will/ work when it gets there22:31
JayFjust -infra doesn't want to publish all of the builds22:32
jrollJayF: no, I was just thinking it was in the check and multiple patches would clobber each other22:34
jrollJayF: it's totally fine as is :)22:34
JayFdevananda: https://review.openstack.org/#/c/101654 when this merges, we're going to have automatic CoreOS agent image builds in the post-merge for ipa22:34
JayFdevananda: as you guys have requested for a /very long/ time22:35
devanandaJayF: woot!22:35
devanandaJayF: next up: devstack testing using said image + IPA driver22:36
jrolldevananda: that's... in the works :P22:36
jrolldevananda: did you ever glance at https://review.openstack.org/#/c/101020/ ?22:36
openstackgerritJay Faulkner proposed a change to openstack/ironic-python-agent: Fix comments to reflect reality  https://review.openstack.org/10166422:37
jrolldevananda: I'd like you to poke at that a little before I keep going much22:37
JayFjroll: JoshNang: https://review.openstack.org/#/c/101664/ +2 but no +A yet, please22:37
JayFJust a small comment fix commit in order to test our builder... but we need to wait for the builder to merge :)22:37
* jroll +A's anyway22:37
jrollJayF: just poke me when your other thing lands?22:38
devanandaJayF: got a walkthrough to stand that up on my hardware?22:38
*** eguz has quit IRC22:38
devananda*that i can follow22:38
jrollJayF: line 17 trailing whitespace22:38
jrolldevananda: just pxe boot it and grab a console22:38
*** eghobo has joined #openstack-ironic22:38
jroll(once there's a build to use)22:38
JayFdevananda: https://github.com/openstack/ironic-python-agent/blob/master/imagebuild/coreos/README.md is reasonably close but a little out of date22:38
*** Sam_S has quit IRC22:38
JayFdevananda: I'll add improving that readme to my list of things todo22:38
jrolloh, right, that's a thing too22:39
openstackgerritJay Faulkner proposed a change to openstack/ironic-python-agent: Fix comments to reflect reality  https://review.openstack.org/10166422:39
JayFjroll: ^ fixed22:39
jrolldevananda: it currently doesn't really start if it can't lookup its node22:39
jrolldevananda: but theoretically it should work with 101020 running (haven't actually tested that yet)22:41
* devananda gets pulled away by meatspace22:41
*** sysexit has quit IRC22:43
openstackgerritJay Faulkner proposed a change to openstack/ironic-python-agent: Improve readme for CoreOS imagebuild  https://review.openstack.org/10166722:54
*** Penick has quit IRC22:55
JayFdevananda: https://review.openstack.org/101667 improves that readme for ipa / coreos images. That should work given a working pxe environment. LMK if you have any questions.22:55
JayFAs an aside, we should migrate *.md in the repo to *.rst22:55
openstackgerritA change was merged to openstack/ironic-python-agent: Fix comments to reflect reality  https://review.openstack.org/10166422:57
JayF^ filed a bug for the .md -> .rst stuff here: https://bugs.launchpad.net/ironic/+bug/133274123:01
*** ellenh has quit IRC23:03
*** ellenh has joined #openstack-ironic23:06
openstackgerritA change was merged to openstack/ironic-python-agent: Improve readme for CoreOS imagebuild  https://review.openstack.org/10166723:22
*** ellenh has quit IRC23:24
*** foexle has joined #openstack-ironic23:43
*** ellenh has joined #openstack-ironic23:53

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