Wednesday, 2015-09-30

rloojlvillal, mgagne: I didn't really follow what you were doing, but is this somewhat similar: https://github.com/openstack/oslo-incubator/blob/master/tools/clone_openstack.sh00:00
*** openstackgerrit has quit IRC00:01
jlvillalrloo: I think it is openstack re-inventing repo :)  Sort of.00:01
mgagnerloo: repo is a tool written for the android project and looks to offer more features00:01
mgagneyea00:01
jlvillalrloo: But in bash :(00:01
mgagneand I don't want to clone the world00:01
*** openstackgerrit has joined #openstack-ironic00:02
mgagneI know you can pass in projects but it's a manual work and you can't organize them as you want00:02
*** openstackstatus has quit IRC00:02
*** openstack has joined #openstack-ironic00:04
jlvillalmgagne: I think you can also use 'repo init -g ironic ....'00:05
jlvillalmgagne: Since you created an ironic group00:05
mgagneyou need to have a manifest first which my tool generates00:05
mgagneall depends on your taste =)00:06
jlvillalmgagne: Right. But if had your everything manifest, could use the -g option00:06
mgagnetrue00:06
*** devlaps has quit IRC00:09
openstackgerritMerged openstack/ironic: Fix documentation for installing mariaDB  https://review.openstack.org/22903200:09
openstackgerritMerged openstack/ironic: Fix documentation for installing mariaDB  https://review.openstack.org/22903200:09
*** Sukhdev has quit IRC00:09
*** penick has quit IRC00:11
jlvillalMerge it twice!00:12
jlvillalOr maybe that is my ZNC client going funky....00:12
rloojlvillal: no, it is twice. cuz that is such an important patch! (Seriously I've seen it do that before.)00:13
jlvillal:)00:13
rloociao jlvillal, talk to you tomorrow.00:14
*** rloo has left #openstack-ironic00:14
jlvillalrloo: ciao!00:14
*** karimb has quit IRC00:14
*** amotoki has joined #openstack-ironic00:18
*** Sukhdev has joined #openstack-ironic00:19
*** tsekiyam_ has joined #openstack-ironic00:19
*** mtanino has quit IRC00:19
*** amotoki has quit IRC00:22
*** shadower has quit IRC00:23
*** shadower has joined #openstack-ironic00:23
*** tsekiyama has quit IRC00:23
*** tsekiyam_ has quit IRC00:23
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP  https://review.openstack.org/22919600:34
*** sdake_ has joined #openstack-ironic00:38
openstackgerritJohn L. Villalovos proposed openstack/ironic: Add 'whitelist_externals = bash' for two testenvs  https://review.openstack.org/22919700:39
jlvillaljroll: I think ^^^ is kind of important00:39
jlvillalAt least it was failing for me.00:40
jlvillalHaomeng: Maybe you can look at it too? ^^^^00:40
jlvillalhttps://review.openstack.org/22919700:41
*** sdake has quit IRC00:43
openstackgerritMerged openstack/python-ironicclient: Fix the bug of incorrect spelling  https://review.openstack.org/22873700:43
Haomengjlvillal: sure:)00:43
jlvillalHaomeng: Thank you much. Maybe you can reproduce the: tox -epep8 error I was seeing locally without the patch00:43
* jlvillal is unsure how it snuck through the gate00:43
Haomengjlvillal: ok, will do, thanks:)00:44
*** naohirot has joined #openstack-ironic00:45
*** Sukhdev has quit IRC00:45
*** Sukhdev has joined #openstack-ironic00:46
*** sdake_ has quit IRC00:53
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP  https://review.openstack.org/22919600:55
openstackgerritJohn L. Villalovos proposed openstack/ironic: Add 'whitelist_externals = bash' for two testenvs  https://review.openstack.org/22919701:00
jlvillaljroll, Haomeng: Strange, the pep8 test says it passes at the bottom, but it doesn't actually run the test. Submitted bug on this and added it to the patch.01:00
*** Sukhdev has quit IRC01:03
jlvillaljroll, Haomeng: Okay. Seems like pep8 does work. So false alarm. Error message being printed doesn't mean it isn't running pep8 tests.01:04
Haomengjlvillal: yes, it is strange01:04
Haomengjlvillal: some time, local pep8 != jekins pep8 behavior01:05
*** saripurigopi has quit IRC01:05
*** saripurigopi has joined #openstack-ironic01:05
jlvillalHaomeng: Could be. So I think that it is just a warning message. So my patch is not critical. PEP8 tests are being done correctly as I put in PEP8 errors and it did fail.01:07
jlvillalWithout my patch.01:07
jlvillalSo my fix is more of a cosmetic fix to get rid of the warning message.01:07
*** baoli has joined #openstack-ironic01:10
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP: Move unit tests to correct directory  https://review.openstack.org/22919601:10
Haomengjlvillal: good:)01:11
*** praneshp has quit IRC01:15
*** harshs has quit IRC01:16
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP: Move unit tests to correct directory  https://review.openstack.org/22919601:17
*** harshs has joined #openstack-ironic01:20
krotscheckOh, that lovely moment when you make a change and run into your own test threshold01:26
*** saripurigopi has quit IRC01:27
*** morgan is now known as notreallymorgan01:30
*** notreallymorgan is now known as morgan01:30
*** mrda has joined #openstack-ironic01:33
*** amotoki has joined #openstack-ironic01:48
*** harshs has quit IRC01:48
*** amotoki has quit IRC01:53
*** lazy_prince has joined #openstack-ironic01:56
*** sdake has joined #openstack-ironic01:58
*** killer_prince has joined #openstack-ironic02:03
*** zhenguo has joined #openstack-ironic02:06
*** lazy_prince has quit IRC02:07
*** mrda has quit IRC02:27
*** achanda has joined #openstack-ironic02:43
*** dims_ has quit IRC02:57
*** amotoki has joined #openstack-ironic03:02
*** amotoki has quit IRC03:07
*** harshs has joined #openstack-ironic03:16
*** puranamr has joined #openstack-ironic03:17
*** shuquan has joined #openstack-ironic03:19
*** puranamr has quit IRC03:19
*** puranamr has joined #openstack-ironic03:20
openstackgerritOm Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - RPC  https://review.openstack.org/20624303:20
openstackgerritOm Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - API  https://review.openstack.org/20624403:20
openstackgerritOm Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - net  https://review.openstack.org/20624503:20
openstackgerritOm Kumar proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326203:21
openstackgerritOm Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - DB  https://review.openstack.org/20623203:21
openstackgerritOm Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - objs  https://review.openstack.org/20623803:21
openstackgerritOm Kumar proposed openstack/ironic: Add network provider interface and implementations  https://review.openstack.org/13968703:21
*** puranamr has quit IRC03:23
*** puranamr has joined #openstack-ironic03:24
*** puranamr has quit IRC03:28
*** killer_prince has quit IRC03:29
*** Sukhdev has joined #openstack-ironic03:35
*** puranamr has joined #openstack-ironic03:35
*** sdake has quit IRC03:38
*** saripurigopi has joined #openstack-ironic03:40
*** sdake has joined #openstack-ironic03:41
*** shuquan has quit IRC03:41
*** amotoki has joined #openstack-ironic03:42
*** shuquan has joined #openstack-ironic03:44
*** harshs has quit IRC03:45
*** sdake has quit IRC03:46
*** puranamr has quit IRC03:53
*** links has joined #openstack-ironic03:53
*** baoli has quit IRC03:54
*** rameshg87 has joined #openstack-ironic03:57
*** dims has joined #openstack-ironic03:57
*** Nisha has joined #openstack-ironic03:59
*** shuquan has quit IRC04:00
*** amotoki has quit IRC04:01
*** shuquan has joined #openstack-ironic04:02
*** dims has quit IRC04:03
*** VikasC has joined #openstack-ironic04:05
*** pradipta has joined #openstack-ironic04:05
*** amotoki has joined #openstack-ironic04:11
*** amotoki has quit IRC04:20
*** ramineni has joined #openstack-ironic04:29
*** amotoki has joined #openstack-ironic04:30
*** ramineni1 has joined #openstack-ironic04:32
*** ramineni has quit IRC04:33
*** ramineni has joined #openstack-ironic04:34
*** links has quit IRC04:36
*** ramineni1 has quit IRC04:36
*** ramineni2 has joined #openstack-ironic04:36
*** ramineni1 has joined #openstack-ironic04:37
*** ramineni has quit IRC04:38
*** ramineni has joined #openstack-ironic04:39
*** ramineni2 has quit IRC04:40
*** ramineni2 has joined #openstack-ironic04:40
*** ramineni1 has quit IRC04:41
*** ramineni1 has joined #openstack-ironic04:42
*** ramineni has quit IRC04:43
*** ramineni has joined #openstack-ironic04:45
*** ramineni2 has quit IRC04:45
*** ramineni2 has joined #openstack-ironic04:46
*** ramineni1 has quit IRC04:46
*** amotoki has quit IRC04:48
*** ramineni has quit IRC04:49
*** ramineni2 has quit IRC04:50
Sukhdevlazy_prince: Ping04:54
openstackgerritShinn'ya Hoshino proposed openstack/ironic: Fix a bug error by passwords includes just numbers  https://review.openstack.org/22038505:03
openstackgerritJohn L. Villalovos proposed openstack/ironic: Move unit tests to correct directory  https://review.openstack.org/22919605:04
*** praneshp_ has joined #openstack-ironic05:05
openstackgerritJohn L. Villalovos proposed openstack/ironic: Move unit tests to correct directory  https://review.openstack.org/22919605:07
*** thiagop has quit IRC05:08
*** sinval has quit IRC05:10
*** afaranha has quit IRC05:10
*** amotoki has joined #openstack-ironic05:11
*** links has joined #openstack-ironic05:12
*** shuquan has quit IRC05:13
*** shuquan has joined #openstack-ironic05:14
jlvillalrameshg87, Have you looked at https://review.openstack.org/#/c/220385 ?  I don't understand how it fixes what the commit says it fixes. Have you looked at it before?05:14
jlvillalIf you have time today to look at it :)05:14
* jlvillal gets ready for sleep...05:15
rameshg87jlvillal: hi05:17
rameshg87jlvillal: will have a look, haven't looked at it before05:18
*** enikanorov has joined #openstack-ironic05:18
*** enikanorov_ has quit IRC05:21
*** shuquan has quit IRC05:25
*** olaph has quit IRC05:26
*** shuquan has joined #openstack-ironic05:26
*** Sukhdev has quit IRC05:29
*** alexpilotti has quit IRC05:33
*** olaph has joined #openstack-ironic05:41
*** lsmola has joined #openstack-ironic05:42
*** lazy_prince has joined #openstack-ironic05:43
*** benhur_marcel has joined #openstack-ironic06:01
*** itsuugo has joined #openstack-ironic06:04
openstackgerritMerged openstack/ironic: Update help strings for DRAC configs  https://review.openstack.org/22897606:12
openstackgerritMerged openstack/ironic: Add 'whitelist_externals = bash' for two testenvs  https://review.openstack.org/22919706:14
*** itsuugo has quit IRC06:28
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Imported Translations from Zanata  https://review.openstack.org/22925806:31
*** shuquan has quit IRC06:35
*** yog_ has joined #openstack-ironic06:35
*** x3k` has quit IRC06:46
openstackgerritNaohiro Tamura proposed openstack/ironic: Add hardware inspection module for iRMC driver  https://review.openstack.org/19648006:48
*** Marga_ has quit IRC06:56
*** Marga_ has joined #openstack-ironic06:56
*** karimb has joined #openstack-ironic06:58
*** Marga__ has joined #openstack-ironic06:59
*** kevinbenton has quit IRC07:03
*** Marga_ has quit IRC07:03
*** jlvillal has quit IRC07:05
*** Marga__ has quit IRC07:18
*** kevinbenton has joined #openstack-ironic07:21
*** Haomeng has quit IRC07:23
*** Haomeng has joined #openstack-ironic07:23
*** alexpilotti has joined #openstack-ironic07:24
*** ifarkas has joined #openstack-ironic07:26
*** itsuugo has joined #openstack-ironic07:34
*** stendulker has joined #openstack-ironic07:35
*** praneshp_ has quit IRC07:36
*** praneshp has joined #openstack-ironic07:36
*** mgoddard has joined #openstack-ironic07:43
*** praneshp has quit IRC07:44
openstackgerritShivanand Tendulker proposed openstack/ironic: Refactor iscsi_ilo driver to use new boot interface  https://review.openstack.org/22927407:46
*** romainh has joined #openstack-ironic07:47
*** achanda has quit IRC07:48
*** alexpilotti has quit IRC07:58
*** benhur_marcel has quit IRC07:58
openstackgerritShivanand Tendulker proposed openstack/ironic: Refactor iscsi_ilo driver to use new boot interface  https://review.openstack.org/21653808:00
*** jistr has joined #openstack-ironic08:02
*** e0ne has joined #openstack-ironic08:03
*** itsuugo has quit IRC08:08
openstackgerritShivanand Tendulker proposed openstack/ironic: Refactor iscsi_ilo driver to use new boot interface  https://review.openstack.org/22928408:13
openstackgerritAnton Arefiev proposed openstack/python-ironicclient: WIP: Support Ironic API under proxy  https://review.openstack.org/22928608:14
openstackgerritAnton Arefiev proposed openstack/python-ironicclient: WIP: Support Ironic API behind proxy  https://review.openstack.org/22928608:15
*** alexpilotti has joined #openstack-ironic08:15
*** ukalifon has joined #openstack-ironic08:15
*** derekh has joined #openstack-ironic08:16
*** athomas has joined #openstack-ironic08:16
openstackgerritImre Farkas proposed openstack/ironic: DRAC BIOS vendor_passthru: enable rebooting the node  https://review.openstack.org/16307108:20
openstackgerritShivanand Tendulker proposed openstack/ironic: Refactor agent_ilo driver to use new boot interface  https://review.openstack.org/21710208:23
openstackgerritShivanand Tendulker proposed openstack/ironic: Refactor iLO driver console interface into new module  https://review.openstack.org/22928408:25
openstackgerritShivanand Tendulker proposed openstack/ironic: Refactor iLO drivers to use new boot interface  https://review.openstack.org/22927408:26
openstackgerritShivanand Tendulker proposed openstack/ironic: Refactor iscsi_ilo driver to use new boot interface  https://review.openstack.org/21653808:28
openstackgerritShivanand Tendulker proposed openstack/ironic: Refactor iLO driver console interface into new module  https://review.openstack.org/22928408:29
openstackgerritShivanand Tendulker proposed openstack/ironic: Refactor agent_ilo driver to use new boot interface  https://review.openstack.org/21710208:29
*** amotoki_ has joined #openstack-ironic08:49
openstackgerritAnton Arefiev proposed openstack/python-ironicclient: WIP: Support Ironic API behind proxy  https://review.openstack.org/22928608:54
*** ukalifon has quit IRC08:58
*** e0ne has quit IRC09:00
*** e0ne has joined #openstack-ironic09:00
*** alexpilotti has quit IRC09:04
*** amotoki_ has quit IRC09:07
*** amotoki has quit IRC09:08
*** dtantsur|afk is now known as dtantsur09:10
dtantsurMorning Ironic09:11
vdrokmorning ironic and dtantsur09:12
dtantsuro/09:12
*** amotoki has joined #openstack-ironic09:13
*** pelix has joined #openstack-ironic09:16
*** amotoki has quit IRC09:18
*** digimonde has joined #openstack-ironic09:22
*** dims has joined #openstack-ironic09:32
*** killer_prince has joined #openstack-ironic09:37
*** lazy_prince has quit IRC09:38
*** dims has quit IRC09:39
*** dims has joined #openstack-ironic09:42
*** amotoki has joined #openstack-ironic09:46
*** enikanorov_ has joined #openstack-ironic09:48
*** marzif has joined #openstack-ironic09:51
*** enikanorov has quit IRC09:52
openstackgerritShuquan Huang proposed openstack/ironic: Fix ubuntu install command in install guide  https://review.openstack.org/22933309:55
*** itsuugo has joined #openstack-ironic09:56
*** amotoki has quit IRC09:58
*** naohirot has quit IRC10:00
*** marzif has quit IRC10:03
*** romcheg has joined #openstack-ironic10:08
*** thrash|g0ne is now known as thrash10:10
*** sambetts_ is now known as sambetts10:10
sambettsMorning dtantsur and everyone else o/10:10
dtantsurmorning sambetts10:11
*** killer_prince has quit IRC10:15
*** lazy_prince has joined #openstack-ironic10:15
*** jasondotstar_afk is now known as jasondotstar10:19
rameshg87morning all10:19
*** Nisha has quit IRC10:19
rameshg87sambetts: any updates from the grub2 issue yesterday ?10:20
*** amotoki has joined #openstack-ironic10:20
sambettsHey rameshg87, I didn't get much further, I checked the grub probe with -vv and the results were the same as with only 1 -v10:21
rameshg87oh :(10:21
sambettsrameshg87: I pulled the tripleo_elements down and the localboot element that they provide works out of the box10:21
rameshg87sambetts: okay10:22
openstackgerritRamakrishnan G proposed openstack/ironic: Add support for in-band cleaning in ISCSIDeploy  https://review.openstack.org/22089810:22
sambettsrameshg87: I really don't get why the grub2 element doesn't work, I found this http://edoceo.com/notabene/grub-probe-error-cannot-find-device-for-root but I don't know if its helpful or not10:24
rameshg87sambetts: yeah, and your /dev was mounted and it was showing /dev/sd*10:25
sambettsrameshg87: yup10:26
rameshg87sambetts: it should have been some other issue10:26
sambettsrameshg87: Yeah it must be something else, I went through the logs line by line and couldn't spot anything that might be triggering it though10:27
rameshg87sambetts: hmm..yeah, we should triage it10:27
sambettsrameshg87: http://git.openstack.org/cgit/openstack/tripleo-image-elements/tree/elements/localboot/install.d/99-boot-helpers#n7 I don't know if the comment here is related10:28
*** ukalifon1 has joined #openstack-ironic10:28
rameshg87sambetts: oh yeah, but I still wonder why it goes through fine for me :)10:29
rameshg87sambetts: may be we need something of that sort there in grub2 element as well10:29
sambettsrameshg87: I wonder that too, I'm surprised that there are 2 elements which essentially do the same thing10:30
rameshg87sambetts: it was me and lucas who wrote the grub2 module when we added localboot support10:30
rameshg87sambetts: but honestly I didn't know another element exists in tripleo-image-elements :)10:30
sambettsrameshg87: I don't know which came first :/10:31
*** MattMan has joined #openstack-ironic10:32
*** MattMan has left #openstack-ironic10:34
*** e0ne has quit IRC10:36
*** trown is now known as trown|relocating10:44
openstackgerritMerged openstack/ironic-inspector: Add .eggs/ to gitignore  https://review.openstack.org/22898510:50
*** VikasC has quit IRC10:53
*** VikasC has joined #openstack-ironic10:53
openstackgerritMerged openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/22906710:54
*** MattMan has joined #openstack-ironic10:57
*** MattMan has left #openstack-ironic10:58
openstackgerritRamakrishnan G proposed openstack/ironic: Add support for in-band cleaning in ISCSIDeploy  https://review.openstack.org/22089811:05
*** killer_prince has joined #openstack-ironic11:07
* rameshg87 goes home11:08
*** rameshg87 has quit IRC11:08
*** baoli has joined #openstack-ironic11:10
*** baoli has quit IRC11:10
*** lazy_prince has quit IRC11:10
*** e0ne has joined #openstack-ironic11:18
*** baoli has joined #openstack-ironic11:18
*** itsuugo has quit IRC11:21
*** MattMan has joined #openstack-ironic11:25
*** baoli has joined #openstack-ironic11:27
*** baoli_ has joined #openstack-ironic11:28
*** pradipta has quit IRC11:29
*** baoli has quit IRC11:32
*** karimb has quit IRC11:33
*** stendulker has quit IRC11:33
*** karimb has joined #openstack-ironic11:34
*** aarefiev has quit IRC11:35
*** stendulker has joined #openstack-ironic11:35
*** aarefiev has joined #openstack-ironic11:38
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Support for getting introspection data in Python library  https://review.openstack.org/22407511:39
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Stop assuming that all shell unit tests are for rules API  https://review.openstack.org/22937811:47
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Copy gitignore file from ironic-inspector  https://review.openstack.org/22937911:50
*** stendulker has quit IRC11:52
*** romainh has left #openstack-ironic11:53
*** romainh has joined #openstack-ironic11:54
*** sinval has joined #openstack-ironic12:02
*** amotoki has quit IRC12:06
*** yog_ has quit IRC12:07
*** ukalifon1 has quit IRC12:10
openstackgerritVladyslav Drok proposed openstack/ironic: Enable radosgw support in ironic  https://review.openstack.org/22938912:12
*** trown|relocating is now known as trown12:29
*** VikasC has quit IRC12:31
*** saripurigopi has quit IRC12:32
*** marzif has joined #openstack-ironic12:43
*** achanda has joined #openstack-ironic12:44
*** itsuugo has joined #openstack-ironic12:46
*** saripurigopi has joined #openstack-ironic12:54
jrollmorning everyone \o12:55
sambettsHey jroll o/12:56
jrollohai sambetts !12:56
*** achanda has quit IRC13:00
*** jaypipes has quit IRC13:00
*** rloo has joined #openstack-ironic13:06
dtantsurmorning jroll!13:09
jrollheya dtantsur :)13:10
dtantsurjroll, could you have one more look at https://review.openstack.org/225092 today please? I think I've fixed what could be fixed :)13:11
openstackgerritShinn'ya Hoshino proposed openstack/ironic: Fix a bug error by passwords includes just numbers  https://review.openstack.org/22038513:11
jrolloh yeah, I meant to come back to that13:11
jrollthanks for reminder13:11
dtantsurcool!13:11
jrolldtantsur: I still want something for the hardware module13:11
jrollmaybe docs + optional-requirements.txt? idk13:11
jrollif the docs are under inspector, that's cool too13:12
jrollI also owe you guys a release :|13:12
trownjroll: its ok we have it building from trunk right now anyways13:14
trownwhich will continue even post release13:15
jrollyeah13:15
jrollbut the first release is the hardest13:15
jrollafter that it's a four line patch that anyone can do13:15
*** killer_prince has quit IRC13:17
dtantsurjroll, something like plugin-requirements.txt?13:18
dtantsurlike in ironic... maybe13:18
jrolldtantsur: yeah, I'm not opinionated to the name13:19
jrolldriver-requirements sounds weird, but is consistent with ironic13:19
jrollfor example debian packagers use driver-requirements13:19
dtantsurwell, ipa does not have "drivers", so I'd stick with plugin-requirements (as in inspector btw)13:19
jrollah, ok13:20
jroll+113:20
* jroll reviews in the meantime13:20
jrollno temp file \o/13:20
dtantsuryep, seems like I managed to do it :)13:21
jrollalso TIL journalctl -b13:21
jrollgood to know13:21
NobodyCamgood morning Ironicers13:23
rloogood morning NobodyCam, jroll, dtantsur13:23
NobodyCamgood morning rloo13:23
dtantsurmorning NobodyCam, rloo13:24
NobodyCamand morning to dtantsur13:24
jrollNobodyCam: rloo: good morning!13:24
NobodyCamalso to jroll13:24
NobodyCam:)13:24
NobodyCam lol13:24
rloodtantsur: here's the first patch I know of that touches code in ironic-lib: https://review.openstack.org/#/c/216478/13:24
*** Nisha has joined #openstack-ironic13:24
NobodyCamI clearlly need more coffee13:24
rloodtantsur: have you heard from ramesh wrt status of those patches? we should probably start using it13:25
rloodtantsur: patches to migrate to ironic-lib i mean.13:25
dtantsurrloo, he was planning to update the whiteboard, seems like he didn't13:26
dtantsuryeah, I'm pretty sure we should switch to ironic-lib asap13:26
jroll++13:26
rloodtantsur: ok. I am regretting offering to fix that bug now. Should have taken jroll's offer to do it :)13:27
dtantsur:)13:27
rloodtantsur: seems like a plot to get me involved with ironic-lib :D13:27
jrolllol, welp13:27
jrollyeah we should block that and move it to ironic-lib13:27
dtantsurjroll, anything else to fix on that patch before I press 'git review'?13:28
*** itsuugo has quit IRC13:28
rloojroll: yeah, am looking at ironic-lib now ;)13:28
jrolldtantsur: /me clicks publish13:28
jrolljust one little nit in the tests13:28
dtantsuryeah, lemme fix13:29
rloooh, jroll: wrt that patch: https://review.openstack.org/#/c/216478/. is it serious enough to backport?13:29
jrollrloo: good question...13:29
jrollrloo: I guess it depends if we think any production environments will be using not-english13:30
rloojroll: if it is, it might make sense to patch in ironic & ironic-lib to make the cherry-pick easier. not a big deal though.13:30
jrollrloo: yeah, that's what I'm thinking13:30
jrollfixing it on master is a requirement for backport13:30
rloojroll: well, we will be fixing it on master. just that the code will have diverged between master and stable13:31
jrollrloo: right, huge divergence, and probably won't be done before liberty final13:31
jrollI say land it on both13:31
rloojroll: right, i was going to say that. not sure when it will land.13:31
jrollrloo: if you put up the ironic-lib patch, maybe dtantsur and I can land both at the same time :)13:32
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Add "logs" and "extra-hardware" inspection collectors  https://review.openstack.org/22509213:33
dtantsurjroll, ^^13:33
jrollthanks13:33
rloojroll: ack13:34
jrolldtantsur: +213:38
dtantsurwoohoo13:38
*** marzif has quit IRC13:43
*** william has joined #openstack-ironic13:44
*** william is now known as Guest4963913:45
*** dims has quit IRC13:49
*** dims has joined #openstack-ironic13:49
openstackgerritRuby Loo proposed openstack/ironic-lib: Use standard locale when executing 'dd' command  https://review.openstack.org/22943413:49
jrollI've never looked at ironic-lib in depth before13:52
* jroll sees things to fix13:52
jrollthanks for the patch ruby13:52
openstackgerritMatthew Treinish proposed openstack/ironic: Workaround eventlet bug with subunit on python 3  https://review.openstack.org/22943613:52
TheJuliaHmm... ipxe seems broken13:52
*** mtreinish has joined #openstack-ironic13:52
TheJuliaor my config is broken :(13:53
jrollruh roh13:53
*** digimonde has quit IRC13:54
* TheJulia nukes everything and runs the magical script to see what results13:55
*** jaypipes has joined #openstack-ironic13:56
jrollI believe we use ipxe in the gate, so it shouldn't get too broken :)13:57
jrolldtantsur: wanna land both of those dd patches?13:59
jrollI have a +2 on both13:59
jrolland then we can backport the one in ironic to liberty14:00
*** Nisha has quit IRC14:01
*** romcheg has quit IRC14:02
rloojroll: wrt ironic-lib, what were you going to fix? cuz i was going to delete TODO and update setup.cfg14:04
jrollrloo: I was going to move the tests directory and use oslo.i18n instead of incubator14:04
jrollI can base mine on top of yours14:05
jroll(if needed)14:05
rloojroll: ok, no need to base, they don't conflict14:05
jrollcool14:05
jrollrloo: maybe add a python 3.4 classifier while you're in there :)14:06
rloojroll: oh, do we want to do that now? cuz ironic isn't.14:06
jrollorly?14:06
*** amotoki has joined #openstack-ironic14:06
jrollmehhhh.14:06
TheJuliajroll: it might be me, I'm trying to add a new feature to bifrost and naturally have changed the config... rolling back to see if I get the same behavior14:06
rloojroll: yeah.14:06
rlooi was going to remove 2.614:06
jrollrloo: don't worry about it then, for now14:06
jroll+114:06
rloojroll: ok14:06
jrollTheJulia: gotcha14:06
openstackgerritRuby Loo proposed openstack/ironic-lib: Update setup.cfg  https://review.openstack.org/22944714:07
TheJuliaJust... none of it was even close to initial boot sequence14:07
*** itsuugo has joined #openstack-ironic14:08
*** saripurigopi has quit IRC14:08
*** amotoki has quit IRC14:10
dtantsurjroll, landing14:11
*** [1]cdearborn has joined #openstack-ironic14:11
jrolldtantsur: cool, thanks14:11
BadCubMorning Ironicville14:12
* jroll passes BadCub the coffee14:12
jrollgood morning!14:12
NobodyCam:) oh sure14:12
NobodyCamlol14:12
BadCubahhhh, coffee. The nectar of the gods!14:12
jroll++14:13
dtantsurmorning BadCub14:13
dtantsurapproved both dd patches, and now I have to go14:14
BadCubhiya jroll and dtantsur hows things?14:14
dtantsursee you later/tomorrow14:14
dtantsurBadCub, good enough :) hope you're fine to14:14
BadCubI am very well, thanks :)14:14
NobodyCammorning and good night dtantsur14:14
jrollg'night dtantsur14:15
*** romcheg has joined #openstack-ironic14:16
*** albertoffb has joined #openstack-ironic14:17
*** dtantsur is now known as dtantsur|afk14:17
*** amotoki has joined #openstack-ironic14:17
*** athomas has quit IRC14:22
*** amotoki has quit IRC14:24
openstackgerritRuby Loo proposed openstack/ironic-lib: Remove TODO  https://review.openstack.org/22945414:26
sambettso/ dtantsur|afk14:26
*** athomas has joined #openstack-ironic14:27
*** amotoki has joined #openstack-ironic14:28
*** mgoddard1 has joined #openstack-ironic14:30
*** mgoddard has quit IRC14:32
*** amotoki has quit IRC14:33
*** amotoki has joined #openstack-ironic14:37
*** [2]Rick has joined #openstack-ironic14:40
openstackgerritMerged openstack/ironic-lib: Use standard locale when executing 'dd' command  https://review.openstack.org/22943414:44
*** tsekiyama has joined #openstack-ironic14:45
*** mtanino has joined #openstack-ironic14:49
NobodyCammorning sambetts :)14:50
*** linggao has joined #openstack-ironic14:54
*** [1]cdearborn has quit IRC14:55
*** jlvillal_noznc has joined #openstack-ironic14:55
* jlvillal_noznc grumbles about his $4/month VPS hosting ZNC being un-reachable this morning...14:55
NobodyCamjlvillal_noznc: :(14:56
jlvillal_nozncNobodyCam, Yeah :(14:56
* NobodyCam uses hpcloud for is irssi :)14:56
jlvillal_nozncNobodyCam, show off! ;)14:57
NobodyCam:p we just happen to run a public cloud :p14:57
sambettso/ NobodyCam14:57
* sambetts uses a raspberry pi 14:58
* jroll injects a plug about rackspace cloud and fanatical support14:58
NobodyCamsambetts: \o/14:58
NobodyCamlol14:58
*** mgoddard1 has quit IRC14:59
rloojroll etc. what are we going to do with specs in approved/ ?15:00
jrollrloo: implement them? :)15:00
rloojroll: so we leave them in approved still? no need to re-evaluate or whatever?15:01
*** r-daneel has joined #openstack-ironic15:01
rloojroll: I'm fine with that. I thought there was a diff process before.15:01
jrollrloo: yeah, that's the goal15:01
jrollthere was, when things were targeted to a release cycle15:01
rloojroll: https://wiki.openstack.org/wiki/Ironic/Specs_Process15:02
rloojroll: should that be updated?15:02
rloojroll: 'Specifications have to be re-proposed for every release. The review may be quick, but even if something was previously approved, it should be re-reviewed to make sure it still makes sense as written. '15:02
jrollrloo: yes, apparently15:02
jrolldocs in too many places15:03
jrollrloo: the readme in ironic-specs should be correct as to the process15:03
rloojroll: oh, then the wiki can point to the readme for more info.15:03
jrollya probably15:03
rloojroll: we need to clarify what 'release' is...15:03
jrollrloo: it should eventually point to devref15:04
jrollrloo: yeah, we should use 'release' for our releases, and 'development cycle' for Liberty, Mitaka, etc15:04
rloojroll: what is 'devref'?15:04
jrollrloo: developer docs, the name comes from nova: http://docs.openstack.org/developer/nova/devref/15:05
jroll'how to develop on ironic'15:05
jrollthat's something I'm going to focus on a lot during M15:05
rloojroll: oh, so you mean our devrefs15:05
jrollyes15:05
rloothx for doing it jroll!15:06
jrollrloo: welcome, I may need some help but am leading the charge :)15:07
*** jistr has quit IRC15:07
jrollrloo: if you don't mind updating that wiki, though, that would be great :)15:07
rloojroll: yeah, i'll update that wiki. might be good if you sent out email about it though.15:07
jrollrloo: thanks, will do15:09
* jroll puts on list15:09
*** sdake has joined #openstack-ironic15:10
*** [1]cdearborn has joined #openstack-ironic15:10
*** Sukhdev has joined #openstack-ironic15:14
Sukhdevlazy_prince: ping15:16
*** dims_ has joined #openstack-ironic15:16
*** itsuugo has quit IRC15:17
*** amotoki has quit IRC15:18
*** dims has quit IRC15:20
*** links has quit IRC15:21
rloojroll: updated. I left a caveat about re-reviewing. https://wiki.openstack.org/wiki/Ironic/Specs_Process#Ironic_Specs_Process15:27
jvmariogood morning ironic15:28
rloomornin' jvmario15:28
linggaoHi ironic folks. how do you setup {u'is_whole_disk_image': True}  for a node? it is a driver_internal_info15:30
NobodyCamgood morning jvmario15:30
jrollrloo: thanks15:30
jrolllinggao: nova/ironic should do it for you; I believe it's instance_info but am not 100% sure15:30
jrolllinggao: I think maybe just ironic does it, not nova15:31
linggaojroll, I got the following error. Error: Node ebe7c567-b5f7-4e42-8618-87020fccceb0 is configured to use the agent_ipmitool driver which currently does not support deploying partition images.15:32
linggaojroll, what does it mean?15:32
*** Sukhdev_ has joined #openstack-ironic15:32
linggaoMy user image has kernel, intird associated with it.15:32
jrolllinggao: that driver does not support partition-based images15:32
jrolllinggao: does your image have separate images for kernel and initrd?15:33
jrollor is it all-in-one?15:33
linggaoI got the images from someone. It has 3 images. a kernal, a ramdsik and a image.15:34
linggaojroll: ubuntu-image.initrd  ubuntu-image.vmlinuz and ubuntu-image.qcow215:35
*** amotoki has joined #openstack-ironic15:35
jrolllinggao: right, that is a partition image, not a whole disk image15:36
jrolllinggao: this should be supported by the end of mitaka, hopefully sooner :)15:36
Sukhdev_lazy_prince: Ping15:36
linggaojroll, got it. so it is not currently supported, even for liberty, right?15:37
devanandag'morning, all15:38
openstackgerritJim Rollenhagen proposed openstack/ironic-lib: Move tests/ under ironic_lib/, use ostestr  https://review.openstack.org/22948315:38
openstackgerritJim Rollenhagen proposed openstack/ironic-lib: Better optimize test runs  https://review.openstack.org/22948415:38
openstackgerritJim Rollenhagen proposed openstack/ironic-lib: Add a cover target to tox  https://review.openstack.org/22948515:38
openstackgerritJim Rollenhagen proposed openstack/ironic-lib: Move to oslo.i18n  https://review.openstack.org/22948615:38
jrollwhee!15:38
NobodyCamgood morning devananda15:38
jrolllinggao: correct :(15:38
linggaojroll, does pxe driver support it?15:38
rloomorning devananda15:39
jrolllinggao: yes, pxe_ipmitool supports partition images. it can also be used with the agent ramdisk, so you just need to switch node.driver15:39
jrolldevananda: morning \o15:39
linggaomorning devananda15:39
linggaojroll, thanks.15:40
jrollno problem15:40
jrollalso added https://review.openstack.org/#/c/229473/ related to the above patches15:40
jrollurgh, merge conflicts :|15:41
*** marzif has joined #openstack-ironic15:41
openstackgerritJim Rollenhagen proposed openstack/ironic-lib: Add a cover target to tox  https://review.openstack.org/22948515:42
openstackgerritJim Rollenhagen proposed openstack/ironic-lib: Better optimize test runs  https://review.openstack.org/22948415:42
openstackgerritJim Rollenhagen proposed openstack/ironic-lib: Move to oslo.i18n  https://review.openstack.org/22948615:42
openstackgerritJim Rollenhagen proposed openstack/ironic-lib: Move tests/ under ironic_lib/, use ostestr  https://review.openstack.org/22948315:42
jrollgerrit doesn't do rebases well eh15:42
jrolldevananda: fyi when this lands we're going to backport it https://review.openstack.org/#/c/216478/15:42
jrollit's already landed in ironic-lib15:42
jrollwe landed it in master for backport easy-peasy-ness15:43
devanandajroll: wdyt of reverting specs/kilo-implemented/new-ironic-state-machine to the content it actually had at the end of Kilo?15:44
devanandawe've been updating it during liberty, but the file is still called "kilo-implemented" ... rather misleading IMHO15:44
devanandaI'd rather we have a separate file saying what was done in Liberty, what we still need to do, tec15:44
jrolldevananda: yeah that spec is weird :/15:45
jrolldevananda: whatever you think it's best, I'm going to dislike any solution :)15:45
devanandahah15:45
devanandafair15:45
*** alexpilotti has joined #openstack-ironic15:46
*** Nisha has joined #openstack-ironic15:48
openstackgerritDevananda van der Veen proposed openstack/ironic-specs: Introduce a "living" document for the state machine  https://review.openstack.org/21213215:51
devanandajroll: updated my spec from last cycle, with changes15:52
jrollthanks15:52
* jroll adds to list15:52
openstackgerritDevananda van der Veen proposed openstack/ironic-specs: Add spec for user-settable Node affinity  https://review.openstack.org/19326315:52
devanandaalso updated that one for Mitaka15:52
*** alexpilotti has quit IRC15:55
*** alexpilotti has joined #openstack-ironic15:55
*** alexpilotti has quit IRC16:01
*** marzif has quit IRC16:04
*** alexpilotti has joined #openstack-ironic16:04
MattManHi, spec question, guessing as liberty is done and dusted , wondering when the mitaka specs directory is being created in https://github.com/openstack/ironic-specs  for Mitaka specs ?16:05
*** mgoddard has joined #openstack-ironic16:06
jvmarioMattMan: i'd guess when the first spec is approved?16:06
MattMangood point16:06
*** dims_ has quit IRC16:06
*** ifarkas has quit IRC16:07
rlooMattMan: actually, we don't need the mitaka directory until there is an implemented/completed spec16:07
jvmario++, new specs go in approved/, right?16:08
*** mgoddard has quit IRC16:08
*** lazy_prince has joined #openstack-ironic16:08
rlooMattMan, jvmario: yes, if you want to propose a spec, put it in approved/16:08
*** mgoddard has joined #openstack-ironic16:08
rlooMattMan, jvmario: and once it is approved, it'll show up in approved/. It'll get moved after completion.16:09
*** jlvillal has joined #openstack-ironic16:10
devananda^ yes16:10
MattManGreat thanks16:11
devanandaoh hey - my oneplus2 arrived! that'll make a good lunchtime distraction16:11
jlvillalAnd maybe I'm back online....16:12
*** zhenguo has quit IRC16:14
jlvillalWoo hoo :)16:15
NobodyCamnice ++16:15
NobodyCamwb jlvillal16:15
NobodyCamhehehehe16:15
* jlvillal is embarrassed to admit that he hadn't paid his VPS bill. In his defense it had previously been setup to auto-pay but that somehow got stopped :(16:15
jlvillaldevananda: Those really exist? ;)16:16
jlvillalAnd lucky you!16:16
devanandajlvillal: yah16:16
jlvillalNobodyCam: Thanks. I setup auto-pay again. Hopefully it sticks :)16:16
*** olaph has quit IRC16:18
*** Nisha has quit IRC16:18
*** trown is now known as trown|lunch16:18
jlvillalAnyone else seeing errors running 'tox -epy34' locally?  I get: UnicodeEncodeError: 'ascii' codec can't encode characters in position 39-41: ordinal not in range(128)16:19
jlvillalI did try mtreinish patch with no joy16:19
sambettsjlvillal: Just trying it out16:20
jlvillalsambetts: Thanks!16:21
openstackgerritMerged openstack/ironic-webclient: Added code coverage output directory to eslintignore  https://review.openstack.org/21520916:23
openstackgerritMerged openstack/ironic-webclient: Added coverage threshold failure  https://review.openstack.org/21381616:23
openstackgerritMerged openstack/ironic-webclient: Removed libs from eslintignore  https://review.openstack.org/21521416:23
*** saripurigopi has joined #openstack-ironic16:23
openstackgerritMerged openstack/ironic-webclient: Removed X-Client header  https://review.openstack.org/21440616:25
openstackgerritRuby Loo proposed openstack/ironic-specs: Use doc8 style checker  https://review.openstack.org/22951116:25
openstackgerritMerged openstack/ironic-webclient: Updated eslint rules.  https://review.openstack.org/21579516:26
openstackgerritMerged openstack/ironic-webclient: Removed gitignore files for previous build system  https://review.openstack.org/21518016:26
*** olaph has joined #openstack-ironic16:26
sambettsjlvillal: py34 passed succesfully for me16:26
jlvillalsambetts: Hmmm :(  I'm on Fedora 22.  You?16:26
sambettsjlvillal: Debian Jessie16:27
jlvillalI'll try in Ubuntu 14.0416:27
sambettsjlvillal: have you --recreated recently? I ran tox -e py34 --recreate to ensure a clean run16:28
jlvillalsambetts: Yeah I did: tox -v -v -epy34 -r16:28
jrolljlvillal: now that you mention it, that reminds me that my comcast autopay keeps breaking16:28
* jroll pays that right quick16:28
jlvillal:)16:28
openstackgerritNate House proposed openstack/ironic-specs: [WIP] Proposes exposing additional api filter options for collections based on db node attributes.  https://review.openstack.org/22951216:29
natoriouswoot!16:29
* jlvillal thinks he needs to setup the wheel cache on his Ubuntu system too... This is slow.16:29
jrolldoeeeet16:29
jlvillalYep\16:30
jrolljlvillal: heh, "payment due by september 25; click here to pay early"16:30
jlvillalLOL. Ouch, that could have been bad16:30
jlvillalsambetts: Same error on Ubuntu 14.04 with latest openstack/ironic16:31
sambettsjlvillal: just ran you exact command and it passed :/16:31
* jlvillal wonders if it is because he likes to do LANG=C16:31
jlvillalsambetts: Can you try for fun, LANG=C tox ......16:32
jrollheh16:32
jrollLANG=en_US.UTF-8 ftw16:32
*** romcheg has quit IRC16:32
jlvillaljroll: I'll try that over here16:33
jrolljlvillal: I can't promise it works, that's just what I always have set16:33
*** dims_ has joined #openstack-ironic16:33
jlvillalWell whoopdy something do.  It worked :)16:34
*** Nisha has joined #openstack-ironic16:34
sambettsjlvillal: running it now16:34
jroll\o/16:34
jrollstepping away for a while16:35
*** harshs has joined #openstack-ironic16:36
sambettsjlvillal: LANG=C broke mine16:36
*** garthb has joined #openstack-ironic16:36
* jroll fixes a thing before stepping away16:36
jlvillalsambetts: Woo hoo. You should keep it that way ;)16:36
jlvillalOkay, maybe not...16:37
jlvillalsambetts: Thanks for the help.16:37
sambetts:D16:37
*** saripurigopi has quit IRC16:37
* jlvillal thinks he will propose a patch to tox.ini to hide the issue16:37
openstackgerritJim Rollenhagen proposed openstack/ironic-lib: Add a cover target to tox  https://review.openstack.org/22948516:37
openstackgerritJim Rollenhagen proposed openstack/ironic-lib: Better optimize test runs  https://review.openstack.org/22948416:37
openstackgerritJim Rollenhagen proposed openstack/ironic-lib: Move to oslo.i18n  https://review.openstack.org/22948616:37
openstackgerritJim Rollenhagen proposed openstack/ironic-lib: Move tests/ under ironic_lib/, use ostestr  https://review.openstack.org/22948316:37
jrollthere we go, bbiab16:38
*** [1]cdearborn has quit IRC16:39
*** romainh has left #openstack-ironic16:42
*** nicodemos has joined #openstack-ironic16:43
*** lsmola has quit IRC16:44
openstackgerritJohn L. Villalovos proposed openstack/ironic: Python 3.4 unit tests fail with LANG=C  https://review.openstack.org/22952016:44
*** lekha||away is now known as lekha16:46
*** killer_prince has joined #openstack-ironic16:48
mtreinishjlvillal: it's likely something with the local python3.4 setup and your locale setting with the unicode encoding16:49
*** devlaps has joined #openstack-ironic16:50
openstackgerritJohn L. Villalovos proposed openstack/ironic: Python 3.4 unit tests fail with LANG=C  https://review.openstack.org/22952016:50
jlvillalmtreinish: I have LANG=C on my system. Worked before os-testr16:50
jlvillalmtreinish: I just proposed the patch above ^^^16:50
mtreinishjlvillal: the distiction before ostestr was used you weren't running with subunit output16:51
mtreinishjlvillal: ostestr just calls testr run --parallel --subunit16:51
mtreinishand then pipes that output into subunit-trace16:51
*** lazy_prince has quit IRC16:51
*** harlowja has quit IRC16:52
jlvillalmtreinish: Okay, well somehow that broke it with LANG=C. I'm just going to work around the issue by forcing it to LANG=en_US.....16:52
*** alexpilotti has quit IRC16:53
mtreinishjlvillal: it's because LANG=C says uses ASCII, but subunit with py34 is probably only expecting unicode16:53
mtreinishbecause python316:53
mtreinishthat's a subunit bug that you should file16:53
*** killer_prince is now known as lazy_prince16:53
mtreinishjlvillal: https://bugs.launchpad.net/subunit16:54
jlvillalmtreinish: Thanks16:54
jlvillalmtreinish: Added subunit to the bug I created for Ironic16:55
*** karimb has quit IRC16:55
mtreinishjlvillal: honestly it sounds like a fedora bug if it's setting LANG=C by default in your env16:56
jlvillalmtreinish: No I do that intentionally. Not Fedora16:56
mtreinishah, ok16:56
mtreinishwhy?16:56
*** alexpilotti has joined #openstack-ironic16:57
*** derekh has quit IRC16:57
jlvillalI have reasons, because of something which I can't remember at the moment :) And have to prepare for meeting in 3 minutes....16:57
mtreinishok, no worries :)16:57
*** achanda has joined #openstack-ironic16:58
mtreinishoh, looking at the traceback again, the write issue might be in subunit-trace not subunit itself16:58
mtreinishwe probably need to test this case a bit mroe16:58
*** cdearborn has joined #openstack-ironic16:59
*** achanda has quit IRC17:01
*** alexpilotti has quit IRC17:03
*** alexpilotti has joined #openstack-ironic17:04
*** Guest49639 has quit IRC17:04
*** william has joined #openstack-ironic17:04
*** alexpilotti has quit IRC17:04
*** william is now known as Guest1560817:04
*** alexpilotti has joined #openstack-ironic17:05
mtreinishjlvillal: when you get a sec can you try: https://review.openstack.org/#/c/229527/ in your local env17:05
Nishajroll, https://review.openstack.org/#/c/228020/ the documentation patch for liberty feature needs review17:06
*** trown|lunch is now known as trown17:07
*** kkoski has joined #openstack-ironic17:09
*** e0ne has quit IRC17:14
jlvillalmtreinish: Will try it, thanks.17:14
*** killer_prince has joined #openstack-ironic17:16
jlvillalmtreinish: That fixes the issue :)17:18
mtreinishjlvillal: ok, cool. Now to figure out how to unit test it so I can land the patch17:18
jlvillalmtreinish: Thanks!17:19
mtreinishjlvillal: I'll look at that after lunch17:19
mtreinishI'll push a bugfix release as soon as it lands17:19
*** lazy_prince has quit IRC17:20
*** Sukhdev_ has quit IRC17:24
*** shakamunyi has quit IRC17:35
sambettsCalling it a day, see everyone tomorrow o/17:36
jrollNisha: will check it out, thanks for the patch :D17:36
Nishajroll, thanks17:36
*** achanda has joined #openstack-ironic17:37
*** Sukhdev has quit IRC17:39
kkoskiI just filed this bug for nova, but i figured it might interest you guys as well: pad.lv/150144017:39
kkoskiam i correct in thinking that it would be useful to have nova hypervisors get created from ironic nodes with a human-readable name?17:40
*** killer_prince has quit IRC17:42
*** garthb has quit IRC17:46
*** pelix has quit IRC17:51
*** praneshp has joined #openstack-ironic17:52
jrollkkoski: ironic node name is completely optional, and is editable17:55
jrollI don't think it's a good canonical reference for Nova17:55
jrollkkoski: what's your use case for that to be human readable?17:56
jrollNisha: reviewed17:57
NishaThanks jroll rloo for the review17:58
jrollwelcome :D17:58
* Nisha looking at the comments and hope to post the patch soon17:58
*** Marga_ has joined #openstack-ironic17:59
*** Marga_ has quit IRC17:59
*** Marga_ has joined #openstack-ironic18:00
*** shakamunyi has joined #openstack-ironic18:01
*** amotoki has quit IRC18:09
*** harlowja has joined #openstack-ironic18:13
*** e0ne has joined #openstack-ironic18:13
*** openstackgerrit has quit IRC18:16
*** openstackgerrit has joined #openstack-ironic18:17
*** penick has joined #openstack-ironic18:21
linggaojroll, can you have both agent_ipmitool and pxe_ipmitool loaded in ironic at the same time?18:22
*** garthb has joined #openstack-ironic18:22
jrolllinggao: yes, that goes for all drivers18:22
*** e0ne has quit IRC18:23
linggaoand you can have one node use agent and another use pxe at the same time?18:23
*** Guest15608 has quit IRC18:24
jrolllinggao: yes18:27
*** Nisha has quit IRC18:27
*** ijw has joined #openstack-ironic18:29
*** e0ne has joined #openstack-ironic18:29
openstackgerritMario Villaplana proposed openstack/ironic: Allow vendor drivers to acquire shared tasks  https://review.openstack.org/22605818:30
*** ijw has quit IRC18:34
*** tsekiyama has quit IRC18:34
*** harshs has quit IRC18:35
*** ijw has joined #openstack-ironic18:37
*** harshs has joined #openstack-ironic18:38
*** Guest15608 has joined #openstack-ironic18:42
*** Nisha has joined #openstack-ironic18:43
*** Guest15608 has quit IRC18:43
*** william has joined #openstack-ironic18:44
*** william is now known as Guest8237018:45
openstackgerritMichael Krotscheck proposed openstack/ironic-webclient: Separated UI and Configuration Logic  https://review.openstack.org/22956818:45
*** dims_ has quit IRC18:46
*** openstackgerrit has quit IRC18:46
*** dims_ has joined #openstack-ironic18:46
*** openstackgerrit has joined #openstack-ironic18:47
kkoskijroll: the use case is doing something like this:18:47
kkoskinova boot --flavor flavor --image image --availability-zone=nova:hostname:SOME_LONG_UUID18:47
jrollkkoski: is that built into nova, where you can target a node via AZ?18:49
*** achanda has quit IRC18:50
kkoskijroll: i believe so. I don't think we're doing anything special in that regard18:51
*** amotoki has joined #openstack-ironic18:51
*** achanda has joined #openstack-ironic18:53
*** Sukhdev has joined #openstack-ironic18:53
*** Sukhdev has quit IRC18:54
*** david-lyle has quit IRC18:55
*** david-lyle has joined #openstack-ironic18:55
*** amotoki has quit IRC18:57
devanandakkoski: yes. it's controllable by policy, so public clouds often limit access to it18:59
devanandajroll: ^18:59
*** Sukhdev has joined #openstack-ironic19:02
*** achanda has quit IRC19:02
Nishajroll, the architecture doc doesnt define deploy process for any of the drivers19:08
Nishajroll, i am really not sure if it is correct place to move the deploy process for each of the drivers there19:08
*** lqu has joined #openstack-ironic19:08
openstackgerritMerged openstack/ironic-specs: Update RAID spec based on what was coded  https://review.openstack.org/22435819:13
*** jlvillal_noznc has quit IRC19:17
*** david-lyle has quit IRC19:22
*** david-lyle has joined #openstack-ironic19:22
*** david-lyle has quit IRC19:35
*** marzif has joined #openstack-ironic19:40
*** penick has quit IRC19:43
*** amotoki has joined #openstack-ironic19:46
*** Marga_ has quit IRC19:50
*** amotoki has quit IRC19:50
*** david-lyle has joined #openstack-ironic19:52
*** penick has joined #openstack-ironic19:53
*** kkoski has quit IRC19:56
jrollkk<tab> dang20:04
jrolldevananda: I knew scheduler hints were a thing; wasn't sure about via --availability-zone, seems weird20:05
jrolldevananda: anyway, I wonder if we could make the scheduler do that translation20:05
* jroll notes as something to think about20:05
jrollNisha: maybe others have opinions, then. seems like an architecture thing to me :)20:05
*** Sukhdev has quit IRC20:06
*** david-ly_ has joined #openstack-ironic20:07
*** david-lyle has quit IRC20:07
*** david-ly_ is now known as david-lyle20:08
jlvillaljroll: rloo: With all the patches going into ironic-lib, does that mean about ready to start using it in Ironic?20:10
*** harshs has quit IRC20:11
jrolljlvillal: yes, we want to switch over asap20:11
rloojlvillal: yeah, we wanted to use it when the lib was first populated.20:11
jlvillalAwesome :)20:11
jlvillaljroll: rloo: If I can help in any way, please let me know.20:11
rloojlvillal: and i had some patch that touched code in the lib, so that is an extra incentive to get ironic refactored to use it.20:11
rloojlvillal: review :)20:11
jroll^20:11
jlvillalStarting ... :)20:12
jrollrloo: we never figured out current status, did we?20:12
rloojlvillal: ramesh is meant to have an ironic patch that uses the lib. when that is avail, it'd be great if you could review that.20:12
jrollI'd love to get someone hacking on it20:12
jrollaha20:12
jroll\o/20:12
jlvillalSounds good. Thanks.20:12
*** harshs has joined #openstack-ironic20:13
*** Marga_ has joined #openstack-ironic20:15
openstackgerritRuby Loo proposed openstack/ironic-specs: Use doc8 style checker  https://review.openstack.org/22951120:17
*** kkoski has joined #openstack-ironic20:18
*** [1]cdearborn has joined #openstack-ironic20:24
*** baoli_ has quit IRC20:26
*** cdearborn has quit IRC20:28
lquHi guys, can I ask a question here?20:29
openstackgerritJohn L. Villalovos proposed openstack/ironic-specs: Use doc8 style checker  https://review.openstack.org/22951120:29
lquI'm new to ironic, but worked on PXE, NFS, iSCSI. I want to know if Ironic supports diskless nodes?20:30
devanandalqu: hi! not yet, but there are others interested in working on that as well20:31
rloothx jlvillal :)20:31
jlvillalrloo: Glad to help!20:31
lquThanks a lot.20:32
lquSo the image copy process is from glance to node's local disk?20:32
jlvillalrloo: In ironic-specs, the 'requirements.txt' sure looks like what should be in a 'test-requirements.txt' for the most part. Somewhat strange to me, but realize it is not a normal project.20:33
rloojlvillal: yeah. I didn't want to touch it though, cuz it meant having to decide which goes where.20:33
jlvillalrloo: Yep!20:33
rloojlvillal: it could be cleaned up but i don't feel like it.20:34
jlvillalk20:34
devanandalqu: it varies by config. it may be: from ironic-conductor to node via an iSCSI connection, or from an image source (glance or HTTP) to the local disk20:35
*** garthb has quit IRC20:38
*** amotoki has joined #openstack-ironic20:40
krotscheckThat lovely moment when your unit tests show that your code actually don't do that thing you say it does.20:45
*** amotoki has quit IRC20:45
openstackgerritJohn L. Villalovos proposed openstack/ironic-lib: Put py34 first in the env order of tox  https://review.openstack.org/22961620:48
openstackgerritMerged openstack/ironic-specs: Use doc8 style checker  https://review.openstack.org/22951120:48
*** linggao has quit IRC20:49
openstackgerritJohn L. Villalovos proposed openstack/ironic-lib: Remove execute permission from exception.py  https://review.openstack.org/22961920:49
*** albertoffb has quit IRC20:49
*** itsuugo has joined #openstack-ironic20:53
*** ndipanov has quit IRC20:53
openstackgerritJohn L. Villalovos proposed openstack/ironic-lib: Update link for where to report bugs  https://review.openstack.org/22962020:53
openstackgerritJohn L. Villalovos proposed openstack/ironic-lib: Remove README.md and update README.rst  https://review.openstack.org/22962220:55
* jlvillal joined the ironic-lib party. Maybe a little late ;)20:55
jlvillalkrotscheck: At least you know you have good unit tests! :)20:55
openstackgerritNisha Agarwal proposed openstack/ironic: Documentation for Swift-less iLO Virtual Media deploy  https://review.openstack.org/22802020:56
* jlvillal likes unit tests even though they are a pain to write at times20:56
*** penick has quit IRC20:56
krtaylorjlvillal, I just wrote one for api utils, just to blowout the cobwebs20:58
lqudevananda: thanks20:59
jlvillalkrtaylor: Awesome!20:59
rloojlvillal: welcome to the party. Not late at all. In fact, you're early :)20:59
jlvillalrloo: I always think of 'Die Hard' when I hear 'Welcome to the party' :)21:00
rloojlvillal: I think that is a movie that I didn't see...21:00
jlvillalrloo: Could be. Back in the 1980s. Bruce Willis21:01
Nishajroll, rloo i addressed most of the comments.21:01
rloojlvillal: Bruce Willis rings a bell... I'm not much of a movie person!21:01
jrollNisha: thanks, I'll try to get back to it soon21:02
*** nicodemos has quit IRC21:02
Nishajroll, i didnt address the architecture comment as i believe adding deploy process of each driver to architecture doc would be wrong21:02
* jlvillal is liking his new 'git to-origin-master' alias. to-origin-master = !git remote update && git reset --hard origin/master --21:02
*** Nisha has quit IRC21:03
*** david-lyle has quit IRC21:03
*** david-ly_ has joined #openstack-ironic21:03
*** Nisha has joined #openstack-ironic21:03
openstackgerritRuby Loo proposed openstack/ironic-specs: Reorganize the specs  https://review.openstack.org/22962521:04
*** Guest82370 has quit IRC21:04
jvmariojlvillal: that looks dangerous :P21:04
*** praneshp has quit IRC21:04
*** david-ly_ is now known as david-lyle21:05
jlvillaljvmario: true21:05
jlvillaljvmario: But you always have 'git reflog'  if you committed your changes at some point.21:05
jrolldevananda: if we had "how this driver deploys a node" in the docs, should that be a driver thing, architecture thing, somewhere else? I tend to think architecture21:05
jrolldevananda: e.g. line 236 here https://review.openstack.org/#/c/228020/6/doc/source/drivers/ilo.rst21:06
jroll(old patchset, comment on later one if you comment there)21:06
openstackgerritRuby Loo proposed openstack/ironic-specs: Reorganize the specs on main page  https://review.openstack.org/22962521:06
*** Sukhdev has joined #openstack-ironic21:08
devanandajroll: good question. a little of both, but I'd lean towards putting most of the info as architecture, with a note and a link in the driver doc21:09
jroll\o/21:09
devanandaso folks looking at "how does ironic work" can find it all, and someone who's like "how does this driver work" also realizes it may be different21:09
jrollyep21:10
*** trown is now known as trown|outttypeww21:12
krtaylorwe need to get jlvillal to start using git review, resistance is futile21:15
jlvillalkrtaylor: Ha!21:16
jlvillalkrtaylor: git push origin HEAD:refs/publish/master ;)21:17
jrollso terrible21:17
jlvillal:D21:17
*** penick has joined #openstack-ironic21:17
krtaylorhehheh21:18
krtaylorjlvillal, I am going through all the ironic* projects and running cover, the coverage reports are pretty good overall21:19
krtaylorI can publish them as a point in time snapshot if youd like21:19
krtaylormight help test/qa discussions21:20
jlvillalkrtaylor: I would be interested in seeing what they look like. I'll let you decide if you want to publish them on a regular basis21:20
jrolljlvillal: https://review.openstack.org/#/c/221494/21:22
krtaylorhopefully the proposed infra service will take over for the regular part21:23
jlvillaljroll: Thanks21:23
jrollnp21:23
*** gabriel-bezerra has quit IRC21:26
*** david-lyle has quit IRC21:26
*** Marga_ has quit IRC21:26
*** david-lyle has joined #openstack-ironic21:29
*** cdearborn has joined #openstack-ironic21:29
*** [1]cdearborn has quit IRC21:34
*** amotoki has joined #openstack-ironic21:34
rloohey jroll, JayF: wrt stuff that Josh was working on, do we need others (non RS) to take over?21:36
jrollrloo: probably, doesn't have to be non-rackspace21:36
jrolle.g. we could throw jvmario at it21:36
*** Sukhdev has quit IRC21:36
rloojroll: I'm fine with jvmario doing it if you all are. Just want to make sure that someone(s) is.21:37
jrollrloo: that's not to say he's doing it, but yeah we need that worked picked up21:37
jvmarioi'll need some time to catch up but that's fine, if anyone wants to take over a specific patch set lmk21:37
jvmarioor, if there's a specific patch i should look at first21:38
rloojvmario: i didn't look to see all the patches. We didn't get zapping in, and there is a cleaning API that didn't get in yet either.21:38
jvmariook21:39
rloojvmario: the zap spec has to be updated though.21:39
rloojvmario: which may affect the clean API.21:39
*** amotoki has quit IRC21:39
*** garthb has joined #openstack-ironic21:41
*** jobewan has joined #openstack-ironic21:42
rloojvmario: let me take a stab at redoing the zap spec. if i don't get to it tomorrow, i'll do it early next week. or i'll let you know that i don't want to do it :)21:43
*** kkoski has quit IRC21:43
rloojvmario: I think it is still fresh in my mind.21:44
*** kkoski has joined #openstack-ironic21:45
*** david-lyle has quit IRC21:48
*** Marga_ has joined #openstack-ironic21:48
jlvillalrloo: I kind of think of each file as a library. So why I said libraries. But more importantly I was trying to come up with a decent description for the project.21:50
rloojlvillal: oh. I think of the whole 'package' as a library.21:51
jlvillalMaybe: 'A common library to be used by various projects in the Ironic ecosystem'21:51
jlvillal??21:51
jlvillalrloo: ^^21:51
rloojlvillal: yeah, that works for me. i don't quite know what will go in that library so don't know that we want to be more specific.21:51
openstackgerritNisha Agarwal proposed openstack/ironic: Documentation for Swift-less iLO Virtual Media deploy  https://review.openstack.org/22802021:51
jlvillalWorks for me. Thanks21:51
Nishajroll, ^^^21:52
jvmariorloo: thanks, sounds good21:52
openstackgerritJohn L. Villalovos proposed openstack/ironic-lib: Remove README.md and update README.rst  https://review.openstack.org/22962221:53
*** david-lyle has joined #openstack-ironic21:54
*** sdake has quit IRC21:55
*** sdake has joined #openstack-ironic21:58
*** Nisha has quit IRC21:58
*** gabriel-bezerra has joined #openstack-ironic21:58
*** garthb has quit IRC22:03
*** penick has quit IRC22:08
*** aarefiev has quit IRC22:09
*** mgoddard has quit IRC22:10
*** tsekiyama has joined #openstack-ironic22:11
*** mgoddard has joined #openstack-ironic22:12
*** Sukhdev has joined #openstack-ironic22:12
*** vdrok has quit IRC22:14
*** yuriyz has quit IRC22:14
*** david-lyle has quit IRC22:15
openstackgerritMerged openstack/python-ironicclient: Allow 'abort' verb for node-set-provision-state  https://review.openstack.org/20218422:15
*** penick has joined #openstack-ironic22:18
*** alexpilotti has quit IRC22:18
*** garthb has joined #openstack-ironic22:19
*** cdearborn has quit IRC22:20
*** alexpilotti has joined #openstack-ironic22:25
*** [2]Rick has quit IRC22:27
*** kkoski has quit IRC22:27
*** alexpilotti has quit IRC22:29
*** devlaps has quit IRC22:30
*** dims__ has joined #openstack-ironic22:30
*** e0ne has quit IRC22:33
*** dims_ has quit IRC22:33
*** david-lyle has joined #openstack-ironic22:36
*** praneshp has joined #openstack-ironic22:40
*** praneshp_ has joined #openstack-ironic22:43
*** praneshp has quit IRC22:44
*** praneshp_ is now known as praneshp22:44
*** thrash is now known as thrash|g0ne22:45
*** alexpilotti has joined #openstack-ironic22:57
*** alexpilotti has quit IRC23:03
*** penick has quit IRC23:04
*** penick has joined #openstack-ironic23:07
*** jaypipes has quit IRC23:07
*** penick has quit IRC23:08
*** marzif has quit IRC23:10
*** tsekiyama has quit IRC23:10
*** tsekiyama has joined #openstack-ironic23:10
*** amotoki has joined #openstack-ironic23:22
*** amotoki has quit IRC23:27
openstackgerritRuby Loo proposed openstack/python-ironicclient: Use doc8 style checker  https://review.openstack.org/22967623:28
*** karimb has joined #openstack-ironic23:29
*** jobewan has quit IRC23:32
*** karimb has quit IRC23:37
jlvillalrloo: Nice find on that doc8 tool23:40
rloojlvillal: thx. It was on my list of things to look into. Also, harlowja wrote that so I thought I would support him :D23:42
jlvillalFirst I had heard of it. Good thing to add I think.23:43
harlowjai didn't do it23:44
harlowjalol23:44
harlowjaha23:44
harlowjaoh23:44
JayFharlowja: it's okay, you can claim this one23:44
jlvillalharlowja: I even submitted a patch for it ;)  Minor grammar issue23:44
harlowjawoot23:44
harlowjaJayF :)23:44
harlowjajlvillal thx, https://review.openstack.org/#/c/229679/ +223:45
jlvillalWoo hoo! :)23:45
harlowja:)23:46
harlowjai aim to please23:46
harlowjalol23:46
openstackgerritRuby Loo proposed openstack/ironic-python-agent: Use doc8 style checker  https://review.openstack.org/22968323:50
jrollharlowja: I read your TC candidacy after merging the first doc8 thing and everything made sense ;D23:56
harlowjajroll i must of messed up somewhere23:58
harlowjait can't make sense23:58
harlowja:)23:58
harlowja*need moar crazy in my letters*23:59
harlowjalol23:59
jrollharlowja: oh no, just the "ruby proposes doc8 change" thing23:59
harlowjaoh23:59
jrollthe rest was all insanity23:59
jrolldon't worry23:59
harlowjalol23:59
jrollA+ would read again23:59
harlowja:-P23:59

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