Wednesday, 2014-06-25

devanandajroll: that's *exactly* what the tempest folks are saying they think we should do00:00
devanandawell00:00
devanandathat's my understanding00:00
jrollyeah, lets do it00:00
JayFsounds like they stormed in with some good ideas00:00
devanandajroll: see https://review.openstack.org/#/c/100989/1..2/tempest/api/baremetal/README.rst00:00
*** harlowja has quit IRC00:00
jrollsomeone give me X thousand dollars and I'll make it happen :)00:00
jrollJayF: ಠ_ಠ00:00
devanandajroll: would you have a separate cell just for testing?00:01
devanandain the production cloud, i mean. not a separate QA cloud00:01
jrolldevananda: unclear right now :)00:01
jrolldevananda: most of our tests are done in preprod, and just some basic testing in prod afaik00:02
jrolldevananda: I mean, testing in prod is the last line. we want to make sure *the cell we just deployed to* is working as expected00:03
jrollso like, separate cell defeats the purpose00:03
devanandajroll: right00:04
devanandajroll: so what tests do you run against a cell that uou just deployed to prod?00:05
devanandajroll: if you dont mind sharing00:05
JayFI don't think we have much insight into that right now00:05
*** lucas-dinner has quit IRC00:06
jrolldevananda: afaik, today, we support a subset of intended functionality. nova boot and nova delete. so we test that manually, poke around a bit.00:06
JayFI thought he meant a nova cell, with nova deploys00:07
jrolldevananda: as for QE environment, dwalleck runs a whole bunch of stuff against that as far as notifications etc, but I'm not sure exactly what00:07
devanandajroll: right, that's about what i thought00:07
jrolloh, I have no clue about nova cells00:07
jrolldevananda: right, so, once tempest is more fleshed out for ironic purposes, we'll be using that + probably some cloud cafe tests in preprod. not sure about prod but maybe?00:08
jrolldevananda: although, now that I think about it. it's a short window for a race, right?00:09
*** hemna_ has quit IRC00:09
jrolldevananda: maybe this is another reason to figure out the ComputeManager stuff - if we tie some group of nodes to a compute instance, we just tie the test nodes to their own compute instance and set some kind of acl?00:10
jroll(brb, quick phone)00:10
adam_gdevananda, good point00:10
devanandajroll: that's one possibility. another would be node<->tenant assignment00:13
devanandaadam_g: long comment posted to 10098900:14
jrolldevananda: true, that's probably the best way00:15
adam_gdevananda, cool. you may want to post that in the main comments, the reply is tucked away in the old patchset00:16
devanandaadam_g: or just copy/paste the ML ...?00:17
adam_gdevananda, yeah00:17
NobodyCamok  stepping away00:28
*** ellenh has quit IRC00:32
*** ellenh has joined #openstack-ironic00:32
*** matsuhashi has joined #openstack-ironic00:35
*** yu_ has quit IRC00:37
*** ellenh has quit IRC00:41
*** ellenh has joined #openstack-ironic00:41
*** coolsvap is now known as coolsvap|afk00:44
*** takadayuiko has quit IRC00:46
*** Penick has quit IRC00:48
*** faizan has joined #openstack-ironic00:49
*** rloo_gone has quit IRC00:55
*** rloo_gone has joined #openstack-ironic00:55
openstackgerritJim Rollenhagen proposed a change to openstack/ironic-specs: Support long-running deploy ramdisks  https://review.openstack.org/10240500:55
jrollJoshNang: ^00:55
jrollI'm off for the night, out of office tomorrow. see y'all thursday! :)00:56
*** rloo_gone has quit IRC00:56
*** rloo_gone has joined #openstack-ironic00:57
*** rloo_gone has quit IRC00:59
*** rloo_gone has joined #openstack-ironic01:00
*** achanda has quit IRC01:01
*** rloo_gone has quit IRC01:02
*** rloo_gone has joined #openstack-ironic01:03
*** ellenh has quit IRC01:03
*** Penick has joined #openstack-ironic01:23
*** rloo_gone has quit IRC01:24
*** rloo_gone has joined #openstack-ironic01:25
*** faizan has quit IRC01:25
*** Penick has quit IRC01:26
*** harlowja_ has quit IRC01:28
*** Penick has joined #openstack-ironic01:30
*** rloo_gone has quit IRC01:31
*** Penick has quit IRC01:31
*** rloo_gone has joined #openstack-ironic01:31
*** harlowja has joined #openstack-ironic01:32
*** rloo_gone has quit IRC01:32
*** rloo_gone has joined #openstack-ironic01:33
*** rloo_gone has quit IRC01:39
* devananda signs off for the night. starts packign for tomorrow morning's flight01:40
mrdahave fun devananda01:42
*** nosnos has joined #openstack-ironic01:51
*** nosnos has quit IRC01:56
*** nosnos has joined #openstack-ironic01:57
*** bandicot has quit IRC01:57
*** eghobo has quit IRC02:05
*** openstackgerrit has quit IRC02:31
*** mitz_ has joined #openstack-ironic02:36
*** vinbs has joined #openstack-ironic03:01
*** harlowja is now known as harlowja_away03:04
*** bcaldwell has joined #openstack-ironic03:20
*** achanda has joined #openstack-ironic03:25
*** eghobo has joined #openstack-ironic03:36
*** vinbs has quit IRC03:40
*** vinbs has joined #openstack-ironic03:40
*** matsuhashi has quit IRC03:42
*** matsuhashi has joined #openstack-ironic03:43
*** matsuhashi has quit IRC03:47
*** nosnos has quit IRC03:51
*** Poornima has joined #openstack-ironic03:55
*** rameshg871 has joined #openstack-ironic03:57
*** Penick_ has joined #openstack-ironic04:03
*** eghobo has quit IRC04:14
*** eghobo has joined #openstack-ironic04:15
*** lazy_prince has joined #openstack-ironic04:15
*** lazy_prince has quit IRC04:19
*** matsuhashi has joined #openstack-ironic04:31
*** lazy_prince has joined #openstack-ironic04:34
*** Manishanker has joined #openstack-ironic04:35
*** nosnos has joined #openstack-ironic04:37
*** achanda has quit IRC04:39
*** bandicot has joined #openstack-ironic04:40
*** sabah has joined #openstack-ironic04:45
*** rakesh_hs4 has joined #openstack-ironic04:48
*** bandicot has quit IRC04:56
*** achanda has joined #openstack-ironic04:59
*** ajc_ has joined #openstack-ironic04:59
*** Manishanker has quit IRC05:05
*** pradipta_away is now known as pradipta05:17
*** Manishanker has joined #openstack-ironic05:17
*** coolsvap|afk is now known as coolsvap05:19
*** max_lobur has joined #openstack-ironic05:20
*** rwsu has quit IRC05:24
*** faizan has joined #openstack-ironic05:31
*** amitpp has joined #openstack-ironic05:31
*** faizan has quit IRC05:38
*** bcaldwell has quit IRC05:41
*** amitpp has quit IRC05:42
*** bcaldwell has joined #openstack-ironic05:47
*** Manishanker has quit IRC05:55
*** bmahalakshmi has joined #openstack-ironic06:01
*** max_lobur has quit IRC06:06
*** achanda has quit IRC06:10
*** achanda has joined #openstack-ironic06:15
*** achanda has quit IRC06:22
*** Poornima|mtg has joined #openstack-ironic06:42
*** Poornima|afk has joined #openstack-ironic06:45
*** Poornima has quit IRC06:45
*** Poornima|mtg has quit IRC06:48
*** martyntaylor has joined #openstack-ironic06:51
*** Poornima|mtg has joined #openstack-ironic06:52
*** max_lobur has joined #openstack-ironic06:54
*** Poornima|afk has quit IRC06:56
*** Mikhail_D_ltp has joined #openstack-ironic07:05
*** shausy has joined #openstack-ironic07:09
*** ifarkas has joined #openstack-ironic07:20
*** jcoufal has joined #openstack-ironic07:30
*** rameshg87 has quit IRC07:31
*** sabah has quit IRC07:31
*** Haomeng has quit IRC07:46
*** matsuhashi has quit IRC07:57
*** matsuhashi has joined #openstack-ironic07:58
*** Haomeng has joined #openstack-ironic07:59
*** mrda is now known as mrda-away08:02
mrda-awaynight ironic08:03
Haomengmrda-away: night:)08:04
*** matsuhashi has quit IRC08:05
*** matsuhashi has joined #openstack-ironic08:10
*** sabah has joined #openstack-ironic08:12
*** matsuhashi has quit IRC08:17
*** Haomeng has quit IRC08:17
*** matsuhashi has joined #openstack-ironic08:18
*** Haomeng has joined #openstack-ironic08:18
*** Haomeng has quit IRC08:19
*** romcheg has joined #openstack-ironic08:21
*** athomas has joined #openstack-ironic08:28
*** matsuhashi has quit IRC08:31
*** pelix has joined #openstack-ironic08:36
*** lucasagomes has joined #openstack-ironic08:36
*** igordcard has joined #openstack-ironic08:45
*** nosnos has quit IRC08:45
*** openstackgerrit has joined #openstack-ironic08:53
*** sabah has quit IRC09:04
*** Haomeng has joined #openstack-ironic09:13
*** Haomeng|2 has joined #openstack-ironic09:13
*** gcb_ has joined #openstack-ironic09:27
openstackgerritA change was merged to openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/9606309:28
*** nosnos has joined #openstack-ironic09:36
*** sabah has joined #openstack-ironic09:36
*** matsuhashi has joined #openstack-ironic09:36
*** sabah has quit IRC09:41
*** matsuhashi has quit IRC09:54
dtantsur|afkg'afternoon Ironic09:54
*** dtantsur|afk is now known as dtantsur09:54
*** matsuhashi has joined #openstack-ironic09:55
*** matsuhas_ has joined #openstack-ironic09:57
*** matsuhashi has quit IRC09:57
dtantsurFolks, is there any way to get Python logs out of unit tests run result, like: http://logs.openstack.org/21/73121/15/check/gate-ironic-python27/a8c9a47/10:01
dtantsur(the failure can't be reproduced locally)10:02
lucasagomesdtantsur, not sure :/ can't be reproduced locally?10:09
lucasagomesany reason for that?10:09
dtantsurlucasagomes, no idea, tests pass for me here :(10:09
lucasagomesdtantsur, mysql vs sqlite?10:09
dtantsurlucasagomes, maybe... which is used where? sqlite locally, mysql on machines?10:10
lucasagomesdtantsur, I'm not 100% sure but I think that's the case10:10
lucasagomeslocally it will use sqlite10:10
lucasagomesand mysql on gate10:10
lucasagomesso you might be able to change it locally to use mysql local as well10:10
lucasagomes(gotta take a look at how to do it tho)10:10
*** gcb_ has quit IRC10:11
dtantsurlucasagomes, thanks for the hint10:12
*** mkerrin has quit IRC10:12
* dtantsur hates tox >_<10:14
lucasagomesdtantsur, np, it's more a guess tho :)10:14
lucasagomesnot sure if the mysql vs sqlite is the real cause, but worth trying10:14
*** romcheg has quit IRC10:16
*** igordcard has quit IRC10:18
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: Fix leaking DB details to API on error  https://review.openstack.org/7312110:27
*** matsuhas_ has quit IRC10:28
*** matsuhashi has joined #openstack-ironic10:28
*** gcb_ has joined #openstack-ironic10:28
*** amitpp has joined #openstack-ironic10:28
*** ifarkas has quit IRC10:32
openstackgerritA change was merged to openstack/ironic: Implement security groups and firewall filtering methods  https://review.openstack.org/9646610:35
*** matsuhashi has quit IRC10:37
*** matsuhashi has joined #openstack-ironic10:37
*** sabah has joined #openstack-ironic10:44
*** dtantsur is now known as dtantsur|lunch10:48
openstackgerritA change was merged to openstack/ironic: Do not delete pxe_deploy_{kernel, ramdisk} on tear down  https://review.openstack.org/10186410:49
*** Haomeng|2 has quit IRC11:03
*** foexle has joined #openstack-ironic11:08
*** coolsvap is now known as coolsvap|afk11:11
*** gcb_ has quit IRC11:16
*** dtantsur|lunch is now known as dtantsur11:16
*** romcheg has joined #openstack-ironic11:17
*** sabah has quit IRC11:24
*** lucasagomes is now known as lucas-hungry11:54
*** nikunj2512 has joined #openstack-ironic12:00
*** nikunj2512 has quit IRC12:00
*** nikunj2512 has joined #openstack-ironic12:01
*** loki184 has joined #openstack-ironic12:04
*** rakesh_hs4 is now known as rakesh_hs12:05
Shrewsmorning folks12:07
*** vinbs has quit IRC12:08
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: Fix leaking DB details to API on error  https://review.openstack.org/7312112:08
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: Raise appropriate errors on duplicate Node and Chassis creation  https://review.openstack.org/10250612:08
dtantsurShrews, morning12:10
*** loki184 has quit IRC12:16
*** jdob has joined #openstack-ironic12:16
*** linggao has joined #openstack-ironic12:18
*** bcaldwell has quit IRC12:19
*** amitpp has quit IRC12:19
*** amitpp has joined #openstack-ironic12:19
romchegMorning Ironic!12:19
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: Prevent updating UUID of Node, Port and Chassis on DB API level  https://review.openstack.org/10224712:20
dtantsurromcheg, morning!12:21
*** amitpp has quit IRC12:22
*** bmahalakshmi has quit IRC12:23
*** ajc_ has quit IRC12:23
*** shausy has quit IRC12:24
*** shausy has joined #openstack-ironic12:24
*** shausy has quit IRC12:25
*** shausy has joined #openstack-ironic12:25
*** Haomeng|2 has joined #openstack-ironic12:26
dtantsurStill can't figure out, how to run tests locally with MySQL instead of SQLite :( Any ideas?12:26
Shrewsdtantsur: that's in our quickstart guide12:27
dtantsurreally? Oo12:27
*** Haomeng has quit IRC12:27
Shrewsdtantsur: oh, you mean the unit tests12:27
dtantsurShrews, yep12:27
*** geekyogi has joined #openstack-ironic12:29
*** shausy has quit IRC12:31
*** shausy has joined #openstack-ironic12:31
*** shausy has left #openstack-ironic12:31
*** shausy has joined #openstack-ironic12:33
*** nikunj2512 has quit IRC12:40
Shrewsdtantsur: hrm. not sure, but looks like if you set database.connection to a mysql connect string, that should do it. I *think* you set this in a config file and pass it to tox with -c ????12:40
dtantsurShrews, hmmm... worse trying12:40
Shrewstox -c myconfig -ep27   <--- just a guess12:40
Shrews-epy2712:40
*** rakesh_hs has quit IRC12:41
dtantsurfolks, do you know any conditions, where bash will execute different thing from `which X` when call `X` ?12:44
dtantsur* where = when12:44
*** lazy_prince has quit IRC12:47
*** bvivek has joined #openstack-ironic12:48
*** rloo has joined #openstack-ironic12:52
dtantsuryuriyz, any ideas how to make things like https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L520 actually work? Error is not DBDuplicateEntry, it seems like it's required to analyze error message...12:54
*** jbjohnso has joined #openstack-ironic12:54
dtantsuryuriyz, can we have outdated openstack.common.db?12:55
*** pradipta is now known as pradipta_away12:55
yuriyzdtantsur, Im looking now12:57
rameshg871hello dtantsur12:57
dtantsuryuriyz, one more guess: this does not work for primary keys, only for UNIQUE columns...12:57
dtantsuryuriyz, thanks!12:57
dtantsurrameshg871, hi!12:57
rameshg871dtantsur, just got your comment on bash_completion12:58
dtantsurrameshg871, yeah. I am out of luck trying to enable it on my devstack :(12:58
*** amitpp has joined #openstack-ironic12:58
rameshg871dtantsur, i just copy the file tools/ironic.bash_completion to /etc/bash_completion.d/12:58
rameshg871dtantsur, and then source .bashrc again on that terminal12:59
*** matsuhashi has quit IRC12:59
rameshg871dtantsur, source ~/.bashrc12:59
*** matsuhashi has joined #openstack-ironic12:59
rameshg871dtantsur, and then it should work as expected :-)12:59
dtantsurrameshg871, does not work for me, shows files in the current directory :(13:00
rameshg871dtantsur, my ~/.bashrc has ". /etc/bash_completion" which sources all files within /etc/bash_completion.d/13:00
rameshg871dtantsur, i hope you have it too ..13:01
*** lucas-hungry is now known as lucasagomes13:01
rameshg871dtantsur, or you can even try ". /etc/bash_completion" on the current terminal and see whether it works13:02
rameshg871dtantsur, i am using ubuntu btw, i don't know if it differs across distribution13:02
dtantsurrameshg871, sorry for confusion! Installing bash-completion before testing your patch would be a wise step to do :D13:03
rameshg871dtantsur, oh :-)13:03
*** matsuhashi has quit IRC13:04
*** jbjohnso has quit IRC13:04
*** igordcard has joined #openstack-ironic13:06
*** matsuhashi has joined #openstack-ironic13:10
*** mkerrin has joined #openstack-ironic13:20
dtantsuryuriyz, ok, I came to conclusion, that openstack.common.db not always convert pkey violation to DBDuplicateEntry... this requires either fixing them or creating a workaround13:25
dtantsurdon't know what is better13:25
yuriyzdtantsur, my comment inline13:26
*** Haomeng has joined #openstack-ironic13:26
*** matty_dubs|gone is now known as matty_dubs13:27
dtantsuryuriyz, aaah, my bad, thank you! Btw, why are we allowed to set id?13:27
*** Haomeng|2 has quit IRC13:27
yuriyzdtantsur, simple remove it from dict, will be set auto13:28
dtantsuryuriyz, I mean, should not create_* reject calls with ID set?13:28
*** rameshg871 has left #openstack-ironic13:29
yuriyzid used only internally, API dont allow assess13:29
yuriyzaccess13:29
*** amitpp has quit IRC13:30
matty_dubsMorning Ironic'ers (if that's the proper demonym)13:30
dtantsurmatty_dubs, morning :)13:32
*** matsuhashi has quit IRC13:33
*** matsuhashi has joined #openstack-ironic13:33
dtantsuryuriyz, should I raise NodeAssociated or some new exception?13:34
yuriyzfor instance_uuid duplication?13:35
dtantsuryes13:35
matty_dubsdevananda: (probably in absentia this early) I'll revise that blog post. FWIW I've also gotten a few emails with questions about it. But if people are following my random blog post, it makes me think that official docs are probably lacking.13:36
yuriyzI think new like InstanceExists13:36
rloomorning matty_dubs13:36
matty_dubsWhich we should probably work on.13:36
matty_dubsHowdy rloo13:36
rloomatty_dubs: in the absence of deva, do I hear you volunteering to write up an official doc? ;)13:37
dtantsurrloo, morning!13:37
matty_dubslol, I feel like my words are being slightly twisted ;)13:37
*** killer_prince has joined #openstack-ironic13:38
*** matsuhashi has quit IRC13:38
rlooAfternoon dtantsur!13:38
dtantsurmatty_dubs, is https://bugs.launchpad.net/ironic/+bug/1323589 what you want to work on? ;)13:38
dtantsuryuriyz, ack, thanks13:38
rloomatty_dubs: I was just following your train of thought ;)13:38
*** nosnos has quit IRC13:43
*** coolsvap|afk is now known as coolsvap13:43
matty_dubsHeh, https://www.openstack.org/summit/openstack-paris-summit-2014/ -- that photo in the lower left is me! Can I now call myself "the face of OpenStack"? ;)13:43
*** matsuhashi has joined #openstack-ironic13:44
rloomatty_dubs: it's that GREAT SMILE of yours!13:44
*** matsuhashi has quit IRC13:44
*** matsuhashi has joined #openstack-ironic13:45
*** rloo has quit IRC13:47
*** rloo has joined #openstack-ironic13:48
*** rloo has joined #openstack-ironic13:48
*** rloo has quit IRC13:49
*** rloo has joined #openstack-ironic13:49
*** matsuhashi has quit IRC13:49
*** rloo has quit IRC13:55
*** rloo has joined #openstack-ironic13:55
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: Raise appropriate errors on duplicate Node and Chassis creation  https://review.openstack.org/10250613:55
matty_dubsrloo: Actually, do you think there's a better guide out there, that I should just link to, versus revising my blog post from January/13:56
rloomatty_dubs: I forgot what your blog post was about. What's the link?13:56
matty_dubsrloo: Oh, sorry. It was about devstack+Ironic -- http://ma.ttwagner.com/bare-metal-deploys-with-devstack-and-ironic/13:57
matty_dubsPeople keep finding it via Google and stumbling over steps that are now inaccurate. I'm wondering if it makes more sense to revise it, or to just take it down and replace it with a link to something else13:57
dtantsuryuriyz, updated https://review.openstack.org/#/c/102506 please let me know what you think13:57
matty_dubsrloo: I'll poke around; was just curious if you knew of something better off the top of your head. Been a bit since I looked at the documentation out there :-\13:58
rloomatty_dubs: we don't have any doc that goes into such detail.13:58
rloomatty_dubs: so I think this is really useful13:59
matty_dubshttp://docs.openstack.org/developer/ironic/dev/dev-quickstart.html looks promising13:59
matty_dubsAh, okay. I'll revise it a bit then.13:59
rloomatty_dubs: my thinking is that at some point in time (maybe with juno), we could have such a doc in ironic. but for now, i'd be concerned about having a doc like this and having to keep it up-to-date. So I'm quite happy to have you update your blog ;)14:00
rloomatty_dubs: I think it would be nice if in the ironic doc (not sure where), there were pointers to doc outside of ironic, like your blog and jroll's blog.14:01
openstackgerritLucas Alvares Gomes proposed a change to openstack/python-ironicclient: Add pagination support to {node, port, chassis}-list  https://review.openstack.org/9158514:01
rloomatty_dubs: btw, if you're updating your blog. At the end, I think you mean 'Voila', or did you want strings? :-)14:01
matty_dubsrloo: LOL14:02
* rloo thinks that if you get ironic working with devstack, that you might hear strings in the background...14:02
matty_dubsClearly, after getting it all working, mastering basic French required way too much brainpower for me14:03
matty_dubs;)14:03
rloomatty_dubs: human language is way harder than programming languages!14:04
matty_dubsI think long-term this stuff should probably live somewhere in git under Ironic, so the community can keep stuff up-to-date vs. having out-of-date blog posts, but I'm happy to revise this14:04
matty_dubsEspecially now that I'm the face of OpenStack. ;)14:04
* rloo wonders if matty_dubs' head is a bit swollen14:05
matty_dubslol, I came into work and someone asked if I had seen the Paris page.14:05
matty_dubsI have attempted to get as much mileage out of it as possible.14:05
rloomatty_dubs: yeah, that's great. Someone I know is famous!14:05
rloomatty_dubs: people will be asking for your autograph in Paris. You have to be there!14:06
matty_dubsI'm actually not even sure yet if I've been approved to go; I'm trying to use my newfound celebrity status to my advantage, though. ;)14:07
matty_dubs"I'm in a thumbnail! You have to send me!"14:07
rloomatty_dubs: don't forget to say 'voila' first.14:08
rloomatty_dubs: 'voila, je suis dans cette photo. Je dois aller a Paris'.14:08
matty_dubsViola!14:09
matty_dubs;)14:09
*** rloo has quit IRC14:09
*** rloo has joined #openstack-ironic14:09
*** rloo has quit IRC14:15
*** rloo has joined #openstack-ironic14:15
dtantsurlucasagomes, what do you think about creating a new API endpoint for new node callback (in process of autodiscovery)?14:16
*** rloo has quit IRC14:16
dtantsurlucasagomes, IPA is doing it via vendor passthru, but I'm not sure it's actually ramdisk-specific14:16
*** rloo has joined #openstack-ironic14:16
lucasagomesdtantsur, right, yeah it looks kinda generic14:17
lucasagomesdtantsur, I think it's fine yeah, something like /nodes/autodiscovery ?14:17
dtantsurlucasagomes, yeah. Next question to discuss: on a new server, should we just create a Node in maintenance mode or should we introduce some new DB model?14:19
*** martyntaylor has quit IRC14:19
*** Poornima|mtg has quit IRC14:20
NobodyCamgood morning iRonic14:21
dtantsurmorning NobodyCam, when did we get bought by Apple? :)14:22
*** martyntaylor has joined #openstack-ironic14:22
lucasagomesdtantsur, i'd say maintenance is fine14:22
NobodyCamlol14:22
NobodyCammorning dtantsur lucasagomes14:22
lucasagomesdtantsur, we def should have a manual confirmation after the node is being discovered14:22
lucasagomesNobodyCam, morning14:22
dtantsurlucasagomes, maintenance is a good confirmation?14:23
NobodyCamlucasagomes: ++ thou I have heard people also want auto enroll14:23
*** Poornima|mtg has joined #openstack-ironic14:23
NobodyCamfyi I have not read the scroll back14:23
dtantsurNobodyCam, I don't want to address this case in this spec - too many security concerns14:24
*** geekyogi has quit IRC14:25
lucasagomesdtantsur, hmm I think yes?14:25
lucasagomesdtantsur, u think we need something more specific?14:26
lucasagomeslike "recently discovered resources"14:26
dtantsurlucasagomes, I'm more than ok with leaving it with maintenance=True. Maybe just some additional flag, so that we can filter all newly-discovered nodes14:26
dtantsurNobodyCam, context is something appearing here: https://etherpad.openstack.org/p/IronicDownstreamDiscoveryRamdisk14:28
lucasagomesdtantsur, sure, right maybe a flag in extra?14:29
lucasagomesor something14:29
lucasagomesif needed14:30
dtantsurlucasagomes, in extra? Will it allow searching?14:30
lucasagomesdtantsur, not but we have plans to make some filtering there14:30
dtantsurlucasagomes, my concern is allow TripleO UI search all newly discovered HW14:30
matty_dubsathomas: lucasagomes: I think I'm going to submit my NetApp blueprint/spec today. Unless you think that's a bad idea?14:31
dtantsurmatty_dubs, blueprint is never a bad idea14:32
dtantsurmatty_dubs, NEED MOAR BLEUPRINT!!114:32
matty_dubsHa, well I meant if it needed further revisions14:32
matty_dubsBut this was also the wrong window ;)14:32
matty_dubsI didn't want to show up with something that was junk.14:32
* dtantsur likes showing up with junk14:33
matty_dubsMaybe that way people can get their -1's out of their system early ;)14:33
lucasagomesdtantsur, right, we migth need the extra to be filterable anyway14:34
matty_dubsUnless rloo triggers another infinite-loop-of-email bug ;)14:34
lucasagomesdtantsur, to allow bouding nodes with roles14:34
lucasagomesdtantsur,  by using certain tags14:34
lucasagomesmatty_dubs, +1 can't think why it would be a bad idea really14:34
dtantsurlucasagomes, do we want to depend on this yet-to-draft feature? Also, index on this new field won't hurt in some envs14:35
rloomatty_dubs: anytime you want, just let me know. Although I'm not sure I can replicate that. Maybe I should try ;)14:35
* lucasagomes is in the scrum call... 1 sec (it's hard to type and talk on the phone at the same time)14:35
matty_dubslol14:35
lucasagomesdtantsur, def not :)14:35
*** eguz has joined #openstack-ironic14:36
*** eguz has quit IRC14:36
mgagneIs ServiceNet bandwidth billed on onMetal?14:38
lucasagomesjroll, JayF russell_h ^14:39
mgagnelucasagomes: thanks =)14:39
dtantsurmgagne, this folks can answer, but I believe some official support will answer much faster14:39
dtantsurlucasagomes and all interested in discovery: have a look at what is shaping at https://etherpad.openstack.org/p/IronicDownstreamDiscoveryRamdisk please (description and proposed changes)14:40
dtantsurtime to collect some feedback :)14:40
lucasagomes+114:40
*** eghobo has quit IRC14:41
*** rloo has quit IRC14:44
*** rloo has joined #openstack-ironic14:45
*** mdorman has joined #openstack-ironic14:47
*** rwsu has joined #openstack-ironic14:49
*** rloo has quit IRC14:52
*** rloo has joined #openstack-ironic14:53
*** rloo has quit IRC14:54
*** rloo has joined #openstack-ironic14:55
*** rloo has joined #openstack-ironic14:56
*** rloo has quit IRC14:57
*** rloo has joined #openstack-ironic14:57
NobodyCamlucasagomes: is there a ironic yum package?14:59
lucasagomesNobodyCam, I think for rawhide there's yes15:00
lucasagomesNobodyCam, I know a guy who was working on that15:01
NobodyCam:)15:01
NobodyCamalso lucasagomes did you see the paramaiko / eventlet fix15:02
lucasagomesNobodyCam, yup just saw his message on the maillist15:02
lucasagomesgood stuff!15:02
NobodyCam:) ya15:02
*** rloo_ has joined #openstack-ironic15:03
dtantsurfolks, any comments on spec draft? maybe ideas for stated TODO's?15:03
*** rloo has quit IRC15:04
*** Poornima|mtg has quit IRC15:05
*** rloo_ has quit IRC15:06
*** rloo has joined #openstack-ironic15:06
*** rloo has quit IRC15:09
*** rloo has joined #openstack-ironic15:09
NobodyCamdtantsur: do you want a section listing all the info you can / want to auto discover? Ie many bmc's (like iLo, and Drac) have chassis Id's and other info too15:14
NobodyCamor is this just cpu,disk,Mac, and ram15:14
dtantsurNobodyCam, I'd have a list of specs, that are common subset of all technologies15:14
dtantsurNobodyCam, we can always extend it in a particular spec e.g. for IPA15:15
dtantsurI'd like to talk to jroll on what we can put there15:15
NobodyCamdtantsur: also when I was looking at auto discovery I was conserned about detecting which eth ports actually had a cable pluged in. thou I cannt recall why now15:17
*** geekyogi has joined #openstack-ironic15:19
dtantsurmakes sense, yeah15:20
dtantsurone more question: should I account for the reverse flow: when there's some management interface, which can be asked for details on a node15:20
NobodyCamdtantsur: would that be for things like I upgreaded / replaced the cpu / /disk / ram re discover the changes?15:21
openstackgerritMatt Wagner proposed a change to openstack/ironic-specs: Driver for NetApp storage arrays  https://review.openstack.org/10255715:25
*** killer_prince is now known as lazy_prince15:29
romchegMorning guys again :)15:29
dtantsurNobodyCam, e.g. what is supported by iLO: https://review.openstack.org/#/c/100951/7/specs/juno/discover_hardware_properties.rst15:30
dtantsurromcheg, morning again :)15:30
romcheghttp://paste.openstack.org/show/84895/ — that's how flavor update tool is working now15:30
romchegAny suggestions?15:30
NobodyCammorning romcheg15:31
*** lazy_prince is now known as killer_prince15:31
dtantsurromcheg, lgtm15:32
romchegBoth kernel and ramdisk UUIDs are the same because I was too lazy to generate more uuids :)15:32
romchegOk, so I'm going to add some comments and post that as well15:33
* dtantsur is starting to move things to specs repo15:34
*** rloo has quit IRC15:40
*** rloo has joined #openstack-ironic15:41
*** rloo has quit IRC15:41
*** rloo has joined #openstack-ironic15:42
*** killer_prince is now known as lazy_prince15:43
openstackgerritDmitry Tantsur proposed a change to openstack/ironic-specs: Generic Hardware Discovery Bits  https://review.openstack.org/10256515:46
dtantsurlucasagomes, NobodyCam, here we go ^^^15:47
*** lazy_prince is now known as killer_prince15:48
*** jcoufal is now known as jcoufal_bbl15:48
*** rloo has quit IRC15:50
*** rloo has joined #openstack-ironic15:50
*** rloo has quit IRC15:53
*** shausy has quit IRC15:53
*** rloo has joined #openstack-ironic15:53
*** shausy has joined #openstack-ironic15:54
jrollmgagne: this isn't a rackspace channel nor an onmetal channel :/ that said, bandwidth should be billed identical to cloud servers today (which I believe means free servicenet bandwidth)15:54
*** bvivek has quit IRC15:54
*** rloo has quit IRC15:54
mgagnejroll: I'll check with support then, my question was related to the technical aspect of it as AFAIK, both VLANs are trunked on the bond.15:54
*** bvivek has joined #openstack-ironic15:55
*** rloo has joined #openstack-ironic15:55
jrolldtantsur: morning! someone (agordeev?) was working on a super awesome hardware manager that sends tons of info15:55
dtantsurjroll, yeah, I've seen the code15:55
mgagnejroll: and was wondering how ceilometer could make a distinction between both network bandwidth measurement15:55
*** rloo has quit IRC15:56
jrolldtantsur: I'm taking pto today for my girlfriend's birthday so can we talk tomorrow? :)15:56
dtantsurjroll, I want to have some minimum subset, that we _require_ from any implementation (even less awesome that yours :)15:56
*** eghobo has joined #openstack-ironic15:56
dtantsurjroll, sure! I'll concentrate on other things! Say "happy birthday" to her for us :)15:56
jrolldtantsur: sure :)15:56
jrollheh will do15:57
*** eghobo has quit IRC15:57
matty_dubslucasagomes: D'oh, the mem/max_mem difference went right over my head until you explained it15:57
*** eghobo has joined #openstack-ironic15:57
*** rloo has joined #openstack-ironic15:57
jrollmgagne: so we're still working out the details, to be honest. also, we don't use ceilometer for billing15:57
*** viktors|afk is now known as viktors15:58
*** rloo has quit IRC15:58
lucasagomesmatty_dubs, heh :)15:58
lucasagomesmatty_dubs, no problem15:58
mgagnejroll: oh right hehe thanks =)15:58
*** rloo has joined #openstack-ironic15:58
jrollmgagne: we'll need to do interesting things with switches to bill it properly15:58
jrollmgagne: that's about all I know, but I'm not really focused on that stuff. russell_h may be able to elaborate15:59
mgagnejroll: alright, we have the same challenge then =)15:59
jroll:)15:59
*** rloo has quit IRC16:00
*** rloo has joined #openstack-ironic16:00
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Add/Update docstrings in the Nova Ironic Driver  https://review.openstack.org/9753616:01
russell_hmgagne: so we don't actually use ceilometer, but the idea is to use a QoS policy to measure bandwidth separately on each network16:02
mgagnerussell_h: that would be per network and not per port right?16:02
russell_hwe can actually get per-network per-port16:02
lucasagomesjroll, those dependencies you've on ur patch doesn't seems right... those patches factoring out things on the PXE driver are depending on the swift tmp url!?16:02
mgagnerussell_h: cool, I'll check that out, thanks for the info16:03
*** rloo has quit IRC16:04
*** rloo has joined #openstack-ironic16:04
openstackgerritDmitry Tantsur proposed a change to openstack/ironic-specs: Generic Hardware Discovery Bits  https://review.openstack.org/10256516:04
*** notq has joined #openstack-ironic16:04
dtantsur^^^ still incomplete but I have to go. do not hesitate to throw in your ideas!16:05
*** dtantsur is now known as dtantsur|afk16:06
*** matty_dubs is now known as matty_dubs|lunch16:07
JayFIf someone who is core on IPA wants to help a first time Openstack'er get their first patch in, this is a quick+easy review: https://review.openstack.org/#/c/102458/ -- I've already +2'd it.16:17
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Remove from_chassis/from_nodes from the API doc  https://review.openstack.org/9971616:22
*** rloo has quit IRC16:24
*** rloo has joined #openstack-ironic16:25
*** rloo has quit IRC16:25
*** rloo has joined #openstack-ironic16:25
*** rloo has quit IRC16:26
*** vinbs has joined #openstack-ironic16:26
*** rloo has joined #openstack-ironic16:26
*** rloo has quit IRC16:27
*** rloo has joined #openstack-ironic16:27
*** Penick_ has quit IRC16:36
*** shausy has quit IRC16:37
*** vinbs has quit IRC16:38
JoshNangJayF: +A!16:39
*** Mikhail_D_ltp has quit IRC16:40
openstackgerritA change was merged to openstack/ironic-python-agent: Converted documentation in md format to rst  https://review.openstack.org/10245816:41
*** ellenh has joined #openstack-ironic16:41
*** dwalleck has joined #openstack-ironic16:46
JayFAnd another Openstack developer gets their wings16:47
*** matty_dubs|lunch is now known as matty_dubs16:50
*** viktors is now known as viktors|afk16:58
*** achanda has joined #openstack-ironic17:00
*** dwalleck_ has joined #openstack-ironic17:05
*** dwalleck has quit IRC17:06
*** rloo has quit IRC17:06
*** rloo has joined #openstack-ironic17:06
*** rloo has quit IRC17:07
*** rloo has joined #openstack-ironic17:08
*** rloo has quit IRC17:08
*** rloo has joined #openstack-ironic17:08
*** dwalleck_ has quit IRC17:09
*** rloo has quit IRC17:09
*** dwalleck has joined #openstack-ironic17:09
*** rloo has joined #openstack-ironic17:10
*** rloo has quit IRC17:10
*** rloo has joined #openstack-ironic17:11
*** rloo_ has joined #openstack-ironic17:12
lucasagomesalright folks I gotta go17:12
lucasagomesg'night everyone17:12
*** harlowja_away is now known as harlowja17:13
matty_dubsGood night, lucasagomes17:14
*** lucasagomes is now known as lucas-dinner17:14
NobodyCamnight lucas-dinner17:15
*** rloo has quit IRC17:16
*** rloo_ has quit IRC17:23
*** geekyogi has quit IRC17:23
*** rloo has joined #openstack-ironic17:23
openstackgerritEllen Hui proposed a change to openstack/ironic: Add more INFO logging to ironic/conductor  https://review.openstack.org/9974417:25
*** faizan has joined #openstack-ironic17:30
*** digambar has joined #openstack-ironic17:30
*** Mikhail_D_ltp has joined #openstack-ironic17:31
*** rloo has quit IRC17:36
*** rloo has joined #openstack-ironic17:37
*** saripurigopi has joined #openstack-ironic17:37
*** rloo has quit IRC17:38
saripurigopiI'm trying to test Cisco plugin on devstack,  while creating network-profile it is failing with 404 error.17:38
*** rloo has joined #openstack-ironic17:38
saripurigopineutron cisco-network-profile-create netprof vlan --segment_range 100-200 --physical_network physnet117:38
saripurigopi 404 Not Found  The resource could not be found.17:38
saripurigopiany idea why is it failing. ?17:39
*** rloo has quit IRC17:40
*** rloo has joined #openstack-ironic17:40
*** ellenh has quit IRC17:42
*** rloo has left #openstack-ironic17:42
Shrewssaripurigopi: sounds like a question for #openstack-neutron17:43
*** rloo_ has joined #openstack-ironic17:43
*** Penick has joined #openstack-ironic17:47
*** ellenh has joined #openstack-ironic17:47
openstackgerritEllen Hui proposed a change to openstack/ironic: Add more INFO logging to ironic/common  https://review.openstack.org/9949517:50
*** faizan has quit IRC17:53
*** faizan has joined #openstack-ironic17:54
*** digambar has quit IRC17:54
matty_dubsCan someone remind me how to view comments on previous versions of a patch in Gerrit?17:55
matty_dubsOh, nevermind, I'm a dummy17:55
matty_dubsThe answer is "Make sure you're clicking on a patch set that has comments"17:55
rloo_matty_dubs, I was wondering what you meant... :D17:56
matty_dubsHaha, I thought it must have changed. I was looking at the wrong version, which had no comments. *sigh*17:56
rloo_matty_dubs, no comments could be a good thing ;)17:58
matty_dubsHaha, yeah, the *sigh* was at my idiocy.17:58
faizanhello devananda17:59
faizanhello dtantsur18:00
faizandevananda: dtantsur: request you to provide some more review comments on this spec https://review.openstack.org/#/c/99850/18:01
*** rameshg87 has joined #openstack-ironic18:06
*** Nisha has joined #openstack-ironic18:08
romchegGood morning devananda!18:11
devanandamorning, folks18:11
devanandaI'll be semi-afk today and tomorrow18:11
*** coolsvap is now known as coolsvap|afk18:11
*** bvivek has quit IRC18:12
*** faizan has quit IRC18:12
Shrewsdevananda: semi-hi, then  :)18:12
devanandamatty_dubs: thanks for writing docs! ;)18:16
romchegdevananda: What about docs for Nova=>Ironic migration stuff?18:16
romchegShould we start creating docs for that?18:16
devanandaromcheg: we'll need to provide some usage docs with the tool18:17
NobodyCammorning devananda18:20
NobodyCamrloo_: around?18:20
rloo_NobodyCam, yeah, I'm here.18:20
NobodyCamsaw the _ wantedto check18:20
NobodyCam:)18:20
rloo_matty_dubs, yeah, thx for those docs! ;)18:20
rloo_NobodyCam, what _wantedto check?18:21
NobodyCamrloo_: happen to whats up with https://review.openstack.org/#/c/8378818:21
matty_dubsBut... but...18:21
NobodyCamor should I say happen to know what its fate is going to be?18:21
rloo_NobodyCam, oh. I don't know what lucas-dinner decided. I think he was going to use MB18:22
rloo_NobodyCam, I muddied the waters by saying maybe we should support both/mix. allow the user to use whatever units they want.18:23
NobodyCamieek  both :-p18:23
rloo_NobodyCam: cuz I want operators to like me ;)18:23
rameshg87hello devananda18:24
NobodyCamrameshg87: hey18:24
rameshg87hello NobodyCam18:24
NobodyCamI had a question18:24
rameshg87yes NobodyCam18:24
rloo_NobodyCam, it is just a mechanism for specifying information. I don't think it matters much really, but nice to make it easy for folks to specify values. Eg if 10 GB, do you really want to specify that in MB. Why bother?18:24
NobodyCamI was looking over the Ilo driver spec, and see the default port is 443, can Ilo support the advanced feathers over the ssh connection?18:25
NobodyCamI ask bevause I see no key_auth support called for in the spec18:25
*** yu_ has joined #openstack-ironic18:26
rameshg87NobodyCam, ilo communicates over https over 443, but can't support key based authentication for ribcl communication18:26
rameshg87NobodyCam, is that what you meant ?18:26
rameshg87NobodyCam, in short we will need username and password to be communicated everytime we make a ribcl call18:27
*** romcheg has quit IRC18:27
rloo_NobodyCam, do you want to also do the honours and approve 99716?18:28
NobodyCamrameshg87: ok, thank you18:28
rameshg87NobodyCam, thanks. request you also to share your thoughts on the ilo power design spec :-)18:29
NobodyCamrloo_: doh thought I had18:29
NobodyCam:-p18:29
rameshg87devananda, i have addressed your comments on ilo power design spec. please have a look at that.  https://review.openstack.org/#/c/9745518:30
NobodyCamrloo_: fixed18:30
rloo_thx NobodyCam!18:31
openstackgerritEllen Hui proposed a change to openstack/ironic: Add more INFO logging to ironic/drivers/modules  https://review.openstack.org/9943118:32
*** igordcard has quit IRC18:38
*** pelix has quit IRC18:39
*** rameshg87 has left #openstack-ironic18:51
*** ellenh has quit IRC18:52
*** harlowja has quit IRC18:52
*** Penick has quit IRC18:53
*** romcheg has joined #openstack-ironic18:55
*** Penick has joined #openstack-ironic18:59
*** martyntaylor has left #openstack-ironic19:01
openstackgerritA change was merged to openstack/ironic: Remove from_chassis/from_nodes from the API doc  https://review.openstack.org/9971619:03
*** openstackgerrit has quit IRC19:04
*** openstackgerrit has joined #openstack-ironic19:05
*** zdiN0bot has joined #openstack-ironic19:06
*** Penick has quit IRC19:06
*** yjiang5 is now known as yjiang5_away19:08
*** zdiN0bot has quit IRC19:08
*** Penick has joined #openstack-ironic19:11
*** Penick has quit IRC19:11
openstackgerritAdam Gandelman proposed a change to openstack/ironic: Update Nova's available resources at termination  https://review.openstack.org/10237319:11
openstackgerritAdam Gandelman proposed a change to openstack/ironic: Update Nova's available resources at termination  https://review.openstack.org/10237319:13
*** achanda_ has joined #openstack-ironic19:14
*** achanda has quit IRC19:17
devanandaadam_g: comments on rev2 ^19:17
adam_gdevananda, yeah, i caught the typos as soon as i pushed it :)  whats the race condition?19:18
devanandaadam_g: see comments :)19:19
*** jcoufal_bbl has quit IRC19:28
*** jbjohnso has joined #openstack-ironic19:28
openstackgerritAdam Gandelman proposed a change to openstack/ironic: Update Nova's available resources at termination  https://review.openstack.org/10237319:36
*** pcrews has quit IRC19:39
*** pcrews_ has joined #openstack-ironic19:39
*** dwalleck_ has joined #openstack-ironic19:40
*** dwalleck has quit IRC19:40
*** ellenh has joined #openstack-ironic19:47
*** romcheg has quit IRC19:49
openstackgerritlinggao proposed a change to openstack/ironic: Fix exception handling in console start  https://review.openstack.org/10231819:54
*** dwalleck_ has quit IRC19:56
ShrewsIs there anything that states we *must* provide bookmark links?19:58
Shrewstrying to understand the api a bit better and wonder what use those links are19:59
*** Nisha has quit IRC20:00
*** romcheg has joined #openstack-ironic20:01
*** openstackgerrit has quit IRC20:05
*** openstackgerrit has joined #openstack-ironic20:06
*** max_lobur has quit IRC20:07
rloo_Shrews: if I recall correctly, those bookmarks are useful for pagination.20:07
Shrewsrloo_: in nova, they're for "long term storage": http://docs.openstack.org/api/openstack-compute/2/content/LinksReferences.html20:09
Shrewsbut i still don't see the usefulness. especially if we were to have multiple api versions some day20:09
Shrews"A bookmark link provides a permanent link to a resource that is appropriate for long term storage."20:10
Shrewseh, i think i've gone down a rathole with this and should move on  :)20:10
rloo_Shrews: I think the usefulness of bookmarks (or the idea of usefulness) is that if you have multiple versions (we only have v1 now), the bookmark link will know which version to use.20:11
NobodyCambrb quick walkies20:11
rloo_Shrews, if you look at the bookmark link, it doesn't have 'v1' in it. And there's something in the code that will xlate that link to v1 or v2 or whatever.20:11
rloo_Shrews: I think that's what they mean by 'long term storage'. ha ha.20:12
Shrewsrloo_: yep, it's 1-to-1 with the v1 link. but if it's for storing long term, won't it *always* be tied to the first version (v1 in our case)?20:12
rloo_Shrews, nope. it depends on the app. I suspect typically, if you have newer versions, you want the 'general' link to go to the best/newest version.20:13
*** alex_______ has joined #openstack-ironic20:13
rloo_Shrews, so that one could eg deprecate an earlier version or whatever. I don't think we're/ironic needs to think about that for awhile ;)20:14
Shrewsrloo_: for our async api improvements we might  :)20:14
linggaoHi, I have a git question. In order to make a dependency, I did "git review -d xxx", it created a branch like review/linggao/console_fix_exception.  Then how do I update this branch if a new patch is checked in?20:15
rloo_Shrews, are we thinking of v2 for the async stuff? guess i should read that spec one day.20:15
rloo_linggao, do you mean, how do you  rebase your patch if master has changed?20:16
linggaorloo_, no I need to rebase my code this patch, but this patch has been updated.20:17
rloo_linggao, what is the dependency? the patch that you got from doing git review -d ...?20:17
Shrewsrloo_: yeah. but having the "general" link point to the latest version seems to be a valid use case. thx for that20:17
linggaorloo_, yes.20:17
alex_______Hi guys, I have a quick question about the usage of Ironic. Do we absolutely need IPMI to make the solution work?20:17
rloo_Shrews, glad it made sense. Now you can sleep better ;)20:17
*** achanda has joined #openstack-ironic20:18
linggaorloo_, I need to get the latest code for the dependency.20:18
matty_dubsalex_______: Today you mostly do for 'real' servers, though you can test using VMs. There's a spec to support other means of power control, though: https://review.openstack.org/#/c/96838/20:19
*** achanda_ has quit IRC20:20
rloo_linggao, I think (can't recall for sure) that you can do 'git pull --rebase origin master'20:21
linggaorloo_, you mean on the dependency branch?20:21
rloo_linggao, in the branch with your patch.20:22
alex_______matty_dubs: I see thanks. So does that mean that for older hardwares, let's say workstations that are not IPMI compliant, it wouldn't be possible to integrate them as nodes in the Ironic pool at all, even for testing?20:22
rloo_linggao, did you already submit your patch?20:22
*** athomas has quit IRC20:22
matty_dubsalex_______: We were talking about this very thing at Summit, actually. I'm honestly not sure what the answer is right now, if there's some way to do manual power control. I'm not sure that there is.20:24
matty_dubsAnyone else know? ^20:24
linggaorloo_, here is a long story.  I have 2 patches, one depends on the other. Say A depens on B. I checked in set1 for B.20:24
linggaoand put A to depens on A, and checked in A.20:24
matty_dubsIt seems like if there were a no-op power driver, you could do PXE and just manually reboot the machine for testing. Obviously not great in a data center, but it'd make testing on a desktop or whatnot easier.20:24
linggaonow I worked on B and checked in set2 for B.20:25
alex_______Ok thanks.20:25
linggaorloo_, now how do I rebase A on set2 of B?20:25
rloo_matty_dubs, alex_______ : it seems to me that Ironic right now, won't be able to power those nodes. BUT if there is some API to power those nodes, someone could write a driver to do so ;)20:25
alex_______rloo_: What would be the technical requirements to make that work? I'm just starting to get familiar with the Ironic code base right now.20:26
rloo_matty_dubs, alex_______ yeah... i guess if you want to just do testing  you can probably manually fake some stuff. not sure how much.20:26
linggaoI know if A did not have dependency, I can simply go to master branch and run 'git pull origin master' and then go to A and do 'git rebase master'20:27
rloo_alex_______, to add a new driver, or to manually power/on fake as matty_dubs suggested?20:27
rloo_linggao, sorry, i'm not good at multitasking. my brain works slowly. by 'checked in set1 for B', the code has been merged or there is a patch up for review?20:27
linggaorloo_, now since A depends on B, what is the equivalency of "git pull origin master' for B?20:28
linggaorloo_, the code has not been merged.20:28
rloo_ok, linggao. so B patch is OK. You want to update A with the changes from B.20:29
linggaorloo_, yes.20:29
Shrewsalex_______: this is server class h/w?20:29
rloo_linggao, I think if you are in your branch for A, and do that git thing I mentioned, it should work.20:29
openstackgerritEllen Hui proposed a change to openstack/ironic: Add more INFO logging to ironic/conductor  https://review.openstack.org/9974420:30
matty_dubsalex_______: The problem is that I'm not familiar enough with that section of the code to be able to say exactly what needs to be done. You might be able to hack the SSH or IPMI driver to just no-op some of the operations for now?20:30
linggaorloo_, your said 'git pull --rebase origin master', but A depends on B instead of master.20:30
alex_______matty_dubs: Ok I'll look at the IPMI driver and hack around it ;)20:31
JoshNangI factored part of the Swift Temp URL patch into swiftclient: https://review.openstack.org/#/c/102632/20:32
rloo_linggao, sorry, I could be wrong (ie, note that I stashed on another machine may be incorrect). Take a look at https://wiki.openstack.org/wiki/GerritWorkflow#Add_dependency20:33
linggaorloo_, do not worry, I read that doc. I'll ask lucas-dinner tomorrow since he has done a lot of dependencies.20:34
*** Mikhail_D_ltp has quit IRC20:35
rloo_linggao, that link looks correct. So in your working directory, you do the git review -d <yourB>, then git checkout <yourA branch>, then git rebase -i <yourB branch> etc.20:36
Shrewswe should just use cvs20:37
* Shrews giggles20:37
rloo_Shrews: drugs?! :-)20:37
linggaorloo_, I did all that already.  The problem is that B got changed. I need to rebase A with B. I do not know how to get the latest B.20:37
Shrewsrloo_: lol, no, not that cvs  :)20:38
rloo_linggao, that git review -d <b> gets b and puts it in a branch. the 'git rebase <bbranch> rebases the branch you're on, with code from B. Or so I understand it. if it doesn't work for you , then I dunno.20:38
rloo_Shrews, heh.20:38
rloo_linggao, sorry, are you sure that A depends on B?20:39
*** achanda has quit IRC20:39
rloo_linggao, actually, i'm going to disappear. I need to get something done.20:39
linggaorloo_, A depens on the the branch that created by "git review -d <b>"20:40
linggaorloo_, thanks a lot for yor patience on my.20:40
openstackgerritEllen Hui proposed a change to openstack/ironic: Add more INFO logging to ironic/common  https://review.openstack.org/9949520:43
*** alex_______ has quit IRC20:43
NobodyCamShrews: did you ever see the move svn to git joke?20:47
matty_dubsNobodyCam: Did you ever see the TCP joke?20:47
matty_dubshttps://s3.amazonaws.com/uploads.hipchat.com/43791/814522/oOOPppLWZMQv3Q1/TCPJoke.png20:47
matty_dubs^ someone sent me that earlier and you just reminded me of it20:47
NobodyCammatty_dubs: lol20:49
NobodyCamhttps://issues.apache.org/jira/browse/INFRA-752420:49
NobodyCaminital versions didn't have the AF in the title20:49
matty_dubsWow, this is like a tremendous flamewar and I'm not sure if the people commenting on it are in on it or not?20:52
*** yjiang5_away is now known as yjiang520:52
*** romcheg has quit IRC20:52
NobodyCamlol20:52
* matty_dubs just reached the end, concedes that this was pretty awesome20:56
NobodyCam:) ya I loved it :)20:57
matty_dubsSpeaking of TCP jokes... I'd send you a UDP joke, but you might not get it.20:58
NobodyCamlol20:58
NobodyCammy firewall would block for sure20:59
*** linggao has quit IRC20:59
matty_dubsOK, I've got to head out for the day. Have a good one, y'all.21:03
NobodyCamnight matty_dubs21:03
*** matty_dubs is now known as matty_dubs|gone21:04
*** achanda has joined #openstack-ironic21:05
*** Penick has joined #openstack-ironic21:06
*** bandicot has joined #openstack-ironic21:07
*** jdob has quit IRC21:10
*** foexle has quit IRC21:24
*** harlowja has joined #openstack-ironic21:28
*** Penick has quit IRC21:29
*** mrda-away is now known as mrda21:37
mrdaMorning Ironic!21:37
NobodyCammorning mrda21:37
mrda\o21:38
*** alex_______ has joined #openstack-ironic21:39
*** jbjohnso has quit IRC21:40
openstackgerritAdam Gandelman proposed a change to openstack/ironic: Wait for Neutron port updates when using SSHPower  https://review.openstack.org/9171921:55
*** yjiang5 is now known as yjiang5_away22:02
*** yjiang5_away has quit IRC22:02
NobodyCambbiafm22:18
*** achanda has quit IRC22:18
*** achanda_ has joined #openstack-ironic22:20
*** jbjohnso has joined #openstack-ironic22:26
*** ellenh has quit IRC22:31
*** ellenh has joined #openstack-ironic22:32
*** radsy has joined #openstack-ironic22:34
*** radsy has joined #openstack-ironic22:34
*** mdorman has quit IRC22:35
*** openstackgerrit has quit IRC22:35
*** mdorman has joined #openstack-ironic22:36
*** openstackgerrit has joined #openstack-ironic22:36
*** rloo_ has quit IRC22:47
openstackgerritAdam Gandelman proposed a change to openstack/ironic: Wait for Neutron port updates when using SSHPower  https://review.openstack.org/9171922:54
*** pcrews_ has quit IRC23:00
*** pcrews has joined #openstack-ironic23:00
*** mdorman has quit IRC23:07
*** jbjohnso has quit IRC23:17
*** mdorman has joined #openstack-ironic23:26
*** mdorman has quit IRC23:29
NobodyCambrb23:34
*** lucas-dinner has quit IRC23:38
*** r-daneel has joined #openstack-ironic23:48
*** hemna is now known as hemna_23:49
openstackgerritJay Faulkner proposed a change to openstack/ironic-python-agent: Add infra build script to IPA  https://review.openstack.org/10267323:51
*** r-daneel has quit IRC23:52
JayFIf I could get a review on ^ that I'd appreciate it23:54
JayFthe primary reason I made it is so I can test the build of the ipa-coreos image in infra23:54

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