Tuesday, 2014-02-11

« Monday, 2014-02-10 Index Wednesday, 2014-02-12 »
*** rongze has quit IRC00:02
*** jrist has joined #openstack-ironic00:09
*** epim has quit IRC00:26
jog0devananda: we did but don't gate on it00:32
jog0devananda: thanks,00:32
devanandajog0: ack. I +1'd but noted that in coments00:32
jog0NobodyCam: no idea00:32
jog0thanks00:32
NobodyCamjog0: seems it is... fro what I can find00:33
NobodyCamfrom even00:33
NobodyCambeen at keyboard sense 6:30 this morning ... need to step away ... trying to test my bash update script.. should be able to post up tomorrow00:36
*** Wanyen has quit IRC00:38
*** rongze has joined #openstack-ironic00:57
*** rongze has quit IRC01:02
*** epim has joined #openstack-ironic01:09
*** rongze has joined #openstack-ironic01:50
*** epim has quit IRC01:52
*** yongli has joined #openstack-ironic01:59
*** thedodd has joined #openstack-ironic02:06
*** thedodd has quit IRC02:35
*** wusWWW has quit IRC02:36
*** russell_h has quit IRC03:01
*** mikal has quit IRC03:02
*** mikal has joined #openstack-ironic03:03
*** greghaynes has quit IRC03:04
*** greghaynes has joined #openstack-ironic03:07
*** russell_h has joined #openstack-ironic03:09
openstackgerritHaomeng,Wang proposed a change to openstack/ironic: Implements Send Sensor Data to Ceilometer  https://review.openstack.org/7253803:14
*** russell_h has quit IRC03:52
*** russell_h has joined #openstack-ironic03:52
*** harlowja is now known as harlowja_away04:22
*** harlowja_away is now known as harlowja04:30
*** igor_ has joined #openstack-ironic04:35
*** jbjohnso has quit IRC04:52
*** killer_prince has quit IRC05:14
*** killer_prince has joined #openstack-ironic05:15
*** pquerna has quit IRC05:19
*** pquerna has joined #openstack-ironic05:20
*** rongze has quit IRC05:29
*** rongze has joined #openstack-ironic05:30
*** rongze has quit IRC05:34
*** shawal has joined #openstack-ironic05:36
openstackgerritHaomeng,Wang proposed a change to openstack/ironic: Implements bp - Send Sensor Data to Ceilometer  https://review.openstack.org/7253805:41
*** shawal has quit IRC05:58
*** shawal has joined #openstack-ironic06:00
openstackgerritJenkins proposed a change to openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/7119206:06
*** rwsu has quit IRC06:07
*** rwsu has joined #openstack-ironic06:07
*** rongze has joined #openstack-ironic06:15
*** killer_prince has quit IRC06:24
*** rwsu has quit IRC06:25
*** aignatov_ is now known as aignatov06:28
*** loki_ has joined #openstack-ironic06:32
*** harlowja is now known as harlowja_away06:33
*** shawal has quit IRC06:37
*** rwsu has joined #openstack-ironic06:38
*** saju_m has joined #openstack-ironic06:38
*** coolsvap has joined #openstack-ironic06:40
*** saju_m has quit IRC06:43
*** saju_m has joined #openstack-ironic06:50
*** saju_m has quit IRC06:51
*** aignatov is now known as aignatov_07:09
*** shawal has joined #openstack-ironic07:14
*** yongli has quit IRC07:22
*** saju_m has joined #openstack-ironic07:37
*** zigo has joined #openstack-ironic07:47
*** mdurnosvistov_ has joined #openstack-ironic07:51
*** saju_m has quit IRC07:52
*** mdurnosvistov_ has quit IRC08:04
*** killer_prince has joined #openstack-ironic08:06
*** romcheg1 has joined #openstack-ironic08:07
Haomengmorning Ironic:)08:28
*** killer_prince has quit IRC08:34
*** killer_prince has joined #openstack-ironic08:35
zigoDoes the new coinor.pulp dependency in Ironic means I'd have to package coinor, pulp, and coinor.pulp in Debian?08:37
zigoGheRivero: Do you know? ^08:38
GheRiveromorning zigo08:39
zigoHi! :)08:40
zigoI can't even find the sources for coinor. :/08:40
zigohttps://pypi.python.org/pypi/coinor <--- No tarball there.08:40
GheRiverobut in ironic? I though it was used in the new nova scheduler08:41
*** ifarkas has joined #openstack-ironic08:41
GheRiverohttps://pypi.python.org/pypi/coinor.pulp08:41
*** aignatov_ is now known as aignatov08:41
*** aignatov is now known as aignatov_08:42
zigoGheRivero: Yeah, got that one, though doesn't it depend on coinor & pulp?08:42
zigoGheRivero: Indeed, it seems to be in Nova, my bad.08:42
zigoFYI: https://review.openstack.org/#/c/53446/08:42
GheRivero:)08:43
zigoGheRivero: Though, that doesn't answer my question! :) Will ask in the Nova channel.08:43
GheRiveroAFAIUI, it won't neccesary until Juno08:45
*** aignatov_ is now known as aignatov08:45
zigoGheRivero: So, I can skip packaging coinor.pulp for Icehouse?08:48
zigopulp got some non-free files btw.08:49
zigoLike: src/pulp/solverdir/cbc.exe08:49
GheRivero.exe? OMFG!08:49
*** shawal has quit IRC08:49
GheRiveroi don't think they are going to change the scheduler so close to freeze date08:49
*** saju_m has joined #openstack-ironic08:51
*** jistr has joined #openstack-ironic08:58
openstackgerritAndrey Kurilin proposed a change to openstack/python-ironicclient: Reuse cliutils from common code  https://review.openstack.org/7241809:02
*** athomas has joined #openstack-ironic09:22
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Add parameter for filtering nodes by maintenance mode  https://review.openstack.org/6393709:23
GheRiveromorning all09:23
zigoGheRivero: What's the nickname of Robert Collins?09:24
GheRiverolifeless09:24
zigoRight.09:24
zigolifeless: Are you around? :)09:25
lifelesszigo: just going to bed09:30
lifelesszigo: whats up ?09:30
zigolifeless: I'm just wondering about the subunit package in Debian.09:31
zigolifeless: I would have produce a patch for the new version, but I can't find "upstream" tarball ...09:31
lifelesszigo: https://launchpad.net/subunit/09:32
*** lucasagomes has joined #openstack-ironic09:33
zigolifeless: Ok, I'll try to do it and send the debdiff as attachment to a bug in the Debian BTS, if you're fine with that.09:33
zigoOh, you got the Ubuntu package ready ...09:33
zigolifeless: Could you upload that to Debian too? :)09:33
lifelesszigo: no, I didn't touch ubuntu09:34
lifelesszigo: I suspect someone in Ubuntu did09:34
lifelesszigo: NMU in Debian please09:34
zigolifeless: Ok, thanks.09:34
lifelesszigo: viva la cooperative uploads09:34
zigo:)09:34
zigolifeless: Should I commit to the BZR too?09:35
lifelesszigo: pacakging metadata - yes, if you can09:35
lifelesswe really need to sort that out09:35
zigoOk.09:35
zigolifeless: Thanks man, I'm not bothering you more then. Have a good night!09:35
lifelesszigo: gnight!09:35
*** coolsvap has quit IRC09:36
openstackgerritEugeniya Kudryashova proposed a change to openstack/python-ironicclient: Use HTTPClient from common Oslo code  https://review.openstack.org/6428609:43
*** lazy_prince has joined #openstack-ironic10:06
*** saju_m has quit IRC10:25
*** saju_m has joined #openstack-ironic10:26
*** jrist has quit IRC10:30
*** aignatov is now known as aignatov_10:34
*** tatyana has joined #openstack-ironic10:39
*** jrist has joined #openstack-ironic10:42
*** max_lobur_afk is now known as max_lobur10:46
*** aignatov_ is now known as aignatov10:50
*** lazy_prince has quit IRC10:50
*** rsacharya has joined #openstack-ironic10:51
*** rsacharya has quit IRC10:59
*** mgoddard_ has joined #openstack-ironic11:10
*** mgoddard has quit IRC11:13
*** mgoddard_ is now known as mgoddard11:21
*** loki_ has quit IRC11:32
*** romcheg1 has quit IRC11:48
*** mdurnosvistov has quit IRC11:49
*** aignatov has quit IRC11:50
*** lazy_prince has joined #openstack-ironic11:51
*** aignatov has joined #openstack-ironic11:54
*** romcheg1 has joined #openstack-ironic12:04
*** Alexei_987 has joined #openstack-ironic12:07
romchegMorning folks12:13
romchegSorry about the meeting, I fall asleep  :(12:14
*** mdurnosvistov has joined #openstack-ironic12:14
romcheg*is reading the logs*12:14
max_loburmorning romcheg12:18
max_loburand Ironic!12:18
GheRiveromorning all12:19
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Remove None as default value for dict get()  https://review.openstack.org/7263212:26
lucasagomesmorning all12:26
*** lucasagomes is now known as lucas-hungry12:30
Haomengmorning all:)12:33
*** shawal has joined #openstack-ironic12:34
*** walsha has joined #openstack-ironic12:40
*** shawal has quit IRC12:43
*** walsha has quit IRC12:47
mdurnosvistovMorning all! :)12:47
max_loburhi Haomeng, I commented https://bugs.launchpad.net/ironic/+bug/127883912:48
Haomengmax_lobur: thank you Max:)12:50
Haomengmax_lobur: yes, should be env issue12:51
max_loburpls add comment if you find something12:52
Haomengmax_lobur: /opt/stack/oslo.config/oslo is there as well, found another one duplicated12:52
Haomengsure12:52
Haomenghttp://paste.openstack.org/show/64227/12:53
Haomengmax_lobur: remove one of them, then it works now:) "rm -rf /usr/local/lib/python2.7/dist-packages/oslo*"12:54
max_lobur:)12:55
Haomengmax_lobur: thank you help me to verify with your env:)12:56
max_loburnp :_12:56
max_lobur:)12:56
Haomengmax_lobur: :)13:00
*** jdob has joined #openstack-ironic13:26
*** aignatov is now known as aignatov_13:28
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Add timeout for waiting callback in PXE driver  https://review.openstack.org/7239513:28
*** aignatov_ is now known as aignatov13:29
openstackgerritEugeniya Kudryashova proposed a change to openstack/python-ironicclient: Use HTTPClient from common Oslo code  https://review.openstack.org/6428613:32
*** lucas-hungry is now known as lucasagomes13:44
openstackgerritMikhail Durnosvistov proposed a change to openstack/ironic: Get rid object model `dict` methods part 3  https://review.openstack.org/6410814:17
openstackgerritMikhail Durnosvistov proposed a change to openstack/ironic: Get rid object model `dict` methods part 1  https://review.openstack.org/6002514:17
openstackgerritMikhail Durnosvistov proposed a change to openstack/ironic: Get rid object model `dict` methods part 2  https://review.openstack.org/6233114:17
openstackgerritMikhail Durnosvistov proposed a change to openstack/ironic: Get rid object model `dict` methods part 6  https://review.openstack.org/6433614:17
openstackgerritMikhail Durnosvistov proposed a change to openstack/ironic: Get rid object model `dict` methods part 5  https://review.openstack.org/6427814:17
openstackgerritMikhail Durnosvistov proposed a change to openstack/ironic: Get rid object model `dict` methods part 4  https://review.openstack.org/6425114:17
*** rongze has quit IRC14:21
*** jbjohnso has joined #openstack-ironic14:24
*** ifarkas has quit IRC14:24
*** ifarkas has joined #openstack-ironic14:25
openstackgerritMax Lobur proposed a change to openstack/ironic: Replace assertEqual(None, *) with assertIsNone in tests  https://review.openstack.org/7038214:27
*** saju_m has quit IRC14:27
*** saju_m has joined #openstack-ironic14:29
*** rloo has joined #openstack-ironic14:30
openstackgerritMax Lobur proposed a change to openstack/ironic: Remove unnecessary json dumps/loads from tests  https://review.openstack.org/7041314:33
*** igor___ has joined #openstack-ironic14:33
*** igor_ has quit IRC14:34
*** lazy_prince has quit IRC14:35
*** rongze has joined #openstack-ironic14:36
*** romcheg1 has quit IRC14:52
devanandag'morning, all14:56
*** matty_dubs|gone is now known as matty_dubs14:57
matty_dubsMorning devananda14:57
*** jrist has quit IRC14:59
lucasagomesdevananda, morning15:00
max_loburmorning devananda15:00
lucasagomesdevananda, I took a look at how nova bm does start/stop the console, seems they don't expose anything via API, the console is started when the machine is powered on and stopped when it's powered off15:01
lucasagomesdevananda, but in our case we have the console interface, so I'm going to the start/stop to the ironic api15:02
lucasagomesI just dunno what needs to be returned to the user15:02
lucasagomeswhich information he would need in order to access the console15:02
*** pquerna has quit IRC15:03
devanandalucasagomes: http://docs.openstack.org/user-guide/content/instance_console.html15:04
lucasagomeslemme see15:04
devanandai'm pretty sure there was a bug open about console-starts-when-machine-starts15:05
*** jbjohnso has quit IRC15:05
devanandawe don't always want that, and in fact, some hardware simply won't support it15:05
devanandaeg, dense compute chassis where the # nodes > # max console sessions15:05
lucasagomesI see15:06
devanandai need to run to an appt in ~15 min, and will probably be offline most of the day15:06
lucasagomesdevananda, ack I will try to come up with something15:07
lucasagomesthanks for pointing that15:07
lucasagomesthat link*15:07
devanandalucasagomes: welcome15:07
jrollyou can't get a VNC console on bare metal, can you?15:07
jrollI thought that was a hypervisor thing15:08
lucasagomesjroll, yea that link is the VNC for the instances15:08
lucasagomesfor baremetal we r going to use shellinabox + ipmitool15:08
jrollright15:08
jrollah, ok15:08
jrolldo we really want to restrict it to html?15:09
devanandajroll: that API is merely returning a URL. i dont think anyone has said it has to be console-via-HTTP15:10
jrolldevananda: isn't that what shellinabox is?15:11
devanandajroll: yep.15:11
devanandajroll: that's what nova-baremetal is using today. low hanging fruit - just do the same thing initially. add later.15:12
jrollsure, makes sense15:12
*** jbjohnso has joined #openstack-ironic15:18
*** pquerna has joined #openstack-ironic15:18
devanandaok, gotta run ... will try to be online tonight, otherwise see ya'll tmw15:19
openstackgerritRohan Kanade proposed a change to openstack/ironic: Implement the SeaMicro Power driver  https://review.openstack.org/7071915:20
jrollI'm having some issues getting set up developing on ironic. I'm following: http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html and getting errors on the tox command: https://gist.github.com/jimrollenhagen/7bf2e6cee00c0874000915:24
jrollhas anyone seen something like that?15:24
jrollthe weird uid/gid is an artifact of NFS sharing to virtualbox and shouldn't cause any issues15:25
*** sgrasley has quit IRC15:25
NobodyCamgood morning Ironic15:40
*** jrist has joined #openstack-ironic15:42
lucasagomesjroll, hmm haven't seem that15:42
lucasagomesweird it says Error parsing /data/dev/ironic/setup.cfg: OSError: [Errno 2] No such file or directory15:42
lucasagomesthe file is there?15:42
lucasagomesNobodyCam, morning!15:42
jrollyeah, that's why I showed the ls15:42
jrollsuper weird15:42
jrolld15:42
*** coolsvap has joined #openstack-ironic15:42
jrolloops15:42
lucasagomesI see, yea very odd indeed15:44
lucasagomesjroll, permission maybe?15:44
*** saju_m has quit IRC15:45
jrollI don't think so, but I'll run with root to be sure15:45
*** saju_m has joined #openstack-ironic15:45
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Add ability to break TaskManager locks via REST API  https://review.openstack.org/7121215:47
*** jrist has quit IRC15:47
jrolllucasagomes: fwiw I've worked on other projects with the same sort of pbr/tox stack and haven't had issues with it15:48
lucasagomesjroll, I see...15:48
lucasagomeshmm problem is that idk even hw to reproduce that problem15:48
jrollha, yeah15:49
jrollthis is a fresh ubuntu precise VM with apt-get update/upgrade ran15:50
NobodyCamjroll: where is the ls output?15:52
NobodyCamahh I see it15:54
openstackgerritRohan Kanade proposed a change to openstack/ironic: Implement the SeaMicro Power driver  https://review.openstack.org/7071915:56
openstackgerritRohan Kanade proposed a change to openstack/ironic: Implements SeaMicro VendorPassThru functionality  https://review.openstack.org/7072015:56
*** jrist has joined #openstack-ironic16:02
jrolllucasagomes: same error running with root :/16:05
lucasagomes:/16:05
jrollso definitely not permissions16:05
jrollI'm going to head into work and then I'll play with it from there16:05
lucasagomesyea, ouch what odd error16:05
lucasagomesack16:05
*** jrist has quit IRC16:06
NobodyCamjroll: jroll can you "cat /data/dev/ironic/setup.py"16:07
jrollNobodyCam: yeah, it matches master on github16:08
jrollhttps://github.com/openstack/ironic/blob/master/setup.py16:08
*** aignatov is now known as aignatov_16:09
NobodyCamcan you past ERROR: invocation failed, logfile: /data/dev/ironic/.tox/venv/log/venv-2.log16:10
NobodyCamthat file16:10
jrollit's identical to the error that was spit out in the first place, but yeah16:11
jrollone sec16:11
*** jbjohnso has quit IRC16:11
jrolloh, no, I destroyed the venv since then16:11
jrollrunning again now, it'll be a few16:11
NobodyCamsure16:11
*** jrist has joined #openstack-ironic16:12
* NobodyCam is taking in coffee atm16:12
*** jrist has quit IRC16:13
*** jrist has joined #openstack-ironic16:17
*** mrda_away is now known as mrda16:18
*** jog0 has left #openstack-ironic16:18
mrdamorning all16:18
NobodyCamgood morning mrda :)16:18
lucasagomesmorning mrda16:19
mrdamorning NobodyCam and lucasagomes16:20
NobodyCam:)16:20
*** jbjohnso has joined #openstack-ironic16:22
*** ndipanov has joined #openstack-ironic16:23
*** hemna has joined #openstack-ironic16:27
openstackgerritMax Lobur proposed a change to openstack/ironic: Fix race condition when changing node states  https://review.openstack.org/6913516:36
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Add get_console() method  https://review.openstack.org/7269816:37
jrollNobodyCam, lucasagomes, turns out my VM didn't have git installed :P16:38
jrollI'll send a patch for the docs when I get to the office16:38
lucasagomesjroll, hahaha16:38
NobodyCamlol :)16:38
lucasagomesjroll, thank you for that16:38
jrollnp :)16:38
*** yuriyz has quit IRC16:45
* NobodyCam makes second cup16:45
NobodyCamgrr CREATE_FAILED16:47
*** jistr has quit IRC16:47
*** matty_dubs is now known as matty_dubs|lunch16:52
NobodyCamand now bbt... brb16:52
max_loburbbiaw16:52
*** max_lobur has quit IRC16:52
NobodyCam:)16:54
*** thedodd has joined #openstack-ironic17:04
*** athomas has quit IRC17:07
*** digambar_ has joined #openstack-ironic17:10
*** ifarkas has quit IRC17:20
NobodyCamhahaha guess you have to actually register a node to be able to deploy to it :-p17:21
*** max_lobur has joined #openstack-ironic17:21
max_loburback17:21
NobodyCamWb max_lobur17:21
max_lobur:)17:21
*** rongze has quit IRC17:22
openstackgerritJim Rollenhagen proposed a change to openstack/ironic: Add git dependency to quickstart docs.  https://review.openstack.org/7270917:33
jrollNobodyCam: that's the doc fix ^17:34
NobodyCamawesome jroll :)17:34
jrollI got "(W) ccd7e3b: commit message lines >70 characters; manually wrap lines" when running git review17:34
jrolldo you want me to amend that now or just adhere to it in the future?17:34
jrollI was thinking the limit was 7217:34
NobodyCamlol17:35
NobodyCamoh and no . at the end please!17:35
NobodyCam:-p17:36
jrollah, right17:36
jrollok, I'll amend real quick17:36
*** max_lobur is now known as max_lobur_afk17:36
openstackgerritJim Rollenhagen proposed a change to openstack/ironic: Add git dependency to quickstart docs  https://review.openstack.org/7270917:37
jrollfixed17:37
NobodyCam:)17:37
NobodyCamgood catch jroll :)17:37
jrollthanks17:38
NobodyCamlucasagomes: is get a default install for RH17:43
lucasagomesjroll, thanks for the patch, I -1 because I think that it would be useful to set git to be installed for fedora/rhel as well17:43
NobodyCams/get/git/17:43
NobodyCam:-p17:43
lucasagomesNobodyCam, for sure it's not on the cloud image17:43
NobodyCam:)17:44
lucasagomesI don't remember on the default iso, but I bet it's not17:44
NobodyCamya17:44
jrolllucasagomes: I agree, but I don't have a fedora/rhel install handy to test17:44
jrollis the package just 'git', I assume?17:44
jrollI'm happy to add it if someone can verify it won't explode17:44
lucasagomesjroll, it's git yes :) (I use fedora)17:44
NobodyCamlucasagomes: hehehhee17:45
lucasagomeshttp://paste.openstack.org/show/64325/17:45
lucasagomes:)17:45
jrollk, one minute17:46
lucasagomesjroll, NobodyCam just booted an f19 iso here, git is not installed by default17:47
* lucasagomes was curious as well17:47
NobodyCam:) TY lucasagomes for checking17:48
openstackgerritJim Rollenhagen proposed a change to openstack/ironic: Add git dependency to quickstart docs  https://review.openstack.org/7270917:48
jrollyes, thanks to both of you17:48
lucasagomesthanks +2 :)17:49
jroll:)17:51
*** matty_dubs|lunch is now known as matty_dubs17:51
NobodyCamwill +a after mr J is done17:52
jrolllucasagomes: do I need to anything on my end from here or do core reviewers perform the merge?17:52
jroll(this is my first openstack contribution)17:52
NobodyCamjroll: have you signed the contrib agrement17:53
jrollyep17:53
NobodyCam:)17:53
lucasagomesjroll, nop, after it's approved jenkins will launch a job to verify and merge the change17:53
digambar_Hey I am working on https://bugs.launchpad.net/ironic/+bug/127639317:53
jrollcool, thanks17:53
NobodyCamhi digambar_ :)17:54
digambar_let paste here command for node-update17:54
digambar_openstack@openstack:/opt/stack$ ironic node-update ab49e735-af2b-484d-940c-841c66b8fcaf replace address=MAC Couldn't apply patch '[{'path': '/address', 'value': u'MAC', 'op': 'replace'}]'. Reason: can't replace non-existant object 'address'17:54
lucasagomesjroll, np and welcome to openstack, now you're an active contributor :D17:54
NobodyCamdigambar_: port-update not node-update17:54
jroll:D thanks lucasagomes17:55
digambar_ohh17:55
digambar_sorry17:55
NobodyCam:) NP17:55
lucasagomesdigambar_, btw, port-update should fail if you submit that value ("MAC") there's a regex checking the address syntax17:56
NobodyCamoh good point needs to be mac address ("aa:bb:cc:dd:ee:ff") like17:57
lucasagomesyup17:58
digambar_yep17:58
digambar_http://paste.openstack.org/show/64327/17:59
digambar_check this17:59
digambar_This is the expected behavior17:59
NobodyCamlol yes if somehting else running / accessing the node..18:00
digambar_ok18:00
NobodyCamdigambar_: is that uuid the ports uuid or the nodes?18:00
digambar_Then in this case it should say like this18:00
digambar_right?18:00
lucasagomesdigambar_, is it related to the bug ur fixing? if so yes... you shouldn't be able to change the address of a port if the node that contains that port is locked18:01
digambar_yes18:01
digambar_so let me describe what I understand18:04
digambar_port is locked then we cannot change the address18:05
lucasagomess/port/node18:06
NobodyCambrb ... quick walkies18:07
lucasagomesdigambar_, yea sometimes the node is locked, meaning it's doing some operation (for e.g being deployed)18:07
*** JayF has joined #openstack-ironic18:07
lucasagomesso that should prevent us from updating the address of the ports contained in that node18:07
*** aignatov_ is now known as aignatov18:07
digambar_ok18:07
lucasagomesbecause the address is needed for the deploy18:07
lucasagomesit's used to generate the pxe config files18:08
digambar_yes18:08
lucasagomesand also the address is passed to neutron in order to make it asnwer the dhcp request for the pxe boot18:08
digambar_okk18:09
digambar_Then this behavior is expected right?18:09
digambar_in case of deployment18:09
lucasagomesyes18:09
digambar_What is the fix is expected here ?18:09
lucasagomesdigambar_, by reading that bug... it expected an error (NodeLocked) if you try to update the port address when the node that contains that port is locked18:11
lucasagomesthat's the first thing18:11
lucasagomesbut it also mention triggering the neutron updates etc18:11
lucasagomesit can be done in another patch, because the neutron part is not ocmplete yet (almost)18:11
digambar_okay18:12
digambar_Should I update it or leave it till the neutron patch ?18:13
lucasagomes digambar_ I see it as 2 diff problems, so you can split the fix in 2 patches18:14
lucasagomesone to check for the lock18:14
lucasagomesand a second one to add the neutron integration18:14
digambar_okay18:14
JayFHas anyone done any work on creating a Chef cookbook for Ironic yet? I'm looking on stackforge and not seeing anything, but wanted to ensure it wasn't somewhere else before diving in.18:15
digambar_I'll start working one first one18:15
lucasagomesdigambar_, ack :) thanks18:16
lucasagomesJayF, I don't know if there's any chef cookbook18:16
lucasagomesthere's a puppet module tho (https://github.com/stackforge/puppet-ironic)18:16
lucasagomesJayF, also dunno about anybody working on it18:17
digambar_NP luca18:17
digambar_Thanks to you18:17
JayFYeah, I suspect not given what I've looked around for. I will start looking into it some over the next bit. Have to become a bit more familiar with the overall openstack cookbook ecosystem.18:17
lucasagomesJayF, :) ack18:20
digambar_Hey luca, NobodyCam18:25
digambar_I'll be on leave from tomorrow till sunday18:26
digambar_Will be back by monday18:26
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Add topic for change_node_maintenance_mode() RPC method  https://review.openstack.org/7272418:27
*** aignatov is now known as aignatov_18:28
lucasagomesdigambar_, nice! holidays?18:28
digambar_Thanks :)18:28
lucasagomesenjoy it ^^18:29
digambar_yep18:29
*** mdurnosvistov_ has joined #openstack-ironic18:29
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Add topic to the change_node_maintenance_mode() RPC method  https://review.openstack.org/7272418:30
lucasagomesand I'm done for today :) (getting late here, and I'm hungry heh)18:31
NobodyCamok night lucasagomes18:31
lucasagomeshave a good night/afternoon everyone18:31
lucasagomesNobodyCam, good night18:31
*** tatyana has quit IRC18:33
*** lucasagomes has quit IRC18:33
NobodyCamanyone tested 7046818:36
*** epim has joined #openstack-ironic18:37
*** digambar_ has quit IRC18:44
*** romcheg1 has joined #openstack-ironic18:45
NobodyCamromcheg: max_lobur_afk do you have any time to review https://review.openstack.org/#/c/70468/18:48
*** jistr has joined #openstack-ironic18:59
openstackgerritRohan Kanade proposed a change to openstack/ironic: Implement the SeaMicro Power driver  https://review.openstack.org/7071919:09
openstackgerritRohan Kanade proposed a change to openstack/ironic: Implements SeaMicro VendorPassThru functionality  https://review.openstack.org/7072019:09
*** max_lobur has joined #openstack-ironic19:09
romcheg1NobodyCam: will take a look19:10
NobodyCam:)19:11
*** coolsvap has quit IRC19:17
openstackgerritA change was merged to openstack/ironic: Add git dependency to quickstart docs  https://review.openstack.org/7270919:17
NobodyCamjroll: you are now a ATC ^^^19:17
jroll\o/19:19
NobodyCam:)19:19
*** rloo has quit IRC19:20
*** jrist has quit IRC19:20
*** rloo has joined #openstack-ironic19:20
*** pquerna has quit IRC19:24
*** igor___ has quit IRC19:29
*** pquerna has joined #openstack-ironic19:31
*** pquerna has quit IRC19:36
*** rloo has quit IRC19:40
*** rloo has joined #openstack-ironic19:41
*** pquerna has joined #openstack-ironic19:41
*** rloo has quit IRC19:46
*** rloo has joined #openstack-ironic19:46
*** rloo has quit IRC19:47
*** rloo has joined #openstack-ironic19:48
*** saju_m has quit IRC19:49
romcheg1NobodyCam: had to -1 because of node['id'], sorry19:50
*** saju_m has joined #openstack-ironic19:51
max_loburcould someone paste proper keystone authorization section for ironic.conf ?19:52
romcheg1max_lobur: Give me a second19:53
max_loburk, thanks19:53
NobodyCamahh19:54
NobodyCamlooks19:54
romcheg1max_lobur: http://paste.openstack.org/show/64357/19:56
romcheg1Replace the IP addresses with your ones19:56
max_loburty!19:57
*** mdurnosvistov_ has quit IRC19:59
NobodyCamromcheg1: fixing now!20:01
*** jistr has quit IRC20:02
*** thedodd has quit IRC20:02
romcheg1NobodyCam: Sorry if I'm too pedantic :)20:02
max_lobur'X-Auth-Token: <function <lambda> at 0x33d2668>'20:03
max_loburromcheg1: have you seen that?20:03
NobodyCam:)20:04
max_loburwhat am I missed?20:04
*** saju_m has quit IRC20:04
* max_lobur tries to revive his old env20:04
romcheg1max_lobur: I remember that bug20:04
romcheg1It was fixed a while ago20:04
max_loburI remember something like that too20:04
max_loburbut don't remember how to fix20:04
*** saju_m has joined #openstack-ironic20:04
NobodyCamromcheg1: pep 257 for single line doc strings too?20:05
romcheg1NobodyCam: Let me refresh that in my head20:07
romcheg1No, I was wrong!20:07
NobodyCamI think its mainly multi line20:07
romcheg1yup20:08
max_loburromcheg1: could you check what keystone branch do you use and paste keystone.conf as well :)20:10
romcheg1I believe I use master because it's devstack20:10
romcheg1yup, it's master20:11
romcheg1As far as I remember the problem was not in keystone20:13
NobodyCamromcheg1: running tests now20:13
romcheg1The header is set in the keystone client20:13
max_loburyup, not in keystone20:14
max_loburhit it on master as well20:15
romcheg1it might be old keystone client20:15
romcheg1did you try to update it?20:15
max_loburnope, at master20:16
max_loburup to date20:16
max_loburhmm20:16
max_loburgah20:16
max_loburit's installed not as "develop"20:17
romcheg1That explains the problem20:17
*** Alexei_987 has quit IRC20:20
*** epim has quit IRC20:22
*** harlowja_away is now known as harlowja20:24
max_loburyup, it's fixed20:25
max_loburthanks!20:25
openstackgerritChris Krelle proposed a change to openstack/ironic: Implement _update_neutron in PXE driver  https://review.openstack.org/7046820:25
NobodyCamromcheg1: max_lobur ^^^^20:25
* max_lobur looking20:25
NobodyCam:)20:26
NobodyCamTy20:26
romcheg1NobodyCam: looks good but I'm going to wait Jenkins20:31
NobodyCamyes!!!!20:31
NobodyCam:)20:32
max_loburI have 1 suggestion, going to check if it will look better20:32
*** saju_m has quit IRC20:32
romcheg1Just one question: why {'node': node.uuid} instead of node=node.uuid ?20:33
romcheg1I don't think that's a reson for -1, just curious20:33
NobodyCamromcheg1: TypeError: _log() got an unexpected keyword argument 'node'20:34
romcheg1Hmm20:34
NobodyCam:-p20:34
romcheg1It's late here, so I can suggest stupid things, nevermind :)20:35
NobodyCami'm happy to fix with subsequent patches20:35
NobodyCamwe need this :-p20:35
* max_lobur still looking20:42
*** rloo has quit IRC20:42
*** rloo has joined #openstack-ironic20:42
NobodyCam:) watching tc meeting atm20:42
romcheg1*is watching Hobbit in Spanish*20:45
*** epim has joined #openstack-ironic20:48
rlooNobodyCam: maybe I am tired, but _get_node_vif_ids() looks odd: https://review.openstack.org/#/c/70468/8/ironic/drivers/modules/pxe.py20:51
NobodyCamone sec rloo in tc meeting20:52
rloothe return is under the for loop. so if the first if r.node.id == node.id is not true, will result be returned (well, it'll err cuz result will be undefined)20:52
rlooNobodyCam: no worries, I'll add a comment to the review.20:53
max_loburrloo20:54
max_loburyou're right)20:54
max_loburI almost published that =)20:54
romcheg1Oh, nice catch rloo :)20:54
romcheg1At least 3 people didn't notice that :)20:55
rloomax_lobur. thx. I thought maybe I was tired cuz it didn't look right.20:55
max_loburnope20:55
max_loburcheck my comment20:55
*** aignatov_ is now known as aignatov20:55
rloomax_lobur. your comment is more complete than mine ;)20:57
max_lobur:)20:58
romcheg1max_lobur: for..else. Maybe we should avoid that? :)20:58
rlooalthough max_lobur, i'm not sure your solution is correct either.20:58
*** mdurnosvistov_ has joined #openstack-ironic20:59
*** jdob_ has joined #openstack-ironic21:01
max_loburwhy21:02
NobodyCamok back21:03
rloomax_lobur. I'm not sure if at most r.node.id == node.id. Do you know?21:03
max_loburromcheg1: well21:03
rloomax_lobur. I mean, at most 1 occurs.21:03
max_loburwe may replace it with if node_resource is None: raise21:03
max_loburrloo, there should be only one resource with such node21:04
max_loburlemme find on github21:05
rloomax_lobur. if there is at most one resource, then won't the orig code work, but with the return indented?21:05
rloomax_lobur. in this case, your code will work, but it seems overly complicated ;)21:06
max_loburhttps://github.com/openstack/ironic/blob/master/ironic/conductor/resource_manager.py#L66-L7221:06
max_loburhttps://github.com/openstack/ironic/blob/master/ironic/conductor/task_manager.py#L124-L12621:07
* devananda tries to read scrollback21:07
devanandahi guys! i'm sorta back :)21:07
NobodyCamhi devananda21:07
max_loburhttps://github.com/openstack/ironic/blob/master/ironic/conductor/task_manager.py#L14121:08
max_loburwell21:08
NobodyCamwe are trying to work on 7046821:08
max_loburduplicate node ids are possible :(21:08
max_loburI thought we have set() there21:08
NobodyCamdevananda: see current comments on https://review.openstack.org/#/c/70468/8/ironic/drivers/modules/pxe.py21:08
max_loburthere should be set21:08
*** hstimer has joined #openstack-ironic21:09
max_loburas for my example21:14
max_loburif we have no node with such node_id acquired in task - there will be an excetpion (this should not happen, maybe only on debugging stage)21:15
*** jdob_ has quit IRC21:15
max_loburif we have more than one node with such node_id - it will select the first21:15
max_loburlast one is not normal too, but will be fixed if we change task_manager.resources to set21:16
max_loburthis is how I imagine it…21:16
devanandaok, caught up on scroll back21:17
devanandaanesthesia is still wearing off... going rather slowly at the moment...21:17
NobodyCam:(21:18
devanandaNobodyCam: looking at comments...21:18
devanandamax_lobur, rloo - so your comments on https://review.openstack.org/#/c/70468/8/ironic/drivers/modules/pxe.py are spot on21:20
devanandathis is a result of the awkwardness that we discussed last week -- we sort of support milti-node tasks, but we never really use them21:20
max_loburyep21:21
max_loburI totally agree21:21
max_loburdata structures does not correspond to real use cases21:21
devanandahow about refactor and just reject multinode tasks for now in this method21:21
max_loburI can manage refactoring21:22
max_loburlemme comment how it would be without multi node tasks21:23
devanandaNobodyCam: I see you tossed up the last rev of that patchset. if either you or max_lobur want to keep working on it, it's fine with me21:23
NobodyCamdevananda: are you in projects meeting?21:24
NobodyCamalso see pm21:24
*** thedodd has joined #openstack-ironic21:30
*** max_lobur has quit IRC21:31
*** max_lobur has joined #openstack-ironic21:32
max_loburNobodyCam: i just posted the last comment21:32
NobodyCamok in project meeting now21:32
*** anniec has joined #openstack-ironic21:39
NobodyCammax_lobur: feel like pushing up a new rev :-p21:39
NobodyCamhhehehe21:39
NobodyCamstill in project meeting21:39
*** anniec has quit IRC21:40
NobodyCami need to get some food21:47
NobodyCambbiafm21:47
max_lobur:)21:50
devanandaback from project meeting21:58
devanandamostly talked about graduation rquirements21:58
NobodyCamdevananda: if you were writing a bash script that updates projects, like nova, with patches would you have this script update more then one project? oh say also pull ironic patches for neutron, etc.. ???22:02
*** harlowja has quit IRC22:03
devanandaNobodyCam: no22:05
devanandaNobodyCam: i would have one script for each project22:06
NobodyCam:)22:06
devanandaso max_lobur, are you working on next iteration of 70468 or shall I?22:11
max_loburno, I'm not22:13
devanandak22:13
max_loburi thought NobodyCam going to finish it22:13
devanandaoh! NobodyCam, are you ^ ?22:13
max_loburnoticed very annoying thing about our tempest testing :(22:14
NobodyCamdevananda: I can22:14
NobodyCamdevananda: do you agree with max on removeing the node parm?22:15
*** anniec has joined #openstack-ironic22:16
max_loburin tempest I checked for ironic chassis-update 80302e6c-de58-49ba-91c0-63f079cc6537 replace uuid=TEST scenario22:24
max_loburasserted that it will respond 40022:24
max_loburbecause uuid cannot be replaced22:24
max_loburit responded 400 =)22:25
max_loburInvalidUUID: Expected a uuid but received TEST.22:25
NobodyCammax_lobur: we validate that is looks like a uuid22:25
NobodyCamlike with mac addresses22:25
max_loburwill need to check messages too22:25
*** aignatov is now known as aignatov_22:26
*** jbjohnso has quit IRC22:26
max_loburI think it should be "Chassis UUID cannot be replaced" prior to all other things22:26
*** jrist has joined #openstack-ironic22:26
NobodyCamit cant,22:27
NobodyCamwhy not22:27
max_loburironic chassis-update 80302e6c-de58-49ba-91c0-63f079cc6537 replace uuid=80302e6c-de58-49ba-91c0-63f079cc654422:27
max_lobur(second uuid does not exist)22:28
max_lobur(but valid - not malformed)22:28
NobodyCamoh22:28
*** romcheg1 has quit IRC22:28
NobodyCamya can cant change a uuid like that22:28
NobodyCami was thinking a nodes chassis uuid22:28
*** romcheg1 has joined #openstack-ironic22:28
max_loburHTTP/1.0 404 Not Found22:30
max_loburChassisNotFound: Chassis 80302e6c-de58-49ba-91c0-63f079cc6544 could not be found.22:30
max_loburthe messages are misleading22:30
max_loburwill fill a bug for this22:30
max_loburNobodyCam: yup22:30
max_loburthis is about replacing primary key22:30
NobodyCam:-p22:31
*** mdurnosvistov_ has quit IRC22:34
*** max_lobur1 has joined #openstack-ironic22:35
*** max_lobur has quit IRC22:35
*** matty_dubs is now known as matty_dubs|gone22:52
*** harlowja has joined #openstack-ironic23:09
*** rloo has quit IRC23:11
*** rloo has joined #openstack-ironic23:12
*** anniec has quit IRC23:29
*** anniec has joined #openstack-ironic23:32
*** mrda is now known as mrda_away23:37
*** thedodd has quit IRC23:37
max_lobur1night folks23:37
*** max_lobur1 has quit IRC23:38
NobodyCamnight max_lobur_afk23:39
*** jdob has quit IRC23:40
devanandag'night, max_lobur_afk23:42
openstackgerritChris Krelle proposed a change to openstack/ironic: Implement _update_neutron in PXE driver  https://review.openstack.org/7046823:43
NobodyCamdevananda: max_lobur_afk romcheg ^^^^^23:44
NobodyCamif your around23:44
NobodyCamrloo: didn't mean to forget you :-p23:47
*** romcheg1 has quit IRC23:47
* devananda looks23:49
*** ndipanov has quit IRC23:57
« Monday, 2014-02-10 Index Wednesday, 2014-02-12 »

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