Monday, 2014-02-17

« Sunday, 2014-02-16 Index Tuesday, 2014-02-18 »
*** jcooley_ has joined #openstack-ironic00:07
*** jcooley_ has quit IRC00:13
*** matsuhashi has joined #openstack-ironic00:23
*** jcooley_ has joined #openstack-ironic00:30
*** matsuhashi has quit IRC00:33
*** matsuhashi has joined #openstack-ironic00:34
*** jcooley_ has quit IRC00:37
*** matsuhashi has quit IRC00:38
*** matsuhashi has joined #openstack-ironic01:04
*** dkehn__ has joined #openstack-ironic01:04
*** dkehn has quit IRC01:05
*** nosnos has joined #openstack-ironic01:05
*** dkehn__ has quit IRC01:05
*** dkehn__ has joined #openstack-ironic01:05
*** dkehn_ has quit IRC01:06
*** dkehn has joined #openstack-ironic01:07
*** dkehn__ is now known as dkehn_01:09
*** jcooley_ has joined #openstack-ironic01:30
openstackgerritA change was merged to openstack/ironic: Use StringType from WSME  https://review.openstack.org/7324801:37
*** jcooley_ has quit IRC01:37
openstackgerritLin Tan proposed a change to openstack/ironic: Use six.StringIO/BytesIO instead of StringIO.StringIO  https://review.openstack.org/7389602:16
*** matsuhashi has quit IRC02:18
*** jcooley_ has joined #openstack-ironic02:26
*** matsuhashi has joined #openstack-ironic02:26
*** jcooley_ has quit IRC02:31
*** jcooley_ has joined #openstack-ironic03:01
*** dkehn__ has joined #openstack-ironic03:38
*** dkehn_ has quit IRC03:41
*** dkehn has quit IRC03:41
*** dkehn has joined #openstack-ironic03:42
*** matsuhashi has quit IRC03:48
*** jcooley_ has quit IRC03:53
*** dkehn__ is now known as dkehn_04:16
*** matsuhashi has joined #openstack-ironic04:32
*** Haomeng has quit IRC05:24
*** killer_prince has quit IRC05:35
openstackgerritLin Tan proposed a change to openstack/ironic: Use six.StringIO/BytesIO instead of StringIO.StringIO  https://review.openstack.org/7389605:36
*** killer_prince has joined #openstack-ironic05:39
openstackgerritJenkins proposed a change to openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/7119206:06
*** saju_m has joined #openstack-ironic06:15
*** nosnos has quit IRC06:42
*** nosnos has joined #openstack-ironic06:52
*** ifarkas has joined #openstack-ironic07:30
*** mdurnosvistov has joined #openstack-ironic07:37
*** Alexei_987 has joined #openstack-ironic07:38
*** ndipanov has joined #openstack-ironic07:48
openstackgerrityangxurong proposed a change to openstack/python-ironicclient: py3kcompat: remove in python-ceilometerclient  https://review.openstack.org/7395607:49
*** dshulyak has joined #openstack-ironic07:49
*** loki__ has joined #openstack-ironic08:18
*** yuriyz has joined #openstack-ironic08:25
*** mdurnosvistov has quit IRC08:25
*** saju_m has quit IRC08:31
openstackgerritAndrey Kurilin proposed a change to openstack/python-ironicclient: Sync cliutils from oslo  https://review.openstack.org/7241708:39
openstackgerritHaomeng,Wang proposed a change to openstack/ironic: sync oslo rpc to ironic  https://review.openstack.org/7397108:49
*** Haomeng has joined #openstack-ironic08:49
openstackgerritHaomeng,Wang proposed a change to openstack/ironic: sync oslo rpc to ironic  https://review.openstack.org/7397108:52
openstackgerritAndrey Kurilin proposed a change to openstack/python-ironicclient: Reuse cliutils from common code  https://review.openstack.org/7241808:56
*** saju_m has joined #openstack-ironic08:58
openstackgerritHaomeng,Wang proposed a change to openstack/ironic: sync oslo rpc to ironic  https://review.openstack.org/7397108:58
*** jistr has joined #openstack-ironic09:01
openstackgerritAndrey Kurilin proposed a change to openstack/python-ironicclient: Reuse module `cliutils` from common code  https://review.openstack.org/7241809:01
*** derekh has joined #openstack-ironic09:14
*** mdurnosvistov has joined #openstack-ironic09:17
*** lucasagomes has joined #openstack-ironic09:18
*** shausy has joined #openstack-ironic09:33
openstackgerritAndrey Kurilin proposed a change to openstack/python-ironicclient: Sync latest code and reuse exceptions from oslo  https://review.openstack.org/7150009:37
*** matsuhashi has quit IRC09:54
*** lazy_prince has joined #openstack-ironic09:57
*** killer_prince has quit IRC09:58
*** lazy_prince is now known as killer_prince09:58
openstackgerritAndrey Kurilin proposed a change to openstack/python-ironicclient: Sync cliutils from oslo  https://review.openstack.org/7241710:00
*** athomas has joined #openstack-ironic10:02
*** matsuhashi has joined #openstack-ironic10:02
*** aignatov is now known as aignatov_10:02
*** max_lobur has joined #openstack-ironic10:04
openstackgerritAndrey Kurilin proposed a change to openstack/python-ironicclient: Reuse module `cliutils` from common code  https://review.openstack.org/7241810:16
*** matsuhashi has quit IRC10:17
*** matsuhashi has joined #openstack-ironic10:17
GheRiverotox10:19
GheRiveroups... not here10:19
*** matsuhashi has quit IRC10:22
max_lobur:)10:25
max_loburmorning Ironic10:25
openstackgerritGhe Rivero proposed a change to openstack/ironic: Generic Multiplexed method  https://review.openstack.org/7332210:30
openstackgerritGhe Rivero proposed a change to openstack/ironic: Set boot device to PXE when deploying  https://review.openstack.org/7133210:30
openstackgerritGhe Rivero proposed a change to openstack/ironic: Move ipmitool _set_boot_device to VendorPassthru  https://review.openstack.org/7158510:30
openstackgerritGhe Rivero proposed a change to openstack/ironic: Move ipminative _set_boot_device to VendorPassthru  https://review.openstack.org/7249510:30
Haomengmorning GheRivero, max_lobur :)10:33
mdurnosvistovMorning all! :)10:36
*** aignatov_ is now known as aignatov10:37
*** martyntaylor has joined #openstack-ironic10:45
*** vkozhukalov has joined #openstack-ironic10:46
*** matsuhashi has joined #openstack-ironic10:47
vkozhukalovCan 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
Haomengvkozhukalov: extra is used for node's meta data10:58
Haomengvkozhukalov: that is not required fields for ironic, can be used for customer driver10:59
*** tatyana has joined #openstack-ironic11:00
max_loburvkozhukalov: hi!11:01
vkozhukalovHaomeng, thanks. And another question. What kind of metadata is that?11:01
max_loburtypically we track deploy progress through node states11:01
max_loburare you going to provide percentage ?11:02
max_loburvkozhukalov: metadata like11:04
max_lobur        "priority": "low",11:04
max_lobur        "role": "webserver"11:04
max_loburand so on11:04
vkozhukalovmax_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
vkozhukalovmax_lobur, thanks very much.11:06
max_loburhttps://github.com/openstack/ironic/blob/master/ironic/common/states.py11:07
max_loburI see11:07
max_loburwell, I think extra is suitable for this11:08
*** athomas has quit IRC11:34
*** matsuhashi has quit IRC11:37
*** aignatov is now known as aignatov_11:40
max_loburlucasagomes: r u around?11:41
*** athomas has joined #openstack-ironic11:44
*** shausy has quit IRC11:48
*** shausy has joined #openstack-ironic11:48
*** aignatov_ is now known as aignatov11:50
*** aignatov is now known as aignatov_11:51
*** aignatov_ is now known as aignatov11:57
*** killer_prince has quit IRC12:01
lucasagomesmax_lobur, howdy!12:02
openstackgerritLucas Alvares Gomes proposed a change to openstack/python-ironicclient: Add set-console-mode, get-console commands  https://review.openstack.org/7400712:07
max_loburhi lucasagomes :)12:08
max_loburmaybe you know12:08
max_loburhttps://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L25212:08
max_loburwhy we use shared lock for vendor passthru?12:09
max_loburlooks like an error, because we changing node state, not just read i12:09
openstackgerritLucas Alvares Gomes proposed a change to openstack/python-ironicclient: Add set-console-mode, get-console commands  https://review.openstack.org/7400712:09
max_lobur*if12:09
max_loburgah12:09
max_lobur* it :)12:09
*** loki__ has quit IRC12:10
lucasagomesmax_lobur, uu, yea looks like a potential bug :)12:10
lucasagomescontinue_deploy for e.g would change the node states as u said12:11
lucasagomesI bet it should be shared=False12:11
lucasagomesmax_lobur, good catch :)12:12
max_loburthx :)12:14
*** lucasagomes is now known as lucas-hungry12:32
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Sync Oslo service module to Ironic  https://review.openstack.org/7402512:55
*** dshulyak has quit IRC12:56
*** shausy has quit IRC13:04
*** shausy has joined #openstack-ironic13:05
Haomengyuriyz:ping13:07
*** nosnos has quit IRC13:08
*** jdob has joined #openstack-ironic13:41
*** lucas-hungry is now known as lucasagomes13:42
*** aignatov is now known as aignatov_13:43
agordeevHello. Could anyone tell me will the meeting happen today?13:45
max_loburagordeev: it will happen as usually13:46
agordeevmax_lobur: very good, thanks!13:47
max_loburnp :)13:47
*** aignatov_ is now known as aignatov14:03
*** saju_m has quit IRC14:15
*** jdob has quit IRC14:19
*** jdob has joined #openstack-ironic14:19
*** jdob_ has joined #openstack-ironic14:37
*** jdob has quit IRC14:37
*** blamar has joined #openstack-ironic14:38
*** jdob_ is now known as jdob14:38
*** blamar has quit IRC14:40
*** blamar has joined #openstack-ironic14:40
*** dkehn__ has joined #openstack-ironic14:41
*** dkehn has quit IRC14:43
*** dkehn_ has quit IRC14:44
*** dkehn has joined #openstack-ironic14:46
*** dkehn__ is now known as dkehn_14:46
*** matty_dubs|gone is now known as matty_dubs14:47
*** anniec has joined #openstack-ironic14:48
NobodyCamgood morning Ironic14:49
matty_dubsHowdy NobodyCam14:50
NobodyCamhey hey matty_dubs14:51
*** killer_prince has joined #openstack-ironic14:52
openstackgerritImre Farkas proposed a change to openstack/ironic: Add support for custom libvirt uri  https://review.openstack.org/7406214:53
*** loki__ has joined #openstack-ironic14:53
openstackgerritMax Lobur proposed a change to openstack/ironic: Refactor vendor_passthru to use conductor async workers.  https://review.openstack.org/7406314:55
openstackgerritMax Lobur proposed a change to openstack/ironic: Refactor vendor_passthru to use conductor async workers.  https://review.openstack.org/7406315:04
*** rustlebee is now known as russellb15:08
*** rwsu has joined #openstack-ironic15:09
*** linggao has joined #openstack-ironic15:14
*** loki__ has quit IRC15:20
*** loki__ has joined #openstack-ironic15:21
*** loki__ has quit IRC15:26
*** coolsvap has joined #openstack-ironic15:26
*** shausy has quit IRC15:26
max_loburbbiaw15:37
*** max_lobur has quit IRC15:37
*** max_lobur has joined #openstack-ironic15:37
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Add string representation method to MultiType  https://review.openstack.org/7407515:38
NobodyCamlucasagomes: 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
lucasagomesNobodyCam, ack thanks :D15:48
lucasagomesbtw I'm adding tests to the volume driver15:49
NobodyCamawesome :)15:49
lucasagomesbut yeah, it sucks to not be able to add dependecy on multiple patchs in gerrit :(15:51
lucasagomesonly a linear dependency15:51
NobodyCamyes15:52
NobodyCamI think a rebase on the volume driver will fix the issue I am having with the update (pull patches) script I've almost got working15:53
*** aignatov is now known as aignatov_15:54
lucasagomesNobodyCam, ack, yea I will rebase it15:59
devanandamorning, all15:59
NobodyCamgood morning devananda :)15:59
*** lazy_prince has joined #openstack-ironic15:59
lucasagomesmorning devananda16:00
*** loki__ has joined #openstack-ironic16:01
devanandalooks like jenkins is happy again?16:01
lucasagomesyeah I think it's grand now16:02
NobodyCamya the -1's look real now16:03
devanandaawesome16:04
NobodyCamany 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 linux16:06
lazy_princeNobodyCam: yes. for it to work we need to have chain.c32 copied to tftp directory..16:08
lazy_princeAnd we have it tested for linux disk images as well..16:09
NobodyCam:) that requirment should be commented on the reviews commit message16:10
lucasagomeslazy_prince, awesome16:10
NobodyCamyes! awesome to see :)16:10
*** dshulyak has joined #openstack-ironic16:14
* NobodyCam makes mo coffee16:14
devanandaso, we doing a review jam again today?16:14
NobodyCamI think max_lobur is afk atm!16:19
max_loburback16:21
max_loburmorning / afternoon Everyone16:21
NobodyCammorning max_lobur :)16:21
max_loburare we started already?16:22
*** creiht has left #openstack-ironic16:22
devanandai dont think so ...16:23
devanandai've been catching up on some of the recently posted reviews16:23
* devananda just -2'd the py3compat reviews16:23
devanandaetherpad: https://etherpad.openstack.org/p/IronicReviewDay16:24
NobodyCamI was looking at: https://review.openstack.org/#/c/7197416:24
devanandaoh ya - i was going to update that16:24
devanandaNobodyCam: marking that as WIP for now16:25
max_loburthere's unfinished discussion in those patch I think16:25
devanandaNobodyCam: max_lobur's changes that landed will conflict with that. it needs to be reworked anyway16:25
devanandahttps://review.openstack.org/#/c/73279/3 and https://review.openstack.org/#/c/73284/16:26
devanandalook fairly easy16:26
* max_lobur looking16:27
NobodyCamok... need to step away for a minute... brb16:27
devanandaas do i ... bbiam16:27
max_lobur+2d https://review.openstack.org/#/c/73279/ (Add wsme custom BooleanType type)16:31
lucasagomesbtw NobodyCam, I know it's not ideal, but what about make #72007 (refactor iscsi_initiator) depend on the #51328 (Ironic driver)16:32
lucasagomesto have a linear dependency16:32
lucasagomeshost_manager -> ironic driver -> iscsi refactor -> volume driver16:32
lucasagomesotherwise the volume driver can't be reviewed until the iscsi refactor or the ironic driver gets merged16:33
devanandalucasagomes: does the order make any sense if 'ironic driver' is the last one?16:33
devanandahost_mgr -> iscsi -> volume -> driver ?16:33
NobodyCamhumm16:34
NobodyCamvol needs driver16:34
lucasagomesdevananda, hmm it could be refactored like that, but the problem would remain16:34
lucasagomesiscsi doesnt really depend on host_mgr16:34
lucasagomeswe are adding as a dep just to have this linear dependency (cause multi dependency is not possible in gerrit afaik)16:35
devanandaright16:35
devanandaso put iscsi refactor as the root then16:35
*** jistr has quit IRC16:35
lucasagomesack16:35
devanandaiscsi -> host_mgr -> driver -> volume16:35
lucasagomesyea it would make it easier to get merged at least16:35
NobodyCam:) that would work16:36
lucasagomesdevananda, cheers, will do that16:36
*** bearhands is now known as comstud16:36
NobodyCamlucasagomes: awesome Thank you :)16:36
lucasagomesNobodyCam, heh thank YOU, you did the hard work16:37
NobodyCam:-p16:38
devanandamax_lobur: https://review.openstack.org/#/c/73284/ also +2'd16:38
devanandaI tested both locally, at least the "associated" part, since that is what client supports right now16:39
devanandaworks fine16:39
devanandawe'll need a patch to the client to add --maintenance [true|false]16:39
devanandahere's an important patch chain we should review16:40
devanandahttps://review.openstack.org/#/c/71297/ -> https://review.openstack.org/#/c/72395/16:40
NobodyCammax_lobur: I'll +a 7328416:44
max_loburNobodyCam: yep, ty!16:45
devanandaNobodyCam: https://review.openstack.org/#/c/71879/3 could use your eyes16:47
*** anniec has quit IRC16:47
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: API: Expose a way to start/stop the console  https://review.openstack.org/7299816:52
*** lazy_prince has quit IRC16:55
*** vkozhukalov has quit IRC17:00
NobodyCamhttps://review.openstack.org/#/c/7322317:01
*** digambar has joined #openstack-ironic17:02
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: PXE clean_up() to remove the pxe_deploy_key parameter  https://review.openstack.org/7187917:02
devanandalucasagomes: updated the unit test on you rpatch ^17:02
lucasagomesdevananda, ack thanks17:02
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: PXE clean_up() to remove the pxe_deploy_key parameter  https://review.openstack.org/7187917:03
*** Alexei_987 has quit IRC17:05
NobodyCamhttps://review.openstack.org/#/c/70348/17:07
openstackgerritA change was merged to openstack/ironic: Add wsme custom BooleanType type  https://review.openstack.org/7327917:08
devanandamax_lobur: let's talk about your comment on https://review.openstack.org/#/c/68018/5/ironic/api/controllers/v1/node.py17:08
*** martyntaylor has left #openstack-ironic17:11
*** lazy_prince has joined #openstack-ironic17:11
digambarHello17:14
digambargetting forbidden error when I run any cli command in ironic17:14
digambarBefore re-installing devstack, it is working well17:15
digambarafter re-installation, it is giving error every CLI command17:15
max_loburdigambar: hi!17:15
openstackgerritA change was merged to openstack/ironic: Use the custom wsme BooleanType on the nodes api  https://review.openstack.org/7328417:15
digambarhi17:15
max_loburcan you post error?17:16
NobodyCamdigambar: check the keystone creds in nova conf17:16
max_loburhttps://review.openstack.org/#/c/71297/ -> https://review.openstack.org/#/c/72395/17:16
max_loburfolks17:16
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: API validates driver name for both POST and PATCH  https://review.openstack.org/6801817:16
max_lobur1st one is merging now17:16
digambarall nova cli working well17:17
digambarit is failing @ only keystone side17:17
max_loburthe second one - I found one uncovered pathway17:17
max_loburyuriyz: ping17:17
NobodyCamdigambar: there should be an ironic block in nova.conf with keystone u/p17:17
digambarironic node-create -d fake_ssh -i ssh_address=192.168.122.1 -i ssh_username=root -i ssh_password=root123 -i ssh_virt_type=virsh17:18
digambarHTTPForbidden (HTTP 403)17:18
digambarok17:18
digambarlet me check17:18
devanandadigambar, NobodyCam - that failure is not in the nova.conf17:18
digambarokay17:18
digambarbefore this I am not getting this error, after installing new devstack this error is coming17:19
devanandadigambar: if python-ironicclient is geting HTTPForbidden talkign to ironic API, it is auth error. what privileges does your OS_USERNAME accout have?17:19
digambaroops17:19
devanandadigambar: "installing new devstack" is not very specific. what changed?17:19
digambarlet me check17:19
digambarok17:19
digambarhey sorry17:20
digambarI am trying it using demo user17:20
devanandayep17:20
digambargot it working using admin user17:21
devananda:)17:21
digambarThanks :)17:21
NobodyCam:)17:21
max_lobur:)17:21
NobodyCamhttps://review.openstack.org/#/c/7121217:26
*** mdurnosvistov_ has joined #openstack-ironic17:26
*** loki__ has quit IRC17:28
digambarhttps://bugs.launchpad.net/ironic/+bug/127639317:28
digambarworking on this bug17:28
digambarupdating port-update required17:28
digambarironic port-update 94809fcb-afbd-4116-80cc-20f24b6fbf4a replace17:29
digambarfor last argument17:29
digambarpath=??17:29
NobodyCammac=aa:bb...17:30
digambarok17:31
NobodyCamoh wait i think its address=17:31
*** dshulyak has quit IRC17:31
lucasagomesNobodyCam, you can hear me?17:31
devanandalucasagomes: https://review.openstack.org/#/c/73642/17:31
NobodyCamnope17:32
lucasagomesdevananda, feck lemme try17:32
digambarlet me paste here whole command17:32
lucasagomesdevananda, ok lemme rejoin the hangout17:32
devanandadigambar: please use pastebin for large pastes17:32
digambarok17:32
lucasagomesdevananda, yea I'm not mutted it just doesnt work17:33
lucasagomesdevananda, so17:33
digambarhttp://paste.openstack.org/show/66554/17:33
lucasagomesdevananda, that patch collide with another patch that ruby posted17:33
lucasagomesdevananda, so i abandoned that, and I'm comment at ruby's17:33
NobodyCamdigambar: my bad use address=17:34
* lucasagomes tries to fix his mic17:34
digambarokk17:34
digambar:)17:35
NobodyCam:)17:35
*** matty_dubs is now known as matty_dubs|lunch17:39
*** ndipanov is now known as ndipanov_gone17:48
NobodyCamhttps://review.openstack.org/#/c/6898017:49
digambarhttp://paste.openstack.org/show/66566/17:51
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Fix dev-quick-start.rst post landing for 9d81333fd0  https://review.openstack.org/7364217:52
NobodyCamdigambar: is it locking now17:53
digambarno17:54
*** derekh has quit IRC17:54
*** lazy_prince has quit IRC17:55
digambarbefore this, if I run port-update, it will give nodelocked error17:55
digambarnow it is showing the output17:55
NobodyCamdigambar: yep the bug is so it should be putting a lock on the node for that type of change17:59
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Task object as paramater to validate() methods  https://review.openstack.org/6898017:59
digambarok18:00
digambarso in that case, when anybody fires port-update, then it should put the lock on the node18:01
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: PXE and SSH validate() method to check for a port  https://review.openstack.org/6898518:01
NobodyCamyep18:01
devanandamax_lobur: https://review.openstack.org/#/c/68413/ commented. LGTM aside from commit message18:02
devanandalucasagomes: https://review.openstack.org/#/c/68018/ could use your eyes18:03
max_loburhttps://review.openstack.org/#/c/74063/18:09
NobodyCamhttps://review.openstack.org/#/c/7158518:12
*** athomas has quit IRC18:12
*** matty_dubs|lunch is now known as matty_dubs18:14
openstackgerritA change was merged to openstack/ironic: Allow to tear-down a node waiting to be deployed  https://review.openstack.org/7129718:14
max_loburhttps://review.openstack.org/#/c/73256/18:17
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: Correct assertEqual order from patch e69e41c99fb  https://review.openstack.org/7411418:26
*** tatyana has quit IRC18:26
openstackgerritMax Lobur proposed a change to openstack/ironic: Add JSONEncodedType with enforced type checking  https://review.openstack.org/6841318:27
NobodyCambbiab18:28
max_loburfolks ^^ easy one18:28
*** lucasagomes is now known as lucas-afk18:32
max_loburbrb18:33
openstackgerritA change was merged to openstack/ironic: Add common.service config options to sample  https://review.openstack.org/7348818:40
max_loburback18:40
*** vkozhukalov has joined #openstack-ironic18:44
*** digambar has quit IRC18:46
*** k4n0 has joined #openstack-ironic18:48
*** aignatov_ is now known as aignatov18:48
openstackgerritA change was merged to openstack/ironic: API validates driver name for both POST and PATCH  https://review.openstack.org/6801818:51
*** agordeev2 has joined #openstack-ironic18:52
*** openstackgerrit has quit IRC18:53
*** openstackgerrit has joined #openstack-ironic18:54
devanandaback18:55
devanandamax_lobur: +2'd18:56
max_loburthx18:58
*** lucas-afk is now known as lucasagomes18:58
devanandaheads up -- meeting in a minute18:58
mrdamorning all!19:01
devanandamrda: g'morning!19:02
devanandamrda: we're all in the meeting :)19:02
openstackgerritA change was merged to openstack/ironic: Task object as paramater to validate() methods  https://review.openstack.org/6898019:02
openstackgerritA change was merged to openstack/ironic: PXE and SSH validate() method to check for a port  https://review.openstack.org/6898519:03
mrdadevananda: 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-ironic19:15
openstackgerritA change was merged to openstack/ironic: Correct assertEqual order from patch e69e41c99fb  https://review.openstack.org/7411419:30
lucasagomesNobodyCam, devananda max_lobur when you guys have some time https://review.openstack.org/#/c/73642/19:34
max_loburack19:35
*** aignatov_ is now known as aignatov19:38
*** anniec has joined #openstack-ironic19:41
*** aignatov is now known as aignatov_19:42
* NobodyCam needs to do a quick walkies .. brb20:00
k4n0devananda: So the drivers will be exposed via API?20:00
devanandak4n0: they are: http://docs.openstack.org/developer/ironic/webapi/v1.html#drivers20:01
devanandatoday it's read only20:01
lucasagomesok folks I'm also going take a break/eat something (/me not feeling very well today)20:01
devanandalucasagomes: ack. hope you feel better :)20:01
lucasagomesdevananda, thanks!20:01
lucasagomesg'night!20:01
NobodyCamnight lucasagomes20:01
lucasagomesNobodyCam, night buddy!20:02
* lucasagomes will leave it open to read the scrollback laters20:02
*** lucasagomes is now known as lucas-afk20:02
k4n0devananda: Any other reason that the chassis object cant be made to do discovery?20:02
devanandak4n0: there's been discussion with the Tuskar folks about doing different kinds of node grouping20:02
devanandaeg, modelling of multiple failure domains20:03
devanandarather than just grouping by chassis20:03
k4n0devananda: 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
devanandak4n0: probably not20:04
devanandak4n0: or rather -- i dont know :)20:05
devanandaVendorPassthru interface requires a Node and a Task, like all the current driver interfaces20:05
devanandai haven't looked at how we might pass a request down to a driver without having either of those20:06
devanandawhat happens if, eg, two API requests are received at /drivers/seamicro/discover at the same time?20:06
devanandawhat 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
k4n0devananda: valid concerns20:07
devanandak4n0: so i think it's a great topic for the summit20:08
k4n0devananda: let me think along those lines.20:08
k4n0devananda: isnt the last date for submitting topics 14th Feb :)20:08
devanandak4n0: for talks, yes. for design sessions -- no :)20:08
devanandawhen i say "topic" i mean for us all to discuss in the design summit20:09
k4n0devananda: ohh, sure, we should discuss this20:09
k4n0devananda: alright , lets keep this topic alive in the mailing lists  till then, lets see if something can be done20:10
devanandak4n0: sounds good20:10
k4n0devananda: thanks for your time :) , cya soon20:10
* NobodyCam is back20:17
max_loburdevananda, NobodyCam https://review.openstack.org/#/c/74063/ take a look when you have time20:22
max_loburthere's opened question - do we need to guarantee that there should not be intrusion between vendor.validate and vendor.vendor_passthru20:23
*** mdurnosvistov_ has quit IRC20:23
*** jcooley_ has quit IRC20:25
max_loburk, I'm done for today. see you tomorrow!20:32
max_loburg'night20:32
*** max_lobur has quit IRC20:34
devanandag'night!20:34
*** k4n0 has left #openstack-ironic20:38
*** agordeev2 has quit IRC20:42
*** tatyana has joined #openstack-ironic20:42
*** ifarkas has quit IRC20:44
*** aignatov_ is now known as aignatov20:48
lifelesshey so21:06
lifelesswindows support21:06
lifelessI think that patch conflates a bunch of stuff21:06
lifelessthe blueprint too21:06
lifelesswhole disk is something that I think many folk may want (ill advised or not :P)21:06
lifelesslike - if you have that you can't use a flavor with ephemeral21:07
devanandaso, yes, it borks ephemeral support21:07
lifelessdevananda: how would you feel if we add an actual etherpad spec21:07
devanandabut for some deployments, that may be OK. I think it's a valid option (and you seem to agree)21:07
lifelesswhere we can do design21:07
lifelessand get a set of changes21:07
lifelessand then point the author of that patch at it21:08
devanandaand in looking at the patch now (which i hadn't really done before) i agree, it's conflating windows boot with whole-disk deploy21:08
lifelessok, I'll add a etherpad, sec21:08
openstackgerritA change was merged to openstack/ironic: Fix dev-quick-start.rst post landing for 9d81333fd0  https://review.openstack.org/7364221:09
lifelessalso I thought primeministerp said windows had a PXE mode?21:09
lifelessdevananda: https://etherpad.openstack.org/p/icehouse-ironic-windows-support21:10
devanandalifeless: this patch is based on primeministerp's work21:11
lifelessah21:11
devanandalifeless: his PXE mode is https://review.openstack.org/#/c/73054/4/ironic/drivers/modules/pxe_config_disk.template21:11
devanandaCOM32 chain.c32 APPEND hd021:11
devanandaAIUI, that ties us to whole-disk image here21:12
lifelessright thats not pxe booting, thats local booting21:12
lifelesswith a pxe pointer21:12
devanandaright21:12
lifelessis he around ?21:12
devanandawhich is what windows supports21:12
devanandaAIUI, we can't feed a kernel & ramdisk to windows over pXE21:12
devanandaPXE points windows at the local boot sector21:13
devanandaor more accurately, at some boot sector on a mounted media21:13
lifelesslinux chains into the local boot sector21:13
lifelessdevananda: btw I'm writing in https://etherpad.openstack.org/p/icehouse-ironic-windows-support21:15
*** mdurnosvistov_lt has joined #openstack-ironic21:20
lifelessok, I think I've got enough laid out in my head to review the patch now21:23
devanandaack21:26
devanandagood stuff in the etherpad. i have a concern about auto-detecting boot from localdisk, though21:26
lifelessgo on ?21:27
devanandalifeless: L2421:27
openstackgerritJosh Gachnang proposed a change to openstack/ironic: Removing unused get_node_power_state  https://review.openstack.org/7309721:27
lifelessdevananda: answered there21:28
lifelessdevananda: there is a race condition with neutron updating dnsmasq to care for21:28
lifelessdevananda: OTOH if a node power cycles faster than neutron updates dnsmasq....21:28
lifelessthats a problem for *everyone*21:28
devanandalifeless: we can prevent booting from local disk easily enough21:29
devanandajust by changing boot order via IPMI21:29
lifelessdevananda: we'd want that optional for TripleO21:29
devanandaright21:29
lifelessdevananda: has anyone asked 'please don't fall back to local disk if local disk is bootable' ?21:29
devanandaand 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
devanandaIIRC, you did :)21:30
devanandaor more specifically21:30
devanandayou asked tjat local disk not be bootable on some nodes21:30
lifelessyes21:30
lifelesswhich we get today by not having boot blocks21:31
lifelessuhm21:31
lifelessso IIRC what I was getting at21:31
devanandaah21:31
*** derekh has joined #openstack-ironic21:31
lifelessis that running the users stuff in a secure context is a bit odd21:31
lifelessand that could happen if they reboot locally21:31
lifelessbut21:31
lifelesswe don't have any secure context today21:31
devanandaheh, fair21:31
lifelessand, if we do get one, I'd expect it to be turned on in series with PXE config in Neutron etc.21:31
devanandaso, effectively, linux = if k&r in glance, set netboot, else set localboot. windows = no k&r in glance, always set localboot21:32
NobodyCamthe node is stil pxe booting to get the local boot flag21:35
NobodyCamjust no k&r being servied21:36
openstackgerritJosh Gachnang proposed a change to openstack/ironic: Removing unused get_node_power_state  https://review.openstack.org/7309721:36
NobodyCamif no K & R attached to image in glance then assume local boot?21:36
devanandaNobodyCam: that's what lifeless is proposing -- if no k&r in glance, then have the deploy driver actually set boot device to local disk21:37
devanandaNobodyCam: rather than have the node needlessly PXE boot just to be told to chain hd021:38
devanandathat's where my security concern comes in21:38
lifelessdevananda: actually no21:38
devanandaerm21:38
lifelessdevananda: I was not proposing changing the node's IPMI boot order21:38
lifelessdevananda: I was proposing just deleting the PXE entries from Neutron21:38
lifelessdevananda: but - changing the IPMI boot order might be better21:39
devanandalifeless: 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
devanandabeing explicit here is best, IMO21:40
devanandain each case, what do we want the node to boot from?21:40
lifelessdevananda: I think we should *also* change Neutron though21:40
lifelessdevananda: for clarity21:40
devanandasure :)21:40
devanandaif we're setting it to localboot, we should definitely unset the neutron boot option21:41
lifelessok, I think we're agreed on that aspect21:41
NobodyCamand then reset up for maintanince and firmware updates?21:41
lifelessyou've got a security concern ?21:41
lifelessNobodyCam: yes, we do anyway for pxe - we edit an option on disk, not conceptually any different21:42
NobodyCam:)21:42
devanandalifeless: if we infer localboot from the lack of k&r, then we lose the programatic control of where a node boots from21:42
devanandalifeless: eg, windows image won't hve k&r, so all windows instances will localboot, even if they're not essential tripleo infrastructure nodes21:43
devanandas/we lose/the operator loses/21:43
lifelessdevananda: We don't currently expose programmatic control of where to boot from21:44
lifelessso I'm not sure I follow21:44
*** coolsvap has quit IRC21:44
NobodyCamdevananda: dont all windows images HAVE to local boot21:44
lifelessdevananda: FWIW I believe the VM images in Nova work this way (IMBW)21:44
devanandalifeless: that's because everything boots from network today21:44
lifelessdevananda: I know :)21:44
devanandaNobodyCam: the NODE can still net boot, even if we chain to local disk.21:45
devanandaNobodyCam: that would prevent eg. the node from booting before neutron is online21:45
*** tatyana has quit IRC21:45
lifelessonly if we explicitly disable localboot21:46
devanandalifeless: right21:46
NobodyCamso pxe'ing the chain command..21:46
lifelessbecause pxe will time out21:46
openstackgerritA change was merged to openstack/ironic: Add JSONEncodedType with enforced type checking  https://review.openstack.org/6841321:46
lifelessand we don't explicily disable localboot today21:46
lifelessat least in nova-bm21:46
devanandasure21:46
devanandaok21:46
devanandalet's step back -- its geting confusing with the "we do today" vs "we want" vs ...21:46
devanandalifeless: 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 plane21:47
devanandayes?21:47
lifelessmaybe21:47
lifelesswhat I want is the ability to pull the plug on a DC21:47
devananda:-/21:47
lifelessand put it back in21:48
lifelessand have everything work21:48
devanandahah21:48
lifelesswithout complex/fragile manual steps21:48
lifelesseverything else is tactics :)21:48
lifelessso21:48
lifelessperhaps it will help if I layout first principles for our mutual memory refresh21:48
lifelessin a tripleo DC we have a large number of machines21:49
lifelesswhich are 'tenants'21:49
lifelessnova will try to restore their state when nova starts up. This is probably a bug, but we'll come back to that :P21:49
lifelessto boot properly they depend on DHCP - Neutron - and nova-api (metadata) and heat (more metadata) from the undercloud21:50
lifelessthen we have the undercloud21:50
lifelessthe undercloud is some number of machines scale to the DC size21:50
lifelesswhere the machines themselves are also tenants of the undercloud21:50
lifelessso to boot properly they need DHCP, nova-API, heat running21:50
lifelessDHCP not for PXE - DHCP for 'whats my IP within Linux'21:51
*** vkozhukalov has quit IRC21:53
lifelessto bring dhcp up we need neutron up which needs DB and rabbit up21:53
lifelessto bring DB up we need IP address + hostname->IP for other galera cluster members21:53
lifelesslikewise for rabbit21:53
lifelessto bring nova-api we need db + rabbit21:53
lifelessditto heat21:54
lifelesswe also need the previously running metadata config to be cached on the machines21:54
lifelessso - we basically need to freeze enough data to make that whole set of dependencies, within each undercloud machine survive booting with no other machines running21:54
*** jdob has quit IRC21:54
*** mdurnosvistov_lt has quit IRC21:54
lifelessand have the machines come up and self assemble, hit quorum, and become operational21:55
lifelessthen we can start asserting 'nova start' or 'nova reboot' or whatever all the other tenants21:55
*** mdurnosvistov_lt has joined #openstack-ironic21:55
lifelesswe don't need log servers, swift backends etc to come up21:55
NobodyCamlifeless: cache as NOT in the db. but on local media(coould be local db)21:56
lifelessNobodyCam: something persistent locally yes21:57
lifelessso, most machines in that setup can't boot locally at all.21:57
devanandalifeless: 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 to21:57
lifelesswhich ones can? The ones we've added local boot blocks for I guess - today.21:57
lifelessdevananda: maybe; will answer in detail in a sec21:58
lifelessif we scale the undercloud down21:58
lifelesswe would leave behind a machine with a local boot block21:58
lifelessthat we wanted to boot locally in the past21:58
lifelessbut that isn't part of the cluster anymore21:58
lifelessand *that* shouldn't be allowed to boot21:59
lifelessOTOH21:59
lifelesswe would want scale down to wipe its disk to delete passwords etc too21:59
lifelesswhich would quite naturally stop it booting ;)21:59
lifelessdevananda: so your possible assertions above22:01
lifelessa) boot the minimal closure from local disk first22:01
lifelessb) boot the rest only when desired22:01
lifelessfor 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
lifelessOTOH22:02
lifelesswe won't know that it will work unless we boot them with the network cable pulled out and DHCP etc not answering.22:02
lifelessso 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
lifelesss/thing/think/22:02
lifelessfor b) if we set the action on power reconnection to 'do nothing' for most machines, we'd get that22:03
lifelessthat would be a good thing for most DC's anyway - they usually can't handle an all-at-once-power-on event.22:03
lifelessdevananda: ok, fin22:03
devanandalifeless: ah. so you'd prefer "chassis policy off" for those machines22:06
lifelessdevananda: that seems like an appropriate thing22:07
devanandaindeed22:07
lifelessdevananda: 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
devanandait separates the "where should it boot from" from "what should happen on DC power on" concerns22:07
devanandayep22:07
lifelessbtw22:08
lifelesswill the nova driver also run a hash ring ?22:08
lifelessor how will two nova compute instances interact with Ironic?22:08
lifelessTOTAL CHANGE OF TOPIC :)22:08
*** linggao has quit IRC22:08
*** aignatov is now known as aignatov_22:10
devanandaheh22:11
devanandathe ironic api -> rpc pipe will aggregate requests for the same node22:12
lifelessI mean22:13
lifelesssomething lists all the nodes in the sceheduler right?22:13
lifelessis that Ironic or nova-compute?22:13
devanandaso if multiple nova-compute instances are asked to operate on a single node, some will encounter a NodealreadyLocked error22:13
devanandaah22:13
devanandayou mean22:13
lifelessdo all nova computes list all ironic nodes in the scheduler22:14
devanandawhich nova-compute will advertise which ironic nodes to the scheduler22:14
devanandaright22:14
lifelessso if we have 10K nodes and 50 conductors, nova-compute dies?22:14
lifelessbecause we have say 3 nova-computes for redundancy, but that lists 30K nodes in the secheduler ?22:14
devanandawhat's the rough limit on a cell size today -- O(100) nova-compute hosts, right?22:15
lifelesspick a number22:15
lifelessits going to be wrong, but whatever. C for Cell.22:15
lifelessC hosts.22:15
devanandak22:15
devanandaso we have C compute hosts22:15
devanandado you have any expectation of the ration of C to I, where I is # of ironic nodes?22:16
devanandarephrase22:16
devanandaif the scheduler today can handle only C hosts, do we expect it to handle more than C nodes ?22:17
devanandafrom teh scheduler's POV, each ironic node is a compute host22:17
devanandayour 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 issue22:18
lifelessso, I was pulling on a thread22:18
devanandaas you do :)22:18
lifelessI don't have a specific X is a problem point22:18
lifelessuhm22:19
lifelessI expect in a BM cloud that nova is basically a proxy22:19
devananda(it's a good thing)22:19
devanandaright22:19
lifelesstranslating queries to Ironic and back22:19
devanandayep22:19
lifelessappying some business logic etc.22:19
lifelessso22:19
lifelesssmall # of nova-compute22:19
lifelessI suspect I'd be happy to try 10K nodes with 3 nova computes.22:19
devanandaeverything 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 yet22:19
lifeless(so that a failure or deploy doesn't degrade to one node)22:19
devananda++22:20
lifelessI'd be a little wary of direct scheduler stuff for now; possibly when gantt is more mature22:20
devanandaright22:20
devananda+eventually22:20
lifelessscheduler fallover - I think I'm more worried by two-at-once races (even with locking - it can get hairy)22:20
devanandaso layer violation -- everything in nova, AFAIK, expects each nova driver instance to present a unique view of resources it owns22:21
lifelessbut we need the hypervisor id's to be constant in the scheduler right ?22:21
devanandanot a shared view of communla resources22:21
lifelessyes, this is an issue for vmware etc too AIUI22:21
devanandado you now how theyr'e solving it?22:21
lifelessNFI22:22
lifelesshow long does it take a node to time out of the scheduler ?22:23
lifelesslike22:23
lifelessI'm wondering if a fairly simple 'register anything thats not registered' approach would work for now22:23
devanandahumm22:24
devanandaso, typicaly, a nova driver registers (host, node) tuple for every resource it owns22:25
devanandaeg, kvm registers ($hostname, '')22:25
devanandasince it has only one resource, the local machine22:25
devanandanova-bm has a mapping of n-cpu<->node22:26
devanandatake over is a problem, etc... because nodes are registered to one n-cpu22:26
devanandawhat if all ironic nova-compute hosts registered the same hostname22:26
devanandaeg, ("ironic", $node_uuid)22:26
devanandait might bork the scheduler's ability to pick which n-cpu to send a request to, though22:27
devanandaor it might just all route to the first entry in a list ... i dont know ..22:27
devanandaNobodyCam: ^22:28
devanandalifeless: two summits ago, comstud proposed removing the first portion of that tuple22:29
devanandalifeless: instead of (host, node) the mapping would just be (node)22:29
devanandai dont think anyone worked on that since, but it would solve this problem for ironic, vmware, etc22:29
NobodyCamha I was thinking about how we expose nodes to nova this morning not for this reason thu22:30
NobodyCamdeva not to sside track but have you seen http://ipxe.org/wimboot22:32
devanandaNobodyCam: no, but again, there's another benefit of ipxe22:33
NobodyCam:)22:33
devanandaNobodyCam: take a look at https://www.kernel.org/pub/linux/kernel/people/geoff/petitboot/petitboot.html22:34
* NobodyCam clicks22:34
devanandasome great things in there. the kexec bit is questionable though22:35
* NobodyCam thinks after graduation ... After Graduation ... AFTER GRADUATION....22:35
devanandaheh22:35
* devananda goes back to testing devstack22:36
*** mdurnosvistov_lt has quit IRC22:49
*** derekh has quit IRC22:49
lucas-afkdevananda, 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 grub4dos23:20
lucas-afkand from there you can boot the local windows23:20
lucas-afkso it would work just like we have for linux now23:21
lucas-afktho it looks a bit hacky :)23:21
*** matty_dubs is now known as matty_dubs|gone23:22
lifelesslucas-afk: thats basically chain loading though right ?23:24
lucas-afklifeless, yes, that's it pretty much23:25
lifelesswhich ^ we don't need23:25
devanandalifeless: 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
devanandawhich doesn't support it23:36
devanandathink that'll break anything?23:36
*** jcooley_ has joined #openstack-ironic23:37
lifelessprecise won't work.23:37
lifelesslibvirt there doesn't support ovs networks at all.23:37
devanandai see23:38
lifelessyou'll need the ubuntu cloud archive enabled and ovs + libvirt from that pulled in.23:38
lifelesswhich will give you --persistent anyhow, I expect.23:38
devanandaso23:38
devanandainfra CI runs on precise23:38
devanandathat rules out using tripleo-incubator scripts to help prepare our CI environment23:39
devanandai'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 env23:40
lifelessso we use ovs switches rather than linux bridge23:59
lifelessbecause we got better L2 behaviour from them.23:59
« Sunday, 2014-02-16 Index Tuesday, 2014-02-18 »

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!