*** 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!