| *** datajerk has joined #openstack-ironic | 00:06 | |
| *** jcooley has quit IRC | 00:08 | |
| *** jcooley has joined #openstack-ironic | 00:12 | |
| *** dkehnx has quit IRC | 00:14 | |
| *** dkehnx has joined #openstack-ironic | 00:14 | |
| *** zdiN0bot has quit IRC | 00:19 | |
| *** matsuhashi has joined #openstack-ironic | 00:20 | |
| *** zdiN0bot has joined #openstack-ironic | 00:20 | |
| *** jcooley has quit IRC | 00:22 | |
| *** dwalleck has joined #openstack-ironic | 00:24 | |
| *** dwalleck_ has joined #openstack-ironic | 00:27 | |
| *** newell_ has quit IRC | 00:29 | |
| *** dwalleck has quit IRC | 00:30 | |
| *** matsuhashi has quit IRC | 00:32 | |
| *** datajerk1 has joined #openstack-ironic | 00:32 | |
| *** zdiN0bot has quit IRC | 00:33 | |
| *** matsuhashi has joined #openstack-ironic | 00:33 | |
| *** datajerk has quit IRC | 00:34 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Adding a reference driver for the agent https://review.openstack.org/84795 | 00:56 |
|---|---|---|
| *** neillc is now known as neillc_taxi | 01:00 | |
| *** neillc_taxi has quit IRC | 01:02 | |
| *** rloo has quit IRC | 01:16 | |
| *** rloo has joined #openstack-ironic | 01:16 | |
| *** rloo has quit IRC | 01:18 | |
| *** rloo has joined #openstack-ironic | 01:18 | |
| *** rloo has joined #openstack-ironic | 01:19 | |
| *** nosnos has joined #openstack-ironic | 01:43 | |
| *** rloo has quit IRC | 01:52 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Factoring out PXE and TFTP functions https://review.openstack.org/90233 | 02:07 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Adding a reference driver for the agent https://review.openstack.org/84795 | 02:11 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Adding a reference driver for the agent https://review.openstack.org/84795 | 02:11 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Factoring out PXE and TFTP functions https://review.openstack.org/90233 | 02:38 |
| *** datajerk1 has quit IRC | 02:39 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Adding a reference driver for the agent https://review.openstack.org/84795 | 02:39 |
| *** jgrimm has joined #openstack-ironic | 02:41 | |
| *** datajerk has joined #openstack-ironic | 02:41 | |
| *** datajerk has quit IRC | 02:46 | |
| *** harlowja is now known as harlowja_away | 02:50 | |
| *** epim has joined #openstack-ironic | 02:51 | |
| *** hemna has quit IRC | 02:52 | |
| *** harlowja_away has quit IRC | 02:58 | |
| *** russell_h has quit IRC | 03:01 | |
| *** datajerk has joined #openstack-ironic | 03:10 | |
| *** epim has quit IRC | 03:13 | |
| *** datajerk has quit IRC | 03:18 | |
| *** rameshg87 has joined #openstack-ironic | 03:22 | |
| *** rameshg87 has quit IRC | 03:23 | |
| *** rameshg87 has joined #openstack-ironic | 03:24 | |
| *** matsuhashi has quit IRC | 03:33 | |
| *** coolsvap|afk is now known as coolsvap | 03:34 | |
| *** neillc has joined #openstack-ironic | 03:36 | |
| *** nosnos has quit IRC | 03:43 | |
| openstackgerrit | Ramakrishnan G proposed a change to openstack/ironic: Add IloDriver and its utils https://review.openstack.org/89500 | 03:53 |
| *** datajerk has joined #openstack-ironic | 04:07 | |
| *** datajerk has quit IRC | 04:11 | |
| *** datajerk has joined #openstack-ironic | 04:12 | |
| *** datajerk has quit IRC | 04:18 | |
| *** ams0 has quit IRC | 04:20 | |
| *** epim has joined #openstack-ironic | 04:22 | |
| *** epim has quit IRC | 04:25 | |
| *** ramineni has joined #openstack-ironic | 04:26 | |
| *** ams0 has joined #openstack-ironic | 04:28 | |
| *** epim has joined #openstack-ironic | 04:34 | |
| *** epim has quit IRC | 04:38 | |
| *** matsuhashi has joined #openstack-ironic | 04:39 | |
| *** mdenny has quit IRC | 04:40 | |
| *** lazy_prince has joined #openstack-ironic | 04:42 | |
| *** neillc is now known as neillc_physio | 04:44 | |
| *** epim has joined #openstack-ironic | 04:46 | |
| *** nosnos has joined #openstack-ironic | 04:51 | |
| *** Mikhail_D_ltp has joined #openstack-ironic | 05:03 | |
| *** sabah has joined #openstack-ironic | 05:08 | |
| *** datajerk has joined #openstack-ironic | 05:12 | |
| *** romcheg1 has joined #openstack-ironic | 05:14 | |
| openstackgerrit | Anusha proposed a change to openstack/ironic: Add IloPower module for IloDriver https://review.openstack.org/90294 | 05:16 |
| *** datajerk has quit IRC | 05:16 | |
| *** ams0 has quit IRC | 05:22 | |
| *** ams0 has joined #openstack-ironic | 05:23 | |
| *** Mikhail_D_ltp has quit IRC | 05:24 | |
| *** ams0 has quit IRC | 05:25 | |
| *** rameshg87 has quit IRC | 05:27 | |
| *** epim has quit IRC | 05:28 | |
| *** Mikhail_D_ltp has joined #openstack-ironic | 05:34 | |
| *** Mikhail_D_ltp has quit IRC | 05:48 | |
| *** eghobo has joined #openstack-ironic | 05:53 | |
| *** eghobo has quit IRC | 05:54 | |
| *** eghobo has joined #openstack-ironic | 05:54 | |
| *** pradipta_away is now known as pradipta | 06:02 | |
| *** datajerk has joined #openstack-ironic | 06:06 | |
| *** sseago has quit IRC | 06:06 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/88508 | 06:07 |
| *** datajerk has quit IRC | 06:11 | |
| *** datajerk has joined #openstack-ironic | 06:12 | |
| *** ams0 has joined #openstack-ironic | 06:15 | |
| *** datajerk has quit IRC | 06:16 | |
| *** sseago has joined #openstack-ironic | 06:25 | |
| *** dkehn_ has joined #openstack-ironic | 06:25 | |
| *** ams0 has quit IRC | 06:25 | |
| *** ams0 has joined #openstack-ironic | 06:26 | |
| *** ams0 has quit IRC | 06:27 | |
| *** dkehnx has quit IRC | 06:29 | |
| *** neillc_physio is now known as neillc | 06:37 | |
| *** epim has joined #openstack-ironic | 06:40 | |
| GheRivero | morning all | 06:48 |
| *** epim has quit IRC | 06:52 | |
| *** sabah has quit IRC | 06:57 | |
| *** rameshg87 has joined #openstack-ironic | 07:04 | |
| *** rameshg87 has left #openstack-ironic | 07:04 | |
| *** eghobo has quit IRC | 07:04 | |
| *** sabah has joined #openstack-ironic | 07:08 | |
| *** lsmola has joined #openstack-ironic | 07:09 | |
| *** ifarkas has joined #openstack-ironic | 07:10 | |
| *** datajerk has joined #openstack-ironic | 07:12 | |
| *** datajerk has quit IRC | 07:17 | |
| *** foexle has joined #openstack-ironic | 07:25 | |
| *** neillc is now known as neillc_away | 07:30 | |
| *** mrda is now known as mrda-away | 07:31 | |
| *** ndipanov_gone is now known as ndipanov | 07:49 | |
| *** jistr has joined #openstack-ironic | 07:57 | |
| *** athomas has joined #openstack-ironic | 07:58 | |
| *** athomas has quit IRC | 07:58 | |
| romcheg1 | Morning GheRivero! | 08:05 |
| *** datajerk has joined #openstack-ironic | 08:06 | |
| *** datajerk has quit IRC | 08:11 | |
| *** datajerk has joined #openstack-ironic | 08:12 | |
| *** datajerk has quit IRC | 08:16 | |
| *** killer_prince has quit IRC | 08:17 | |
| *** derekh_ has joined #openstack-ironic | 08:20 | |
| dtantsur | Morning Ironic, morning GheRivero, romcheg1 | 08:23 |
| *** lucasagomes has joined #openstack-ironic | 08:26 | |
| *** athomas has joined #openstack-ironic | 08:33 | |
| *** sabah has quit IRC | 08:37 | |
| *** datajerk has joined #openstack-ironic | 09:06 | |
| *** datajerk has quit IRC | 09:10 | |
| *** datajerk has joined #openstack-ironic | 09:11 | |
| *** lazy_prince2 has joined #openstack-ironic | 09:18 | |
| *** lazy_prince has quit IRC | 09:19 | |
| *** datajerk has quit IRC | 09:19 | |
| *** lazy_prince2 has quit IRC | 09:26 | |
| *** coolsvap is now known as coolsvap|afk | 09:30 | |
| *** romcheg1 has quit IRC | 09:31 | |
| *** sabah has joined #openstack-ironic | 09:40 | |
| *** matsuhashi has quit IRC | 09:41 | |
| *** matsuhas_ has joined #openstack-ironic | 09:42 | |
| *** dwalleck_ has quit IRC | 09:50 | |
| *** coolsvap|afk is now known as coolsvap | 09:58 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Implement the reboot command on the Ironic Driver https://review.openstack.org/92554 | 09:59 |
| agordeev | morning Ironic :) | 09:59 |
| *** lazy_prince has joined #openstack-ironic | 10:03 | |
| *** matsuhas_ has quit IRC | 10:09 | |
| *** matsuhashi has joined #openstack-ironic | 10:10 | |
| *** matsuhashi has quit IRC | 10:21 | |
| *** matsuhashi has joined #openstack-ironic | 10:22 | |
| Mikhail_D_wk | Morning all! :) | 10:24 |
| *** martyntaylor has joined #openstack-ironic | 10:25 | |
| *** lazy_prince2 has joined #openstack-ironic | 10:39 | |
| *** sabah has quit IRC | 10:39 | |
| *** lazy_prince has quit IRC | 10:41 | |
| *** athomas has quit IRC | 10:48 | |
| lucasagomes | morning agordeev Mikhail_D_wk | 10:52 |
| *** coolsvap is now known as coolsvap|afk | 10:54 | |
| *** athomas has joined #openstack-ironic | 10:57 | |
| *** ramineni has quit IRC | 11:02 | |
| *** ifarkas has quit IRC | 11:07 | |
| *** ifarkas has joined #openstack-ironic | 11:07 | |
| agordeev | morning lucasagomes Mikhail_D_wk | 11:21 |
| openstackgerrit | Imre Farkas proposed a change to openstack/ironic: Cleanup images after deployment https://review.openstack.org/92340 | 11:21 |
| openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Using system call for downloading files https://review.openstack.org/92560 | 11:33 |
| NobodyCam | good morning Ironic | 11:46 |
| lucasagomes | morning NobodyCam sorry I missed ur ping yesterday | 11:51 |
| NobodyCam | morning lucasagomes :) .. what the one that must have been at like 11 pm your time | 11:52 |
| NobodyCam | :-p | 11:52 |
| lucasagomes | NobodyCam, when u get some time; https://review.openstack.org/#/c/83726/ | 11:52 |
| lucasagomes | NobodyCam, yeah that one | 11:52 |
| NobodyCam | now if I could remember what it was for :-p | 11:52 |
| NobodyCam | ahh yes ... check-tempest-dsvm-virtual-ironic failing on https://review.openstack.org/#/c/83788/ | 11:54 |
| lucasagomes | check-tempest-dsvm-virtual-ironic ? | 11:55 |
| NobodyCam | ya it had errors I hadn't seen | 11:55 |
| lucasagomes | yeah looking at those logs I can't spot anything really obvious | 11:55 |
| lucasagomes | I can try a recheck | 11:55 |
| NobodyCam | I did | 11:56 |
| lucasagomes | no dice? | 11:57 |
| lucasagomes | :( | 11:57 |
| NobodyCam | same errors :( | 11:58 |
| lucasagomes | at one point it was working | 11:58 |
| lucasagomes | patch set 15 | 11:58 |
| NobodyCam | ones I hadn't seen before. but really seem unreleated to the patch | 11:58 |
| lucasagomes | yeah | 11:58 |
| lucasagomes | I will try to recheck just to see | 11:58 |
| lucasagomes | cause really patch set 15 to 16 I think was just a rebase | 11:59 |
| NobodyCam | I just wanted to ask if you had seen that error b4 | 11:59 |
| *** jgrimm has quit IRC | 11:59 | |
| lucasagomes | NobodyCam, right | 12:00 |
| lucasagomes | it seems that other patches are also failing with a similar error | 12:00 |
| lucasagomes | NobodyCam, https://review.openstack.org/#/c/92367/ | 12:00 |
| NobodyCam | wounder if infra changed something | 12:01 |
| lucasagomes | yeah, no idea really | 12:01 |
| NobodyCam | gah we have to land that patch so I so swaping mb fo gb in my mind | 12:02 |
| *** Alexei_987 has joined #openstack-ironic | 12:02 | |
| NobodyCam | s/so i so/so I can/ | 12:02 |
| NobodyCam | can/can stop | 12:03 |
| lucasagomes | heh | 12:11 |
| lucasagomes | yeah would be good to land those, they r hanging there for a while already | 12:11 |
| lucasagomes | the first one, to get rid of swap already has a +2 | 12:11 |
| NobodyCam | just a question on the test_deploy_utils.py on 83726 | 12:12 |
| Shrews | NobodyCam, lucasagomes: morning | 12:12 |
| lucasagomes | Shrews, morning | 12:12 |
| NobodyCam | you added test_deploy_without_swap wich is testing with EP=0 you see any reason to test have that same with EP >0? | 12:13 |
| NobodyCam | morning Shrews | 12:13 |
| NobodyCam | s/you see any reason to test have/you see any reason to have/ | 12:14 |
| lucasagomes | NobodyCam, hmmm... that's definitely one case that we could test | 12:14 |
| NobodyCam | :-p | 12:15 |
| lucasagomes | what I want to do with those tests is to create some helper functions as well | 12:15 |
| lucasagomes | cause they r way too big | 12:15 |
| lucasagomes | and share a lot of things in common | 12:15 |
| NobodyCam | ahh so a test clean up patch | 12:16 |
| lucasagomes | yeah | 12:16 |
| NobodyCam | I'm good with landing as is if we can add that same test with EP >0 in the clean up patch | 12:16 |
| lucasagomes | NobodyCam, right, I can do some work on it | 12:17 |
| * lucasagomes writes it down | 12:17 | |
| NobodyCam | lucasagomes: you are awesome!!!! | 12:17 |
| *** dkehn_ is now known as dkehnx | 12:18 | |
| lucasagomes | :D | 12:18 |
| lucasagomes | 1 sec I will eat something, brb | 12:18 |
| *** lucasagomes is now known as lucas-lunch | 12:19 | |
| *** pradipta is now known as pradipta_away | 12:20 | |
| *** foexle has quit IRC | 12:31 | |
| Shrews | it sometimes seems that ir-api isn't started before n-cpu under devstack | 12:33 |
| Shrews | odd | 12:33 |
| NobodyCam | isn't started or just slow about starting? | 12:36 |
| * NobodyCam has seen slow to start with devTEST | 12:36 | |
| Shrews | NobodyCam: hrm, may just be slow | 12:36 |
| *** jdob has joined #openstack-ironic | 12:37 | |
| *** jgrimm has joined #openstack-ironic | 12:50 | |
| *** jbjohnso has joined #openstack-ironic | 12:50 | |
| *** nosnos has quit IRC | 12:53 | |
| *** rloo has joined #openstack-ironic | 12:54 | |
| openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Using system call for downloading files https://review.openstack.org/92560 | 12:54 |
| *** Mikhail_D_wk has left #openstack-ironic | 13:00 | |
| *** lucas-lunch is now known as lucasagomes | 13:00 | |
| *** rloo has quit IRC | 13:02 | |
| -openstackstatus- NOTICE: Zuul is stuck due to earlier networking issues with Gerrit server, work in progress. | 13:02 | |
| *** ChanServ changes topic to "Zuul is stuck due to earlier networking issues with Gerrit server, work in progress." | 13:02 | |
| *** rloo has joined #openstack-ironic | 13:02 | |
| *** Mikhail_D_wk has joined #openstack-ironic | 13:03 | |
| *** coolsvap|afk is now known as coolsvap | 13:07 | |
| *** ChanServ changes topic to "OpenStack Bare Metal Provisioning | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic | Status: https://etherpad.openstack.org/p/IronicWhiteBoard" | 13:11 | |
| -openstackstatus- NOTICE: Zuul is processing changes now; some results were lost. Use "recheck bug 1317089" if needed. | 13:11 | |
| openstackgerrit | Dirk Mueller proposed a change to openstack/ironic: Remove explicit dependency on amqplib https://review.openstack.org/92153 | 13:12 |
| *** rloo has quit IRC | 13:17 | |
| *** rloo has joined #openstack-ironic | 13:17 | |
| *** datajerk has joined #openstack-ironic | 13:17 | |
| openstackgerrit | Imre Farkas proposed a change to openstack/ironic: Fix workaround for the "device is busy" problem https://review.openstack.org/90126 | 13:22 |
| *** rloo has quit IRC | 13:32 | |
| *** lazy_prince2 has quit IRC | 13:32 | |
| *** rloo has joined #openstack-ironic | 13:33 | |
| jroll | morning ironic :) | 13:34 |
| NobodyCam | good morning jroll | 13:34 |
| jroll | is Imre Farkas on irc? :/ | 13:34 |
| dtantsur | jroll, morning | 13:34 |
| dtantsur | jroll, ifarkas | 13:34 |
| NobodyCam | morning dtantsur :) | 13:34 |
| jroll | ah thanks dtantsur :) and good morning | 13:34 |
| dtantsur | oh, morning NobodyCam :) | 13:34 |
| NobodyCam | :) | 13:34 |
| ifarkas | morning jroll and NobodyCam | 13:34 |
| NobodyCam | morning ifarkas | 13:34 |
| *** matsuhashi has quit IRC | 13:35 | |
| jroll | morning ifarkas :) I still think that interval is too high in https://review.openstack.org/90126 | 13:35 |
| jroll | originally we waited 3 seconds | 13:35 |
| jroll | a major point of this patch is to wait less if the device is busy, no? | 13:35 |
| ifarkas | jroll, I think the point of the patch was to create a less error prone solution as 3 secs might not be enough in every cases | 13:36 |
| jroll | well, that too. but why not speed it up while we are at it? :) | 13:36 |
| ifarkas | let me check the launchpad ticket | 13:36 |
| jroll | lucasagomes: wdyt ^ | 13:37 |
| ifarkas | jroll, I see your point but I think 6 secs in total might be to low. As a solution for both cases we might use 1 secs for interval and 20 for the number for retries | 13:38 |
| lucasagomes | jroll, hey, yeah I agree that 30 seconds is too much | 13:38 |
| jroll | sure, I'm ok with that | 13:38 |
| jroll | lucasagomes: it's now 4 | 13:38 |
| lucasagomes | 30 * 6 actually, so 180 | 13:38 |
| jroll | 4 * 5 now | 13:39 |
| lucasagomes | right, 4 sounds resonable no? | 13:39 |
| jroll | well | 13:39 |
| lucasagomes | I mean, 20 seconds in total | 13:39 |
| jroll | 20 total sounds fine | 13:39 |
| jroll | but I think 1 * 20 might be better | 13:39 |
| jroll | it will be faster at least, and should be a rare case so probably not much extra load | 13:40 |
| ifarkas | jroll, yeah, that works for me | 13:41 |
| jroll | lucasagomes: it was hardcoded to 'wait 3 seconds' before - did you ever see this code exercised? was 3 seconds generally enough? | 13:41 |
| *** krtaylor_ has joined #openstack-ironic | 13:41 | |
| jroll | ifarkas: cool, I'd kind of like lucas to agree as I'm not core :) | 13:41 |
| lucasagomes | right, jroll no I never hit that problem afaict | 13:41 |
| *** krtaylor_ has quit IRC | 13:42 | |
| jroll | ok | 13:42 |
| lucasagomes | jroll, sounds reasonable to me yeah | 13:42 |
| jroll | awesome. thanks :) | 13:42 |
| *** krtaylor has quit IRC | 13:42 | |
| ifarkas | ifarkas, cool, that's an ack by everyone ;-) | 13:42 |
| ifarkas | jroll, will update the patch in a minute | 13:42 |
| jroll | for some definition of everyone :P | 13:42 |
| ifarkas | hehe, right | 13:43 |
| openstackgerrit | Imre Farkas proposed a change to openstack/ironic: Fix workaround for the "device is busy" problem https://review.openstack.org/90126 | 13:44 |
| ifarkas | jroll, ^ ;-) | 13:45 |
| jroll | thanks :) | 13:46 |
| jroll | +1'd | 13:46 |
| ifarkas | jroll, cool, thank you | 13:47 |
| openstackgerrit | A change was merged to openstack/ironic: Stop creating a swap partition when none was specified https://review.openstack.org/83726 | 13:49 |
| NobodyCam | lucasagomes: ^^^ \o/ | 13:49 |
| lucasagomes | NobodyCam, w00ts! | 13:49 |
| lucasagomes | NobodyCam, thanks | 13:49 |
| *** krtaylor has joined #openstack-ironic | 13:52 | |
| *** foexle has joined #openstack-ironic | 13:52 | |
| rloo | NobodyCam, or any non-lucasagomes core: easy one: https://review.openstack.org/#/c/90675/. | 14:04 |
| rloo | lucasagomes: I hope this doesn't need to be rebased: https://review.openstack.org/#/c/90675/ | 14:04 |
| NobodyCam | lucasagomes: just a question on test_deploy_utils on 90675 | 14:09 |
| NobodyCam | lines 486/487 you change the size from 1025 to 513? the next patch mb -> gb will round to 1024? | 14:12 |
| NobodyCam | oh wait is ithat start and end positions | 14:12 |
| NobodyCam | never mind | 14:12 |
| NobodyCam | :-p | 14:12 |
| *** mdenny has joined #openstack-ironic | 14:13 | |
| *** shakamunyi has joined #openstack-ironic | 14:16 | |
| *** athomas has quit IRC | 14:21 | |
| *** athomas has joined #openstack-ironic | 14:22 | |
| lucasagomes | rloo, morning | 14:22 |
| rloo | hi lucasagomes! | 14:22 |
| lucasagomes | NobodyCam, sorry I missed the ping (I'm now in a call) | 14:22 |
| NobodyCam | oh I missed that too... Morning rloo | 14:22 |
| NobodyCam | :( | 14:22 |
| NobodyCam | all good lucasagomes .... | 14:23 |
| NobodyCam | I miss read the change | 14:23 |
| rloo | morning NobodyCam! You're up awfully early. What timezone are you in now? | 14:23 |
| *** linggao has joined #openstack-ironic | 14:23 | |
| NobodyCam | early. its 10:23 here in Atlanta | 14:24 |
| NobodyCam | :-p | 14:24 |
| NobodyCam | heheheh | 14:24 |
| rloo | nice; my time zone :-) | 14:24 |
| NobodyCam | hehehehe :) | 14:24 |
| dtantsur | oh, morning rloo | 14:48 |
| rloo | hi dtantsur! | 14:49 |
| openstackgerrit | A change was merged to openstack/ironic: Place root partition last so that it can always be expanded https://review.openstack.org/90675 | 14:53 |
| NobodyCam | lucasagomes: ^^^ today is the day! | 14:54 |
| lucasagomes | wow!! hah yay cleaning up my open patches :D | 14:56 |
| openstackgerrit | Dmitry Tantsur proposed a change to openstack/ironic: Simplify calls to ImageCache in PXE module https://review.openstack.org/92367 | 14:56 |
| openstackgerrit | Dmitry Tantsur proposed a change to openstack/ironic: Add periodic task to check whether PXE image cache is up-to-date https://review.openstack.org/92115 | 14:56 |
| lucasagomes | NobodyCam, devananda have you guys tried it out? http://www8.hp.com/us/en/cloud/hphelion-openstack-community.html | 15:01 |
| lucasagomes | "...and experience a deployment based on TripleO" | 15:01 |
| NobodyCam | I have not.. but have see the internal info on it | 15:02 |
| lucasagomes | sounds pretty cool | 15:03 |
| NobodyCam | :) | 15:03 |
| *** datajerk has quit IRC | 15:04 | |
| *** shakamunyi has quit IRC | 15:20 | |
| *** shakamunyi has joined #openstack-ironic | 15:23 | |
| rloo | lucasagomes: https://review.openstack.org/#/c/83788/. This means that it is not possible to specify a swap partition to be < 1 GB? | 15:30 |
| lucasagomes | rloo, pretty much... the minimum would be 1GB | 15:32 |
| lucasagomes | rloo, me devananda lifeless talked about it, my first idea was to have everything to be MB | 15:32 |
| rloo | lucasagomes: is that clear to everyone? the bug and commit don't say that. | 15:32 |
| lucasagomes | but then we agreed that having everything as GB would be better | 15:32 |
| rloo | lucasagomes: I thought you'd allow fractions, eg 0.5 | 15:32 |
| lucasagomes | rloo, oh, I can clarify that | 15:32 |
| rloo | so why not allow pxe_swap_gb = 0.2 ? | 15:33 |
| lucasagomes | hmm cause then it would be better to have everything in MB instead of GB? | 15:34 |
| lucasagomes | I mean, I still think that MB is more flexible | 15:34 |
| lucasagomes | I gotta remember the arguments against it | 15:34 |
| rloo | lucasagomes: I think that's a different issue. The units, vs the values for those units. | 15:34 |
| lucasagomes | rloo, right, so right now we convert that value to int | 15:34 |
| rloo | lucasagomes: I'm not comfortable +2'ing it if I'm not sure others realize it means can't specify a swap partition < 1 GB. | 15:35 |
| rloo | lucasagomes: although whoever reviews it should realize it doesn't allow for < 1 GB, but still... | 15:35 |
| lucasagomes | rloo, yeah definetely the commit message is not clear about that | 15:36 |
| lucasagomes | rloo, the problem of having fractions is that | 15:36 |
| rloo | lucasagomes: commit and bug itself. | 15:37 |
| lucasagomes | the amout of swap is part of the nova flavor | 15:37 |
| rloo | lucasagomes: what about baremetal. is/do we care that it isn't backwards compatible then? | 15:37 |
| rloo | lucasagomes: the nova flavor has swap in MB, right? | 15:37 |
| lucasagomes | rloo, right, in nova it's still MB | 15:38 |
| lucasagomes | so the ironic driver will translate that for us | 15:38 |
| lucasagomes | and round it up when needed | 15:38 |
| rloo | lucasagomes: so if a user or whoever specifies the nova flavor swap (in MB) as 100, and it gets translated to a swap of 1 GB, is that what the user expects? | 15:38 |
| lucasagomes | rloo, that's what's going to happen yeah... doesn't seems very user friendly indeed | 15:39 |
| lucasagomes | :/ | 15:39 |
| * lucasagomes thinks about the fractions | 15:39 | |
| NobodyCam | brb | 15:46 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Return the HTTP Location for newly created resources https://review.openstack.org/92615 | 15:47 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Return the HTTP Location for accepted requestes https://review.openstack.org/92616 | 15:47 |
| *** eghobo has joined #openstack-ironic | 15:48 | |
| Shrews | I'm seriously contemplating wiping my laptop and installing Xubuntu 14.04 over my existing Ubuntu 13.10 install. I like the iface, my h/w is better supported, and I'm getting MUCH better battery life. Somebody needs to convince me to NOT do this... and soon. | 15:49 |
| *** datajerk has joined #openstack-ironic | 15:52 | |
| lucasagomes | Shrews, :) maybe trying a diff distro? | 15:52 |
| Shrews | lucasagomes: gee, i wonder which one you'd suggest ;) | 15:53 |
| lucasagomes | I mean not ubuntu based | 15:53 |
| lucasagomes | Shrews, fedora? | 15:53 |
| Shrews | lucasagomes: what kernel is in the latest fedora? | 15:54 |
| lucasagomes | (not because I work for RH really, I use fedora before that) | 15:54 |
| lucasagomes | 3.11.10-301 | 15:55 |
| lucasagomes | oh actually not | 15:55 |
| lucasagomes | that was my vm which I didn't update for a while | 15:55 |
| lucasagomes | 3.14.2 | 15:56 |
| lucasagomes | on my laptop | 15:56 |
| Shrews | hrm, 3.13.0 in ubuntu | 15:56 |
| Shrews | might be worth considering | 15:56 |
| lucasagomes | :) | 15:57 |
| Shrews | but i need reasons NOT to update, not alternate paths :-P | 15:57 |
| *** lsmola has quit IRC | 15:58 | |
| lucasagomes | heh yeah... well u won't hear good things about ubuntu, not from me at least | 16:00 |
| * lucasagomes would use debian instead | 16:01 | |
| *** hemna_ is now known as hemna | 16:03 | |
| *** dwalleck has joined #openstack-ironic | 16:03 | |
| *** dwalleck_ has joined #openstack-ironic | 16:04 | |
| *** ifarkas has quit IRC | 16:07 | |
| *** dwalleck has quit IRC | 16:09 | |
| jroll | Shrews: don't update because rabbit holes :) | 16:11 |
| *** dwalleck_ has quit IRC | 16:13 | |
| Shrews | wascally wabbits | 16:14 |
| JayF | Shrews: #1 rule: never upgrade your OS before going to a conference | 16:14 |
| NobodyCam | brb | 16:14 |
| JayF | Shrews: you'll find out you need some big package you forgot to install, and will have to download it over assumedly horrible conference wifi | 16:14 |
| jroll | lol | 16:14 |
| jroll | this is true | 16:14 |
| lucasagomes | lol JayF +1 | 16:16 |
| Shrews | very true, but in this case, it's _because_ of the conf that i want to update. currently get horrible battery life | 16:16 |
| Shrews | eh, i'll sleep on it | 16:17 |
| openstackgerrit | Dmitry Tantsur proposed a change to openstack/ironic: Consider free disk space before downloading images into cache https://review.openstack.org/92625 | 16:26 |
| JayF | Shrews: maybe install something less battery-killing onto a USB stick or SD card? That way you can have a minimal awesome battery live environment for irc and browsing, but can boot into your configured os if you need to do 'real work' or have powah | 16:28 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic-python-agent: Use LLDP to get switch port mapping https://review.openstack.org/92627 | 16:29 |
| *** dwalleck_ has joined #openstack-ironic | 16:29 | |
| *** jistr has quit IRC | 16:31 | |
| dtantsur | Shrews, re Ubuntu, I prefer to wait for LTS.1 (but that's probably because I use it in the mission-critical environment, i.e. on my wife's laptop) | 16:32 |
| Shrews | lol | 16:33 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic-python-agent: Use LLDP to get switch port mapping https://review.openstack.org/92627 | 16:35 |
| *** dwalleck__ has joined #openstack-ironic | 16:42 | |
| *** coolsvap is now known as coolsvap|afk | 16:46 | |
| *** dwalleck_ has quit IRC | 16:46 | |
| rloo | lucasagomes: are you looking into the RCP_API_VERSION in conductor/manager.py being wrong, or should I? | 16:52 |
| *** derekh_ has quit IRC | 16:52 | |
| lucasagomes | rloo, I just pointed it out, ur patch is actually going to fix it | 16:52 |
| lucasagomes | rloo, you think we need a patch fixing that before? if so can you put one up? | 16:53 |
| rloo | lucasagomes: i think it should be fixed outside my patch. because my patch could take many months before it gets merged at the rate it is going ;) | 16:53 |
| lucasagomes | rloo, heh yeah | 16:53 |
| rloo | lucasagomes: I'll put one up in a few minutes. | 16:53 |
| lucasagomes | I hope it doesn't take months to get merged tho | 16:53 |
| lucasagomes | rloo, cool, thank you | 16:53 |
| rloo | lucasagomes: thx for noticing! | 16:54 |
| lucasagomes | rloo, np :) | 16:54 |
| NobodyCam | lucasagomes: rloo just a version update | 16:54 |
| lucasagomes | NobodyCam, currently in trunk the RPC_API_VERSION from the manager.py and rpcapi.py doesn't match | 16:55 |
| lucasagomes | rloo, is going to put a patch up to fix that | 16:55 |
| NobodyCam | let land it :) | 16:55 |
| lucasagomes | NobodyCam, +1 | 16:55 |
| NobodyCam | s/let/lets/ | 16:55 |
| NobodyCam | :) | 16:55 |
| * NobodyCam drums fingers on table and stares at the screen waiting for rloo's patch .... (j/k) | 16:56 | |
| * rloo ignores NobodyCam :D | 16:57 | |
| *** harlowja has joined #openstack-ironic | 16:58 | |
| *** epim has joined #openstack-ironic | 16:58 | |
| openstackgerrit | Ruby Loo proposed a change to openstack/ironic: RPC_API_VERSION out of sync https://review.openstack.org/92636 | 17:00 |
| rloo | NobodyCam, lucasagomes: writing the commit took longer than the fix. Hope it has enough info. https://review.openstack.org/92636 | 17:01 |
| *** epim has quit IRC | 17:03 | |
| *** martyntaylor has left #openstack-ironic | 17:03 | |
| *** datajerk has quit IRC | 17:04 | |
| *** athomas has quit IRC | 17:04 | |
| *** Alexei_987 has quit IRC | 17:06 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Remove 'fake' and 'ssh' drivers from default enabled list https://review.openstack.org/88711 | 17:09 |
| NobodyCam | remove ssh NO!!! | 17:10 |
| NobodyCam | why | 17:10 |
| jroll | defaults should be sane defaults for production | 17:11 |
| NobodyCam | rloo: lgtm just need to wait on Mr. J | 17:11 |
| NobodyCam | jroll: :-p | 17:12 |
| lucasagomes | rloo, NobodyCam do you think that we need to open a bug about that problem and link on the commit message? | 17:12 |
| lucasagomes | or is it overkill? | 17:12 |
| NobodyCam | lucasagomes: might be a good idea just to track that we got out of sync | 17:12 |
| lucasagomes | ack I will open it quickly so rloo can link that | 17:13 |
| NobodyCam | but ... could also easly be overkill | 17:13 |
| NobodyCam | :) lucasagomes awesome | 17:13 |
| *** newell_ has joined #openstack-ironic | 17:13 | |
| *** zdiN0bot has joined #openstack-ironic | 17:14 | |
| *** russell_h has joined #openstack-ironic | 17:14 | |
| *** russell_h has quit IRC | 17:15 | |
| *** russell_h has joined #openstack-ironic | 17:15 | |
| NobodyCam | does anyone know if a qcow2 needs to be converted to raw before converting to tarball.. ie as is done here: https://github.com/openstack/diskimage-builder/blob/master/elements/rhel/root.d/10-rhel-cloud-image#L32 | 17:15 |
| NobodyCam | or could I just mount with qemu-nbd? | 17:15 |
| JayF | So what exactly are you trying to do? | 17:16 |
| JayF | I had to futz around quite a bit with qcow-convert for ipa | 17:16 |
| NobodyCam | convert a suse qcow2 to tar | 17:16 |
| JayF | to /tar/. | 17:16 |
| JayF | hmm | 17:16 |
| JayF | I know you could just mount it wtih qemu-nbd and tar up the filesystem | 17:17 |
| JayF | or similarly covert to raw, mount as loop, tar it up | 17:17 |
| JayF | but it wouldn't surprise me if qcow-convert would just convert it to tar | 17:17 |
| JayF | er, qemu | 17:17 |
| lucasagomes | rloo, can you tag the bug https://bugs.launchpad.net/ironic/+bug/1317202 with that patch? | 17:17 |
| lucasagomes | rloo, you can edit the commit message on gerrit directly | 17:17 |
| NobodyCam | ahhh paste.openstack.org is down? | 17:18 |
| JayF | I mean qemu-img convert aka https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/shell/write_image.sh#L48 | 17:18 |
| devananda | lifeless: "concept of verifying that a node works" -- yes, but it's only partial atm. | 17:19 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Drivers determine acceptable power states https://review.openstack.org/86744 | 17:19 |
| devananda | lifeless: we have the "validate" method, which tickles all the node's driver's interfaces | 17:19 |
| devananda | lifeless: however, afaik, only the power interface actually tickles the node -- the deploy interface, for example, just checks the node.driver_info to see if all the parameterls *look* good | 17:20 |
| openstackgerrit | Ruby Loo proposed a change to openstack/ironic: RPC_API_VERSION out of sync https://review.openstack.org/92636 | 17:20 |
| devananda | lifeless: we've been talking about moving taht to a periodic task, applied to unprovisioned nodeds, toand making GET /node/state totally async as a result -- so, then, i think a test deploy would be reasonable | 17:21 |
| JayF | NobodyCam: afaict there's no way to get qemu-img convert to give you a tarball directly :( so I'd just mount it with qemu-nbd and tar it up | 17:21 |
| JayF | at least in the version of qemu-img that ships with precise | 17:21 |
| devananda | also - g'morning, all | 17:22 |
| lucasagomes | devananda, morning! | 17:22 |
| NobodyCam | morning deva | 17:22 |
| rloo | lucasagomes: thx. I wondered about opening a bug. I've updated the commit. | 17:22 |
| lucasagomes | rloo, :) I saw, +2'd already | 17:22 |
| lucasagomes | I won't have time to wait for jenkins | 17:22 |
| lucasagomes | but we should merge that after it | 17:23 |
| devananda | lucasagomes: the field which stores root/swap/eph size in nova is an INT. for what ever taht's worth | 17:23 |
| * devananda is reading a few days of scrollback | 17:23 | |
| rloo | morning devananda. or er evening. | 17:23 |
| lucasagomes | devananda, yeah :( so no float | 17:23 |
| devananda | Shrews: lemme know how xubuntu turns out - i've been thinking of the same thing, mostly for the reports i hear of better battery life | 17:23 |
| devananda | ugh - when did RPC_API get out of syunc? | 17:24 |
| devananda | yea, it's evening here | 17:25 |
| devananda | i walked to potzdamerplatz to see the new architecture BUT there's nothing i can eat here | 17:25 |
| rloo | wrt RPC_API getting out of sync. Maybe we should add a test. | 17:25 |
| devananda | rloo: ++. cause this is the second time we've broken it | 17:25 |
| devananda | reviewer fail -- we need a machine to save us | 17:26 |
| rloo | nothing to eat? sauerkraut? | 17:26 |
| devananda | rloo: veg + GF. in germany, this is hard to find. | 17:26 |
| devananda | and sauerkraut != breakfast | 17:26 |
| rloo | yeah, the version thing should be a hack or something. i don't see why we should all remember to make sure it is in sync. | 17:26 |
| devananda | lucasagomes: i jsut replied on https://review.openstack.org/#/c/90174/ -- sorry for the long delay | 17:27 |
| devananda | rloo: it should be easy to add a unit test | 17:27 |
| lucasagomes | devananda, will take a look | 17:27 |
| rloo | devananda: from what i remember, veggies could be found, but they were almost always marinated or pickled or something. hard to find a fresh salad. | 17:27 |
| devananda | just import those 2 modules and assertEqual or something | 17:27 |
| lucasagomes | oh that's a good idea ^ | 17:27 |
| lucasagomes | rloo, mind doing that? | 17:27 |
| devananda | rloo: i found a place with decent curry -- but the gewurtzmittel all has gluten | 17:27 |
| rloo | devananda: yeah, i'll open a bug and write that test. beats reviewing, ha ha. | 17:28 |
| devananda | :P | 17:28 |
| devananda | thanks | 17:28 |
| NobodyCam | JayF: this is what I am doing: http://paste.openstack.org/show/f1OgZRg1yOiIIVzm7pZb/ | 17:28 |
| rloo | yw devananda. thx for the tip on what to write :-) | 17:28 |
| lucasagomes | devananda, ah I see... hmm makes sense if u ran the tests with --parallel | 17:29 |
| NobodyCam | brb | 17:29 |
| devananda | lucasagomes: which they are, in teh gate | 17:29 |
| lucasagomes | devananda, yeah | 17:29 |
| lucasagomes | devananda, ok I'm convinced will change my vote, thanks for pointing that out | 17:29 |
| devananda | that's why i did the sys.modules thing at import time | 17:30 |
| lucasagomes | I dunno any other way to mock that without tweaking sys.modules | 17:30 |
| devananda | yw! thanks for thinking of alternates :) | 17:30 |
| devananda | yea | 17:30 |
| devananda | i looked but didn't find anything else -- would love another way, tho | 17:30 |
| rloo | devananda, lucasagomes. wrt the seamicro mock, i was fine with it but i wondered whether that should be in a separate file, or make it more obvious that it will affect all unit tests. | 17:31 |
| *** jdob has quit IRC | 17:31 | |
| *** jdob_ has joined #openstack-ironic | 17:31 | |
| devananda | rloo: it's mocking seamicro library when it isn't present on the system | 17:32 |
| devananda | rloo: and not affecting other unit tests per-se --= unless they use the seamicroclient library,w hich, well, theys houldn't! | 17:32 |
| devananda | rloo: but if at some point, we have multiple files testing seamicro, then yea, we'd want to factor that out | 17:33 |
| lucasagomes | I'm grand in having the mock on the test_seamicro.py file (as-is) | 17:33 |
| rloo | devananda: but for that API for driverinfo properties, i have a test that uses pxe_seamicro (or something like that), and it uses that mocked library too. well it would if i enabled it. | 17:33 |
| devananda | ooh | 17:33 |
| *** krtaylor has quit IRC | 17:33 | |
| devananda | rloo: good point | 17:34 |
| devananda | also, if anyone is curious, here are my slides for my talk on friday -- http://bit.ly/os-ironic-linuxtag-2014 | 17:34 |
| devananda | I've WIP'd it | 17:35 |
| * NobodyCam clicks | 17:35 | |
| devananda | rloo: if you feel like doing some recactoring, you're welcome to clobber that patch. otherwise i'll get to it when i have time | 17:36 |
| devananda | *refactoring | 17:36 |
| devananda | speaking of time - it's time I find food i can eat ... bbl | 17:36 |
| rloo | devananda: ok. | 17:36 |
| lucasagomes | rloo, ur going to add that RPC_API_VERSION test in the #92636? | 17:38 |
| lucasagomes | I think that the test would be important | 17:39 |
| rloo | lucasagomes: i was going to open another bug for the test, but yeah, I could add it to the same patch. | 17:39 |
| lucasagomes | rloo, I see, whatever is easier for u | 17:39 |
| rloo | lucasagomes: just finishing lunch. will get to it in a few... | 17:39 |
| rloo | lucasagomes: easier to add to same bug/review. one less bug to open then :-) | 17:40 |
| lucasagomes | rloo, ack, just saying because if you would use the same review we need might want to update the bug report to say that a test should be added to avoid this problem from appear again | 17:40 |
| lucasagomes | ack I will add a note to the bug | 17:40 |
| lucasagomes | rloo, thank you, sorry for interrupting ur lunch | 17:41 |
| lucasagomes | and I'm also going to find some food :D | 17:45 |
| lucasagomes | have a good night everyone! | 17:45 |
| *** lucasagomes is now known as lucas-dinner | 17:45 | |
| *** zdiN0bot has quit IRC | 17:46 | |
| *** zdiN0bot has joined #openstack-ironic | 17:50 | |
| *** romcheg1 has joined #openstack-ironic | 17:53 | |
| *** matty_dubs|gone is now known as matty_dubs | 17:59 | |
| jroll | devananda: good stuff :) | 18:03 |
| *** jdob has joined #openstack-ironic | 18:04 | |
| *** jdob_ has quit IRC | 18:06 | |
| *** Mikhail_D_ltp has joined #openstack-ironic | 18:20 | |
| *** krtaylor has joined #openstack-ironic | 18:24 | |
| *** ndipanov has quit IRC | 18:24 | |
| openstackgerrit | Ruby Loo proposed a change to openstack/ironic: RPC_API_VERSION out of sync https://review.openstack.org/92636 | 18:41 |
| Shrews | rloo: thanks for cleaning up my mess :) | 18:42 |
| rloo | Shrews: you and several other people, missed it. I even missed it when I updated my patch due to your change. No worries; gave me something to do today :-) | 18:44 |
| rloo | Shrews: anyway, that's why we have computers. To do the boring stuff. | 18:44 |
| Shrews | indeed | 18:45 |
| jroll | devananda: I see 6 ironic sessions scheduled now? is that right? | 18:47 |
| jroll | devananda: I thought we only had 4 slots | 18:47 |
| jroll | devananda: or maybe because a few are consolidated but all marked as scheduled? | 18:50 |
| JayF | NobodyCam: +1 that's probably what I would've done as well | 18:51 |
| NobodyCam | :) | 18:52 |
| NobodyCam | brb | 18:53 |
| *** rloo has quit IRC | 18:57 | |
| *** rloo has joined #openstack-ironic | 18:58 | |
| *** zdiN0bot has quit IRC | 19:30 | |
| *** dwalleck__ has quit IRC | 19:32 | |
| *** zdiN0bot has joined #openstack-ironic | 19:35 | |
| *** Mikhail_D_ltp has quit IRC | 19:39 | |
| *** epim has joined #openstack-ironic | 19:48 | |
| devananda | jroll: yes - merged a few | 19:54 |
| NobodyCam | devananda: your here | 19:55 |
| NobodyCam | :) | 19:55 |
| devananda | i'm back | 19:55 |
| NobodyCam | hey hey mind if I ask how you tested the ephemeral stuff | 19:56 |
| devananda | nope, dont mind | 19:56 |
| NobodyCam | you ran nova boot with --ephemeral then nova rebuild? | 19:57 |
| devananda | change flavor; nova boot; ssh into instance; write data to files on /vda1 and /vda3; nova rebuild; confirm if data still present in files on /vda1 and /vda3 | 19:58 |
| devananda | NobodyCam: you dont run "nova boot --ephemeral" | 19:58 |
| devananda | that shouldn't ever work | 19:58 |
| devananda | NobodyCam: you need a flavor with ephemeral_gb!=0, then you run "nova boot" and later run "nova rebuild --preserve-ephemeral" | 19:58 |
| NobodyCam | devananda: ack TY | 19:59 |
| *** epim has quit IRC | 20:06 | |
| *** ams0 has joined #openstack-ironic | 20:22 | |
| *** dwalleck__ has joined #openstack-ironic | 20:26 | |
| *** shakamunyi has quit IRC | 20:27 | |
| openstackgerrit | linggao proposed a change to openstack/ironic: Handle API error for XML client https://review.openstack.org/88448 | 20:35 |
| *** dwalleck_ has joined #openstack-ironic | 20:37 | |
| *** dwalleck__ has quit IRC | 20:40 | |
| *** dwalleck_ has quit IRC | 20:43 | |
| *** jdob has quit IRC | 20:45 | |
| *** ams0_ has joined #openstack-ironic | 20:54 | |
| *** ams0 has quit IRC | 20:55 | |
| *** shakamunyi has joined #openstack-ironic | 20:56 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Drivers determine acceptable power states https://review.openstack.org/86744 | 20:56 |
| *** linggao has quit IRC | 20:57 | |
| *** matty_dubs is now known as matty_dubs|gone | 21:04 | |
| *** shakamunyi has quit IRC | 21:07 | |
| *** krtaylor has quit IRC | 21:08 | |
| *** ams0_ has quit IRC | 21:10 | |
| *** ams0 has joined #openstack-ironic | 21:11 | |
| *** foexle has quit IRC | 21:13 | |
| *** jbjohnso has quit IRC | 21:13 | |
| devananda | rloo: just a thought -- a while back, we had only a single module constant for RPC API version. Since we clearly need that to be the same in both modules, why not just define it in one place? | 21:18 |
| devananda | rloo: after digesting my lunch a bit, the unit test to force them to stay in sync seems like just an artefact of poor design. What's the benefit of having two identical constants? | 21:19 |
| rloo | devananda: I was wondering about that myself. It seems like I asked about it, or maybe I combined something else that was duplicated. Can't remember. | 21:19 |
| rloo | devananda: yeah, I agree. | 21:19 |
| rloo | devananda: any idea why it was duplicated in the first place? | 21:20 |
| rloo | just poor design? :-) | 21:20 |
| Shrews | i think that's an excellent idea, and kicking myself for not having thought of that. ++ to the single constant | 21:25 |
| devananda | it was originally a single constant when I wrote i t... | 21:30 |
| openstackgerrit | A change was merged to openstack/ironic: RPC_API_VERSION out of sync https://review.openstack.org/92636 | 21:31 |
| rloo | devananda: fwiw, I took a quick look in nova code. Their 'rpcapi' code uses BASE_RPC_API_VERSION, value can be <= the manager's RPC_API_VERSION. I think. | 21:41 |
| *** shakamunyi has joined #openstack-ironic | 21:41 | |
| devananda | actually, it was defined in both files when I originally wrote it. that was lame of me. | 21:42 |
| devananda | rloo: I'm not sure what nova is doing with their versions tbh | 21:45 |
| devananda | they have a set of aliases to demarcate each release | 21:46 |
| rloo | devananda: I'm afraid to know. but it might be good to know as a heads up wrt something that may happen in ironic. | 21:46 |
| devananda | the ConductorManager and ComputeManager classes both define a class variable like this: target = messaging.Target(version='3.25') | 21:46 |
| NobodyCam | devananda: fyi I seem to be getting Error: Partition(s) on /dev/sdb are being used.\n in my ephemeral testing ... not sure why yet | 21:47 |
| devananda | then the respective rpcapi clients do | 21:47 |
| devananda | target = messaging.Target(topic=CONF.compute_topic, version='3.0') | 21:47 |
| devananda | version_cap = self.VERSION_ALIASES.get( ... | 21:47 |
| devananda | self.client = self.get_client(target, version_cap, serializer) | 21:47 |
| devananda | so the client defines a base version and a version cap | 21:47 |
| NobodyCam | but its dinner time now... :) | 21:47 |
| *** rloo has quit IRC | 21:48 | |
| devananda | whereas the server/service side just has a string for its actual current version | 21:48 |
| *** rloo has joined #openstack-ironic | 21:48 | |
| devananda | the only constant I see is the VERSION_ALIASES list | 21:48 |
| devananda | NobodyCam: i need more info to understand why you're gettnig that error. i dont have a /dev/sdb in my testing | 21:49 |
| *** zdiN0bot has quit IRC | 21:50 | |
| NobodyCam | I shouldn't either :-p so I need to see whats up | 21:52 |
| rloo | devananda: there's all this docn about the versioning: https://git.openstack.org/cgit/openstack/oslo-incubator/tree/openstack/common/rpc/dispatcher.py | 21:53 |
| rloo | devananda: client side vs server side. | 21:53 |
| devananda | NobodyCam: did you see derek's post on the ML regarding tripleo tests // moving check-tripleo-ironic-undercloud-precise into experimental queue? | 21:58 |
| devananda | NobodyCam: i recall you were working on that at one point, so thought you might have thoughts on it | 21:59 |
| devananda | rloo: i bet there's a similar doc for oslo.messaging (which we're now using) | 21:59 |
| rloo | devananda: oh. I didn't realize we were using oslo.messaging. Can't keep track of the oslo* stuff. | 22:01 |
| devananda | rloo: http://git.openstack.org/cgit/openstack/ironic/commit/?id=69dc8400db6e0c2b1b2be83f06f8ef15638cd54e | 22:02 |
| *** zdiN0bot has joined #openstack-ironic | 22:03 | |
| *** rloo has quit IRC | 22:04 | |
| *** rloo has joined #openstack-ironic | 22:04 | |
| rloo | being summoned. dinner time. ciao. maybe lucas-dinner will have some ideas ;) | 22:06 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Factoring out PXE and TFTP functions https://review.openstack.org/90233 | 22:11 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Adding a reference driver for the agent https://review.openstack.org/84795 | 22:11 |
| devananda | ugh | 22:15 |
| devananda | so after reading some of the oslo.messagign docs and nova code and tinkering a bit | 22:15 |
| devananda | i think we've introduced several non-backwards-compatible changes without bumping the major version (which we could argue is OK right now) | 22:18 |
| devananda | BUT we have also kept the minimum version at 1.0 this whole time, without taggign what version each method was implemented in | 22:19 |
| devananda | so really, that is what would have prevented any rolling upgrade from working | 22:19 |
| devananda | IIUC, the rpcapi settings have meant: supports talking to any server that speaks a version between 1.0 and RPC_API_VERSION, inclusively | 22:21 |
| devananda | which appears to be untrue for multiple reasons | 22:22 |
| *** radsy has joined #openstack-ironic | 22:28 | |
| *** radsy has joined #openstack-ironic | 22:28 | |
| *** zdiN0bot has quit IRC | 22:32 | |
| *** rwsu has quit IRC | 22:42 | |
| *** jgrimm has quit IRC | 22:43 | |
| *** shakamunyi has quit IRC | 22:51 | |
| *** lucas-dinner has quit IRC | 22:55 | |
| *** boris-42 has quit IRC | 22:57 | |
| *** boris-42 has joined #openstack-ironic | 22:58 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Pin RPC client version min == max https://review.openstack.org/92723 | 23:06 |
| devananda | rloo: ^ | 23:06 |
| *** neillc_away is now known as neillc | 23:09 | |
| *** epim has joined #openstack-ironic | 23:11 | |
| *** epim has quit IRC | 23:13 | |
| NobodyCam | devananda: ahhh sdb if from the conductor (seed in my case) so the nodes disk is sdb | 23:13 |
| NobodyCam | s/if/is/ | 23:13 |
| rloo | devananda: interesting. guess i have some background-code reading to do :-) | 23:18 |
| devananda | NobodyCam: huh? | 23:19 |
| *** romcheg2 has joined #openstack-ironic | 23:19 | |
| NobodyCam | that error is when the conductor has mounted the nodes disk.. so the conductors own disk is sda and the mounted nodes disk is sdb | 23:20 |
| *** romcheg1 has quit IRC | 23:21 | |
| devananda | NobodyCam: conductor should be addressing the node's disk as dev = "/dev/disk/by-path/ip-%s:%s-iscsi-%s-lun-%s | 23:27 |
| devananda | not as /dev/sdb | 23:27 |
| devananda | NobodyCam: oh. I think I know what's going on | 23:28 |
| devananda | NobodyCam: pastebin the full debug log from conductor during a rebuild | 23:28 |
| devananda | pls | 23:28 |
| devananda | NobodyCam: also, within the VM, try running "sudo sync" before you initiate the rebuild | 23:30 |
| devananda | see if that prevents the error | 23:30 |
| JoshNang | anyone run into something like "NoSuchOptError: no such option: my_ip"? | 23:30 |
| devananda | my suspicion is libvirt is still doing something under the hood with the FS, but it's just a hunch | 23:30 |
| devananda | JoshNang: there are a few spots where we import that option's value into other option(s) | 23:31 |
| devananda | but i haven't seen that error lately, no | 23:31 |
| *** zdiN0bot has joined #openstack-ironic | 23:31 | |
| JoshNang | devananda: right. i think tftp.tftp_server is one. i'll keep debugging | 23:32 |
| *** zdiN0bot has quit IRC | 23:34 | |
| *** mrda-away is now known as mrda | 23:38 | |
| NobodyCam | devananda: http://paste.openstack.org/show/KHCYpEsNNKXO9Bziew8f/ | 23:39 |
| NobodyCam | thats all I have so far will try the sync | 23:39 |
| devananda | NobodyCam: I get a 500 error | 23:39 |
| devananda | on that page | 23:39 |
| NobodyCam | oh and it is addressing correctly error is just getting reported that way | 23:39 |
| NobodyCam | http://paste.openstack.org/show/KHCYpEsNNKXO9Bziew8f | 23:40 |
| NobodyCam | ?? | 23:40 |
| NobodyCam | devananda: you may need to clear cookies for paste | 23:41 |
| *** ams0 has quit IRC | 23:42 | |
| *** ams0 has joined #openstack-ironic | 23:43 | |
| *** ams0 has quit IRC | 23:44 | |
| *** ams0 has joined #openstack-ironic | 23:45 | |
| *** ams0 has quit IRC | 23:45 | |
| *** ams0 has joined #openstack-ironic | 23:46 | |
| *** ams0 has quit IRC | 23:47 | |
| *** Hefeweizen has joined #openstack-ironic | 23:47 | |
| *** ams0 has joined #openstack-ironic | 23:47 | |
| devananda | clear cookies? | 23:53 |
| *** rloo has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!