*** derekh has joined #openstack-ironic | 00:00 | |
*** ChuckC_ has quit IRC | 00:01 | |
*** Haomeng|2 has joined #openstack-ironic | 00:05 | |
lifeless | russell_h: so the conductors need to be subscribed to a queue that tells them when conductors join | 00:05 |
---|---|---|
lifeless | russell_h: and recalculate appropriately | 00:06 |
*** Haomeng has quit IRC | 00:06 | |
jroll | let's use zaqar | 00:07 |
* jroll ducks | 00:08 | |
greghaynes | russell_h: You wouldnt happen to still have your patch for making hashringmanager live across requests, would you? | 00:08 |
greghaynes | since it sounds like its now wanted | 00:08 |
*** derekh has quit IRC | 00:09 | |
russell_h | greghaynes: https://review.openstack.org/#/c/82915/ | 00:09 |
russell_h | it'll need some rebasing for sure | 00:09 |
russell_h | actually | 00:10 |
russell_h | https://review.openstack.org/#/c/82915/3/ironic/api/hooks.py | 00:10 |
russell_h | I guess thats the important part, although until the HashRingManager refreshes state (periodically, in response to events, or both) its probably undesireable | 00:11 |
greghaynes | wow, old patch is old | 00:11 |
*** chuckC has quit IRC | 00:11 | |
greghaynes | ok, so thats for rpcapi entirely | 00:12 |
russell_h | right | 00:12 |
russell_h | we could move that down some | 00:12 |
greghaynes | yea, I just want to break HashRingManager outside of rpcapi | 00:12 |
*** takadayuiko has joined #openstack-ironic | 00:17 | |
takadayuiko | Good morning, Ironic | 00:18 |
takadayuiko | rloo, are you here? | 00:18 |
rloo | hi takadayuiko | 00:18 |
takadayuiko | rloo, hi, thank you for giving me a message in Launchpad. I'll do it soon. | 00:20 |
rloo | takadayuiko: thx! | 00:20 |
takadayuiko | rloo, Is it night over there? morning? Anyway, I would be glad if you'll do review my patch, anytime OK. | 00:23 |
rloo | takadayuiko: it is almost 8:30pm here; I'm basically done for the rest of the week. Will look at your patch on Monday. | 00:23 |
rloo | takadayuiko: (unless others approve it before then!) | 00:24 |
takadayuiko | rloo, oh, it's 9:00am in Japan, opposite :) OK, OK. | 00:28 |
jroll | takadayuiko: what patch is your? | 00:28 |
jroll | yours* | 00:28 |
rloo | takadayuiko: ;) | 00:29 |
rloo | jroll: I asked takadayuiko if he was going to submit a patch to close this bug: https://bugs.launchpad.net/python-ironicclient/+bug/1310446 | 00:30 |
takadayuiko | jroll, hi :) I haven't post a new patch, yet. I'm writing it now, so I'll post it soon! | 00:30 |
jroll | takadayuiko: I'm just curious :) | 00:36 |
jroll | ok | 00:36 |
*** smoriya has joined #openstack-ironic | 00:37 | |
takadayuiko | ]jroll, thank you, don't miss it! | 00:39 |
openstackgerrit | greghaynes proposed a change to openstack/ironic: Make HashRings Singleton https://review.openstack.org/120951 | 00:54 |
greghaynes | russell_h: ^ | 00:55 |
*** chuckC has joined #openstack-ironic | 00:56 | |
NobodyCam | rloo: want to give https://wiki.openstack.org/wiki/Ironic/NovaBaremetalIronicMigration a read over if you have a free minutes or two? | 01:01 |
rloo | NobodyCam: I'll take a look in a few minutes | 01:02 |
openstackgerrit | Yuiko Takada proposed a change to openstack/python-ironicclient: Add 'rebuild' option to node-set-provision-state https://review.openstack.org/120953 | 01:05 |
openstackgerrit | Yuiko Takada proposed a change to openstack/python-ironicclient: Add unit tests for "ironic node-delete" shell cmd https://review.openstack.org/120710 | 01:13 |
openstackgerrit | Yuiko Takada proposed a change to openstack/python-ironicclient: Add unit tests for "ironic node-create" shell cmd https://review.openstack.org/120312 | 01:14 |
*** shakamunyi has joined #openstack-ironic | 01:16 | |
*** chenglch has joined #openstack-ironic | 01:22 | |
*** shakamunyi has quit IRC | 01:24 | |
*** shakamunyi has joined #openstack-ironic | 01:31 | |
rloo | hi NobodyCam, just looking at the migration wiki. | 01:35 |
rloo | NobodyCam: I think it is better to block off some time on Monday, to give feedback. (Or is there some way to annotate on wikis?) | 01:36 |
*** Haomeng|2 has quit IRC | 01:38 | |
*** pcrews has quit IRC | 01:39 | |
*** Haomeng has joined #openstack-ironic | 01:39 | |
*** nosnos has joined #openstack-ironic | 01:46 | |
*** takadayuiko has quit IRC | 02:00 | |
*** pcrews has joined #openstack-ironic | 02:05 | |
*** rloo has quit IRC | 02:08 | |
*** pcrews has quit IRC | 02:16 | |
*** jjmanzer has joined #openstack-ironic | 02:19 | |
*** killer_prince is now known as lazy_prince | 02:24 | |
*** ramineni has joined #openstack-ironic | 02:45 | |
*** nosnos has quit IRC | 02:53 | |
*** nosnos has joined #openstack-ironic | 02:53 | |
*** harlowja is now known as harlowja_away | 02:56 | |
*** nosnos has quit IRC | 02:58 | |
*** rushiagr_away is now known as rushiagr | 03:07 | |
*** shakamunyi has quit IRC | 03:22 | |
*** jjmanzer has quit IRC | 03:26 | |
*** rushiagr is now known as rushiagr_away | 03:28 | |
*** nikunj2512 has joined #openstack-ironic | 04:00 | |
*** lazy_prince is now known as killer_prince | 04:02 | |
openstackgerrit | greghaynes proposed a change to openstack/ironic: Make HashRings Singleton https://review.openstack.org/120951 | 04:08 |
*** stendulker has joined #openstack-ironic | 04:09 | |
*** rushiagr_away is now known as rushiagr | 04:12 | |
*** nosnos has joined #openstack-ironic | 04:13 | |
*** todd_dsm has quit IRC | 04:14 | |
*** Poornima has joined #openstack-ironic | 04:46 | |
openstackgerrit | Syed Ismail Faizan Barmawer proposed a change to openstack/ironic: Add uefi boot mode support in IloVirtualMediaIscsiDeploy https://review.openstack.org/116561 | 04:49 |
*** Nisha has joined #openstack-ironic | 04:53 | |
*** HenryG is now known as HenryG_afk | 04:55 | |
*** yuanying_ has joined #openstack-ironic | 04:56 | |
*** yuanying_ has quit IRC | 04:56 | |
*** yuanying_ has joined #openstack-ironic | 04:58 | |
*** rakesh_hs has joined #openstack-ironic | 04:59 | |
*** yuanying has quit IRC | 05:00 | |
Nisha | devananda: hi | 05:00 |
*** nikunj2512 has quit IRC | 05:03 | |
*** HenryG_afk has quit IRC | 05:04 | |
*** nikunj2512 has joined #openstack-ironic | 05:10 | |
openstackgerrit | Syed Ismail Faizan Barmawer proposed a change to openstack/ironic: Support for setting boot mode in pxe_ilo driver https://review.openstack.org/118578 | 05:20 |
*** HenryG has joined #openstack-ironic | 05:23 | |
*** killer_prince is now known as lazy_prince | 05:24 | |
*** HenryG is now known as HenryG_afk | 05:24 | |
*** dlaube has joined #openstack-ironic | 05:25 | |
*** pensu has joined #openstack-ironic | 05:35 | |
*** lazy_prince is now known as killer_prince | 05:43 | |
openstackgerrit | Syed Ismail Faizan Barmawer proposed a change to openstack/ironic: Add uefi boot mode support in IloVirtualMediaIscsiDeploy https://review.openstack.org/116561 | 05:48 |
openstackgerrit | greghaynes proposed a change to openstack/ironic: Make HashRings Singleton https://review.openstack.org/120951 | 05:48 |
openstackgerrit | Syed Ismail Faizan Barmawer proposed a change to openstack/ironic: Support for setting boot mode in pxe_ilo driver https://review.openstack.org/118578 | 05:49 |
openstackgerrit | greghaynes proposed a change to openstack/ironic: Make HashRings Singleton https://review.openstack.org/120951 | 05:49 |
openstackgerrit | greghaynes proposed a change to openstack/ironic: Make HashRings Singleton https://review.openstack.org/120951 | 05:52 |
*** nikunj2513 has joined #openstack-ironic | 05:58 | |
*** killer_prince is now known as lazy_prince | 05:59 | |
*** nikunj2512 has quit IRC | 05:59 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/120701 | 06:06 |
*** dlaube has quit IRC | 06:26 | |
*** nikunj2514 has joined #openstack-ironic | 06:33 | |
openstackgerrit | Yuiko Takada proposed a change to openstack/python-ironicclient: Add unit test for "ironic node-update" shell cmd https://review.openstack.org/120989 | 06:33 |
*** rakesh_hs has quit IRC | 06:35 | |
*** nikunj2513 has quit IRC | 06:35 | |
*** rakesh_hs has joined #openstack-ironic | 06:36 | |
*** jcoufal has joined #openstack-ironic | 06:39 | |
*** nikunj2514 is now known as nikunj2512 | 06:40 | |
*** k4n0 has joined #openstack-ironic | 06:45 | |
openstackgerrit | Yuiko Takada proposed a change to openstack/python-ironicclient: Add unit test for "ironi port-update" shell cmd https://review.openstack.org/120997 | 07:03 |
openstackgerrit | Yuiko Takada proposed a change to openstack/python-ironicclient: Add unit test for "ironic port-update" shell cmd https://review.openstack.org/120997 | 07:11 |
*** dtantsur|afk is now known as dtantsur | 07:20 | |
dtantsur | Morning Ironic | 07:20 |
*** bluex-pl has joined #openstack-ironic | 07:21 | |
Nisha | Morning dtantsur | 07:27 |
dtantsur | Nisha, morning | 07:27 |
Nisha | looking for reviews for https://review.openstack.org/116561 and https://review.openstack.org/118578 | 07:27 |
Nisha | :) | 07:27 |
*** romcheg has joined #openstack-ironic | 07:27 | |
romcheg | Good morning Ironic! | 07:28 |
Nisha | dtantsur: ^^^ | 07:28 |
dtantsur | Nisha, yeah, I remember, gimme some time please | 07:28 |
dtantsur | romcheg, morning | 07:28 |
*** early has quit IRC | 07:29 | |
*** ifarkas has joined #openstack-ironic | 07:32 | |
*** early has joined #openstack-ironic | 07:32 | |
Nisha | dtantsur: sure. | 07:32 |
*** bluex-pl has quit IRC | 07:34 | |
*** bluex-pl has joined #openstack-ironic | 07:35 | |
openstackgerrit | Yuiko Takada proposed a change to openstack/python-ironicclient: Add unit tests for "ironic node-show" shell cmd https://review.openstack.org/120001 | 07:41 |
*** jistr has joined #openstack-ironic | 07:47 | |
*** stendulker has quit IRC | 07:52 | |
*** nikunj2512 has quit IRC | 07:53 | |
*** bluex-pl has quit IRC | 07:53 | |
openstackgerrit | Yuiko Takada proposed a change to openstack/python-ironicclient: Add unit test for "ironic node-update" shell cmd https://review.openstack.org/120989 | 07:54 |
*** derekh has joined #openstack-ironic | 07:57 | |
*** ndipanov_gone is now known as ndipanoff | 07:58 | |
openstackgerrit | Yuiko Takada proposed a change to openstack/python-ironicclient: Add unit test for "ironic port-update" shell cmd https://review.openstack.org/120997 | 08:00 |
Haomeng | Nisha: morning:) | 08:03 |
Haomeng | Nisha: I have comments with the patch - https://review.openstack.org/#/c/118578/ | 08:04 |
Nisha | morning Haomeng | 08:04 |
Nisha | Haomeng: tell me | 08:04 |
Haomeng | Nisha: just think about if our IPMIManagement.get_sensor_data() works with ilo systems, I have no such ilo systems, can not do the testing | 08:04 |
Haomeng | Nisha: I understand the 'ipmitool' command can run on ilo system, right? | 08:05 |
Nisha | Haomeng: yes ipmitool can be run on ilo systems | 08:06 |
Haomeng | Nisha: ok, can you help to paste the command output if you have the ilo system and have time - "ipmitool -I lanplus -H $bmc -U USERID -P PASSW0RD sdr -v", thank you. | 08:07 |
Haomeng | Nisha: I am not sure if my code work with the ilo system:) | 08:07 |
Nisha | Haomeng: it works on the ilo systems also with the same syntax. | 08:08 |
Haomeng | Nisha: ok, that is fine:) | 08:09 |
Haomeng | Nisha: thanks for your confirmation:) | 08:09 |
Nisha | Haomeng: thanks | 08:09 |
*** nikunj2512 has joined #openstack-ironic | 08:09 | |
Haomeng | Nisha: :) | 08:10 |
*** MattMan has quit IRC | 08:12 | |
*** rakesh_hs2 has joined #openstack-ironic | 08:12 | |
*** MattMan has joined #openstack-ironic | 08:13 | |
*** rakesh_hs has quit IRC | 08:14 | |
*** yuanying_ has quit IRC | 08:15 | |
*** dtantsur is now known as dtantsur|bbl | 08:15 | |
*** yuanying has joined #openstack-ironic | 08:17 | |
*** andreykurilin has joined #openstack-ironic | 08:19 | |
*** lucasagomes has joined #openstack-ironic | 08:29 | |
Nisha | lucasagomes: morning | 08:30 |
openstackgerrit | A change was merged to openstack/ironic: Remove bypassing of H302 for gettextutils markers https://review.openstack.org/120910 | 08:39 |
Nisha | lucasagomes: looking for reviews for https://review.openstack.org/116561 and https://review.openstack.org/118578 | 08:41 |
*** igordcard has joined #openstack-ironic | 08:41 | |
lucasagomes | Nisha, morning, sure I will review it :) just trying to wake up here heh | 08:42 |
Nisha | lucasagomes: :) | 08:43 |
*** aswadr has joined #openstack-ironic | 08:43 | |
*** rushiagr is now known as rushiagr_away | 09:01 | |
*** athomas has joined #openstack-ironic | 09:09 | |
*** dnikishov has joined #openstack-ironic | 09:11 | |
*** rushiagr_away is now known as rushiagr | 09:13 | |
*** teju has joined #openstack-ironic | 09:16 | |
teju | hi all... ironic deploy failed....logs : http://pastebin.com/uQCMMa8S | 09:18 |
*** yuanying has quit IRC | 09:21 | |
*** pelix has joined #openstack-ironic | 09:28 | |
*** teju has quit IRC | 09:31 | |
*** chenglch has quit IRC | 09:57 | |
*** teju has joined #openstack-ironic | 10:18 | |
*** yuanying has joined #openstack-ironic | 10:19 | |
teju | i see "failed to detect root device UUID " in ironic conductor logs....how to fix this? | 10:19 |
*** andreykurilin has quit IRC | 10:21 | |
*** faizan has joined #openstack-ironic | 10:32 | |
*** rakesh_hs2 has quit IRC | 10:38 | |
*** Nisha has quit IRC | 10:43 | |
*** stendulker has joined #openstack-ironic | 10:43 | |
*** nikunj2512 has quit IRC | 10:44 | |
*** nosnos has quit IRC | 10:44 | |
*** nosnos has joined #openstack-ironic | 10:45 | |
*** Poornima has quit IRC | 10:48 | |
*** nosnos has quit IRC | 10:49 | |
*** rushiagr is now known as rushiagr_away | 10:56 | |
*** jcoufal has quit IRC | 11:00 | |
*** ramineni has quit IRC | 11:05 | |
teju | how to build images for ironic bare metal deployment? | 11:05 |
Haomeng | teju: run dib | 11:07 |
Haomeng | teju: let me find the document for you | 11:07 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Don't write python bytecode while testing https://review.openstack.org/121050 | 11:07 |
teju | Haomeng : ok....pls provide link if possible | 11:08 |
Haomeng | teju: yes | 11:08 |
teju | Haomeng : r u familiar with the error "Failed to detect root device UUID" in ironic conductor logs? | 11:09 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Don't write python bytecode while testing https://review.openstack.org/121050 | 11:09 |
Haomeng | teju: I guess it is root paration uuid | 11:10 |
lucasagomes | teju, https://github.com/openstack/diskimage-builder | 11:11 |
lucasagomes | there's a nice readme there about how to create an image | 11:11 |
teju | lucasagomes : thanks... | 11:12 |
teju | lucasagomes : i built images using disk image builder (link : http://ma.ttwagner.com/bare-metal-deploys-with-devstack-and-ironic/ ).....but, i got error "Failed to detect root device UUID" in ironic conductor logs....can this be because of the image? | 11:13 |
Haomeng | teju: you should install https://github.com/openstack/diskimage-builder | 11:19 |
Haomeng | teju: I guess you use wrong image to deploy to baremetal | 11:20 |
lucasagomes | teju, hmm very unlikely | 11:21 |
Haomeng | teju: what arguments when you run disk-image-creater ? | 11:21 |
teju | Haomeng : bin/ramdisk-image-create -a amd64 fedora deploy-ironic -o /tmp/deploy-ramdisk | 11:21 |
*** stendulker has quit IRC | 11:21 | |
Haomeng | teju: looks fine | 11:22 |
*** yuanying has quit IRC | 11:22 | |
Haomeng | teju: how about the baremetal console output? | 11:23 |
Haomeng | Haomeng: where is blocking at? | 11:23 |
teju | Haomeng : on baremetal console , i see 1. IP is assgined 2. images are fetched from tftp server ....and at some later point, the node is powered off | 11:25 |
Haomeng | teju: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/deploy_utils.py#L326 | 11:26 |
Haomeng | fail at block_uuid | 11:27 |
Haomeng | teju: can you paste more logs around "Failed to detect root device UUID" with this url http://paste.openstack.org/ ? | 11:28 |
Haomeng | teju: we have to check the 'blkid' command call output | 11:28 |
teju | Haomeng : http://pastebin.com/JbB5mPrZ | 11:28 |
teju | Haomeng : u r right...that command failed with status 2 | 11:29 |
Haomeng | teju: thanks, let me check the context | 11:29 |
Haomeng | yes this is root cause | 11:29 |
Haomeng | can not get the root uuid | 11:29 |
teju | how can i fix this? | 11:30 |
Haomeng | teju: let me what means for the error code returned with 2 | 11:30 |
Haomeng | it is - If the specified token was not found, or no (specified) devices could be identified, an exit code of 2 is returned. | 11:31 |
Haomeng | teju: it can be recreated for each time? or just one time? | 11:33 |
teju | Haomeng : i got the same error 2 times | 11:33 |
Haomeng | teju: not stable? | 11:33 |
Haomeng | teju: or for each time? | 11:33 |
Haomeng | teju: is your network stable during the testing? | 11:34 |
teju | Haomeng : yes, my network is stable during deployment | 11:34 |
Haomeng | teju: because the disk operation is over iscsi | 11:34 |
Haomeng | teju: can you run "blkid -s UUID" from your conductor machine by manually? | 11:35 |
*** dtantsur|bbl is now known as dtantsur | 11:35 | |
teju | Haomeng : yes... | 11:36 |
Haomeng | teju: dont care, this is *readonly* operation | 11:36 |
Haomeng | to check if it works with return code 0 - echo $? | 11:36 |
teju | Haomeng : blkid -s UUID is working fine | 11:37 |
Haomeng | teju: ok, that means at that time, remote iscsi disk is offline | 11:37 |
teju | Haomeng : so, the disk in the bare metal node is offline? | 11:38 |
Haomeng | teju: should be related with iscsi I think | 11:38 |
Haomeng | teju: the command issued by ironic is - "blkid -s UUID -o value /dev/disk/by-path/ip-192.168.10.30:3260-iscsi-iqn-266ddb3e-1673-4b98-a178-81e1f1f9f482-lun-1-part1" | 11:39 |
Haomeng | teju: the last argument is the iscsi disk | 11:39 |
Haomeng | teju: we got return code 2, that means blkid can not find the disk at all | 11:40 |
teju | Haomeng : yes.....and it failed to export iscsi disk...right? | 11:40 |
Haomeng | teju: I think so | 11:40 |
teju | Haomeng : may be because of corrupt disk? | 11:41 |
Haomeng | teju: however it is diffcult to check the log from baremetal side, it is handled by our agent | 11:41 |
Haomeng | teju: have to kindly check the machine console output | 11:42 |
teju | Haomeng : bare metal console output? | 11:42 |
teju | ok | 11:42 |
Haomeng | teju: yes | 11:42 |
Haomeng | teju: did you run successfuly with same case? | 11:42 |
Haomeng | teju: or failed always? | 11:43 |
teju | Haomeng : failed always | 11:43 |
*** lucasagomes is now known as lucas-hungry | 11:43 | |
Haomeng | teju: Unfortunately:) | 11:43 |
teju | Haomeng: i will try to try again by replacing the disk in the bare metal node | 11:45 |
Haomeng | teju: I guess, but we have no agent side log to see why remote iscsi disk is offline | 11:45 |
Haomeng | teju: do you have qq number? | 11:46 |
Haomeng | teju: let me chat wich Chinese:) | 11:46 |
Haomeng | teju: one question | 11:46 |
Haomeng | teju: what is image you are using to deploy to bm? | 11:47 |
Haomeng | teju: ramdisk-image-create is used to create ramdisk/kernel for deploy tool agent | 11:47 |
*** ramineni has joined #openstack-ironic | 11:49 | |
lazy_prince | 2014-09-12 17:17:32.642 TRACE ironic.openstack.common.threadgroup DriverLoadError: Driver fake could not be loaded. Reason: name 'i18n' is not defined. | 11:49 |
lazy_prince | any one seen above error..? | 11:49 |
lazy_prince | This is with devstack.. | 11:50 |
Haomeng | teju: you should run "disk-image-create" to create the *user* image which will deploy to bm | 11:50 |
teju | Haomeng : http://ma.ttwagner.com/bare-metal-deploys-with-devstack-and-ironic/ | 11:51 |
teju | Haomeng : i installed ironic in ubuntu 14.04, but i followed the instructions in the above link to build images and deploy | 11:52 |
Haomeng | teju: let me check the guide, looks cool | 11:52 |
teju | Haomeng : ok.. | 11:52 |
dtantsur | teju, I hope you're aware that this instruction can be seriusly outdated? | 11:52 |
dtantsur | It's written when Ironic was different | 11:53 |
Haomeng | teju: yes, it is Posted on January 30, 2014 | 11:54 |
Haomeng | teju: can you run devstack to setup the ironic env? | 11:54 |
teju | Haomeng : no | 11:55 |
Haomeng | teju: agree with dtantsur, our code changed a lot from Jan | 11:55 |
dtantsur | I guess I should ask Matt to drop this article... | 11:56 |
Haomeng | dtantsur: +1 | 11:56 |
Haomeng | dtantsur: :) | 11:56 |
Haomeng | teju: for this step - "ironic node-create" | 11:56 |
Haomeng | teju: how do you provide the argument for "ironic node-create" and "-i pxe_image_source" | 11:56 |
teju | dtantsur : i gave the id of cirros image for pxe_image_source | 11:57 |
teju | Haomeng : i gave the id of cirros image for pxe_image_source | 11:57 |
Haomeng | teju: the image should be created by "disk-image-create" | 11:57 |
teju | Haomeng : so, how should this command be modified? ramdisk-image-create -a amd64 fedora deploy-ironic -o /tmp/deploy-ramdisk | 11:58 |
Haomeng | teju: where the cirros image you get? | 11:58 |
*** rushiagr_away is now known as rushiagr | 11:58 | |
Haomeng | teju: default cirros image is for vm not bm | 11:59 |
Haomeng | teju: the image ironic can be used to deploy to bm is created by "disk-image-create" not "ramdisk-image-create" | 11:59 |
Haomeng | teju: so we have the 'deploy' image and 'user' image | 12:00 |
Haomeng | teju: deploy image is created by ramdisk-image-create | 12:00 |
Haomeng | teju: user image is created by disk-image-create | 12:00 |
Haomeng | teju: do you think if I am clear or not? | 12:00 |
Haomeng | teju: and I dont think the guide is good reference, can you check with devstack ironic scripts? | 12:02 |
Haomeng | teju: this is official reference - https://github.com/openstack-dev/devstack/blob/master/lib/ironic | 12:03 |
teju | Haomeng : ok....if possible, can u provide the commands u used to create images | 12:03 |
Haomeng | teju: sure | 12:04 |
teju | Haomeng : is cirros image available for bm? | 12:04 |
Haomeng | teju: dont think | 12:04 |
Haomeng | teju: in case our dib supports cirros :) | 12:04 |
Haomeng | teju: sorry | 12:05 |
*** rushiagr is now known as rushiagr_away | 12:05 | |
Haomeng | teju: devstack support cirros bm image I think | 12:05 |
Haomeng | teju: let me find the command line to create bm image | 12:05 |
Haomeng | teju: try with this command " disk-image-create -a amd64 -o ubuntu-amd64 ubuntu" | 12:08 |
Haomeng | teju: this can create the ubuntu amd 64 bm image | 12:08 |
Haomeng | teju: can you have a try | 12:08 |
teju | Haomeng: ok... | 12:09 |
Haomeng | teju: but this will be long time to create the ubuntu image | 12:09 |
Haomeng | teju: let me help you find the cirros bm image | 12:09 |
teju | Haomeng : ok...after that, I should register with glance? | 12:09 |
Haomeng | teju: if you run devstack, you will get cirros-0.3.2-x86_64-uec cirros-0.3.2-x86_64-uec-kernel cirros-0.3.2-x86_64-uec-ramdisk three images | 12:10 |
Haomeng | teju: let me check these image's url | 12:10 |
Haomeng | teju: we can download them directly I think | 12:10 |
*** rushiagr_away is now known as rushiagr | 12:12 | |
teju | Haomeng : ok....direct links to download cirros would be great | 12:12 |
Haomeng | teju: https://launchpad.net/cirros/+download | 12:15 |
Haomeng | teju: this one - cirros-0.3.0-i386-uec.tar.gz (md5, sig) | 12:16 |
Haomeng | teju: including the uec style tarball (kernel, ramdisk, rootfs.img) for i386 | 12:16 |
Haomeng | teju: you have to import them into glance, and set rootfs image ext parameters to point kernel image and ramdisk image | 12:17 |
Haomeng | teju: totaly 5 images are required for our ironic bm provision | 12:18 |
teju | Haomeng :i extracted the tar.gz and it has 3 files : cirros-0.3.0-i386-blank.img cirros-0.3.0-i386-initrd cirros-0.3.0-i386-vmlinuz | 12:19 |
*** yuanying has joined #openstack-ironic | 12:19 | |
Haomeng | teju: devstack is download them from - http://download.cirros-cloud.net/0.3.2/cirros-0.3.2-x86_64-uec.tar.gz | 12:20 |
Haomeng | teju: yes | 12:21 |
Haomeng | teju: they are - kernel, ramdisk, rootfs.img | 12:22 |
teju | Haomeng : now, I have 3 images....u said I need 5 images... | 12:24 |
Haomeng | teju: yes | 12:24 |
Haomeng | teju: two deploy images | 12:24 |
Haomeng | teju: three *user* images | 12:24 |
Haomeng | teju: did you run latest ironic code? | 12:25 |
Haomeng | teju: we have new agent | 12:25 |
Haomeng | teju: so suggest you run devstack to have a first try | 12:25 |
Haomeng | teju: and then run these steps by manually | 12:25 |
teju | Haomeng : i cant install devstack now...i already have installed all openstack services by following the standard docs in Ubuntu 14.04 | 12:26 |
Haomeng | teju: I mean you install devstack from another machine, then we have the reference between the devstack env and manually env | 12:27 |
Haomeng | teju: have a reference to check the difference | 12:28 |
Haomeng | teju: without devstack, all the steps are done by manually, so that is diffcult to run all these commands with correct arguments and order | 12:28 |
teju | Haomeng : ok....cirros-0.3.0-i386-blank.img cirros-0.3.0-i386-initrd cirros-0.3.0-i386-vmlinuz..... these 3 are user images? | 12:28 |
Haomeng | teju: yes | 12:29 |
teju | Haomeng : now, how to get deploy images? | 12:29 |
Haomeng | teju: the deploy image is created by ramdisk-image-create with arguments *deploy-ironic* | 12:30 |
Haomeng | teju: if you followed http://ma.ttwagner.com/bare-metal-deploys-with-devstack-and-ironic/ section "Build some images" | 12:31 |
dtantsur | fyi I asked Matt to delete this page | 12:31 |
Haomeng | teju: but dont suggest you take this as a guide, because it is outofdated | 12:31 |
Haomeng | dtantsur: :) | 12:32 |
Haomeng | dtantsur: +1 | 12:32 |
*** dprince has joined #openstack-ironic | 12:32 | |
teju | Haomeng : ok... | 12:32 |
teju | Haomeng : one last question...what is the corresponding ramdisk-image-create command for the 3 cirros user images i just extracted from cirros-0.3.0-i386-uec.tar.gz? | 12:34 |
Haomeng | teju: devstack ironic scripts are openstack official deploy tool | 12:34 |
Haomeng | teju: I dont think ramdisk-image-create support cirros image creation, let me check the dib elements | 12:35 |
Haomeng | teju: dib supports fedora,ubuntu,centos - https://github.com/openstack/diskimage-builder/tree/master/elements | 12:37 |
Haomeng | teju: can not find the cirros | 12:38 |
Haomeng | teju: cirros is not *production* os for customer | 12:38 |
Haomeng | teju: for example, you can run "disk-image-create -a amd64 -o ubuntu-amd64 ubuntu" to create ubuntu image | 12:40 |
teju | Haomeng : ok.. | 12:40 |
Haomeng | teju: good luck and nice weekend:) | 12:41 |
teju | Haomeng : thanks for ur time... have a great weekend | 12:43 |
Haomeng | night ironic:) | 12:43 |
*** lucas-hungry is now known as lucasagomes | 12:43 | |
Haomeng | teju: a ironic reference - http://www.slideshare.net/enigmadragon/ironic - hope this help you to understand ironic concepts | 12:43 |
Haomeng | teju: bye | 12:43 |
Haomeng | teju: any time:) | 12:43 |
teju | Haomeng : bye...talk to u on Monday | 12:44 |
Haomeng | teju: you can add my qq number if you want | 12:44 |
*** HenryG_afk is now known as HenryG | 12:51 | |
teju | Haomeng : what's qq number? | 12:51 |
*** yuanying has quit IRC | 12:52 | |
*** dlaube has joined #openstack-ironic | 13:07 | |
*** jjmanzer has joined #openstack-ironic | 13:20 | |
faizan | Hi Lucas, | 13:29 |
faizan | Lucas, Request you to take a look at https://review.openstack.org/116561 and https://review.openstack.org/118578 | 13:30 |
*** smoriya has quit IRC | 13:37 | |
*** pensu has quit IRC | 13:39 | |
lucasagomes | faizan, ack, looking now | 13:47 |
faizan | lucasagomes, thanks | 13:48 |
*** jjmanzer has quit IRC | 13:48 | |
*** yuanying has joined #openstack-ironic | 13:49 | |
*** Nisha has joined #openstack-ironic | 13:51 | |
lucasagomes | faizan, we won't need the mkisofs thing there? | 13:55 |
lucasagomes | faizan, forget just saw ur comment son patch-set 14 | 13:56 |
lucasagomes | faizan, +a both of them :) | 14:00 |
lucasagomes | def removing the efi ISO creation made things much more easier there | 14:00 |
lucasagomes | we can work on introducing the generation of the efi iso in Ironic later once we are more confident about the process to do it | 14:01 |
faizan | lucasagomes, things were getting complicated in the code, | 14:01 |
lucasagomes | yeah | 14:02 |
lucasagomes | it was always trying to generate the efi iso even if the boot mode was bios | 14:02 |
lucasagomes | and the mkisofs version was also a hurdle | 14:02 |
*** stendulker has joined #openstack-ironic | 14:02 | |
faizan | anyway, as part of documentation I will try to give the steps to create efi iso and later we can merge it in the code | 14:02 |
lucasagomes | faizan, +1! | 14:03 |
lucasagomes | awesome | 14:03 |
lucasagomes | faizan, there's any other patch for that uefi blueprint or we are all done with code? | 14:03 |
faizan | There is one more patch - https://review.openstack.org/118578 | 14:03 |
lucasagomes | faizan, already reviewed that | 14:04 |
lucasagomes | (reviewed both patches you pointed me to) | 14:04 |
faizan | lucasagomes, just saw you approval, thanks | 14:04 |
lucasagomes | yeah I don't see anything in the queue as well :) | 14:04 |
lucasagomes | so I think uefi is done! | 14:04 |
lucasagomes | faizan, thanks for the hard working | 14:05 |
*** r-daneel has joined #openstack-ironic | 14:05 | |
faizan | yes, thats it, we can call it done for juno | 14:05 |
lucasagomes | faizan, w00t | 14:05 |
* lucasagomes run and grab a beer | 14:06 | |
*** stendulker_ has joined #openstack-ironic | 14:06 | |
faizan | lucasagomes, I think I need to get approval from one more core-reviewer for these patches. | 14:06 |
lucasagomes | faizan, both has 2 +2 already | 14:07 |
faizan | as these patches are in FFE | 14:07 |
lucasagomes | oh do we have that rule!? | 14:07 |
lucasagomes | well I can cancel the approval until we sort that out | 14:07 |
lucasagomes | dtantsur, ^ | 14:08 |
faizan | I dont know, need to check with devananda | 14:08 |
lucasagomes | faizan, ack, I revoked the approval | 14:09 |
lucasagomes | I will investigate and if not I re-approve it asap :) | 14:09 |
lucasagomes | faizan, cheers | 14:09 |
faizan | lucasagomes, thats fine. Lets check it first and then merge. Thanks | 14:10 |
lucasagomes | faizan, yup | 14:10 |
*** stendulker has quit IRC | 14:11 | |
dtantsur | lucasagomes, never heard about it, but it won't harm to give folks ability to have one last look | 14:11 |
lucasagomes | yeah, I just revoked it | 14:11 |
*** stendulker has joined #openstack-ironic | 14:11 | |
lucasagomes | trying to find if we said that somewhere | 14:11 |
faizan | here is the comment from rloo while moving these patches to ffe | 14:14 |
faizan | 2014-09-04T16:03:45 <rloo> hey, we need 3 cores for ffe :-) | 14:14 |
lucasagomes | faizan, oh right... I think is something else | 14:15 |
lucasagomes | faizan, because in nova, in order to a blueprint to go to FFE 3 cores has to sign up as "sponsors" of that blueprint | 14:15 |
lucasagomes | and we had to find cores over there to sponsor our nova ironic driver blueprint in nova | 14:15 |
lucasagomes | in ironic uefi was the only one in ffe so we didn't even needed to do that ^ I think rloo was joking there | 14:16 |
*** romcheg has quit IRC | 14:16 | |
*** yuanying_ has joined #openstack-ironic | 14:16 | |
faizan | oh, I got it wrong then, since I was reading it in the uefi context. | 14:17 |
faizan | lucasagomes, thanks for clarification. | 14:17 |
*** yuanying has quit IRC | 14:19 | |
*** pcrews has joined #openstack-ironic | 14:19 | |
*** yuanying_ has quit IRC | 14:22 | |
*** dlaube has quit IRC | 14:22 | |
NobodyCam | good morrning Ironic .. TGIF | 14:24 |
dtantsur | NobodyCam, morning) | 14:24 |
NobodyCam | Thank you for the comments on 120433 lucasagomes | 14:24 |
NobodyCam | morning dtantsur | 14:24 |
lucasagomes | NobodyCam, no bothers, can you add that comment and simplify that function please? | 14:25 |
NobodyCam | will do.. | 14:25 |
lucasagomes | thanks | 14:26 |
lucasagomes | yeah just say that the CONF.compute_driver option needs to be present in the API configuration in order to the proxy to work | 14:26 |
lucasagomes | NobodyCam, as a NOTE :) | 14:27 |
lucasagomes | NobodyCam, and morning :D | 14:27 |
*** rwsu has quit IRC | 14:28 | |
*** teju has left #openstack-ironic | 14:32 | |
Nisha | lucasagomes: I see verified marked as cross for faizan's one patch https://review.openstack.org/#/c/118578/. Did you revoked the approval as discussed above ? | 14:46 |
lucasagomes | Nisha, yup... but I will re-approve it just giving people a chance to take a look at it | 14:46 |
lucasagomes | if nobody does soonish I will just re-approve it | 14:47 |
Nisha | lucasagomes: ok thanks | 14:47 |
*** athomas has quit IRC | 14:47 | |
lucasagomes | Nisha, thank you for the hard working | 14:47 |
jroll | morning ironic :) | 14:47 |
dtantsur | jroll, morning | 14:48 |
jroll | morning lucasagomes dtantsur faizan Nisha :) | 14:50 |
Nisha | lucasagomes: after reapproval the patches will go merge automatically? or any step required by us | 14:50 |
Nisha | morning jroll | 14:50 |
lucasagomes | Nisha, merges automatically | 14:50 |
Nisha | :) | 14:50 |
jroll | lucasagomes: I don't think we have a 3-core rule | 14:50 |
lucasagomes | jroll, yeah, me neither | 14:50 |
Nisha | lucasagomes: ok | 14:50 |
lucasagomes | but if you want to do a final look at https://review.openstack.org/116561 and https://review.openstack.org/118578 | 14:50 |
lucasagomes | and approve if you are happy with it, it will be great | 14:51 |
jroll | also, nova's rule is 3 cores for ffe, not for +A on patches | 14:51 |
jroll | yeah | 14:51 |
jroll | one sec | 14:51 |
lucasagomes | yup | 14:51 |
*** athomas has joined #openstack-ironic | 14:53 | |
*** rwsu has joined #openstack-ironic | 14:53 | |
jroll | faizan: I still don't understand why we can't just set pxe persistently in deploy | 14:54 |
*** linggao has joined #openstack-ironic | 14:56 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Remove the objectify decorator https://review.openstack.org/120489 | 14:56 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add reserve() and release() to Node object https://review.openstack.org/120488 | 14:56 |
Shrews | lucasagomes: this PYTHONDONTWRITEBYTECODE patch is interesting. Does this have any performance implications? | 14:57 |
Shrews | noticeable* performance | 14:57 |
faizan | jroll, because in deploy the node is still in power off and in prepare method we might have done a boot mode switch from bios to uefi | 14:57 |
lucasagomes | Shrews, I don't think so, AFAIK pyc is not a python optimization | 14:58 |
jroll | faizan: does changing to uefi reset boot mode away from pxe? | 14:58 |
lucasagomes | Shrews, lemme do a quick google to make sure | 14:58 |
jroll | lucasagomes: afaik it will otherwise compile at boot time | 14:59 |
jroll | err | 14:59 |
jroll | s/boot/import/ | 14:59 |
*** igordcard has quit IRC | 14:59 | |
*** overlayer has joined #openstack-ironic | 14:59 | |
lucasagomes | jroll, yeah, it will opmize startup time indeed | 15:00 |
lucasagomes | pyc files | 15:00 |
faizan | jroll, the persistent boot devices are stored differently for uefi and bios boot mode, | 15:00 |
jroll | faizan: aha, ok | 15:00 |
jroll | thanks | 15:00 |
lucasagomes | http://www.network-theory.co.uk/docs/pytut/CompiledPythonfiles.html | 15:00 |
lucasagomes | Shrews, ^ yeah it makes loading files faster | 15:02 |
jroll | boom, +A'd | 15:03 |
jroll | (both of them) | 15:03 |
jroll | faizan: ^^ | 15:03 |
lucasagomes | :D | 15:03 |
jroll | lucasagomes: what's this PYTHONDONTWRITEBYTECODE patch? | 15:04 |
dtantsur | \o/ | 15:04 |
lucasagomes | jroll, https://review.openstack.org/#/c/121050/ | 15:04 |
jroll | thanks | 15:04 |
jroll | oh, just for tox | 15:04 |
lucasagomes | I like it because pyc is really annoying, even when you switch branches | 15:04 |
jroll | I'm fine with that but will let Shrews approve | 15:04 |
lucasagomes | yeah 1 line change :) | 15:05 |
jroll | yeah, I was more worried about non-test envs | 15:05 |
lucasagomes | jroll, oh no, just for venv | 15:05 |
jroll | right :) | 15:05 |
*** jjmanzer has joined #openstack-ironic | 15:07 | |
lucasagomes | jroll, btw, tests added for the reserve() release() thing and docstrings updated | 15:07 |
jroll | cool, will look | 15:07 |
lucasagomes | no rush to review tho | 15:07 |
lucasagomes | take ur time I know it's early there | 15:07 |
jroll | ok | 15:07 |
Nisha | jroll: lucasagomes: the patch https://review.openstack.org/#/c/118578/ still shows verified marked as cross ... is it not required to remove for patch to merge? | 15:08 |
lucasagomes | Nisha, it will be marked like that until the Jenkins runs agains (after re-approval) | 15:09 |
lucasagomes | Nisha, it's now running check zuul http://status.openstack.org/zuul/ | 15:09 |
Shrews | lucasagomes: not seeing any big difference in the py27 run times. should be ok i guess | 15:09 |
lucasagomes | Shrews, :) thanks for verifying it | 15:09 |
Nisha | lucasagomes: thanks | 15:10 |
lucasagomes | Nisha, also, as that patch has a dependency | 15:10 |
lucasagomes | the first one will merge first and then this one will be merged | 15:10 |
Shrews | lucasgomes: i guess since the tests only run once, there would never be anything to take advantage of the .pyc anyway :) | 15:10 |
Nisha | ok | 15:11 |
lucasagomes | Shrews, yup :) pyc's for tests are just there to confuse us in case something is deleted | 15:11 |
ramineni | JayF : Hi, | 15:16 |
*** MattMan has quit IRC | 15:16 | |
jroll | 25 hour gate. | 15:17 |
*** faizan has quit IRC | 15:19 | |
*** yuanying has joined #openstack-ironic | 15:19 | |
* Shrews wonder why chrome is going bat-sh** crazy today when accessing review.o.o | 15:20 | |
jroll | I'm not having issues | 15:21 |
Shrews | maybe it doesn't like that i'm working from a cafe this morning | 15:22 |
*** r-daneel_ has joined #openstack-ironic | 15:24 | |
jroll | ah yeah, too much ajax | 15:24 |
*** r-daneel has quit IRC | 15:25 | |
*** r-daneel__ has joined #openstack-ironic | 15:26 | |
*** ndipanoff has quit IRC | 15:28 | |
*** r-daneel_ has quit IRC | 15:30 | |
*** Ugallu has joined #openstack-ironic | 15:35 | |
Shrews | jroll: qq... agent.IronicPythonAgent.get_agent_mac_addr() calls self.hardware.get_primary_mac_address()... where is that method?? | 15:36 |
jroll | Shrews: hardware.py | 15:36 |
jroll | oh | 15:37 |
jroll | wtf | 15:37 |
Shrews | like, am i being stupid? | 15:37 |
* Shrews totally does that often | 15:37 | |
jroll | no | 15:37 |
jroll | so like | 15:37 |
jroll | the hardware manager is pluggable | 15:38 |
Shrews | right | 15:38 |
dtantsur | before I finally go | 15:38 |
jroll | apparently we forgot to put that in the base class | 15:38 |
Shrews | LOL | 15:38 |
dtantsur | jroll, any progress updates on https://bugs.launchpad.net/ironic/+bug/1300939 ? | 15:38 |
jroll | dtantsur: I've poked around at it, I have a path to fix | 15:38 |
jroll | was waiting for the oslo.i18n._ stuff to land | 15:39 |
jroll | (which it has) | 15:39 |
dtantsur | looking forward to it ;) | 15:39 |
jroll | :P | 15:39 |
dtantsur | oh then, have a nice weekend everyone :) | 15:40 |
Shrews | bye dtantsur! | 15:40 |
jroll | you too dtantsur :) | 15:40 |
*** dtantsur is now known as dtantsur|afk | 15:40 | |
lucasagomes | dtantsur|afk, g'night enjoy the weekend | 15:41 |
jroll | Shrews: wait... this isn't downstream either, wtf | 15:41 |
* Shrews shrugs | 15:41 | |
jroll | Shrews: oh, this isn't called | 15:42 |
*** Nisha has quit IRC | 15:42 | |
*** Ugallu has quit IRC | 15:44 | |
devananda | morning, all | 15:44 |
jroll | morning devananda | 15:45 |
lucasagomes | devananda, morning | 15:45 |
Shrews | hey devananda | 15:46 |
*** todd_dsm has joined #openstack-ironic | 15:46 | |
jroll | Shrews: oh, this isn't called at all :| | 15:46 |
*** todd_dsm has quit IRC | 15:46 | |
Shrews | yeah, i can't find get_agent_mac_addr() getting called | 15:47 |
jroll | right | 15:47 |
* jroll wonders why that was there | 15:47 | |
Shrews | jroll: specifically to confuse me, i'm sure | 15:47 |
Shrews | :) | 15:47 |
jroll | oh | 15:48 |
jroll | we get them here: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/hardware.py#L169 | 15:48 |
jroll | that other thing is redundant | 15:48 |
* jroll deletes | 15:48 | |
jroll | Shrews: are you reviewing my docstrings or just poking around | 15:48 |
Shrews | jroll: reviewing your docstrings | 15:48 |
Shrews | which led to the poking | 15:48 |
jroll | ok, I'll delete it on top of that | 15:48 |
jroll | don't want to rebase a bunch of stuff around | 15:49 |
Shrews | however you want to handle it is fine | 15:50 |
jroll | ok | 15:50 |
devananda | looks like the UEFI patches are in the merge queue? | 15:51 |
*** yuanying has quit IRC | 15:52 | |
*** rushiagr is now known as rushiagr_away | 15:52 | |
openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic-python-agent: Delete unused method get_agent_mac_address() https://review.openstack.org/121153 | 15:52 |
jroll | Shrews: ^ :) | 15:53 |
jroll | devananda: yep, the 25 hour merge queue | 15:53 |
devananda | jroll: ironic's merge queue is separate (for now) | 15:53 |
jroll | oh, right, cool :) | 15:54 |
devananda | 11651 looks like it'll merge in ~20min | 15:54 |
devananda | https://review.openstack.org/#/c/118578/ is blocked by jenkins tho | 15:54 |
devananda | I'm seeing comments from <xcat@cn.ibm.com> on Ironic reviews | 15:56 |
jroll | right | 15:56 |
jroll | I think that's ipminative CI | 15:57 |
devananda | that's fantastic | 15:57 |
jroll | but it doesn't know how to run git merge, it seems | 15:57 |
devananda | jroll: those are unrelated | 15:58 |
devananda | i mean, that 118578 isn't merging shouldn't be related to <xcat@cn.ibm.com> | 15:58 |
jroll | no, what I mean is | 15:58 |
jroll | every comment from xcat is "failed to merge blablabla" | 15:58 |
jroll | (that I've seen so far) | 15:59 |
Shrews | switching locations. biab | 15:59 |
jroll | oh, maybe it's working now | 15:59 |
jroll | nice | 15:59 |
devananda | jroll: oh. I'm looking for some recent comments from it still | 15:59 |
jroll | ha, it stores logs on sourceforge | 15:59 |
jroll | devananda: https://review.openstack.org/#/c/121050/ | 15:59 |
jroll | it passed there | 16:00 |
devananda | \o/ | 16:00 |
jroll | wow, sourceforge is really bad :( | 16:00 |
devananda | oh, that's a horrible place to store log files | 16:00 |
jroll | click a log | 16:00 |
jroll | "Your download will start in 0 seconds..." | 16:00 |
devananda | in a project repo? | 16:00 |
jroll | (counts down from 5) | 16:00 |
jroll | and then downloads instead of rendering in the browsers :/ | 16:00 |
jroll | browser* | 16:00 |
jroll | http://sourceforge.net/projects/xcat/files/OpenStack/CI/121050/2/check/check-ironic-xcat-third-party/40/ | 16:01 |
devananda | right | 16:01 |
jroll | maybe we should talk to jbjohnso__ about it | 16:01 |
*** dlaube has joined #openstack-ironic | 16:03 | |
devananda | jroll: do you know who's running the 3rdparty CI there? I suspect it would be linggao or someone on her team | 16:04 |
jroll | yeah, dunno | 16:05 |
*** bluex-pl has joined #openstack-ironic | 16:08 | |
openstackgerrit | A change was merged to openstack/ironic: Add uefi boot mode support in IloVirtualMediaIscsiDeploy https://review.openstack.org/116561 | 16:08 |
*** Haomeng has quit IRC | 16:09 | |
*** wanyen has joined #openstack-ironic | 16:10 | |
linggao | Hi devananda, jroll, yes, someone in my team is running the 3rd party CI. | 16:10 |
*** overlayer has quit IRC | 16:11 | |
jroll | linggao: cool :) request to serve the logs with apache or something, rather than storing them on sourceforge :) | 16:11 |
*** rushiagr_away is now known as rushiagr | 16:12 | |
*** ifarkas has quit IRC | 16:12 | |
linggao | devananda, jroll, currently we face the logs on sourceforge. | 16:13 |
linggao | face/save | 16:13 |
jroll | linggao: that makes them really difficult to read :( | 16:13 |
linggao | why? | 16:13 |
devananda | linggao: awesome - I'm really excited to see that it is running! | 16:13 |
jroll | it waits for 5 seconds before allowing you to download them, and it downloads rather than rendering it in the browser | 16:14 |
linggao | devananda, for now it tests ipminative with real baremetal hardware. | 16:14 |
*** Haomeng has joined #openstack-ironic | 16:14 | |
*** r-daneel__ has quit IRC | 16:16 | |
linggao | devananda, here is the test site. https://review.openstack.org/#/c/118938/ we still need fix something. | 16:17 |
linggao | please check the "IBM xCAT CI" | 16:18 |
*** stendulker_ has quit IRC | 16:19 | |
*** linggao_ has joined #openstack-ironic | 16:19 | |
*** stendulker has quit IRC | 16:19 | |
linggao | jroll, can you be more specific on " request to serve the logs with apache or something"? where do we send the request? | 16:20 |
* devananda updates the Ironic/Drivers wiki page | 16:20 | |
jroll | linggao: you run a web server and put the logs there, like openstack-infra does | 16:22 |
*** mitz has quit IRC | 16:23 | |
bluex-pl | jroll: thanks for +2 in Enhance decorators... patchset :) | 16:25 |
jroll | :) | 16:26 |
*** jistr has quit IRC | 16:26 | |
*** r-daneel__ has joined #openstack-ironic | 16:28 | |
devananda | https://wiki.openstack.org/wiki/Ironic/Drivers | 16:29 |
linggao | jroll, thanks. | 16:30 |
devananda | jroll: linggao: please check for correctness. I've updated for all the drivers' status in Juno that I'm aware of | 16:30 |
jroll | deploy | agent | N/A | 16:31 |
jroll | :( | 16:31 |
jroll | devananda: looks fine, do we want to add a POC for agent_*? | 16:32 |
jroll | or is that just for CI'd drivers? | 16:32 |
devananda | jroll: when there's CI for agent, please update that | 16:33 |
jroll | ok | 16:33 |
devananda | jroll: feel free to add a note, if you want, about the current status (in production, blah blah) | 16:33 |
jroll | ok | 16:33 |
lucasagomes | nova proxy is now approved!! | 16:34 |
jroll | \o/ | 16:34 |
JayF | ramineni: you rang? | 16:38 |
*** derekh has quit IRC | 16:38 | |
*** k4n0 has quit IRC | 16:40 | |
linggao | devananda, looks good to me. thanks. I'll talk to the team to make sure it works properly. | 16:40 |
JayF | ramineni: responded to you in PM | 16:40 |
*** bluex-pl has quit IRC | 16:42 | |
*** yuanying has joined #openstack-ironic | 16:49 | |
lucasagomes | folk I will call it a day | 16:51 |
*** viktors is now known as viktors|afk | 16:51 | |
lucasagomes | have a good night everyone! enjoy the weekend | 16:51 |
jroll | night lucas :) | 16:51 |
*** lucasagomes is now known as lucas-beer | 16:52 | |
*** igordcard has joined #openstack-ironic | 16:59 | |
*** lazy_prince is now known as killer_prince | 16:59 | |
*** Hefeweizen has joined #openstack-ironic | 17:00 | |
*** dnikisho1 has joined #openstack-ironic | 17:15 | |
*** lsmola_ has quit IRC | 17:18 | |
*** dnikishov has quit IRC | 17:18 | |
*** lsmola_ has joined #openstack-ironic | 17:19 | |
*** penick has joined #openstack-ironic | 17:21 | |
*** dtantsur|afk has quit IRC | 17:22 | |
*** yuanying has quit IRC | 17:22 | |
Shrews | hrm, this PYTHONDONTWRITEBYTECODE thing is facing quite a bit of opposition on the ML. Wonder if we should unapprove https://review.openstack.org/121050 for the time being? | 17:28 |
jroll | super low-hanging fruit review if anyone wants it: https://review.openstack.org/#/c/121153/ | 17:28 |
jroll | Shrews: yeah, dunno | 17:28 |
jroll | Shrews: I'm in favor of the change, so I won't be the one to revert it :P | 17:29 |
jroll | I almost say we leave it until they decide | 17:29 |
*** penick has quit IRC | 17:32 | |
*** penick has joined #openstack-ironic | 17:33 | |
*** harlowja_away is now known as harlowja | 17:34 | |
*** rushiagr is now known as rushiagr_away | 17:45 | |
*** Nisha has joined #openstack-ironic | 17:47 | |
openstackgerrit | A change was merged to openstack/ironic-python-agent: Enhance decorators in agent and automate creation of command_map https://review.openstack.org/117529 | 17:56 |
jroll | ugh, tempest, y u hate me | 17:59 |
*** athomas has quit IRC | 18:02 | |
*** igordcard has quit IRC | 18:13 | |
*** pelix has quit IRC | 18:15 | |
*** yuanying has joined #openstack-ironic | 18:19 | |
openstackgerrit | Josh Gachnang proposed a change to openstack/ironic-python-agent: Remove PyPy from default tox runlist https://review.openstack.org/121195 | 18:30 |
*** Nisha has quit IRC | 18:39 | |
*** pensu has joined #openstack-ironic | 18:44 | |
openstackgerrit | A change was merged to openstack/ironic: Add reserve() and release() to Node object https://review.openstack.org/120488 | 18:48 |
*** yuanying has quit IRC | 18:51 | |
*** romcheg has joined #openstack-ironic | 18:54 | |
romcheg | Hi guys! | 18:54 |
romcheg | If someone didn't touch the patch for 1 month but mentioned that they are about to apply my suggestions, is it good to update it myself? | 18:55 |
romcheg | I'm talking about this particular one https://review.openstack.org/#/c/97731/ | 18:55 |
openstackgerrit | A change was merged to openstack/ironic: Remove the objectify decorator https://review.openstack.org/120489 | 18:56 |
jroll | romcheg: I think it's fine in general to update patches, especially for small changes | 19:01 |
jroll | you should try to let them know if possible, but if you can't that's ok | 19:01 |
romcheg | jroll: Well, I actually proposed to change the patch completely but yongli agreed :) | 19:02 |
jroll | we should all do what we need to do to push the patch forward :) | 19:02 |
romcheg | yongli: ^ | 19:02 |
jroll | s/the patch/ironic/ | 19:02 |
jroll | in this case, it's been a month, it's probably ok | 19:02 |
jroll | to do it without asking | 19:03 |
jroll | in general we shouldn't be afraid to touch other folks' code | 19:03 |
*** ChuckC_ has joined #openstack-ironic | 19:03 | |
romcheg | jroll: Yes, but it's been a while so I'd like to be sure whether it's ok :) | 19:04 |
romcheg | I think I have to make my announcement public. As I've already told some folks, I no longed work in the upstream group in my company. So I can only work on Ironic only in my personal time. | 19:04 |
romcheg | Which I will try to do cuz I like working with all of you guys :) | 19:04 |
jroll | oh no :( | 19:05 |
jroll | glad you'll be around, though | 19:05 |
ChuckC_ | jroll: hi. I'm still having some trouble with devstack. Do you have some time? | 19:07 |
jroll | ChuckC_: a bit, what's up? | 19:07 |
ChuckC_ | When I try to nova boot, the ironic node never gets to ACTIVE | 19:08 |
ChuckC_ | I see that dnsmasq gives it an address, but after quite a long time, it just goes back to None provisioning state | 19:09 |
jroll | are you following our devstack guides or? | 19:09 |
*** coolsvap has quit IRC | 19:09 | |
ChuckC_ | yes, pretty much exactly what's in the dev guide, except I turned up logging. | 19:10 |
jroll | is there an error on the ironic node? any errors in logs? (nova-api, nova-scheduler, nova-compute, ironic-api, ironic-conductor, ironic node's console) | 19:10 |
ChuckC_ | http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#deploying-ironic-with-devstack | 19:10 |
ChuckC_ | 2014-09-11 21:00:11.530 ERROR ironic.conductor.utils [-] Timeout reached while waiting for callback for node 324497b3-0cea-4ba4-ae1e-c8a2de8f932f | 19:11 |
openstackgerrit | A change was merged to openstack/ironic: Support for setting boot mode in pxe_ilo driver https://review.openstack.org/118578 | 19:12 |
jroll | ChuckC_: check the console logs | 19:13 |
jroll | should be at $DEVSTACK_HOME/ironic-bm-logs/ | 19:13 |
jroll | that usually means the agent isn't starting | 19:14 |
jroll | ChuckC_: to be clear, what OS are you running this on? (I doubt it matters, but) | 19:14 |
ChuckC_ | trusty | 19:14 |
jroll | ok, cool | 19:15 |
jroll | yeah, check console logs | 19:15 |
jroll | could also check tftp logs or something, dunno | 19:16 |
Shrews | ChuckC_: running on a laptop, vm, or other? | 19:17 |
Shrews | if it's something underpowered, it might just take longer for a provision and you'd need to adjust the timeout | 19:17 |
* Shrews guesses wildly | 19:17 | |
ChuckC_ | thanks, looking (don't see ironic-bm-logs anywhere) | 19:18 |
jroll | huh... mine are /opt/stack/ironic-bm-logs | 19:18 |
ChuckC_ | Shrews: yeah, this is on my laptop, which is 8 core i7-2860QM | 19:19 |
Shrews | ChuckC_: cpu is definitely enough. how much ram? | 19:19 |
jroll | Shrews: working on this tempest stuff, it appears to work but ssh times out on the last check :( | 19:20 |
jroll | ChuckC_: maybe you can paste your localrc somewhere, too | 19:20 |
ChuckC_ | Shrews: 16G, and the VM devstack is running on is 8GB | 19:20 |
jroll | yeah, should be fine | 19:20 |
Shrews | jroll: last check? | 19:21 |
jroll | Shrews: after the rbeuild | 19:21 |
jroll | but watching the logs... cirros boots just fine | 19:21 |
jroll | so dunno | 19:21 |
Shrews | jroll: review # again? | 19:21 |
devananda | so folks it's probably time for us to start thinking about the summit, and what we all want to talk about | 19:22 |
devananda | we won't know until after next week what day we're on, and it may be longer until we know exactly how many slots we get | 19:22 |
ChuckC_ | jroll: http://paste.openstack.org/show/110772/ | 19:22 |
devananda | but this year it's going to be a different layout anyway | 19:22 |
devananda | s/year/summit/ | 19:22 |
jroll | Shrews: https://review.openstack.org/#/c/120814/ | 19:23 |
jroll | Shrews: don't think it's related to my change, though | 19:23 |
jroll | devananda: +1 | 19:23 |
Shrews | jroll: so, the self.verify_connectivity() after the rebuild is failing for you? | 19:24 |
jroll | ChuckC_: looks fine | 19:24 |
jroll | Shrews: yeah | 19:24 |
jroll | oh wait | 19:24 |
jroll | IT PASSED | 19:24 |
jroll | idk what happened, restacked and it's fine | 19:24 |
jroll | I did have to bump timeouts, though :( | 19:24 |
*** jjmanzer has left #openstack-ironic | 19:26 | |
Shrews | ChuckC_: hrm, i'd study the /opt/stack/logs/screen/screen-ir-cond.log file a bit and look for any errors for that node | 19:27 |
ChuckC_ | Shrews: jroll: I see http://paste.openstack.org/show/110783/ happening repeatedly in screen-ir-cond.log (even now, after provisioning state is None) | 19:29 |
jroll | that's normal | 19:29 |
devananda | https://docs.google.com/spreadsheets/d/1XBKdeDeGfaRYaThjIIoYRwe_zPensECnxsKUuqdoVmQ/edit#gid=0 | 19:31 |
devananda | what do ya'll think of the format of that | 19:31 |
devananda | is it missing columns? | 19:31 |
devananda | I stuck one example in there to get it started | 19:31 |
ChuckC_ | no errors in that file other than what I posted here | 19:31 |
devananda | (everyone should have edit rights) | 19:31 |
jroll | ChuckC_: what's your nova boot command? | 19:31 |
ChuckC_ | jroll: 2014-09-11 21:00:11.530 ERROR ironic.conductor.utils [-] Timeout reached while waiting for callback for node 324497b3-0cea-4ba4-ae1e-c8a2de8f932f | 19:31 |
devananda | or would ya'll prefer an etherpad over a spreadsheet? | 19:31 |
romcheg | devananda: I think the summary should be a separate column | 19:32 |
jroll | ChuckC_: oh, right | 19:32 |
devananda | romcheg: separate from? | 19:32 |
devananda | I mean, it is a column right now | 19:32 |
ChuckC_ | chuckC: right out of the doc: nova boot --flavor baremetal --image $image --key-name default testing | 19:32 |
romcheg | devananda: from Notes, Goals, Etc. | 19:32 |
romcheg | In order to be able to easily find the summary after lots of edits | 19:33 |
jroll | ChuckC_: run 'nova boot' again, while that's happening run "sudo virsh list --all" repeatedly, one should eventually power on | 19:33 |
romcheg | I assume Notes, Goals, Etc will be changes during the discussions | 19:33 |
ChuckC_ | jroll: yes, virsh showed it powered on | 19:34 |
jroll | ChuckC_: ok, need to track down ironic-bm-logs | 19:34 |
ChuckC_ | jroll: note: I set some logging params in localrc | 19:35 |
jroll | yeah, this is different, I think | 19:36 |
jroll | hmm | 19:36 |
ChuckC_ | jroll: ./localrc:IRONIC_VM_LOG_DIR=/home/stack/ironic-bm-logs | 19:38 |
jroll | ChuckC_: oh right IRONIC_VM_LOG_DIR=/home/stack/ironic-bm-logs | 19:38 |
jroll | yeah | 19:38 |
ChuckC_ | jroll: the log files are empty, though! | 19:39 |
jroll | ooooo | 19:40 |
jroll | I think that means they aren't pxe booting | 19:40 |
jroll | something is probably f'd with neutron | 19:40 |
jroll | or tftp | 19:40 |
ChuckC_ | any network setup required before running stack.sh? | 19:40 |
jroll | no | 19:41 |
jroll | I'd unstack.sh && stack.sh | 19:41 |
jroll | and try again | 19:41 |
jroll | just to be sure it's not something weird | 19:41 |
jroll | # ENABLE_TENANT_VLANS=True | 19:41 |
jroll | # OVS_BRIDGE_MAPPINGS=br-eth1:vlannet | 19:41 |
jroll | were those there when you stack'd | 19:41 |
jroll | ? | 19:41 |
ChuckC_ | yes, I was planning to add vlan after I got the documented one running | 19:42 |
jroll | might be related | 19:45 |
jroll | s/might be/likely | 19:45 |
jroll | devananda: that format is fine with me | 19:45 |
ChuckC_ | jroll: will remove and restack | 19:45 |
ChuckC_ | jroll: but I'm skeptical :s | 19:47 |
Shrews | ChuckC_: if that doesn't work, might want to check that your vm's can actually contact the devstack vm over the br-int interface using your networking prowess. | 19:47 |
ChuckC_ | Shrews: sure | 19:48 |
jroll | ChuckC_: I mean, if they were commented out, then yeah, not related | 19:48 |
jroll | but like | 19:48 |
jroll | this is clearly a networking issue | 19:48 |
jroll | maybe not clearly | 19:48 |
jroll | probably | 19:48 |
Shrews | it's clearly probably a networking issue maybe | 19:48 |
*** yuanying has joined #openstack-ironic | 19:49 | |
* Shrews has jroll's back | 19:49 | |
harlowja | kylestev hey, for https://review.openstack.org/#/c/117576/ would u instead be able to use tooz to do this, imho it can help ensure this is done in a sane manner | 19:49 |
jroll | lol | 19:49 |
ChuckC_ | Shrews: like all issues ;) | 19:50 |
*** romcheg1 has joined #openstack-ironic | 19:53 | |
*** romcheg has quit IRC | 19:53 | |
*** penick has quit IRC | 19:58 | |
*** penick has joined #openstack-ironic | 20:01 | |
*** lucas-beer has quit IRC | 20:04 | |
*** andreykurilin_ has joined #openstack-ironic | 20:05 | |
jroll | Shrews: do you know where tempest timeouts are set? devstack-gate? | 20:13 |
jroll | (for CI, not locally) | 20:13 |
jroll | ignore me, found it | 20:15 |
*** yuanying_ has joined #openstack-ironic | 20:16 | |
*** yuanying has quit IRC | 20:19 | |
jroll | I'm signing off for the weekend, y'all have a good one :) | 20:21 |
ChuckC_ | jroll: bye and thanks | 20:21 |
*** pensu has quit IRC | 20:21 | |
*** yuanying_ has quit IRC | 20:22 | |
*** igordcard has joined #openstack-ironic | 20:23 | |
ChuckC_ | Shrews: you there? | 20:27 |
*** r-daneel__ has quit IRC | 20:29 | |
*** ramineni has quit IRC | 20:33 | |
openstackgerrit | Josh Gachnang proposed a change to openstack/ironic-python-agent: Remove PyPy from default tox runlist https://review.openstack.org/121195 | 20:38 |
openstackgerrit | Josh Gachnang proposed a change to openstack/ironic-python-agent: Remove PyPy from default tox runlist https://review.openstack.org/121195 | 20:39 |
*** ChuckC_ has quit IRC | 20:41 | |
devananda | adam_g: a few days ago you WIP'd https://review.openstack.org/#/c/118700/ then uploaded more patches, but didn't post links to the supporting changes -- have they all merged now? | 20:48 |
adam_g | devananda, yeah, that should be okay to merge now https://review.openstack.org/#/q/topic:ironic_infra,n,z has everythign thats pending | 20:49 |
adam_g | devananda, the only thing dependent on merge ordering is the WIP'd disable cinder patch | 20:50 |
devananda | awesome. looks ilke one of them is still WIPd | 20:50 |
*** penick has quit IRC | 20:50 | |
adam_g | devananda, that WIP'd cant merge before 112134 | 20:50 |
adam_g | Consolidate and rename Ironic jobs | 20:51 |
devananda | gotcha | 20:51 |
devananda | sounds like a catch-22 tho | 20:51 |
devananda | IIUC, landing 112134 will make things fail until 115803 is landed | 20:52 |
devananda | as long as -infra is aware of this and approves both of them at the same time, that seems OK though | 20:52 |
adam_g | no, that will let us disable cinder in the meantime | 20:53 |
adam_g | 112134 reconfigures the current check-tempest-dsvm-full the same as the current virutal-ironic job (VIRT_DRIVER=ironic, cinder tests skipped with testr regex) | 20:53 |
adam_g | and drops the virtual-ironic job | 20:53 |
*** penick has joined #openstack-ironic | 20:54 | |
adam_g | we can disable cinder after | 20:54 |
devananda | ah | 20:54 |
devananda | so 112134 is good to go now | 20:54 |
adam_g | and build confidence in the non-voting ironic-parallel job (https://review.openstack.org/118700) | 20:54 |
*** dprince has quit IRC | 20:55 | |
adam_g | after thats stable, we reconfigure check-tempest-dsvm-full same as ironic-parallel , and drop that.. then we're running a 'vanilla' tempest smoke | 20:55 |
adam_g | devananda, yeah, should be. been sitting a while with no love and may need a rebase | 20:55 |
devananda | adam_g: looks like 112134 had +2's from clarkb earlier this week | 20:55 |
adam_g | devananda, it did, yeah. but needed rebasing at some point | 20:56 |
devananda | ah | 20:56 |
*** ChuckC_ has joined #openstack-ironic | 20:58 | |
devananda | adam_g: remind me - you took time off ,or are going to take time off soon? (wondering if I should take up the list of infra changes to champion through next week) | 21:05 |
adam_g | devananda, i was out the last 2 days but im back and will be around next week | 21:05 |
*** penick has quit IRC | 21:05 | |
devananda | great (welcome back!) | 21:06 |
devananda | adam_g: assuming ironic graduates, I think we'll want to have the tempest stuff worked out and consolidated, so we can make some of them voting very soon | 21:06 |
openstackgerrit | greghaynes proposed a change to openstack/ironic: Store HashRings in HashRingManager class state https://review.openstack.org/120951 | 21:06 |
adam_g | devananda, but any help pesteri^Wchampioning stuff thru -infra queue would be great | 21:06 |
devananda | hehe | 21:06 |
adam_g | devananda, so im looking closer at the migration thing now, with the help of devtest | 21:08 |
*** penick has joined #openstack-ironic | 21:08 | |
adam_g | we currently migrate node's state correctly, but nova instances hypervisor still points to the old nova-bm hostname (uuid) | 21:09 |
adam_g | im wondering if there were a way to update that in nova, if that would give us the ability to migrate, reconfig + restart nova,, stop/start instances and have ownership transfer to ironic | 21:10 |
devananda | interesting | 21:10 |
devananda | what exactly do you mean by "nova instance hypervisor" | 21:10 |
adam_g | the instance to hypervisor association in nova | 21:11 |
adam_g | | OS-EXT-SRV-ATTR:hypervisor_hostname | ce31002a-3373-49b6-8274-76c1d9db89a4 | | 21:11 |
adam_g | i dont remember off the top of my head what that points to in the ironic case | 21:11 |
adam_g | wait, thats the node uuid, right? | 21:11 |
adam_g | theres still an association somewhere between the nova instance and the hypervisor that currently hosts the instance. so even if a nova-bm node is migrated to ironic, the nova-bm hypervisor will be DOWN and the instance can no longer be acted upon (deletion may be impossible at that point, too?) | 21:13 |
*** jrist has quit IRC | 21:16 | |
devananda | yep | 21:17 |
devananda | ooh wow | 21:17 |
devananda | no | 21:17 |
devananda | OS-EXT-SRV-ATTR:hypervisor_hostname becomes the node uuid | 21:17 |
*** coolsvap has joined #openstack-ironic | 21:17 | |
devananda | i mean, that attribute exposes the node uuid for an instance in ironic | 21:18 |
devananda | when the instance is created WITH ironic | 21:18 |
devananda | oh :( | 21:18 |
devananda | so | 21:18 |
devananda | we could (a) update the OS-EXT-SRV-ATTR:hypervisor_hostname in Nova during the migration, or (b) copy the OS-EXT-SRV-ATTR:hypervisor_hostname into Ironic when we import that node | 21:18 |
devananda | the API allows POSTing a Node with a preset UUID | 21:19 |
*** yuanying has joined #openstack-ironic | 21:19 | |
devananda | that bypasses the code which generates the node UUID | 21:19 |
adam_g | i think we alreayd preserve the uuid from the database | 21:19 |
adam_g | on migration | 21:19 |
devananda | I think that's an easy answer | 21:19 |
devananda | good | 21:19 |
devananda | adam_g: there may be a separate reference to the hypervisor driver that we need to update | 21:20 |
adam_g | yeah thats what im trying to dig out of my head | 21:20 |
devananda | but i think this handles the hypervisor_hostname | 21:20 |
adam_g | i know there is a (hypervisor, $id_or_something) tuple somewhere in nova | 21:20 |
devananda | (host, hypervisor_hostname) | 21:21 |
adam_g | ah right | 21:21 |
devananda | as long as the hsotname of the compute host didn't change | 21:21 |
devananda | and the node uuid was imported to ironic | 21:21 |
devananda | it should map the same, I think | 21:21 |
adam_g | okay, yeah that may actually work | 21:21 |
adam_g | cool | 21:21 |
adam_g | getting keystone endpoints setup and will test this | 21:21 |
*** dhellmann is now known as dhellmann_ | 21:22 | |
devananda | awesome | 21:22 |
JayF | devananda: I'm assuming that code checks to see if the provided uuid was valid? | 21:23 |
JayF | devananda: I ask because I know when we bootstrapped our production, we did it by posting an object that had uuid:FAKE | 21:23 |
devananda | JayF: yes. type checking and uniqueness are still enforced | 21:23 |
JayF | and it generated one :) | 21:23 |
devananda | ah | 21:23 |
devananda | if it's not erroring when receiving an invalid UUID, i think that's a bug in the API | 21:24 |
JayF | it threw an error for sure, but then happily gave us a uuid | 21:24 |
JayF | iirc | 21:24 |
JayF | jroll: ^^ | 21:24 |
* JayF goes and looks at jroll's bootstrap script | 21:24 | |
JayF | ah, he's gone for the day, I forgot | 21:24 |
devananda | JayF: looking at api/v1/types.py I think it should be just throwing an InvalidUUID exception and failing out | 21:25 |
devananda | JayF: unless wsme is mangling the error and continuing on | 21:26 |
*** andreykurilin_ has quit IRC | 21:26 | |
JayF | devananda: I think you're right. Looking at the copy of that script at master in our repo, it doesn't do that | 21:26 |
JayF | devananda: so probably I'm just misremembering a bug we had in that script while bootstrapping | 21:26 |
JayF | Those were a pretty crazy couple of weeks. hehe | 21:26 |
* devananda looks at /v1/node.py:NodesController.post | 21:27 | |
devananda | 739 if not node.uuid: | 21:27 |
devananda | 740 node.uuid = utils.generate_uuid() | 21:27 |
devananda | 751 new_node = objects.Node(context=pecan.request.context, | 21:28 |
devananda | 752 **node.as_dict()) | 21:28 |
*** romcheg1 has quit IRC | 21:28 | |
devananda | nope - it's not doing type checking in the API | 21:28 |
devananda | :( | 21:28 |
devananda | JayF: want to file a bug? | 21:28 |
*** romcheg has joined #openstack-ironic | 21:29 | |
devananda | JayF: never mind -- it is behaving as I expect | 21:37 |
devananda | even if i'm now unsure as to why | 21:38 |
JayF | heh, in that case, I'll claim the reason I hadn't filed the bug yet to be clairvoyance as opposed to forgetfulness+distraction | 21:38 |
devananda | :) | 21:42 |
devananda | ahh | 21:42 |
devananda | I see why | 21:42 |
openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Improve IPA client library https://review.openstack.org/111118 | 21:42 |
devananda | wsme is converting the POST body into an api.controller.v1.Node object | 21:42 |
devananda | which is where the type checkign is happening | 21:42 |
devananda | and so it raises: 2014-09-12 14:41:19.349 2755 WARNING wsme.api [-] Client-side error: Expected a uuid but received aaaa-bbbb-cccc. | 21:42 |
devananda | yay! | 21:43 |
*** aswadr has quit IRC | 21:43 | |
JoshNang | JayF: ^ That should be a much improved version | 21:43 |
*** linggao has quit IRC | 21:46 | |
*** yuanying has quit IRC | 21:52 | |
*** igordcard has quit IRC | 22:00 | |
devananda | JoshNang: hi! I was just looking at that | 22:01 |
devananda | a few quick notes, unrelated to the code | 22:01 |
JoshNang | devananda: sweet | 22:01 |
devananda | JoshNang: please tag bugs ya'll file about the agent with "agent" -- I just tagged that one | 22:01 |
devananda | also, we are now in string freeze -- https://wiki.openstack.org/wiki/StringFreeze | 22:02 |
devananda | so adding new exceptions really shouldn't be done | 22:02 |
*** igordcard has joined #openstack-ironic | 22:02 | |
JoshNang | devananda: ahhh didn't realize we were in string freeze. i'll fix that | 22:02 |
JoshNang | and thanks for tagging that | 22:02 |
devananda | JoshNang: if you can split this patch so that the new strings can be held off till Kilo, that'd be graet | 22:03 |
JoshNang | devananda: np! | 22:03 |
*** romcheg has quit IRC | 22:06 | |
JayF | devananda: how hard is it to get something approved past the stringfreeze? looking over this it seems like it'll be hard to do some of the better error checking (like https://review.openstack.org/#/c/111118/8/ironic/drivers/modules/agent.py,cm line 448) without adding a string indicating the error | 22:09 |
JayF | aha Add or change of log messages. | 22:10 |
JayF | is excluded from the freeze per that wiki | 22:10 |
JayF | JoshNang: ^^ | 22:10 |
JayF | So as long as you don't _set_failed_state() with that message it should technically be OK | 22:11 |
*** Hefeweiz1n has joined #openstack-ironic | 22:14 | |
JoshNang | JayF: I think the bigger problem is 4 exceptions with translated messages | 22:15 |
*** dividebin has joined #openstack-ironic | 22:15 | |
JoshNang | though, with not changing _set_failed_state, I'm not sure those log messages ever make it up to the API, so it might be ok, according to the wiki | 22:16 |
JayF | That's what I was saying :) | 22:18 |
*** vdrok has quit IRC | 22:18 | |
*** dhellmann_ has quit IRC | 22:18 | |
*** marios has quit IRC | 22:18 | |
*** devanand1 has joined #openstack-ironic | 22:19 | |
*** boris-42_ has joined #openstack-ironic | 22:22 | |
*** marios has joined #openstack-ironic | 22:24 | |
*** openstack has joined #openstack-ironic | 23:37 | |
*** zer0c00l has joined #openstack-ironic | 23:42 | |
*** NobodyCam has joined #openstack-ironic | 23:42 | |
*** coolsvap has joined #openstack-ironic | 23:42 | |
*** Hefeweizen has joined #openstack-ironic | 23:42 | |
*** ekarlso- has joined #openstack-ironic | 23:42 | |
*** pcrews has joined #openstack-ironic | 23:42 | |
*** dhellmann has joined #openstack-ironic | 23:42 | |
*** tonycoffman has joined #openstack-ironic | 23:42 | |
*** sirushti has joined #openstack-ironic | 23:42 | |
*** openstackgerrit has joined #openstack-ironic | 23:42 | |
*** slagle has joined #openstack-ironic | 23:42 | |
*** SpamapS has joined #openstack-ironic | 23:42 | |
*** Mike_D_wk has joined #openstack-ironic | 23:42 | |
*** HenryG has joined #openstack-ironic | 23:42 | |
*** marios has joined #openstack-ironic | 23:42 | |
*** Shrews has joined #openstack-ironic | 23:42 | |
*** mkerrin has joined #openstack-ironic | 23:42 | |
*** stevebaker has joined #openstack-ironic | 23:42 | |
*** davidlenwell has joined #openstack-ironic | 23:42 | |
*** marzif_ has joined #openstack-ironic | 23:42 | |
*** Guest74618 has joined #openstack-ironic | 23:42 | |
*** greghaynes has joined #openstack-ironic | 23:42 | |
*** chuckC has joined #openstack-ironic | 23:42 | |
*** dlaube has joined #openstack-ironic | 23:42 | |
*** romcheg1 has joined #openstack-ironic | 23:42 | |
*** jroll has joined #openstack-ironic | 23:42 | |
*** adam_g has joined #openstack-ironic | 23:42 | |
*** boris-42 has joined #openstack-ironic | 23:42 | |
*** BadCub_ has joined #openstack-ironic | 23:42 | |
*** kevinbenton has joined #openstack-ironic | 23:42 | |
*** harlowja_ has joined #openstack-ironic | 23:42 | |
*** devananda has joined #openstack-ironic | 23:42 | |
*** dividebin has joined #openstack-ironic | 23:42 | |
*** lsmola_ has joined #openstack-ironic | 23:42 | |
*** dnikisho1 has joined #openstack-ironic | 23:42 | |
*** Haomeng has joined #openstack-ironic | 23:42 | |
*** wanyen has joined #openstack-ironic | 23:42 | |
*** rwsu has joined #openstack-ironic | 23:42 | |
*** early has joined #openstack-ironic | 23:42 | |
*** bigjools has joined #openstack-ironic | 23:42 | |
*** ryanpetre has joined #openstack-ironic | 23:42 | |
*** JayF has joined #openstack-ironic | 23:42 | |
*** krtaylor has joined #openstack-ironic | 23:42 | |
*** killer_prince has joined #openstack-ironic | 23:42 | |
*** datajerk has joined #openstack-ironic | 23:42 | |
*** wendar has joined #openstack-ironic | 23:42 | |
*** pradipta_away has joined #openstack-ironic | 23:42 | |
*** comstud has joined #openstack-ironic | 23:42 | |
*** victor_lowther_ has joined #openstack-ironic | 23:42 | |
*** yuriyz has joined #openstack-ironic | 23:42 | |
*** yongli has joined #openstack-ironic | 23:42 | |
*** sbadia has joined #openstack-ironic | 23:42 | |
*** enterprisedc has joined #openstack-ironic | 23:42 | |
*** toabctl has joined #openstack-ironic | 23:42 | |
*** GheRivero_pto has joined #openstack-ironic | 23:42 | |
*** enikanorov has joined #openstack-ironic | 23:42 | |
*** agordeev has joined #openstack-ironic | 23:42 | |
*** mrda-vacation has joined #openstack-ironic | 23:42 | |
*** mmitchell has joined #openstack-ironic | 23:42 | |
*** Ng has joined #openstack-ironic | 23:42 | |
*** aweeks has joined #openstack-ironic | 23:42 | |
*** rushiagr_away has joined #openstack-ironic | 23:42 | |
*** viktors|afk has joined #openstack-ironic | 23:42 | |
*** keekz has joined #openstack-ironic | 23:42 | |
*** EmilienM has joined #openstack-ironic | 23:42 | |
*** pleia2 has joined #openstack-ironic | 23:42 | |
*** soren has joined #openstack-ironic | 23:42 | |
*** JoshNang has joined #openstack-ironic | 23:42 | |
*** zz_naotok has joined #openstack-ironic | 23:42 | |
*** russell_h has joined #openstack-ironic | 23:42 | |
*** dvorak has joined #openstack-ironic | 23:42 | |
*** mikal has joined #openstack-ironic | 23:42 | |
*** Madasi has joined #openstack-ironic | 23:42 | |
*** tteggel has joined #openstack-ironic | 23:42 | |
*** kylestev has joined #openstack-ironic | 23:42 | |
*** dguerri has joined #openstack-ironic | 23:42 | |
*** aignatov has joined #openstack-ironic | 23:42 | |
*** lifeless has joined #openstack-ironic | 23:42 | |
*** jeblair has joined #openstack-ironic | 23:42 | |
*** anteaya has joined #openstack-ironic | 23:42 | |
*** jbjohnso__ has joined #openstack-ironic | 23:42 | |
*** zigo has joined #openstack-ironic | 23:42 | |
*** gilliard has joined #openstack-ironic | 23:42 | |
*** lynxman has joined #openstack-ironic | 23:42 | |
*** cinerama has joined #openstack-ironic | 23:42 | |
*** Isotopp has joined #openstack-ironic | 23:42 | |
*** mordred has joined #openstack-ironic | 23:42 | |
*** morgabra has joined #openstack-ironic | 23:42 | |
*** d0ugal has joined #openstack-ironic | 23:42 | |
*** marcoemorais has joined #openstack-ironic | 23:44 | |
*** notq has joined #openstack-ironic | 23:44 | |
*** harlowja has joined #openstack-ironic | 23:44 | |
*** annegentle_ has joined #openstack-ironic | 23:44 | |
*** harlowja has quit IRC | 23:44 | |
*** annegentle_ has quit IRC | 23:44 | |
*** annegentle has joined #openstack-ironic | 23:45 | |
*** pcrews has quit IRC | 23:52 | |
* NobodyCam is back, He had forgotten that today was the day the house got new breaks | 23:53 | |
*** mitz has joined #openstack-ironic | 23:54 | |
*** mitz has quit IRC | 23:54 | |
*** mitz has joined #openstack-ironic | 23:55 | |
*** pcrews has joined #openstack-ironic | 23:57 | |
*** harlowja_ has quit IRC | 23:58 | |
*** harlowja_ has joined #openstack-ironic | 23:58 | |
*** marzif_ has quit IRC | 23:59 | |
*** marzif_ has joined #openstack-ironic | 23:59 | |
*** mkerrin has quit IRC | 23:59 | |
*** mkerrin has joined #openstack-ironic | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!