Monday, 2014-07-28

*** pcrews has joined #openstack-ironic01:04
*** nosnos has joined #openstack-ironic01:43
*** pcrews has quit IRC01:50
*** Haomeng|2 has quit IRC02:46
*** pquerna has quit IRC02:47
*** pquerna has joined #openstack-ironic02:48
*** Haomeng has joined #openstack-ironic02:54
*** enikanorov_ has quit IRC02:56
*** enikanorov__ has joined #openstack-ironic02:56
*** nosnos has quit IRC03:37
*** eghobo has joined #openstack-ironic03:41
*** jcoufal has joined #openstack-ironic03:44
*** mitz_ has joined #openstack-ironic03:58
*** mitz has quit IRC03:58
*** rameshg87 has joined #openstack-ironic04:04
*** nosnos has joined #openstack-ironic04:15
openstackgerritA change was merged to openstack/ironic: Add a test case for DB schema comparison  https://review.openstack.org/10762804:22
*** Poornima has joined #openstack-ironic04:26
openstackgerritSyed Ismail Faizan Barmawer proposed a change to openstack/ironic-specs: UEFI support for Ironic deploy drivers  https://review.openstack.org/9985004:29
*** pradipta_away is now known as pradipta04:34
*** ramineni has joined #openstack-ironic04:36
*** jcoufal has quit IRC04:47
openstackgerritMichael Davies proposed a change to openstack/python-ironicclient: Export 'client' and 'exc' modules  https://review.openstack.org/10897404:58
*** k4n0 has joined #openstack-ironic05:17
*** rakesh_hs has joined #openstack-ironic05:23
*** bmahalakshmi has joined #openstack-ironic05:26
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/10890106:10
*** bvivek has joined #openstack-ironic06:16
*** Mikhail_D_ltp has joined #openstack-ironic06:44
*** Mikhail_D_ltp has quit IRC06:46
*** pradipta is now known as pradipta_away06:46
*** Mikhail_D_ltp has joined #openstack-ironic06:49
*** sabah has joined #openstack-ironic06:56
*** lsmola has joined #openstack-ironic07:02
*** mkerrin has joined #openstack-ironic07:04
*** eghobo has quit IRC07:11
*** Poornima has quit IRC07:30
*** nikunj2512 has joined #openstack-ironic07:30
*** romcheg1 has joined #openstack-ironic07:37
*** sabah has quit IRC07:39
openstackgerritHaomeng,Wang proposed a change to openstack/ironic: Implements send-data-to-ceilometer  https://review.openstack.org/7253807:39
*** jistr has joined #openstack-ironic07:42
*** ifarkas has joined #openstack-ironic07:46
*** ndipanov has joined #openstack-ironic07:53
*** william has joined #openstack-ironic07:55
*** william has left #openstack-ironic07:55
*** pelix has joined #openstack-ironic07:56
*** sabah has joined #openstack-ironic08:10
*** dguerri is now known as _dguerri08:12
*** bvivek has quit IRC08:15
*** _dguerri is now known as dguerri08:18
*** dtantsur|afk is now known as dtantsur08:19
dtantsurMorning Ironic!08:19
*** derekh_ has joined #openstack-ironic08:21
*** Zerpex has joined #openstack-ironic08:22
ZerpexHi awesome people! Does anyone got ironic provisioning working on a CentOS 6 OS? I'm currently making a POC using the packstack tool - for yeah, looking into if we're gonna use Openstack and ironic for deploying servers - but I've searched the interwebs thin about how to get ironic working on CentOS without any luck sadly :/08:24
*** igordcard has joined #openstack-ironic08:26
*** bvivek has joined #openstack-ironic08:28
*** MattMan has joined #openstack-ironic08:29
dtantsurZerpex, hi! Well, I did some work on supporting CentOS and Fedora using DevStack. There is nothing in Ironic itself that would prevent you from using it on CentOS 608:31
dtantsurZerpex, I suspect the biggest problem would be that Ironic is still trying to graduate as a project, I am not sure that packstack will support it out-of-box currently (instead of nova-baremetal)08:32
dtantsurBut again, that's mostly about how to install and configure Ironic08:32
dtantsurthat reminds me, that I have to do one more round of checks of the current situation on Fedora and CentOS...08:33
openstackgerritHaomeng,Wang proposed a change to openstack/ironic: Implements send-data-to-ceilometer  https://review.openstack.org/7253808:36
Zerpexdtantsur: cool! I don't mind if I need to configure some stuff to get ironic working just for a prove of concept - but sadly I haven't been able to find much information about ironic as well - and I would love to see the speeds of openstack deploying operating systems etc :) Could turn out that another large hosting provider would base infrastructure on openstack :D08:40
dtantsurZerpex, sounds great :) out docs are not perfect for now, we realize. What we have (and try to improve) is http://docs.openstack.org/developer/ironic/deploy/install-guide.html08:42
dtantsurZerpex, it may miss some points, you're free to ping us here and ask08:42
ZerpexCool - main issue I have, is where to find the code for ironic-api and ironic-conductor08:43
dtantsurZerpex, what do you mean by code? Source code? https://github.com/openstack/ironic08:45
Zerpexdtantsur: yeah - so I can actually get it up and running on our test machines :D08:48
dtantsurZerpex, also DevStack source code (https://github.com/openstack-dev/devstack) is your friend: it contains ready-to-use bash commands to set up things, including Ironic08:49
Zerpexcool! Will take a look! You rock08:49
dtantsur:)08:51
*** bmahalakshmi has quit IRC08:56
*** bmahalakshmi has joined #openstack-ironic08:57
*** bmahalakshmi2 has joined #openstack-ironic09:03
*** bmahalakshmi has quit IRC09:04
*** bvivek has quit IRC09:06
*** early has quit IRC09:08
*** bvivek has joined #openstack-ironic09:10
*** early has joined #openstack-ironic09:12
*** igordcard has quit IRC09:22
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: Do not rely on hash ordering in tests  https://review.openstack.org/10993909:25
*** openstackgerrit has quit IRC10:01
*** saripurigopi has joined #openstack-ironic10:04
*** jcoufal has joined #openstack-ironic10:05
*** yuriyz|2 has quit IRC10:09
saripurigopiis there any simple rst tool available that helps with spec format?10:12
*** mitz_ has quit IRC10:17
*** mitz has joined #openstack-ironic10:18
*** mitz has quit IRC10:18
*** mitz has joined #openstack-ironic10:20
saripurigopiIS there a way to run jenkins tests before submitting for review?10:24
*** tteggel_ is now known as tteggel10:25
*** dtantsur is now known as dtantsur|lunch10:25
raminenisaripurigopi: you can create venev and run testr run and build_sphinx ..10:28
raminenisaripurigopi: # activate your development virtualenv source .tox/venv/bin/activate .. testr init , testr run, python setup.py build_sphinx10:29
raminenisaripurigopi: this should help to ensure jenkins pass successfully10:29
saripurigopiramineni, sure thanks will try it out.10:30
*** Nisha has joined #openstack-ironic10:31
*** Nisha has quit IRC10:56
*** ramineni has quit IRC10:58
*** rameshg87 has left #openstack-ironic10:59
IsotoppHello. I am looking at systems for the automated installation of machines such as MAAS or Ironic. The task that I need to solve is run a preboot-thingie, that is, updating the firmwares of the machines (or ensure that they are recent)11:00
Isotoppand running a program to configure the RAID controller setup.11:00
IsotoppThe machines are DL380G8 and ibm blades, and I do have programs such as hpssacli,11:00
Isotoppbut I do see no state engine that would automatically boot into a preboot image, and run a setup script11:00
Isotoppexpect a callback and then boot into the actual installation11:01
IsotoppThat's what I would have expected to see, so how is this being done in ironic instead?11:01
*** dguerri is now known as _dguerri11:02
*** bmahalakshmi2 has quit IRC11:15
*** bmahalakshmi has joined #openstack-ironic11:15
*** Halacs has quit IRC11:23
*** Halacs has joined #openstack-ironic11:23
*** sabah has quit IRC11:23
*** jcoufal has quit IRC11:25
*** dtantsur|lunch is now known as dtantsur11:36
dtantsurIsotopp, hi. You may want to have a look at Ironic Python Agent (https://review.openstack.org/98506) which is our future ramdisk and some more specs (https://review.openstack.org/#/c/100842/ etc)11:37
dtantsurI guess you'd want to talk to jroll or JayF on this IPA thing11:38
*** bvivek2 has joined #openstack-ironic11:43
*** bvivek has quit IRC11:44
Isotoppdtantsur: thanks. looking11:44
*** yuriyz has joined #openstack-ironic11:46
*** nikunj2512 has quit IRC11:48
*** viktors|afk is now known as viktors11:48
*** krtaylor_ is now known as krtaylor11:56
*** _dguerri is now known as dguerri12:05
*** saripurigopi has quit IRC12:09
*** dtantsur has quit IRC12:14
*** jcoufal has joined #openstack-ironic12:18
*** bmahalakshmi has quit IRC12:27
*** jdob has joined #openstack-ironic12:37
*** dguerri is now known as _dguerri12:46
*** _dguerri is now known as dguerri12:46
*** dguerri is now known as _dguerri12:46
*** _dguerri is now known as dguerri12:46
*** k4n0 has quit IRC12:50
*** jasondotstar has joined #openstack-ironic12:52
*** dguerri is now known as dguerri`afk12:54
*** dguerri`afk is now known as dguerri12:54
Shrews_lifeless: took the liberty of rebasing 106906 for you13:06
*** nosnos has quit IRC13:07
*** rloo has joined #openstack-ironic13:09
*** datajerk has quit IRC13:23
*** jbjohnso has quit IRC13:31
*** datajerk has joined #openstack-ironic13:32
*** openstackgerrit has joined #openstack-ironic13:33
*** pcrews has joined #openstack-ironic13:34
IsotoppHm, are there any ubuntu 12 or 14 packages for ironic?13:40
rlooIsotopp: ubuntu 14, ircehouse packages.13:41
rlooIsotopp: http://docs.openstack.org/developer/ironic/deploy/install-guide.html#install-the-bare-metal-service13:42
Isotoppthanks. let's see how i integrate that. i am on precise at the moment, because contrail13:43
rlooIsotopp: but I think the latest version of ironic is better than the icehouse version.13:43
Shrews_rloo: morning13:46
*** Shrews_ is now known as Shrews13:46
rlooMorning Shrews!13:47
rlooShrews, did you have fun last week?13:47
Shrewsrloo: at the meetup? yeah. rest of the week was hectic13:52
rlooShrews: too bad about the hectic part, but hopefully this week will be relaxing ;)13:52
*** mdorman has joined #openstack-ironic13:57
*** rakesh_hs has quit IRC14:00
*** dtantsur has joined #openstack-ironic14:09
*** dguerri is now known as dguerri`afk14:10
*** dguerri`afk is now known as dguerri14:13
*** bmahalakshmi has joined #openstack-ironic14:15
* dtantsur is online again. Hello to everyone, who I didn't see :)14:15
*** jgrimm has joined #openstack-ironic14:15
rloohi dtantsur!14:20
dtantsurrloo, hi14:20
*** jistr has quit IRC14:22
boris-42devananda okay here is spec for OSprfoiler14:23
boris-42devananda https://review.openstack.org/#/c/103825/314:23
boris-42devananda so if you would like to discuss it in specs that is the right place14:23
*** jistr has joined #openstack-ironic14:23
*** rainya has joined #openstack-ironic14:24
*** rainya has quit IRC14:24
*** rainya has joined #openstack-ironic14:25
Shrewshey dtantsur14:29
dtantsurShrews, hi!14:30
*** bvivek2 has quit IRC14:35
*** dtantsur has quit IRC14:36
*** dtantsur has joined #openstack-ironic14:37
boris-42jroll ping14:39
IsotoppI have just installed ironic in a virtualenv on ubuntu 12.04, which has been clean installed before.14:40
Isotoppthe dbsync failed because14:40
IsotoppMySQLdb is missing14:40
IsotoppI had to14:40
Isotopp 1309  aptitude install libmysqlclient-dev14:40
Isotopp 1310  pip install MySQL-python14:40
Isotoppthen the dbsync works.14:40
IsotoppI do believe that is a missing dependency14:40
Isotopp(but maybe a variable one, depending on the SQLAlchemy options used, right?)14:41
dtantsurIsotopp, I guess the default (though not perfect) is sqlite14:46
Isotoppi just want to make sure that this is not an actual bug as in 'missing dependency' but a thing that comes from a configuration choice.14:47
*** bvivek has joined #openstack-ironic14:51
openstackgerritA change was merged to openstack/ironic: Push the image cache ttl way up  https://review.openstack.org/10690614:52
Shrewsrloo: no meeting today, right?14:53
rlooShrews: right.14:54
Shrewssince everyone is partying in portland14:54
Shrewsi mean... "working", of course  :)14:54
rlooShrews: you mean, having lunch in portland ;)14:54
*** mrda-traveling is now known as mrda14:55
NobodyCamgood morning Ironic15:06
dtantsurNobodyCam, morning15:12
*** dkehnx has quit IRC15:12
*** bvivek has quit IRC15:12
*** dkehn_ has joined #openstack-ironic15:15
jrollboris-42: hi15:15
*** dkehn_ is now known as dkehnx15:15
boris-42jroll hi there15:15
jrollmornin everybody :)15:15
boris-42jroll I made a spec for osprofiler https://review.openstack.org/#/c/103825/15:16
jrollyep, I see that15:17
jrollI'll take a look when I get a chance15:17
boris-42jroll ok thanks15:17
jrollIsotopp: what doc are you reading for dependencies?15:17
rloojroll, Isotopp: our deploy instructions assume MySQL (and assumes it was already installed/setup due to other OpenStack components).15:21
rloojroll, Isotopp: http://docs.openstack.org/developer/ironic/deploy/install-guide.html#set-up-the-database-for-bare-metal15:21
jrollyeah, exactly, that's why I want to fix whatever docs Isotopp is reading :)15:21
jrollI would assume that nearly every project requires mysqldb15:22
rloojroll: yeah15:22
NobodyCammornign dtantsur jroll, boris-42 ,rloo and Shrews15:24
jrollmorning NobodyCam15:24
ShrewsNobodyCam: hey hey15:24
dtantsurjroll, morning15:24
rlooMorning NobodyCam. You at the meetup?15:24
NobodyCamsi15:24
jrollheya dtantsur15:24
boris-42NobodyCam hi there-)15:24
rlooNobodyCam: great. I expect great things from all of you this week :-)15:24
NobodyCam:) hehehe we will try :)15:25
viktorsdtantsur: hi!15:27
dtantsurviktors, hi!15:28
viktorsdtantsur: I have a question as for your comment to patch https://review.openstack.org/#/c/10762915:28
viktorsdtantsur: ad for engine in create_schema()15:29
viktors*as15:29
dtantsursure15:29
viktorsdtantsur: your comments sounds ok for me, but I see that we create engine also in version() function in ironic/db/sqlalchemy/migration.py file. Should I change signature of version() function also?15:30
dtantsurme is looking15:32
ryanpetrelloIronic folks: any more thoughts on this backport? https://review.openstack.org/#/c/105583/15:32
ryanpetrellonot totally sure who has +2 on these backports15:32
Shrewsryanpetrello: yeah, not sure who controls that (ttx?). we can't +2 it15:32
dtantsurviktors, I'm not sure, if get_engine() actually returns new engine each time...15:33
dtantsurviktors, but yes, it makes sense to do consistently15:34
dtantsurviktors, judging by version(), maybe the same engine is returned each time... or it does not matter15:34
viktorsdtantsur: I suppose to add engine as an optional argument to these two functions15:36
dtantsurviktors, makes sense to me15:36
viktorsdtantsur: >> get_engine() actually returns new engine each time...   -- see https://github.com/openstack/oslo.db/blob/master/oslo/db/sqlalchemy/session.py#L690-L70415:36
viktorsI'm gust supposed, that we can work with multiple engines in a same time15:36
dtantsuryeah, makes sense15:37
viktorsdtantsur: ok, will fix, thanks!15:38
dtantsurnp)15:38
jrollryanpetrello: there's some sort of "stable release" team - not sure who that is but maybe poke infra or ttx... that has four +1s from ironic cores15:38
ryanpetrellookay15:39
ryanpetrelloI’ll figure it out15:39
jroll:)15:39
*** eghobo has joined #openstack-ironic15:40
*** lsmola has quit IRC15:43
*** rainya has quit IRC15:45
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: EXPERIMENTAL Implement set up for hardware discovery in PXE driver  https://review.openstack.org/11003115:49
*** Mikhail_D_ltp has quit IRC15:49
*** eghobo has quit IRC15:52
*** ndipanov has quit IRC16:00
*** eghobo has joined #openstack-ironic16:00
*** rainya has joined #openstack-ironic16:01
openstackgerritVictor Sergeyev proposed a change to openstack/ironic: Use metadata.create_all() to get database schema  https://review.openstack.org/10762916:02
rloojroll, you have time to look at my comment for 81391?16:03
*** ifarkas has quit IRC16:09
*** dtantsur is now known as dtantsur|afk16:11
jrollrloo: I'll make JoshNang take a look16:12
rloojroll: ok thx. I know you guys want it merged. I'm on the fence about it.16:12
jroll;)16:12
jrollyeah16:12
rloojroll: if others feel like that is ok (or can be done in a subsequent patch), I'll +2 it.16:12
jrollrloo: ok, it shouldn't be a problem to add16:13
JoshNangrloo: looking now. just got on the wifi16:13
jrollI think it's ok but I'm biased16:13
rloothx jroll, JoshNang. jroll, that's why I asked you, to see what your biased opinion would be ;)16:14
jroll:)16:14
JoshNangrloo: i think i'm going to have to upload another patchset anyway. when i was debugging direct_url support, i went source diving and found 700 lines of glance code to deal with swift direct_urls and 5 different formats :/ I don't think we can feasibly support that16:14
rlooJoshNang: ouch.16:15
rlooJoshNang: maybe WIP it then.16:15
JoshNangrloo: done. i'll have a patch removing it for now shortly16:16
rloothx JoshNang. You also at the meetup?16:17
*** ellenh has joined #openstack-ironic16:17
JoshNangrloo: i am! i mentioned direct_url in the spec. how ok is it to drop that support?16:19
jrollnoooooooo not WIP :|16:19
rlooJoshNang, I think you have to huddle with jroll. Make sure jroll doesn't cry.16:19
jrollremove direct_url and land it :|16:19
* jroll cries16:19
*** matty_dubs|gone is now known as matty_dubs16:19
JoshNangjroll: don't cry. i just have to fix tests and make the docs a bit better16:20
jrollgogogo16:20
rlooJoshNang: that's a good question about the spec. I suspect you could for now, put in a patch that deals with the non-direct-url part of the spec, and then face the core-spec folks wrt not implementing the direct-url part of the spec ;)16:20
* jroll banishes JoshNang to the corner16:20
* JoshNang is at the corner of the table. close enough16:21
JoshNangrloo: wfm16:21
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Update_port should expect MACAlreadyExists  https://review.openstack.org/10910716:28
*** eguz has joined #openstack-ironic16:28
*** eguz has quit IRC16:28
*** lucasagomes has joined #openstack-ironic16:28
*** Penick has joined #openstack-ironic16:29
*** eguz has joined #openstack-ironic16:29
*** viktors is now known as viktors|afk16:29
*** eghobo has quit IRC16:32
*** jcoufal has quit IRC16:33
mrdaJust FYI - those linking ironic t-shirts and stickers look here: http://www.redbubble.com/people/mrda/works/12207321-ironic16:33
rloomrda: shouldn't ironic be first in that list?16:34
JayFjroll: JoshNang ^^ we should all get red copies of that shirt16:34
mrdarloo: well, it's levereging off an existing design16:37
mrdabut certainly the highlight is ironic :)16:37
mrdaJayF: Feel free :)16:37
mrdarloo: Also, wish you were here in PDX! :)16:38
rloomrda: ok, if there was an existing design, I won't comment about it any more.16:38
rloomrda: ahh yeah, if it was a short hop to PDX I might have considered it, but it is more like a hop, skip, and big jump.16:39
mrdarloo: 30 hours for me :-P16:39
rloomrda: you're a real trooper!16:39
*** ndipanov has joined #openstack-ironic16:40
*** rameshg87 has joined #openstack-ironic16:42
jrolllucasagomes: was it you that was looking at booting from disk in virsh? the internet is telling me "just edit the config"16:46
*** bearhands is now known as comstud16:46
lucasagomesjroll, hmm? not that I remember16:46
lucasagomesjroll, the set boot device will edit the config16:46
jrollheh, ok16:46
lucasagomesfor e.g16:47
jrollsomebody was looking at it16:47
lucasagomesif you set disk as a boot device16:47
jrollahh, I see16:47
jrollthe problem is... I don't think a 'set boot device' command exists for virsh :|16:49
jrollI should probably like, just make ipa pxe boot the instance image to begin with16:50
*** jistr has quit IRC16:51
*** marzif_ has quit IRC16:52
*** shakamunyi has joined #openstack-ironic16:54
JayFjroll: if it's helpful to get it done quickly, I can show you a quick and dirty pxeboot chainload16:55
JayFjroll: I've implemented it before in the hardware lab16:55
*** derekh_ has quit IRC16:58
*** dguerri is now known as dguerri`afk17:00
*** rwsu has joined #openstack-ironic17:13
*** romcheg1 has quit IRC17:17
*** harlowja_away is now known as harlowja17:20
*** Nisha has joined #openstack-ironic17:23
*** nikunj2512 has joined #openstack-ironic17:26
ryanpetrello\o/ merged https://review.openstack.org/#/c/105583/17:28
jroll\o/17:33
*** nikunj2512 has quit IRC17:49
jrollstack@devstack:~$ sudo virsh dumpxml baremetalbrbm_1 | sed "s/boot dev='network'/boot dev='hd'/" | sudo virsh update-device baremetalbrbm_017:51
jrollerror: command 'update-device' requires <file> option17:51
jrollhate you, vrish17:51
jrollvirsh, too17:51
*** Nisha has quit IRC17:55
*** shakamunyi has quit IRC17:58
*** achanda has joined #openstack-ironic17:58
*** rainya has quit IRC17:59
*** rainya has joined #openstack-ironic17:59
lucasagomesjroll, take a look at https://review.openstack.org/#/c/89884/18/ironic/drivers/modules/ssh.py18:10
openstackgerritMichael Davies proposed a change to openstack/python-ironicclient: Export 'client' and 'exc' modules  https://review.openstack.org/10897418:10
*** rainya has quit IRC18:11
*** achanda has quit IRC18:13
*** achanda has joined #openstack-ironic18:14
openstackgerritJosh Gachnang proposed a change to openstack/ironic: Adding swift temp url support  https://review.openstack.org/8139118:20
*** rainya has joined #openstack-ironic18:22
openstackgerritA change was merged to openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/10890118:22
openstackgerritRamakrishnan G proposed a change to openstack/ironic-specs: UEFI support for Ironic deploy drivers  https://review.openstack.org/9985018:23
jrolllucasagomes+++++++18:23
lucasagomesjroll, btw, can't add you to the group there's something wrong with ur launchpad id18:23
lucasagomesjroll, tried using ur full name and launchpad id but didn't work18:24
*** rameshg87 has quit IRC18:25
*** rameshg87 has joined #openstack-ironic18:27
*** rainya has quit IRC18:27
*** achanda has quit IRC18:28
rameshg87hi devananda, all, we have just updated the uefi boot spec removing the dependencies on the nova ironic virt driver  - https://review.openstack.org/#/c/9985018:28
rameshg87please have a look at the spec ...18:28
rlooJoshNang: yt? wrt 81391, I don't think you rebased it against master.18:32
*** rainya has joined #openstack-ironic18:32
jrolllucasagomes: try 1034318:32
*** bmahalakshmi has quit IRC18:33
*** rameshg87 has quit IRC18:33
JoshNangrloo: oh gah. missed the jroll rebased18:33
jrollJoshNang: rloo might be right, I had rebased18:33
jrollyeah18:34
jrollha, grab that18:34
JoshNangstarted on the code on friday but didn't finish18:34
jrolloops :|18:34
lucasagomesjroll, LOL ok it works18:34
jrolllucasagomes: yeahhhh, it's messed up :/18:36
openstackgerritJosh Gachnang proposed a change to openstack/ironic: Adding swift temp url support  https://review.openstack.org/8139118:36
JoshNangrloo: should be fixed!18:37
rlooJoshNang: ok, it's in the back of my queue now ;)18:37
JoshNangrloo: heh no worries18:38
*** achanda has joined #openstack-ironic18:43
*** rainya has quit IRC18:55
*** scubacuda has joined #openstack-ironic18:57
*** achanda has quit IRC18:57
*** scubacuda has joined #openstack-ironic18:58
*** rainya has joined #openstack-ironic19:04
*** ekarlso has quit IRC19:04
*** ellenh has quit IRC19:09
*** pelix has quit IRC19:14
*** ekarlso has joined #openstack-ironic19:16
*** Penick has quit IRC19:18
*** wanyen has joined #openstack-ironic19:27
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements  https://review.openstack.org/10656919:34
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/8872219:34
*** achanda has joined #openstack-ironic19:44
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements  https://review.openstack.org/10656919:48
*** ndipanov has quit IRC19:48
mrdarloo: Just getting py33 running before I pit up a new patch for ironicclient19:52
rloocool mrda.19:52
*** achanda has quit IRC19:53
openstackgerritMichael Davies proposed a change to openstack/python-ironicclient: Export 'client' and 'exc' modules  https://review.openstack.org/10897419:59
*** achanda has joined #openstack-ironic20:04
*** tatyana has joined #openstack-ironic20:08
*** igordcard has joined #openstack-ironic20:21
*** lucasagomes has quit IRC20:22
*** ellenh has joined #openstack-ironic20:23
*** jdob has quit IRC20:23
*** lucasagomes has joined #openstack-ironic20:31
openstackgerritMichael Davies proposed a change to openstack/python-ironicclient: Export 'client' and 'exc' modules  https://review.openstack.org/10897420:45
*** igordcard has quit IRC20:57
*** igordcard has joined #openstack-ironic21:10
openstackgerritJosh Gachnang proposed a change to openstack/ironic: Adding swift temp url support  https://review.openstack.org/8139121:16
*** jasondotstar has quit IRC21:23
lucasagomesrloo, yes for the iPXE driver I'm going to address the comments21:30
rloothx lucasagomes21:30
rloobut lucasagomes, get the nova-ironic-driver merged first ;)21:30
lucasagomesrloo, hah ack21:37
mrdarloo: +121:40
rloomrda: you're there. One of your tasks is to make sure the others know what the priorities are :D21:41
jrolllucasagomes: btw, your ssh set_boot_device works :)21:41
lucasagomesjroll, :) yay21:41
*** jgrimm has quit IRC21:41
jrolllucasagomes: so I guess we depend on swift tempurl thing, and now ssh management interface, do we want to just try to land one/both today or make a patch chain?21:42
jrolland if we want to make a patch chain, what order?21:43
lucasagomesjroll, yeah... I think I have to address some small nits on the ssh driver and it's pretty much ready... for the swift temp url it's also almost there no?21:43
lucasagomeswill review it21:43
jrolllucasagomes: yes, I think tempurl is good to go, rloo found some nits but it should be fine21:46
rloojroll, that swift url thingy? I just +221:47
rloo'd it. Let someone else -1 it now ;)21:47
*** tatyana has quit IRC21:48
openstackgerritMichael Davies proposed a change to openstack/python-ironicclient: Export 'client' and 'exc' modules  https://review.openstack.org/10897421:49
JoshNangrloo: thanks for the handful of reviews :)21:49
rlooJoshNang: yw. I'm exhausted now ;)21:50
JoshNangrloo: me too!21:52
rlooJoshNang: ha ha. Get it right the first time then ;)21:53
rlooJoshNang: it ain't quite done yet... let's hope it gets merged soon.21:53
jrollrloo: awesome, thanks :)21:54
*** dhellmann_ has joined #openstack-ironic21:55
*** harlowja is now known as harlowja_away21:57
*** dhellmann has quit IRC21:57
*** dhellmann_ is now known as dhellmann21:57
*** harlowja_away is now known as harlowja22:03
rloomrda: why'd you remove the -1 from the test in 108974?22:05
*** harlowja has quit IRC22:10
*** harlowja has joined #openstack-ironic22:11
openstackgerritGhe Rivero proposed a change to openstack/ironic: Raise MissingParameterValue instead of invalid  https://review.openstack.org/10845522:14
jroll+2'd https://review.openstack.org/#/c/89884/22:22
jrolljust needs one more :)22:23
lifelessShrews: good idea :)22:28
*** lucasagomes has quit IRC22:38
jrollrloo: I had to -1 the swift url thing :(22:42
mrdarloo: It's defaulted anyway, and in py33 -1 is not supported22:43
mrdaso taking the defaults, we get what we want, and not specifying it means we can have one solution across py27/33 without having to do a six.blah test22:43
mrdarloo ^^22:44
*** lucasagomes has joined #openstack-ironic22:44
JoshNangjroll: rloo and I debated the 5 mins vs 15 mins. the spec was written with 5, i'm open to either22:50
JayFI think landing it with either default is better than not landing it.22:54
JayFPersonally I'd go even higher than 15m, but that's probably because I don't think you should keep secret info in images :)22:54
JoshNangJayF: right, but if deployers want users to be able to use their own images, i'm sure some of those will have secret info, which is why we dropped it from 60mins22:55
jrollJayF, JoshNang, my point is that 5 almost certainly will not work. in devstack and in real hardware.22:57
JayFjroll: why not?22:58
JayFWhen is the temp url generated vs when it's used?22:58
jroll^ answered in meatspace23:06
rchew23:08
openstackgerritJosh Gachnang proposed a change to openstack/ironic: Adding swift temp url support  https://review.openstack.org/8139123:12
openstackgerritJosh Gachnang proposed a change to openstack/ironic: Adding swift temp url support  https://review.openstack.org/8139123:17
openstackgerritJosh Gachnang proposed a change to openstack/ironic: Adding swift temp url support  https://review.openstack.org/8139123:20
*** antonym has quit IRC23:26
*** achanda has quit IRC23:35
*** achanda has joined #openstack-ironic23:37
*** igordcard has quit IRC23:38
*** antonym has joined #openstack-ironic23:40
lucasagomesrloo, left a comment on the iPXE review23:46
lucasagomesrloo, so yeah the name is not great but i think the suggested ones are not good too. I suggested changing that name to chainload.ipxe, what you think? (there's some explanation on the comment)23:47
* lucasagomes brb23:47
*** lucasagomes has quit IRC23:59

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