Tuesday, 2014-03-04

*** jcooley_ has quit IRC00:04
*** jcooley_ has joined #openstack-ironic00:05
*** antonym has quit IRC00:10
*** antonym has joined #openstack-ironic00:24
*** rwsu has quit IRC00:27
*** rwsu has joined #openstack-ironic00:30
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Fix SeaMicro driver post landing for ba207b4aa0  https://review.openstack.org/7773900:33
*** jdob has quit IRC00:34
*** jdob has joined #openstack-ironic00:35
*** jdob has quit IRC00:40
*** matty_dubs|sfo is now known as matty_dubs|gone00:41
*** dhellmann is now known as dhellmann_00:43
*** openstackgerrit has quit IRC00:44
*** openstackgerrit has joined #openstack-ironic00:44
*** epim has quit IRC01:05
*** rwsu has quit IRC01:17
*** lucasagomes has quit IRC01:24
*** nosnos has joined #openstack-ironic01:29
*** derekh has quit IRC01:30
*** jcooley_ has quit IRC01:49
*** jcooley_ has joined #openstack-ironic01:49
*** jcooley_ has quit IRC01:53
*** jcooley_ has joined #openstack-ironic01:56
*** jcooley_ has quit IRC02:02
*** rloo has joined #openstack-ironic02:02
*** jcooley_ has joined #openstack-ironic02:07
*** jcooley_ has quit IRC02:08
*** sjing has joined #openstack-ironic02:12
*** jcooley_ has joined #openstack-ironic02:13
*** jcooley_ has quit IRC02:13
*** jcooley_ has joined #openstack-ironic02:26
*** jcooley_ has quit IRC02:26
openstackgerritShuangtai Tian proposed a change to openstack/ironic: Sync the log_handler from oslo  https://review.openstack.org/7650002:33
*** rloo has quit IRC03:17
openstackgerritSun Jing proposed a change to openstack/ironic: Support serial console access  https://review.openstack.org/6410003:20
*** vkozhukalov has joined #openstack-ironic03:32
openstackgerritZhang Yang proposed a change to openstack/ironic: Remove redundant default value None for dict.get  https://review.openstack.org/7777103:37
*** Manishanker has quit IRC04:09
*** sjing has quit IRC04:26
*** sjing has joined #openstack-ironic04:27
*** harlowja is now known as harlowja_away04:30
*** harlowja_away is now known as harlowja04:35
*** jcooley_ has joined #openstack-ironic04:37
*** killer_prince has joined #openstack-ironic04:55
*** jcooley_ has quit IRC05:02
*** mrda is now known as mrda_away05:09
*** vkozhukalov has quit IRC05:10
*** lazy_prince has quit IRC05:12
openstackgerritSun Jing proposed a change to openstack/ironic: Support serial console access  https://review.openstack.org/6410005:12
*** lazy_prince has joined #openstack-ironic05:17
*** sjing has quit IRC05:20
*** vkozhukalov has joined #openstack-ironic05:21
*** rmiki has joined #openstack-ironic05:24
*** wallnr has joined #openstack-ironic05:29
*** wallnr has left #openstack-ironic05:29
*** vkozhukalov has quit IRC05:35
*** jcooley_ has joined #openstack-ironic05:36
*** loki_ has joined #openstack-ironic05:42
*** Manishanker has joined #openstack-ironic05:44
openstackgerritJenkins proposed a change to openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/7119206:06
*** harlowja is now known as harlowja_away06:16
*** yongli has quit IRC06:21
*** wanyen has quit IRC06:22
*** wallnr has joined #openstack-ironic06:33
*** wallnr has quit IRC06:46
*** romcheg has joined #openstack-ironic06:59
*** saju_m has joined #openstack-ironic07:12
*** lsmola has joined #openstack-ironic07:23
*** nosnos has quit IRC07:31
*** nosnos_ has joined #openstack-ironic07:31
*** nosnos_ has quit IRC08:04
*** nosnos has joined #openstack-ironic08:04
*** max_lobur has joined #openstack-ironic08:15
*** tatyana has joined #openstack-ironic08:22
*** ndipanov has joined #openstack-ironic08:23
*** saju_m has quit IRC08:26
*** max_lobur has quit IRC08:49
*** athomas has joined #openstack-ironic08:50
*** jistr has joined #openstack-ironic08:55
openstackgerrityangxurong proposed a change to openstack/ironic: Fix params order in assertEqual  https://review.openstack.org/7607709:17
*** rmiki has quit IRC09:24
*** saju_m has joined #openstack-ironic09:41
*** Manishanker has quit IRC09:51
*** martyntaylor has joined #openstack-ironic09:57
*** martyntaylor has quit IRC10:07
*** martyntaylor has joined #openstack-ironic10:12
*** max_lobur has joined #openstack-ironic10:15
*** athomas has quit IRC10:21
openstackgerritRohan Kanade proposed a change to openstack/ironic: Adds max retry limit to sync_power_state task  https://review.openstack.org/7742010:28
*** max_lobur has quit IRC10:32
*** max_lobur has joined #openstack-ironic10:33
*** athomas has joined #openstack-ironic10:39
*** tatyana has quit IRC10:58
*** Manishanker has joined #openstack-ironic11:40
Manishankerhi I am trying to install devstack with ironic and getting the error "[ERROR] /home/test/devstack/lib/ironic:198 ir-api did not start"11:41
ManishankerI followed the instructions from https://gist.github.com/max-lobur/778607411:42
Manishankercan anyone help me with this ??11:42
max_loburhi, could you please post api logs11:44
max_loburscreen -x11:44
max_loburir-api screen11:44
Manishankertest@openstackvm7:/opt/stack/tempest$ cd /opt/stack/ironic; /usr/local/bin/ironic-api --config-file=/etc/ironic/ironic.conf || echo "ir-api failed to start" | tee "/opt/stack/status/stack/ir-api.failure" 2014-03-04 21:25:27.006 24411 DEBUG ironic.common.utils [-] Reloading cached file /etc/ironic/policy.json read_cached_file /opt/stack/ironic/ironic/common/utils.py:310 2014-03-04 21:25:27.019 24411 CRITICAL ironic [-]11:45
ManishankerI can see this might be the issue "2014-03-04 22:19:28.062 24711 CRITICAL ironic [-] __init__() got an unexpected keyword argument 'readonly' ir-api failed to start"11:46
*** loki_ has quit IRC11:51
*** romcheg has left #openstack-ironic11:54
*** romcheg has joined #openstack-ironic11:55
*** vkozhukalov has joined #openstack-ironic11:55
*** romcheg has quit IRC12:08
yuriyzmorning Ironic12:08
*** ekarlso has quit IRC12:09
yuriyzManishanker: what is WSME version (pip show wsme)?12:09
Manishanker0.5b512:10
Manishankeri just checked it12:10
yuriyzshould be 0.612:10
ManishankerI will upgrade this and let you know12:10
openstackgerritRohan Kanade proposed a change to openstack/ironic: Adds max retry limit to sync_power_state task  https://review.openstack.org/7742012:15
ManishankerI have upgraded checked Wsme version and i was able to run ir-api,but horizon failed with this error " cd /opt/stack/horizon && sudo tail -f /var/log/apache2/horizon_error.log || echo "horizon failed to start" | tee "/opt/stack/status/stack/horizon.failure" [Tue Mar 04 16:28:38 2014] [error]     resp, body = self.request(url, method, **kwargs) [Tue Mar 04 16:28:38 2014] [error]   File "/opt/stack/python-novaclient/nov12:16
max_loburI always solving this by "python manage.py runserver your_ip:horizon_port (seems 6000)"12:18
max_loburfrom those screen horizon dir12:18
max_lobur* or horizon dir12:18
max_loburalso currently Ironic is not represented on horizon in neither way12:20
ManishankerHey max how do you provision the Baremetal instacnes ?12:25
max_loburthere's a more complete guide12:26
max_loburthe first one covers devstack installation only12:26
max_loburlemme find link12:26
Manishankerokay12:26
*** romcheg has joined #openstack-ironic12:26
max_loburhttps://etherpad.openstack.org/p/IronicDeployDevstack12:28
max_lobur.12:29
ManishankerHey max this is the error when i try to login Horizon "Connection to neutron failed: Maximum attempts reached"12:29
max_loburI believe GUIDE 1 is ok12:29
ManishankerSo you are saying we cant use Ironic with Horizon ?12:30
Manishankerokay i will try that :)12:30
max_loburthere's nothing to do on horizon for Ironic :)12:30
romchegMorning everyone!12:31
Haomengmorning all12:32
max_loburmorning :)12:32
openstackgerritRoman Prykhodchenko proposed a change to openstack/ironic: Process public API list as regular expressions  https://review.openstack.org/6662112:36
romchegFor some reason I lost this patch :)12:37
romchegThought it was already merged :)12:37
*** saju_m has quit IRC12:40
*** saju_m has joined #openstack-ironic12:51
mdurnosvistovMorning all! :)12:55
*** Alexei_9871 has joined #openstack-ironic12:59
*** saju_m has quit IRC13:04
*** dhellmann_ is now known as dhellmann13:11
*** lsmola has quit IRC13:23
*** ekarlso has joined #openstack-ironic13:23
*** linggao has joined #openstack-ironic13:27
*** dhellmann is now known as dhellmann_13:36
*** lsmola has joined #openstack-ironic13:36
*** krtaylor has joined #openstack-ironic13:51
*** nosnos has quit IRC13:56
*** lsmola has quit IRC14:24
devanandamorning, all14:29
openstackgerritA change was merged to openstack/ironic: Do not delete a Node which is not powered off  https://review.openstack.org/7625814:33
*** lsmola has joined #openstack-ironic14:33
*** lsmola has quit IRC14:33
*** lsmola has joined #openstack-ironic14:34
mdurnosvistovdevananda: morning :)14:42
devanandamatty_dubs|gone: around?14:44
linggaoMorning devananda.14:49
*** tatyana has joined #openstack-ironic14:49
*** lsmola has quit IRC14:50
*** tatyana has quit IRC14:51
NobodyCamgood morning Ironic14:57
GheRiveromorning Ironic15:00
pquernamorning15:00
linggaomorning NobodyCam, GheRivero and all15:01
*** killer_prince has quit IRC15:13
*** lazy_prince is now known as killer_prince15:13
openstackgerritA change was merged to openstack/ironic: Python 3: replace "im_self" by "__self__"  https://review.openstack.org/7651715:14
romchegMorning devananda NobodyCam!15:18
romchegHow is the meet up going?15:18
devanandagood!15:19
devanandayesterday was focused on feature freeze crunch.15:19
devanandaaiming to get more integration with tripleo today15:20
NobodyCammorning GheRivero devananda linggao pquerna and romcheg15:20
*** jgrimm has joined #openstack-ironic15:20
linggaodevananda,NobodyCam, I just finished up the assignements for my company. Now I am able to switch back to Ironic again. Is there anything I can help with? I know you guys are busy getting Ironic graduated.15:21
NobodyCamlinggao: awesome and welcome back15:22
linggao:)15:22
devanandalinggao: for the next month we will be focused on testing and stability of ironic, and writing documentation15:23
linggaodevananda, then shoud I work on bug fixing?15:25
devanandalinggao: sure :)15:25
NobodyCamlinggao: ya15:26
linggaodevananda, ok, I will have my advanture in the bug-land :-)15:27
linggaodevananda NobodyCam, if there are specific ones that you think I can work on, plesse let me know.15:28
*** lsmola has joined #openstack-ironic15:32
NobodyCamlinggao: take a look at https://launchpad.net/ironic/+milestone/icehouse-315:35
NobodyCami'm not 100% sure this is complete https://bugs.launchpad.net/ironic/+bug/126459615:35
NobodyCamIPMI set_boot_device is not called or tested15:36
linggaoNobodyCam, it is assgined to Michael Davies already?15:37
linggaoLooks like most of bugs have assignees already.15:37
NobodyCamlinggao: there is a patch or two up for that I seem to recall that they are nit complete however15:38
NobodyCamthou I am still on first cup of coffee15:38
*** coolsvap has joined #openstack-ironic15:38
linggaolol15:38
* Shrews throws coffee at NobodyCam15:39
NobodyCamlol morning Shrews :)15:39
*** Alexei_9871 has quit IRC15:41
devanandacoffee? what's that?15:41
NobodyCamits that stuff I keep in the my IV drip.15:43
NobodyCam:-p15:44
*** vkozhukalov has quit IRC15:45
linggaoNobodyCam, I'll take a look at this bug in details.15:45
NobodyCamThank you linggao :)15:46
linggaoNobodayCam, I also need coffee or tea :)15:48
openstackgerritA change was merged to openstack/ironic: Implements SeaMicro VendorPassThru functionality  https://review.openstack.org/7072015:57
*** coolsvap1 has joined #openstack-ironic15:58
NobodyCamok starting to pack up will be hitting the road in a few min. if I am late getting to  Yahoo it will be do to traffic16:00
*** coolsvap has quit IRC16:00
NobodyCamdevananda: fyi ^^^16:01
devanandahmm, i should go downstairs and have food/coffee ....16:01
devanandaNobodyCam: ack. see you soon!16:01
NobodyCam:)16:01
*** dhellmann_ is now known as dhellmann16:12
*** hemna has quit IRC16:31
*** jistr has quit IRC16:31
*** harlowja_away is now known as harlowja16:40
*** harlowja has quit IRC16:41
*** hemna has joined #openstack-ironic16:41
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Update Neutron if mac address of the port changed  https://review.openstack.org/7793916:55
*** harlowja has joined #openstack-ironic16:56
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Add TaskManager lock on change port data  https://review.openstack.org/7762317:01
*** wallnr has joined #openstack-ironic17:03
*** jdob has joined #openstack-ironic17:06
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Update Neutron if mac address of the port changed  https://review.openstack.org/7793917:06
*** yuriyz has quit IRC17:07
*** wallnr has quit IRC17:07
jrollmornin!17:09
*** lucasagomes has joined #openstack-ironic17:10
*** romcheg has quit IRC17:11
*** rwsu has joined #openstack-ironic17:14
GheRiveromorning17:15
lucasagomesmorning GheRivero17:16
*** digambar has joined #openstack-ironic17:19
JoshNang_https://etherpad.openstack.org/p/tripleo-icehouse-sprint17:21
*** derekh has joined #openstack-ironic17:23
*** derekh has quit IRC17:23
*** matty_dubs|gone is now known as matty_dubs|sfo17:27
matty_dubs|sfodevananda: Ha, sorry, I was sound asleep when you pinged me earlier.17:27
*** martyntaylor has quit IRC17:32
*** derekh has joined #openstack-ironic17:32
*** Manishanker has quit IRC17:52
*** digambar has quit IRC17:53
jrolldevananda: we talked about that node model's instance info class yesterday (for configdrive data, etc) - I remember you wanted to move over root_gb, were there other fields too?17:54
jrollwriting up a blueprint right now17:54
*** rloo has joined #openstack-ironic17:55
jrolls/instance info class/instance info/17:55
*** ndipanov has quit IRC18:00
*** vkozhukalov has joined #openstack-ironic18:05
NobodyCamMorning Ironic18:08
matty_dubs|sfomorning NobodyCam18:18
*** max_lobur has quit IRC18:21
*** wallnr has joined #openstack-ironic18:29
devanandajroll: node.instance_info -- yes. need to move root_gb, swap_gb, ephemeral_gb, ephemeral_format, image_source18:29
*** wallnr has left #openstack-ironic18:32
jrollthanks18:42
pquernajroll: so, on this18:51
jrollhttps://gist.github.com/jimrollenhagen/971d981648b15498d88118:52
jroll^ instance_info blueprint draft18:52
pquernaimplies config drive is a base64 blob right?18:52
jrollright18:52
jrollwith the thought of "this is how nova does this everywhere else"18:52
jrollbut I'm not tied to that by any means18:53
jrollthis field would not require a configdrive key18:53
pquernahrm, don't feel awesome about a base64 blob, inside a json blob, inside a string field :x18:53
jrollbecause that is driver specific18:53
jrollsure18:53
jrollbut that's a teeth agent driver discussion, not really an ironic core discussion, yeah?18:53
jrollbasically a driver can put any instance-specific data in this field18:53
jrolland use it later when deploying18:54
russell_hare some of these fields like required core fields, and some are teeth-agent specific?18:54
jrollall the fields moving into that are pxe-driver specific18:54
jrollthey're currently in the driver_info field, but are specific to the instance18:54
russell_hgotcha18:55
JoshNang_checked by the pxe-driver's validate()?18:55
jrollthis is coming out of a chat that devananda and I had yesterday about how to get instance info to deploy()18:55
russell_hso comstud was saying that nova may require all drivers to implement configdrive18:55
JoshNang_as in, each driver can validate that info separately18:55
russell_hwhich, to me, suggests that ironic must require all drivers to do so as well18:56
jrollJoshNang_: correct18:56
jrollhrm18:56
pquernarussell_h: an interesting turn of events.18:56
russell_hbut I'm making at least 2 leaps there which should be validated18:56
jrollrussell_h: that sucks18:56
pquernarussell_h: but nova doesn't today?18:56
pquernarussell_h: i mean, if DefCore says ConfigDrive all the things.18:56
*** jistr has joined #openstack-ironic18:57
jrollrussell_h: I know that today, for example, root_gb is only validated in the pxe driver18:57
jrolloh, you mean ironic must require all drivers to implement configdrive18:59
*** athomas has quit IRC18:59
jrollhrm19:00
pquernawell, given a 'configdrive' like thing, make an os think of it as a configdrive like thing.19:00
*** rloo has quit IRC19:00
*** rloo has joined #openstack-ironic19:01
pquernacomstud: is DefCore the right reason for something like ConfigDrive being a requirement, or is there another driver.19:01
*** rloo has quit IRC19:05
comstudpquerna: I don't think 'configdrive' is actually a DefCore thing19:05
comstudIt was a decision made by nova only19:05
comstudpromoting the config drive extension to be core API19:06
comstudin v3 (although there is likely not a v3 now for a while:)19:06
*** rloo has joined #openstack-ironic19:11
jrollpquerna: so, back to where we were./b 6319:16
jrolloops19:17
jrollbut yeah19:17
jrollthis field is kind of defined as arbitrary data for the driver to use19:17
jrolllike, an example JSON blob is good to have in the blueprint, but I don't think it should specify exactly what should be in there19:17
jrollbecause that's up to the driver19:17
pquernahrm.19:23
*** romcheg has joined #openstack-ironic19:28
openstackgerritGhe Rivero proposed a change to openstack/ironic: Generic Multiplexed method  https://review.openstack.org/7332219:34
*** lsmola has quit IRC19:35
*** vkozhukalov has quit IRC19:36
*** rloo has quit IRC19:39
*** martyntaylor has joined #openstack-ironic19:45
*** rloo has joined #openstack-ironic19:48
openstackgerritRohan Kanade proposed a change to openstack/ironic: Fix wrong exception raised by conductor for node  https://review.openstack.org/7742620:04
*** martyntaylor has left #openstack-ironic20:19
*** jistr has quit IRC20:20
*** dwalleck has joined #openstack-ironic20:39
*** jcooley_ has quit IRC20:43
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: deploy() to fail if image is larger than root partition  https://review.openstack.org/7800120:57
*** mrda_away is now known as mrda20:57
jrolldevananda: so, on the instance_info thing we discussed. you were talking about passing that data on create()? that's not going to work for all drivers - there's plenty of info that you don't have at create() time that you do have at deploy() time. for example, configdrive.20:57
mrdamorning all20:58
NobodyCammorning mrda20:58
mrdasprint going well NobodyCam ?20:58
jrolland about storing that data in the DB, I'm not sure if that's necessary. does nova do the retries if a deploy fails, or does ironic handle that? if nova handles it, we can just pass the data into deploy each time, yeah?20:59
jrollheya mrda20:59
mrdahey jroll21:00
NobodyCamhey hey lucasagomes how good are you with tox settings?21:10
lucasagomesNobodyCam, ahn not that good, what's up?21:10
openstackgerritChris Krelle proposed a change to openstack/ironic: Add Ironic Nova driver  https://review.openstack.org/7800221:22
*** jcooley_ has joined #openstack-ironic21:23
pquernacomstud: devananda: https://review.openstack.org/#/c/77504/  <- adding neutron networks to metadata service.21:28
openstackgerritChris Krelle proposed a change to openstack/ironic: Add Ironic Nova driver  https://review.openstack.org/7800221:28
pquernaactually sorry, thats just adding one more field.21:28
pquernahttps://review.openstack.org/#/c/77504/21:29
pquernahas adding network info21:29
pquernabalh, confusing tabs and urls.21:29
pquernahttps://review.openstack.org/#/c/74002/21:30
*** jcooley_ has quit IRC21:57
*** rloo has quit IRC22:08
*** linggao has quit IRC22:10
*** dhellmann is now known as dhellmann_22:18
*** dwalleck has quit IRC22:36
*** dwalleck has joined #openstack-ironic22:38
devanandamrda: are you working on https://bugs.launchpad.net/ironic/+bug/1285507 ?22:38
*** dwalleck_ has joined #openstack-ironic22:39
NobodyCammrda: sorry I missed your question... Yes it is :)22:41
*** dwalleck has quit IRC22:43
openstackgerritSirushti Murugesan proposed a change to openstack/ironic: Adds support for deploying Disk Images  https://review.openstack.org/7305422:44
*** jcooley_ has joined #openstack-ironic22:44
russell_hjroll: https://gist.github.com/jimrollenhagen/971d981648b15498d881 <-- incidentally, I think I'm +1 to this22:54
jrollrussell_h: well, we talked during lunch about maybe not storing this in the db22:54
russell_hjroll: Ironic should do the retry if a conductor dies22:54
jrolldepending if nova or ironic does the retries22:54
jrolloh22:54
russell_hjroll: oh... right22:54
jrollwelp22:54
russell_hyeah, I think its gotta go in the db22:55
jrollare you saying ironic currently does the retry, or should do the retry?22:55
russell_hI think thats also OK... I think its reasonable to record the parameters an instance was built with22:55
jrollyeah, fair22:55
*** blamar has joined #openstack-ironic22:56
jrolldevananda: did you see my questions for you way in the scrollback?22:56
devanandajroll: no22:56
jrolldevananda: so, on the instance_info thing we discussed. you were talking about passing that data on create()? that's not going to work for all drivers - there's plenty of info that you don't have at create() time that you do have at deploy() time. for example, configdrive.22:57
jrolland about storing that data in the DB, I'm not sure if that's necessary. does nova do the retries if a deploy fails, or does ironic handle that? if nova handles it, we can just pass the data into deploy each time, yeah?22:57
comstudnova does retries right now22:59
comstudif the virt driver spawn() raises22:59
comstudit'll re-schedule22:59
jrollcomstud: assuming ironic causes spawn() to raise an exception23:00
comstudI haven't looked.. I assume ironic virt driver issues create.. then polls to wait for it to be done23:01
comstudor error23:01
comstudif error.. it should raise so nova can retry23:01
*** dwalleck has joined #openstack-ironic23:03
*** dwalleck_ has quit IRC23:03
*** romcheg has quit IRC23:04
*** rloo has joined #openstack-ironic23:10
*** rloo has joined #openstack-ironic23:10
jrolldevananda: here's the blueprint for the stuff we were just talking about: https://blueprints.launchpad.net/ironic/+spec/add-node-instance-info23:13
*** dwalleck has quit IRC23:19
openstackgerritA change was merged to openstack/ironic: Fix wrong exception raised by conductor for node  https://review.openstack.org/7742623:32
openstackgerritA change was merged to openstack/ironic: Fix params order in assertEqual  https://review.openstack.org/7607723:32
*** blamar has quit IRC23:48
*** blamar has joined #openstack-ironic23:49

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