*** jcooley_ has joined #openstack-ironic | 00:07 | |
*** jcooley_ has quit IRC | 00:13 | |
*** matsuhashi has joined #openstack-ironic | 00:23 | |
*** jcooley_ has joined #openstack-ironic | 00:30 | |
*** matsuhashi has quit IRC | 00:33 | |
*** matsuhashi has joined #openstack-ironic | 00:34 | |
*** jcooley_ has quit IRC | 00:37 | |
*** matsuhashi has quit IRC | 00:38 | |
*** matsuhashi has joined #openstack-ironic | 01:04 | |
*** dkehn__ has joined #openstack-ironic | 01:04 | |
*** dkehn has quit IRC | 01:05 | |
*** nosnos has joined #openstack-ironic | 01:05 | |
*** dkehn__ has quit IRC | 01:05 | |
*** dkehn__ has joined #openstack-ironic | 01:05 | |
*** dkehn_ has quit IRC | 01:06 | |
*** dkehn has joined #openstack-ironic | 01:07 | |
*** dkehn__ is now known as dkehn_ | 01:09 | |
*** jcooley_ has joined #openstack-ironic | 01:30 | |
openstackgerrit | A change was merged to openstack/ironic: Use StringType from WSME https://review.openstack.org/73248 | 01:37 |
---|---|---|
*** jcooley_ has quit IRC | 01:37 | |
openstackgerrit | Lin Tan proposed a change to openstack/ironic: Use six.StringIO/BytesIO instead of StringIO.StringIO https://review.openstack.org/73896 | 02:16 |
*** matsuhashi has quit IRC | 02:18 | |
*** jcooley_ has joined #openstack-ironic | 02:26 | |
*** matsuhashi has joined #openstack-ironic | 02:26 | |
*** jcooley_ has quit IRC | 02:31 | |
*** jcooley_ has joined #openstack-ironic | 03:01 | |
*** dkehn__ has joined #openstack-ironic | 03:38 | |
*** dkehn_ has quit IRC | 03:41 | |
*** dkehn has quit IRC | 03:41 | |
*** dkehn has joined #openstack-ironic | 03:42 | |
*** matsuhashi has quit IRC | 03:48 | |
*** jcooley_ has quit IRC | 03:53 | |
*** dkehn__ is now known as dkehn_ | 04:16 | |
*** matsuhashi has joined #openstack-ironic | 04:32 | |
*** Haomeng has quit IRC | 05:24 | |
*** killer_prince has quit IRC | 05:35 | |
openstackgerrit | Lin Tan proposed a change to openstack/ironic: Use six.StringIO/BytesIO instead of StringIO.StringIO https://review.openstack.org/73896 | 05:36 |
*** killer_prince has joined #openstack-ironic | 05:39 | |
openstackgerrit | Jenkins proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/71192 | 06:06 |
*** saju_m has joined #openstack-ironic | 06:15 | |
*** nosnos has quit IRC | 06:42 | |
*** nosnos has joined #openstack-ironic | 06:52 | |
*** ifarkas has joined #openstack-ironic | 07:30 | |
*** mdurnosvistov has joined #openstack-ironic | 07:37 | |
*** Alexei_987 has joined #openstack-ironic | 07:38 | |
*** ndipanov has joined #openstack-ironic | 07:48 | |
openstackgerrit | yangxurong proposed a change to openstack/python-ironicclient: py3kcompat: remove in python-ceilometerclient https://review.openstack.org/73956 | 07:49 |
*** dshulyak has joined #openstack-ironic | 07:49 | |
*** loki__ has joined #openstack-ironic | 08:18 | |
*** yuriyz has joined #openstack-ironic | 08:25 | |
*** mdurnosvistov has quit IRC | 08:25 | |
*** saju_m has quit IRC | 08:31 | |
openstackgerrit | Andrey Kurilin proposed a change to openstack/python-ironicclient: Sync cliutils from oslo https://review.openstack.org/72417 | 08:39 |
openstackgerrit | Haomeng,Wang proposed a change to openstack/ironic: sync oslo rpc to ironic https://review.openstack.org/73971 | 08:49 |
*** Haomeng has joined #openstack-ironic | 08:49 | |
openstackgerrit | Haomeng,Wang proposed a change to openstack/ironic: sync oslo rpc to ironic https://review.openstack.org/73971 | 08:52 |
openstackgerrit | Andrey Kurilin proposed a change to openstack/python-ironicclient: Reuse cliutils from common code https://review.openstack.org/72418 | 08:56 |
*** saju_m has joined #openstack-ironic | 08:58 | |
openstackgerrit | Haomeng,Wang proposed a change to openstack/ironic: sync oslo rpc to ironic https://review.openstack.org/73971 | 08:58 |
*** jistr has joined #openstack-ironic | 09:01 | |
openstackgerrit | Andrey Kurilin proposed a change to openstack/python-ironicclient: Reuse module `cliutils` from common code https://review.openstack.org/72418 | 09:01 |
*** derekh has joined #openstack-ironic | 09:14 | |
*** mdurnosvistov has joined #openstack-ironic | 09:17 | |
*** lucasagomes has joined #openstack-ironic | 09:18 | |
*** shausy has joined #openstack-ironic | 09:33 | |
openstackgerrit | Andrey Kurilin proposed a change to openstack/python-ironicclient: Sync latest code and reuse exceptions from oslo https://review.openstack.org/71500 | 09:37 |
*** matsuhashi has quit IRC | 09:54 | |
*** lazy_prince has joined #openstack-ironic | 09:57 | |
*** killer_prince has quit IRC | 09:58 | |
*** lazy_prince is now known as killer_prince | 09:58 | |
openstackgerrit | Andrey Kurilin proposed a change to openstack/python-ironicclient: Sync cliutils from oslo https://review.openstack.org/72417 | 10:00 |
*** athomas has joined #openstack-ironic | 10:02 | |
*** matsuhashi has joined #openstack-ironic | 10:02 | |
*** aignatov is now known as aignatov_ | 10:02 | |
*** max_lobur has joined #openstack-ironic | 10:04 | |
openstackgerrit | Andrey Kurilin proposed a change to openstack/python-ironicclient: Reuse module `cliutils` from common code https://review.openstack.org/72418 | 10:16 |
*** matsuhashi has quit IRC | 10:17 | |
*** matsuhashi has joined #openstack-ironic | 10:17 | |
GheRivero | tox | 10:19 |
GheRivero | ups... not here | 10:19 |
*** matsuhashi has quit IRC | 10:22 | |
max_lobur | :) | 10:25 |
max_lobur | morning Ironic | 10:25 |
openstackgerrit | Ghe Rivero proposed a change to openstack/ironic: Generic Multiplexed method https://review.openstack.org/73322 | 10:30 |
openstackgerrit | Ghe Rivero proposed a change to openstack/ironic: Set boot device to PXE when deploying https://review.openstack.org/71332 | 10:30 |
openstackgerrit | Ghe Rivero proposed a change to openstack/ironic: Move ipmitool _set_boot_device to VendorPassthru https://review.openstack.org/71585 | 10:30 |
openstackgerrit | Ghe Rivero proposed a change to openstack/ironic: Move ipminative _set_boot_device to VendorPassthru https://review.openstack.org/72495 | 10:30 |
Haomeng | morning GheRivero, max_lobur :) | 10:33 |
mdurnosvistov | Morning all! :) | 10:36 |
*** aignatov_ is now known as aignatov | 10:37 | |
*** martyntaylor has joined #openstack-ironic | 10:45 | |
*** vkozhukalov has joined #openstack-ironic | 10:46 | |
*** matsuhashi has joined #openstack-ironic | 10:47 | |
vkozhukalov | Can anybody inform me, what for the node "extra" is. By default it is empty. Is it ok, if my custom driver uses this field to store deploy progress there? | 10:50 |
Haomeng | vkozhukalov: extra is used for node's meta data | 10:58 |
Haomeng | vkozhukalov: that is not required fields for ironic, can be used for customer driver | 10:59 |
*** tatyana has joined #openstack-ironic | 11:00 | |
max_lobur | vkozhukalov: hi! | 11:01 |
vkozhukalov | Haomeng, thanks. And another question. What kind of metadata is that? | 11:01 |
max_lobur | typically we track deploy progress through node states | 11:01 |
max_lobur | are you going to provide percentage ? | 11:02 |
max_lobur | vkozhukalov: metadata like | 11:04 |
max_lobur | "priority": "low", | 11:04 |
max_lobur | "role": "webserver" | 11:04 |
max_lobur | and so on | 11:04 |
vkozhukalov | max_lobur, yes, i want it to be able to provide percentage. my custom driver makes comprehensive disk partitioning and some other tasks. so i need to have something more detailed than just a set of states. | 11:06 |
vkozhukalov | max_lobur, thanks very much. | 11:06 |
max_lobur | https://github.com/openstack/ironic/blob/master/ironic/common/states.py | 11:07 |
max_lobur | I see | 11:07 |
max_lobur | well, I think extra is suitable for this | 11:08 |
*** athomas has quit IRC | 11:34 | |
*** matsuhashi has quit IRC | 11:37 | |
*** aignatov is now known as aignatov_ | 11:40 | |
max_lobur | lucasagomes: r u around? | 11:41 |
*** athomas has joined #openstack-ironic | 11:44 | |
*** shausy has quit IRC | 11:48 | |
*** shausy has joined #openstack-ironic | 11:48 | |
*** aignatov_ is now known as aignatov | 11:50 | |
*** aignatov is now known as aignatov_ | 11:51 | |
*** aignatov_ is now known as aignatov | 11:57 | |
*** killer_prince has quit IRC | 12:01 | |
lucasagomes | max_lobur, howdy! | 12:02 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/python-ironicclient: Add set-console-mode, get-console commands https://review.openstack.org/74007 | 12:07 |
max_lobur | hi lucasagomes :) | 12:08 |
max_lobur | maybe you know | 12:08 |
max_lobur | https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L252 | 12:08 |
max_lobur | why we use shared lock for vendor passthru? | 12:09 |
max_lobur | looks like an error, because we changing node state, not just read i | 12:09 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/python-ironicclient: Add set-console-mode, get-console commands https://review.openstack.org/74007 | 12:09 |
max_lobur | *if | 12:09 |
max_lobur | gah | 12:09 |
max_lobur | * it :) | 12:09 |
*** loki__ has quit IRC | 12:10 | |
lucasagomes | max_lobur, uu, yea looks like a potential bug :) | 12:10 |
lucasagomes | continue_deploy for e.g would change the node states as u said | 12:11 |
lucasagomes | I bet it should be shared=False | 12:11 |
lucasagomes | max_lobur, good catch :) | 12:12 |
max_lobur | thx :) | 12:14 |
*** lucasagomes is now known as lucas-hungry | 12:32 | |
openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Sync Oslo service module to Ironic https://review.openstack.org/74025 | 12:55 |
*** dshulyak has quit IRC | 12:56 | |
*** shausy has quit IRC | 13:04 | |
*** shausy has joined #openstack-ironic | 13:05 | |
Haomeng | yuriyz:ping | 13:07 |
*** nosnos has quit IRC | 13:08 | |
*** jdob has joined #openstack-ironic | 13:41 | |
*** lucas-hungry is now known as lucasagomes | 13:42 | |
*** aignatov is now known as aignatov_ | 13:43 | |
agordeev | Hello. Could anyone tell me will the meeting happen today? | 13:45 |
max_lobur | agordeev: it will happen as usually | 13:46 |
agordeev | max_lobur: very good, thanks! | 13:47 |
max_lobur | np :) | 13:47 |
*** aignatov_ is now known as aignatov | 14:03 | |
*** saju_m has quit IRC | 14:15 | |
*** jdob has quit IRC | 14:19 | |
*** jdob has joined #openstack-ironic | 14:19 | |
*** jdob_ has joined #openstack-ironic | 14:37 | |
*** jdob has quit IRC | 14:37 | |
*** blamar has joined #openstack-ironic | 14:38 | |
*** jdob_ is now known as jdob | 14:38 | |
*** blamar has quit IRC | 14:40 | |
*** blamar has joined #openstack-ironic | 14:40 | |
*** dkehn__ has joined #openstack-ironic | 14:41 | |
*** dkehn has quit IRC | 14:43 | |
*** dkehn_ has quit IRC | 14:44 | |
*** dkehn has joined #openstack-ironic | 14:46 | |
*** dkehn__ is now known as dkehn_ | 14:46 | |
*** matty_dubs|gone is now known as matty_dubs | 14:47 | |
*** anniec has joined #openstack-ironic | 14:48 | |
NobodyCam | good morning Ironic | 14:49 |
matty_dubs | Howdy NobodyCam | 14:50 |
NobodyCam | hey hey matty_dubs | 14:51 |
*** killer_prince has joined #openstack-ironic | 14:52 | |
openstackgerrit | Imre Farkas proposed a change to openstack/ironic: Add support for custom libvirt uri https://review.openstack.org/74062 | 14:53 |
*** loki__ has joined #openstack-ironic | 14:53 | |
openstackgerrit | Max Lobur proposed a change to openstack/ironic: Refactor vendor_passthru to use conductor async workers. https://review.openstack.org/74063 | 14:55 |
openstackgerrit | Max Lobur proposed a change to openstack/ironic: Refactor vendor_passthru to use conductor async workers. https://review.openstack.org/74063 | 15:04 |
*** rustlebee is now known as russellb | 15:08 | |
*** rwsu has joined #openstack-ironic | 15:09 | |
*** linggao has joined #openstack-ironic | 15:14 | |
*** loki__ has quit IRC | 15:20 | |
*** loki__ has joined #openstack-ironic | 15:21 | |
*** loki__ has quit IRC | 15:26 | |
*** coolsvap has joined #openstack-ironic | 15:26 | |
*** shausy has quit IRC | 15:26 | |
max_lobur | bbiaw | 15:37 |
*** max_lobur has quit IRC | 15:37 | |
*** max_lobur has joined #openstack-ironic | 15:37 | |
openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Add string representation method to MultiType https://review.openstack.org/74075 | 15:38 |
NobodyCam | lucasagomes: devananda just fyi I added a try to nova driver that should address the issue deva had starting nova ( http://paste.openstack.org/show/JQR3BWFNJl5mLl2x97Hq ) | 15:47 |
lucasagomes | NobodyCam, ack thanks :D | 15:48 |
lucasagomes | btw I'm adding tests to the volume driver | 15:49 |
NobodyCam | awesome :) | 15:49 |
lucasagomes | but yeah, it sucks to not be able to add dependecy on multiple patchs in gerrit :( | 15:51 |
lucasagomes | only a linear dependency | 15:51 |
NobodyCam | yes | 15:52 |
NobodyCam | I think a rebase on the volume driver will fix the issue I am having with the update (pull patches) script I've almost got working | 15:53 |
*** aignatov is now known as aignatov_ | 15:54 | |
lucasagomes | NobodyCam, ack, yea I will rebase it | 15:59 |
devananda | morning, all | 15:59 |
NobodyCam | good morning devananda :) | 15:59 |
*** lazy_prince has joined #openstack-ironic | 15:59 | |
lucasagomes | morning devananda | 16:00 |
*** loki__ has joined #openstack-ironic | 16:01 | |
devananda | looks like jenkins is happy again? | 16:01 |
lucasagomes | yeah I think it's grand now | 16:02 |
NobodyCam | ya the -1's look real now | 16:03 |
devananda | awesome | 16:04 |
NobodyCam | any one happen to know off the top of their head if line #9 of https://review.openstack.org/#/c/73054/4/ironic/tests/drivers/pxe_config_disk.template is valid for linux | 16:06 |
lazy_prince | NobodyCam: yes. for it to work we need to have chain.c32 copied to tftp directory.. | 16:08 |
lazy_prince | And we have it tested for linux disk images as well.. | 16:09 |
NobodyCam | :) that requirment should be commented on the reviews commit message | 16:10 |
lucasagomes | lazy_prince, awesome | 16:10 |
NobodyCam | yes! awesome to see :) | 16:10 |
*** dshulyak has joined #openstack-ironic | 16:14 | |
* NobodyCam makes mo coffee | 16:14 | |
devananda | so, we doing a review jam again today? | 16:14 |
NobodyCam | I think max_lobur is afk atm! | 16:19 |
max_lobur | back | 16:21 |
max_lobur | morning / afternoon Everyone | 16:21 |
NobodyCam | morning max_lobur :) | 16:21 |
max_lobur | are we started already? | 16:22 |
*** creiht has left #openstack-ironic | 16:22 | |
devananda | i dont think so ... | 16:23 |
devananda | i've been catching up on some of the recently posted reviews | 16:23 |
* devananda just -2'd the py3compat reviews | 16:23 | |
devananda | etherpad: https://etherpad.openstack.org/p/IronicReviewDay | 16:24 |
NobodyCam | I was looking at: https://review.openstack.org/#/c/71974 | 16:24 |
devananda | oh ya - i was going to update that | 16:24 |
devananda | NobodyCam: marking that as WIP for now | 16:25 |
max_lobur | there's unfinished discussion in those patch I think | 16:25 |
devananda | NobodyCam: max_lobur's changes that landed will conflict with that. it needs to be reworked anyway | 16:25 |
devananda | https://review.openstack.org/#/c/73279/3 and https://review.openstack.org/#/c/73284/ | 16:26 |
devananda | look fairly easy | 16:26 |
* max_lobur looking | 16:27 | |
NobodyCam | ok... need to step away for a minute... brb | 16:27 |
devananda | as do i ... bbiam | 16:27 |
max_lobur | +2d https://review.openstack.org/#/c/73279/ (Add wsme custom BooleanType type) | 16:31 |
lucasagomes | btw NobodyCam, I know it's not ideal, but what about make #72007 (refactor iscsi_initiator) depend on the #51328 (Ironic driver) | 16:32 |
lucasagomes | to have a linear dependency | 16:32 |
lucasagomes | host_manager -> ironic driver -> iscsi refactor -> volume driver | 16:32 |
lucasagomes | otherwise the volume driver can't be reviewed until the iscsi refactor or the ironic driver gets merged | 16:33 |
devananda | lucasagomes: does the order make any sense if 'ironic driver' is the last one? | 16:33 |
devananda | host_mgr -> iscsi -> volume -> driver ? | 16:33 |
NobodyCam | humm | 16:34 |
NobodyCam | vol needs driver | 16:34 |
lucasagomes | devananda, hmm it could be refactored like that, but the problem would remain | 16:34 |
lucasagomes | iscsi doesnt really depend on host_mgr | 16:34 |
lucasagomes | we are adding as a dep just to have this linear dependency (cause multi dependency is not possible in gerrit afaik) | 16:35 |
devananda | right | 16:35 |
devananda | so put iscsi refactor as the root then | 16:35 |
*** jistr has quit IRC | 16:35 | |
lucasagomes | ack | 16:35 |
devananda | iscsi -> host_mgr -> driver -> volume | 16:35 |
lucasagomes | yea it would make it easier to get merged at least | 16:35 |
NobodyCam | :) that would work | 16:36 |
lucasagomes | devananda, cheers, will do that | 16:36 |
*** bearhands is now known as comstud | 16:36 | |
NobodyCam | lucasagomes: awesome Thank you :) | 16:36 |
lucasagomes | NobodyCam, heh thank YOU, you did the hard work | 16:37 |
NobodyCam | :-p | 16:38 |
devananda | max_lobur: https://review.openstack.org/#/c/73284/ also +2'd | 16:38 |
devananda | I tested both locally, at least the "associated" part, since that is what client supports right now | 16:39 |
devananda | works fine | 16:39 |
devananda | we'll need a patch to the client to add --maintenance [true|false] | 16:39 |
devananda | here's an important patch chain we should review | 16:40 |
devananda | https://review.openstack.org/#/c/71297/ -> https://review.openstack.org/#/c/72395/ | 16:40 |
NobodyCam | max_lobur: I'll +a 73284 | 16:44 |
max_lobur | NobodyCam: yep, ty! | 16:45 |
devananda | NobodyCam: https://review.openstack.org/#/c/71879/3 could use your eyes | 16:47 |
*** anniec has quit IRC | 16:47 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: API: Expose a way to start/stop the console https://review.openstack.org/72998 | 16:52 |
*** lazy_prince has quit IRC | 16:55 | |
*** vkozhukalov has quit IRC | 17:00 | |
NobodyCam | https://review.openstack.org/#/c/73223 | 17:01 |
*** digambar has joined #openstack-ironic | 17:02 | |
openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: PXE clean_up() to remove the pxe_deploy_key parameter https://review.openstack.org/71879 | 17:02 |
devananda | lucasagomes: updated the unit test on you rpatch ^ | 17:02 |
lucasagomes | devananda, ack thanks | 17:02 |
openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: PXE clean_up() to remove the pxe_deploy_key parameter https://review.openstack.org/71879 | 17:03 |
*** Alexei_987 has quit IRC | 17:05 | |
NobodyCam | https://review.openstack.org/#/c/70348/ | 17:07 |
openstackgerrit | A change was merged to openstack/ironic: Add wsme custom BooleanType type https://review.openstack.org/73279 | 17:08 |
devananda | max_lobur: let's talk about your comment on https://review.openstack.org/#/c/68018/5/ironic/api/controllers/v1/node.py | 17:08 |
*** martyntaylor has left #openstack-ironic | 17:11 | |
*** lazy_prince has joined #openstack-ironic | 17:11 | |
digambar | Hello | 17:14 |
digambar | getting forbidden error when I run any cli command in ironic | 17:14 |
digambar | Before re-installing devstack, it is working well | 17:15 |
digambar | after re-installation, it is giving error every CLI command | 17:15 |
max_lobur | digambar: hi! | 17:15 |
openstackgerrit | A change was merged to openstack/ironic: Use the custom wsme BooleanType on the nodes api https://review.openstack.org/73284 | 17:15 |
digambar | hi | 17:15 |
max_lobur | can you post error? | 17:16 |
NobodyCam | digambar: check the keystone creds in nova conf | 17:16 |
max_lobur | https://review.openstack.org/#/c/71297/Â -> https://review.openstack.org/#/c/72395/ | 17:16 |
max_lobur | folks | 17:16 |
openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: API validates driver name for both POST and PATCH https://review.openstack.org/68018 | 17:16 |
max_lobur | 1st one is merging now | 17:16 |
digambar | all nova cli working well | 17:17 |
digambar | it is failing @ only keystone side | 17:17 |
max_lobur | the second one - I found one uncovered pathway | 17:17 |
max_lobur | yuriyz: ping | 17:17 |
NobodyCam | digambar: there should be an ironic block in nova.conf with keystone u/p | 17:17 |
digambar | ironic node-create -d fake_ssh -i ssh_address=192.168.122.1 -i ssh_username=root -i ssh_password=root123 -i ssh_virt_type=virsh | 17:18 |
digambar | HTTPForbidden (HTTP 403) | 17:18 |
digambar | ok | 17:18 |
digambar | let me check | 17:18 |
devananda | digambar, NobodyCam - that failure is not in the nova.conf | 17:18 |
digambar | okay | 17:18 |
digambar | before this I am not getting this error, after installing new devstack this error is coming | 17:19 |
devananda | digambar: if python-ironicclient is geting HTTPForbidden talkign to ironic API, it is auth error. what privileges does your OS_USERNAME accout have? | 17:19 |
digambar | oops | 17:19 |
devananda | digambar: "installing new devstack" is not very specific. what changed? | 17:19 |
digambar | let me check | 17:19 |
digambar | ok | 17:19 |
digambar | hey sorry | 17:20 |
digambar | I am trying it using demo user | 17:20 |
devananda | yep | 17:20 |
digambar | got it working using admin user | 17:21 |
devananda | :) | 17:21 |
digambar | Thanks :) | 17:21 |
NobodyCam | :) | 17:21 |
max_lobur | :) | 17:21 |
NobodyCam | https://review.openstack.org/#/c/71212 | 17:26 |
*** mdurnosvistov_ has joined #openstack-ironic | 17:26 | |
*** loki__ has quit IRC | 17:28 | |
digambar | https://bugs.launchpad.net/ironic/+bug/1276393 | 17:28 |
digambar | working on this bug | 17:28 |
digambar | updating port-update required | 17:28 |
digambar | ironic port-update 94809fcb-afbd-4116-80cc-20f24b6fbf4a replace | 17:29 |
digambar | for last argument | 17:29 |
digambar | path=?? | 17:29 |
NobodyCam | mac=aa:bb... | 17:30 |
digambar | ok | 17:31 |
NobodyCam | oh wait i think its address= | 17:31 |
*** dshulyak has quit IRC | 17:31 | |
lucasagomes | NobodyCam, you can hear me? | 17:31 |
devananda | lucasagomes: https://review.openstack.org/#/c/73642/ | 17:31 |
NobodyCam | nope | 17:32 |
lucasagomes | devananda, feck lemme try | 17:32 |
digambar | let me paste here whole command | 17:32 |
lucasagomes | devananda, ok lemme rejoin the hangout | 17:32 |
devananda | digambar: please use pastebin for large pastes | 17:32 |
digambar | ok | 17:32 |
lucasagomes | devananda, yea I'm not mutted it just doesnt work | 17:33 |
lucasagomes | devananda, so | 17:33 |
digambar | http://paste.openstack.org/show/66554/ | 17:33 |
lucasagomes | devananda, that patch collide with another patch that ruby posted | 17:33 |
lucasagomes | devananda, so i abandoned that, and I'm comment at ruby's | 17:33 |
NobodyCam | digambar: my bad use address= | 17:34 |
* lucasagomes tries to fix his mic | 17:34 | |
digambar | okk | 17:34 |
digambar | :) | 17:35 |
NobodyCam | :) | 17:35 |
*** matty_dubs is now known as matty_dubs|lunch | 17:39 | |
*** ndipanov is now known as ndipanov_gone | 17:48 | |
NobodyCam | https://review.openstack.org/#/c/68980 | 17:49 |
digambar | http://paste.openstack.org/show/66566/ | 17:51 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Fix dev-quick-start.rst post landing for 9d81333fd0 https://review.openstack.org/73642 | 17:52 |
NobodyCam | digambar: is it locking now | 17:53 |
digambar | no | 17:54 |
*** derekh has quit IRC | 17:54 | |
*** lazy_prince has quit IRC | 17:55 | |
digambar | before this, if I run port-update, it will give nodelocked error | 17:55 |
digambar | now it is showing the output | 17:55 |
NobodyCam | digambar: yep the bug is so it should be putting a lock on the node for that type of change | 17:59 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Task object as paramater to validate() methods https://review.openstack.org/68980 | 17:59 |
digambar | ok | 18:00 |
digambar | so in that case, when anybody fires port-update, then it should put the lock on the node | 18:01 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: PXE and SSH validate() method to check for a port https://review.openstack.org/68985 | 18:01 |
NobodyCam | yep | 18:01 |
devananda | max_lobur: https://review.openstack.org/#/c/68413/ commented. LGTM aside from commit message | 18:02 |
devananda | lucasagomes: https://review.openstack.org/#/c/68018/ could use your eyes | 18:03 |
max_lobur | https://review.openstack.org/#/c/74063/ | 18:09 |
NobodyCam | https://review.openstack.org/#/c/71585 | 18:12 |
*** athomas has quit IRC | 18:12 | |
*** matty_dubs|lunch is now known as matty_dubs | 18:14 | |
openstackgerrit | A change was merged to openstack/ironic: Allow to tear-down a node waiting to be deployed https://review.openstack.org/71297 | 18:14 |
max_lobur | https://review.openstack.org/#/c/73256/ | 18:17 |
openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Correct assertEqual order from patch e69e41c99fb https://review.openstack.org/74114 | 18:26 |
*** tatyana has quit IRC | 18:26 | |
openstackgerrit | Max Lobur proposed a change to openstack/ironic: Add JSONEncodedType with enforced type checking https://review.openstack.org/68413 | 18:27 |
NobodyCam | bbiab | 18:28 |
max_lobur | folks ^^ easy one | 18:28 |
*** lucasagomes is now known as lucas-afk | 18:32 | |
max_lobur | brb | 18:33 |
openstackgerrit | A change was merged to openstack/ironic: Add common.service config options to sample https://review.openstack.org/73488 | 18:40 |
max_lobur | back | 18:40 |
*** vkozhukalov has joined #openstack-ironic | 18:44 | |
*** digambar has quit IRC | 18:46 | |
*** k4n0 has joined #openstack-ironic | 18:48 | |
*** aignatov_ is now known as aignatov | 18:48 | |
openstackgerrit | A change was merged to openstack/ironic: API validates driver name for both POST and PATCH https://review.openstack.org/68018 | 18:51 |
*** agordeev2 has joined #openstack-ironic | 18:52 | |
*** openstackgerrit has quit IRC | 18:53 | |
*** openstackgerrit has joined #openstack-ironic | 18:54 | |
devananda | back | 18:55 |
devananda | max_lobur: +2'd | 18:56 |
max_lobur | thx | 18:58 |
*** lucas-afk is now known as lucasagomes | 18:58 | |
devananda | heads up -- meeting in a minute | 18:58 |
mrda | morning all! | 19:01 |
devananda | mrda: g'morning! | 19:02 |
devananda | mrda: we're all in the meeting :) | 19:02 |
openstackgerrit | A change was merged to openstack/ironic: Task object as paramater to validate() methods https://review.openstack.org/68980 | 19:02 |
openstackgerrit | A change was merged to openstack/ironic: PXE and SSH validate() method to check for a port https://review.openstack.org/68985 | 19:03 |
mrda | devananda: yes, that's why I'm here at 5:30am :) | 19:03 |
max_lobur | :) | 19:03 |
*** aignatov is now known as aignatov_ | 19:07 | |
*** jcooley_ has joined #openstack-ironic | 19:15 | |
openstackgerrit | A change was merged to openstack/ironic: Correct assertEqual order from patch e69e41c99fb https://review.openstack.org/74114 | 19:30 |
lucasagomes | NobodyCam, devananda max_lobur when you guys have some time https://review.openstack.org/#/c/73642/ | 19:34 |
max_lobur | ack | 19:35 |
*** aignatov_ is now known as aignatov | 19:38 | |
*** anniec has joined #openstack-ironic | 19:41 | |
*** aignatov is now known as aignatov_ | 19:42 | |
* NobodyCam needs to do a quick walkies .. brb | 20:00 | |
k4n0 | devananda: So the drivers will be exposed via API? | 20:00 |
devananda | k4n0: they are: http://docs.openstack.org/developer/ironic/webapi/v1.html#drivers | 20:01 |
devananda | today it's read only | 20:01 |
lucasagomes | ok folks I'm also going take a break/eat something (/me not feeling very well today) | 20:01 |
devananda | lucasagomes: ack. hope you feel better :) | 20:01 |
lucasagomes | devananda, thanks! | 20:01 |
lucasagomes | g'night! | 20:01 |
NobodyCam | night lucasagomes | 20:01 |
lucasagomes | NobodyCam, night buddy! | 20:02 |
* lucasagomes will leave it open to read the scrollback laters | 20:02 | |
*** lucasagomes is now known as lucas-afk | 20:02 | |
k4n0 | devananda: Any other reason that the chassis object cant be made to do discovery? | 20:02 |
devananda | k4n0: there's been discussion with the Tuskar folks about doing different kinds of node grouping | 20:02 |
devananda | eg, modelling of multiple failure domains | 20:03 |
devananda | rather than just grouping by chassis | 20:03 |
k4n0 | devananda: I see, that looks helpful, I will look into the /drivers api. I assume the /drivers/seamicro will still hit the vendor passthru? | 20:04 |
devananda | k4n0: probably not | 20:04 |
devananda | k4n0: or rather -- i dont know :) | 20:05 |
devananda | VendorPassthru interface requires a Node and a Task, like all the current driver interfaces | 20:05 |
devananda | i haven't looked at how we might pass a request down to a driver without having either of those | 20:06 |
devananda | what happens if, eg, two API requests are received at /drivers/seamicro/discover at the same time? | 20:06 |
devananda | what schedules them or handles conflicts? how do we determine which ConductorManager to route the request(s) to? or do we broadcast it to all? | 20:07 |
k4n0 | devananda: valid concerns | 20:07 |
devananda | k4n0: so i think it's a great topic for the summit | 20:08 |
k4n0 | devananda: let me think along those lines. | 20:08 |
k4n0 | devananda: isnt the last date for submitting topics 14th Feb :) | 20:08 |
devananda | k4n0: for talks, yes. for design sessions -- no :) | 20:08 |
devananda | when i say "topic" i mean for us all to discuss in the design summit | 20:09 |
k4n0 | devananda: ohh, sure, we should discuss this | 20:09 |
k4n0 | devananda: alright , lets keep this topic alive in the mailing lists till then, lets see if something can be done | 20:10 |
devananda | k4n0: sounds good | 20:10 |
k4n0 | devananda: thanks for your time :) , cya soon | 20:10 |
* NobodyCam is back | 20:17 | |
max_lobur | devananda, NobodyCam https://review.openstack.org/#/c/74063/ take a look when you have time | 20:22 |
max_lobur | there's opened question - do we need to guarantee that there should not be intrusion between vendor.validate and vendor.vendor_passthru | 20:23 |
*** mdurnosvistov_ has quit IRC | 20:23 | |
*** jcooley_ has quit IRC | 20:25 | |
max_lobur | k, I'm done for today. see you tomorrow! | 20:32 |
max_lobur | g'night | 20:32 |
*** max_lobur has quit IRC | 20:34 | |
devananda | g'night! | 20:34 |
*** k4n0 has left #openstack-ironic | 20:38 | |
*** agordeev2 has quit IRC | 20:42 | |
*** tatyana has joined #openstack-ironic | 20:42 | |
*** ifarkas has quit IRC | 20:44 | |
*** aignatov_ is now known as aignatov | 20:48 | |
lifeless | hey so | 21:06 |
lifeless | windows support | 21:06 |
lifeless | I think that patch conflates a bunch of stuff | 21:06 |
lifeless | the blueprint too | 21:06 |
lifeless | whole disk is something that I think many folk may want (ill advised or not :P) | 21:06 |
lifeless | like - if you have that you can't use a flavor with ephemeral | 21:07 |
devananda | so, yes, it borks ephemeral support | 21:07 |
lifeless | devananda: how would you feel if we add an actual etherpad spec | 21:07 |
devananda | but for some deployments, that may be OK. I think it's a valid option (and you seem to agree) | 21:07 |
lifeless | where we can do design | 21:07 |
lifeless | and get a set of changes | 21:07 |
lifeless | and then point the author of that patch at it | 21:08 |
devananda | and in looking at the patch now (which i hadn't really done before) i agree, it's conflating windows boot with whole-disk deploy | 21:08 |
lifeless | ok, I'll add a etherpad, sec | 21:08 |
openstackgerrit | A change was merged to openstack/ironic: Fix dev-quick-start.rst post landing for 9d81333fd0 https://review.openstack.org/73642 | 21:09 |
lifeless | also I thought primeministerp said windows had a PXE mode? | 21:09 |
lifeless | devananda: https://etherpad.openstack.org/p/icehouse-ironic-windows-support | 21:10 |
devananda | lifeless: this patch is based on primeministerp's work | 21:11 |
lifeless | ah | 21:11 |
devananda | lifeless: his PXE mode is https://review.openstack.org/#/c/73054/4/ironic/drivers/modules/pxe_config_disk.template | 21:11 |
devananda | COM32 chain.c32 APPEND hd0 | 21:11 |
devananda | AIUI, that ties us to whole-disk image here | 21:12 |
lifeless | right thats not pxe booting, thats local booting | 21:12 |
lifeless | with a pxe pointer | 21:12 |
devananda | right | 21:12 |
lifeless | is he around ? | 21:12 |
devananda | which is what windows supports | 21:12 |
devananda | AIUI, we can't feed a kernel & ramdisk to windows over pXE | 21:12 |
devananda | PXE points windows at the local boot sector | 21:13 |
devananda | or more accurately, at some boot sector on a mounted media | 21:13 |
lifeless | linux chains into the local boot sector | 21:13 |
lifeless | devananda: btw I'm writing in https://etherpad.openstack.org/p/icehouse-ironic-windows-support | 21:15 |
*** mdurnosvistov_lt has joined #openstack-ironic | 21:20 | |
lifeless | ok, I think I've got enough laid out in my head to review the patch now | 21:23 |
devananda | ack | 21:26 |
devananda | good stuff in the etherpad. i have a concern about auto-detecting boot from localdisk, though | 21:26 |
lifeless | go on ? | 21:27 |
devananda | lifeless: L24 | 21:27 |
openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Removing unused get_node_power_state https://review.openstack.org/73097 | 21:27 |
lifeless | devananda: answered there | 21:28 |
lifeless | devananda: there is a race condition with neutron updating dnsmasq to care for | 21:28 |
lifeless | devananda: OTOH if a node power cycles faster than neutron updates dnsmasq.... | 21:28 |
lifeless | thats a problem for *everyone* | 21:28 |
devananda | lifeless: we can prevent booting from local disk easily enough | 21:29 |
devananda | just by changing boot order via IPMI | 21:29 |
lifeless | devananda: we'd want that optional for TripleO | 21:29 |
devananda | right | 21:29 |
lifeless | devananda: has anyone asked 'please don't fall back to local disk if local disk is bootable' ? | 21:29 |
devananda | and some folks may consider it a security issue to allow nodes to boot from local disk before the network comes online (eg, in a power cycle scenario) | 21:30 |
devananda | IIRC, you did :) | 21:30 |
devananda | or more specifically | 21:30 |
devananda | you asked tjat local disk not be bootable on some nodes | 21:30 |
lifeless | yes | 21:30 |
lifeless | which we get today by not having boot blocks | 21:31 |
lifeless | uhm | 21:31 |
lifeless | so IIRC what I was getting at | 21:31 |
devananda | ah | 21:31 |
*** derekh has joined #openstack-ironic | 21:31 | |
lifeless | is that running the users stuff in a secure context is a bit odd | 21:31 |
lifeless | and that could happen if they reboot locally | 21:31 |
lifeless | but | 21:31 |
lifeless | we don't have any secure context today | 21:31 |
devananda | heh, fair | 21:31 |
lifeless | and, if we do get one, I'd expect it to be turned on in series with PXE config in Neutron etc. | 21:31 |
devananda | so, effectively, linux = if k&r in glance, set netboot, else set localboot. windows = no k&r in glance, always set localboot | 21:32 |
NobodyCam | the node is stil pxe booting to get the local boot flag | 21:35 |
NobodyCam | just no k&r being servied | 21:36 |
openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Removing unused get_node_power_state https://review.openstack.org/73097 | 21:36 |
NobodyCam | if no K & R attached to image in glance then assume local boot? | 21:36 |
devananda | NobodyCam: that's what lifeless is proposing -- if no k&r in glance, then have the deploy driver actually set boot device to local disk | 21:37 |
devananda | NobodyCam: rather than have the node needlessly PXE boot just to be told to chain hd0 | 21:38 |
devananda | that's where my security concern comes in | 21:38 |
lifeless | devananda: actually no | 21:38 |
devananda | erm | 21:38 |
lifeless | devananda: I was not proposing changing the node's IPMI boot order | 21:38 |
lifeless | devananda: I was proposing just deleting the PXE entries from Neutron | 21:38 |
lifeless | devananda: but - changing the IPMI boot order might be better | 21:39 |
devananda | lifeless: if we set next-boot to net (and remove local boot) then deleting the neutron option would prevent boot. if we didn't remove lcoal boot, it'd just make the boot take longer. | 21:39 |
devananda | being explicit here is best, IMO | 21:40 |
devananda | in each case, what do we want the node to boot from? | 21:40 |
lifeless | devananda: I think we should *also* change Neutron though | 21:40 |
lifeless | devananda: for clarity | 21:40 |
devananda | sure :) | 21:40 |
devananda | if we're setting it to localboot, we should definitely unset the neutron boot option | 21:41 |
lifeless | ok, I think we're agreed on that aspect | 21:41 |
NobodyCam | and then reset up for maintanince and firmware updates? | 21:41 |
lifeless | you've got a security concern ? | 21:41 |
lifeless | NobodyCam: yes, we do anyway for pxe - we edit an option on disk, not conceptually any different | 21:42 |
NobodyCam | :) | 21:42 |
devananda | lifeless: if we infer localboot from the lack of k&r, then we lose the programatic control of where a node boots from | 21:42 |
devananda | lifeless: eg, windows image won't hve k&r, so all windows instances will localboot, even if they're not essential tripleo infrastructure nodes | 21:43 |
devananda | s/we lose/the operator loses/ | 21:43 |
lifeless | devananda: We don't currently expose programmatic control of where to boot from | 21:44 |
lifeless | so I'm not sure I follow | 21:44 |
*** coolsvap has quit IRC | 21:44 | |
NobodyCam | devananda: dont all windows images HAVE to local boot | 21:44 |
lifeless | devananda: FWIW I believe the VM images in Nova work this way (IMBW) | 21:44 |
devananda | lifeless: that's because everything boots from network today | 21:44 |
lifeless | devananda: I know :) | 21:44 |
devananda | NobodyCam: the NODE can still net boot, even if we chain to local disk. | 21:45 |
devananda | NobodyCam: that would prevent eg. the node from booting before neutron is online | 21:45 |
*** tatyana has quit IRC | 21:45 | |
lifeless | only if we explicitly disable localboot | 21:46 |
devananda | lifeless: right | 21:46 |
NobodyCam | so pxe'ing the chain command.. | 21:46 |
lifeless | because pxe will time out | 21:46 |
openstackgerrit | A change was merged to openstack/ironic: Add JSONEncodedType with enforced type checking https://review.openstack.org/68413 | 21:46 |
lifeless | and we don't explicily disable localboot today | 21:46 |
lifeless | at least in nova-bm | 21:46 |
devananda | sure | 21:46 |
devananda | ok | 21:46 |
devananda | let's step back -- its geting confusing with the "we do today" vs "we want" vs ... | 21:46 |
devananda | lifeless: AIUI, you want the ability for TripleO to determine which nodes boot from local disk,a nd which nodes will fail to boot at all, if they come up before the control plane | 21:47 |
devananda | yes? | 21:47 |
lifeless | maybe | 21:47 |
lifeless | what I want is the ability to pull the plug on a DC | 21:47 |
devananda | :-/ | 21:47 |
lifeless | and put it back in | 21:48 |
lifeless | and have everything work | 21:48 |
devananda | hah | 21:48 |
lifeless | without complex/fragile manual steps | 21:48 |
lifeless | everything else is tactics :) | 21:48 |
lifeless | so | 21:48 |
lifeless | perhaps it will help if I layout first principles for our mutual memory refresh | 21:48 |
lifeless | in a tripleo DC we have a large number of machines | 21:49 |
lifeless | which are 'tenants' | 21:49 |
lifeless | nova will try to restore their state when nova starts up. This is probably a bug, but we'll come back to that :P | 21:49 |
lifeless | to boot properly they depend on DHCP - Neutron - and nova-api (metadata) and heat (more metadata) from the undercloud | 21:50 |
lifeless | then we have the undercloud | 21:50 |
lifeless | the undercloud is some number of machines scale to the DC size | 21:50 |
lifeless | where the machines themselves are also tenants of the undercloud | 21:50 |
lifeless | so to boot properly they need DHCP, nova-API, heat running | 21:50 |
lifeless | DHCP not for PXE - DHCP for 'whats my IP within Linux' | 21:51 |
*** vkozhukalov has quit IRC | 21:53 | |
lifeless | to bring dhcp up we need neutron up which needs DB and rabbit up | 21:53 |
lifeless | to bring DB up we need IP address + hostname->IP for other galera cluster members | 21:53 |
lifeless | likewise for rabbit | 21:53 |
lifeless | to bring nova-api we need db + rabbit | 21:53 |
lifeless | ditto heat | 21:54 |
lifeless | we also need the previously running metadata config to be cached on the machines | 21:54 |
lifeless | so - we basically need to freeze enough data to make that whole set of dependencies, within each undercloud machine survive booting with no other machines running | 21:54 |
*** jdob has quit IRC | 21:54 | |
*** mdurnosvistov_lt has quit IRC | 21:54 | |
lifeless | and have the machines come up and self assemble, hit quorum, and become operational | 21:55 |
lifeless | then we can start asserting 'nova start' or 'nova reboot' or whatever all the other tenants | 21:55 |
*** mdurnosvistov_lt has joined #openstack-ironic | 21:55 | |
lifeless | we don't need log servers, swift backends etc to come up | 21:55 |
NobodyCam | lifeless: cache as NOT in the db. but on local media(coould be local db) | 21:56 |
lifeless | NobodyCam: something persistent locally yes | 21:57 |
lifeless | so, most machines in that setup can't boot locally at all. | 21:57 |
devananda | lifeless: and I'd imagine you want a way to assert that those crucial undercloud machines boot from local disk first, and that all non-crucial machines (particularly overcloud instances) don't start coming online until they are expected to | 21:57 |
lifeless | which ones can? The ones we've added local boot blocks for I guess - today. | 21:57 |
lifeless | devananda: maybe; will answer in detail in a sec | 21:58 |
lifeless | if we scale the undercloud down | 21:58 |
lifeless | we would leave behind a machine with a local boot block | 21:58 |
lifeless | that we wanted to boot locally in the past | 21:58 |
lifeless | but that isn't part of the cluster anymore | 21:58 |
lifeless | and *that* shouldn't be allowed to boot | 21:59 |
lifeless | OTOH | 21:59 |
lifeless | we would want scale down to wipe its disk to delete passwords etc too | 21:59 |
lifeless | which would quite naturally stop it booting ;) | 21:59 |
lifeless | devananda: so your possible assertions above | 22:01 |
lifeless | a) boot the minimal closure from local disk first | 22:01 |
lifeless | b) boot the rest only when desired | 22:01 |
lifeless | for a) no - I guess so. If we don't boot them locally-only, we wouldn't know that a power off situation would work. | 22:02 |
lifeless | OTOH | 22:02 |
lifeless | we won't know that it will work unless we boot them with the network cable pulled out and DHCP etc not answering. | 22:02 |
lifeless | so I'm not convinced. I kindof thing DR testing should be explicit - like - ok, we're turning off the undercloud entirely and back on again now. Lets see what happens. | 22:02 |
lifeless | s/thing/think/ | 22:02 |
lifeless | for b) if we set the action on power reconnection to 'do nothing' for most machines, we'd get that | 22:03 |
lifeless | that would be a good thing for most DC's anyway - they usually can't handle an all-at-once-power-on event. | 22:03 |
lifeless | devananda: ok, fin | 22:03 |
devananda | lifeless: ah. so you'd prefer "chassis policy off" for those machines | 22:06 |
lifeless | devananda: that seems like an appropriate thing | 22:07 |
devananda | indeed | 22:07 |
lifeless | devananda: we should probably set that for all machines in fact, and have 'I want a local boot with no infrastructure' special case to do *differently* | 22:07 |
devananda | it separates the "where should it boot from" from "what should happen on DC power on" concerns | 22:07 |
devananda | yep | 22:07 |
lifeless | btw | 22:08 |
lifeless | will the nova driver also run a hash ring ? | 22:08 |
lifeless | or how will two nova compute instances interact with Ironic? | 22:08 |
lifeless | TOTAL CHANGE OF TOPIC :) | 22:08 |
*** linggao has quit IRC | 22:08 | |
*** aignatov is now known as aignatov_ | 22:10 | |
devananda | heh | 22:11 |
devananda | the ironic api -> rpc pipe will aggregate requests for the same node | 22:12 |
lifeless | I mean | 22:13 |
lifeless | something lists all the nodes in the sceheduler right? | 22:13 |
lifeless | is that Ironic or nova-compute? | 22:13 |
devananda | so if multiple nova-compute instances are asked to operate on a single node, some will encounter a NodealreadyLocked error | 22:13 |
devananda | ah | 22:13 |
devananda | you mean | 22:13 |
lifeless | do all nova computes list all ironic nodes in the scheduler | 22:14 |
devananda | which nova-compute will advertise which ironic nodes to the scheduler | 22:14 |
devananda | right | 22:14 |
lifeless | so if we have 10K nodes and 50 conductors, nova-compute dies? | 22:14 |
lifeless | because we have say 3 nova-computes for redundancy, but that lists 30K nodes in the secheduler ? | 22:14 |
devananda | what's the rough limit on a cell size today -- O(100) nova-compute hosts, right? | 22:15 |
lifeless | pick a number | 22:15 |
lifeless | its going to be wrong, but whatever. C for Cell. | 22:15 |
lifeless | C hosts. | 22:15 |
devananda | k | 22:15 |
devananda | so we have C compute hosts | 22:15 |
devananda | do you have any expectation of the ration of C to I, where I is # of ironic nodes? | 22:16 |
devananda | rephrase | 22:16 |
devananda | if the scheduler today can handle only C hosts, do we expect it to handle more than C nodes ? | 22:17 |
devananda | from teh scheduler's POV, each ironic node is a compute host | 22:17 |
devananda | your question of which nova-compute is advertising which ironic node is very interesting (and i suspect we haven't addressed that) but separate to the scheduler-falls-over-when-C-is-large issue | 22:18 |
lifeless | so, I was pulling on a thread | 22:18 |
devananda | as you do :) | 22:18 |
lifeless | I don't have a specific X is a problem point | 22:18 |
lifeless | uhm | 22:19 |
lifeless | I expect in a BM cloud that nova is basically a proxy | 22:19 |
devananda | (it's a good thing) | 22:19 |
devananda | right | 22:19 |
lifeless | translating queries to Ironic and back | 22:19 |
devananda | yep | 22:19 |
lifeless | appying some business logic etc. | 22:19 |
lifeless | so | 22:19 |
lifeless | small # of nova-compute | 22:19 |
lifeless | I suspect I'd be happy to try 10K nodes with 3 nova computes. | 22:19 |
devananda | everything could be done directly -- and I suspect we may want the scheduler to talk directly to ironic to avoid some of this, but we haven't done that yet | 22:19 |
lifeless | (so that a failure or deploy doesn't degrade to one node) | 22:19 |
devananda | ++ | 22:20 |
lifeless | I'd be a little wary of direct scheduler stuff for now; possibly when gantt is more mature | 22:20 |
devananda | right | 22:20 |
devananda | +eventually | 22:20 |
lifeless | scheduler fallover - I think I'm more worried by two-at-once races (even with locking - it can get hairy) | 22:20 |
devananda | so layer violation -- everything in nova, AFAIK, expects each nova driver instance to present a unique view of resources it owns | 22:21 |
lifeless | but we need the hypervisor id's to be constant in the scheduler right ? | 22:21 |
devananda | not a shared view of communla resources | 22:21 |
lifeless | yes, this is an issue for vmware etc too AIUI | 22:21 |
devananda | do you now how theyr'e solving it? | 22:21 |
lifeless | NFI | 22:22 |
lifeless | how long does it take a node to time out of the scheduler ? | 22:23 |
lifeless | like | 22:23 |
lifeless | I'm wondering if a fairly simple 'register anything thats not registered' approach would work for now | 22:23 |
devananda | humm | 22:24 |
devananda | so, typicaly, a nova driver registers (host, node) tuple for every resource it owns | 22:25 |
devananda | eg, kvm registers ($hostname, '') | 22:25 |
devananda | since it has only one resource, the local machine | 22:25 |
devananda | nova-bm has a mapping of n-cpu<->node | 22:26 |
devananda | take over is a problem, etc... because nodes are registered to one n-cpu | 22:26 |
devananda | what if all ironic nova-compute hosts registered the same hostname | 22:26 |
devananda | eg, ("ironic", $node_uuid) | 22:26 |
devananda | it might bork the scheduler's ability to pick which n-cpu to send a request to, though | 22:27 |
devananda | or it might just all route to the first entry in a list ... i dont know .. | 22:27 |
devananda | NobodyCam: ^ | 22:28 |
devananda | lifeless: two summits ago, comstud proposed removing the first portion of that tuple | 22:29 |
devananda | lifeless: instead of (host, node) the mapping would just be (node) | 22:29 |
devananda | i dont think anyone worked on that since, but it would solve this problem for ironic, vmware, etc | 22:29 |
NobodyCam | ha I was thinking about how we expose nodes to nova this morning not for this reason thu | 22:30 |
NobodyCam | deva not to sside track but have you seen http://ipxe.org/wimboot | 22:32 |
devananda | NobodyCam: no, but again, there's another benefit of ipxe | 22:33 |
NobodyCam | :) | 22:33 |
devananda | NobodyCam: take a look at https://www.kernel.org/pub/linux/kernel/people/geoff/petitboot/petitboot.html | 22:34 |
* NobodyCam clicks | 22:34 | |
devananda | some great things in there. the kexec bit is questionable though | 22:35 |
* NobodyCam thinks after graduation ... After Graduation ... AFTER GRADUATION.... | 22:35 | |
devananda | heh | 22:35 |
* devananda goes back to testing devstack | 22:36 | |
*** mdurnosvistov_lt has quit IRC | 22:49 | |
*** derekh has quit IRC | 22:49 | |
lucas-afk | devananda, lifeless btw, there's a way to boot windows with a linux k&r... you could pxe boot a special linux k+r which contains kexec and grub4dos | 23:20 |
lucas-afk | and from there you can boot the local windows | 23:20 |
lucas-afk | so it would work just like we have for linux now | 23:21 |
lucas-afk | tho it looks a bit hacky :) | 23:21 |
*** matty_dubs is now known as matty_dubs|gone | 23:22 | |
lifeless | lucas-afk: thats basically chain loading though right ? | 23:24 |
lucas-afk | lifeless, yes, that's it pretty much | 23:25 |
lifeless | which ^ we don't need | 23:25 |
devananda | lifeless: any objection to changing http://git.openstack.org/cgit/openstack/tripleo-incubator/tree/scripts/setup-network so --persistent is hidden on ubuntu precise? | 23:36 |
devananda | which doesn't support it | 23:36 |
devananda | think that'll break anything? | 23:36 |
*** jcooley_ has joined #openstack-ironic | 23:37 | |
lifeless | precise won't work. | 23:37 |
lifeless | libvirt there doesn't support ovs networks at all. | 23:37 |
devananda | i see | 23:38 |
lifeless | you'll need the ubuntu cloud archive enabled and ovs + libvirt from that pulled in. | 23:38 |
lifeless | which will give you --persistent anyhow, I expect. | 23:38 |
devananda | so | 23:38 |
devananda | infra CI runs on precise | 23:38 |
devananda | that rules out using tripleo-incubator scripts to help prepare our CI environment | 23:39 |
devananda | i'm not clear on exactly which fucntionality of neutron (that i'm accustomed to using with tripleo) is going to be absent in a precise env | 23:40 |
lifeless | so we use ovs switches rather than linux bridge | 23:59 |
lifeless | because we got better L2 behaviour from them. | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!