Thursday, 2014-08-21

rlooJayF: that's why I'm asking. Don't know if you guys didn't want it, or if it was an omission.00:00
jrollJayF: it's a separate interface00:00
jrollit was an omission, rloo00:00
jrollnot intentional00:00
jrolljust... not something we noticed00:00
rloojroll: ok, then. i can fix it. i have the powah.00:00
jroll\o/00:00
jrollhooray for open source.00:00
rloo:D00:01
openstackgerritRuby Loo proposed a change to openstack/ironic: Add console interface to agent_ipmitool driver  https://review.openstack.org/11582600:11
jrollrloo: no tests? :P00:14
*** harlowja has quit IRC00:15
*** harlowja_ has joined #openstack-ironic00:15
openstackgerritGregory Haynes proposed a change to openstack/ironic: More helpful failure for tests on noexec /tmp  https://review.openstack.org/11582700:17
*** christop1eraedo is now known as christopheraedo00:18
rloojroll: nope, cuz there weren't any tests for the other non-agent stuff :-)00:22
jrollhuh, ok00:24
rloojroll: do you want me to ask the folks that wrote the agent stuff, to add tests?00:27
jrollyeah00:28
jrollthose guys suck00:28
rloojroll: ok, will do. Nah, they're nice guys. It is the reviewers that are the problelm.00:28
rloohey jroll: would you please add unit tests for the agent, that test all the interfaces? It only tests the agent-related interfaces now. Some jroll guy wants them.00:29
jrolllol00:30
jrollfile a bug :P00:30
jroll(mostly kidding)00:30
rloojroll: huh, 'mostly', not 'totally'? ;)00:31
jroll:P00:31
greghaynesaye, I need to learn how to db migration00:35
*** penick has quit IRC00:37
*** eghobo has quit IRC00:39
*** penick has joined #openstack-ironic00:39
*** eghobo has joined #openstack-ironic00:39
openstackgerritGregory Haynes proposed a change to openstack/ironic: WIP: Add conductor_hostname property to nodes  https://review.openstack.org/10968800:41
*** penick has quit IRC00:51
*** penick has joined #openstack-ironic00:54
*** jcoufal has quit IRC00:55
*** dlaube has quit IRC01:00
*** penick has quit IRC01:01
*** shakamunyi has joined #openstack-ironic01:17
*** nosnos has joined #openstack-ironic01:44
*** rloo has quit IRC01:51
*** lazy_prince has quit IRC01:53
*** killer_prince has joined #openstack-ironic01:55
*** killer_prince is now known as lazy_prince01:55
greghaynesAny reason why my migration isnt being run in CI on https://review.openstack.org/#/c/109688/5 ?01:59
greghaynesoh, it is running I just fail at writing them ;)02:01
openstackgerritGregory Haynes proposed a change to openstack/ironic: WIP: Add conductor_hostname property to nodes  https://review.openstack.org/10968802:02
*** christopheraedo has quit IRC02:03
*** christopheraedo has joined #openstack-ironic02:04
*** christopheraedo has left #openstack-ironic02:05
openstackgerritEllen Hui proposed a change to openstack/ironic: Make DHCP provider pluggable  https://review.openstack.org/11235102:08
*** lazy_prince has quit IRC02:15
*** killer_prince has joined #openstack-ironic02:16
*** killer_prince is now known as lazy_prince02:16
*** rushiagr_away is now known as rushiagr02:23
*** shakayumi has joined #openstack-ironic02:25
*** shakamunyi has quit IRC02:28
*** alexiz has joined #openstack-ironic02:31
*** shakayumi has quit IRC02:34
*** harlowja_ is now known as harlowja_away02:42
*** eghobo has quit IRC02:53
*** yuanying has quit IRC03:00
openstackgerritHaomeng,Wang proposed a change to openstack/ironic: Add send-data-to-ceilometer support for pxe_ipminative driver  https://review.openstack.org/11248603:04
*** Poornima has joined #openstack-ironic03:08
*** vinbs has joined #openstack-ironic03:09
*** ramineni has joined #openstack-ironic03:15
*** yuanying has joined #openstack-ironic03:16
*** yuanying has quit IRC03:21
*** shakamunyi has joined #openstack-ironic03:54
*** alexiz has quit IRC03:57
*** yuanying has joined #openstack-ironic04:34
*** rameshg87 has joined #openstack-ironic04:38
*** rushiagr is now known as rushiagr_away04:58
*** rushiagr_away is now known as rushiagr05:01
*** rushiagr is now known as rushiagr_away05:01
*** Poornima has quit IRC05:09
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements  https://review.openstack.org/11560005:16
*** pcrews has quit IRC05:16
openstackgerritVinay B S proposed a change to openstack/ironic: Adds network creation instructions  https://review.openstack.org/11488605:17
Haomengvinbs: morning:)05:21
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements  https://review.openstack.org/11560005:25
*** shakamunyi has quit IRC05:37
*** Poornima has joined #openstack-ironic05:43
*** killer_prince has joined #openstack-ironic05:45
*** pradipta_away is now known as pradipta05:45
vinbsHaomeng: morning! :)05:45
Haomengvinbs: :)05:45
*** killer_prince has quit IRC05:46
*** teju has joined #openstack-ironic05:50
*** bmahalakshmi has joined #openstack-ironic05:53
*** dtantsur|afk is now known as dtantsur06:03
dtantsurMorning Ironic06:04
mrdahi dtantsur06:05
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/11587806:08
dtantsurmrda, g'evening06:09
*** rushiagr_away is now known as rushiagr06:20
*** k4n0 has joined #openstack-ironic06:25
openstackgerritVinay B S proposed a change to openstack/ironic: Adds network creation instructions  https://review.openstack.org/11488606:29
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: EXPERIMENTAL Implement hardware discovery setup in PXE driver  https://review.openstack.org/11003106:31
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: Add newly_discovered column to Node object  https://review.openstack.org/10738906:31
*** tatyana has joined #openstack-ironic06:40
openstackgerritRamakrishnan G proposed a change to openstack/ironic: Add support for creating vfat and iso images  https://review.openstack.org/10541306:46
openstackgerritRamakrishnan G proposed a change to openstack/ironic: Add support for interacting with swift  https://review.openstack.org/10579506:50
openstackgerritRamakrishnan G proposed a change to openstack/ironic: IloVirtualMediaIscsi deploy driver  https://review.openstack.org/11386506:54
*** jcoufal has joined #openstack-ironic07:02
openstackgerritRamakrishnan G proposed a change to openstack/ironic: IloVirtualMediaIscsi deploy driver  https://review.openstack.org/11386507:03
*** shakamunyi has joined #openstack-ironic07:04
*** yongli has joined #openstack-ironic07:06
openstackgerritRamakrishnan G proposed a change to openstack/ironic: IloVirtualMediaIscsi deploy driver  https://review.openstack.org/11386507:08
*** shakamunyi has quit IRC07:08
*** athomas has joined #openstack-ironic07:10
*** ndipanov has joined #openstack-ironic07:13
openstackgerritRamakrishnan G proposed a change to openstack/ironic: IloVirtualMediaAgent deploy driver  https://review.openstack.org/11588507:30
yuriyzmorning Ironic07:32
dtantsuryuriyz, morning07:58
*** derekh has joined #openstack-ironic08:16
*** lucasagomes has joined #openstack-ironic08:16
*** viktors|afk is now known as viktors08:28
*** igordcard has joined #openstack-ironic08:48
*** pelix has joined #openstack-ironic08:51
*** Haomeng has quit IRC08:55
openstackgerritVladyslav Drok proposed a change to openstack/ironic: Add driver name on driver load exception  https://review.openstack.org/11204909:04
*** igordcard has quit IRC09:20
*** Haomeng has joined #openstack-ironic09:22
*** igordcard has joined #openstack-ironic09:25
*** yuanying has quit IRC09:31
*** yuanying has joined #openstack-ironic09:35
*** pradipta is now known as pradipta_away09:41
*** yuanying has quit IRC10:10
*** aswadr has quit IRC10:12
*** dtantsur is now known as dtantsur|lunch10:26
*** rameshg87 has quit IRC10:28
*** yuanying has joined #openstack-ironic10:41
*** lazy_prince is now known as killer_prince10:48
*** killer_prince is now known as lazy_prince10:53
*** teju has quit IRC11:00
*** yuanying has quit IRC11:15
*** mikedillion has joined #openstack-ironic11:18
*** Poornima has quit IRC11:29
*** mikedillion has quit IRC11:35
*** mkerrin1 has quit IRC11:38
marioslucasagomes: 'ironic node-set-power-state e99fe1b3-4cca-49e5-9774-7cb164aeece9 on' (or should that be 'power_on' or something else?)11:46
marioslucasagomes: nm sorry got the help message11:47
marioslucasagomes: apologies for the noise11:47
lucasagomesmarios, heh no worries11:47
lucasagomesit's 'on' 'off' 'reboot' afair11:47
marioslucasagomes: \o/ it asks for dhcp service and dnsmasq replies from the 'discovery range' (i.e. the distinct range from the 'normal' undercloud one)11:54
lucasagomesmarios, w00t!11:54
marioswe can play with the tags and see if we can serve up different pxe options as well (should just be getting config right)11:55
lucasagomesmarios, a nice one, yeah... for iPXE I have a test where I check if the pxe request comes from iPXE or standard PXE and if it's standard I send the iPXE boot image so machine boot can chainload it11:58
lucasagomesthe machine booting*11:58
marioslucasagomes: cool, lets talk later11:58
lucasagomesmarios, aight11:58
*** vinbs has quit IRC12:02
*** yuanying has joined #openstack-ironic12:11
*** nosnos has quit IRC12:13
*** octavian1001 has joined #openstack-ironic12:13
*** ramineni1 has joined #openstack-ironic12:14
*** ramineni has quit IRC12:16
dtantsur|lunchmarios, \o/12:30
*** dtantsur|lunch is now known as dtantsur12:30
marios:)12:31
dtantsurnow to something completely different: after we increased default image cache values, I'm seeing complains from people with hard drive overflow12:31
dtantsurI wonder if we should revisit caching logic or revert to lower values12:32
dtantsurlifeless, lucasagomes ^^^12:32
lucasagomesdtantsur, they were running it in production or just testing?12:35
dtantsurlucasagomes, testing. That's why they have little disc space.12:35
dtantsurAnd seems like that *2 logic to account for converting does not always work either :(12:36
lucasagomesdtantsur, right so they have to tune down the config because in OpenStack the defaults should be production defaults12:36
lucasagomesafaiui12:36
dtantsurwell, yes... I wonder if we should set them down in devstack12:37
lucasagomesright... so it's buggy yeah we gotta try to fix it12:37
dtantsurlucasagomes, also see https://bugs.launchpad.net/ironic/+bug/135636212:37
lucasagomesdtantsur, yeah, makes sense in devstack12:37
lucasagomesbecause it's mostly used for tests12:37
*** bmahalakshmi has quit IRC12:43
*** yuanying has quit IRC12:44
*** octavian1001 has left #openstack-ironic12:45
dtantsurhmmm... who are we waiting for approval of https://review.openstack.org/#/c/115493/ ? I doubt we should wait for devananda :)12:45
agordeevgood morning/evening folks! is anyone here familiar with `dracut`?12:57
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Control extra space for images conversion in image_cache  https://review.openstack.org/11597413:00
yuriyzdtantsur, https://review.openstack.org/#/c/115974/13:08
*** rloo has joined #openstack-ironic13:08
dtantsuryuriyz, already looking, thanks a lot!13:10
*** jasondotstar has joined #openstack-ironic13:19
*** pcrews has joined #openstack-ironic13:20
*** openstackgerrit has quit IRC13:21
lucasagomesagordeev, hey... hmm not super familiar but whats up?13:32
*** linggao has joined #openstack-ironic13:34
rloomorning ironickers13:37
lucasagomesrloo, morning13:37
rlooI see quite a few jenkins failures. Does anyone know why?13:37
*** lazy_prince is now known as killer_prince13:38
*** krtaylor has quit IRC13:41
*** yuanying has joined #openstack-ironic13:41
linggaomorning rloo lucasagomes13:45
lucasagomeslinggao, good morning13:46
linggaorloo, I saw your review comments on 97331. Do you really need a bug for it?13:47
linggao(console for ipminative)13:47
rloomorning linggao.13:47
agordeevlucasagomes: it may not too related with ironic at all, but accoring to the last news it becomes capable of loading root live-images via torrents. I'm just wondering how that can possibly help us in a case of provisioning.13:47
rloolinggao: that is adding a new feature. it is at least a bug.13:48
lucasagomesagordeev, nice... actually dracut is pretty cool we could do many things using it13:48
lucasagomeseven generating our ramdisk instead of using DIB13:48
lucasagomesyou can use only dracut13:48
rloolinggao: it isn't difficult to add a bug. be glad no one has asked you to write a spec for it ;)13:49
lucasagomesdef something to keep an eye at13:49
dtantsurrloo, linggao, agordeev, morning!13:49
linggaorloo, I do not mind create a bug for it as long as you guys/girls can approve. I am just afraid that one bug is open, someone will say it needs a spec. This patch has been there for a long time. I just want to finish it up.13:49
rloohi dtantsur!13:49
agordeevdtantsur: morning :)13:49
linggaogood moring dtantsur agordeev13:50
agordeevlinggao: morning too!13:50
rloolinggao: I cannot speak for others. Someone may ask for a spec. that would be regardless if you opened a bug or not though.13:50
agordeevrloo: morning also :)13:50
rloolinggao: I don't think it needs a spec though ;)13:50
jrollmorning ironic :)13:50
rloohi agordeev (not sure what time zone you're in).13:50
jrollagordeev! long time no see13:50
rloomorning jroll13:50
jrollmorning rloo lucasagomes linggao dtantsur13:51
linggaorloo, thanks. I'll open a bug then.13:51
lucasagomesjroll, morning13:51
agordeevjroll: morning, yup! i'm still being too busy with our product. Not sure what to do with my work for IPA13:51
linggaogood morning jroll.13:51
jrollagordeev: all inventory sorts of things have been bumped to kilo, so it will sit some more :P13:52
jrolldtantsur: we could probably approve https://review.openstack.org/#/c/115493/13:52
jrolldtantsur: the only person I would want to check with is mrda, but I'm going to assume he wants it landed13:53
*** datajerk has quit IRC13:53
* jroll jfdi13:53
* NobodyCam wakes up slowly13:54
NobodyCamgood morning Ironic13:54
jrollmorning NobodyCam :)13:55
*** k4n0 has quit IRC13:55
rloomorning NobodyCam13:56
linggaogood morning NobodyCam13:56
NobodyCammorning jroll rloo and linggao :)13:57
*** openstackgerrit has joined #openstack-ironic13:59
*** datajerk has joined #openstack-ironic14:00
dtantsurNobodyCam, jroll, morning14:01
NobodyCamgood morning dtantsur :)14:02
rlooanyone seen gate_tempest_dsvm-virtual-ironic failing in tearDownClass? Looks like it is trying to delete the chassis but it has nodes.14:07
rlooI guess our tempest experts are on vacation this week?14:07
jrollrloo: got a link?14:07
rloojroll: http://logs.openstack.org/92/115692/1/gate/gate-tempest-dsvm-virtual-ironic/0dfc429/logs/testr_results.html.gz14:08
rloojroll: under tearDownClass14:08
rloojroll: this is the first time I'm looking at this tempest page.14:08
jrollthat's a lot of failures14:10
jrollmight be unrelated14:10
rloojroll: I just looked at another patch where that gate fails, but it is failing in tempest in different place.14:10
jrollyeah14:10
jrolltempest fails everywhere :)14:10
jrollhttp://logs.openstack.org/92/115692/1/gate/gate-tempest-dsvm-virtual-ironic/0dfc429/console.html#_2014-08-21_13_25_44_23914:10
jrollfailed 78 tests14:11
rloojroll: do I ignore it? I see quite a few folks doing 'recheck no bug'.14:11
jrollwhich means it's likely something didn't get set up right14:11
jrollwell14:11
jrollwe should try to track down the actual issue14:11
jrolland see if there's a bug, if so do 'recheck bug xxx'14:11
rloojroll: yeah, i know. we should try to track it down :-(14:11
jrollrloo: I'll take a look :)14:11
rloojroll: well, this is good experience. ha ha.14:11
*** chuckC_ has quit IRC14:11
rloojroll: I also saw another gate-docs fail, haven't yet looked into that one. wanted to see how frequent it was failing first.14:12
jrollheh14:12
jrollso with this first one14:12
jrollI usually like to go to the logs, console first (which is what I linked to you above)14:12
jrollsince there's so many failures, something was probably set up wrong14:12
jrollso starting from here: http://logs.openstack.org/92/115692/1/gate/gate-tempest-dsvm-virtual-ironic/0dfc429/14:13
jrollI go to logs, then to devstack-gate-setup-host.txt.gz or devstack-gate-setup-workspace-new.txt.gz14:13
jrollfirst one looks a bit short to me :|14:14
*** yuanying has quit IRC14:14
rloojroll: so do you get the logs from a good session and compare?14:15
jrollrloo: I have in the past, turns out that first log is actually normal14:16
jroll(checked against a good one)14:16
rloojroll: ok. I just got a good one to compare ;)14:17
jrollk :)14:17
*** krtaylor has joined #openstack-ironic14:17
* rloo thinks a tool to smartly-diff the logs between two gate runs might be nice.14:18
jrollhmm, could be14:19
jrollso looking at the tempest logs14:19
jrollthere's a ton of14:19
jroll2014-08-21 13:16:06.648 22733 TRACE tempest.test IdentityError: Got identity error14:19
jroll2014-08-21 13:16:06.648 22733 TRACE tempest.test Details: Unexpected status code 40014:19
*** ChuckC has quit IRC14:20
rloojroll: I was thinking those failures were due to the first failed cleanup.14:23
jrollrloo: I think it's keystone being dumb14:23
*** pradipta_away is now known as pradipta14:26
jrollrloo: I'm having trouble actually finding anything, though, I might just 'recheck no bug' for now14:28
NobodyCamI have a question, what would be the best way of getting our client into https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/contrib/baremetal_nodes.py14:28
jrollNobodyCam: go to sleep and dream about it?14:29
rloojroll: ok thx. I might look a bit more. or not ;)14:29
jroll:P14:29
jrollNobodyCam: you mean, python-ironicclient?14:29
NobodyCamlol. yeppers14:29
rlooNobodyCam: why?14:30
jrollwhy14:30
*** pradipta is now known as pradipta_away14:30
jrollI don't think that's valuable, then we have two places to update14:30
*** rushiagr is now known as rushiagr_away14:30
NobodyCamso we can proxy nova baremetal-node-list, and nova baremetal-show commands14:30
NobodyCamI'm trying not to import https://github.com/openstack/ironic/blob/master/ironic/nova/virt/ironic/driver.py#L51-L7714:31
jrolljust... import the client?14:31
jrollmmmmmmmm14:31
jrolloh14:32
jrollso you're saying best way to import the client14:32
NobodyCamI'm if I import the conf values I end up with a ValueError: Circular reference detected14:32
NobodyCamyes!14:32
jrollgrr circular imports14:32
jrollNobodyCam: can I see the code?14:34
rlooNobodyCam: do we need this for graduation?14:34
jrollalso14:34
jrolldo we import baremetal_nodes in the virt driver?14:34
jrollrloo: yeah :|14:34
lucasagomessomeone knows Stig Telfer IRC name?14:36
rloowhat exactly does 'proxy' mean? That someone has openstack with baremetal, or openstack with ironic, and issuing nova baremetal* commands?14:36
jrollrloo: something like that, it's for upgrade testing iirc14:36
*** jgrimm has joined #openstack-ironic14:37
NobodyCamyes, we only need to handle simple command node-list and node-show14:37
NobodyCamadd node/port can error14:37
rloojroll: would be good to nail down the requirements. Is it just for our testing, or is it something our users will use?14:37
jrollidk14:38
jrollask NobodyCam :P14:38
rloojroll: sorry, i thought maybe you all knew ;)14:38
jrollI don't wanna know :P14:38
NobodyCamnova would like the node-list and node-show command to be proxied, As I understand it, its for testing and users14:39
jrolloh, goody14:39
rlooNobodyCam: so what's the scenario. I haven't actually thought about it. Someone has openstack w baremetal w ironic? Then 1. they use the db-migrate-scripts to get bm node info into ironic db.14:40
NobodyCamjroll: just some initial hacking attempts: http://paste.openstack.org/show/X7F8rz7SsZs4DII4wS5F/14:41
rlooNobodyCam: then 2. they issue nova bm* commands and we proxy to ironic API?14:41
NobodyCamyep, We just need to handle the baremetal-node-list & baremetal-node-show commands14:42
rlooNobodyCam: so its possible to have openstack+bm+ironic at the same time?14:43
NobodyCamno14:44
NobodyCamthis will be for once NBM is removed14:44
lucasagomesNobodyCam, u def have to convert the values coming from ironic to a nova bm node style14:44
lucasagomeslike that commented dict you have there before the return14:44
NobodyCamoh ya!14:45
NobodyCamlucasagomes: my first hack http://paste.openstack.org/show/hNLkJ4Yjb33Z11H9LHJn/14:45
NobodyCamlol14:45
lucasagomesNobodyCam, ah awesome, almost there then ;)14:45
lucasagomes(surprised that there's no tests checking the type of the fields there...)14:46
NobodyCamlol shhhh14:46
rlooNobodyCam: ha ha.14:46
jrollNobodyCam: have the traceback from the circular import?14:46
NobodyCamsure one sec14:46
NobodyCamjroll: http://paste.openstack.org/show/bMfaM5MMKswQLoZKIHpV/14:47
jrollOH14:48
jrollthat's not a circular import14:48
jrollthat's like, bad json or something14:48
NobodyCamI haven't dug into it a whole lot. I hit that at like 5 lastnight14:49
lucasagomes+114:49
lucasagomesNobodyCam, when you do a node = icli.node.list() and then nodes.append(node)14:51
lucasagomesI bet ur appending a list of objects in a list14:52
lucasagomesand that may not be serializable14:52
lucasagomeshence the error ur getting14:52
lucasagomesAFAIUI, you have to get the node.list() and then convert each of the items to the nova bm format14:52
lucasagomesand not append with another list, just return the list of converted items14:52
NobodyCamoh let me test that. I was thinking the errors where comming from the CONF.import_opt stuff14:53
jrolllucasagomes: yeah, sounds right14:54
NobodyCambut first a conf call :-p14:55
lucasagomes:)14:55
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements  https://review.openstack.org/11560014:55
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/11405814:55
*** coolsvap has quit IRC14:58
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: EXPERIMENTAL Endpoint for accepting discovered data  https://review.openstack.org/11600314:59
jrolldtantsur really wants this discovery stuff14:59
*** ChuckC has joined #openstack-ironic14:59
dtantsurjroll, not really _me_, but well yeah :)14:59
jrollheh15:00
*** chuckC_ has joined #openstack-ironic15:00
dtantsurit proved to be faaaaar more complex, than I expected ~ month ago :(15:01
jrollright, we punted it for a reason :P15:01
*** coolsvap has joined #openstack-ironic15:04
*** mikedillion has joined #openstack-ironic15:05
*** yuanying has joined #openstack-ironic15:11
*** rwsu has quit IRC15:23
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Control extra space for images conversion in image_cache  https://review.openstack.org/11597415:24
*** dtantsur is now known as dtantsur|afk15:31
*** rwsu has joined #openstack-ironic15:39
*** yuanying has quit IRC15:44
*** mikedillion has quit IRC15:58
*** mikedillion has joined #openstack-ironic16:00
*** krtaylor has quit IRC16:00
NobodyCamgah locked up my mac book :-p16:01
lifelessdtantsur|afk: tuning down in devstack would be fine IMO16:02
lifelessdtantsur|afk: the 2x thing we knew was wrong, not surprising ;)16:02
JayFA question for the folks here; I know when we started IPA the idea was that we'd do releases periodically similar to how the clients do16:15
JayFbut I'm starting to wonder if we should cut a release for Juno, so we have an agent that's 'blessed' for use in installs16:16
JayFI personally would not use this, as we obviously run master/master+unmerged patches, but I'm curious what other users of the agent think16:16
jrollI'd like to hear from lifeless and lucasagomes on this ^^16:16
jrolland NobodyCam16:16
NobodyCamhumm16:21
NobodyCamipa has landed so I could see a juno release16:21
JayFLike I continue to think that master of ironic should work with any agent master (well, from merge onwards, at least) due to versioning and such16:22
JayFplus most big or multitenant installs will want to build their own agent with their own hardware manager16:22
NobodyCamJayF: with a release, wouldn't that be more like the refference version?16:24
JayFI don't know, honestly16:25
JayFthat's why I'm asking :)16:25
JayFI still don't understand why someone would want to run a release from an arbitrary date because it has a name instead of running the latest and greatest ;)16:26
lucasagomesjroll, NobodyCam JayF does TripleO goes through the release process... if so having a release of IPA for Juno is the same of having a release for Juno of DIB (and it's elements, which is == our default ramdisk)16:26
lucasagomesso I would say yes, it makes sense to have a release for IPA16:27
JayFthe second question that follows is what form is the release?16:27
JayFreference images? pypi (I don't love this but understand why it could be useful)?16:27
lucasagomesyeah... well that's the thing we would probably have to follow the whole release process16:28
lucasagomeshttps://wiki.openstack.org/wiki/Release_Cycle16:28
lucasagomeswhich we haven't done (I think)... we didn't have any milestone release for IPA etc...16:28
jrollwell16:28
jrollthe goal is to release like the client16:29
jrollthe reason I see a juno release as valuable:16:29
jroll1) stable/juno CI16:29
jroll2) to say "this agent works with juno"16:29
rloojroll: question about ironic.drivers.modules.agent wrt driver_info properties16:29
jrollpypi makes sense e.g. to pip install in a ramdisk / docker container16:30
jrolldeb/rpm makes absolutely no sense16:30
JayFlike if a juno bug is reported, step 1 is to test with the 'reference' version for juno, to see if it's a new regression in the agent or something in ironic16:30
jrollrloo: shoot16:30
JayFright?16:30
*** dlaube has joined #openstack-ironic16:30
rloojroll: the agent code returns {} for get_properties(). but...16:30
*** viktors is now known as viktors|afk16:30
jrollrloo: yeah, so deva and I chatted about this, since the deployer doesn't enter anything in driver_info it makes sense for get_properties to return {}16:31
rloojroll: I see that 'agent_url' should be in driver_info?16:31
rloojroll: who puts in agent_url?16:31
JayFthe agent does.16:31
JayFon it's lookup/heartbeat16:31
jrollrloo: computers :)16:31
rloojroll: ahhh. ok, got it.16:32
kylestevJayF: what do you think about the agent being available in pypi?16:32
JayFI think I personally wouldn't use it, but I can see the value for folks wanting to build their own agents to have a 'stable' base16:34
JayFbut honestly IDK if I'd want to encourage "build a hardware manager on this arbitrary SHA1" as a pattern vs having those users follow master or pick a spot to integrate themselves16:34
jrollbrb16:34
*** ramineni1 has quit IRC16:39
*** yuanying has joined #openstack-ironic16:41
*** vdrok_ has joined #openstack-ironic16:43
*** krtaylor has joined #openstack-ironic16:45
*** athomas has quit IRC16:45
vdrok_good evening Ironic!16:48
NobodyCammorning vdrok_ :)16:49
NobodyCam:-p16:49
vdrok_morning NobodyCam :)16:49
vdrok_There is a question about how to import translation functions from i18n. rloo suggested to add i18n module to tox.ini as import_exception and then import functions like _, _LW directly from there. What do you guys think about that?16:52
rloovdrok_: I think it is a grand idea ;)16:53
vdrok_:)16:53
*** derekh has quit IRC16:56
lucasagomesaight I will call it a day16:57
lucasagomesI've to take the train back home yet16:57
lucasagomeshave a good night everybody!16:57
rloobye lucasagomes16:57
lucasagomesbye bye16:58
*** lucasagomes has quit IRC16:58
vdrok_rloo: seems that it's done this way in nova17:06
*** bandicot has joined #openstack-ironic17:06
rloovdrok_: yeah. not that we always want to copy nova, but :-). It is also similar to how we handled it with the old gettextutils.17:06
rloovdrok_: of course, if others think we ought to do it differently, we can discuss.17:07
vdrok_rloo: sure17:07
*** tonycoffman has joined #openstack-ironic17:07
rloovdrok_: it would be good to get agreement before you make changes.  I don't want you to get frustrated by different opinions.17:08
*** bandicot is now known as bensonmiller17:09
*** tatyana has quit IRC17:10
lifelessjroll: I'm a huge CD fan17:10
lifelessjroll: so I totally support treating IPA as always-stable always-working-with-all-supported-Ironics17:11
vdrok_rloo: ok :)17:11
*** ndipanov has quit IRC17:11
lifelessjroll: and cutting releases out of trunk to keep inventory under control17:11
russell_h+117:12
lifelessjroll: assuming thats what you meant :)17:12
*** yuanying has quit IRC17:14
*** harlowja_away is now known as harlowja_17:25
jrolllifeless: +1 :)17:26
lifelessjroll: I suggest starting a -dev thread on this17:27
jrolllifeless: yeah, today is crazy but will try17:27
jrollmaybe I can con JayF into doing so17:27
lifelessjroll: as ttx may have thoughts/concerns and it will ensure we've had wide visibility17:27
lifelessjroll: JayF: also -operators should be given a chance to scream noooooo17:27
jrollindeed17:28
*** jcoufal has quit IRC17:33
*** pelix has quit IRC17:42
aweeksgreghaynes: I've got a question for you about https://review.openstack.org/#/c/109688/ if you have a moment18:00
*** mikedillion has quit IRC18:05
*** igordcard has quit IRC18:05
* JayF dodges the bus18:07
greghaynesaweeks: Hey18:09
greghaynesaweeks: At the airport, so I have a little time while I wait18:09
aweeksgreghaynes: I was just wondering if the idea is to make it more efficient for API nodes to decide which conductor requests need to be forwarded to18:10
greghaynesaweeks: No, its for rebalancing when a conductor fails (its in the commit message)18:10
greghaynesif we dont have this index we have to walk all the nodes when a failure happens18:11
*** yuanying has joined #openstack-ironic18:11
*** penick has joined #openstack-ironic18:14
aweeksgreghaynes: ok, I'm probably just missing something, or am not familiar enough with the code, but how is the failure detected?18:14
JayFI think it's more that18:15
JayFhe wants to know what nodes are impacted via a mysql query18:15
greghaynesI dont believe its implemented yet but the plan is periodic task.18:15
greghaynesyes18:15
JayFif he knows shiny-conductor17.mydomain.tld goes down18:15
aweeksok, that makes more sense18:16
greghaynesThe only way to get the list of nodes that belong to a conductor right now is to compile a list by evaluating the conductor every node should belong to18:16
aweeksyeah18:16
lifelessits also for ensuring tha tfailures are handled18:27
*** mikedillion has joined #openstack-ironic18:28
*** mikedill_ has joined #openstack-ironic18:32
*** mikedillion has quit IRC18:33
*** mikedill_ has quit IRC18:34
*** mikedillion has joined #openstack-ironic18:36
*** yuanying has quit IRC18:44
NobodyCambrb18:51
*** mikedillion has quit IRC19:05
* jroll shakes a fist at JayF19:07
*** mikedillion has joined #openstack-ironic19:08
*** harlowja_ has quit IRC19:08
*** harlowja has joined #openstack-ironic19:15
*** vdrok_ has quit IRC19:17
*** harlowja has quit IRC19:19
*** harlowja has joined #openstack-ironic19:28
*** harlowja has quit IRC19:34
*** harlowja has joined #openstack-ironic19:40
openstackgerritKyle Stevenson proposed a change to openstack/ironic: Add implementation of pluggable distributed sync  https://review.openstack.org/11514819:40
*** yuanying has joined #openstack-ironic19:41
adam_gNobodyCam, FYI WRT soft poweroff, not sure if you've seen it.. just landed in nova https://review.openstack.org/#/c/68942/3219:58
* NobodyCam looks19:58
NobodyCamoh happy happy joy jpy19:59
NobodyCamadam_g: TY19:59
NobodyCam:)19:59
adam_gincidentally, it breaks our driver and im getting lots of new tracebacks :|19:59
NobodyCam:(20:06
NobodyCamhumm. teh non-voting job did pass20:07
NobodyCamon that review20:07
adam_ghmm20:08
adam_gour gate looks mostly broken ATM20:08
adam_ghttp://no-carrier.net/~adam/openstack/ironic_gate_status.html20:08
NobodyCamare thouse our error? http://logs.openstack.org/39/115339/1/check/check-tempest-dsvm-virtual-ironic-nv/54e6dd5/console.html#_2014-08-21_19_56_55_71320:10
adam_gthats what im seeing20:10
NobodyCams/error/errors/20:10
adam_g(unrelated to that nova driver change)20:10
JayFdid a requirement bump? like perhaps a new release of keystroneclient getting installed now? </guess>20:12
*** yuanying has quit IRC20:15
openstackgerritAdam Gandelman proposed a change to openstack/ironic: Update nova driver's power_off() parameters  https://review.openstack.org/11609320:15
adam_gmrda, ^^^20:16
adam_gsort of looks like a tempest issue20:16
NobodyCamadam_g: Ty for ^^ and the bug too :)20:16
rloosomeone filed a bug about the keystone thing in tempest: https://bugs.launchpad.net/ironic/+bug/135992920:18
adam_gyeah20:20
adam_gjust found that20:20
openstackgerritKyle Stevenson proposed a change to openstack/ironic: Add implementation of pluggable distributed sync  https://review.openstack.org/11514820:21
*** penick has quit IRC20:45
*** stevebaker has quit IRC20:49
*** stevebaker has joined #openstack-ironic20:49
openstackgerritAdam Gandelman proposed a change to openstack/ironic: Update nova driver's power_off() parameters  https://review.openstack.org/11609320:55
*** stevebaker has quit IRC21:01
*** stevebaker has joined #openstack-ironic21:02
rlooadam_g: wrt 116093, sorry to be a PITA, the other docstrings use 'Ignored by this driver.' Should we do the same here?21:02
adam_grloo, sure. one min21:03
rlooadam_g: am also wondering about opening a bug to handle them in the future?21:03
rlooadam_g: I looked in nova code. the other drivers were updated with this: "# TODO(PhilDay): Add support for timeout (clean shutdown)"21:06
NobodyCamits all true I tell you: http://www.caninedistractions.com/when-danes-destroy-30-photos-that-capture-the-wreckage/21:07
*** yuanying has joined #openstack-ironic21:11
rlooI updated the gate status in our IronicWhiteBoard.21:23
adam_gi think ive found the issue21:27
*** jasondotstar has quit IRC21:27
rlooadam_g: you da man21:27
adam_ghttps://review.openstack.org/#/c/116103/21:34
adam_g^ that should hopefully pass the virtual-ironic job and fix us21:36
rlooadam_g: that doesn't just affect ironic jobs?21:38
adam_grloo, it sort of does21:39
adam_grloo, devstack-vm-gate.sh calls different tox envs differently depending on the job. i think ironic is the only one that uses the code path to call tox -eall21:40
rlooadam_g: ahh. ok. and what changed yesterday that caused this to break?21:40
*** krtaylor has quit IRC21:41
adam_grloo, tox was unpinned and upgraded to 1.7.2 across slaves. that now sets the a random hashseed, and apparently lots of test code can't handle that :)21:41
rlooadam_g: apparently :-)21:42
adam_grloo, http://lists.openstack.org/pipermail/openstack-dev/2014-July/041283.html21:42
*** linggao has quit IRC21:44
rlooadam_g: glad you can keep track of these changes. Although I don't see how dict-entry-ordering caused the keystone-xml error but hopefully it did.21:44
*** yuanying has quit IRC21:44
openstackgerritKyle Stevenson proposed a change to openstack/ironic-specs: Add spec for pluggable distributed synchronization  https://review.openstack.org/11463221:50
NobodyCamgah check-tripleo-ironic-undercloud-precise-nonha failed21:53
NobodyCambrb21:59
openstackgerritAdam Gandelman proposed a change to openstack/ironic: Update nova driver's power_off() parameters  https://review.openstack.org/11609322:02
*** rloo is now known as rloo_afk22:12
* NobodyCam sobs just a little: http://paste.openstack.org/show/RyDz3eHQ2ydnkxZaAzCX/22:15
jrolladam_g: nice find22:15
jrollNobodyCam: please tell me that's local and not the gate22:15
NobodyCamyes thats local22:15
NobodyCamlol22:15
adam_gNobodyCam, tripleo?22:15
NobodyCamsi22:15
NobodyCamadam_g: it's my seed vm. oh :-p22:16
NobodyCamoot@ubuntu:~# uptime 22:15:56 up 7 days,  1:53,  1 user,  load average: 0.07, 0.10, 0.1322:16
*** scubacuda has joined #openstack-ironic22:17
adam_gNobodyCam, wonder how big the keystone token table is22:17
NobodyCamadam_g: you know the command to check?22:17
adam_gNobodyCam, i usually just mysql -c keystone -e 'select count(*) from token'22:19
NobodyCamlol22:22
NobodyCam+----------+22:22
NobodyCam| count(*) |22:22
NobodyCam+----------+22:22
NobodyCam|        2 |22:22
NobodyCam+----------+22:22
*** jgrimm has quit IRC22:22
* NobodyCam dose happy dance22:25
NobodyCamhttp://paste.openstack.org/show/vodkbUatKRI5siIrJJnE/22:25
jrollNobodyCam: http://www.downloadmoreram.com/22:25
NobodyCamlol22:25
jrollwhat's sad, is when I searched for that, google's autocomplete wanted to add "legit" and "virus"22:26
jrolloh ha https://answers.yahoo.com/question/index?qid=20111123175542AA3CoOH22:26
NobodyCamlol You make me weep for humanity.22:27
NobodyCamlov that one22:27
jrollyeahhhhhh22:28
NobodyCamtho with gmailfs (http://en.wikipedia.org/wiki/GmailFS) and a swap file it could look like downloaded ram... lol22:29
jrollhahaha22:30
jrollraas - ram as a service22:30
JayFI bet you could really break a machine22:30
JayFwith a flakey fuse driver22:30
NobodyCamlol.. that would actually be really cool22:30
JayFand a swapfile on that filesystem22:30
NobodyCamoh ya22:30
jrolloh god22:30
NobodyCamand quick it is not lol22:31
*** yuanying has joined #openstack-ironic22:41
mrdaadam_g: is this something we MUST have in the ironci driver for J?22:50
adam_gmrda, yeah22:50
adam_gmrda, tempest wont pass without it22:51
adam_g:(22:51
mrdaok, I'm on a plane today, but I should get that done somewhere where's there's internets :)22:51
* mrda makes a note to himself22:51
*** shakamunyi has joined #openstack-ironic22:51
adam_gmrda, coo.. thanks and sorry. nova updated the driver yesterday22:51
mrdaif this comes up again, add it to the Whiteboard and ping me where too.  That way it won't get skipped :)22:52
NobodyCamquestion: do we need to fill out all the fields? RE: http://paste.openstack.org/show/jNQpecPEYVEvxAeFx5B122:59
NobodyCamjroll: JayF: JoshNang: TY for the list(detail=true) :) +123:02
NobodyCamlol anyone have to have a nova baremetal up. I need a sample of nova baremetal-node-show <blah>23:06
NobodyCams/have to/happen to/23:07
NobodyCamnm: http://docs.openstack.org/user-guide/content/cli_nova_baremetal-details.html23:08
*** yuanying has quit IRC23:14
adam_ghmm. looks like we need a patch to land in stable/havana tempest before https://review.openstack.org/#/c/116103/ can merge and fix us23:20
*** yuanying has joined #openstack-ironic23:30
NobodyCam:(23:30
NobodyCamanyone have a minute to take a quick look at http://paste.openstack.org/show/ijF44Q6nzoJwYAWOGakP23:30
NobodyCammrda: ^^^^^^23:31
*** chuckC_ has quit IRC23:31
NobodyCambrb23:31
NobodyCamback :)23:47
adam_gNobodyCam, could the interfaces field of the nova node show the result of a node-port-list call?23:53

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