| NobodyCam | I always mix up finally and else | 00:00 |
|---|---|---|
| NobodyCam | else is only on fail | 00:00 |
| JayF | finally happens no matter what | 00:01 |
| JayF | I just think of someone saying "finallly" esasperated after a dozen things have gone wrong | 00:01 |
| jroll | correct, else is on fail, finally is always | 00:02 |
| jroll | I only have the else there to avoid catching exceptions for that code, can move it to the try block if that's easier for folks to parse | 00:02 |
| NobodyCam | +2'd | 00:03 |
| jroll | whee | 00:03 |
| jroll | thanks | 00:03 |
| NobodyCam | lol deva also +2'd | 00:03 |
| jroll | nice | 00:03 |
| NobodyCam | jroll: no need to move | 00:04 |
| jroll | k :) | 00:04 |
| * devananda afk's for dinner | 00:10 | |
| NobodyCam | enjoy | 00:10 |
| NobodyCam | :) | 00:10 |
| jroll | NobodyCam: do we want to approve this, then? https://review.openstack.org/#/c/121615/ | 00:42 |
| jroll | no rush, I suppose | 00:42 |
| * jroll goes home for the day | 00:45 | |
| rloo | jroll: qq about 121615. line 403. did you want node or node.uuid? https://review.openstack.org/#/c/121615/2/ironic/drivers/modules/pxe.py | 00:45 |
| jroll | guh | 00:45 |
| * jroll wants rloo to review everything | 00:45 | |
| jroll | one sec | 00:45 |
| * rloo doesn't ;) | 00:45 | |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Allow clean_up with missing image ref https://review.openstack.org/121615 | 00:46 |
| jroll | rloo: all yours ^ | 00:46 |
| *** mitz_ has joined #openstack-ironic | 00:46 | |
| rloo | thx jroll. i'll wait til you leave before I look any more ;) | 00:46 |
| jroll | heh | 00:46 |
| jroll | leaving now :P | 00:47 |
| *** marcoemorais has quit IRC | 01:26 | |
| *** penick has joined #openstack-ironic | 01:32 | |
| *** eghobo has joined #openstack-ironic | 01:33 | |
| *** dlaube has quit IRC | 01:34 | |
| devananda | back for a bit | 01:39 |
| *** pcrews has quit IRC | 01:42 | |
| *** penick_ has joined #openstack-ironic | 01:42 | |
| *** penick has quit IRC | 01:46 | |
| *** penick_ is now known as penick | 01:46 | |
| *** eghobo has quit IRC | 01:49 | |
| *** chenglch has joined #openstack-ironic | 01:54 | |
| *** Poornima_ has joined #openstack-ironic | 02:15 | |
| *** killer_prince is now known as lazy_prince | 02:35 | |
| rloo | NobodyCam: I modified the migration doc, merged two sections into one: https://wiki.openstack.org/wiki/Ironic/NovaBaremetalIronicMigration#Migration_Steps | 02:38 |
| rloo | NobodyCam: If you don't like it, please undo it. If it is OK, I'll update the sections before it, so that it makes more sense with the change I just made. | 02:39 |
| rloo | NobodyCam: I'll check with you tomorrow AM (or look at your comments in IRC). I'm off now. | 02:40 |
| *** rloo has quit IRC | 02:40 | |
| *** penick has quit IRC | 02:43 | |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Allow clean_up with missing image ref https://review.openstack.org/121615 | 02:44 |
| *** ramineni has joined #openstack-ironic | 02:45 | |
| *** dlaube has joined #openstack-ironic | 03:14 | |
| *** dlaube has quit IRC | 03:15 | |
| openstackgerrit | Haomeng,Wang proposed a change to openstack/ironic: Sync the doc with latest code https://review.openstack.org/121742 | 03:35 |
| openstackgerrit | Haomeng,Wang proposed a change to openstack/ironic: Sync the doc with latest code https://review.openstack.org/121742 | 03:37 |
| *** nikunj2512 has joined #openstack-ironic | 03:45 | |
| *** lazy_prince is now known as killer_prince | 03:49 | |
| openstackgerrit | A change was merged to openstack/ironic: Stop using intersphinx https://review.openstack.org/121296 | 03:50 |
| *** rushiagr_away is now known as rushiagr | 03:56 | |
| *** Poornima_ has quit IRC | 04:16 | |
| *** teju has joined #openstack-ironic | 04:18 | |
| teju | ironic node-show $NODE shows last_error : Failed to deploy. Error: [Errno 28] No space left on device....how to fix this? | 04:19 |
| *** chenglch has quit IRC | 04:21 | |
| *** rakesh_hs has joined #openstack-ironic | 04:27 | |
| *** smoriya has joined #openstack-ironic | 04:37 | |
| *** chenglch has joined #openstack-ironic | 04:37 | |
| *** rushiagr is now known as rushiagr_away | 04:43 | |
| *** Poornima_ has joined #openstack-ironic | 04:46 | |
| *** harlowja_ is now known as harlowja_away | 04:59 | |
| *** rushiagr_away is now known as rushiagr | 05:16 | |
| *** killer_prince is now known as lazy_prince | 05:23 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/120701 | 06:09 |
| *** vinbs has joined #openstack-ironic | 06:23 | |
| vinbs | Morning Ironic! | 06:30 |
| *** pensu has joined #openstack-ironic | 06:30 | |
| openstackgerrit | Anusha Ramineni proposed a change to openstack/ironic-python-agent: Add support to build iso image https://review.openstack.org/121433 | 06:48 |
| *** dtantsur|afk is now known as dtantsur | 07:07 | |
| dtantsur | Morning Ironic | 07:08 |
| pensu | Morning dtantsur! | 07:10 |
| *** Haomeng has joined #openstack-ironic | 07:27 | |
| *** nikunj2512 has quit IRC | 07:30 | |
| openstackgerrit | Dmitry Tantsur proposed a change to openstack/ironic: EXPERIMENTAL Implement hardware discovery in PXE driver https://review.openstack.org/110031 | 07:44 |
| *** igordcard has joined #openstack-ironic | 07:45 | |
| *** dtantsur is now known as dtantsur|brb | 07:48 | |
| *** jistr has joined #openstack-ironic | 07:50 | |
| *** vinbs has quit IRC | 07:51 | |
| *** vinbs has joined #openstack-ironic | 07:52 | |
| *** derekh has joined #openstack-ironic | 07:59 | |
| *** ifarkas has joined #openstack-ironic | 08:00 | |
| *** ndipanov_gone is now known as ndipanov | 08:08 | |
| *** andreykurilin_ has joined #openstack-ironic | 08:14 | |
| *** pradipta_away is now known as pradipta | 08:20 | |
| *** pensu has quit IRC | 08:20 | |
| *** athomas has joined #openstack-ironic | 08:20 | |
| *** lucasagomes has joined #openstack-ironic | 08:25 | |
| *** romcheg has joined #openstack-ironic | 08:27 | |
| *** Haomeng has quit IRC | 08:29 | |
| *** ramineni1 has joined #openstack-ironic | 08:31 | |
| *** ramineni1 has quit IRC | 08:31 | |
| *** ramineni1 has joined #openstack-ironic | 08:32 | |
| *** ramineni has quit IRC | 08:33 | |
| *** igordcard has quit IRC | 08:35 | |
| *** andreykurilin_ has quit IRC | 08:35 | |
| *** ramineni1 has quit IRC | 08:36 | |
| *** pensu has joined #openstack-ironic | 08:37 | |
| *** rakesh_hs has quit IRC | 08:38 | |
| *** pensu1 has joined #openstack-ironic | 08:39 | |
| *** pensu has quit IRC | 08:42 | |
| *** pelix has joined #openstack-ironic | 08:43 | |
| *** rakesh_hs has joined #openstack-ironic | 08:44 | |
| *** viktors|afk is now known as viktors | 08:44 | |
| *** bluex-pl has joined #openstack-ironic | 08:54 | |
| pensu1 | lucasagomes: could you please have a look: http://lists.openstack.org/pipermail/openstack-dev/2014-September/046094.html | 09:04 |
| *** athomas has quit IRC | 09:06 | |
| *** athomas has joined #openstack-ironic | 09:14 | |
| lucasagomes | pensu1, ack | 09:17 |
| lucasagomes | pensu1, hmmmmm this error looks related to neutron and not ironic | 09:17 |
| lucasagomes | DCannot open network namespace: No such file or directory; evice "ovs-tap1" does not exist. | 09:17 |
| lucasagomes | pensu1, have you tried running on trusty instead of precise? | 09:18 |
| pensu1 | lucasagomes: yeah, I guess so, looks like some issues with ovs, just wanted to confirm it's not happening because of something specific to precise... | 09:19 |
| lucasagomes | pensu1, hmm I'm not sure really, maybe asking at #openstack-neutron | 09:20 |
| pensu1 | lucasagomes: yeah, I have tried with trusty and it works completely fine, after applying the patch we saw yesterday morning... | 09:20 |
| lucasagomes | I don't know if they do have a problem like that on precise | 09:20 |
| lucasagomes | I see | 09:20 |
| pensu1 | lucasagomes: hmm....will have to check, may be something wrong with neutron only, thanks anyways...:) | 09:21 |
| lucasagomes | pensu1, no problem | 09:22 |
| *** igordcard has joined #openstack-ironic | 09:31 | |
| *** rakesh_hs2 has joined #openstack-ironic | 09:36 | |
| *** rakesh_hs has quit IRC | 09:37 | |
| openstackgerrit | A change was merged to openstack/ironic: Update ironic states and documentation https://review.openstack.org/118467 | 09:38 |
| *** ramineni has joined #openstack-ironic | 09:41 | |
| ramineni | dtantsur: Hi | 09:41 |
| *** MattMan has quit IRC | 09:41 | |
| *** MattMan has joined #openstack-ironic | 09:42 | |
| teju | hui all...i am trying to do PXE boot using ironic ....the bare metal server's console is stuck with the message "request ironic api to deploy image curl 3 url malformed waiting for notice of complete"....any ideas how to fix this? | 09:53 |
| lucasagomes | teju, hmm it seems to be a problem in the deploy ramdisk | 10:00 |
| lucasagomes | teju, you've built it recently using DIB? | 10:00 |
| teju | lucasagomes : yes... i've build the images recently | 10:01 |
| lucasagomes | teju, right... hmm | 10:01 |
| * lucasagomes check dib logs see if something changed recently | 10:02 | |
| teju | lucasagomes : ok....if possible , can u give me commands to build initrd,kernel and user images for deploying ubuntu on a bare metal node? | 10:03 |
| lucasagomes | teju, diskimage-builder/bin/ramdisk-image-create -a <arch> <ubuntu/fedora> deploy-ironic -o deploy-ironic | 10:04 |
| lucasagomes | teju, they have a nice README here https://github.com/openstack/diskimage-builder | 10:04 |
| teju | lucasagomes; Yes...i've read that but, i just wanna make sure that I am running correct commands | 10:05 |
| *** chenglch has quit IRC | 10:09 | |
| *** rushiagr is now known as rushiagr_away | 10:09 | |
| lucasagomes | teju, right... the last change to the deploy-ramdisk was https://review.openstack.org/#/c/114124/3 but hmm I don't see any problem there | 10:13 |
| lucasagomes | teju, do you know more or less how to debug a ramdisk? | 10:13 |
| *** romcheg has quit IRC | 10:14 | |
| lucasagomes | teju, usually what I do is to extract the content, modify the init script with some messages (also you can call bash there to have access to the console) | 10:14 |
| lucasagomes | it's very manual :/ | 10:15 |
| teju | lucasagomes : no...r u sure the issue is with deploy-ramdisk? ,.,,,i am asking bcoz i see the message "deployment to node xxxxx done " in ironic conductor logs | 10:15 |
| lucasagomes | teju, ouch, and the node is active? | 10:16 |
| lucasagomes | teju, I was suspecting of the ramdisk... but if the POST got to the node it may be something else | 10:16 |
| teju | lucasagomes : yes | 10:16 |
| lucasagomes | teju, hmm | 10:17 |
| teju | lucasagomes : i built user disk with the command ....bin/disk-image-create -a amd64 -o ubuntu-amd64 ubuntu | 10:17 |
| teju | lucasagomes : and upload with glance ... | 10:18 |
| lucasagomes | teju, ohhh | 10:18 |
| lucasagomes | teju, when you uploaded the image ramdisk to glance, did you use the same ramdisk for deployment ? | 10:18 |
| lucasagomes | we have 2 ramdisks in Ironic | 10:18 |
| lucasagomes | one to deploy (the one you built with the ironic-deploy element) | 10:19 |
| lucasagomes | and the final ramdisk of the image, which is just extracted from the user image after you build it | 10:19 |
| teju | lucasagomes : yes... i see these files in my tftp dir : config deploy_kernel deploy_ramdisk kernel ramdisk | 10:20 |
| *** romcheg has joined #openstack-ironic | 10:20 | |
| lucasagomes | check if deploy_ramdisk is the same as ramdisk | 10:21 |
| lucasagomes | (using diff or something) | 10:21 |
| teju | lucasagomes : they are of the same size | 10:22 |
| lucasagomes | right... ok I think the mistake is that ur using the same ramdisk for deployement and to the final image | 10:22 |
| teju | diff deploy_ramdisk ramdisk and diff deploy_kernel kernel | 10:22 |
| teju | no output | 10:22 |
| lucasagomes | teju, right yeah they are same | 10:22 |
| lucasagomes | so... | 10:23 |
| lucasagomes | teju, take a look at how we do it using the tripleo scripts https://github.com/openstack/tripleo-incubator/blob/master/scripts/load-image#L51-L118 | 10:23 |
| lucasagomes | after you build the user image you should extract the ramdisk/kernel of it | 10:24 |
| lucasagomes | and that's what you upload to glance with ur image | 10:24 |
| teju | lucasagomes : u r right....my mistake.... | 10:25 |
| lucasagomes | we have 2 ramdisks in the Ironic, the one to do the deployment (which you built using the deploy-ironic element, and after set it to the flavor) | 10:25 |
| lucasagomes | and the ramdisk of the image | 10:25 |
| lucasagomes | teju, yeah, sorry I know it's complicated :/ we need to improve our docs to reflect that as well | 10:25 |
| teju | lucasagomes : one last question....i got a qcow2 file when i ran the command bin/disk-image-create -a amd64 -o ubuntu-amd64 ubuntu | 10:26 |
| teju | lucasagomes : that qcow2 file is the user image....so, how do i extract kernel & ramdisk from that qcow2 file? | 10:26 |
| lucasagomes | teju, yes exactly | 10:27 |
| lucasagomes | there's a script on DIB to extract it for u | 10:27 |
| lucasagomes | disk-image-get-kernel I think | 10:27 |
| lucasagomes | yeah that's correct | 10:27 |
| teju | lucasagomes : ok...does disk-image-get-kernel get me both kernel & ramdisk images? | 10:28 |
| lucasagomes | yes | 10:29 |
| teju | lucasagomes : thanks for ur time...i will give a try.... | 10:29 |
| lucasagomes | teju, here a usage example https://github.com/openstack/tripleo-incubator/blob/master/scripts/load-image#L71-L87 | 10:29 |
| lucasagomes | teju, cool, no problem | 10:29 |
| *** yuanying has quit IRC | 10:36 | |
| *** yuanying has joined #openstack-ironic | 10:36 | |
| *** yuanying has quit IRC | 10:41 | |
| *** romcheg1 has joined #openstack-ironic | 10:43 | |
| *** romcheg has quit IRC | 10:43 | |
| *** romcheg1 has quit IRC | 10:48 | |
| ramineni | lucasagomes: hi | 10:51 |
| *** ramineni has quit IRC | 10:58 | |
| lucasagomes | ram<tab> :( | 11:13 |
| *** yuanying has joined #openstack-ironic | 11:20 | |
| *** rushiagr_away is now known as rushiagr | 11:23 | |
| *** Poornima_ has quit IRC | 11:28 | |
| *** lucasagomes is now known as lucas-hungry | 11:29 | |
| *** pradipta is now known as pradipta_away | 11:34 | |
| *** vinbs has quit IRC | 11:40 | |
| *** dtantsur|brb is now known as dtantsur | 11:43 | |
| *** rakesh_hs2 has quit IRC | 11:53 | |
| *** rakesh_hs has joined #openstack-ironic | 11:53 | |
| *** yuanying has quit IRC | 11:53 | |
| *** ramineni has joined #openstack-ironic | 12:02 | |
| ramineni | dtantsur , lucas-hungry : around? | 12:08 |
| dtantsur | yes | 12:08 |
| *** bluex-pl has quit IRC | 12:08 | |
| ramineni | dtantsur , have one quick question , if we want to submit a lib to global-requirements.txt , is there any prerequisite before adding to it? | 12:09 |
| *** jcoufal has joined #openstack-ironic | 12:10 | |
| dtantsur | ramineni, I think we're not the right folks to ask about global-requirements sorry :( | 12:10 |
| dtantsur | not sure who is responsible, maybe infra? | 12:10 |
| ramineni | dtantsur, ooh ok | 12:10 |
| *** vinbs has joined #openstack-ironic | 12:12 | |
| *** vinbs has quit IRC | 12:17 | |
| *** bluex-pl has joined #openstack-ironic | 12:19 | |
| *** romcheg has joined #openstack-ironic | 12:20 | |
| *** linggao has joined #openstack-ironic | 12:25 | |
| *** romcheg1 has joined #openstack-ironic | 12:28 | |
| *** enterprisedc has left #openstack-ironic | 12:29 | |
| *** lucas-hungry is now known as lucasagomes | 12:32 | |
| lucasagomes | ramineni, probably things like license etc needs to be considered | 12:32 |
| lucasagomes | should be on pip as well | 12:32 |
| ramineni | lucasagomes: its there on pip already | 12:33 |
| ramineni | lucasagomes: do i need to raise a bug and add it in global-requirements.txt? what is the process? | 12:34 |
| lucasagomes | ramineni, just propose to the requirements project (will get the link) | 12:34 |
| lucasagomes | ramineni, https://review.openstack.org/#/q/project:openstack/requirements,n,z | 12:34 |
| lucasagomes | same flow as Ironic, clone github openstack/requirements and propose the change there | 12:35 |
| ramineni | lucasagomes: thanks , will raise it . but on what basis review happens ? any idea | 12:36 |
| lucasagomes | ramineni, hmm, well no idea. You can always ping people on infra if you get no reviews in a while | 12:37 |
| lucasagomes | ramineni, also idk if because projects are on RC it will affect the review process there | 12:38 |
| ramineni | lucasagomes: ok , | 12:39 |
| *** romcheg1 has quit IRC | 12:46 | |
| *** dprince has joined #openstack-ironic | 12:46 | |
| *** yuanying has joined #openstack-ironic | 12:50 | |
| jroll | morning ironic :) | 12:54 |
| lucasagomes | jroll, yo morning | 12:56 |
| *** rakesh_hs has quit IRC | 12:56 | |
| *** ifarkas has quit IRC | 12:57 | |
| * jroll fixes up 121615 | 12:57 | |
| lucasagomes | jroll, on that called_once_with() | 12:58 |
| lucasagomes | jroll, I find another place using it, I was about to fix both lines in a path | 12:59 |
| jroll | lucasagomes: responded here about the try/except/else https://review.openstack.org/#/c/121615/4/ironic/drivers/modules/pxe.py | 12:59 |
| jroll | lucasagomes: I can rebase on yours, tis fine | 12:59 |
| *** smoriya has quit IRC | 12:59 | |
| lucasagomes | jroll, it won't conflict I bet | 12:59 |
| lucasagomes | oh I see the else there | 13:00 |
| *** ifarkas has joined #openstack-ironic | 13:01 | |
| jroll | true, but would like these tests to be running correctly :P | 13:01 |
| lucasagomes | jroll, +1! | 13:01 |
| lucasagomes | 1 sec | 13:01 |
| * jroll brb | 13:02 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: mock.called_once_with() is not a valid method https://review.openstack.org/121828 | 13:03 |
| lucasagomes | jroll, ^ | 13:03 |
| lucasagomes | jroll, I just separated the fix in another patch because there's more the 1 place | 13:03 |
| dtantsur | jroll, morning | 13:04 |
| teju | lucasagomes : thanks for your help.... | 13:04 |
| lucasagomes | teju, yvw, did it work now? | 13:05 |
| teju | lucasagomes : yes....one question,.,.,what is the provision_state if deployment is successful? | 13:05 |
| dtantsur | lucasagomes, hmm.. I actually prefer try..else in this situations, but don't mind pxe_info = [] too | 13:06 |
| lucasagomes | teju, ACTIVE and target_provision_state will be None | 13:06 |
| lucasagomes | dtantsur, yeah me too | 13:06 |
| lucasagomes | dtantsur, I see it now, I was kinda blind when I first looked | 13:07 |
| lucasagomes | but yeah only execute that code if the try block suceed | 13:07 |
| lucasagomes | succeed* | 13:07 |
| teju | lucasagomes : yes....provision state is active and target_provision_state is None.....once again....thanks a lot....bye | 13:09 |
| lucasagomes | teju, no problem yvw :) see ya | 13:09 |
| *** teju has left #openstack-ironic | 13:10 | |
| *** rloo has joined #openstack-ironic | 13:12 | |
| *** yuanying has quit IRC | 13:23 | |
| *** rakesh_hs has joined #openstack-ironic | 13:24 | |
| *** igordcard has quit IRC | 13:24 | |
| *** rakesh_hs has quit IRC | 13:28 | |
| rloo | bonjour Ironickers | 13:29 |
| rloo | Shrews, wrt https://review.openstack.org/#/c/118467/3, are you going to make the change in nova too? | 13:30 |
| jroll | lucasagomes: thanks! | 13:31 |
| jroll | morning rloo :) | 13:31 |
| rloo | hiya jroll | 13:31 |
| jroll | lucasagomes: so, another thing, if we use autospec=True or provide a spec to the mocks I think it will catch this problem | 13:31 |
| jroll | not 100% sure though | 13:31 |
| rloo | from what i remember, using autospec=True seemed to make sense but we rarely use it. | 13:32 |
| jroll | yeah | 13:33 |
| *** rwsu has quit IRC | 13:33 | |
| jroll | dtantsur: still not sure why this test doesn't fail :/ | 13:35 |
| lucasagomes | jroll, ah, 1 sec I'm in a call (but yeah spec=true may help_ | 13:36 |
| lucasagomes | or a hacking rules | 13:36 |
| jroll | yeah indeed | 13:37 |
| jroll | I wonder if newer hacking has this rule | 13:37 |
| *** rakesh_hs has joined #openstack-ironic | 13:41 | |
| jroll | dtantsur: I wonder if some other code is cleaning this up, or deleting the master dir | 13:42 |
| Shrews | rloo: maybe? i'll ask deva what we should do. | 13:47 |
| openstackgerrit | Jarrod Johnson proposed a change to stackforge/pyghmi: Handle custom keepalive modifications on the fly https://review.openstack.org/121838 | 13:47 |
| *** r-daneel__ has joined #openstack-ironic | 13:48 | |
| rloo | Shrews: thx. It seems to me that the state file ought to be kept in sync. | 13:48 |
| jroll | dtantsur: aha, https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/iscsi_deploy.py#L200 | 13:50 |
| jroll | though I thought we cache deploy ramdisk/kernel outside of that dir | 13:51 |
| *** igordcard has joined #openstack-ironic | 13:53 | |
| *** jasondotstar has joined #openstack-ironic | 13:55 | |
| jroll | paths: [u'/tmp/tmpcyq_au/tmpz5TeHZ/1be26c0b-03f2-4d2e-ae87-c02d7f33c123/config', u'/tmp/tmpcyq_au/tmpz5TeHZ/1be26c0b-03f2-4d2e-ae87-c02d7f33c123/deploy_kernel', u'/tmp/tmpcyq_au/tmpXZoXU7/1be26c0b-03f2-4d2e-ae87-c02d7f33c123/disk', '/tmp/tmpcyq_au/tmpz5TeHZ/pxelinux.cfg/01-aa-bb-cc', u'/tmp/tmpcyq_au/tmpXZoXU7/1be26c0b-03f2-4d2e-ae87-c02d7f33c123', | 13:55 |
| jroll | u'/tmp/tmpcyq_au/tmpz5TeHZ/1be26c0b-03f2-4d2e-ae87-c02d7f33c123', u'/tmp/tmpcyq_au/tmpz5TeHZ/token-1be26c0b-03f2-4d2e-ae87-c02d7f33c123'] | 13:55 |
| jroll | dtantsur: we're caching deploy kernel/ramdisk under the node? | 13:56 |
| jroll | e.g. v | 13:56 |
| jroll | /tmp/tmpcyq_au/tmpz5TeHZ/1be26c0b-03f2-4d2e-ae87-c02d7f33c123/deploy_kernel | 13:56 |
| dtantsur | wait a bit please, I'm on call | 13:56 |
| jroll | irc is async, I'm just talking, respond when you can | 13:56 |
| jroll | yeah, think maybe the tests are just broken :| | 13:58 |
| jroll | posted on the review | 14:00 |
| Shrews | jroll: why are you cluttering up my IRC window? do you hate me???? | 14:00 |
| Shrews | :P | 14:00 |
| * jroll promises to never talk again :P | 14:00 | |
| jroll | Shrews: be happy my $TEMP is not /tmp/Shrews | 14:01 |
| Shrews | lol | 14:01 |
| Shrews | jroll: usually i'm more like /dev/null... i don't care what you send my way, i'll just ignore it | 14:01 |
| jroll | heh, but do you highlight on /dev/null :P | 14:02 |
| Shrews | yes. yes i do | 14:02 |
| jroll | well then | 14:02 |
| jroll | this test is really not great | 14:03 |
| *** pcrews has joined #openstack-ironic | 14:05 | |
| Shrews | For anybody who may have missed it, a new bootstrapping hour begins this friday: http://lists.openstack.org/pipermail/openstack-dev/2014-September/046062.html | 14:13 |
| lucasagomes | rloo, Shrews morning | 14:15 |
| Shrews | lucasagomes: hey hey | 14:15 |
| lucasagomes | jroll, yeah idk if there's a hacking rules for that (will take a look) | 14:15 |
| rloo | hi lucasagomes. | 14:16 |
| rloo | Shrews: that bootstrapping stuff sounds like a great idea. Thx. | 14:17 |
| *** yuanying has joined #openstack-ironic | 14:20 | |
| NobodyCam | Good Morning Ironic | 14:21 |
| jroll | hrm. idgi | 14:21 |
| jroll | NobodyCam: morning :) | 14:21 |
| NobodyCam | morning jroll | 14:22 |
| rloo | morning NobodyCam | 14:23 |
| NobodyCam | morning rloo | 14:23 |
| NobodyCam | and Shrews too | 14:23 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Allow clean_up with missing image ref https://review.openstack.org/121615 | 14:23 |
| jroll | ok that should do it | 14:24 |
| jroll | I still don't know if it's completely correct :| | 14:24 |
| jroll | dtantsur ^ | 14:24 |
| Shrews | morning NobodyCam | 14:24 |
| * jroll bbl | 14:25 | |
| NobodyCam | :) ok enjoy jroll | 14:25 |
| *** rakesh_hs has quit IRC | 14:26 | |
| dtantsur | NobodyCam, morning | 14:27 |
| * dtantsur reading scrollback | 14:27 | |
| NobodyCam | morning dtantsur | 14:27 |
| NobodyCam | rloo: I like the rework of the migration page :) | 14:29 |
| rloo | NobodyCam: good. I'll clean up the first part of that page then, unless you want to. | 14:30 |
| rloo | NobodyCam: I was wondering if we needed to describe the $variables used in the examples. | 14:30 |
| *** rwsu has joined #openstack-ironic | 14:31 | |
| lucasagomes | NobodyCam, morning | 14:31 |
| NobodyCam | I tried to use al few $variables as possiable | 14:31 |
| NobodyCam | Morning lucasagomes | 14:32 |
| NobodyCam | :) | 14:32 |
| rloo | NobodyCam: few is still > 0. Maybe we can leave them in and see if anyone comments on them. | 14:32 |
| NobodyCam | Yea, the dkid and drid (deploy kernel and deploy raamdisk id's) are not supper clear | 14:33 |
| *** krtaylor has quit IRC | 14:34 | |
| lazy_prince | JoshNang: Hi, I finished configuring ironic-neutron plugin. and used code pinted out by you. However, i see that the DHCP options are not added to the provisioning network but its getting added to instance network. | 14:35 |
| dtantsur | jroll, left a comment. this whole test is super-confusing :( | 14:35 |
| dtantsur | rloo, and morning to you as well | 14:35 |
| rloo | hi dtantsur! | 14:35 |
| lazy_prince | JoshNang: Due to this, node is not able to pxe boot for deployment. i think update for dhcp options needs to addressed seperately.. | 14:37 |
| openstackgerrit | A change was merged to stackforge/pyghmi: Handle custom keepalive modifications on the fly https://review.openstack.org/121838 | 14:37 |
| openstackgerrit | Ruby Loo proposed a change to openstack/python-ironicclient: Updates to CLI doc https://review.openstack.org/121880 | 14:38 |
| openstackgerrit | Ruby Loo proposed a change to openstack/python-ironicclient: Add defaults to the CLI help strings https://review.openstack.org/121881 | 14:42 |
| NobodyCam | do other projects put the default like that? | 14:44 |
| rloo | NobodyCam: 'like that' vs ?? | 14:44 |
| NobodyCam | not having the default at all | 14:45 |
| rloo | So some of the strings already have 'Defaults to...' | 14:45 |
| NobodyCam | ya | 14:45 |
| rloo | I actually wonder if those OS_* ones should have more description than 'Defaults to..', but I don't want to tackle those. | 14:46 |
| rloo | I added the defaults for service_type & endpoint_type cuz our docn doesn't say to specify those, so I wondered why not. | 14:46 |
| NobodyCam | how about ones like --ironic_api_version | 14:46 |
| rloo | and then I saw the timeout one, so added that too | 14:46 |
| NobodyCam | not help string at all | 14:47 |
| NobodyCam | s/not/no/ | 14:47 |
| rloo | that one says "Defaults to env[] or 1' that's the help string. You mean, it doesn't say much? | 14:47 |
| NobodyCam | oh doh | 14:47 |
| NobodyCam | my bad there | 14:47 |
| rloo | dtantsur: (and anyone else that likes exceptions), you have a few minutes to discuss exceptions of third party libraries? | 14:51 |
| rloo | I've been trying to clean up our exception handling (bug 1267693). I'm just paranoid about 3rd party libraries. I'm wondering if it makes sense to wrap all calls to 3rd-party libraries with Exception instead of specific 3rd-party exceptions | 14:52 |
| rloo | eg https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/snmp.py#L105 | 14:52 |
| *** yuanying has quit IRC | 14:53 | |
| rloo | I've been/can read the library code and see what it does/raises, but that doesn't help if the library changes and new things happen... | 14:53 |
| Shrews | rloo: For stuff like that, I usually do the Exception route, but log the *real* actual exception, like: LOG("Bad stuff: %s, %s", (e.__class__, e)) | 14:55 |
| Shrews | where e is the Exception. iirc, __class__ will get the actual exception | 14:55 |
| Shrews | i could be mis-remembering though | 14:56 |
| rloo | Shrews: that sounds good to me. I think you might be right about the class thing. I would check of course ;) | 14:56 |
| * Shrews does a quick test | 14:56 | |
| NobodyCam | brb | 14:56 |
| rloo | if no one objects, I'll do it the Shrews-way. | 14:57 |
| dtantsur | rloo, well, I don't really like it | 14:57 |
| dtantsur | if you don't know the exception, what are you going to do with the exception? | 14:58 |
| rloo | dtantsur: what do you suggest? | 14:58 |
| dtantsur | 1. if we expect some exception - wrap it with proper message; 2. if you don't want what to expect - do nothing | 14:59 |
| rloo | dtantsur: its an exception from a 3rd party. Log it and raise an IronicException. | 14:59 |
| rloo | dtantsur: the question becomes, do we want unknown exceptions from 3rd party, to percolate all the way up the stack? | 14:59 |
| dtantsur | rloo, like any unexpected exception - yes | 15:00 |
| dtantsur | IMO again :) | 15:00 |
| rloo | dtantsur: so should we void the bug https://bugs.launchpad.net/ironic/+bug/1267693 | 15:00 |
| openstackgerrit | A change was merged to openstack/ironic: mock.called_once_with() is not a valid method https://review.openstack.org/121828 | 15:01 |
| rloo | dtantsur: i'm not an snmp person. I don't know what is expected or unexpected from those libraries and I don't want to have to know what changes between library versions. And I have no faith in the quality of coding of those third party libraries. How do I deal with it? | 15:01 |
| dtantsur | rloo, I don't like it title at all, but what we did before was reasonable: if we expect snmp to raise some SNMPError, we should wrap it | 15:01 |
| dtantsur | rloo, we do rely on quality of libraries we use, that's inevitable | 15:02 |
| *** ndipanov is now known as ndipanov_gone | 15:02 | |
| dtantsur | if they change everything in some release, we'll have much worse problems than exceptions :) | 15:02 |
| dtantsur | expecially since we mock them in tests | 15:03 |
| rloo | so we shouldn't land drivers if the authors haven't handled their exceptions. | 15:04 |
| dtantsur | rloo ++ | 15:04 |
| dtantsur | it's pity we didn't come to this agreement before | 15:04 |
| dtantsur | but IIRC we followed it for some drivers | 15:05 |
| Shrews | rloo: actually, i agree with dtantsur after thinking about it. in cases where we need to handle 3rd party exceptions, we should use their specific exceptions. | 15:05 |
| rloo | well, i tried to follow it. I'm just worried that as we continue to add new drivers, the situation could become worse. | 15:05 |
| rloo | Shrews, dtantsur: so that means someone needs to read the 3rd party library code, cuz there isn't always docn. And what scares me are the possible exceptions that the 3rd party library doesn't even catch themselves. | 15:07 |
| *** ifarkas has quit IRC | 15:07 | |
| dtantsur | rloo, if we know that some library is really behaving badly, we can surround their code with tr.. except Exception | 15:08 |
| Shrews | rloo: if the library is that bad, perhaps we shouldn't allow its use? :) | 15:08 |
| dtantsur | rloo, but I think people adding drivers should read docs :) | 15:08 |
| rloo | dtantsur: but how do I know the library is behaving badly. Who has actually tried out all the drivers? | 15:08 |
| dtantsur | Shrews ++ | 15:08 |
| *** bluex-pl has quit IRC | 15:08 | |
| jroll | rloo: I'm more scared of swallowing exceptions than them actually happening | 15:09 |
| dtantsur | rloo, well, this questions goes far beyond of just catching exceptions... who has cheched that this snmp library even works? | 15:09 |
| rloo | dtantsur: yeah, that's what i mean. | 15:09 |
| dtantsur | we don't know. we have to trust em and rely on bug reports. or take nova path and be much stricter (e.g. require CI) | 15:09 |
| *** viktors is now known as viktors|afk | 15:10 | |
| *** pelix has quit IRC | 15:11 | |
| rloo | ok so it seems like majority of vocal folks want to handle *known* 3rd party exceptions only. | 15:12 |
| *** pelix has joined #openstack-ironic | 15:15 | |
| dtantsur | +1 | 15:15 |
| Shrews | rloo: yes. if something unknown is thrown, that's a bug that we will deal with at that time | 15:15 |
| rloo | ok. sigh, was hoping not to read the snmp library ;) | 15:16 |
| jroll | dtantsur: "the same as master_deploy_kernel_path. and master copy is never deleted - it's cached." | 15:16 |
| jroll | dtantsur: thing is... it's not. | 15:16 |
| jroll | or appears not, anyway | 15:17 |
| Shrews | rloo: you can leave an "except Exception" as the last handler if you're that concerned | 15:17 |
| Shrews | try: | 15:17 |
| dtantsur | jroll, hmmm... but why? | 15:17 |
| Shrews | except SoemthingSpecific: | 15:17 |
| Shrews | except Exception: | 15:17 |
| jroll | Shrews: nooooooooo, that's only going to hide things :( | 15:17 |
| jroll | dtantsur: I have no idea | 15:17 |
| jroll | dtantsur: I logged cache_(ttl|size), those look fine | 15:18 |
| Shrews | jroll: wha? | 15:18 |
| rloo | Shrews: I think we should try to be consistent with how we handle all of them. Maybe we aren't ready for handling unknown exceptions yet. Will wait to see how many might bite us. | 15:18 |
| jroll | Shrews: if we don't know what the exception being raised is, what should we do? log and move on, with unexpected behavior? | 15:19 |
| dtantsur | Shrews, I was taught never to catch exceptions, if I don't know what to do with them :) | 15:19 |
| Shrews | jroll: i'm not saying that AT ALL | 15:19 |
| jroll | we don't know what the state will be after that exception is thrown | 15:19 |
| jroll | Shrews: what do you suggest? :P | 15:19 |
| jroll | if it's just to re-raise it... what's the point | 15:19 |
| Shrews | jroll: it's situation dependent, but it forces you to deal with the unexpected rather than just letting things blow up on their own | 15:20 |
| jroll | I'd just rather it blows up | 15:20 |
| jroll | fail fast, etc | 15:20 |
| * Shrews blows up jroll | 15:20 | |
| dtantsur | lol | 15:20 |
| * jroll dies | 15:20 | |
| * rloo didn't mean to start a war | 15:22 | |
| openstackgerrit | A change was merged to openstack/ironic: Fix Devstack docs for zsh users https://review.openstack.org/121712 | 15:23 |
| jroll | dtantsur: so... I think this is almost fine | 15:23 |
| jroll | I agree with your last comment | 15:23 |
| jroll | but maybe this shows a bug | 15:23 |
| jroll | idk | 15:24 |
| dtantsur | oh well | 15:24 |
| jroll | dtantsur: you're more familiar with the image caching stuff, do you mind taking a look at these? | 15:24 |
| NobodyCam | ok just added a line to the kilo planning sheet.. please let me know any thoughts | 15:24 |
| rloo | QQ for all you English-speaking folks out there. Is 'upgrade' == 'migrate' in the context of baremetal -> ironic? | 15:24 |
| NobodyCam | good / bad / or ugly | 15:24 |
| jroll | dunno, rloo, I think so | 15:24 |
| NobodyCam | rloo: I would say maybe | 15:24 |
| jroll | NobodyCam: link? :) | 15:24 |
| NobodyCam | https://docs.google.com/spreadsheets/d/1XBKdeDeGfaRYaThjIIoYRwe_zPensECnxsKUuqdoVmQ/edit#gid=0 | 15:25 |
| jroll | rloo: it is a migration, it's also a sideways upgrade.... so kind of? | 15:25 |
| JayF | rloo: I think migration is a better word, because "upgrades" tend to require less interaction than "migrations" but that just may be my own connotations on the words | 15:25 |
| dtantsur | NobodyCam, I'm not sure we're even close to the same problems as nova, but why not discuss :) | 15:25 |
| jroll | NobodyCam: good idea, dunno if it's high enough priority, but yeah we should talk sometime | 15:25 |
| NobodyCam | dtantsur: yes we are deff not at that point .... just want to avoid ever getting there | 15:26 |
| jroll | I mean, there's no sense in making it complicated if we'll never get to that point (but we probably will? idk) | 15:26 |
| dtantsur | jroll, ok will play with your patch (though I believe the best we can do is to refactor that test) | 15:27 |
| rloo | thx JayF, jroll, NobodyCam. | 15:27 |
| jroll | dtantsur: I think I agree, test needs to be much smaller, and preferably not touch my filesystem :) | 15:28 |
| jroll | dtantsur: I also noticed that test_clean_up_no_master_images is redundant :) | 15:28 |
| dtantsur | oh... | 15:28 |
| dtantsur | much test | 15:29 |
| dtantsur | so redundant | 15:29 |
| dtantsur | not wow | 15:29 |
| jroll | JayF: https://review.openstack.org/#/c/121433/3/imagebuild/coreos/Makefile will make automatically pick up the target for ramdisk/kernel, or would I need to run 'make docker coreos iso' | 15:29 |
| JayF | jroll: no, you'd have to make && make iso | 15:29 |
| jroll | hm, why not specify deps so we can just to 'make iso' | 15:30 |
| *** vinbs has joined #openstack-ironic | 15:39 | |
| *** viktors|afk is now known as viktors | 15:39 | |
| *** vinbs has quit IRC | 15:39 | |
| *** krtaylor has joined #openstack-ironic | 15:40 | |
| *** marcoemorais has joined #openstack-ironic | 15:42 | |
| *** rushiagr is now known as rushiagr_away | 15:43 | |
| devananda | morning, all | 15:45 |
| dtantsur | jroll, this test makes me sick :( I guess I'll just refactor it tomorrow. feel free to land patch as it is in the meanwhile | 15:45 |
| dtantsur | devananda, morning | 15:45 |
| NobodyCam | good morning devananda | 15:45 |
| lucasagomes | morning devananda | 15:46 |
| devananda | rloo: i would say "migration" is the more precise word. But it may be overloaded -- going from Nova in Icehouse to Juno is clearly an upgrade, but also requires database migrations, for example | 15:47 |
| rloo | devananda: thx. I changed a few, but left 'upgrade' in a few places in that BM-IronicMigration doc. Hmm. I'll change them all to migrate. Probably not a big deal though. | 15:49 |
| *** yuanying has joined #openstack-ironic | 15:50 | |
| lucasagomes | devananda, are we still being very strict about changing objects/base.py? e.g see comments at https://review.openstack.org/#/c/120773/ | 15:50 |
| jroll | dtantsur: hm, ok, thanks | 15:51 |
| lucasagomes | I agree that we should make remotable not need a context, and I think nova is going towards that direction as well | 15:51 |
| lucasagomes | I'm afraid whether I should modify Ironic already to do it, or wait for nova (maybe work on nova to get that there first) | 15:51 |
| lucasagomes | opinions? | 15:52 |
| * devananda makes a small pot of espresso | 15:52 | |
| dtantsur | mine stays the same - I hate passing context twice in a row to the same object :) | 15:52 |
| devananda | lucasagomes: I would like comstud's opinion on that | 15:53 |
| lucasagomes | comstud, ^ you've been invoked | 15:53 |
| devananda | admittedly I haven't followed the nova object code much lately - but i think theyr'e just objectifying more things, not changing the base class that much | 15:53 |
| lucasagomes | dtantsur, I know... I agree that this is the best options. But we need to consider whether we should divert from nova | 15:53 |
| lucasagomes | because that would make things hard to oslofy in the future | 15:54 |
| lucasagomes | devananda, yeah... well there's a couple of changes | 15:54 |
| lucasagomes | they have fields of each attribute type etc | 15:54 |
| devananda | shrews, rloo - on exceptions, I think ya'll reached a reasonable conclusion. drivers need to wrap their libraries' exceptions and raise common ones. What I do -not- want is for conductor/manager.py to have to catch a bunch of driver-specific exceptions | 15:54 |
| devananda | the moment we start that, we've broken the Driver API abstraction layer | 15:54 |
| lucasagomes | I'm planning to sync with nova once we get this finished in our objects | 15:55 |
| dtantsur | lucasagomes, well if answer is no, we should stop passing context to __init__ IMO | 15:55 |
| *** romcheg1 has joined #openstack-ironic | 15:55 | |
| devananda | lucasagomes: /me adds oslo.objects to paris agenda | 15:55 |
| lucasagomes | dtantsur, yeah, right now the constructor doens't require a context | 15:55 |
| lucasagomes | dtantsur, tho we pass it in some cases | 15:55 |
| lucasagomes | dtantsur, maybe we should stop passing the context to the constructor, and make create mandatory. I talked a bit with dan smith yesterday | 15:56 |
| *** andreykurilin has joined #openstack-ironic | 15:56 | |
| lucasagomes | devananda, cheers, year lets talk about it | 15:56 |
| dtantsur | lucasagomes, I would prefer passing to __init__, looks more sane to me | 15:56 |
| lucasagomes | I'm ok on working on oslofy that | 15:56 |
| lucasagomes | dtantsur, yup I've a patch for that, but that requires a a couple of changes on the base | 15:56 |
| lucasagomes | (will upload it) | 15:57 |
| rloo | devananda: ok, so we do our best, but no guarantees about handling ALL exceptions from 3rd party libraries, in which case they will make it to the top of the stack | 15:57 |
| dtantsur | lucasagomes, I would just land it, if there's no strong objections _right now_, as it's not really a huge change | 15:57 |
| devananda | dhellmann: any opinion if oslo has bandwidth in Kilo to take on the object base classes from nova & ironic? is that worth a session in Paris, or just a nod-and-start-work? | 15:57 |
| *** lsmola_ is now known as lsmola | 15:57 | |
| devananda | rloo: correct. any time we encounter a leaked exception, it's a bug | 15:58 |
| rloo | devananda: gotcha. thx. | 15:58 |
| *** lsmola is now known as lsmola______ | 15:58 | |
| lucasagomes | dtantsur, that's what I'm trying to figure out, when we started with the RPC objects we were very strict about changing base.py | 15:58 |
| devananda | wow. this is like drinking turkish coffee | 15:59 |
| lucasagomes | so I don't know whether we should do it now or not | 15:59 |
| lucasagomes | I think no but | 15:59 |
| NobodyCam | devananda: lol... | 15:59 |
| lucasagomes | Sep 15 15:18:06 <dansmith> lucasagomes: you mean mandatory to be passed during create and not ever during a remotable? then yeah, we're on the same page :) | 15:59 |
| lucasagomes | dtantsur, ^ I think that's the plan to remove the context from remotable in the future | 15:59 |
| devananda | NobodyCam: except I have a lot of it :) | 15:59 |
| *** viktors is now known as viktors|afk | 15:59 | |
| devananda | lucasagomes: the object must have a context to be invoked remotely | 16:00 |
| devananda | lucasagomes: also - are we actually using the remotable feature of objects at all? | 16:00 |
| lucasagomes | devananda, yeah right now we do need it | 16:00 |
| devananda | (I thought we weren't) | 16:00 |
| dtantsur | ok folks, see you tomorrow | 16:01 |
| *** dtantsur is now known as dtantsur|afk | 16:01 | |
| lucasagomes | dtantsur|afk, see ya g'night | 16:01 |
| lucasagomes | devananda, so we use remotable for things like create/destroy/save/refresh | 16:01 |
| *** dlaube has joined #openstack-ironic | 16:01 | |
| NobodyCam | night dtantsur|afk | 16:01 |
| jroll | night dtantsur|afk | 16:01 |
| devananda | lucasagomes: oh? so api service is not actually doing db connections now? | 16:02 |
| * devananda tests that | 16:02 | |
| lucasagomes | devananda, not for those cases afaict | 16:02 |
| lucasagomes | devananda, we are close to get the api not accesing the dbapi directly | 16:03 |
| lucasagomes | that's part of my plans | 16:03 |
| devananda | I see | 16:03 |
| lucasagomes | we are very close to that | 16:03 |
| lucasagomes | so we won't need to have that dbapi instance on the api | 16:03 |
| lucasagomes | which is good | 16:03 |
| devananda | well, yes and no | 16:03 |
| devananda | Nova did that for security concerns | 16:03 |
| devananda | because their API endpoint is exposed to untrusted users | 16:04 |
| devananda | ours, at least right now, is not | 16:04 |
| lucasagomes | yeah, it's not a big concern for us | 16:04 |
| devananda | using RPC for DB access has a few downsides to it -- notably, the additional network latency and work done by the conductor | 16:04 |
| lucasagomes | it's more about being consistent on the way we access things | 16:05 |
| lucasagomes | indeed | 16:05 |
| devananda | and potential for RPC troubles (saturation of the AMQP broker, or the conductor running out of available threads) to result in API service failure | 16:05 |
| jroll | this will be interesting to test at scale | 16:06 |
| lucasagomes | devananda, right, what was the biggest motivation for us to have the RPC object copied from nova? | 16:06 |
| lucasagomes | isolating db calls to only conductor seems fair | 16:07 |
| devananda | lucasagomes: my motivation was cleaner code | 16:07 |
| devananda | I don't have a problem with both the API and DB services accessing the DB directly | 16:08 |
| jroll | are remoteables always remoted? | 16:08 |
| lucasagomes | I see | 16:09 |
| devananda | jroll: dunno? | 16:09 |
| jroll | yeah, idk either | 16:09 |
| lucasagomes | jroll, needs some tests but my assumption is yes | 16:09 |
| jroll | just curious... in the middle of some other things but watching this conversation :) | 16:09 |
| devananda | lucasagomes: I think there was a flag to toggle remoting of object access in Nova at some point, but I could be thinking of something else | 16:09 |
| *** romcheg1 has quit IRC | 16:10 | |
| devananda | i dont know if that was removed, though | 16:10 |
| lucasagomes | will take a look see if I can find something | 16:10 |
| devananda | jroll: what's up with the tests on https://review.openstack.org/#/c/121615/ | 16:10 |
| jroll | devananda: everything is fucked :) | 16:11 |
| NobodyCam | jroll: :( | 16:11 |
| devananda | jroll: awesome | 16:11 |
| jroll | I mean | 16:11 |
| jroll | basically, the tests are unparseable | 16:12 |
| jroll | they seem to work ok | 16:12 |
| jroll | dtantsur|afk said if we go ahead and land that as is, he can refactor those tomorrow | 16:12 |
| lucasagomes | jroll, which I think is ok? I mean splitting up/refactoring that test is a bit outside the scope of the patch anyway no? | 16:14 |
| jroll | lucasagomes: yeah, the question is if that test is valid at all | 16:14 |
| devananda | jroll: I was happier with rev2 | 16:14 |
| jroll | but I've tested it in devstack, sooo | 16:14 |
| devananda | jroll: LOL | 16:15 |
| jroll | :) | 16:15 |
| rloo | jroll, lucasagomes: +1. (although fix test first, then jroll's patch might be better, but in the interests of time...) | 16:15 |
| jroll | devananda: the only difference from rev 2 is fixing logs and making the tests a bit better | 16:15 |
| devananda | jroll: so in general, I think this test is adding a nondeterministic race | 16:19 |
| devananda | *adding to a | 16:20 |
| jroll | mmm | 16:20 |
| jroll | that's a good point | 16:20 |
| jroll | well | 16:20 |
| jroll | each tempdir should be different | 16:20 |
| jroll | but the master thing might be racy | 16:20 |
| devananda | sorry, right | 16:20 |
| devananda | yep - anything in tftp_master_dir is racy | 16:21 |
| jroll | yeah | 16:21 |
| jroll | though that should be cached, so... | 16:21 |
| jroll | but I think you're right | 16:21 |
| devananda | and there will be two tests nwo that call | 16:21 |
| devananda | assert_false_path.append(d_kernel_path) | 16:21 |
| devananda | 3 | 16:21 |
| devananda | 2 | 16:22 |
| * devananda needs more coffee | 16:22 | |
| jroll | heh | 16:22 |
| NobodyCam | :) /me can't do more then one cup of turkish coffe per day | 16:23 |
| devananda | wait, nope. not a problem | 16:23 |
| devananda | self.config(tftp_root=temp_dir, group='pxe') | 16:23 |
| jroll | oh, right | 16:24 |
| *** yuanying has quit IRC | 16:24 | |
| rloo | NobodyCam: wrt the migration wiki, do we want to mention the proxying of baremetal-node-list/show? | 16:25 |
| devananda | rloo: ++ | 16:26 |
| NobodyCam | hummm... we could | 16:26 |
| rloo | devananda, NobodyCam: Ok, I'll add something. They will be deprecated when? | 16:26 |
| NobodyCam | once BM is removed | 16:27 |
| rloo | NobodyCam: that's good enough ;) | 16:27 |
| devananda | rloo: so actually, you shouldn't add anything about when that API will go away yet, IMO | 16:29 |
| devananda | rloo: Nova may choose to keep the proxy api around for a while. It seems silly to us -- who doesn't love and want to use our awesome API -- but that's their call | 16:29 |
| NobodyCam | :) | 16:29 |
| rloo | devananda: ? Oh. Can I mention that it *might* be deprecated? | 16:30 |
| devananda | rloo: I wouldn't even do that until Nova makes a statement about it | 16:30 |
| devananda | rloo: we /can/ say that baremetal driver will be removed, though | 16:30 |
| JayF | When is the TC meeting? 1pm? | 16:31 |
| JayF | or is it 11pm? | 16:31 |
| devananda | JayF: what tz are you in? | 16:31 |
| JayF | erm, am | 16:31 |
| rloo | devananda: sigh. ok. I wanted to mention it so that people would change. oh well. I'll mention the BM driver being removed ;) | 16:31 |
| JayF | haha, it's 9:30am right now | 16:31 |
| JayF | devananda: I forget we all aren't on IRCst yet | 16:31 |
| *** marcoemorais has quit IRC | 16:32 | |
| devananda | JayF: my IRC server runs UTC | 16:32 |
| *** marcoemorais has joined #openstack-ironic | 16:32 | |
| devananda | so - fromthe TC wiki - Next meeting: Tuesday September 16th, 20:00 UTC | 16:32 |
| JayF | all my production boxes run IRC | 16:32 |
| JayF | so graduation review == normal TC meeting? | 16:32 |
| devananda | agendas can be seen here: https://wiki.openstack.org/wiki/Governance/TechnicalCommittee | 16:33 |
| jroll | JayF: sounds like 1pm | 16:33 |
| jroll | just like last week | 16:33 |
| devananda | jroll: yep | 16:33 |
| JayF | sweet, alright | 16:33 |
| lucasagomes | so close!! | 16:33 |
| *** ramineni has quit IRC | 16:35 | |
| comstud | lucasagomes, devananda: i agree with making context mandatory on create() | 16:36 |
| devananda | comstud: cool, ty | 16:36 |
| comstud | as far as remotable(), i'm wondering if we should just remove that from everything for now. | 16:36 |
| comstud | we don't use it | 16:36 |
| lucasagomes | comstud, right, but not changing the constructor in base.py to make it passing the context mandatory when instantiating the object right? | 16:37 |
| comstud | oh | 16:37 |
| comstud | let me think. | 16:37 |
| comstud | (i just woke up) | 16:37 |
| lucasagomes | (that was one suggestion too^) | 16:37 |
| comstud | i guess that's where I want it | 16:37 |
| comstud | is on __init__ | 16:37 |
| comstud | not on individual methods | 16:38 |
| comstud | that's the direction i've been trying to move things to | 16:38 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: DO NOT MERGE: Make context mandatory when instantiating a RPC object https://review.openstack.org/121923 | 16:38 |
| lucasagomes | comstud, ^ I did it locally, that's how it will look like | 16:38 |
| lucasagomes | i was just concerned whether we should change base.py or not | 16:39 |
| lucasagomes | if you think that's fine I agree that it would be the best approach | 16:39 |
| comstud | yeah | 16:39 |
| lucasagomes | so we don't need any context on create/destroy/refresh etc.. | 16:39 |
| comstud | That's what we've started doing in nova | 16:39 |
| comstud | correct | 16:39 |
| lucasagomes | ah awesome, alright move things in that direction then | 16:40 |
| lucasagomes | comstud, ta much! | 16:40 |
| lucasagomes | if you have a time and want to review that patch (tho I -2ed just in case) it would be great | 16:41 |
| lucasagomes | and the patch is small too | 16:41 |
| comstud | yeah | 16:41 |
| * devananda watches live feed from openstacksv | 16:48 | |
| lucasagomes | alright I'll call it day, but will be back in time for the meeting :) | 16:49 |
| lucasagomes | g'night everyone! | 16:49 |
| JayF | later lucas | 16:49 |
| NobodyCam | have a good night lucasagomes | 16:49 |
| jroll | ooo, mordred and pvo are back to back this afternoon | 16:49 |
| JayF | What are you talking about? openstacksv? | 16:50 |
| JayF | oh, nice | 16:50 |
| *** jistr has quit IRC | 16:50 | |
| NobodyCam | devananda: link? | 16:50 |
| JayF | NobodyCam: http://openstacksv.com/ | 16:51 |
| NobodyCam | TY | 16:51 |
| *** derekh has quit IRC | 17:00 | |
| *** lazy_prince is now known as killer_prince | 17:00 | |
| rloo | NobodyCam: I added the nova baremetal cmds to migration doc https://wiki.openstack.org/wiki/Ironic/NovaBaremetalIronicMigration | 17:01 |
| rloo | NobodyCam: couple of questions. Why should they start using ironic commands. I'm hoping you'll add that part (eg, cuz they'll get more info in the response?) | 17:02 |
| jroll | because the proxy is read-only ;) | 17:03 |
| NobodyCam | rloo: vs the proxyed nova commands | 17:03 |
| NobodyCam | ya only node list node-show are proxied | 17:03 |
| rloo | NobodyCam: also, I'm not totally sure how baremetal-interface-list is coded, but I don't see it (no _ironic_proxy() call) in your proxy patch 120433. I'm guessing that if that is invoked, they'll get an empy list. | 17:03 |
| NobodyCam | oh did I miss that one | 17:04 |
| *** harlowja_away is now known as harlowja_ | 17:04 | |
| jroll | isn't index() the list? | 17:05 |
| devananda | rloo: i don't see baremetal-interface-list in the original code either | 17:05 |
| rloo | it's in the CLI and i did a 'nova' command and it was listed. | 17:06 |
| rloo | devananda: I've already forgotten, but I saw somewhere that it was getting the ports from node.interfaces. | 17:07 |
| devananda | rloo: NobodyCam: http://git.openstack.org/cgit/openstack/python-novaclient/tree/novaclient/v1_1/contrib/baremetal.py#n138 | 17:07 |
| devananda | it uses the /os-baremetal-nodes/ endpoint to fetch all the nodes | 17:07 |
| rloo | devananda: yeah, that's where I saw it :-) | 17:07 |
| devananda | *fetch the whole node | 17:07 |
| devananda | then prints only its interfaces | 17:08 |
| devananda | NobodyCam: is the proxy returning all the interfaces when it returns a node? | 17:09 |
| NobodyCam | devananda: yes I think so: https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/contrib/baremetal_nodes.py#L188 | 17:11 |
| rloo | NobodyCam: it doesn't look like it has the ports in the node.interfaces there, but line 230? | 17:13 |
| NobodyCam | oh so we only grab the port on a show? | 17:15 |
| NobodyCam | sorry several things going on atm | 17:15 |
| devananda | NobodyCam: 'interfaces': [], | 17:16 |
| devananda | NobodyCam: I think that needs to be populated | 17:16 |
| devananda | NobodyCam: eg, in the old code, we had: node['interfaces'] = [_interface_dict(i) for i in ifs] | 17:16 |
| devananda | NobodyCam: that is how the client was able to list interfaces, but now it's hardcoded to an empty list | 17:17 |
| *** yuanying has joined #openstack-ironic | 17:20 | |
| *** ChuckC has quit IRC | 17:25 | |
| NobodyCam | devananda: ack. | 17:25 |
| Shrews | devananda: do you want this (https://review.openstack.org/118467) ported to the nova driver as well? | 17:25 |
| *** ChuckC has joined #openstack-ironic | 17:26 | |
| *** MattMan has left #openstack-ironic | 17:26 | |
| *** kevinbenton has quit IRC | 17:27 | |
| *** romcheg1 has joined #openstack-ironic | 17:29 | |
| *** kevinbenton has joined #openstack-ironic | 17:32 | |
| *** athomas has quit IRC | 17:33 | |
| openstackgerrit | A change was merged to openstack/ironic: Allow clean_up with missing image ref https://review.openstack.org/121615 | 17:37 |
| *** athomas has joined #openstack-ironic | 17:39 | |
| *** linggao has quit IRC | 17:41 | |
| *** linggao has joined #openstack-ironic | 17:41 | |
| *** killer_prince is now known as lazy_prince | 17:42 | |
| *** igordcard has quit IRC | 17:49 | |
| devananda | Shrews: yup | 17:49 |
| *** yuanying has quit IRC | 17:53 | |
| *** andreykurilin_ has joined #openstack-ironic | 17:54 | |
| *** slagle_ is now known as slagle | 17:55 | |
| *** pelix has quit IRC | 17:56 | |
| *** andreykurilin has quit IRC | 17:56 | |
| * devananda spams the dev list | 17:58 | |
| Shrews | devananda: rloo: https://review.openstack.org/121937 | 18:01 |
| *** romcheg1 has quit IRC | 18:06 | |
| *** romcheg1 has joined #openstack-ironic | 18:07 | |
| rloo | thx Shrews | 18:08 |
| adam_g | hope this one goes smooth https://review.openstack.org/#/c/112134/ | 18:18 |
| adam_g | *fingers crossed* | 18:18 |
| devananda | ... :) | 18:19 |
| Shrews | devananda: i now no longer feel bad when i mistype your name, seeing you do it as well (Devananad) :-P | 18:20 |
| openstackgerrit | Sergey Lupersolsky proposed a change to openstack/ironic: Attempt to import necessary modules for IMPINative and iBoot fake drivers https://review.openstack.org/121715 | 18:24 |
| devananda | Shrews: LOL | 18:29 |
| devananda | that's what i get for replying to 5? messages at once | 18:29 |
| openstackgerrit | Sergey Lupersolsky proposed a change to openstack/ironic: Attempt to import modules for IMPINative and iBoot fake drivers https://review.openstack.org/121715 | 18:31 |
| *** tatyana has joined #openstack-ironic | 18:39 | |
| openstackgerrit | Sergey Lupersolsky proposed a change to openstack/ironic: Import modules for fake IPMINative/iBoot drivers https://review.openstack.org/121715 | 18:42 |
| *** romcheg1 has quit IRC | 18:45 | |
| *** yuanying has joined #openstack-ironic | 18:50 | |
| *** lazy_prince is now known as killer_prince | 19:00 | |
| *** enterprisedc has joined #openstack-ironic | 19:20 | |
| Madkiss | hello adam_g! good to read you! | 19:20 |
| adam_g | Madkiss, martin! :) hows it going? | 19:20 |
| *** harlowja_ has quit IRC | 19:22 | |
| *** harlowja has joined #openstack-ironic | 19:22 | |
| *** yuanying has quit IRC | 19:24 | |
| openstackgerrit | greghaynes proposed a change to openstack/ironic: WIP: Add conductor_id index to nodes https://review.openstack.org/109688 | 19:24 |
| *** pcrews has quit IRC | 19:25 | |
| devananda | ChuckC: re your email for resilient boot - -what about port bonding? | 19:28 |
| adam_g | i need to run out and will miss the meeting | 19:34 |
| adam_g | devananda, we're one patch away from having the grenade CI stuff functional : https://review.openstack.org/#/c/118700/ | 19:34 |
| devananda | awesome, ty | 19:35 |
| adam_g | i did some testing this morning with jroll's fix, it looks like post-migration start, stop, rebuild should all work | 19:35 |
| devananda | double awesome, thanks! | 19:35 |
| jroll | \o/ | 19:36 |
| adam_g | i cant think of other things that may be affected by the migration | 19:36 |
| NobodyCam | is the tc meeting in -alt today? | 19:36 |
| adam_g | but wanna poke more at that this afternoon | 19:36 |
| jroll | I think so, NobodyCam | 19:36 |
| adam_g | good luck !! | 19:36 |
| NobodyCam | :) ty jroll | 19:36 |
| NobodyCam | gah anyone used a tripleo seed (kvm) with real hardware undercloud where the ethernet port for pxe booting (on the seeds host) is eth1? | 19:39 |
| * NobodyCam feels he has forgotten something quite simple | 19:39 | |
| Shrews | NobodyCam always has the most specific problems :) | 19:40 |
| jroll | NobodyCam: I think the pxe driver just chooses the first port or something | 19:40 |
| rloo | anyone knows what check-tempest-dsvm-ironic-pxe_ssh is? It shows 'NOT_REGISTERED" for https://review.openstack.org/#/c/120764/ | 19:44 |
| NobodyCam | Shrews: I try... | 19:44 |
| NobodyCam | i think it somehting like reconfiguring one of the bridges | 19:45 |
| NobodyCam | Shrews: I do strange things | 19:45 |
| jroll | rloo: ooo! | 19:47 |
| jroll | rloo: that's part of adam_g's CI refactor | 19:47 |
| rloo | jroll: ?? | 19:47 |
| rloo | oh... so not good then | 19:47 |
| jroll | rloo: dunno if that NOT_REGISTERED thing is intentional, or why it happens | 19:48 |
| jroll | it did get merged | 19:48 |
| jroll | hmm | 19:48 |
| rloo | i couldn't get access to any logs | 19:48 |
| jroll | right, I don't believe it ran | 19:48 |
| rloo | which makes me wonder if it didn't get very far into the test. | 19:48 |
| jroll | oh wait | 19:49 |
| jroll | that's why jenkins -1'd | 19:49 |
| jroll | uh oh | 19:49 |
| jroll | adam_g: ^^ | 19:49 |
| * jroll brb | 19:49 | |
| rloo | jroll: adam_g had to run out. We'll have to wait til he's back. That same test is probably used for ironic code too, not just the client. | 19:49 |
| rloo | well, we don't have anything urgent that needs to get merged soon ;) | 19:50 |
| lucasagomes | it's almost timeeee! | 19:55 |
| JayF | rloo: just nobody mention in the TC meeting that our gate is broken and it's our fault :P | 19:55 |
| * NobodyCam get party hats ready | 19:55 | |
| rloo | JayF: mum's the word. And we don't know for sure who's fault it is. | 19:55 |
| JayF | :) | 19:56 |
| devananda | lol | 19:56 |
| JayF | sshhhh, don't tell devananda | 19:57 |
| JayF | he's on the tc | 19:57 |
| lucasagomes | :P | 19:57 |
| NobodyCam | even in this channel | 19:57 |
| * NobodyCam hasn't seen devananda switch hats | 19:58 | |
| lucasagomes | doens't look like it's our fault that NOT_REGISTERED thing tho | 19:58 |
| lucasagomes | seems like J lost track of the test | 19:58 |
| *** pcrews has joined #openstack-ironic | 19:58 | |
| jroll | lol | 19:58 |
| JayF | so is our meeting in -alt or -meeting? | 19:59 |
| * JayF is in both to be safe | 19:59 | |
| lucasagomes | -meeting | 19:59 |
| NobodyCam | gah | 19:59 |
| * jroll is always in both :P | 19:59 | |
| lucasagomes | my pint of smithwicks is ready | 20:00 |
| * jroll passes around the whiskey | 20:00 | |
| mordred | JayF: you should be careful - sometimes TC members lurk in your channel ... | 20:05 |
| JayF | mordred: it's not lurking if you say something :P | 20:06 |
| Shrews | mordred: not important ones | 20:06 |
| * JayF was kidding anyway | 20:06 | |
| * mordred throws wet cats at JayF and Shrews | 20:06 | |
| rloo | yeah, JayF is always joking around. We never believe anything he says. | 20:06 |
| jroll | I just assume the TC has access to NSA data and knows everything | 20:06 |
| NobodyCam | lol | 20:06 |
| jroll | rloo: you have no idea how true that is | 20:06 |
| JayF | /o\ | 20:06 |
| BadCub_ | hehehe | 20:06 |
| jroll | (source: I sit next to him) | 20:07 |
| rloo | jroll: lucky errrrr you | 20:07 |
| jroll | :P | 20:07 |
| jroll | he's punny, I don't mind | 20:07 |
| JayF | JoshNang jroll JayF and aweeks are a rectangular force for Ironic in Rackspace SFO office :P | 20:08 |
| JayF | we all sit beside each other | 20:08 |
| jroll | heh | 20:08 |
| aweeks | JayF: we form more of a rhombus | 20:08 |
| JayF | you're a rhombus | 20:08 |
| rloo | ahh, the 3-Js and A | 20:08 |
| lucasagomes | I'm the only one that thinks that "message system" looks yet more confusing than queue | 20:12 |
| * lucasagomes is talking about zaqar | 20:12 | |
| NobodyCam | lol | 20:12 |
| lucasagomes | message system implies in a lot of other assumptions like the delivery order of the messages etc | 20:13 |
| JayF | rloo: yeah, kylestev was part of the rhombus of power but he went back to school :P | 20:13 |
| lucasagomes | where a queue you already know such things, and it's common/well understood | 20:14 |
| kylestev | JayF: :P | 20:14 |
| JayF | kylestev: aweeks already moved into your desk | 20:14 |
| kylestev | welp | 20:15 |
| kylestev | i thought JoshNang had dibs? | 20:15 |
| JayF | This is the real world, no such thing as dibs. Only getting what you take. | 20:16 |
| JoshNang | kylestev: i considered, but i'm lazy :P | 20:17 |
| kylestev | gotcha haha | 20:17 |
| *** dprince has quit IRC | 20:20 | |
| *** yuanying has joined #openstack-ironic | 20:20 | |
| *** jasondotstar has quit IRC | 20:21 | |
| *** athomas has quit IRC | 20:23 | |
| JayF | What is the voting requirement for graduation? majority? 2/3rds? unanimous? | 20:30 |
| * NobodyCam is not sure | 20:32 | |
| lucasagomes | not sure too | 20:34 |
| * jroll votes -1 on https://review.openstack.org/#/c/120225/ just to see what happens | 20:35 | |
| NobodyCam | here we go | 20:35 |
| * Shrews stomps on jroll's head | 20:35 | |
| JayF | jroll: it'd involve Deva and a cluebat | 20:35 |
| jroll | this hatred implies I get a vote | 20:36 |
| rloo | jroll: I dare you | 20:36 |
| lucasagomes | lol | 20:36 |
| * jroll hides | 20:36 | |
| lucasagomes | I think we can't vote there | 20:37 |
| lucasagomes | yeah we can't :P | 20:37 |
| jroll | oh, interesting | 20:37 |
| jroll | TIL | 20:37 |
| NobodyCam | nice : https://review.openstack.org/#/c/120225 | 20:37 |
| NobodyCam | :) | 20:37 |
| JayF | I can't find anything on the wiki about how many votes it takes to win | 20:38 |
| * jroll pours champagne on deva's head | 20:39 | |
| devananda | :-D | 20:40 |
| NobodyCam | wooo looks good | 20:40 |
| *** pensu1 has quit IRC | 20:40 | |
| lucasagomes | uhul! | 20:40 |
| * lucasagomes opens a beer! | 20:40 | |
| devananda | we nwo have 9 +1's | 20:40 |
| NobodyCam | nice | 20:40 |
| lucasagomes | 10! | 20:40 |
| NobodyCam | congratz to ALL!!!!!! | 20:40 |
| lucasagomes | uhul | 20:40 |
| Shrews | Great! NOW we can haz voting jobz pleeeeeeez????????? | 20:40 |
| NobodyCam | its official 20:40 > ttx approves it | 20:41 |
| JoshNang | \o/ | 20:41 |
| jroll | Shrews: go put in the patch :P | 20:41 |
| JayF | http://i.kinja-img.com/gawker-media/image/upload/s--1cV3qjUF--/19c35oidyf35igif.gif | 20:41 |
| lucasagomes | Shrews, +1 | 20:41 |
| jroll | wait | 20:41 |
| jroll | don't do that yet | 20:41 |
| jroll | (broken jobs, remember) | 20:42 |
| NobodyCam | lol | 20:42 |
| Shrews | jroll: i'm not touching any jobs until adam's stuff is merged | 20:42 |
| JayF | devananda: I wonder how much having a public cloud deployment of Ironic helped as well w/r/t graduation :P | 20:42 |
| devananda | JayF: who can say ;) | 20:42 |
| * jroll pets Shrews, good boy | 20:42 | |
| JayF | or perhaps it's more that Ironic is good enough to power a public cloud | 20:42 |
| devananda | JayF: probably more than it hurt :p | 20:42 |
| JayF | proof of concept that Ironic works and can deal with ornery contributors | 20:42 |
| NobodyCam | is that ross pero | 20:42 |
| JayF | NobodyCam: he's like the ross perot of the 2000s | 20:43 |
| JayF | NobodyCam: aka Ron Paul :P | 20:43 |
| NobodyCam | lol | 20:43 |
| lucasagomes | Change has been successfully merged into the git repository. | 20:43 |
| lucasagomes | ohh yeah | 20:43 |
| NobodyCam | we are official | 20:43 |
| BadCub_ | Congrats! | 20:43 |
| Shrews | great. let's break stuff | 20:44 |
| NobodyCam | lol Shrews we did (our gate) | 20:44 |
| lucasagomes | alright I will finish my pint somewhere else :) | 20:49 |
| lucasagomes | thank you all, congratssss | 20:49 |
| NobodyCam | have a great night lucasagomes | 20:50 |
| devananda | lucasagomes: cheers! g'night | 20:50 |
| lucasagomes | good night! | 20:50 |
| *** tatyana has quit IRC | 20:50 | |
| lucasagomes | \o/ | 20:50 |
| jroll | night lucas :) | 20:50 |
| *** lucasagomes is now known as lucas-w00t | 20:50 | |
| *** marcoemorais has quit IRC | 20:52 | |
| openstackgerrit | David Shrewsbury proposed a change to openstack/ironic: Remove 'incubated' documentation theme https://review.openstack.org/121977 | 20:52 |
| *** marcoemorais has joined #openstack-ironic | 20:52 | |
| NobodyCam | Shrews: +2'd ^^^ | 20:53 |
| lucas-w00t | Shrews, ^ that's for K right!? | 20:53 |
| *** yuanying has quit IRC | 20:53 | |
| * lucas-w00t is still here talking | 20:53 | |
| Shrews | lucas-w00t: do we need to wait? hrm... | 20:54 |
| NobodyCam | lol nice nick lucas-w00t | 20:54 |
| lucas-w00t | I think so because it's in J (RC) we still not integrated | 20:54 |
| lucas-w00t | AFAIUI | 20:54 |
| *** marcoemorais1 has joined #openstack-ironic | 20:54 | |
| Shrews | lucas-w00t: you are probably right | 20:54 |
| NobodyCam | ya | 20:55 |
| NobodyCam | we could ask devananda | 20:55 |
| NobodyCam | he's on the TC I hear | 20:55 |
| devananda | I would punt that to annegentle as the docs PTL | 20:55 |
| NobodyCam | :-p | 20:55 |
| devananda | but fwiw, I think it's fine | 20:55 |
| Shrews | I WIP'd it for now | 20:56 |
| Shrews | i'll ask anne | 20:56 |
| lucas-w00t | ignore my vote and land it if it's not the case | 20:56 |
| lucas-w00t | now I'll be afk for real | 20:56 |
| NobodyCam | have a pint for /me lucas-w00t | 20:56 |
| *** andreykurilin_ has quit IRC | 20:57 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Add a doc note about the vendor_passthru endpoint https://review.openstack.org/121678 | 20:57 |
| lucas-w00t | NobodyCam, ++ Will do! !! | 20:57 |
| *** marcoemorais has quit IRC | 20:58 | |
| NobodyCam | :) | 20:58 |
| NobodyCam | brb | 20:58 |
| *** marcoemorais1 has quit IRC | 21:02 | |
| *** marcoemorais has joined #openstack-ironic | 21:02 | |
| *** marcoemorais has quit IRC | 21:02 | |
| *** marcoemorais has joined #openstack-ironic | 21:03 | |
| anteaya | congratulations! | 21:06 |
| JayF | thanks | 21:07 |
| devananda | anteaya: thanks!! | 21:09 |
| *** linggao has quit IRC | 21:09 | |
| anteaya | :D | 21:11 |
| NobodyCam | thank you anteaya :) | 21:11 |
| anteaya | so proud of all of you | 21:11 |
| anteaya | you did it | 21:11 |
| NobodyCam | we all did it, | 21:11 |
| JayF | devananda: very congrats | 21:11 |
| jroll | thanks anteaya :) | 21:14 |
| anteaya | :D | 21:16 |
| adam_g | back now | 21:19 |
| adam_g | check-tempest-dsvm-ironic-pxe_ssh-postgres should be running now | 21:20 |
| adam_g | i think there was a window where the new configuration was applied to zuul but not to the jenkins slaves yet | 21:20 |
| rloo | adam_g: so recheck? | 21:20 |
| adam_g | rloo, yeah, it looks like its at least running now for the jobs that are in the check pipeline | 21:21 |
| adam_g | 121678 | 21:21 |
| adam_g | 121977 | 21:22 |
| jroll | nice | 21:22 |
| rloo | will do adam_g. thx. | 21:23 |
| * devananda sees lots of passing jenkins tests | 21:39 | |
| adam_g | reconfiged jobs look okay, at least according to https://review.openstack.org/#/c/109688/ | 21:40 |
| devananda | adam_g: http://no-carrier.net/~adam/openstack/ironic_gate_status.html should be updated to the new test names :) | 21:40 |
| adam_g | devananda, doing that now :) | 21:40 |
| devananda | I did it in the URL to see the overall stats | 21:40 |
| devananda | mostly green since the change | 21:40 |
| devananda | adam_g: I suspect the answer is "no" but can you do something similar for the ibm-xcat test results? | 21:41 |
| jroll | adam_g: nice work :) | 21:41 |
| devananda | I couldn't find them in the graphite list (but didn't look that hard) | 21:41 |
| adam_g | devananda, not sure. URL? | 21:42 |
| *** penick has joined #openstack-ironic | 21:47 | |
| *** yuanying has joined #openstack-ironic | 21:50 | |
| devananda | adam_g: url? I dunno - I just see their third-party systems posting in Jenkins now | 21:52 |
| devananda | "check-ironic-xcat-third-party" | 21:53 |
| adam_g | devananda, oh, sorry. misread you. you mean add those jobs to the tracking page? | 21:53 |
| devananda | adam_g: or have a separate page for them | 21:54 |
| adam_g | devananda, sure | 21:54 |
| devananda | adam_g: I think I'd like to link to graphs of each driver's tests from somewhere on https://wiki.openstack.org/wiki/Ironic/Drivers | 21:54 |
| devananda | so that page becomes a clearinghouse for info on each driver's status (how well tested is it? are they passing? etc) | 21:54 |
| * devananda is still formulating ideas on how to represent that | 21:55 | |
| adam_g | devananda, sure | 21:55 |
| adam_g | the js graphite stuff jogo is using is much prettier http://jogo.github.io/gate/ | 21:55 |
| *** ChuckC has quit IRC | 21:56 | |
| *** penick has quit IRC | 22:00 | |
| *** marcoemorais has quit IRC | 22:05 | |
| *** marcoemorais has joined #openstack-ironic | 22:06 | |
| *** marcoemorais has quit IRC | 22:06 | |
| *** marcoemorais has joined #openstack-ironic | 22:06 | |
| *** r-daneel__ has quit IRC | 22:09 | |
| devananda | so it is | 22:11 |
| devananda | well | 22:11 |
| devananda | i like them both, but his has more colours :) | 22:11 |
| *** ChuckC has joined #openstack-ironic | 22:15 | |
| adam_g | devananda, oh hmm. we cant really easily scrape the third party systems using the same script | 22:18 |
| devananda | adam_g: that's what I was afraid of | 22:19 |
| adam_g | we're combing through job statuses on the openstack jenkins server | 22:19 |
| adam_g | and oh man, xcat uses sourceforge to host its CI results | 22:20 |
| adam_g | 10 seconds of add spam before i can look at any of them :( | 22:20 |
| jroll | adam_g: and you can't view them in browser | 22:21 |
| jroll | it's great | 22:21 |
| adam_g | better than nothing? | 22:21 |
| jroll | meh | 22:22 |
| adam_g | cant debug failure, must buy a toyota | 22:22 |
| jroll | I'd have to care really hard about a fail to dive into those logs | 22:22 |
| *** yuanying has quit IRC | 22:23 | |
| adam_g | devananda, http://no-carrier.net/~adam/openstack/ironic_gate_status.html updated. easy to add new jobs now, so let me know if there are others we should track | 22:26 |
| adam_g | one or two of the jenkins slaves looks like they are lagging behind updates, so that UNREGISTERED error might show up now and again | 22:26 |
| devananda | yea, hosting that on sf is terrible | 22:27 |
| devananda | they've already been asked to move it somewhere browsable (irc conversation) | 22:28 |
| adam_g | what is xcat testing specifically? | 22:28 |
| devananda | adam_g: awesome, ty | 22:28 |
| devananda | adam_g: good question. I believe the intent is to actually test with ipminative / pyghmi | 22:29 |
| devananda | but I'd have to trawl through sf ads to find out | 22:29 |
| adam_g | maybe it can be moved to angelfire or geocities | 22:30 |
| *** jcoufal has quit IRC | 22:32 | |
| devananda | :) | 22:33 |
| devananda | anyone planning to stay a few days in Paris after the summit? | 22:33 |
| *** derekh has joined #openstack-ironic | 22:34 | |
| JayF | I think I have a day on each side | 22:34 |
| JayF | barring flight issues | 22:34 |
| jroll | devananda: my SO and I leave the following wednesday | 22:34 |
| adam_g | was gonna stay a while but wife cant come after all. may take a train / flight somewhere new for a few days before heading home | 22:34 |
| devananda | nice | 22:35 |
| devananda | I'll be there with my SO wednesday before the summit | 22:35 |
| devananda | am debating flying back sunday or monday | 22:35 |
| jroll | I'd be up for dinner or something, dunno about her :) | 22:36 |
| ChuckC | devananda: hi, sorry, I was heads down on a problem for a while today | 22:36 |
| JayF | The summits are never aligned with a Summer so I'll never be able to take my wife :( | 22:37 |
| * JayF jealous of all the discount Paris vacations people are having | 22:38 | |
| ChuckC | devananda: I decided to take small bites, so just doing resilient boot in that spec | 22:38 |
| *** r-daneel__ has joined #openstack-ironic | 22:39 | |
| ChuckC | devananda: and I don't really know how to address issues coming from reviewers who say bonding is in the guest OS, so neutron can't reach it | 22:39 |
| ChuckC | devananda: thoughts? | 22:39 |
| JayF | ChuckC: I missed a lot of context; but I care a lot about neutron and bonding | 22:40 |
| JayF | ChuckC: primarily because we run an ironic cloud that uses both of those things (Rackspace OnMetal) | 22:40 |
| * jroll replies to that email and adds [Ironic] | 22:40 | |
| ChuckC | jroll: thanks | 22:41 |
| devananda | ChuckC: bonding has to be understood by the switch when we're talking about physical ports and VLANs and whatnaught. | 22:41 |
| devananda | at least that's my understanding. IANANE | 22:42 |
| * JayF links relevant things ChuckC may care about | 22:42 | |
| jroll | JayF: openstack-dev, Subject: [openstack-dev] [neutron] support resilient network boot of ironic server | 22:42 |
| jroll | devananda: this is true | 22:42 |
| JayF | ChuckC: If you figured out a way to pxe boot bond something, that'd be pretty wonderful | 22:42 |
| jroll | ChuckC: your blueprint link doesn't work for me, got a different one? | 22:43 |
| jroll | ChuckC: or somewhere discussion is happening? | 22:43 |
| ChuckC | jroll: no, it's just part of the neutron way: spec first, then blueprint | 22:44 |
| *** r-daneel__ has quit IRC | 22:44 | |
| jroll | ChuckC: right... where is the spec? :) | 22:44 |
| jroll | (other than email) | 22:44 |
| ChuckC | jroll: I'd hoped an email discussion would happen first, but no replies today | 22:44 |
| jroll | ChuckC: got it, I saw "I don't really know how to address issues coming from reviewers who say..." and thought maybe I could read that somewhere :) | 22:45 |
| jroll | anyhow, as devananda said, the switch has to know about the bonding, as does the ironic instance | 22:46 |
| jroll | we've done some work toward this at rackspace, if you're interested | 22:46 |
| ChuckC | jroll: I need to understand better what neutron can/should know about physical switches | 22:47 |
| ChuckC | jroll: sure, I'd be glad to take a look! | 22:47 |
| * jroll will find some things | 22:47 | |
| ChuckC | jroll: JayF: many thanks :) | 22:48 |
| * jroll makes an etherpad | 22:51 | |
| openstackgerrit | A change was merged to openstack/python-ironicclient: fixes help string for driver-list https://review.openstack.org/120764 | 22:54 |
| *** r-daneel__ has joined #openstack-ironic | 22:56 | |
| * devananda books flights | 22:58 | |
| * ChuckC had already booked flights, then found out I might not go! | 22:59 | |
| devananda | d'oh | 23:01 |
| JayF | get your celebratory +1s in https://review.openstack.org/#/c/122007/1 (deprecation of nova-bm) | 23:01 |
| JayF | ChuckC: My suggestion: Go anyway; find a company there to work for that will pay your way next time | 23:01 |
| ChuckC | either my boss jumped the gun a bit, or we'll find a way to get me there, I guess ;) | 23:03 |
| *** marcoemorais has quit IRC | 23:05 | |
| *** marcoemorais has joined #openstack-ironic | 23:05 | |
| *** marcoemorais has quit IRC | 23:08 | |
| *** marcoemorais has joined #openstack-ironic | 23:08 | |
| *** marcoemorais has quit IRC | 23:08 | |
| *** marcoemorais has joined #openstack-ironic | 23:09 | |
| jroll | ChuckC: https://etherpad.openstack.org/p/ironic-neutron-bonding | 23:10 |
| jroll | devananda: I expect you'll be interested in this ^ | 23:10 |
| jroll | JayF JoshNang please to correct any wrong-ness here | 23:11 |
| * ChuckC looking | 23:11 | |
| jroll | ChuckC: I'll brb, feel free to spam with questions while I'm gone :) | 23:12 |
| JayF | me, JoshNang and morgabra know things about this setup too | 23:12 |
| JayF | so we can answer as well | 23:12 |
| * JayF brb | 23:12 | |
| *** yuanying has joined #openstack-ironic | 23:18 | |
| *** ryanpetrello has quit IRC | 23:23 | |
| *** marcoemorais has quit IRC | 23:36 | |
| *** marcoemorais has joined #openstack-ironic | 23:36 | |
| * ChuckC needs some time to digest this stuff | 23:38 | |
| ChuckC | maybe tomorrow :) | 23:38 |
| jroll | heh | 23:38 |
| jroll | I just braindumped in there | 23:38 |
| jroll | dunno if everything makes sense | 23:38 |
| devananda | jroll: added a link on the whiteboard | 23:39 |
| jroll | nice | 23:39 |
| jroll | thanks | 23:39 |
| *** Guest72739 is now known as mgagne | 23:42 | |
| *** mgagne has joined #openstack-ironic | 23:42 | |
| *** derekh has quit IRC | 23:45 | |
| *** ekarlso has quit IRC | 23:48 | |
| *** ekarlso has joined #openstack-ironic | 23:48 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
