| *** naohirot has joined #openstack-ironic | 00:03 | |
| *** meghal has joined #openstack-ironic | 00:10 | |
| *** meghal has quit IRC | 00:11 | |
| *** ParsectiX has quit IRC | 00:13 | |
| *** ParsectiX has joined #openstack-ironic | 00:14 | |
| jlvillal | GheRivero: That's a big patch you got there :) | 00:17 |
|---|---|---|
| GheRivero | but only 1-2 lines per file | 00:19 |
| *** meghal has joined #openstack-ironic | 00:19 | |
| GheRivero | anyway, I just found a typo so... | 00:20 |
| *** BadCub_ has quit IRC | 00:20 | |
| jlvillal | GheRivero: Okay. https://review.openstack.org/#/c/157602/1/ironic/api/config.py Intentional deletion? | 00:21 |
| * jlvillal thinks so | 00:22 | |
| GheRivero | yes and no. I was having some strange behaviour with the oslo_log lib, so with that change it passed all local tests, but wanted to test it in the gate | 00:24 |
| naohirot | good morning ironic, jlvillal. GheRivero | 00:25 |
| NobodyCam | morning naohirot :) | 00:26 |
| jlvillal | naohirot: Good morning. Ohayō. | 00:26 |
| GheRivero | morning all | 00:26 |
| naohirot | good evening NobodyCam :) | 00:27 |
| NobodyCam | :) morning GheRivero :) (you up in seattle? | 00:27 |
| NobodyCam | naohirot: :) | 00:27 |
| naohirot | jlvillal: ohayo :) | 00:27 |
| GheRivero | NobodyCam: yep | 00:27 |
| NobodyCam | :) | 00:27 |
| jlvillal | naohirot: I know how to use google translate ;) | 00:28 |
| naohirot | GheRivero: good evening :) | 00:28 |
| jlvillal | NobodyCam: GheRivero : We got people all along the west coast :) I'm in Oregon. | 00:29 |
| NobodyCam | nice :) | 00:29 |
| NobodyCam | there is a TripleO meetup in seattle ATm ... that is why I asked | 00:29 |
| * jlvillal would rather be working in Amsterdam :) | 00:29 | |
| NobodyCam | lol | 00:29 |
| GheRivero | jlvillal: I doubt I could do any job from Amsterdam, but now that Seattle has Weed Vending Machines... | 00:30 |
| NobodyCam | omg do they GheRivero :O | 00:31 |
| GheRivero | http://www.theatlantic.com/technology/archive/2015/02/the-radical-efficiency-of-the-pot-vending-machine/385165/ | 00:31 |
| NobodyCam | ROFL | 00:32 |
| GheRivero | although I prefer the coffe shops :) | 00:32 |
| NobodyCam | :-p | 00:32 |
| openstackgerrit | Merged openstack/ironic: Add concept of stable states to the state machine https://review.openstack.org/155529 | 00:37 |
| *** david-lyle is now known as david-lyle_afk | 00:59 | |
| meghal | NobodyCam: sorry I had to step out | 01:05 |
| meghal | NobodyCam: yes, I wanted to understand how are ironic nodes presented to the scheduler and how is the selection made ? | 01:05 |
| meghal | NobodyCam: I was under the assumption that nova-compute makes the selection | 01:05 |
| meghal | NobodyCam: that is what I saw in ironic icehouse | 01:05 |
| *** alex_xu has quit IRC | 01:06 | |
| NobodyCam | meghal: I would start with something like https://github.com/openstack/nova/blob/master/nova/scheduler/ironic_host_manager.py | 01:07 |
| NobodyCam | meghal: have you played with the nova scheduler? | 01:08 |
| *** alex_xu has joined #openstack-ironic | 01:09 | |
| meghal | NobodyCam: yes, I have played around with it, primarily for vms which pick up a compute node | 01:09 |
| NobodyCam | in a simple form we have ram, cpu, and disk | 01:10 |
| meghal | right | 01:10 |
| meghal | NobodyCam: so in ironic, I would register ironic node, which would show up in ironic node-list, do I need to do anything special while registering ? | 01:11 |
| meghal | because for scheduler it would be looking at picking up nova-compute | 01:12 |
| NobodyCam | so you need to setup a baremetal flavor : http://docs.openstack.org/developer/ironic/deploy/install-guide.html#flavor-creation | 01:13 |
| meghal | ahh so cpu_arch is the key, for bm flavor | 01:14 |
| *** dprince has quit IRC | 01:15 | |
| *** achanda has quit IRC | 01:16 | |
| NobodyCam | also see the enrollment section of that same page on creating a node | 01:16 |
| NobodyCam | but ya we do need a scheduler explanatory doc | 01:19 |
| meghal | NobodyCam: ok…ahh now I understood, doing above steps registers ironic nodes as hypervisors | 01:21 |
| *** pcrews has quit IRC | 01:22 | |
| NobodyCam | yep | 01:22 |
| meghal | so where does nova-compute fit in boot flow ? | 01:23 |
| NobodyCam | btw.. that whole doc isn't a bad read :) | 01:23 |
| NobodyCam | for that I would point you to: http://docs.openstack.org/developer/ironic/deploy/user-guide.html | 01:24 |
| NobodyCam | :-p | 01:24 |
| meghal | cool…thanks will go through it | 01:26 |
| NobodyCam | :) | 01:26 |
| *** killer_prince is now known as lazy_prince | 01:29 | |
| *** saripurigopi has joined #openstack-ironic | 01:33 | |
| meghal | "Nova scheduler applies filter and finds the eligible compute node." this statement is what confused me | 01:33 |
| meghal | in reality it is picking up an eligible ironic node and since that ironic node is linked to nova-compute, request goes to appropriate nova-compute | 01:34 |
| meghal | I was talking about step 2. here - http://docs.openstack.org/developer/ironic/deploy/user-guide.html#deploy-process | 01:35 |
| meghal | but thanks for the explanation and pointers to the docs, will still go through them in detail | 01:35 |
| meghal | but have gotten the idea | 01:35 |
| NobodyCam | ok | 01:35 |
| NobodyCam | remember to take a look at the first link too | 01:35 |
| meghal | yep…that install guide as well | 01:36 |
| NobodyCam | the nova code | 01:36 |
| NobodyCam | :) | 01:37 |
| *** penick has quit IRC | 01:40 | |
| meghal | ahh yes | 01:40 |
| NobodyCam | ;) | 01:40 |
| *** chlong has quit IRC | 01:40 | |
| *** chlong has joined #openstack-ironic | 01:41 | |
| NobodyCam | lol it could just that I tend to find reading the code helps a lot (at least for /me) | 01:41 |
| NobodyCam | ...could just BE that... | 01:42 |
| NobodyCam | :-p | 01:42 |
| meghal | yes…devs are always curious as to how does it work and tend to read the code | 01:43 |
| *** ParsectiX has quit IRC | 01:44 | |
| *** ParsectiX has joined #openstack-ironic | 01:45 | |
| meghal | I was just looking at wrong place from nova-compute, assuming that request would be reaching nova-compute just like normal vm flow and then nova-compute would be making the decision by calling ironic client, getting list of nodes and taking some decision on the list to select one | 01:45 |
| NobodyCam | ahh | 01:45 |
| *** lazy_prince is now known as killer_prince | 01:47 | |
| *** korekhov_ has quit IRC | 01:51 | |
| *** saripurigopi has quit IRC | 01:55 | |
| *** saripurigopi has joined #openstack-ironic | 01:55 | |
| *** killer_prince is now known as lazy_prince | 02:10 | |
| *** pensu has quit IRC | 02:13 | |
| *** meghal has quit IRC | 02:29 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: Consistent names of args and metavars, and help strings https://review.openstack.org/155031 | 02:34 |
| *** spandhe has quit IRC | 02:35 | |
| *** ramineni has joined #openstack-ironic | 02:41 | |
| *** saripurigopi has quit IRC | 02:45 | |
| *** yuanying has quit IRC | 02:53 | |
| NobodyCam | rloo: are you still here? | 02:55 |
| rloo | NobodyCam: sadly yes | 02:56 |
| NobodyCam | :-p me too | 02:56 |
| NobodyCam | :) on the ^^^ patch are you still going thou things? | 02:56 |
| NobodyCam | https://github.com/openstack/python-ironicclient/blob/master/ironicclient/tests/test_shell.py#L164-L168 | 02:56 |
| rloo | did that fail? | 02:57 |
| NobodyCam | not sure I just saw it while looking | 02:57 |
| NobodyCam | zuul is passing | 02:58 |
| rloo | the unit tests pass on my dev box. | 02:59 |
| NobodyCam | ya they are passing | 02:59 |
| rloo | NobodyCam: if it fails, you have my permission to fix it ;) | 03:00 |
| NobodyCam | that for bash completion | 03:00 |
| NobodyCam | it is passing ;) hehehe | 03:00 |
| NobodyCam | lgtm :) | 03:00 |
| rloo | Are you actually looking at the output as the tests are being run? | 03:00 |
| NobodyCam | I checked zuul web page and all the tests that had run passed | 03:01 |
| rloo | phew. thanks! | 03:01 |
| NobodyCam | :) | 03:02 |
| *** Nisha has joined #openstack-ironic | 03:05 | |
| rloo | g'night NobodyCam. I'm off (finally). Talk to you on Monday! | 03:14 |
| *** rloo has quit IRC | 03:15 | |
| *** coolsvap_ is now known as coolsvap | 03:15 | |
| openstackgerrit | Anusha Ramineni proposed openstack/ironic-specs: Add get/set boot mode to Management Interface https://review.openstack.org/129529 | 03:18 |
| *** Marga_ has quit IRC | 03:26 | |
| *** coolsvap is now known as coolsvap_ | 03:34 | |
| *** yog_ has joined #openstack-ironic | 03:34 | |
| *** yuanying has joined #openstack-ironic | 03:38 | |
| *** coolsvap_ is now known as coolsvap | 03:43 | |
| *** zz_jgrimm is now known as jgrimm | 03:46 | |
| *** saripurigopi has joined #openstack-ironic | 03:51 | |
| *** pcrews has joined #openstack-ironic | 04:00 | |
| *** rameshg87 has joined #openstack-ironic | 04:09 | |
| openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: Add Proliant Hardware Manager for IPA https://review.openstack.org/157291 | 04:20 |
| *** Marga_ has joined #openstack-ironic | 04:21 | |
| *** spandhe has joined #openstack-ironic | 04:23 | |
| *** harlowja is now known as harlowja_away | 04:28 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for pxe_ilo driver https://review.openstack.org/154808 | 04:31 |
| *** lazy_prince is now known as killer_prince | 04:35 | |
| *** killer_prince is now known as lazy_prince | 04:36 | |
| rameshg87 | goot morning ironic | 04:36 |
| openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: Add Proliant Hardware Manager for IPA https://review.openstack.org/157291 | 04:36 |
| rameshg87 | i meant good morning :) | 04:36 |
| mrda | hi rameshg87 | 04:40 |
| *** coolsvap is now known as coolsvap_ | 04:41 | |
| rameshg87 | hello mrda | 04:41 |
| *** coolsvap_ is now known as coolsvap | 04:48 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for iscsi_ilo driver https://review.openstack.org/154814 | 04:49 |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: Add Proliant Hardware Manager for IPA https://review.openstack.org/157291 | 04:50 |
| *** sandhya has joined #openstack-ironic | 04:58 | |
| sandhya | devananda, NobodyCam, dtantsur: Could you suggest points to come to a consensus on the Chassis Discovery tool blueprint - https://review.openstack.org/#/c/134866/ | 05:00 |
| *** spandhe has quit IRC | 05:01 | |
| rameshg87 | sandhya, i guess better to put in on mailing list once and then add an agenda in next week's team meeting for this | 05:01 |
| rameshg87 | sandhya, we have come to consensus on things like this in last few team meetings | 05:02 |
| sandhya | Thank you... | 05:02 |
| *** achanda has joined #openstack-ironic | 05:08 | |
| *** yog_ has quit IRC | 05:30 | |
| *** cinerama has quit IRC | 05:42 | |
| *** cinerama has joined #openstack-ironic | 05:43 | |
| *** yog_ has joined #openstack-ironic | 05:46 | |
| openstackgerrit | Merged stackforge/proliantutils: Add Proliant Hardware Manager for IPA https://review.openstack.org/157291 | 05:51 |
| *** Marga_ has quit IRC | 05:55 | |
| *** ukalifon1 has joined #openstack-ironic | 06:01 | |
| *** mrda is now known as mrda-weekend | 06:01 | |
| *** dlpartain has joined #openstack-ironic | 06:09 | |
| *** lazy_prince is now known as killer_prince | 06:10 | |
| *** killer_prince is now known as lazy_prince | 06:11 | |
| *** sandhya has quit IRC | 06:28 | |
| *** achanda has quit IRC | 06:35 | |
| *** achanda has joined #openstack-ironic | 06:38 | |
| *** ParsectiX has quit IRC | 06:38 | |
| *** ParsectiX has joined #openstack-ironic | 06:39 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for agent_ilo driver https://review.openstack.org/154816 | 06:45 |
| *** dlpartain has quit IRC | 06:52 | |
| *** gridinv has joined #openstack-ironic | 06:53 | |
| *** ParsectiX has quit IRC | 07:00 | |
| *** yog_ has quit IRC | 07:04 | |
| *** ndipanov has quit IRC | 07:04 | |
| openstackgerrit | Anusha Ramineni proposed openstack/ironic-specs: Add get/set boot mode to Management Interface https://review.openstack.org/129529 | 07:10 |
| *** yog_ has joined #openstack-ironic | 07:17 | |
| *** ndipanov has joined #openstack-ironic | 07:20 | |
| *** meghal has joined #openstack-ironic | 07:27 | |
| *** ukalifon1 has quit IRC | 07:29 | |
| *** lazy_prince has quit IRC | 07:29 | |
| *** meghal1 has joined #openstack-ironic | 07:30 | |
| *** killer_prince has joined #openstack-ironic | 07:31 | |
| *** killer_prince is now known as lazy_prince | 07:32 | |
| *** meghal has quit IRC | 07:33 | |
| *** andreykurilin_ has joined #openstack-ironic | 07:40 | |
| *** abc_ has joined #openstack-ironic | 07:47 | |
| abc_ | hi | 07:47 |
| abc_ | i m trying to install ironic service using packstack ( RDO-Juno) on a single node (CentOS7) | 07:48 |
| abc_ | but the installation failed with some neutron error | 07:48 |
| abc_ | as soon as I edited answer file to "not" install ironic and re-run, installation was successful | 07:49 |
| *** abc_ has quit IRC | 07:52 | |
| *** lazy_prince has quit IRC | 07:54 | |
| *** ndipanov has quit IRC | 07:57 | |
| *** ndipanov has joined #openstack-ironic | 07:58 | |
| *** killer_prince has joined #openstack-ironic | 08:02 | |
| *** killer_prince is now known as lazy_prince | 08:02 | |
| *** athomas has joined #openstack-ironic | 08:04 | |
| *** meghal1 has quit IRC | 08:06 | |
| *** ndipanov has quit IRC | 08:08 | |
| *** yuanying has quit IRC | 08:15 | |
| *** achanda has quit IRC | 08:15 | |
| *** yuanying_ has joined #openstack-ironic | 08:15 | |
| *** dtantsur|afk is now known as dtantsur | 08:16 | |
| dtantsur | Morning Ironic | 08:16 |
| *** stendulker has joined #openstack-ironic | 08:20 | |
| *** ndipanov has joined #openstack-ironic | 08:20 | |
| *** andreykurilin_ has quit IRC | 08:21 | |
| Nisha | dtantsur, hi | 08:24 |
| *** ifarkas has joined #openstack-ironic | 08:24 | |
| dtantsur | o/ | 08:24 |
| Nisha | i wanted to disccus your comment on InspectInterface method inspect_hardware() | 08:25 |
| Nisha | dtantsur, about what it shall return | 08:25 |
| dtantsur | anything that will allow you distinguish between in-band and OOB case.. | 08:26 |
| Nisha | dtantsur, now as INSPECTED is not there, does the method need to return anything | 08:26 |
| Nisha | dtantsur, we have two options | 08:27 |
| Nisha | dtantsur, 1. return MANAGEABLE on success , error on failure | 08:27 |
| Nisha | dtantsur, 2. just return failure and on success dont return anything | 08:28 |
| dtantsur | I'm not talking about failure at all (and never did) | 08:28 |
| Nisha | yes | 08:28 |
| dtantsur | how do you distinguish between OOB (proceed with 'done' event) and in-band (do nothing) case? | 08:28 |
| Nisha | dtantsur, on succes it can either return manageable or it can return none | 08:28 |
| *** coolsvap is now known as coolsvap_ | 08:29 | |
| Nisha | dtantsur, why in in-band conductor shouldnt wait for the inband driver method to return? | 08:29 |
| *** bauzas is now known as bauwser | 08:29 | |
| dtantsur | how can it fail, that's async process | 08:29 |
| dtantsur | well, we probably could, but that will mean occupying the green thread for indefinite amount of time with polling | 08:30 |
| dtantsur | which will limit concurrency of the whole conductor | 08:31 |
| dtantsur | (in first comments s/fail/wait/) | 08:34 |
| Nisha | dtantsur, i think then both can follow same correct | 08:34 |
| Nisha | letme explain | 08:34 |
| *** coolsvap_ is now known as coolsvap | 08:35 | |
| *** coolsvap is now known as coolsvap_ | 08:35 | |
| Nisha | dtantsur, ok before that how does in-band execute 'done' event? | 08:35 |
| Nisha | or 'fail' event? | 08:36 |
| dtantsur | Nisha, in a periodic task after it detects that discoverd has finished (by polling it from this periodic task) | 08:36 |
| Nisha | dtantsur, so thast done by conductor async method | 08:37 |
| Nisha | not by the inband InspectHardware.inspect_hardware() | 08:38 |
| Nisha | ? | 08:38 |
| dtantsur | not sure I understand you, it's done by a periodic task. that means inspect_hardware() will return much earlier | 08:38 |
| dtantsur | have a look how deploy is done in Ironic, it's the same stuff | 08:38 |
| Nisha | yes the same way i have implemented | 08:38 |
| Nisha | please see https://review.openstack.org/#/c/149823/18/ironic/conductor/manager.py | 08:38 |
| Nisha | the periodic task is at the end | 08:40 |
| dtantsur | Nisha, it's all correct, except for you report error when inspect_hardware returns INSPECTING, that's wrong | 08:41 |
| * dtantsur left a comment | 08:41 | |
| Nisha | i dont erro when its INSPECTING | 08:41 |
| Nisha | it is when it is not INSPECTING or MANAGEABLE | 08:41 |
| dtantsur | ah sorry, I didn't notice it's !=, not == | 08:42 |
| * dtantsur has to wake up before discussion stuff | 08:42 | |
| Nisha | dtantsur, :) | 08:42 |
| Nisha | so to you it looks ok if we return MANAGEABLE?...IMO that is not required. when the async call has finished without raising error, we can directly call event 'done' | 08:43 |
| dtantsur | so the logic seems to be correct | 08:43 |
| dtantsur | no, we can't, it may be not finished (i.e. return INSPECTING) | 08:43 |
| Nisha | dtantsur, means "if new_state == states.MANAGEABLE:" this is not required | 08:43 |
| dtantsur | it is | 08:44 |
| *** yuanying_ has quit IRC | 08:44 | |
| dtantsur | in-band inspection won't finish at this point | 08:44 |
| dtantsur | the code you have now is correct | 08:44 |
| *** eghobo has joined #openstack-ironic | 08:44 | |
| dtantsur | sorry, have to run now, see you later | 08:46 |
| *** dtantsur is now known as dtantsur|bbl | 08:46 | |
| *** jistr has joined #openstack-ironic | 08:46 | |
| Nisha | i think we can do this way http://paste.openstack.org/show/178238/ | 08:47 |
| dtantsur|bbl | no, you can't process 'done' before inspection is actually done... | 08:49 |
| * dtantsur|bbl gone for real | 08:49 | |
| *** romcheg has joined #openstack-ironic | 08:57 | |
| *** jcoufal has joined #openstack-ironic | 09:07 | |
| *** Marga_ has joined #openstack-ironic | 09:09 | |
| *** erwan_taf has joined #openstack-ironic | 09:12 | |
| *** lucasagomes has joined #openstack-ironic | 09:15 | |
| *** MattMan has joined #openstack-ironic | 09:17 | |
| *** pas-ha has joined #openstack-ironic | 09:21 | |
| *** derekh has joined #openstack-ironic | 09:27 | |
| *** eghobo has quit IRC | 09:29 | |
| *** Marga_ has quit IRC | 09:31 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add InspectInterface for node-introspection https://review.openstack.org/149817 | 09:34 |
| rameshg87 | lucasagomes, hi | 09:40 |
| rameshg87 | lucasagomes, i had a question | 09:40 |
| lucasagomes | rameshg87, yes? | 09:46 |
| rameshg87 | lucasagomes, yesterday Nisha's patch for nova got merged | 09:46 |
| rameshg87 | lucasagomes, with that the selections in flavor will come down to instance_info | 09:47 |
| lucasagomes | right | 09:47 |
| Nisha | lucasagomes, how do you expect user to give boot_option in flavor? | 09:48 |
| rameshg87 | lucasagomes, for this, first the operator has to put properties/capbilities=boot_option:local | 09:48 |
| lucasagomes | Nisha, setting capabilities:boot_option=local in the flavor? | 09:48 |
| rameshg87 | lucasagomes, then the node will be matched if user selects a flavor with capbilities:boot_option=local | 09:48 |
| lucasagomes | rameshg87, yes | 09:48 |
| Nisha | and what will node contain? | 09:49 |
| rameshg87 | lucasagomes, so if there is nothing in the flavor, then it will be netboot | 09:49 |
| lucasagomes | rameshg87, yeah, that's the fallback to be backwards compat | 09:49 |
| rameshg87 | Nisha, node will contain properties/capbilities=boot_option:local | 09:49 |
| rameshg87 | lucasagomes, boot_option is not a boolean (eventhough it has only two values) | 09:50 |
| rameshg87 | lucasagomes, such a use-case makes more sense for a boolean actually | 09:50 |
| lucasagomes | rameshg87, like local=False or True? | 09:50 |
| rameshg87 | lucasagomes, exactly | 09:50 |
| rameshg87 | lucasagomes, localboot=True, something like that | 09:50 |
| rameshg87 | lucasagomes, doesn't that sound better ? | 09:51 |
| lucasagomes | well, it could be, I think the boot_option thing came when I took over the spec for local boot | 09:51 |
| rameshg87 | lucasagomes, do we expect more ways to boot the machine ? :) | 09:51 |
| lucasagomes | rameshg87, yeah, I wonder if it's flexible enough tho... | 09:51 |
| lucasagomes | for example and if we want full disk images to be able to choose between pxe and local boot | 09:51 |
| lucasagomes | right now they only do localboot afaict | 09:51 |
| lucasagomes | rameshg87, not of the top of my head really | 09:52 |
| rameshg87 | lucasagomes, okay .. do you feel localboot=True is better than current boot_option=local ? | 09:52 |
| rameshg87 | lucasagomes, localboot=True sounded better to me. | 09:53 |
| lucasagomes | rameshg87, yes, since we only support 2 options | 09:53 |
| lucasagomes | but we need to think whether more options can be used there in the future | 09:53 |
| rameshg87 | lucasagomes, how about proposing a change to localboot spec to rename it | 09:53 |
| lucasagomes | leaving it flexible local or netboot might be a good thing | 09:53 |
| lucasagomes | rameshg87, idk, like what if people want to have a separated boot partition | 09:54 |
| lucasagomes | so they pass a diff value and we can do that | 09:54 |
| rameshg87 | lucasagomes, okay .. | 09:54 |
| lucasagomes | just an stupid example here, but I think that leaving it flexible is not a bad idea | 09:54 |
| lucasagomes | rameshg87, yeah sure, proposing something is always good | 09:54 |
| rameshg87 | lucasagomes, okay .. | 09:55 |
| lucasagomes | but let's also think about it a little, how it could potentially be used in the future | 09:55 |
| rameshg87 | lucasagomes, yeah may be a spec change opens up to get everyone's thoughts | 09:56 |
| lucasagomes | rameshg87, yeah def... the boot_option was in the spec and all proposing a change there is fine | 09:56 |
| lucasagomes | +1 | 09:56 |
| rameshg87 | lucasagomes, okay | 09:56 |
| rameshg87 | thanks | 09:56 |
| lucasagomes | rameshg87, yw | 09:57 |
| *** ndipanov is now known as ndipanoff | 09:58 | |
| vdrok | morning ironic | 09:59 |
| *** mdbooth has joined #openstack-ironic | 10:00 | |
| *** yog_ has quit IRC | 10:18 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Generic changes for Node Inspection https://review.openstack.org/149823 | 10:19 |
| openstackgerrit | Alexis Lee proposed openstack/ironic: Don't write PXE config during ACTIVE takeover https://review.openstack.org/156250 | 10:28 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Generic changes for Node Inspection https://review.openstack.org/149823 | 10:36 |
| *** athomas has quit IRC | 10:37 | |
| *** ukalifon1 has joined #openstack-ironic | 10:41 | |
| *** cinerama has quit IRC | 10:43 | |
| *** cinerama has joined #openstack-ironic | 10:43 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: iLO implementation for hardware inspection https://review.openstack.org/151596 | 10:44 |
| *** athomas has joined #openstack-ironic | 10:47 | |
| *** Nisha has quit IRC | 10:48 | |
| *** romcheg1 has joined #openstack-ironic | 10:50 | |
| *** romcheg has quit IRC | 10:51 | |
| *** romcheg has joined #openstack-ironic | 10:53 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add driver interface for RAID configuration https://review.openstack.org/155230 | 10:53 |
| *** romcheg1 has quit IRC | 10:55 | |
| *** dlpartain has joined #openstack-ironic | 10:57 | |
| *** dlpartain has left #openstack-ironic | 10:59 | |
| *** viktors is now known as viktors|lunch | 11:01 | |
| openstackgerrit | Anusha Ramineni proposed openstack/ironic: Implement Cleaning in DriverInterfaces https://review.openstack.org/157208 | 11:01 |
| openstackgerrit | Anusha Ramineni proposed openstack/ironic: Implement Cleaning States https://review.openstack.org/153444 | 11:01 |
| openstackgerrit | Anusha Ramineni proposed openstack/ironic: Add Cleaning Operations for iLO drivers https://review.openstack.org/157715 | 11:01 |
| openstackgerrit | Anusha Ramineni proposed openstack/ironic: Add Cleaning Operations for iLO drivers https://review.openstack.org/157715 | 11:05 |
| rameshg87 | lucasagomes, hi | 11:06 |
| rameshg87 | lucasagomes, what do you think about jrolls comment on https://review.openstack.org/#/c/156587/1/ironic/drivers/modules/deploy_utils.py | 11:06 |
| *** ramineni has quit IRC | 11:06 | |
| rameshg87 | lucasagomes, mbr has bytes 510 and 511 as boot signature | 11:07 |
| rameshg87 | lucasagomes, but can't guarantee all bios will honour this | 11:08 |
| rameshg87 | lucasagomes, i guess it's better to change bootable=True only for localboot | 11:08 |
| lucasagomes | rameshg87, right, I didn't read. But ok... instead of passing the node_uuid we could pass the node object to the function partitioning the disk | 11:09 |
| lucasagomes | and look if localboot is set | 11:09 |
| lucasagomes | rameshg87, don't get the mbr thing too, its 512 bytes, 440 I think the bootloader code, then 6 bytes are for signature and optional stuff | 11:10 |
| lucasagomes | than 4 entries of 14 bytes for the part table | 11:10 |
| lucasagomes | IIRC | 11:10 |
| rameshg87 | lucasagomes, yeah i think i will change than going round of this. | 11:13 |
| rameshg87 | lucasagomes, even i can't guarantee setting bootable=true for all cases will always cause the expected behaviour on all hardware :) | 11:13 |
| *** ukalifon1 has quit IRC | 11:15 | |
| lucasagomes | rameshg87, yeah exactly | 11:16 |
| lucasagomes | rameshg87, commented and changed my vote then | 11:17 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Support for non-Glance image references https://review.openstack.org/136741 | 11:18 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Add the image extension (for local boot) https://review.openstack.org/157469 | 11:18 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: PXE drivers support for IPA https://review.openstack.org/155728 | 11:18 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Localboot support for IPA https://review.openstack.org/157723 | 11:18 |
| rameshg87 | lucasagomes, okay | 11:24 |
| * rameshg87 leaves for home | 11:24 | |
| *** rameshg87 has quit IRC | 11:24 | |
| lucasagomes | rameshg87, safe trip back home | 11:24 |
| *** stendulker has quit IRC | 11:25 | |
| *** openstack has joined #openstack-ironic | 20:23 | |
| *** linggao_ has joined #openstack-ironic | 20:23 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!