Monday, 2014-10-06

*** shakamunyi has joined #openstack-ironic00:22
*** shakamunyi has quit IRC00:29
*** shakamunyi has joined #openstack-ironic00:30
*** shakamunyi has quit IRC00:39
*** shakamunyi has joined #openstack-ironic01:04
*** lazy_prince is now known as killer_prince01:09
*** shakamunyi has quit IRC01:14
*** killer_prince has quit IRC01:18
*** shakamunyi has joined #openstack-ironic01:39
*** shakamunyi has quit IRC01:54
*** shakamunyi has joined #openstack-ironic02:20
*** killer_prince has joined #openstack-ironic02:24
*** killer_prince is now known as lazy_prince02:24
*** shakamunyi has quit IRC02:36
*** shakamunyi has joined #openstack-ironic03:03
*** Haomeng|2 has joined #openstack-ironic03:08
*** Haomeng has quit IRC03:08
*** shakamunyi has quit IRC03:11
*** Poornima has joined #openstack-ironic04:04
*** ramineni has joined #openstack-ironic04:07
*** shakamunyi has joined #openstack-ironic04:08
*** spandhe has joined #openstack-ironic04:17
*** nosnos has joined #openstack-ironic04:18
*** nikunj2512 has joined #openstack-ironic04:18
*** shakamunyi has quit IRC04:20
*** rushiagr_away is now known as rushiagr04:21
*** rameshg87 has joined #openstack-ironic04:43
*** rushiagr is now known as rushiagr_away04:47
*** spandhe has quit IRC04:59
*** shakamunyi has joined #openstack-ironic05:08
*** shakamunyi has quit IRC05:24
*** lazy_prince has quit IRC05:26
*** takadayuiko has joined #openstack-ironic05:32
*** pensu has joined #openstack-ironic05:34
*** coolsvap|afk is now known as coolsvap05:35
*** killer_prince has joined #openstack-ironic05:36
*** killer_prince is now known as lazy_prince05:36
*** k4n0 has joined #openstack-ironic05:44
*** rushiagr_away is now known as rushiagr05:48
*** shakamunyi has joined #openstack-ironic05:50
*** lazy_prince has quit IRC05:51
*** pradipta has joined #openstack-ironic05:52
*** lazy_prince has joined #openstack-ironic05:53
*** shakamunyi has quit IRC05:57
*** ndipanov has joined #openstack-ironic06:03
*** pensu has quit IRC06:16
*** pensu has joined #openstack-ironic06:35
GheRiveromorning all06:49
*** shakamunyi has joined #openstack-ironic06:53
takadayuikoHi, GheRivero!06:54
*** andreykurilin_ has joined #openstack-ironic07:04
*** ifarkas has joined #openstack-ironic07:06
*** andreykurilin_ has quit IRC07:08
*** shakamunyi has quit IRC07:19
*** jistr has joined #openstack-ironic07:23
*** vinbs has joined #openstack-ironic07:26
*** dtantsur|afk is now known as dtantsur07:37
dtantsurMorning Ironic07:37
takadayuikodtantsur, Hi!07:37
*** shakamunyi has joined #openstack-ironic07:40
*** romcheg has joined #openstack-ironic07:40
*** romcheg has quit IRC07:47
*** shakamunyi has quit IRC07:55
*** romcheg has joined #openstack-ironic07:56
*** lucasagomes has joined #openstack-ironic07:58
*** igordcard has joined #openstack-ironic08:13
*** jcoufal has joined #openstack-ironic08:19
*** igordcard has quit IRC08:24
*** derekh has joined #openstack-ironic08:26
*** ndipanov has quit IRC08:28
*** lsmola has joined #openstack-ironic08:37
*** coolsvap is now known as coolsvap|afk08:40
*** athomas has joined #openstack-ironic08:41
*** Poornima has quit IRC08:44
*** k4n0 has quit IRC08:46
*** Poornima has joined #openstack-ironic08:46
*** coolsvap|afk is now known as coolsvap08:51
*** k4n0 has joined #openstack-ironic09:00
*** viktors|afk is now known as viktors09:02
*** yuanying has quit IRC09:32
*** yuanying has joined #openstack-ironic09:32
*** pradipta is now known as pradipta_away09:33
*** yuanying has quit IRC09:37
*** shakamunyi has joined #openstack-ironic09:41
*** Poornima_ has joined #openstack-ironic09:41
*** k4n0 has quit IRC09:41
*** Poornima has quit IRC09:42
*** shakamunyi has quit IRC09:45
*** pelix has joined #openstack-ironic09:48
*** takadayuiko has quit IRC09:49
*** k4n0 has joined #openstack-ironic09:57
*** killer_prince has joined #openstack-ironic09:58
*** yuanying has joined #openstack-ironic10:33
*** dividehex has quit IRC10:34
*** bradjones has joined #openstack-ironic10:40
*** mkerrin has joined #openstack-ironic10:46
openstackgerritGhe Rivero proposed a change to openstack/python-ironicclient: Add keystone v3 CLI support  https://review.openstack.org/11117510:47
*** harlowja_away has quit IRC10:47
*** nikunj2512 has quit IRC10:48
*** dividehex has joined #openstack-ironic10:49
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: WIP: DRAC introspect hardware  https://review.openstack.org/12625710:55
*** rameshg87 has quit IRC10:59
*** ramineni has quit IRC10:59
*** romcheg has quit IRC10:59
*** shakamunyi has joined #openstack-ironic11:03
*** yuanying has quit IRC11:06
*** lucasagomes is now known as lucas-afk11:11
*** shakamunyi has quit IRC11:12
*** ifarkas has quit IRC11:17
*** coolsvap is now known as coolsvap|afk11:23
*** romcheg has joined #openstack-ironic11:24
*** vinbs has quit IRC11:27
*** ifarkas has joined #openstack-ironic11:28
openstackgerritRoman Prykhodchenko proposed a change to openstack/ironic: Refactoring for Ironic policy  https://review.openstack.org/12626511:30
*** nosnos has quit IRC11:34
romchegGood morning/evening Ironic!11:37
romcheg^ is a preview. When I come home from work I'll fix it and write more tests.11:38
*** krtaylor_away is now known as krtaylor11:40
dtantsurromcheg, morning, good job!11:44
romchegdtantsur: I like removing code :)11:54
romchegdtantsur: *if after all it's still doing the same job :)11:54
dtantsur:)11:54
*** Poornima_ has quit IRC11:59
*** pensu has quit IRC12:02
*** yuanying has joined #openstack-ironic12:03
*** dprince has joined #openstack-ironic12:16
*** shakamunyi has joined #openstack-ironic12:26
*** lazy_prince has quit IRC12:33
*** killer_prince is now known as lazy_prince12:33
*** yuanying has quit IRC12:36
*** shakamunyi has quit IRC12:40
*** Mike_D_wk has left #openstack-ironic12:40
*** ifarkas has quit IRC12:41
*** ifarkas has joined #openstack-ironic12:46
*** rloo has joined #openstack-ironic13:09
romchegdtantsur: Do you want to go to PyCon PL? :)13:11
romchegdtantsur: we could meet and drink some beer there.13:12
romchegdtantsur: … I mean… discuss openstack13:12
romcheghttp://pl.pycon.org/2014/en/13:12
dtantsurromcheg, 16-19 Oct? oh that's bad, I already have plans for these dates...13:15
romcheg:(13:17
*** mikedillion has joined #openstack-ironic13:26
*** yuanying has joined #openstack-ironic13:33
*** todd_dsm has joined #openstack-ironic13:34
*** shakamunyi has joined #openstack-ironic13:37
viktorsromcheg: as for pycon - do you suppose to visit this - http://ua.pycon.org/ ?13:41
viktorsor maybe someone else from this chat will be there?13:42
*** lucas-afk is now known as lucasagomes13:42
*** dhellmann is now known as dhellmann_13:43
*** dhellmann_ is now known as dhellmann13:44
dtantsurPyCon in Lviv, mmm.... :) pity I can't be there13:44
romchegdtantsur: You can ;)13:44
viktorsdtantsur: why not?13:45
*** r-daneel has joined #openstack-ironic13:45
dtantsur1. the summit; 2. it's not the best time to appear in Ukraine with Russian passport :)13:46
romchegdtantsur: I previously called border control and they said that if you fly from Czech Republic with a ticket to PyCon and an accommodation booking it won't be a problem13:46
romchegWe have a bunch of folks from Moscow in Kharkiv office right now :)13:46
viktorsdtantsur: well, we supposed to go to summit directly from Lviv13:46
*** kbyrne has joined #openstack-ironic13:47
dtantsurmy air tickets are already booked :(13:47
viktorsdtantsur: oh :(13:47
romchegdtantsur: November is not a good time for visiting Lviv I suppose but we can go there if the weather is fine.13:52
romchegdtantsur: No one eats you because of your Russian passport, it's only on Russian TV :)13:52
*** shakamunyi has quit IRC13:53
viktorsdtantsur: confirm :)13:53
dtantsurwell, I'm not judging by Russian TV, rather by words of my wife's grandparents that live in Lviv13:53
* dtantsur hasn't watched Russian TV for years now13:54
*** igordcard has joined #openstack-ironic13:55
romchegdtantsur: You could have some problems with border control a few months ago when a lot of pro-Russian bandits were coming through border control, now it's almost not the case13:55
dtantsurwell, good to hear, will see then :)13:55
romchegUnless you carry a mobile SAM :)13:55
romchegNow those guys have part of south-eastern border to cross, they don't need border control anymore :)13:56
*** romcheg has left #openstack-ironic13:57
*** romcheg has joined #openstack-ironic13:57
*** rameshg87 has joined #openstack-ironic14:01
*** yuanying has quit IRC14:06
*** shakamunyi has joined #openstack-ironic14:07
*** pensu has joined #openstack-ironic14:13
*** sambetts has joined #openstack-ironic14:15
*** rameshg87_ has joined #openstack-ironic14:15
*** rameshg87 has quit IRC14:16
*** rameshg87_ has quit IRC14:18
*** shakamunyi has quit IRC14:21
NobodyCamGood Morning Ironic14:27
romchegMorning NobodyCam!14:27
sambettsAfternoon NobodyCam!14:27
NobodyCammorning romcheg sambetts :)14:28
*** romcheg has left #openstack-ironic14:28
NobodyCamhow was your weekend?14:28
*** jistr has quit IRC14:28
lucasagomesNobodyCam, morning14:29
NobodyCammorning lucasagomes :)14:29
*** jistr has joined #openstack-ironic14:30
*** jcoufal has quit IRC14:31
GheRiveromorning all14:32
*** pensu has quit IRC14:32
NobodyCammorning GheRivero :)14:32
*** jcoufal has joined #openstack-ironic14:34
lucasagomesGheRivero, morning14:35
*** coolsvap|afk is now known as coolsvap14:41
*** linggao has joined #openstack-ironic14:41
*** pcrews has joined #openstack-ironic14:45
*** pensu has joined #openstack-ironic14:50
NobodyCamanyone have any items for the agenda14:55
*** viktors is now known as viktors|afk14:56
lucasagomesNobodyCam, I may have one based on the last comment by devananda on #9942614:58
lucasagomeshaven't added to the agenda yet tho14:58
*** coolsvap is now known as coolsvap|afk14:58
*** pensu has quit IRC15:00
NobodyCam:)15:01
*** jistr has quit IRC15:02
*** jistr has joined #openstack-ironic15:02
*** yuanying has joined #openstack-ironic15:03
devanandamorning, all15:14
lucasagomesdevananda, morning15:14
NobodyCamgood morning devananda15:15
lucasagomesdevananda, have a min? I was thinking about start https://blueprints.launchpad.net/ironic/+spec/new-boot-interface15:15
lucasagomesdo I need to write a simple spec for it first? or I can go ahead and write the full one?15:15
*** pensu has joined #openstack-ironic15:16
lucasagomesI don't think people would oppose to the idea of splitting the deploy interface into boot and deploy interfaces15:16
lucasagomesand it's needed to pave the work to the ipxe driver, so I wanna tackle it asap15:16
devanandalucasagomes: I think full is fine. I dont think anyone's objected to that refactoring15:16
devanandaand yea, sooner is good15:16
lucasagomesdevananda, ack, cool yeah I will write the spec and try to put some code asap for that15:17
lucasagomescheers15:17
*** pensu has quit IRC15:20
*** pensu has joined #openstack-ironic15:35
*** igordcard has quit IRC15:36
JayFdevananda: curious if we want to ask ttx to move one of our design summit sections... IDK about jroll's talk, but I know one of our design summit slots is right over the top of mine and JoshNang's talk on Decom, which I'm assuming will be very popular among ironic folk15:36
*** yuanying has quit IRC15:37
*** jcoufal has quit IRC15:43
lucasagomesdevananda, http://paste.openstack.org/show/119021/ re ironic send sensor data bug15:44
devanandaJayF: yes15:52
devanandaJayF: can you send me links to those talk slots?15:52
NobodyCambrb15:53
devanandalucasagomes: oh, I see. thanks15:54
lucasagomesdevananda, np15:54
*** ifarkas has quit IRC15:59
*** romcheg has joined #openstack-ironic16:00
*** jistr has quit IRC16:02
devanandaJayF: I only see one of yours at 11am Tuesday. lemme know if there are others and I'll coordinate with ttx16:06
*** coolsvap|afk is now known as coolsvap16:06
devanandanm, found it16:06
romchegMorning devananda!16:06
NobodyCammorning romcheg :)16:07
devanandaromcheg: hi! o/16:07
romchegNobodyCam: Morning 2nd time :-P16:07
openstackgerritRoman Prykhodchenko proposed a change to openstack/ironic: Refactoring for Ironic policy  https://review.openstack.org/12626516:07
romchegSo as I promised ^ finished that to make the tests pass :)16:08
*** spandhe has joined #openstack-ironic16:11
lucasagomesromcheg, w00t, will take a look soon thanks16:12
romcheglucasagomes: there's one place I think should be done thread safe there, but I'd like to see your comments on that16:14
lucasagomescool16:14
lucasagomesI will take a look when I get some time here16:14
*** romcheg has quit IRC16:20
devanandaJayF: how do you feel about moving your talk back a few hours?16:21
NobodyCam_LE is LOG.execption or LOG.error?16:28
rlooNobodyCam: I have wondered. I have only see _LE described for ERROR, but I haven't seen anything for exceptions: http://docs.openstack.org/developer/oslo.i18n/guidelines.html#examples16:31
rlooNobodyCam: and I've seen folks use _LE for exceptions.16:31
rlooNobodyCam: if you scroll down that page, you'll see an example with _LE for exceptions: LOG.exception(_LE('There was an error.'))16:33
*** yuanying has joined #openstack-ironic16:33
*** marcoemorais has joined #openstack-ironic16:36
NobodyCamrloo: ya16:37
NobodyCamok was looking at lines 510 & 520 of https://review.openstack.org/#/c/126265/2/ironic/openstack/common/policy.py16:37
*** dlaube has joined #openstack-ironic16:37
dlaubeg'morning16:38
NobodyCamgood morning dlaube16:38
dtantsurfolks, maybe you know, is eventlet _required_ for anything using our client?16:39
dtantsur(and morning/afternoon to everyone I didn't notice)16:39
*** rushiagr is now known as rushiagr_away16:39
openstackgerritA change was merged to openstack/python-ironicclient: Stop using intersphinx  https://review.openstack.org/12615616:40
devanandadtantsur: i dont see why the client should need eventlet16:41
devanandarloo: that sounds like we're misusing _LE16:41
dtantsurhmm... ack thanks16:41
rloodevananda: misusing in what way?16:41
devanandahttp://docs.openstack.org/developer/oslo.i18n/guidelines.html16:42
devananda_LE = ERROR16:42
devanandaoh16:42
rloodevananda: yes, but they don't show anything for exceptions, and then they have an example that uses _LE for exceptions.16:42
devanandarloo: ok, never mind. the doc uses both ...16:42
rloodevananda: I suppose we should ask whoever to update/clarify that page.16:43
devanandarloo: or we follow what the page indicates and use it for both16:44
rloodevananda: if the intent is to use for both, I think it would help if that page explicitly showed that at the top.16:44
*** rwsu has joined #openstack-ironic16:44
dtantsurdevananda, also: any ideas on whether get_client (not client itself) is thread-safe?16:46
devanandadtantsur: not sure off hand16:49
dtantsurah ok..16:50
JayFdevananda: I don't personally have a preference, have to check with JoshNang too16:51
JayFdevananda: also jroll has a talk but idk when that's scheduled16:51
devanandaJayF: jroll's is tuesday, no confilct with our design sessions16:52
JayFI am very OK with moving the decom talk if that helps, as long as JoshNang is on board16:53
JoshNangJayF: devananda no complains here16:53
JoshNang*complaints16:53
devanandacool16:53
*** athomas has quit IRC16:55
*** spandhe has quit IRC16:59
*** marcoemorais has quit IRC17:01
*** marcoemorais has joined #openstack-ironic17:02
*** marcoemorais has quit IRC17:02
*** marcoemorais has joined #openstack-ironic17:02
*** spandhe has joined #openstack-ironic17:03
*** derekh has quit IRC17:05
*** yuanying has quit IRC17:07
*** rushiagr_away is now known as rushiagr17:17
*** romcheg has joined #openstack-ironic17:24
*** ryanpetrello_ has joined #openstack-ironic17:24
*** pensu has quit IRC17:24
*** pensu has joined #openstack-ironic17:25
*** coolsvap is now known as coolsvap|afk17:27
*** ryanpetrello_ has quit IRC17:27
*** praneshp has joined #openstack-ironic17:30
*** harlowja has joined #openstack-ironic17:34
* dtantsur afk till the meeting17:35
*** marcoemorais has quit IRC17:37
*** marcoemorais has joined #openstack-ironic17:38
*** sambetts has quit IRC17:39
dlaubeHey guys, can anyone point me in the right direction as to why my ironic calls are failing with "Invalid OpenStack Identity credentials"?  When I run with debug, it appears that keystore auth is working properly now (I think) since I'm receiving a token. Looks like when the token is provided to the ironic API, it throws an HTTP 401.17:41
dlaubemore details at http://pastie.org/private/fa3fsj2fgiatusqlpgz1mw17:41
jrolldlaube: check the roles for that keystone user17:43
dlaubechecking now17:44
*** romcheg has quit IRC17:44
* JayF WTB log messages that when authorization failed, says "you have roles x,y,z, you need role banana"17:44
dlaubehehehe17:44
dlaubehttp://pastie.org/private/vhbyf0qqcqsxtmkcr4j1oq17:44
devanandaJayF: thanks, now I want a banana17:45
dlaubewill confirm admin and tenant vars in ironic.conf17:45
jrolldevananda: aren't you glad bananas are easy to acquire?17:45
dlaubeI have   admin_user=ironic    and admin_tenant_name=service    in my ironic.conf17:45
jrolldlaube: also check policy.json, but it should have role:admin17:45
JayFdevananda: ever been to France? Did some research and found something that basically said France is the worst Euro nation to travel to if you can't eat gluten17:45
* jroll throws a baguette at JayF17:46
dlaubejroll: policy.json inside my ironic dir, or the keystone dir?17:46
Shrewsgluten-free baguette?17:46
JayFThey exist, and I had a hoagie on one this weekend \o/17:46
jrolldlaube: whatever ironic is using, there's a path to it in the ironic conf afaik17:46
jrollhuh.17:46
jrollseems wrong.17:46
jrolllike, sacrelig17:47
JayFjroll: http://udisglutenfree.com/products/french-baguettes/17:47
jrollso so so wrong17:47
JayFjroll: in the freezer section of your local overpriced organic grocer17:47
jrollfrozen baguettes17:47
jrollagain, so much wrong here17:47
jrollif it's older than 2 hours, it doesn't exist to me17:48
dlaubeI bet this is a problem… my policy.json  ->  http://pastie.org/private/vzib3rbjanypfcfvx4kbra17:48
jrolldlaube: that looks sane, role:admin should match just fine17:48
jrollI now have no idea :)17:49
jrollalthough!17:49
jrollmaybe make sure ironic is auth'ing to keystone properly17:49
jrollsorry, I think that's the key17:49
jrollcheck out ironic-api log17:49
jrolls17:49
* jroll should have known that17:49
*** romcheg has joined #openstack-ironic17:51
devanandaJayF: you're correct17:51
devanandaJayF: we're going to die. painfully.17:51
devanandaJayF: also, they have AMAZING wine, so at least we have one gluten free source of calories.17:51
jrollhahahaha17:51
JayFdevananda: I forsee lots of "steak with no sauce whatsoever because you certainly started it with a roux" orders17:51
dlaubejroll: good call! I didnt think to look there since I saw the request being made to the ironic api in general17:52
devanandawell... i may be exaggerating in some way ...17:52
dlaubelooks like its trying to use SSL even though I've specified plain http17:52
dlaubehttp://pastie.org/private/bnmhxugiogucpmjwjuat1w17:52
devanandaJayF: even soup will be a challenge. and forget pre-shredded cheese on your salad, bc there is often flour as an anti-caking agent.17:52
*** praneshp has quit IRC17:53
dlaubemy ironic.conf has   auth_uri=http://198.x.x.202:5000/     but perhaps I need to set #auth_protocol=http ?17:54
jrolldlaube: weird, I think auth_uri is maybe deprecated?17:54
JayFdevananda: I'm dairy free nowadays too, so at least that's not a total loss17:54
dlaubehmm..17:54
jrollno, auth_uri is the new thing17:55
jrollor identity_uri maybe17:55
dlaubejroll:  i'll play with #insecure=false    and #auth_protocol=https17:55
jrollhard to tell :/17:55
jrolldlaube: if your keystone server doesn't serve with ssl, that won't work :P17:55
dlaubeim going to try auth_protocol=http17:56
jrolldlaube: I vote use identity_uri=http://198.x.x.202:5000/17:56
dlaubeoh ok, I'll try this first —^17:56
jrolllooking at https://github.com/openstack/keystonemiddleware/blob/master/keystonemiddleware/auth_token.py#L60617:56
devanandaJayF: hm. butter is also in everything.17:57
JayFdevananda: butter isn't dairy, it's golden deliciousness17:57
JayFdevananda: (I eat butter, cream, just not milk and soft cheese)17:57
JayFand it's more of a I-feel-better-if thing, not with gluten where if I eat it I would have prefered eating a hot coal17:58
NobodyCamdevananda: just looking at the agenda? do you see any reason we should add DRAC to the drivers report area?17:59
dlaubejroll: looks like I needed to set auth_protocol=http  as identity_uri did not work for me …but I'm currently playing around with ironic 0.1.218:01
jrolldlaube: weird... you mean ironic client 0.1.2 yes?18:01
dlaubeyessir.  have the new RCs dropped for the latest juno awesomeness or should I consider forking from master as I really start to get into ironic?18:02
jrollyeah, they're out18:02
jrollI mean, I believe in continuous deployment, but if releases are your thing, :)18:02
*** marcoemorais has quit IRC18:03
*** marcoemorais has joined #openstack-ironic18:03
*** yuanying has joined #openstack-ironic18:03
*** marcoemorais has quit IRC18:04
*** marcoemorais has joined #openstack-ironic18:04
*** marcoemorais has quit IRC18:05
dlaubesweet18:05
dlaubethanks jroll18:05
*** marcoemorais has joined #openstack-ironic18:06
NobodyCambrb18:08
*** praneshp has joined #openstack-ironic18:14
*** marcoemorais has quit IRC18:17
*** rushiagr is now known as rushiagr_away18:17
NobodyCamadam_g: still want https://review.openstack.org/#/c/118700 on the agenda.. it landed a while ago.18:21
*** rushiagr_away is now known as rushiagr18:21
adam_gNobodyCam, nah18:21
adam_gwonder how that got carried over18:21
* NobodyCam clears it out18:22
NobodyCamlucasagomes: did you want me to add #99426 to the agenda?18:24
*** chuckC_ has quit IRC18:27
*** mikedillion has quit IRC18:29
* NobodyCam added it to Open Discussion 18:30
*** mikedillion has joined #openstack-ironic18:30
*** rushiagr is now known as rushiagr_away18:31
*** marcoemorais has joined #openstack-ironic18:33
* NobodyCam grabs quick coffee and runs out for smoke befor meeting18:34
*** ChuckC has quit IRC18:35
*** yuanying has quit IRC18:38
*** dprince has quit IRC18:38
* devananda breaks before meeting18:40
NobodyCamlast chance to get your item on the agenda!18:44
*** pelix has quit IRC18:45
lucasagomesNobodyCam, oh sorry... hmm well I may leave it for the next because this week I will focus on the boot interface18:48
*** jgrimm has joined #openstack-ironic18:49
NobodyCamlucasagomes: I add 99426 to Open Discussion18:51
lucasagomesNobodyCam, cheers18:52
NobodyCam:-p18:54
*** BertieFulton has joined #openstack-ironic18:57
* devananda heads over to the meeting room18:58
*** ChuckC has joined #openstack-ironic19:04
*** praneshp has quit IRC19:04
*** marcoemorais has quit IRC19:05
*** marcoemorais has joined #openstack-ironic19:06
*** marcoemorais has quit IRC19:06
*** marcoemorais has joined #openstack-ironic19:06
*** marcoemorais has quit IRC19:07
*** marcoemorais has joined #openstack-ironic19:07
*** praneshp has joined #openstack-ironic19:07
*** praneshp has quit IRC19:07
*** chuckC_ has joined #openstack-ironic19:07
*** ChuckC has quit IRC19:10
*** Haomeng has joined #openstack-ironic19:13
*** Haomeng|2 has quit IRC19:14
*** ChuckC has joined #openstack-ironic19:19
*** praneshp has joined #openstack-ironic19:27
*** yuanying has joined #openstack-ironic19:33
*** ryanpetrello_ has joined #openstack-ironic19:35
*** praneshp has quit IRC19:35
*** ryanpetrello_ has quit IRC19:36
*** marcoemorais has quit IRC19:41
*** praneshp has joined #openstack-ironic19:59
NobodyCamthank you all for a great meeting20:00
lucasagomesdevananda, jroll oh yeah, so we need to convert in the nova driver too (missed that bit)20:00
devanandaso yea, changing unit sizes is going to break tripleo and any existing deployments20:00
dtantsurnow to get some sleep! g'night, see you tomorrow20:01
devanandabecause it's not changing what Nova writes to Ironic -- it's changing how Nova interprets what Ironic exposes20:01
*** dtantsur is now known as dtantsur|afk20:01
jrollright, the backwards compat part is hard20:01
devanandaand we don't haev a means in Ironic to indicate the unit size for that property20:01
devanandadtantsur|afk: g'night!20:01
jrollbut like... there's probably a way to do it20:01
lucasagomesyeah worthy taking a look20:01
*** BertieFulton has quit IRC20:01
jrollthere's going to be migrations and whatnot involved anyway, deploying this is going to have an impact already20:01
devanandait's a text field in Ironic and an INT in Nova20:02
jrollmmm20:02
jrolloh because json20:02
devanandaright20:02
*** praneshp has quit IRC20:02
jrollso... we'll have to write an upgrade script to munge those into things containing units?20:02
devanandanova statically assumes the unit sizes of cpu/ram/disk/swap/ephemeral -- and they're not all the same20:02
*** ryanpetrello_ has joined #openstack-ironic20:02
jrollright20:02
devanandaif we munge the data in Ironic, it breaks old deployments of Nova20:03
devanandaso ironic needs to expose data in a way that old-nova still understands while allowing new-nova to understand something new20:03
jrollright, I don't see an easy path either way20:03
devanandaright20:03
lucasagomeshmm yeah20:03
devanandaif this were moved out of JSON and into a separate table in Ironic, we could handle it more intelligently20:04
jrollso let's do the right thing (IMO, use all MBs) and figure out the upgrade path20:04
JayFWhy is picking MB as the unit the right thing?20:04
devanandastore once, expose it both as "local_gb:INT" and "disk: ${size}${unit}"20:04
jrollJayF: because it's better than text fields with units or using floats (again, imho)20:04
devanandaold nova would ignore the "disk" property20:04
devanandanew nova would ignore the "local_gb" property20:04
devanandaAPI supports writing to either one20:05
devanandait's terrible20:05
*** praneshp has joined #openstack-ironic20:05
JayFjroll: I vote bytes, in that case20:05
devanandabut that would be backwards compat20:05
*** marcoemorais has joined #openstack-ironic20:05
lucasagomesdevananda, yeah looks pretty bad indeed20:05
devanandaalso - this is what versioned APIs would give us20:05
lucasagomesduplicated info20:05
devanandalucasagomes: duplciated API properties, but not in our DB20:05
jrolldevananda: right... we could do the same with converting everything to (mega)bytes then, no?20:06
devanandawe could abstract that within the client lib, but without a versioned API, its still a pain20:06
devanandajroll: how would old-nova continue to function though?20:06
*** praneshp has quit IRC20:06
devanandajroll: if we simply convert the data stored in Ironic to a new base unit, it breaks existing clouds immediately, sicne the flavors wouldn't match the properties20:07
jrollstore once, expose it both as "local_gb:${size}*1024" and "disk: ${size}"20:07
jrollerrr, /1-2420:07
jrollwow20:07
jrollcan't type20:07
lucasagomeshah20:07
jroll/1024, you get the idea20:07
*** yuanying has quit IRC20:07
devanandajroll: I'm saying, how do we avoid requiring operators to perform a coordinated upgrade of both ironic and nova at the same time20:07
*** praneshp has joined #openstack-ironic20:07
jrollit doesn't require that20:08
devananda?20:08
jrollif we continue to expose local_gb as gigabytes20:08
jrollold nova will be fine20:08
jrollstore once, expose it both as "local_gb:${size}/1024" and "disk: ${size}", where size is in MB20:08
devanandaright, that's what I suggested :)20:09
*** ryanpetrello_ has quit IRC20:09
JayFI think the key diff b/w what jroll and devananda suggested20:09
JayFis that devananda wants to still specify units20:09
lucasagomesjroll, so user would input properties/disk=1024MB and we will expose local_gb=1 in the properties automagically?20:09
jrolldevananda: right, you suggested it with a unit string20:09
JayFjroll: wants to sticky units to (mega)bytes20:09
devanandaah20:09
lucasagomesJayF, yeah I think ur correct20:09
jrolllucasagomes: yes, for backwards compat20:09
lucasagomesgotcha20:09
* devananda needs to step afk for a few hours and take car into shop20:09
jrolllike, using units will be fine with standard tools20:10
NobodyCam:)20:10
jrollbut anyone that talks to ironic with an alternate client will need to handle this20:10
devanandaso I think either of those would work, but both are icky from an API perspective20:10
jrolland it's just... annoying20:10
devanandabbiafh20:10
lucasagomesyeah, well as properties is a json blob we would need to start intropescting into it to find out if the user had input "disk"... but that's ok I think20:10
jrolllike disk size as string is annoying to me20:10
lucasagomesright yeah... well it's a good discussion20:11
jrollindeed :)20:11
lucasagomesmaybe we should continue on the bug there20:11
lucasagomesI also will go afk now :(20:11
jrolland I can fix my tools to work around it, if we do use units :P20:11
lucasagomesor tomorrow we can talk more20:11
jrollsure20:11
* lucasagomes is hungry and it's late here20:11
NobodyCamhave a good night lucasagomes20:11
jrollhave a good night :)20:11
lucasagomesjroll, NobodyCam thanks!20:11
lucasagomesyeah let's talk more tomorrow about it then!20:11
lucasagomeshave a good afternoon there20:12
lucasagomesenjoy the day20:12
*** lucasagomes is now known as lucas-dinner20:12
NobodyCam:) brb too20:12
*** jmank has joined #openstack-ironic20:17
*** Haomeng|2 has joined #openstack-ironic20:24
*** Haomeng has quit IRC20:24
* NobodyCam is back20:25
*** linggao has quit IRC20:46
*** pensu has quit IRC21:01
*** marcoemorais has quit IRC21:03
*** marcoemorais has joined #openstack-ironic21:03
*** yuanying has joined #openstack-ironic21:03
NobodyCambrb again21:11
NobodyCam:-p21:20
*** Haomeng|2 has quit IRC21:23
NobodyCamrloo: are you around?21:24
*** todd_dsm has quit IRC21:24
rlooNobodyCam: some oslo person agrees about using _LE() for exceptions: https://bugs.launchpad.net/bugs/137805321:24
rlooNobodyCam: yeah, I'm around ;)21:24
NobodyCamnice :)21:25
*** Haomeng|2 has joined #openstack-ironic21:25
NobodyCamrloo: got a free minute to take a look at https://review.openstack.org/#/c/12601921:25
dhellmannrloo: actually, exceptions should be _() if they have any chance to showing up in front of a user21:25
rloodhellmann: yes, that's right. but for exception strings that are only logged -- _LE()?21:26
dhellmannrloo: is it worth figuring out whether an exception never makes it to a user?21:26
dhellmannrloo: I mean, you could, but it seems simpler to just say "use _() for exceptions" and use your brain power for other things :-)21:27
rloodhellmann: so to minimize brain power, I just looked at the doc. And it doesn't explicitly mention what to do for exceptions, except in the case you mentioned. and then there's an example where _LE() is used for log.exception. So... my brain is a bit confused.21:28
rloodhellmann: the thing is, people are writing code using _LE() for exceptions.21:29
dhellmannrloo: ok, let me look at the docs and see how we can improve them21:29
rloodhellmann: so I figured I should open a bug about it and hopefully get some answer, moving forward... ;)21:29
rloodhellmann: it came up today cuz NobodyCam asked ;)21:30
dhellmannrloo: first example on http://docs.openstack.org/developer/oslo.i18n/guidelines.html#examples shows _LE() for exceptions as wrong, is that where you were looking?21:30
dhellmannah, no, 2nd example21:30
dhellmannthat's not consistent21:30
dhellmannhrm, well, sort of21:30
NobodyCamdhellmann: this spawned from lines 510 & 520 of https://review.openstack.org/#/c/126265/2/ironic/openstack/common/policy.py21:31
dhellmannthe exception itself is using _() in that 2nd case, but the error associated with logging the exception is _LE21:31
dhellmannNobodyCam: looking21:31
dhellmannNobodyCam: right, those are consistent with the 2nd example from the page linked above21:31
rloodhellmann: so use _LE() for log.exceptions?21:32
rloodhellmann: except for blah blah21:32
dhellmannrloo: use _() inside exception classes or when passing a string to an exception class and _LE for messages sent directly to the log21:32
dhellmannmaybe there's a clearer way to say that21:32
rlooheh. thx dhellmann!21:33
NobodyCam:)21:33
dhellmannif that's clear, I'll toss that text into the bug and we'll see if we can get the docs updated21:33
NobodyCamdhellmann: this helps alot: use _() inside exception classes or when passing a string to an exception class and _LE for messages sent directly to the log21:34
*** todd_dsm has joined #openstack-ironic21:34
NobodyCam+++ to adding to the docs21:34
dhellmannNobodyCam: ok, good, we'll add that to the docs21:34
rloodhellmann: I'm not quite sure that captures the use _() for logging exceptions if that string is also going to be used in an exception class.21:35
rloomaybe '_LE for messages sent ONLY to the log'?21:36
dhellmannrloo: LOG.exception() doesn't create an exception, and the message is clearly only going to go to the log handler there. If you raise a FooException that takes a string, you don't know where that string might end up.21:36
dhellmannyeah21:36
dhellmannugh, launchpad timeouts21:36
*** yuanying has quit IRC21:37
*** achanda has joined #openstack-ironic21:38
rlooNobodyCam: I'll try to take a look at 126019, but I only have +1 powers there.21:38
dhellmannrloo, NobodyCam : https://review.openstack.org/#/c/126419/21:41
* NobodyCam checks the link21:42
*** jgrimm is now known as zz_jgrimm21:42
rloothx dhellmann21:43
NobodyCamrloo: you don't spec +2?21:44
rlooNobodyCam: nope. Don't want too many powers ;)21:44
NobodyCamlol :) ++++21:44
NobodyCam*cough* hey jroll :)- how are you doing ???!!!21:45
jrolluh hi?21:45
rlooha ha21:45
JayFNobodyCam: that lgtm21:45
jrolldon't make me do work21:45
JayFNobodyCam: need me to land it?21:45
jrollsee, snipe JayF21:45
NobodyCamhehhehe21:45
JayFstop bugging jroll21:45
JayFbug other people too21:45
NobodyCamJayF: sure21:45
JayFI want exclusive access to bug jroll21:45
JayF:P21:45
NobodyCamI can do that21:45
NobodyCam*cough* hey JayF :)- how are you doing ???!!!21:45
jrollwait, what are we looking at?21:45
NobodyCamlol21:45
NobodyCam12601921:46
jrollaha, yeah21:46
* jroll lets y'all do that21:46
openstackgerritA change was merged to openstack/ironic-specs: Automate testing of template titles  https://review.openstack.org/12601921:48
* devananda is back21:48
*** ryanpetrello_ has joined #openstack-ironic21:50
NobodyCamWB DavidHu__21:55
NobodyCamgah21:55
NobodyCamwb devananda21:55
NobodyCam:-p21:55
rlooNobodyCam: just wondering, wrt 126019. It doesn't allow a spec to have a section that isn't in the template. We're ok with that?21:58
rlooNobodyCam: I don't think the original test did that.21:59
rlooNobodyCam: I see that there was a new test added too; not mentioned in the commit.21:59
DavidHu__Need an python-ironicclient core to help move https://review.openstack.org/#/c/111175/ forward.21:59
devanandarloo: correct. a spec shouldn't just invent new sections22:00
NobodyCamDavidHu__: sorry about the mis ping :)22:00
NobodyCamrloo: I am okay with it at this time. if we run into a issue with it we refactor easily22:00
rloodevananda: ok. I was thinking that maybe some bright person would think if a new/useful section. but we can handle that if it happens.22:01
devanandaDavidHu__: Ironic does not support resource association to  user/tenant/project at this time22:01
devanandaDavidHu__: so this patch doesn't make sense, in that context22:01
DavidHu__Sorry, it was unrelated to what you guys were discussing.22:02
devanandaDavidHu__: my context has switched. I'm now looking at your patch. and I dont understand why its adding those fields to the client at this time22:03
devanandaDavidHu__: I'm probably missing something obvious, though22:03
*** lucas-dinner has quit IRC22:07
DavidHu__The fields for for keystone v3 support through the CLI.22:09
*** ryanpetrello_ has quit IRC22:11
NobodyCamDavidHu__: just looking quickly should lines 294 and 300 also be looking at os-tenant-id in https://review.openstack.org/#/c/111175/7/ironicclient/shell.py22:12
devanandaDavidHu__: what does it mean to authenticate with keystone v3 for Ironic for "project X" vs "project Y" when Ironic itself doesn't care about projects?22:13
*** lucas-dinner has joined #openstack-ironic22:19
DavidHu__devananda:  Keystone v3 supports the concept of domain.  From authentication perspective, ironic client simply gathers the domain id or name then pass all that info to keystone for authentication.22:24
DavidHu__Projects and tenants are now used interchangeably.22:25
jrollDavidHu__: what does a domain do?22:31
devanandaDavidHu__: sure. and that'll work. but what does it mean to a user of Ironic?22:31
*** yuanying has joined #openstack-ironic22:33
DavidHu__devananda:  The patch is backward compatible with Keystone v2.0 API.  If a user is added to a domain other then the default domain, that user will need to specify thec domain name that he/she belong to.22:39
devanandaDavidHu__: I think we're talking past each other still22:40
devanandaDavidHu__: Ironic does not associate resources today. A user is either auth'd with the service (they see everything) or not auth'd (they see nothing)22:41
*** lucas-dinner has quit IRC22:42
DavidHu__Here is a good wiki on Keystone domains, https://wiki.openstack.org/wiki/Domains22:43
*** todd_dsm has quit IRC22:44
*** romcheg has quit IRC22:45
*** romcheg has joined #openstack-ironic22:45
*** romcheg has quit IRC22:45
jrollthis wiki page makes it feel like domains are still being discussed in keystone22:46
jrollthough also makes it sound like this could be useful in the future22:46
jrollmaybe22:47
jrollit's unclear22:47
jrollbut like... folks could say "everyone in this domain has access to ironic22:47
jroll"22:47
jrolldomain_id:ironic_ops or something22:47
jrollI tend to think our admin or gtfo approach is kind of meh22:48
devanandajroll: I agree. it was an initial good-enough solution22:48
jrollcompletely agree with that22:48
DavidHu__Domain support is there since Grizzly :)22:48
devanandajroll: looking forward in Kilo, I think I'dlike to see two axis for permissions in Ironic22:49
jrolland you can somewhat limit it today... it's still all or nothing but you can limit it to certain folks22:49
jrollbut saying it's admin-only isn't really true, since you can change policy.json to allow all users or whatever22:49
devanandajroll: by type of information (eg, hide driver_info from some folks) and actually, associating nodes<->projects would be helpful22:49
jrollhmm22:50
jrollI still don't think I want my end users poking directly at the ironic api :)22:50
jrollthough if you aren't using nova, might be more useful22:50
devanandajroll: you could have DC ops taht need to trouble shoot the hardware but shouldn't get the IPMI password22:51
*** rwsu has quit IRC22:51
devanandajroll: so we expose maintenance, power, console states to them22:51
jrollyeah, completely22:51
jrollI meant for the node <-> project thing22:51
devanandaah22:51
devanandano, that I want for static assignments22:51
jrollaha22:51
devananda"these 100 machnes belong to tenant A"22:52
jrollyep22:52
jrollthere could be very interesting cases for that :)22:52
devanandayup22:52
jrollanyhow, this keystone domain thing isn't useful today22:53
jrollbut I think it will be useful at some point22:53
devanandayea22:53
*** lucas-dinner has joined #openstack-ironic22:54
DavidHu__The patch has everything you need for keystone v3 CLI support :)22:55
DavidHu__Ironicclient is one of a remaining clients that does not authenticate domain user.  V2 will be deprecated soon.23:02
*** praneshp has quit IRC23:02
*** rwsu has joined #openstack-ironic23:04
*** yuanying has quit IRC23:07
*** achanda has quit IRC23:07
*** praneshp has joined #openstack-ironic23:08
*** yuanying has joined #openstack-ironic23:10
*** bigjools_ is now known as bigjools23:11
*** bigjools has quit IRC23:11
*** bigjools has joined #openstack-ironic23:11
*** mikedillion has quit IRC23:11
*** lucas-dinner has quit IRC23:15
*** achanda has joined #openstack-ironic23:19
NobodyCam^[j|J] does IPA validate that the image it is getting from glance is a whole disk image, or is it assumed just to be so?23:23
jrollhmm, I don't think so23:25
JayFheh23:25
JayFwe don't even validate it's a qcow23:25
JayFit's fed to qemu-img convert and if it can be converted to raw it's happily written to the disk23:25
jrollalso, I refuse to put that regex on highlight, you can just ping like normal :P23:25
JayFonly validation done is that the hash matches what ironic said it would23:26
NobodyCamack was just courious.23:26
NobodyCamjroll: please dont ever add that to your highlight. The times I have used that is only for those that are looking at the channel23:27
jroll:P23:27
NobodyCamI wouldn't want that to "PING" you at all23:27
* JayF only highlights on ^JayF23:27
NobodyCam;-p23:27
jrollhey, you pinged saturday btw23:27
jrolldid you get it sorted?23:27
jrollNobodyCam: ^23:28
JayFand that goes to my irc client, not my phone, my email, my receptionist, none of that crazy stuff some poeple cook up23:28
NobodyCamI had question about the config drive spec23:28
jrollah23:28
NobodyCamthen saw your comment about it becoming a uuid23:28
JoshNangJayF: you have a receptionist?23:28
JayFJoshNang: "ok google"23:28
JoshNangheh23:28
*** chuckC_ has quit IRC23:37
*** ChuckC has quit IRC23:45
* NobodyCam steps away23:49
*** achanda has quit IRC23:55

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