| devananda | was coding. gonna be in another meeting in ~30 | 00:00 |
|---|---|---|
| devananda | lets see if i cna make some headway on the hash rings now | 00:00 |
| NobodyCam | oh side note: monday after meeting I have to go offline and take the house to the folks making the tow bar for the car :) | 00:01 |
| *** marcoemorais has quit IRC | 00:12 | |
| *** marcoemorais has joined #openstack-ironic | 00:12 | |
| openstackgerrit | Yuiko Takada proposed a change to openstack/python-ironicclient: Add unit tests for "ironic node-show" shell cmd https://review.openstack.org/120001 | 00:13 |
| *** penick has quit IRC | 00:18 | |
| openstackgerrit | Yuiko Takada proposed a change to openstack/python-ironicclient: Add unit tests for "ironic node-create" shell cmd https://review.openstack.org/120312 | 00:24 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic-python-agent: Adding support for decommissioning https://review.openstack.org/104379 | 00:24 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic-python-agent: Add versioning to Agent decommission https://review.openstack.org/106859 | 00:28 |
| *** hemna has quit IRC | 00:33 | |
| *** marcoemorais has quit IRC | 01:14 | |
| *** killer_prince is now known as lazy_prince | 01:18 | |
| *** penick has joined #openstack-ironic | 01:35 | |
| *** penick_ has joined #openstack-ironic | 01:37 | |
| *** penick has quit IRC | 01:39 | |
| *** penick_ is now known as penick | 01:39 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Add HashRingManager to wrap hash ring singleton https://review.openstack.org/120951 | 01:40 |
| * devananda goes out for the evening | 01:41 | |
| *** nosnos has joined #openstack-ironic | 01:45 | |
| *** HenryG has joined #openstack-ironic | 01:49 | |
| *** r-daneel has quit IRC | 01:56 | |
| *** Poornima has joined #openstack-ironic | 02:19 | |
| *** Haomeng has joined #openstack-ironic | 02:24 | |
| rloo | Haomeng: hi. what's the difference between bugs 1374227 & 1374226? | 02:25 |
| *** Haomeng|2 has quit IRC | 02:25 | |
| rloo | Haomeng: also, isn't that what lucas fixed earlier today? | 02:26 |
| rloo | Haomeng: https://review.openstack.org/124090. Or did it break again after that patch? | 02:26 |
| Haomeng | rloo: yes, I see we still have difference with latest ironic.conf.sample | 02:27 |
| Haomeng | rloo: I understand lucus's patch fix the issue, but not sure why jekins not picking the latest sample file | 02:28 |
| rloo | Haomeng: I saw one or more patches being merged recently. | 02:28 |
| Haomeng | rloo: yes, let me check the latest keystoneclient options | 02:29 |
| rloo | Haomeng: eg https://review.openstack.org/122743. So I thought things were fixed. | 02:29 |
| Haomeng | rloo: yes, maybe keystoneclient is changed again | 02:29 |
| Haomeng | rloo: let me check | 02:29 |
| rloo | Haomeng: thx for looking into it. I have to leave. But... don't the patches have to rebase first, to get the updated .sample? | 02:30 |
| Haomeng | rloo: yes, good point | 02:31 |
| Haomeng | rloo: I understand jekins will get latest sample and compare the one which is generated with scripts | 02:32 |
| rloo | Haomeng: but if they have to rebase, then how did we get patches merged recently. could be some machines haven't yet gotten the new keystoneclient or whatever package yet. | 02:32 |
| rloo | Haomeng: sorry to leave you with this. Thanks! | 02:32 |
| *** rloo has left #openstack-ironic | 02:32 | |
| Haomeng | rloo: welcome:) | 02:32 |
| Haomeng | let me check:) good night:) | 02:32 |
| Haomeng | so I have the patch try to remove sample conf https://review.openstack.org/106493, to avoid such conflicts again and again:) | 02:34 |
| *** harlowja is now known as harlowja_away | 02:38 | |
| *** penick has quit IRC | 02:43 | |
| *** marcoemorais has joined #openstack-ironic | 02:43 | |
| *** marcoemorais1 has joined #openstack-ironic | 02:45 | |
| *** marcoemorais has quit IRC | 02:48 | |
| *** ramineni has joined #openstack-ironic | 02:48 | |
| ramineni | devananda : hi | 02:49 |
| ramineni | devananda : want to discuss regarding the bug https://bugs.launchpad.net/ironic/+bug/1367182 | 02:49 |
| Haomeng | ramineni: I understand deva is in dream now:) | 02:51 |
| ramineni | Homeng: :) thought so , still wanted to check , by chance he is available :P | 02:52 |
| Haomeng | ramineni: yes:) | 02:53 |
| ramineni | Homeng: not sure about his timings though | 02:53 |
| Haomeng | ramineni: :) | 02:54 |
| Haomeng | ramineni: np | 02:54 |
| ramineni | Haomeng: any idea, still there is chance to fix bugs for juno release? | 02:55 |
| *** nosnos has quit IRC | 02:55 | |
| *** nosnos_ has joined #openstack-ironic | 02:56 | |
| *** nosnos_ has quit IRC | 02:57 | |
| *** nosnos has joined #openstack-ironic | 02:57 | |
| Haomeng | ramineni: we have bug list - https://bugs.launchpad.net/ironic if it is required to fix in juno, we can see the target with high/critical prioity or juno-rc1 milestone | 02:57 |
| ramineni | Haomeng: ok , thanks | 02:59 |
| Haomeng | ramineni: welcome:) | 02:59 |
| *** nosnos has quit IRC | 03:01 | |
| *** pcrews has quit IRC | 03:13 | |
| *** lazy_prince is now known as killer_prince | 03:28 | |
| *** pcrews has joined #openstack-ironic | 03:43 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements https://review.openstack.org/124258 | 03:54 |
| *** yuanying_ has joined #openstack-ironic | 03:59 | |
| *** yuanyin__ has joined #openstack-ironic | 04:02 | |
| *** yuanying_ has quit IRC | 04:02 | |
| *** yuanying has quit IRC | 04:03 | |
| *** Poornima has quit IRC | 04:04 | |
| *** vinbs has joined #openstack-ironic | 04:06 | |
| *** yuanyin__ has quit IRC | 04:10 | |
| *** yuanying has joined #openstack-ironic | 04:15 | |
| *** nikunj2512 has joined #openstack-ironic | 04:22 | |
| *** killer_prince is now known as lazy_prince | 04:24 | |
| *** nosnos has joined #openstack-ironic | 04:28 | |
| *** Poornima has joined #openstack-ironic | 04:42 | |
| *** Poornima has quit IRC | 04:43 | |
| openstackgerrit | Yuiko Takada proposed a change to openstack/python-ironicclient: Add "ironic node-set-power-state" cmd unit test https://review.openstack.org/124267 | 04:45 |
| *** lazy_prince is now known as killer_prince | 04:48 | |
| *** rakesh_hs has joined #openstack-ironic | 04:49 | |
| *** Poornima has joined #openstack-ironic | 04:59 | |
| *** yuanying_ has joined #openstack-ironic | 05:01 | |
| *** killer_prince is now known as lazy_prince | 05:02 | |
| *** yuanying has quit IRC | 05:04 | |
| *** pcrews has quit IRC | 05:23 | |
| *** pcrews has joined #openstack-ironic | 05:30 | |
| *** rushiagr_away is now known as rushiagr | 05:41 | |
| *** loki184 has joined #openstack-ironic | 05:42 | |
| *** todd_dsm has quit IRC | 06:07 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/123342 | 06:11 |
| openstackgerrit | Motohiro 'Yuanying' Otsuka proposed a change to openstack/python-ironicclient: Remove unused command in tox.ini https://review.openstack.org/124303 | 06:26 |
| *** keekz has quit IRC | 06:28 | |
| *** yarker has quit IRC | 06:28 | |
| *** devananda has quit IRC | 06:29 | |
| *** yarker has joined #openstack-ironic | 06:31 | |
| *** devananda has joined #openstack-ironic | 06:31 | |
| openstackgerrit | A change was merged to openstack/ironic: Remove untranslated PO files https://review.openstack.org/124141 | 06:32 |
| *** keekz has joined #openstack-ironic | 06:35 | |
| *** k4n0 has joined #openstack-ironic | 06:37 | |
| *** pcrews has quit IRC | 06:44 | |
| *** yuanying_ has quit IRC | 06:52 | |
| *** yuanying has joined #openstack-ironic | 06:53 | |
| *** coolsvap|afk is now known as coolsvap | 06:55 | |
| *** yuanying has quit IRC | 06:58 | |
| *** dlaube1 has joined #openstack-ironic | 06:58 | |
| *** dlaube has quit IRC | 06:58 | |
| *** yuanying has joined #openstack-ironic | 07:00 | |
| *** pensu has joined #openstack-ironic | 07:03 | |
| *** pensu has quit IRC | 07:09 | |
| *** pensu has joined #openstack-ironic | 07:12 | |
| *** boris-42 has quit IRC | 07:15 | |
| *** yarker has quit IRC | 07:15 | |
| *** athomas has joined #openstack-ironic | 07:17 | |
| *** yarker has joined #openstack-ironic | 07:18 | |
| *** marcoemorais1 has quit IRC | 07:19 | |
| *** pensu has quit IRC | 07:25 | |
| *** pensu has joined #openstack-ironic | 07:26 | |
| *** jistr has joined #openstack-ironic | 07:28 | |
| *** yuanying has quit IRC | 07:45 | |
| *** yuanying has joined #openstack-ironic | 07:46 | |
| *** ndipanov_gone is now known as ndipanov | 07:53 | |
| *** jcoufal has joined #openstack-ironic | 07:59 | |
| *** yuanying has quit IRC | 08:06 | |
| *** yuanying has joined #openstack-ironic | 08:06 | |
| *** foexle has joined #openstack-ironic | 08:08 | |
| *** bluex-pl has joined #openstack-ironic | 08:10 | |
| *** yuanying has quit IRC | 08:11 | |
| *** andreykurilin_ has joined #openstack-ironic | 08:23 | |
| *** AJaeger has joined #openstack-ironic | 08:23 | |
| AJaeger | devananda: could you abandon https://review.openstack.org/#/c/123342/ now, please? | 08:24 |
| *** lucasagomes has joined #openstack-ironic | 08:28 | |
| Haomeng | AJaeger: done:) | 08:29 |
| Haomeng | AJaeger: but if it is abandoned, we can not get more patch for this, right? | 08:30 |
| Haomeng | AJaeger: could we vote -2? | 08:31 |
| AJaeger | Haomeng: if it's abandoned, a new patch will be proposed whenever new files are there... | 08:33 |
| Haomeng | AJaeger: ok:) | 08:33 |
| *** MattMan has quit IRC | 08:33 | |
| AJaeger | Haomeng: I'll keep an eye on the job so that it does the right thing ;) | 08:33 |
| Haomeng | AJaeger: thank you:) | 08:33 |
| Haomeng | AJaeger: :) | 08:33 |
| AJaeger | you're welcome, Haomeng | 08:33 |
| Haomeng | AJaeger: :) | 08:33 |
| *** MattMan has joined #openstack-ironic | 08:34 | |
| *** derekh has joined #openstack-ironic | 08:35 | |
| *** lazy_prince has quit IRC | 08:37 | |
| *** rameshg87 has joined #openstack-ironic | 08:40 | |
| *** bluex-pl has quit IRC | 08:44 | |
| rameshg87 | lucasagomes, hi | 08:44 |
| *** bluex-pl has joined #openstack-ironic | 08:45 | |
| *** killer_prince has joined #openstack-ironic | 08:46 | |
| *** killer_prince is now known as lazy_prince | 08:46 | |
| AJaeger | Haomeng: I just found one bug in the infra scripts that I introduced, so if there's a patch again tomorrow, I'll -1 it until infra merges my fix... | 08:47 |
| *** dvorak has quit IRC | 08:47 | |
| Haomeng | AJaeger: dont worry, thanks for the info:) | 08:47 |
| *** dvorak has joined #openstack-ironic | 08:50 | |
| lucasagomes | rameshg87, hi there | 08:58 |
| rameshg87 | lucasagomes, had a question on documentation | 08:58 |
| rameshg87 | lucasagomes, where are we documenting how to build ramdisk for ironic deployment ? i am not able to find them in ironic source tree. | 08:59 |
| rameshg87 | lucasagomes, or are there plans for doing it ? | 08:59 |
| lucasagomes | rameshg87, oh, that's a good q... I think it's because devstack already build it for us | 08:59 |
| lucasagomes | we def should have a link to the diskimage-builder tool | 08:59 |
| lucasagomes | https://github.com/openstack/diskimage-builder | 09:00 |
| rameshg87 | lucasagomes, yeah. and how to build it too. i guess required for ipa also. | 09:00 |
| lucasagomes | rameshg87, +1!! | 09:00 |
| * lucasagomes rechecks the docs | 09:00 | |
| lucasagomes | yeah indeed there's none | 09:01 |
| rameshg87 | lucasagomes, yeah :) | 09:01 |
| rameshg87 | lucasagomes, i think it might fit in here, right ? https://github.com/openstack/ironic/blob/master/doc/source/deploy/install-guide.rst#setup-the-drivers-for-bare-metal-service | 09:02 |
| lucasagomes | yeah sounds like a good place indeed | 09:04 |
| *** lazy_prince has quit IRC | 09:06 | |
| *** lazy_prince has joined #openstack-ironic | 09:06 | |
| rameshg87 | lucasagomes, can we have a place to document all about ilo drivers ? a single page i mean | 09:13 |
| lucasagomes | rameshg87, I don't see why not, I mean documentation is always good | 09:14 |
| lucasagomes | now, would it make sense to have a wiki about it? | 09:14 |
| rameshg87 | lucasagomes, in https://wiki.openstack.org/wiki ? | 09:14 |
| lucasagomes | it's easier to update | 09:14 |
| lucasagomes | rameshg87, https://wiki.openstack.org/wiki/Ironic | 09:15 |
| *** lsmola has joined #openstack-ironic | 09:15 | |
| rameshg87 | lucasagomes, okay. minimum information on ironic source tree with pointers to wiki, right ? | 09:15 |
| lucasagomes | rameshg87, sure | 09:15 |
| rameshg87 | lucasagomes, thanks .. | 09:16 |
| lucasagomes | yw | 09:16 |
| *** igordcard has joined #openstack-ironic | 09:18 | |
| rameshg87 | lucasagomes, can i have a link in the main page https://wiki.openstack.org/wiki/Ironic pointing to a new page dedicated to ilo driver(s) ? | 09:23 |
| lucasagomes | rameshg87, yup, maybe we can start a section about drivers? | 09:24 |
| lucasagomes | so we can have iLO, drac etc... | 09:24 |
| * lucasagomes also thinks that the migration page from nova bm should be on that main page | 09:24 | |
| lucasagomes | will include | 09:24 |
| rameshg87 | lucasagomes, we have a page for drivers already https://wiki.openstack.org/wiki/Ironic/Drivers | 09:25 |
| lucasagomes | rameshg87, oh I see | 09:25 |
| rameshg87 | lucasagomes, yeah, but i don't know why it's documented by power/deploy interfaces rather than an 'actual' driver | 09:27 |
| rameshg87 | lucasagomes, may be i can start a section third party drivers here | 09:28 |
| lucasagomes | rameshg87, yup yeah feel free to edit it :) | 09:29 |
| lucasagomes | it's a wiki, so it's all good | 09:29 |
| rameshg87 | lucasagomes, :) | 09:30 |
| *** Haomeng|2 has joined #openstack-ironic | 09:51 | |
| *** Haomeng has quit IRC | 09:53 | |
| *** bluex-pl has quit IRC | 09:58 | |
| *** bluex-pl has joined #openstack-ironic | 09:58 | |
| *** viktors has joined #openstack-ironic | 09:58 | |
| *** nosnos has quit IRC | 10:09 | |
| *** nosnos has joined #openstack-ironic | 10:09 | |
| openstackgerrit | A change was merged to openstack/ironic: Remove unneeded context initialization in tests https://review.openstack.org/124073 | 10:16 |
| *** athomas has quit IRC | 10:21 | |
| *** athomas has joined #openstack-ironic | 10:30 | |
| *** romcheg has quit IRC | 10:38 | |
| *** loki184 has quit IRC | 10:46 | |
| *** romcheg has joined #openstack-ironic | 10:49 | |
| *** andreykurilin_ has quit IRC | 10:55 | |
| *** romcheg has quit IRC | 10:55 | |
| *** rameshg87 has quit IRC | 11:00 | |
| *** ramineni has quit IRC | 11:01 | |
| *** jistr has quit IRC | 11:10 | |
| *** romcheg has joined #openstack-ironic | 11:12 | |
| *** jistr has joined #openstack-ironic | 11:14 | |
| *** lucasagomes is now known as lucas-hungry | 11:19 | |
| *** mmitchell has quit IRC | 11:26 | |
| *** vinbs has quit IRC | 11:29 | |
| *** Poornima has quit IRC | 11:32 | |
| *** bluex-pl has quit IRC | 11:42 | |
| *** bluex-pl has joined #openstack-ironic | 11:43 | |
| *** bluex-pl has quit IRC | 11:44 | |
| *** bluex-pl has joined #openstack-ironic | 11:44 | |
| *** k4n0 has quit IRC | 11:46 | |
| *** viktors is now known as viktors|afk | 11:58 | |
| *** andreykurilin_ has joined #openstack-ironic | 11:59 | |
| *** nikunj2512 has quit IRC | 12:09 | |
| *** Haomeng has joined #openstack-ironic | 12:15 | |
| *** Haomeng|2 has quit IRC | 12:16 | |
| *** slagle has quit IRC | 12:20 | |
| *** slagle has joined #openstack-ironic | 12:20 | |
| *** nosnos has quit IRC | 12:24 | |
| *** nosnos has joined #openstack-ironic | 12:24 | |
| *** nosnos has quit IRC | 12:29 | |
| *** AJaeger has left #openstack-ironic | 12:37 | |
| *** lazy_prince has quit IRC | 12:41 | |
| *** lucas-hungry is now known as lucasagomes | 12:41 | |
| *** coolsvap is now known as coolsvap|afk | 12:44 | |
| *** dprince has joined #openstack-ironic | 12:56 | |
| *** rakesh_hs has quit IRC | 13:01 | |
| *** killer_prince has joined #openstack-ironic | 13:01 | |
| *** killer_prince is now known as lazy_prince | 13:01 | |
| openstackgerrit | Alberto Planas proposed a change to openstack/ironic: Adds openSUSE support for developer documentation https://review.openstack.org/124395 | 13:02 |
| *** dprince has quit IRC | 13:12 | |
| *** dhellmann_ is now known as dhellmann | 13:13 | |
| *** jjohnson2 has joined #openstack-ironic | 13:22 | |
| *** romcheg has quit IRC | 13:28 | |
| *** romcheg has joined #openstack-ironic | 13:31 | |
| *** jjohnson2 has quit IRC | 13:38 | |
| *** dprince has joined #openstack-ironic | 13:39 | |
| *** rushiagr is now known as rushiagr_away | 13:43 | |
| openstackgerrit | Vladyslav Drok proposed a change to openstack/ironic: Use DbTestCase as test base when context needed https://review.openstack.org/124400 | 13:45 |
| openstackgerrit | Alberto Planas proposed a change to openstack/ironic: Adds openSUSE support for developer documentation https://review.openstack.org/124395 | 13:52 |
| *** andreykurilin_ has quit IRC | 14:01 | |
| *** bluex-pl has quit IRC | 14:10 | |
| *** bluex-pl has joined #openstack-ironic | 14:10 | |
| NobodyCam | good morning Ironic... TGIF! | 14:14 |
| openstackgerrit | A change was merged to openstack/python-ironicclient: Remove unused command in tox.ini https://review.openstack.org/124303 | 14:14 |
| openstackgerrit | A change was merged to openstack/python-ironicclient: Add unit tests for "ironic node-create" shell cmd https://review.openstack.org/120312 | 14:14 |
| lucasagomes | NobodyCam, morning! | 14:15 |
| NobodyCam | morning lucasagomes | 14:17 |
| Shrews | NobodyCam: tgif indeed | 14:23 |
| *** ramineni has joined #openstack-ironic | 14:23 | |
| NobodyCam | morning Shrews :) !!! | 14:23 |
| *** dprince has quit IRC | 14:25 | |
| *** dprince has joined #openstack-ironic | 14:25 | |
| NobodyCam | hummm 15five? | 14:32 |
| lucasagomes | Shrews, morning :) | 14:58 |
| lucasagomes | 15five? | 14:58 |
| *** andreykurilin_ has joined #openstack-ironic | 14:58 | |
| *** pcrews has joined #openstack-ironic | 15:07 | |
| *** bluex-pl has quit IRC | 15:08 | |
| *** ndipanov has quit IRC | 15:10 | |
| *** rakesh_hs has joined #openstack-ironic | 15:11 | |
| *** chuckC has joined #openstack-ironic | 15:11 | |
| *** ChuckC_ has joined #openstack-ironic | 15:12 | |
| NobodyCam | slow morning | 15:18 |
| NobodyCam | its a new corp thing | 15:19 |
| Shrews | more a "monty" thing than a "corp" thing | 15:20 |
| NobodyCam | lol | 15:20 |
| NobodyCam | Shrews: ture that | 15:20 |
| Shrews | i mean, it's not internal | 15:20 |
| *** rakesh_hs has quit IRC | 15:22 | |
| *** anteaya has quit IRC | 15:23 | |
| openstackgerrit | David Shrewsbury proposed a change to openstack/ironic-specs: Add a section for RPC API impact https://review.openstack.org/103225 | 15:55 |
| Shrews | before we open kilo for new specs, we should merge that ^^^ | 15:56 |
| Shrews | devananda: since we aren't accepting any more specs for juno, can we go ahead and merge that? | 15:58 |
| NobodyCam | Shrews: will we need to adjust this for every cycle? (files = ['specs/template.rst'] + glob.glob('specs/kilo/*')) | 16:02 |
| Shrews | NobodyCam: yeah. i don't see a way around that | 16:02 |
| Shrews | once a spec is accepted, what's the point of testing it anymore? :) | 16:03 |
| NobodyCam | what if we get all then exclude J | 16:03 |
| devananda | morning, all | 16:03 |
| NobodyCam | morning devananda | 16:03 |
| Shrews | NobodyCam: then you'd have to exclude J *and* K when L comes about. Seems easier to just whitelist instead of blacklist, no? | 16:04 |
| devananda | some projects have created an "attic" of source | 16:04 |
| devananda | of sorts | 16:04 |
| devananda | for specs that were meregd but not completed | 16:04 |
| devananda | Shrews: ++ | 16:04 |
| Shrews | ah, that's a different beast i hadn't considered | 16:04 |
| devananda | we actually completed every spec we merged | 16:05 |
| devananda | which is pretty awesome | 16:05 |
| lucasagomes | devananda, morning | 16:05 |
| JayF | we even completed ~half dozen that aren't even approved yet ;) | 16:05 |
| NobodyCam | we have awesome Folk working in Ironic and Ironic releated projects | 16:05 |
| devananda | i poked at the hash ring things a little yesterday afternoon - https://review.openstack.org/#/c/120951/ | 16:06 |
| JayF | NobodyCam: just the right number of cooks in the kitchen, eh? | 16:06 |
| NobodyCam | :) | 16:06 |
| * devananda fixes merge conflict | 16:08 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Add HashRingManager to wrap hash ring singleton https://review.openstack.org/120951 | 16:10 |
| devananda | so, really, we shouldn't release without the hash ring thing fixed ... | 16:10 |
| devananda | i'm going to hack on that during this flight some more | 16:10 |
| devananda | anyone else been poking at it lately / made any progress? lucasagomes ? Shrews ? | 16:11 |
| lucasagomes | devananda, the rebalancing part of it? or take over? | 16:11 |
| devananda | lucasagomes: all of it. they're interdependent | 16:12 |
| lucasagomes | I haven't played much with the rebalancing part | 16:12 |
| devananda | fixing rebalancign w/o take over doesn't solve the "i cant reboot my node!" problem | 16:12 |
| lucasagomes | the take over I was investigating, but after we talked about the ipxe idea I'm trying to validate it instead | 16:12 |
| lucasagomes | yeah :/ | 16:12 |
| devananda | fixing take over w/o fixing rebalancing makes a LOT of take over happen | 16:12 |
| devananda | lucasagomes: so I really like the ipxe solution. really really. but I dont think we can jump to that now() | 16:13 |
| devananda | and releasing with a massively broken scale-out story is awkward | 16:13 |
| devananda | "awkward" is also really minimizing the problem | 16:13 |
| lucasagomes | devananda, yup no way to do it now() | 16:13 |
| lucasagomes | yeah, so I haven't poked at it late this week :( I did until tuesday | 16:14 |
| lucasagomes | one idea I had when I was looking into it | 16:14 |
| lucasagomes | some assumptions really, was to only mark the node (create a bound conductor->node) when the node is deployed | 16:15 |
| lucasagomes | and remove it when undeployed | 16:15 |
| lucasagomes | we don't need to know which conductor is bounded to which node when it's unprovisioned | 16:15 |
| lucasagomes | from there we need to create a delta of the hash_ring, every time a conductor joins or leave the cluster | 16:16 |
| lucasagomes | we need to have the previous hash_ring saved, so we can generate a new one, compare with it, see what drivers were affected | 16:16 |
| lucasagomes | from there find the nodes that uses that driver and are deployed | 16:16 |
| lucasagomes | check if the condutor bounded to that node have changed and if so call take_over() | 16:17 |
| lucasagomes | if not just continue interacting | 16:17 |
| *** marcoemorais has joined #openstack-ironic | 16:17 | |
| lucasagomes | but look, that's very costy | 16:17 |
| *** chuckC has quit IRC | 16:18 | |
| lucasagomes | devananda, makes sense ^? | 16:19 |
| devananda | lucasagomes: yea | 16:19 |
| lucasagomes | that sounds like something achievable for this cycle | 16:19 |
| lucasagomes | tho it's not a light operator | 16:19 |
| lucasagomes | operation* | 16:19 |
| devananda | what about this: | 16:19 |
| lucasagomes | and if we do also the periodic task to check for the pxe enviroments from time to time and update it, we minimize yet more the problem with take overs | 16:20 |
| devananda | - when node is provisioned, update db record to indicate which conductor did the work (it was mapped there during deploy) | 16:20 |
| devananda | - during periodic task, when conductor refreshes its rings: | 16:20 |
| devananda | -- check for nodes which are a) mapped to that conductor b) deployed c) have db tag != current conductor | 16:20 |
| devananda | - trigger take_over() for that set | 16:21 |
| devananda | no need to calculate delta of hash rings | 16:21 |
| devananda | and no need to walk the whole list of nodes | 16:21 |
| *** andreykurilin_ has quit IRC | 16:21 | |
| devananda | and no action explicitly on join/leave | 16:21 |
| devananda | so eg. a simple restart of the conductor won't trigger sudden migrations | 16:22 |
| lucasagomes | devananda, that's simpler indeed, but there's a gap (which might be fine) that the node can't be rebooted until the periodic task run | 16:22 |
| devananda | yep | 16:22 |
| devananda | i'm OK with that | 16:22 |
| devananda | eventually-consistent | 16:22 |
| lucasagomes | right | 16:22 |
| lucasagomes | yeah | 16:22 |
| devananda | if a reboot fails, the user should be able to retry the reboot | 16:22 |
| devananda | (though that's a separate problem) | 16:22 |
| lucasagomes | well, if u want I can put a patch up for that ^ tomorrow | 16:22 |
| devananda | small-window is better than no-solution | 16:22 |
| devananda | I'm gonna try to do it today | 16:22 |
| lucasagomes | I've been playing with that part of the code anyway, so I think I'm familiar | 16:22 |
| devananda | cool | 16:23 |
| lucasagomes | devananda, alright, if you don't finish I may take ur patch tomorrow | 16:23 |
| lucasagomes | and help with it | 16:23 |
| devananda | I'll post what ever I have tonight. feel free to clobber / build on it | 16:23 |
| lucasagomes | devananda, deal | 16:23 |
| *** marcoemorais has quit IRC | 16:23 | |
| devananda | https://review.openstack.org/120951 is the start, i think | 16:23 |
| devananda | that makes it easier to interact with the HashRign singleton | 16:23 |
| lucasagomes | yeah, adn there's one adding the condutor_id db field to the nodes too | 16:24 |
| devananda | yep | 16:24 |
| lucasagomes | u will need that as well | 16:24 |
| *** marcoemorais has joined #openstack-ironic | 16:24 | |
| *** marcoemorais has quit IRC | 16:24 | |
| *** marcoemorais has joined #openstack-ironic | 16:24 | |
| *** ChuckC has joined #openstack-ironic | 16:24 | |
| *** marcoemorais has quit IRC | 16:25 | |
| devananda | updating etherpad now | 16:25 |
| *** ChuckC_ has quit IRC | 16:26 | |
| *** marcoemorais has joined #openstack-ironic | 16:26 | |
| lucasagomes | cool, for those interested on that ^ https://etherpad.openstack.org/p/68IcuXSqzz | 16:26 |
| * lucasagomes brb 1 sec | 16:26 | |
| devananda | ok - gotta board my flight | 16:30 |
| devananda | probably offline most of the day, but will be hacking on this | 16:30 |
| *** ChuckC has quit IRC | 16:30 | |
| NobodyCam | have a good flight back home devananda | 16:32 |
| *** anteaya has joined #openstack-ironic | 16:34 | |
| *** romcheg has quit IRC | 16:34 | |
| *** ramineni has quit IRC | 16:40 | |
| *** jistr is now known as jistr|afk | 16:45 | |
| *** marcoemorais has quit IRC | 16:58 | |
| lucasagomes | devananda, safe flight | 16:59 |
| *** marcoemorais has joined #openstack-ironic | 16:59 | |
| *** marcoemorais has quit IRC | 16:59 | |
| *** marcoemorais has joined #openstack-ironic | 17:00 | |
| *** romcheg has joined #openstack-ironic | 17:05 | |
| *** marcoemorais has quit IRC | 17:05 | |
| *** marcoemorais has joined #openstack-ironic | 17:06 | |
| lucasagomes | aight folks, have a good night! | 17:07 |
| NobodyCam | have a good weekend lucasagomes | 17:07 |
| lucasagomes | NobodyCam, oh, u too! | 17:07 |
| lucasagomes | enjoy it | 17:07 |
| NobodyCam | :) | 17:08 |
| *** lucasagomes is now known as lucas-dinner | 17:08 | |
| *** lazy_prince has quit IRC | 17:08 | |
| *** derekh has quit IRC | 17:14 | |
| *** athomas has quit IRC | 17:16 | |
| *** dhellmann is now known as dhellmann_ | 17:18 | |
| jroll | morning ironic :) | 17:20 |
| NobodyCam | goood morning and happy friday jroll | 17:21 |
| openstackgerrit | Adam Gandelman proposed a change to openstack/ironic: Testing sideways migration (Do not merge) https://review.openstack.org/124474 | 17:21 |
| *** MattMan has quit IRC | 17:23 | |
| *** ChuckC has joined #openstack-ironic | 17:24 | |
| NobodyCam | brb quick walkies | 17:24 |
| *** andreykurilin_ has joined #openstack-ironic | 17:26 | |
| *** chuckC_ has joined #openstack-ironic | 17:26 | |
| jroll | happy friday indeed | 17:27 |
| *** ChuckC has quit IRC | 17:29 | |
| *** chuckC_ is now known as chuckC | 17:29 | |
| *** ChuckC_ has joined #openstack-ironic | 17:30 | |
| *** marcoemorais has quit IRC | 17:30 | |
| *** marcoemorais has joined #openstack-ironic | 17:31 | |
| *** dhellmann_ is now known as dhellmann | 17:36 | |
| *** foexle has quit IRC | 17:39 | |
| *** pensu has quit IRC | 17:43 | |
| *** igordcard has quit IRC | 17:44 | |
| *** andreykurilin_ has quit IRC | 17:53 | |
| *** pensu has joined #openstack-ironic | 17:59 | |
| *** harlowja_away is now known as harlowja | 17:59 | |
| *** dprince has quit IRC | 18:01 | |
| *** killer_prince has joined #openstack-ironic | 18:09 | |
| *** killer_prince is now known as lazy_prince | 18:09 | |
| *** jcoufal has quit IRC | 18:13 | |
| *** dprince has joined #openstack-ironic | 18:13 | |
| *** jistr|afk is now known as jistr | 18:42 | |
| *** jistr has quit IRC | 18:42 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Fix hash ring stability https://review.openstack.org/118932 | 18:51 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Add "active" tracking to nodes and conductors https://review.openstack.org/124493 | 18:51 |
| devananda | still need to tie that into a periodic task | 18:52 |
| devananda | but I think that'll be it | 18:52 |
| devananda | if anyone wants to poke at ^, it needs an alembic migration added to the last patch | 18:53 |
| *** dhellmann is now known as dhellmann_ | 18:57 | |
| *** rushiagr_away is now known as rushiagr | 19:11 | |
| NobodyCam | oh nice: https://review.openstack.org/#/c/124072 landed | 19:18 |
| NobodyCam | lucas-dinner: want to remove your -2 from https://review.openstack.org/#/c/124061 | 19:21 |
| *** andreykurilin_ has joined #openstack-ironic | 19:24 | |
| *** boris-42 has joined #openstack-ironic | 19:27 | |
| *** rushiagr is now known as rushiagr_away | 19:29 | |
| *** andreykurilin_ has quit IRC | 19:53 | |
| *** andreykurilin_ has joined #openstack-ironic | 19:53 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic-python-agent: Adding support for decommissioning https://review.openstack.org/104379 | 19:53 |
| *** dhellmann_ is now known as dhellmann | 19:55 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic-python-agent: Add versioning to Agent decommission https://review.openstack.org/106859 | 20:07 |
| NobodyCam | yuriyz: just checking on https://review.openstack.org/#/c/103105 do you think you'll have some time for that one? | 20:25 |
| *** todd_dsm has joined #openstack-ironic | 20:26 | |
| *** dprince has quit IRC | 20:26 | |
| openstackgerrit | David Shrewsbury proposed a change to openstack/ironic: Add "active" tracking to nodes and conductors https://review.openstack.org/124493 | 20:28 |
| Shrews | devananda: ^^^^ added the alembic migration for you | 20:28 |
| NobodyCam | Shrews: nice :) | 20:30 |
| *** rwsu has quit IRC | 20:31 | |
| * Shrews had to reference his new ironic-dbsync documentation for that | 20:31 | |
| NobodyCam | sweet it works! | 20:33 |
| Shrews | NobodyCam: i am now going to "like" every one of your status items now that i know how much you enjoy that | 20:33 |
| NobodyCam | lol gah hehehehe | 20:33 |
| * Shrews notes that is also one of the reasons he left FB | 20:34 | |
| NobodyCam | oh man you can spend all day replyig to likes... | 20:35 |
| * NobodyCam pop out of a somke... brb | 20:37 | |
| NobodyCam | *pops even | 20:37 |
| * jroll imagines NobodyCam popping out of a cigarette box | 20:39 | |
| NobodyCam | lol | 20:43 |
| *** marcoemorais has quit IRC | 21:07 | |
| *** marcoemorais has joined #openstack-ironic | 21:07 | |
| *** todd_dsm has quit IRC | 21:11 | |
| pensu | Hi all, is devstack-ironic broken today or something? I have tried twice and it just gives me error... | 21:24 |
| *** marcoemorais has quit IRC | 21:25 | |
| NobodyCam | hey hey pensu not that I have heard .... what the error your getting | 21:28 |
| pensu | Howdy NobodyCam, How are you? Two different errors, first some issues my with mysql, then I tried to make it work again, now it says cinder api did not start.... | 21:30 |
| pensu | NobodyCam: 2014-09-26 21:19:39.929 | /opt/stack/devstack/lib/cinder:452:die | 21:30 |
| pensu | 2014-09-26 21:19:39.949 | [ERROR] /opt/stack/devstack/lib/cinder:452 c-api did not start | 21:30 |
| pensu | 2014-09-26 21:19:40.958 | Error on exit | 21:30 |
| pensu | 2014-09-26 21:19:41.026 | df: '/run/user/1000/gvfs': Permission denied | 21:30 |
| NobodyCam | humm thats nother error, looks like user permissions | 21:31 |
| pensu | NobodyCam: Yeah, not sure if something is changed. Was working fine till yesterday! | 21:32 |
| NobodyCam | is there any more detail on the cinder screen? | 21:36 |
| NobodyCam | also you may want ot give http://unix.stackexchange.com/questions/77453/why-cannot-find-read-run-user-1000-gvfs-even-though-it-is-running-as-root a quick read | 21:38 |
| *** romcheg has quit IRC | 21:39 | |
| pensu | no actually, nothing useful, it just tries to start c-api 2-3 times and prints out the permission denied error, I will have a look at that link....thanks...:) | 21:40 |
| *** todd_dsm has joined #openstack-ironic | 21:41 | |
| *** marcoemorais has joined #openstack-ironic | 21:44 | |
| NobodyCam | ofc course | 21:46 |
| *** marcoemorais has quit IRC | 21:46 | |
| NobodyCam | s/ofc/of/ | 21:46 |
| *** marcoemorais1 has joined #openstack-ironic | 21:46 | |
| *** todd_dsm has quit IRC | 21:52 | |
| *** romcheg has joined #openstack-ironic | 21:54 | |
| *** marcoemorais1 has quit IRC | 21:57 | |
| *** enterprisedc has quit IRC | 21:57 | |
| *** enterprisedc has joined #openstack-ironic | 21:59 | |
| *** marcoemorais has joined #openstack-ironic | 22:02 | |
| *** marcoemorais has quit IRC | 22:02 | |
| *** marcoemorais has joined #openstack-ironic | 22:04 | |
| *** todd_dsm has joined #openstack-ironic | 22:10 | |
| *** marcoemorais has quit IRC | 22:16 | |
| NobodyCam | brb | 22:18 |
| *** todd_dsm has quit IRC | 22:19 | |
| *** pensu has quit IRC | 22:26 | |
| *** dhellmann is now known as dhellmann_ | 22:42 | |
| *** romcheg has quit IRC | 22:43 | |
| *** marcoemorais has joined #openstack-ironic | 22:46 | |
| *** rwsu has joined #openstack-ironic | 22:47 | |
| *** pcrews has quit IRC | 22:54 | |
| *** romcheg has joined #openstack-ironic | 22:57 | |
| *** marcoemorais has quit IRC | 23:05 | |
| NobodyCam | ok for the 4 o'clock friday topic, owuld we like our unit test DAMP or DRY? | 23:13 |
| jroll | NobodyCam: I like them JFDI | 23:14 |
| NobodyCam | lol I just found myself making a thesee tests are all the same comment, and that got me thinking | 23:16 |
| jroll | haha | 23:16 |
| jroll | yeah, DRY them up if it doesn't make them confusing | 23:16 |
| NobodyCam | ++ the block I comment on could be cleaned up | 23:18 |
| NobodyCam | but do we really care? | 23:18 |
| jroll | I don't care enough to -1 a review for it, generally | 23:18 |
| jroll | what are you looking at? | 23:18 |
| *** marcoemorais has joined #openstack-ironic | 23:20 | |
| NobodyCam | | 23:26 |
| NobodyCam | jroll: it started here : https://review.openstack.org/#/c/124162 (with Jay pipes comment) which lead me to find my self writing https://review.openstack.org/#/c/124267/1/ironicclient/tests/v1/test_node_shell.py (second comment) | 23:28 |
| NobodyCam | then I get to really thinking about it I think I more agree with your JFDI way! | 23:29 |
| *** rwsu has quit IRC | 23:30 | |
| jroll | NobodyCam: yeah, I would probably clean that up as a committer | 23:34 |
| jroll | as a reviewer, I'd ask them to do it if they push another rev | 23:34 |
| NobodyCam | :) | 23:35 |
| NobodyCam | I actually think cleaned up would be easier to read too :-p | 23:35 |
| jroll | indeed | 23:35 |
| jroll | that one is asking for it, I think it depends on the tests | 23:36 |
| NobodyCam | yes! | 23:36 |
| *** pcrews has joined #openstack-ironic | 23:43 | |
| *** chuckC has quit IRC | 23:48 | |
| *** jeblair is now known as corvus | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!