*** jcooley_ has quit IRC | 00:04 | |
*** jcooley_ has joined #openstack-ironic | 00:05 | |
*** antonym has quit IRC | 00:10 | |
*** antonym has joined #openstack-ironic | 00:24 | |
*** rwsu has quit IRC | 00:27 | |
*** rwsu has joined #openstack-ironic | 00:30 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Fix SeaMicro driver post landing for ba207b4aa0 https://review.openstack.org/77739 | 00:33 |
---|---|---|
*** jdob has quit IRC | 00:34 | |
*** jdob has joined #openstack-ironic | 00:35 | |
*** jdob has quit IRC | 00:40 | |
*** matty_dubs|sfo is now known as matty_dubs|gone | 00:41 | |
*** dhellmann is now known as dhellmann_ | 00:43 | |
*** openstackgerrit has quit IRC | 00:44 | |
*** openstackgerrit has joined #openstack-ironic | 00:44 | |
*** epim has quit IRC | 01:05 | |
*** rwsu has quit IRC | 01:17 | |
*** lucasagomes has quit IRC | 01:24 | |
*** nosnos has joined #openstack-ironic | 01:29 | |
*** derekh has quit IRC | 01:30 | |
*** jcooley_ has quit IRC | 01:49 | |
*** jcooley_ has joined #openstack-ironic | 01:49 | |
*** jcooley_ has quit IRC | 01:53 | |
*** jcooley_ has joined #openstack-ironic | 01:56 | |
*** jcooley_ has quit IRC | 02:02 | |
*** rloo has joined #openstack-ironic | 02:02 | |
*** jcooley_ has joined #openstack-ironic | 02:07 | |
*** jcooley_ has quit IRC | 02:08 | |
*** sjing has joined #openstack-ironic | 02:12 | |
*** jcooley_ has joined #openstack-ironic | 02:13 | |
*** jcooley_ has quit IRC | 02:13 | |
*** jcooley_ has joined #openstack-ironic | 02:26 | |
*** jcooley_ has quit IRC | 02:26 | |
openstackgerrit | Shuangtai Tian proposed a change to openstack/ironic: Sync the log_handler from oslo https://review.openstack.org/76500 | 02:33 |
*** rloo has quit IRC | 03:17 | |
openstackgerrit | Sun Jing proposed a change to openstack/ironic: Support serial console access https://review.openstack.org/64100 | 03:20 |
*** vkozhukalov has joined #openstack-ironic | 03:32 | |
openstackgerrit | Zhang Yang proposed a change to openstack/ironic: Remove redundant default value None for dict.get https://review.openstack.org/77771 | 03:37 |
*** Manishanker has quit IRC | 04:09 | |
*** sjing has quit IRC | 04:26 | |
*** sjing has joined #openstack-ironic | 04:27 | |
*** harlowja is now known as harlowja_away | 04:30 | |
*** harlowja_away is now known as harlowja | 04:35 | |
*** jcooley_ has joined #openstack-ironic | 04:37 | |
*** killer_prince has joined #openstack-ironic | 04:55 | |
*** jcooley_ has quit IRC | 05:02 | |
*** mrda is now known as mrda_away | 05:09 | |
*** vkozhukalov has quit IRC | 05:10 | |
*** lazy_prince has quit IRC | 05:12 | |
openstackgerrit | Sun Jing proposed a change to openstack/ironic: Support serial console access https://review.openstack.org/64100 | 05:12 |
*** lazy_prince has joined #openstack-ironic | 05:17 | |
*** sjing has quit IRC | 05:20 | |
*** vkozhukalov has joined #openstack-ironic | 05:21 | |
*** rmiki has joined #openstack-ironic | 05:24 | |
*** wallnr has joined #openstack-ironic | 05:29 | |
*** wallnr has left #openstack-ironic | 05:29 | |
*** vkozhukalov has quit IRC | 05:35 | |
*** jcooley_ has joined #openstack-ironic | 05:36 | |
*** loki_ has joined #openstack-ironic | 05:42 | |
*** Manishanker has joined #openstack-ironic | 05:44 | |
openstackgerrit | Jenkins proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/71192 | 06:06 |
*** harlowja is now known as harlowja_away | 06:16 | |
*** yongli has quit IRC | 06:21 | |
*** wanyen has quit IRC | 06:22 | |
*** wallnr has joined #openstack-ironic | 06:33 | |
*** wallnr has quit IRC | 06:46 | |
*** romcheg has joined #openstack-ironic | 06:59 | |
*** saju_m has joined #openstack-ironic | 07:12 | |
*** lsmola has joined #openstack-ironic | 07:23 | |
*** nosnos has quit IRC | 07:31 | |
*** nosnos_ has joined #openstack-ironic | 07:31 | |
*** nosnos_ has quit IRC | 08:04 | |
*** nosnos has joined #openstack-ironic | 08:04 | |
*** max_lobur has joined #openstack-ironic | 08:15 | |
*** tatyana has joined #openstack-ironic | 08:22 | |
*** ndipanov has joined #openstack-ironic | 08:23 | |
*** saju_m has quit IRC | 08:26 | |
*** max_lobur has quit IRC | 08:49 | |
*** athomas has joined #openstack-ironic | 08:50 | |
*** jistr has joined #openstack-ironic | 08:55 | |
openstackgerrit | yangxurong proposed a change to openstack/ironic: Fix params order in assertEqual https://review.openstack.org/76077 | 09:17 |
*** rmiki has quit IRC | 09:24 | |
*** saju_m has joined #openstack-ironic | 09:41 | |
*** Manishanker has quit IRC | 09:51 | |
*** martyntaylor has joined #openstack-ironic | 09:57 | |
*** martyntaylor has quit IRC | 10:07 | |
*** martyntaylor has joined #openstack-ironic | 10:12 | |
*** max_lobur has joined #openstack-ironic | 10:15 | |
*** athomas has quit IRC | 10:21 | |
openstackgerrit | Rohan Kanade proposed a change to openstack/ironic: Adds max retry limit to sync_power_state task https://review.openstack.org/77420 | 10:28 |
*** max_lobur has quit IRC | 10:32 | |
*** max_lobur has joined #openstack-ironic | 10:33 | |
*** athomas has joined #openstack-ironic | 10:39 | |
*** tatyana has quit IRC | 10:58 | |
*** Manishanker has joined #openstack-ironic | 11:40 | |
Manishanker | hi 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 |
Manishanker | I followed the instructions from https://gist.github.com/max-lobur/7786074 | 11:42 |
Manishanker | can anyone help me with this ?? | 11:42 |
max_lobur | hi, could you please post api logs | 11:44 |
max_lobur | screen -x | 11:44 |
max_lobur | ir-api screen | 11:44 |
Manishanker | test@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 |
Manishanker | I 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 IRC | 11:51 | |
*** romcheg has left #openstack-ironic | 11:54 | |
*** romcheg has joined #openstack-ironic | 11:55 | |
*** vkozhukalov has joined #openstack-ironic | 11:55 | |
*** romcheg has quit IRC | 12:08 | |
yuriyz | morning Ironic | 12:08 |
*** ekarlso has quit IRC | 12:09 | |
yuriyz | Manishanker: what is WSME version (pip show wsme)? | 12:09 |
Manishanker | 0.5b5 | 12:10 |
Manishanker | i just checked it | 12:10 |
yuriyz | should be 0.6 | 12:10 |
Manishanker | I will upgrade this and let you know | 12:10 |
openstackgerrit | Rohan Kanade proposed a change to openstack/ironic: Adds max retry limit to sync_power_state task https://review.openstack.org/77420 | 12:15 |
Manishanker | I 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/nov | 12:16 |
max_lobur | I always solving this by "python manage.py runserver your_ip:horizon_port (seems 6000)" | 12:18 |
max_lobur | from those screen horizon dir | 12:18 |
max_lobur | * or horizon dir | 12:18 |
max_lobur | also currently Ironic is not represented on horizon in neither way | 12:20 |
Manishanker | Hey max how do you provision the Baremetal instacnes ? | 12:25 |
max_lobur | there's a more complete guide | 12:26 |
max_lobur | the first one covers devstack installation only | 12:26 |
max_lobur | lemme find link | 12:26 |
Manishanker | okay | 12:26 |
*** romcheg has joined #openstack-ironic | 12:26 | |
max_lobur | https://etherpad.openstack.org/p/IronicDeployDevstack | 12:28 |
max_lobur | . | 12:29 |
Manishanker | Hey max this is the error when i try to login Horizon "Connection to neutron failed: Maximum attempts reached" | 12:29 |
max_lobur | I believe GUIDE 1 is ok | 12:29 |
Manishanker | So you are saying we cant use Ironic with Horizon ? | 12:30 |
Manishanker | okay i will try that :) | 12:30 |
max_lobur | there's nothing to do on horizon for Ironic :) | 12:30 |
romcheg | Morning everyone! | 12:31 |
Haomeng | morning all | 12:32 |
max_lobur | morning :) | 12:32 |
openstackgerrit | Roman Prykhodchenko proposed a change to openstack/ironic: Process public API list as regular expressions https://review.openstack.org/66621 | 12:36 |
romcheg | For some reason I lost this patch :) | 12:37 |
romcheg | Thought it was already merged :) | 12:37 |
*** saju_m has quit IRC | 12:40 | |
*** saju_m has joined #openstack-ironic | 12:51 | |
mdurnosvistov | Morning all! :) | 12:55 |
*** Alexei_9871 has joined #openstack-ironic | 12:59 | |
*** saju_m has quit IRC | 13:04 | |
*** dhellmann_ is now known as dhellmann | 13:11 | |
*** lsmola has quit IRC | 13:23 | |
*** ekarlso has joined #openstack-ironic | 13:23 | |
*** linggao has joined #openstack-ironic | 13:27 | |
*** dhellmann is now known as dhellmann_ | 13:36 | |
*** lsmola has joined #openstack-ironic | 13:36 | |
*** krtaylor has joined #openstack-ironic | 13:51 | |
*** nosnos has quit IRC | 13:56 | |
*** lsmola has quit IRC | 14:24 | |
devananda | morning, all | 14:29 |
openstackgerrit | A change was merged to openstack/ironic: Do not delete a Node which is not powered off https://review.openstack.org/76258 | 14:33 |
*** lsmola has joined #openstack-ironic | 14:33 | |
*** lsmola has quit IRC | 14:33 | |
*** lsmola has joined #openstack-ironic | 14:34 | |
mdurnosvistov | devananda: morning :) | 14:42 |
devananda | matty_dubs|gone: around? | 14:44 |
linggao | Morning devananda. | 14:49 |
*** tatyana has joined #openstack-ironic | 14:49 | |
*** lsmola has quit IRC | 14:50 | |
*** tatyana has quit IRC | 14:51 | |
NobodyCam | good morning Ironic | 14:57 |
GheRivero | morning Ironic | 15:00 |
pquerna | morning | 15:00 |
linggao | morning NobodyCam, GheRivero and all | 15:01 |
*** killer_prince has quit IRC | 15:13 | |
*** lazy_prince is now known as killer_prince | 15:13 | |
openstackgerrit | A change was merged to openstack/ironic: Python 3: replace "im_self" by "__self__" https://review.openstack.org/76517 | 15:14 |
romcheg | Morning devananda NobodyCam! | 15:18 |
romcheg | How is the meet up going? | 15:18 |
devananda | good! | 15:19 |
devananda | yesterday was focused on feature freeze crunch. | 15:19 |
devananda | aiming to get more integration with tripleo today | 15:20 |
NobodyCam | morning GheRivero devananda linggao pquerna and romcheg | 15:20 |
*** jgrimm has joined #openstack-ironic | 15:20 | |
linggao | devananda,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 |
NobodyCam | linggao: awesome and welcome back | 15:22 |
linggao | :) | 15:22 |
devananda | linggao: for the next month we will be focused on testing and stability of ironic, and writing documentation | 15:23 |
linggao | devananda, then shoud I work on bug fixing? | 15:25 |
devananda | linggao: sure :) | 15:25 |
NobodyCam | linggao: ya | 15:26 |
linggao | devananda, ok, I will have my advanture in the bug-land :-) | 15:27 |
linggao | devananda NobodyCam, if there are specific ones that you think I can work on, plesse let me know. | 15:28 |
*** lsmola has joined #openstack-ironic | 15:32 | |
NobodyCam | linggao: take a look at https://launchpad.net/ironic/+milestone/icehouse-3 | 15:35 |
NobodyCam | i'm not 100% sure this is complete https://bugs.launchpad.net/ironic/+bug/1264596 | 15:35 |
NobodyCam | IPMI set_boot_device is not called or tested | 15:36 |
linggao | NobodyCam, it is assgined to Michael Davies already? | 15:37 |
linggao | Looks like most of bugs have assignees already. | 15:37 |
NobodyCam | linggao: there is a patch or two up for that I seem to recall that they are nit complete however | 15:38 |
NobodyCam | thou I am still on first cup of coffee | 15:38 |
*** coolsvap has joined #openstack-ironic | 15:38 | |
linggao | lol | 15:38 |
* Shrews throws coffee at NobodyCam | 15:39 | |
NobodyCam | lol morning Shrews :) | 15:39 |
*** Alexei_9871 has quit IRC | 15:41 | |
devananda | coffee? what's that? | 15:41 |
NobodyCam | its that stuff I keep in the my IV drip. | 15:43 |
NobodyCam | :-p | 15:44 |
*** vkozhukalov has quit IRC | 15:45 | |
linggao | NobodyCam, I'll take a look at this bug in details. | 15:45 |
NobodyCam | Thank you linggao :) | 15:46 |
linggao | NobodayCam, I also need coffee or tea :) | 15:48 |
openstackgerrit | A change was merged to openstack/ironic: Implements SeaMicro VendorPassThru functionality https://review.openstack.org/70720 | 15:57 |
*** coolsvap1 has joined #openstack-ironic | 15:58 | |
NobodyCam | ok 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 traffic | 16:00 |
*** coolsvap has quit IRC | 16:00 | |
NobodyCam | devananda: fyi ^^^ | 16:01 |
devananda | hmm, i should go downstairs and have food/coffee .... | 16:01 |
devananda | NobodyCam: ack. see you soon! | 16:01 |
NobodyCam | :) | 16:01 |
*** dhellmann_ is now known as dhellmann | 16:12 | |
*** hemna has quit IRC | 16:31 | |
*** jistr has quit IRC | 16:31 | |
*** harlowja_away is now known as harlowja | 16:40 | |
*** harlowja has quit IRC | 16:41 | |
*** hemna has joined #openstack-ironic | 16:41 | |
openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Update Neutron if mac address of the port changed https://review.openstack.org/77939 | 16:55 |
*** harlowja has joined #openstack-ironic | 16:56 | |
openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Add TaskManager lock on change port data https://review.openstack.org/77623 | 17:01 |
*** wallnr has joined #openstack-ironic | 17:03 | |
*** jdob has joined #openstack-ironic | 17:06 | |
openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Update Neutron if mac address of the port changed https://review.openstack.org/77939 | 17:06 |
*** yuriyz has quit IRC | 17:07 | |
*** wallnr has quit IRC | 17:07 | |
jroll | mornin! | 17:09 |
*** lucasagomes has joined #openstack-ironic | 17:10 | |
*** romcheg has quit IRC | 17:11 | |
*** rwsu has joined #openstack-ironic | 17:14 | |
GheRivero | morning | 17:15 |
lucasagomes | morning GheRivero | 17:16 |
*** digambar has joined #openstack-ironic | 17:19 | |
JoshNang_ | https://etherpad.openstack.org/p/tripleo-icehouse-sprint | 17:21 |
*** derekh has joined #openstack-ironic | 17:23 | |
*** derekh has quit IRC | 17:23 | |
*** matty_dubs|gone is now known as matty_dubs|sfo | 17:27 | |
matty_dubs|sfo | devananda: Ha, sorry, I was sound asleep when you pinged me earlier. | 17:27 |
*** martyntaylor has quit IRC | 17:32 | |
*** derekh has joined #openstack-ironic | 17:32 | |
*** Manishanker has quit IRC | 17:52 | |
*** digambar has quit IRC | 17:53 | |
jroll | devananda: 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 |
jroll | writing up a blueprint right now | 17:54 |
*** rloo has joined #openstack-ironic | 17:55 | |
jroll | s/instance info class/instance info/ | 17:55 |
*** ndipanov has quit IRC | 18:00 | |
*** vkozhukalov has joined #openstack-ironic | 18:05 | |
NobodyCam | Morning Ironic | 18:08 |
matty_dubs|sfo | morning NobodyCam | 18:18 |
*** max_lobur has quit IRC | 18:21 | |
*** wallnr has joined #openstack-ironic | 18:29 | |
devananda | jroll: node.instance_info -- yes. need to move root_gb, swap_gb, ephemeral_gb, ephemeral_format, image_source | 18:29 |
*** wallnr has left #openstack-ironic | 18:32 | |
jroll | thanks | 18:42 |
pquerna | jroll: so, on this | 18:51 |
jroll | https://gist.github.com/jimrollenhagen/971d981648b15498d881 | 18:52 |
jroll | ^ instance_info blueprint draft | 18:52 |
pquerna | implies config drive is a base64 blob right? | 18:52 |
jroll | right | 18:52 |
jroll | with the thought of "this is how nova does this everywhere else" | 18:52 |
jroll | but I'm not tied to that by any means | 18:53 |
jroll | this field would not require a configdrive key | 18:53 |
pquerna | hrm, don't feel awesome about a base64 blob, inside a json blob, inside a string field :x | 18:53 |
jroll | because that is driver specific | 18:53 |
jroll | sure | 18:53 |
jroll | but that's a teeth agent driver discussion, not really an ironic core discussion, yeah? | 18:53 |
jroll | basically a driver can put any instance-specific data in this field | 18:53 |
jroll | and use it later when deploying | 18:54 |
russell_h | are some of these fields like required core fields, and some are teeth-agent specific? | 18:54 |
jroll | all the fields moving into that are pxe-driver specific | 18:54 |
jroll | they're currently in the driver_info field, but are specific to the instance | 18:54 |
russell_h | gotcha | 18:55 |
JoshNang_ | checked by the pxe-driver's validate()? | 18:55 |
jroll | this is coming out of a chat that devananda and I had yesterday about how to get instance info to deploy() | 18:55 |
russell_h | so comstud was saying that nova may require all drivers to implement configdrive | 18:55 |
JoshNang_ | as in, each driver can validate that info separately | 18:55 |
russell_h | which, to me, suggests that ironic must require all drivers to do so as well | 18:56 |
jroll | JoshNang_: correct | 18:56 |
jroll | hrm | 18:56 |
pquerna | russell_h: an interesting turn of events. | 18:56 |
russell_h | but I'm making at least 2 leaps there which should be validated | 18:56 |
jroll | russell_h: that sucks | 18:56 |
pquerna | russell_h: but nova doesn't today? | 18:56 |
pquerna | russell_h: i mean, if DefCore says ConfigDrive all the things. | 18:56 |
*** jistr has joined #openstack-ironic | 18:57 | |
jroll | russell_h: I know that today, for example, root_gb is only validated in the pxe driver | 18:57 |
jroll | oh, you mean ironic must require all drivers to implement configdrive | 18:59 |
*** athomas has quit IRC | 18:59 | |
jroll | hrm | 19:00 |
pquerna | well, given a 'configdrive' like thing, make an os think of it as a configdrive like thing. | 19:00 |
*** rloo has quit IRC | 19:00 | |
*** rloo has joined #openstack-ironic | 19:01 | |
pquerna | comstud: is DefCore the right reason for something like ConfigDrive being a requirement, or is there another driver. | 19:01 |
*** rloo has quit IRC | 19:05 | |
comstud | pquerna: I don't think 'configdrive' is actually a DefCore thing | 19:05 |
comstud | It was a decision made by nova only | 19:05 |
comstud | promoting the config drive extension to be core API | 19:06 |
comstud | in v3 (although there is likely not a v3 now for a while:) | 19:06 |
*** rloo has joined #openstack-ironic | 19:11 | |
jroll | pquerna: so, back to where we were./b 63 | 19:16 |
jroll | oops | 19:17 |
jroll | but yeah | 19:17 |
jroll | this field is kind of defined as arbitrary data for the driver to use | 19:17 |
jroll | like, an example JSON blob is good to have in the blueprint, but I don't think it should specify exactly what should be in there | 19:17 |
jroll | because that's up to the driver | 19:17 |
pquerna | hrm. | 19:23 |
*** romcheg has joined #openstack-ironic | 19:28 | |
openstackgerrit | Ghe Rivero proposed a change to openstack/ironic: Generic Multiplexed method https://review.openstack.org/73322 | 19:34 |
*** lsmola has quit IRC | 19:35 | |
*** vkozhukalov has quit IRC | 19:36 | |
*** rloo has quit IRC | 19:39 | |
*** martyntaylor has joined #openstack-ironic | 19:45 | |
*** rloo has joined #openstack-ironic | 19:48 | |
openstackgerrit | Rohan Kanade proposed a change to openstack/ironic: Fix wrong exception raised by conductor for node https://review.openstack.org/77426 | 20:04 |
*** martyntaylor has left #openstack-ironic | 20:19 | |
*** jistr has quit IRC | 20:20 | |
*** dwalleck has joined #openstack-ironic | 20:39 | |
*** jcooley_ has quit IRC | 20:43 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: deploy() to fail if image is larger than root partition https://review.openstack.org/78001 | 20:57 |
*** mrda_away is now known as mrda | 20:57 | |
jroll | devananda: 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 |
mrda | morning all | 20:58 |
NobodyCam | morning mrda | 20:58 |
mrda | sprint going well NobodyCam ? | 20:58 |
jroll | and 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 |
jroll | heya mrda | 20:59 |
mrda | hey jroll | 21:00 |
NobodyCam | hey hey lucasagomes how good are you with tox settings? | 21:10 |
lucasagomes | NobodyCam, ahn not that good, what's up? | 21:10 |
openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Add Ironic Nova driver https://review.openstack.org/78002 | 21:22 |
*** jcooley_ has joined #openstack-ironic | 21:23 | |
pquerna | comstud: devananda: https://review.openstack.org/#/c/77504/ <- adding neutron networks to metadata service. | 21:28 |
openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Add Ironic Nova driver https://review.openstack.org/78002 | 21:28 |
pquerna | actually sorry, thats just adding one more field. | 21:28 |
pquerna | https://review.openstack.org/#/c/77504/ | 21:29 |
pquerna | has adding network info | 21:29 |
pquerna | balh, confusing tabs and urls. | 21:29 |
pquerna | https://review.openstack.org/#/c/74002/ | 21:30 |
*** jcooley_ has quit IRC | 21:57 | |
*** rloo has quit IRC | 22:08 | |
*** linggao has quit IRC | 22:10 | |
*** dhellmann is now known as dhellmann_ | 22:18 | |
*** dwalleck has quit IRC | 22:36 | |
*** dwalleck has joined #openstack-ironic | 22:38 | |
devananda | mrda: are you working on https://bugs.launchpad.net/ironic/+bug/1285507 ? | 22:38 |
*** dwalleck_ has joined #openstack-ironic | 22:39 | |
NobodyCam | mrda: sorry I missed your question... Yes it is :) | 22:41 |
*** dwalleck has quit IRC | 22:43 | |
openstackgerrit | Sirushti Murugesan proposed a change to openstack/ironic: Adds support for deploying Disk Images https://review.openstack.org/73054 | 22:44 |
*** jcooley_ has joined #openstack-ironic | 22:44 | |
russell_h | jroll: https://gist.github.com/jimrollenhagen/971d981648b15498d881 <-- incidentally, I think I'm +1 to this | 22:54 |
jroll | russell_h: well, we talked during lunch about maybe not storing this in the db | 22:54 |
russell_h | jroll: Ironic should do the retry if a conductor dies | 22:54 |
jroll | depending if nova or ironic does the retries | 22:54 |
jroll | oh | 22:54 |
russell_h | jroll: oh... right | 22:54 |
jroll | welp | 22:54 |
russell_h | yeah, I think its gotta go in the db | 22:55 |
jroll | are you saying ironic currently does the retry, or should do the retry? | 22:55 |
russell_h | I think thats also OK... I think its reasonable to record the parameters an instance was built with | 22:55 |
jroll | yeah, fair | 22:55 |
*** blamar has joined #openstack-ironic | 22:56 | |
jroll | devananda: did you see my questions for you way in the scrollback? | 22:56 |
devananda | jroll: no | 22:56 |
jroll | devananda: 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 |
jroll | and 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 |
comstud | nova does retries right now | 22:59 |
comstud | if the virt driver spawn() raises | 22:59 |
comstud | it'll re-schedule | 22:59 |
jroll | comstud: assuming ironic causes spawn() to raise an exception | 23:00 |
comstud | I haven't looked.. I assume ironic virt driver issues create.. then polls to wait for it to be done | 23:01 |
comstud | or error | 23:01 |
comstud | if error.. it should raise so nova can retry | 23:01 |
*** dwalleck has joined #openstack-ironic | 23:03 | |
*** dwalleck_ has quit IRC | 23:03 | |
*** romcheg has quit IRC | 23:04 | |
*** rloo has joined #openstack-ironic | 23:10 | |
*** rloo has joined #openstack-ironic | 23:10 | |
jroll | devananda: here's the blueprint for the stuff we were just talking about: https://blueprints.launchpad.net/ironic/+spec/add-node-instance-info | 23:13 |
*** dwalleck has quit IRC | 23:19 | |
openstackgerrit | A change was merged to openstack/ironic: Fix wrong exception raised by conductor for node https://review.openstack.org/77426 | 23:32 |
openstackgerrit | A change was merged to openstack/ironic: Fix params order in assertEqual https://review.openstack.org/76077 | 23:32 |
*** blamar has quit IRC | 23:48 | |
*** blamar has joined #openstack-ironic | 23:49 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!