*** rama_y has quit IRC | 00:01 | |
openstackgerrit | Merged openstack/ironic-lib: Remove tests and common.i18n from autodoc https://review.openstack.org/381090 | 00:08 |
---|---|---|
*** aNuposic has quit IRC | 00:10 | |
*** penick has quit IRC | 00:13 | |
*** penick has joined #openstack-ironic | 00:14 | |
*** sdake_ has quit IRC | 00:16 | |
*** thrash is now known as thrash|g0ne | 00:25 | |
*** pester has quit IRC | 00:30 | |
openstackgerrit | Merged openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/376136 | 00:31 |
*** mtanino has quit IRC | 00:32 | |
*** penick has quit IRC | 00:33 | |
*** hj-hpe has quit IRC | 00:37 | |
*** jvgrant_ has joined #openstack-ironic | 00:39 | |
*** jvgrant has quit IRC | 00:43 | |
*** hoangcx has joined #openstack-ironic | 00:48 | |
*** sdake has joined #openstack-ironic | 00:53 | |
openstackgerrit | Merged openstack/ironic: Fix iLO drivers to not clear local_gb if its not detected https://review.openstack.org/341752 | 00:58 |
openstackgerrit | Merged openstack/ironic: Add more details to MIGRATIONS_TIMEOUT note https://review.openstack.org/375048 | 00:59 |
openstackgerrit | Merged openstack/ironic: Change links to point to new install guide https://review.openstack.org/378640 | 00:59 |
*** BadCub has quit IRC | 01:00 | |
*** phuongnh has joined #openstack-ironic | 01:00 | |
*** Ng has quit IRC | 01:01 | |
*** wajdi has quit IRC | 01:01 | |
*** wajdi has joined #openstack-ironic | 01:02 | |
*** kun_huang has quit IRC | 01:04 | |
*** Sukhdev has quit IRC | 01:04 | |
*** syed_ has quit IRC | 01:05 | |
*** wajdi has quit IRC | 01:06 | |
*** Ng has joined #openstack-ironic | 01:08 | |
*** baoli has quit IRC | 01:09 | |
*** BadCub has joined #openstack-ironic | 01:14 | |
*** kun_huang has joined #openstack-ironic | 01:17 | |
*** syed_ has joined #openstack-ironic | 01:17 | |
*** baoli has joined #openstack-ironic | 01:20 | |
*** tuanluong has joined #openstack-ironic | 01:21 | |
*** rloo has quit IRC | 01:23 | |
*** rama_y has joined #openstack-ironic | 01:24 | |
*** baoli has quit IRC | 01:27 | |
*** [1]cdearborn has quit IRC | 01:29 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/377446 | 01:29 |
*** rbudden has joined #openstack-ironic | 01:35 | |
openstackgerrit | Merged openstack/ironic: Fixing Typo https://review.openstack.org/376564 | 01:38 |
*** bdeschenes has quit IRC | 01:48 | |
*** rama_y has quit IRC | 01:50 | |
*** tiendc has joined #openstack-ironic | 01:58 | |
*** rbudden has quit IRC | 02:00 | |
*** mtanino has joined #openstack-ironic | 02:01 | |
*** rbudden has joined #openstack-ironic | 02:03 | |
*** syed_ has quit IRC | 02:04 | |
openstackgerrit | Merged openstack/ironic-python-agent: Fixed typo in doc file https://review.openstack.org/378137 | 02:15 |
openstackgerrit | Merged openstack/ironic-lib: Add prefix "$" for command examples https://review.openstack.org/378611 | 02:17 |
*** sdake has quit IRC | 02:18 | |
*** amotoki has joined #openstack-ironic | 02:19 | |
*** sdake has joined #openstack-ironic | 02:20 | |
*** amotoki has quit IRC | 02:31 | |
*** rbudden has quit IRC | 02:33 | |
*** yuanying_ has quit IRC | 02:39 | |
*** amotoki has joined #openstack-ironic | 02:43 | |
*** amotoki_ has joined #openstack-ironic | 03:04 | |
*** amotoki has quit IRC | 03:07 | |
*** Sukhdev has joined #openstack-ironic | 03:09 | |
openstackgerrit | Merged openstack/ironic-inspector: TrivialFix: Fix typo in the configuration file https://review.openstack.org/379108 | 03:10 |
openstackgerrit | Merged openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/378819 | 03:10 |
*** Sukhdev has quit IRC | 03:13 | |
*** appu103_ has joined #openstack-ironic | 03:25 | |
*** vikrant has joined #openstack-ironic | 03:30 | |
*** amotoki has joined #openstack-ironic | 03:35 | |
*** amotoki has quit IRC | 03:36 | |
*** amotoki_ has quit IRC | 03:36 | |
*** sdake has quit IRC | 03:43 | |
*** sdake has joined #openstack-ironic | 03:43 | |
*** links has joined #openstack-ironic | 03:45 | |
*** vmud213 has joined #openstack-ironic | 03:46 | |
*** vishwanathj has quit IRC | 03:50 | |
*** yuanying has joined #openstack-ironic | 04:02 | |
*** wajdi has joined #openstack-ironic | 04:02 | |
*** amotoki has joined #openstack-ironic | 04:06 | |
*** yuanying has quit IRC | 04:12 | |
*** yuanying has joined #openstack-ironic | 04:13 | |
*** amotoki_ has joined #openstack-ironic | 04:19 | |
*** amotoki has quit IRC | 04:23 | |
*** sdake has quit IRC | 04:25 | |
*** wajdi has quit IRC | 04:36 | |
*** yuanying has quit IRC | 04:36 | |
*** wajdi has joined #openstack-ironic | 04:36 | |
*** yuanying has joined #openstack-ironic | 04:42 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/377446 | 04:47 |
*** syed_ has joined #openstack-ironic | 04:50 | |
*** sdake has joined #openstack-ironic | 04:51 | |
*** HenryG has quit IRC | 04:51 | |
*** HenryG has joined #openstack-ironic | 04:52 | |
*** bjolo_ has joined #openstack-ironic | 04:58 | |
*** amotoki_ has quit IRC | 04:59 | |
*** jaosorior has joined #openstack-ironic | 05:07 | |
*** wajdi has quit IRC | 05:11 | |
*** penick has joined #openstack-ironic | 05:18 | |
*** jaosorior has quit IRC | 05:35 | |
*** jaosorior has joined #openstack-ironic | 05:35 | |
*** moshele has joined #openstack-ironic | 05:36 | |
*** bks has joined #openstack-ironic | 05:40 | |
*** amotoki has joined #openstack-ironic | 05:43 | |
*** amotoki has quit IRC | 05:57 | |
*** wajdi has joined #openstack-ironic | 06:02 | |
*** mjura has joined #openstack-ironic | 06:06 | |
*** vsaienko has joined #openstack-ironic | 06:08 | |
*** mjura has quit IRC | 06:12 | |
*** mjura has joined #openstack-ironic | 06:12 | |
*** yuanying has quit IRC | 06:16 | |
*** bjolo_ has quit IRC | 06:24 | |
*** jtomasek_ has joined #openstack-ironic | 06:36 | |
*** milan has joined #openstack-ironic | 06:36 | |
milan | morning Ironic! :) | 06:39 |
*** vsaienko has quit IRC | 06:39 | |
*** pcaruana has joined #openstack-ironic | 06:46 | |
*** jtomasek_ has quit IRC | 06:50 | |
*** rcernin has joined #openstack-ironic | 06:53 | |
openstackgerrit | Hironori Shiina proposed openstack/ironic: Keep numbering of list in Install Guide https://review.openstack.org/382169 | 07:00 |
*** fxpester has joined #openstack-ironic | 07:04 | |
*** AlexeyAbashkin has quit IRC | 07:07 | |
*** tesseract- has joined #openstack-ironic | 07:07 | |
*** rbartal has joined #openstack-ironic | 07:08 | |
*** rcernin has quit IRC | 07:12 | |
*** rcernin has joined #openstack-ironic | 07:12 | |
*** athomas has joined #openstack-ironic | 07:15 | |
*** amotoki has joined #openstack-ironic | 07:20 | |
*** hogepodge has quit IRC | 07:21 | |
*** amotoki has quit IRC | 07:32 | |
*** jpich has joined #openstack-ironic | 07:33 | |
*** syed_ has quit IRC | 07:34 | |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector-specs: Endpoint to list introspection statuses https://review.openstack.org/375045 | 07:35 |
*** e0ne has joined #openstack-ironic | 07:36 | |
*** amotoki has joined #openstack-ironic | 07:43 | |
*** AlexeyAbashkin has joined #openstack-ironic | 07:46 | |
*** tuanluong has quit IRC | 07:54 | |
*** amotoki has quit IRC | 07:56 | |
pas-ha | morning Ironic :) and milan :) | 07:59 |
*** yolanda has joined #openstack-ironic | 07:59 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-ironic | 08:00 | |
yolanda | hi, i'm trying some iscsi deployment on ironic, and i'm getting following error: http://paste.openstack.org/show/584381/ | 08:00 |
yolanda | invalid error code 65280 on Re-reading iSCSI luns, anyone has experimented it? | 08:01 |
*** yuanying has joined #openstack-ironic | 08:01 | |
*** dnikishov has joined #openstack-ironic | 08:08 | |
*** dnikishov has quit IRC | 08:10 | |
*** dnikishov has joined #openstack-ironic | 08:10 | |
*** hogepodge has joined #openstack-ironic | 08:11 | |
*** appu103_ has quit IRC | 08:14 | |
*** appu103_ has joined #openstack-ironic | 08:16 | |
*** mgoddard has joined #openstack-ironic | 08:16 | |
*** ccamacho has joined #openstack-ironic | 08:16 | |
*** amotoki has joined #openstack-ironic | 08:17 | |
*** ohamada has joined #openstack-ironic | 08:18 | |
*** Romanenko_K has joined #openstack-ironic | 08:25 | |
*** dtantsur|afk is now known as dtantsur | 08:26 | |
dtantsur | morning Ironic, pas-ha, milan, yolanda | 08:26 |
dtantsur | milan, you're an early bird today, aren't you? | 08:26 |
dtantsur | yolanda, no, never... frankly speaking, the iscsi deployment is more fragile than it should be | 08:27 |
yolanda | dtantsur, so we are testing some use case now, that is to start using full disk images, and partitioned with swap, logical partitions... in ironic | 08:28 |
yolanda | so trying to debug a bit more | 08:28 |
dtantsur | yolanda, I'd recommend using agent_* drivers with whole disk images for now. using pxe_* drivers with them is pretty new. | 08:28 |
openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Tests for testing port-create command https://review.openstack.org/291802 | 08:33 |
yolanda | dtantsur, i'm testing under tripleo scope, can it be customized? | 08:36 |
yolanda | how do you specify pxe_ or agent_ ? | 08:36 |
dtantsur | yolanda, hmm, I think tripleo will need additional configuration to use agent_* drivers... | 08:37 |
yolanda | dtantsur, i'm trying to investigate what is needed in order for TripleO to start using full disk images, instead of partition ones. So if that config change is needed, i'd have to document as well | 08:38 |
dtantsur | yolanda, in theory, whole disk images work with pxe drivers too. your error does not seem to be directly related though | 08:39 |
yolanda | yep, i'm adding more debug messages to get exactly the point of failure | 08:40 |
yolanda | i'm also getting a ERROR ironic.drivers.modules.iscsi_deploy [req-edc31351-0cae-49b6-afd4-d141fe057254 - - - - -] Deploy failed for instance 568ddde4-d46f-4160-9d11-b3929dd67389. Error: int() argument must be a string or a number, not 'tuple' | 08:40 |
yolanda | so i was wondering if the first error was a red herring | 08:40 |
dtantsur | yolanda, int problem was fixed in ironic-lib master, but we can't do a release yet til next Monday :( | 08:41 |
dtantsur | because of #openstack-release processes | 08:42 |
yolanda | ah... so i can install from source | 08:42 |
yolanda | so i better install ironic and ironic-lib from git to test? | 08:43 |
dtantsur | yolanda, you don't need ironic from git, you just need to pick https://github.com/openstack/ironic-lib/commit/7323bf50e60383fb05f0655062cb4b830174888f | 08:43 |
dtantsur | also it was backported to stable/newton, so no need for master, if you use newton | 08:43 |
yolanda | ah thx | 08:43 |
dtantsur | (still, not released, so RDO won't pick it, for example) | 08:44 |
yolanda | you saved my day :) | 08:44 |
*** strigazi_AFK is now known as strigazi | 08:44 | |
dtantsur | I'm not sure it fixes the iscsi problem, but the int() problem is real and breaks deployments with config drives | 08:44 |
yolanda | yep, hitting that | 08:44 |
dtantsur | (and tripleo forces config drives) | 08:44 |
yolanda | so i'll apply that fix and retest to see if i arrive further | 08:45 |
dtantsur | thanks! | 08:45 |
*** derekh has joined #openstack-ironic | 08:46 | |
pas-ha | milan: 've just seen you question re assertRaises as context manager - AFAIK it depends on which version you use. the assertRaises from testtools is not returning the context manager, but the exception itself. the one from Python's stdlib returns the context manager | 08:47 |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Add CRUD and node states notifications to ironic https://review.openstack.org/347242 | 08:47 |
pas-ha | and AFAIU we use testtools' version everywhere, as our TestCases are inherited from testtools.TestCase | 08:48 |
openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with node-get command https://review.openstack.org/342060 | 08:48 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Adds the default implementation of get_supported_power_state https://review.openstack.org/382194 | 08:50 |
*** jaosorior is now known as jaosorior_lunch | 08:50 | |
openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with Chassis. https://review.openstack.org/343796 | 08:55 |
yolanda | dtantsur, that worked! i have ironic nodes on active state now | 08:58 |
dtantsur | \o/ | 08:58 |
milan | morning dtantsur pas-ha yolanda! :) | 08:58 |
milan | pas-ha, the thing is oslotest changed the behaviour "overnight"; before assertRaises would return the ctx manager, now that method is taken from testutils instead (hence different call pattern&no return value) | 08:59 |
*** sdake has quit IRC | 08:59 | |
milan | but never mind, I've updated my patch not to depend on the ctx manager anymore :-/ | 09:00 |
*** amotoki has quit IRC | 09:00 | |
pas-ha | IMO that's a breaking change that should be reported | 09:00 |
milan | pas-ha, maybe | 09:01 |
*** awiddersheim has quit IRC | 09:06 | |
*** awiddersheim has joined #openstack-ironic | 09:11 | |
*** skazi has quit IRC | 09:11 | |
*** dnikishov has quit IRC | 09:13 | |
fxpester | hi all, I`m trying to create rhel7 image for ironic, and it can`t boot - looks exactly like this bug - https://bugs.launchpad.net/diskimage-builder/+bug/1477179 | 09:13 |
openstack | Launchpad bug 1477179 in diskimage-builder "centos 6 image doesn't boot after creation" [Medium,Triaged] | 09:13 |
*** skazi has joined #openstack-ironic | 09:14 | |
dtantsur | fxpester, which exactly command are you using? you can look what tripleo adds to their images: https://github.com/openstack/tripleo-common/blob/master/image-yaml/overcloud-images.yaml#L7 | 09:16 |
milan | dtantsur, would you mind review on: https://review.openstack.org/#/c/375045/ | 09:17 |
patchbot | patch 375045 - ironic-inspector-specs - Endpoint to list introspection statuses | 09:17 |
dtantsur | milan, sure, I'll get to it today | 09:18 |
milan | just 312 changes ;) | 09:18 |
milan | thx, would sort of unblock me | 09:18 |
dtantsur | milan, if you have nothing else to do, I have a lot of exciting stuff in backlog to keep you busy ;) | 09:20 |
milan | dtantsur, I'd love to finish what's pending atm ;) | 09:20 |
dtantsur | hehe | 09:20 |
*** skazi has quit IRC | 09:22 | |
fxpester | dtantsur: I use minimum elements: `rhel7 baremetal dhcp-all-interfaces grub2` tripeo use the same, will try to get latest dib from git... | 09:26 |
dtantsur | yes please.. your list looks correct. maybe the image is not to blame then. | 09:26 |
aarefiev | morning! | 09:27 |
dtantsur | hey-hey aarefiev | 09:27 |
*** amotoki has joined #openstack-ironic | 09:31 | |
*** amotoki has quit IRC | 09:31 | |
*** amotoki has joined #openstack-ironic | 09:31 | |
milan | morning aarefiev! :) | 09:32 |
aarefiev | morning dtantsur, milan | 09:32 |
milan | aarefiev, lemme fix the sorting keys! :) | 09:32 |
aarefiev | milan: thanks, I like proposed filtering | 09:33 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal node commands https://review.openstack.org/301687 | 09:33 |
aarefiev | good work | 09:33 |
aarefiev | dtantsur: what do you think about job with radosgw backend in inspector | 09:34 |
dtantsur | aarefiev, I'm not against, even though I'm not sure what our benefit is | 09:35 |
aarefiev | dtantsur: catching possible bugs, to be sure it works, there are a lot of envs with ceph | 09:36 |
aarefiev | we can switch discovery job for example | 09:36 |
aarefiev | and not adding new one | 09:36 |
dtantsur | makes sense | 09:36 |
aarefiev | it will test ironic as well | 09:38 |
dtantsur | only if we use agent_* drivers there, right? | 09:39 |
* dtantsur is unsure which driver we use | 09:39 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-python-agent: [tinyipa] remove trailing slash from TC repo URL https://review.openstack.org/382227 | 09:40 |
aarefiev | dtantsur: I think so, but unsure too | 09:40 |
*** skazi has joined #openstack-ironic | 09:41 | |
*** mgould|afk is now known as mgould | 09:44 | |
mgould | morning Ironic! | 09:44 |
*** jaosorior_lunch is now known as jaosorior | 09:46 | |
milan | aarefiev, thx! :) | 09:50 |
milan | morning mgould | 09:50 |
mgould | morning milan aarefiev dtantsur fxpester jaosorior | 09:56 |
aarefiev | morning mgould | 09:56 |
openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with Chassis. https://review.openstack.org/343796 | 09:56 |
openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with chassis-update https://review.openstack.org/344715 | 09:56 |
jaosorior | morning mgould | 09:56 |
openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with node-list command https://review.openstack.org/360549 | 09:58 |
openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with node-port-list. https://review.openstack.org/360636 | 10:00 |
vdrok | good morning ironic, milan pas-ha dtantsur aarefiev mgould | 10:04 |
milan | morning vdrok! :) | 10:05 |
aarefiev | hey vdrok | 10:05 |
mgould | morning vdrok | 10:06 |
fxpester | ok, no luck even with fresh dib... | 10:09 |
fxpester | maybe anyone messed with error: "grubby fatal error: unable to find a suitable template" when building rhel images with dib ? this is breaking grub.conf so image can`t boot. | 10:10 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal port commands https://review.openstack.org/365692 | 10:10 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for 'baremetal port create' command https://review.openstack.org/332880 | 10:11 |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector-specs: Endpoint to list introspection statuses https://review.openstack.org/375045 | 10:13 |
milan | aarefiev, ^ I've updated the API accordingly too | 10:14 |
*** sambetts|afk is now known as sambetts | 10:21 | |
sambetts | Morning all | 10:21 |
milan | morning sambetts! :) | 10:21 |
*** mgoddard_ has joined #openstack-ironic | 10:22 | |
*** mgoddard has quit IRC | 10:25 | |
aarefiev | milan: thanks | 10:30 |
aarefiev | morning sambetts | 10:31 |
*** hoangcx has quit IRC | 10:31 | |
milan | aarefiev, my pleasure ;) | 10:31 |
milan | aarefiev, btw the flask thing returns the api queries as a {query_name: [value, value, ...], query_name:....} dict so it should be possible to process that; the pagination code supports multiple sorting directions | 10:33 |
dtantsur | morning vdrok, sambetts | 10:33 |
dtantsur | fxpester, this rings some bells.. lemme try to remember | 10:33 |
dtantsur | hmm, no, it was for Fedora | 10:36 |
dtantsur | fxpester, try asking on #tripleo, there are folks much more experienced with DIB there | 10:36 |
*** athomas has quit IRC | 10:40 | |
vdrok | morning sambetts | 10:44 |
*** thrash|g0ne is now known as thrash | 10:54 | |
*** awiddersheim has quit IRC | 10:57 | |
aarefiev | milan: why do we need to return multiple queries | 11:01 |
*** athomas has joined #openstack-ironic | 11:03 | |
*** tiendc has quit IRC | 11:04 | |
milan | aarefiev, so you can specify e.g multiple states as a query | 11:10 |
aarefiev | milan: yeah, I specify multiple states, and return one query with provided states, no? | 11:11 |
milan | aarefiev, yes, but HTTP doesn't support types over queries | 11:11 |
aarefiev | or your mean provide multiple queries | 11:12 |
milan | so multiplicity is usually expressed like repetition | 11:12 |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Add CRUD and node states notifications to ironic https://review.openstack.org/347242 | 11:12 |
milan | aarefiev, so a sample query would be "?state=waiting&state=processing" | 11:12 |
milan | in python: {'state': ['waiting', 'processing'], ...} | 11:13 |
aarefiev | ahh, ok, I thought about other query | 11:13 |
aarefiev | yeah, nice | 11:13 |
milan | aarefiev, so in the code I'd reduce the lists with an (sql) 'or' statement and reduce the dict of lists with an 'and' statement | 11:14 |
*** ccamacho is now known as ccamacho|lunch | 11:15 | |
* milan 's coffee is ready \o/ :D | 11:16 | |
openstackgerrit | Merged openstack/bifrost: Add read and execute perms to httpboot and tftpboot folder https://review.openstack.org/374015 | 11:16 |
TheJulia | Good morning everyone | 11:19 |
aarefiev | milan: yeah, that make sense, I understand you wrong first time, thanks | 11:19 |
aarefiev | morning TheJulia | 11:19 |
milan | morning TheJulia! :) | 11:19 |
*** bdeschenes has joined #openstack-ironic | 11:19 | |
*** marksei has joined #openstack-ironic | 11:21 | |
openstackgerrit | Merged openstack/bifrost: Allow users to change the default provisioning timeout value https://review.openstack.org/376303 | 11:22 |
mgould | morning TheJulia | 11:23 |
openstackgerrit | Mathieu Mitchell proposed openstack/ironic-specs: RAID configuration during deployment https://review.openstack.org/382091 | 11:26 |
mat128 | good morning TheJulia | 11:26 |
mat128 | dtantsur: ^ | 11:26 |
mgould | morning mat128 | 11:26 |
milan | morning mat128! :) | 11:27 |
dtantsur | mat128, \o/ thanks | 11:30 |
dtantsur | morning TheJulia, mat128 | 11:31 |
*** ohamada has quit IRC | 11:37 | |
*** ohamada has joined #openstack-ironic | 11:37 | |
*** bdeschenes has quit IRC | 11:39 | |
*** vsaienko has joined #openstack-ironic | 11:40 | |
*** vsaienko has quit IRC | 11:42 | |
*** mgoddard_ has quit IRC | 11:44 | |
milan | sambetts, dtantsur we've got +2 on https://review.openstack.org/#/c/375045/7 mind a review? ;) | 11:46 |
patchbot | patch 375045 - ironic-inspector-specs - Endpoint to list introspection statuses | 11:46 |
dtantsur | still on my way to it, sorry | 11:46 |
sambetts | milan: reading it now | 11:46 |
milan | cool thx, I'd proceed w/ https://review.openstack.org/#/c/344921/ | 11:47 |
patchbot | patch 344921 - ironic-inspector - Add API for listing all introspection statuses | 11:47 |
vdrok | morning TheJulia and mat128 | 11:49 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal chassis commands https://review.openstack.org/366158 | 11:49 |
mat128 | o/ vdrok | 11:51 |
*** appu103_ has quit IRC | 11:52 | |
openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add negative test-cases for openstack node create command https://review.openstack.org/381943 | 11:53 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Tests for OSC baremetal node create command with options https://review.openstack.org/382352 | 11:54 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 11:55 |
lucasagomes | morning all | 11:56 |
milan | morning lucasagomes! :) | 11:56 |
vdrok | morning lucasagomes | 11:56 |
*** bdeschenes has joined #openstack-ironic | 11:57 | |
mat128 | morning lucasagomes | 11:57 |
mgould | morning lucasagomes | 11:57 |
mat128 | Anyone mind reviewing 2 docs changes that have already had a +2? I'd like to see those off my "Outgoing reviews" list :) https://review.openstack.org/#/q/topic:dox_updates | 11:57 |
openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with Chassis. https://review.openstack.org/343796 | 11:57 |
*** trown|outtypewww is now known as trown | 12:02 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: [install-guide] describe service clients auth https://review.openstack.org/382358 | 12:02 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Skip create_ovs_taps() for multitenancy case https://review.openstack.org/382360 | 12:03 |
*** appu103_ has joined #openstack-ironic | 12:03 | |
*** bdeschenes has quit IRC | 12:04 | |
*** [1]cdearborn has joined #openstack-ironic | 12:04 | |
*** jcoufal has joined #openstack-ironic | 12:09 | |
*** wajdi has quit IRC | 12:09 | |
*** wajdi has joined #openstack-ironic | 12:10 | |
mat128 | lucasagomes: thank you :) | 12:17 |
*** phuongnh has quit IRC | 12:17 | |
lucasagomes | mat128, :-) yw | 12:17 |
*** appu103_ has quit IRC | 12:18 | |
*** sdake has joined #openstack-ironic | 12:19 | |
openstackgerrit | Merged openstack/ironic: Update external links to developer documentation https://review.openstack.org/378990 | 12:19 |
*** vmud213 has quit IRC | 12:20 | |
openstackgerrit | Merged openstack/ironic: Update configuration reference link to latest draft https://review.openstack.org/378991 | 12:21 |
*** ccamacho|lunch is now known as ccamacho | 12:24 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Correctly count the number of primary partitions https://review.openstack.org/381893 | 12:26 |
sambetts | milan: +2 from me | 12:27 |
milan | sambetts, awesome, thanks! :) | 12:27 |
sambetts | milan: I wonder if we should wait for dtantsur's opinion too | 12:27 |
milan | AFAIK dtantsur wanted to review it today ;) | 12:28 |
sambetts | ok :) I'll wait to +A it until he's had a look, but I hope to merge it today | 12:29 |
milan | sambetts, yeah, that'd be nice | 12:29 |
milan | btw | 12:29 |
milan | w/r the list api patch (code), | 12:29 |
*** krtaylor has quit IRC | 12:30 | |
milan | I wonder how to split it | 12:30 |
*** mgoddard has joined #openstack-ironic | 12:30 | |
*** [1]cdearborn has quit IRC | 12:31 | |
*** dnikishov has joined #openstack-ironic | 12:31 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Correctly count the number of primary partitions https://review.openstack.org/381893 | 12:31 |
sambetts | milan: hmmm, the endpoint with everything except the state= and not_state= filters? and then add the those filters as a separate patch on top of the patches to add the states ? | 12:33 |
milan | sambetts, yeah, would be my preference too: pagination(with the default sorting)&time filtering according to the spec | 12:35 |
milan | states filter & custom sorting as separate patches | 12:36 |
milan | to minimise the work needed | 12:36 |
milan | *rework | 12:36 |
mat128 | lucasagomes: found this in vbmc: https://review.openstack.org/#/c/373687/ | 12:36 |
patchbot | patch 373687 - virtualbmc - Updated from global requirements | 12:36 |
lucasagomes | mat128, yay o/ the bot is now working! | 12:37 |
mat128 | :) | 12:37 |
lucasagomes | mat128, why it;s adding a HP copyright line at setup.py | 12:37 |
mat128 | lucasagomes: I think it was supposed to be there. Maybe the script just copies the file from somewhere else? | 12:38 |
mat128 | Let me check if we have the HP copyright in Ironic | 12:38 |
lucasagomes | yeah probably just diff it... it's odd anyway | 12:38 |
mat128 | lucasagomes: agreed | 12:38 |
mat128 | lucasagomes: https://github.com/openstack/ironic/blob/master/setup.py | 12:38 |
mat128 | theres nothing much to have rights on anyway | 12:38 |
*** [1]cdearborn has joined #openstack-ironic | 12:39 | |
lucasagomes | heh true | 12:39 |
lucasagomes | +2;d | 12:39 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Skip create_ovs_taps() for multitenancy case https://review.openstack.org/382360 | 12:41 |
*** baoli has joined #openstack-ironic | 12:44 | |
*** Goneri has joined #openstack-ironic | 12:45 | |
openstackgerrit | Merged openstack/virtualbmc: Updated from global requirements https://review.openstack.org/373687 | 12:48 |
*** ohamada has quit IRC | 12:48 | |
*** ohamada has joined #openstack-ironic | 12:48 | |
*** wajdi has quit IRC | 12:50 | |
*** wajdi has joined #openstack-ironic | 12:50 | |
*** milan is now known as milan|bbl | 12:52 | |
*** vikrant has quit IRC | 12:54 | |
*** wajdi has quit IRC | 12:55 | |
*** makowals has joined #openstack-ironic | 12:56 | |
mariojv | good morning ironic | 12:57 |
sambetts | o/ mariojv | 12:57 |
mgould | morning mariojv | 12:57 |
*** makowals_ has quit IRC | 12:57 | |
*** iurygregory has joined #openstack-ironic | 12:58 | |
*** jheroux has joined #openstack-ironic | 12:59 | |
*** nicodemos has joined #openstack-ironic | 13:00 | |
*** bks has quit IRC | 13:00 | |
*** wajdi has joined #openstack-ironic | 13:02 | |
*** vsaienko has joined #openstack-ironic | 13:02 | |
*** jcoufal_ has joined #openstack-ironic | 13:03 | |
*** jcoufal has quit IRC | 13:03 | |
jroll | morning y'all | 13:04 |
*** baoli_ has joined #openstack-ironic | 13:05 | |
nicodemos | morning jroll | 13:05 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: [install-guide] describe service clients auth https://review.openstack.org/382358 | 13:07 |
lucasagomes | mariojv, sambetts mgould jroll nicodemos morning | 13:07 |
openstackgerrit | Mathieu Mitchell proposed openstack/ironic-specs: RAID configuration during deployment https://review.openstack.org/382091 | 13:08 |
*** baoli has quit IRC | 13:08 | |
mat128 | thanks mgould and wajdi for the quick feedback | 13:09 |
*** hshiina has joined #openstack-ironic | 13:09 | |
nicodemos | hey, lucasagomes | 13:09 |
wajdi | mat128: +1 | 13:10 |
*** ccamacho has quit IRC | 13:16 | |
*** dprince has joined #openstack-ironic | 13:16 | |
*** mjturek has joined #openstack-ironic | 13:17 | |
*** mjturek has left #openstack-ironic | 13:17 | |
*** mjturek has joined #openstack-ironic | 13:18 | |
*** wajdi has quit IRC | 13:19 | |
mgould | mat128: np | 13:21 |
mgould | morning jroll nicodemos | 13:21 |
openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: Add command `driver properties` to osc https://review.openstack.org/381153 | 13:21 |
*** zhenguo has joined #openstack-ironic | 13:32 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification https://review.openstack.org/348437 | 13:33 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: devstack portgroup support https://review.openstack.org/381743 | 13:34 |
openstackgerrit | Michael Turek proposed openstack/python-ironicclient: Refactor provision state so all actions can use inherited take_action https://review.openstack.org/364552 | 13:34 |
*** ccamacho has joined #openstack-ironic | 13:34 | |
*** jaosorior has quit IRC | 13:36 | |
*** jaosorior has joined #openstack-ironic | 13:36 | |
openstackgerrit | Vadim Hmyrov proposed openstack/python-ironicclient: Add option "--split" to command "ironic node-list" https://review.openstack.org/375579 | 13:37 |
openstackgerrit | Hironori Shiina proposed openstack/ironic: Fix undisplayed notes in Quick-Start https://review.openstack.org/382416 | 13:38 |
*** bnemec has joined #openstack-ironic | 13:40 | |
*** marksei has quit IRC | 13:41 | |
*** mgoddard_ has joined #openstack-ironic | 13:45 | |
*** moshele has quit IRC | 13:46 | |
*** mgoddard has quit IRC | 13:48 | |
*** rbudden has joined #openstack-ironic | 13:48 | |
*** mgoddard has joined #openstack-ironic | 13:52 | |
*** mgoddard_ has quit IRC | 13:53 | |
jroll | I'm off sick again today, ping me on hangouts if you desperately need anything | 13:53 |
sambetts | jroll: get better soon dude | 13:54 |
*** alex_xu has quit IRC | 13:55 | |
lucasagomes | jroll, :-/ hope you get better soon! | 13:57 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add ability to write clouds.yaml https://review.openstack.org/380227 | 13:57 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add support to update keystone for bifrost https://review.openstack.org/380226 | 13:57 |
*** vsaienko has quit IRC | 13:57 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add keystone installation playbook https://review.openstack.org/380225 | 13:57 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Update playbooks to support os_client_config https://review.openstack.org/380224 | 13:57 |
*** vsaienko has joined #openstack-ironic | 13:58 | |
xavierr | good morning Ironic | 14:00 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Update Ansible version to stable-2.1 https://review.openstack.org/378041 | 14:00 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add keystone install and clouds.yaml to testing https://review.openstack.org/380228 | 14:00 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add ability to write clouds.yaml https://review.openstack.org/380227 | 14:00 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add support to update keystone for bifrost https://review.openstack.org/380226 | 14:00 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add keystone installation playbook https://review.openstack.org/380225 | 14:00 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Update playbooks to support os_client_config https://review.openstack.org/380224 | 14:00 |
*** alex_xu has joined #openstack-ironic | 14:02 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Convert ironic.conf to template generated https://review.openstack.org/305634 | 14:02 |
*** vsaienko has quit IRC | 14:04 | |
*** rbrud has joined #openstack-ironic | 14:04 | |
*** makowals has quit IRC | 14:04 | |
*** vsaienko has joined #openstack-ironic | 14:04 | |
*** joprovost has joined #openstack-ironic | 14:05 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: devstack portgroup support https://review.openstack.org/381743 | 14:06 |
mgould | jroll: get well soon! | 14:06 |
*** makowals has joined #openstack-ironic | 14:06 | |
*** vsaienko has quit IRC | 14:07 | |
dtantsur | morning jroll, get better! | 14:07 |
openstackgerrit | Merged openstack/ironic-inspector: LLC Hook: Fix patching Ironic ports https://review.openstack.org/380400 | 14:07 |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 14:07 |
*** wajdi has joined #openstack-ironic | 14:08 | |
*** absubram has quit IRC | 14:08 | |
*** dprince has quit IRC | 14:10 | |
*** dims has quit IRC | 14:11 | |
*** links has quit IRC | 14:14 | |
*** jaosorior has quit IRC | 14:16 | |
*** vsaienko has joined #openstack-ironic | 14:16 | |
*** dims has joined #openstack-ironic | 14:17 | |
*** vsaienko has quit IRC | 14:17 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Move build_instance_info_for_deploy to deploy_utils https://review.openstack.org/377608 | 14:21 |
dtantsur | folks, what are the requirements for running our tempest plugin? nodes enrolled, images, networks? do we have it documented? | 14:21 |
TheJulia | afaik we do not | 14:21 |
*** vsaienko has joined #openstack-ironic | 14:23 | |
lucasagomes | dtantsur, I think I've added some docs to run with devstack | 14:23 |
* lucasagomes finds | 14:23 | |
lucasagomes | dtantsur, http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#running-tempest-tests | 14:24 |
lucasagomes | not much info there, but may be a starting point | 14:24 |
*** rpioso has joined #openstack-ironic | 14:24 | |
dtantsur | well, devstack does a lot of preparation itself, so unfortunately it doesn't help much | 14:24 |
lucasagomes | yeah, I guess the best source is the devstack script itself | 14:25 |
*** hj-hpe has joined #openstack-ironic | 14:28 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification https://review.openstack.org/348437 | 14:29 |
openstackgerrit | Merged openstack/ironic-inspector: LLC Hook: Do not assume interfaces are added to Ironic https://review.openstack.org/380401 | 14:33 |
*** baoli_ has quit IRC | 14:35 | |
*** moshele has joined #openstack-ironic | 14:35 | |
*** vsaienko has quit IRC | 14:36 | |
*** moshele has quit IRC | 14:38 | |
*** hj-hpe has quit IRC | 14:38 | |
*** hogepodge has quit IRC | 14:38 | |
*** tesseract- has quit IRC | 14:38 | |
*** fxpester has quit IRC | 14:38 | |
*** milan|bbl has quit IRC | 14:38 | |
*** mjura has quit IRC | 14:38 | |
*** BadCub has quit IRC | 14:38 | |
*** Ng has quit IRC | 14:38 | |
*** jjohnson2 has quit IRC | 14:38 | |
*** pmannidi has quit IRC | 14:38 | |
*** clenimar has quit IRC | 14:38 | |
*** davidlenwell has quit IRC | 14:38 | |
*** ipukha has quit IRC | 14:38 | |
*** marlinc has quit IRC | 14:38 | |
*** pmannidi has joined #openstack-ironic | 14:39 | |
*** hj-hpe has joined #openstack-ironic | 14:39 | |
*** mjura has joined #openstack-ironic | 14:39 | |
*** fxpester has joined #openstack-ironic | 14:39 | |
*** milan|bbl has joined #openstack-ironic | 14:39 | |
*** clenimar has joined #openstack-ironic | 14:39 | |
*** jjohnson2 has joined #openstack-ironic | 14:39 | |
*** hogepodge has joined #openstack-ironic | 14:39 | |
*** bnemec has quit IRC | 14:39 | |
*** tesseract- has joined #openstack-ironic | 14:39 | |
*** davidlenwell has joined #openstack-ironic | 14:40 | |
*** mkam has joined #openstack-ironic | 14:40 | |
*** ipukha has joined #openstack-ironic | 14:40 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add Keystone support to ironic.conf https://review.openstack.org/382469 | 14:42 |
*** mkam has quit IRC | 14:43 | |
*** bnemec has joined #openstack-ironic | 14:44 | |
*** penick has quit IRC | 14:44 | |
*** rpioso has quit IRC | 14:44 | |
*** mkam has joined #openstack-ironic | 14:45 | |
*** Ng has joined #openstack-ironic | 14:46 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: Add command `driver properties` to osc https://review.openstack.org/381153 | 14:46 |
*** marlinc has joined #openstack-ironic | 14:48 | |
*** BadCub has joined #openstack-ironic | 14:49 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: tempest tests for bonding https://review.openstack.org/382476 | 14:50 |
vsaienk0 | jroll, dtantsur, sambetts: could you please let me know your opinion about: https://review.openstack.org/#/c/382438/1/nova/virt/ironic/driver.py@685 | 14:52 |
patchbot | patch 382438 - nova - WIP: Add portgroups support to ironic virt driver | 14:52 |
*** narasimha_SV has joined #openstack-ironic | 14:52 | |
narasimha_SV | http://paste.openstack.org/show/584478/ | 14:52 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: [install-guide] describe pxe.ipxe_swift_tempurl https://review.openstack.org/382478 | 14:52 |
narasimha_SV | in ironic-api i am getting these logs | 14:53 |
narasimha_SV | it is unable to get the token and not able to validate those tokens | 14:53 |
*** mkam has quit IRC | 14:53 | |
*** wajdi_ has joined #openstack-ironic | 14:53 | |
narasimha_SV | http://paste.openstack.org/show/584479/ there are the configurations in my ironic.conf | 14:54 |
*** wajdi has quit IRC | 14:56 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Clean exceptions handling in conductor manager https://review.openstack.org/381554 | 14:57 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: API Evolution - Error Responses https://review.openstack.org/381989 | 14:57 |
*** wajdi_ has quit IRC | 15:01 | |
*** jistr is now known as jistr|call | 15:01 | |
openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: API Evolution - Implement Tasks Resource https://review.openstack.org/382017 | 15:01 |
*** wajdi has joined #openstack-ironic | 15:01 | |
*** mjura has quit IRC | 15:05 | |
*** mkam has joined #openstack-ironic | 15:06 | |
*** rcernin has quit IRC | 15:12 | |
*** pcaruana has quit IRC | 15:13 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for OSC baremetal chassis create command https://review.openstack.org/382496 | 15:13 |
*** jistr|call is now known as jistr | 15:13 | |
*** hshiina has quit IRC | 15:15 | |
*** ijw has joined #openstack-ironic | 15:16 | |
openstackgerrit | Mathieu Mitchell proposed openstack/ironic-specs: RAID configuration during deployment https://review.openstack.org/382091 | 15:18 |
*** tesseract- has quit IRC | 15:24 | |
openstackgerrit | Merged openstack/ironic: Use six.StringIO instead of six.moves.StringIO https://review.openstack.org/375297 | 15:26 |
jlvillal | vsaienk0: Would you like me to rebase the multi-node patches? I saw you changed the localrc patch. So I think all the other patches need to be updated. | 15:28 |
vdrok | morning jlvillal | 15:30 |
vdrok | devananda: around? | 15:30 |
jlvillal | Hi vdrok :) | 15:30 |
jlvillal | vdrok: I like the Mirantis posse! Six of them go and +1 the devstack-gate patch in less than an hour :) https://review.openstack.org/#/c/364830/ | 15:31 |
patchbot | patch 364830 - openstack-infra/devstack-gate - Setup ssh-key on subnodes for Ironic | 15:31 |
*** vishwanathj has joined #openstack-ironic | 15:31 | |
vdrok | jlvillal: heh, yep, happens sometimes :) | 15:32 |
jlvillal | vdrok: I hope it gets merged! I worked on that patch too :) | 15:32 |
vdrok | jlvillal: bring in intel folks then :) | 15:32 |
jlvillal | heh | 15:33 |
*** rbartal has quit IRC | 15:34 | |
*** lucasagomes is now known as lucas-hungry | 15:35 | |
*** jaosorior has joined #openstack-ironic | 15:37 | |
sambetts | vsaienk0: I don't mind it, however you are missing some of the logic around vif to pif mapping | 15:38 |
*** baoli has joined #openstack-ironic | 15:39 | |
*** dprince has joined #openstack-ironic | 15:39 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for maintenance mode commands https://review.openstack.org/337259 | 15:39 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add negative tests for baremetal node commands https://review.openstack.org/337719 | 15:40 |
sambetts | vsaienk0: e.g. there are some cases where hardware has 3 nics 2 in a portgroup and 1 standalone, current logic states that the vifs should be loadbalanced across both the port and the portgroup | 15:40 |
*** rama_y has joined #openstack-ironic | 15:40 | |
sambetts | s/loadbalanced/round robined | 15:40 |
openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: Add command `driver properties` to osc https://review.openstack.org/381153 | 15:42 |
*** zhenguo has quit IRC | 15:43 | |
mariojv | JayF: was this the BFV spec you were talking about? https://review.openstack.org/#/c/294995/ | 15:45 |
patchbot | patch 294995 - ironic-specs - Boot from Volume - Reference Drivers | 15:45 |
JayF | yes | 15:45 |
mat128 | JayF: you might be interested by this spec: https://review.openstack.org/382091 | 15:45 |
patchbot | patch 382091 - ironic-specs - RAID configuration during deployment | 15:45 |
mat128 | it's what we discussed during the midcycle, re: deploy_steps | 15:45 |
milan|bbl | sambetts, aarefiev w/r dtantsur 's comment, I think the guideline is sort of inconsistent with the list spec :-/ there's especially this: https://github.com/openstack/api-wg/blob/master/guidelines/pagination_filter_sort.rst#filtering so coma-separated items and operations like 'in' are suggested in there. Not sure how to address this, wdyt? | 15:47 |
TheJulia | dtantsur: I take it with roles that inspector would need to be a baremetal admin? | 15:47 |
dtantsur | TheJulia, iirc yes | 15:48 |
TheJulia | excellent | 15:48 |
TheJulia | thank you | 15:48 |
* TheJulia raises eyebrow at BFV chatter | 15:48 | |
milan|bbl | next time we'll get a turing-complete language over queries ;) ?size=gt:8 | 15:49 |
mariojv | TheJulia: was just asking for things that need reviews, jay recommended a couple specs | 15:49 |
*** milan|bbl is now known as milan | 15:49 | |
TheJulia | mariojv: Ahh, excellent! | 15:50 |
* milan feels like no adopter unless there's oslo support for parsing those funky bits | 15:54 | |
openstackgerrit | Annie Lezil proposed openstack/ironic: Adding Timing metrics for DRAC drivers. https://review.openstack.org/377957 | 15:55 |
sambetts | milan: oh dear... | 15:56 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Tests for OSC baremetal node create command with options https://review.openstack.org/382352 | 15:56 |
milan | sambetts, I'd postpone compatibility to APIv2 ;) | 15:57 |
JayF | mat128: do you think we need a more generic spec for deploy steps before your spec? | 15:57 |
dtantsur | milan, I'd prefer ignoring operators for now | 15:57 |
*** sukhdev has joined #openstack-ironic | 15:57 | |
JayF | mat128: like I'm left with a lot of questions: can I define additional deploy steps in a hardwaremanager, like I can with clean steps? etc | 15:58 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for OSC baremetal chassis create command https://review.openstack.org/382496 | 15:58 |
milan | dtantsur, but we do have time interval and state sets in the spec | 15:58 |
mat128 | JayF: I was hoping not, but can see it happening if needed | 15:58 |
mat128 | JayF: yes you will be able , I think I mentioned it | 15:58 |
mat128 | let me find it | 15:58 |
milan | all of those are specified as multi-queries atm | 15:58 |
dtantsur | ugh... | 15:58 |
sambetts | milan, dtantsur: are you suggesting we do things like finished_at=gt:<time> ?? | 15:58 |
mat128 | JayF: L219 | 15:58 |
milan | sambetts, not me | 15:59 |
dtantsur | sambetts, I'm not suggesting, but the API guidelines do | 15:59 |
sambetts | :/ | 15:59 |
* milan facepalms | 15:59 | |
dtantsur | assuming that we care about what API-WG produces, of course | 15:59 |
*** Sukhdev_ has joined #openstack-ironic | 15:59 | |
dtantsur | if we don't, we just copy what Ironic is doing | 15:59 |
dtantsur | we have to be as compatible with either of these two as possible | 15:59 |
milan | Ironic, what are you doing? :D | 15:59 |
* milan digs in code | 16:00 | |
*** jpich has quit IRC | 16:00 | |
sambetts | yeah... I mean I'd prefer if we stuck to API-WG stuff for OpenStack unity, and I don't actually mind the form "finished_at=gt:<time>" instead of the intervals if it covers all the use cases | 16:01 |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session glance and swift https://review.openstack.org/352289 | 16:01 |
* milan desperately needs a beer | 16:01 | |
* sambetts adds a note to buy milan a beer in spain | 16:01 | |
* jlvillal pulls the trigger!!! | 16:02 | |
milan | sambetts, you're my pal | 16:02 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Update iptables rules and services IPs for multinode https://review.openstack.org/368648 | 16:02 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Fix setting custom IRONIC_VM_NETWORK_BRIDGE https://review.openstack.org/365116 | 16:02 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Add devstack setup_vxlan_network() https://review.openstack.org/368647 | 16:02 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Skip some steps for multinode case https://review.openstack.org/368646 | 16:02 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Update ironic node names for multinode case https://review.openstack.org/368645 | 16:02 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Update devstack provision net config for multihost https://review.openstack.org/368644 | 16:02 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Source openrc on subnode in multinode topology https://review.openstack.org/353302 | 16:02 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Skip db configuration on subnodes https://review.openstack.org/353303 | 16:02 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 16:02 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Ignore required_services for multinode topology https://review.openstack.org/352793 | 16:02 |
TheJulia | wow | 16:02 |
sambetts | its a tsunami run!!! | 16:02 |
* jlvillal has hopefully completed a successful rebase to update a var name change :) | 16:03 | |
milan | :D | 16:03 |
sambetts | jlvillal: thats one long chain of patches :-P | 16:03 |
jlvillal | Yeah. The multi-node stuff | 16:03 |
*** rpioso has joined #openstack-ironic | 16:03 | |
jlvillal | We are still trying to work on getting the devstack-gate changes in. Along with one devstack change. | 16:03 |
*** ccamacho is now known as ccamacho|willbeb | 16:05 | |
*** ccamacho|willbeb is now known as ccamacho|willbbc | 16:05 | |
openstackgerrit | Merged openstack/bifrost: Fix a typo in documentation https://review.openstack.org/381383 | 16:05 |
*** rbudden has quit IRC | 16:05 | |
*** ccamacho|willbbc is now known as ccamacho|wllbbck | 16:05 | |
*** Sukhdev_ has quit IRC | 16:06 | |
*** baoli has quit IRC | 16:06 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for 'baremetal port create' command https://review.openstack.org/332880 | 16:06 |
jlvillal | vsaienk0: I rebased the multi-node patches for the var name change in devstack-gate | 16:08 |
milan | sambetts, aarefiev, dtantsur according to https://wiki.openstack.org/wiki/API_Working_Group/Current_Design/Sorting not every one uses this WG suggestion. I need to be running now, let's talk about it tomorrow again, sry | 16:11 |
* milan gone | 16:11 | |
milan | good night Ironic | 16:11 |
dtantsur | g'night! | 16:11 |
TheJulia | goodnight milan | 16:12 |
dtantsur | ugh, I was smart enough to run full disk cleaning on a 50 Gi VM.. I wonder how much it can take, it's been 40 minutes already | 16:12 |
JayF | it can take a /real long time/ in a vm | 16:13 |
JayF | lol | 16:13 |
* dtantsur applies his head to the table | 16:13 | |
JayF | I know the ssh driver is dying | 16:13 |
JayF | but if it wasn't, it'd be awesome to implement "out of band" erase devices on it | 16:13 |
JayF | like just deleting the disk image on disk or something, lol | 16:13 |
dtantsur | lol | 16:14 |
dtantsur | file an RFE against virtualbmc | 16:14 |
dtantsur | (although it will require a specialized driver) | 16:14 |
JayF | yeah, which defeats the point of virtualbmc | 16:14 |
JayF | mainly just a throwaway comment :P | 16:14 |
dtantsur | but yeah, this is good :) why wouldn't qemu implement ATA secure erase btw? | 16:15 |
*** milan has quit IRC | 16:15 | |
*** vsaienko has joined #openstack-ironic | 16:15 | |
JayF | I have no idea, and don't know enough C to make it happen :P | 16:16 |
dtantsur | I regret I was too lazy to rerun puppet on all conductors to disable the full erase... it would be some much faster :D | 16:16 |
*** vsaienko has quit IRC | 16:16 | |
*** rajinir has joined #openstack-ironic | 16:18 | |
dtantsur | I guess I'll just leave it finish overnight. see you all tomorrow | 16:18 |
sambetts | cya dtantsur | 16:19 |
*** dtantsur is now known as dtantsur|afk | 16:19 | |
JayF | o/ | 16:19 |
mat128 | dtantsur|afk, JayF: same kind of issue: SMART doesnt work on virtio | 16:20 |
mat128 | or bonding doesnt work on virtio-net because it has no link speed and duplex | 16:20 |
mat128 | fake problems :) | 16:21 |
*** syed_ has joined #openstack-ironic | 16:22 | |
*** rbudden has joined #openstack-ironic | 16:23 | |
*** Romanenko_K has quit IRC | 16:25 | |
NobodyCam | Good Morning Ironic'ers | 16:27 |
jlvillal | Hi NobodyCam | 16:27 |
JayF | o/ | 16:28 |
NobodyCam | morning jlvillal JayF :) | 16:28 |
*** dougs1 has joined #openstack-ironic | 16:29 | |
dougs1 | Anyone know offhand if Ironic supports Linux Bridges (instead of Open vSwitch) in the current release? | 16:30 |
JayF | How would linux bridges do anything for you? There's no hypervisor to bridge to in Ironic | 16:31 |
sambetts | dougs1: in devstack no, but in production you can use what ever you want | 16:31 |
jlvillal | dougs1: For devstack I think we only support OVS. But in a real bare-metal environment, what JayF said. | 16:31 |
dougs1 | ok, thanks guys | 16:32 |
JayF | And if you're using Ironic outside of devstack with VMs instead of bare metal, please don't :P | 16:32 |
sambetts | jlvillal, JayF: it can affect network node configuration | 16:32 |
sambetts | neutron config | 16:32 |
JayF | what do you mean? | 16:32 |
jlvillal | sambetts: What's that mean? | 16:32 |
jlvillal | JayF: Jinx! | 16:32 |
mat128 | JayF, dtantsur|afk: just checked for fun, qemu has an "ide_cmd_table" in hw/ide/core.c that wouldn't be too hard to extend | 16:32 |
JayF | mat128: I can write just slightly more than hello world in qemu; but you could write the patch :P | 16:33 |
sambetts | JayF, jlvillal: I mean choosing LB or OVS changes how your neutron install is plumbed in, so there are some differences in configuring a LB environment for Ironic against an OVS one | 16:33 |
JayF | mat128: just remember that standard secure erase = drive is zeros, enhanced secure erase = drive is random bits | 16:34 |
mat128 | I'm afraid my C is not cool enough for my feature to land in qemu, but who knows | 16:34 |
JayF | sambetts: but the LB setup would be completely incompatible with ironic ... right? | 16:34 |
*** watanabe_isao has joined #openstack-ironic | 16:34 | |
JayF | or do I not understand? | 16:34 |
JayF | mat128: It's easier to get patches into things than you think. I have code in systemd-nspawn, and I don't even know C :P | 16:34 |
mat128 | JayF: unless you run the linuxbridge plugin on your TOR :) | 16:34 |
JayF | *blink* | 16:35 |
sambetts | JayF: on the network/controller node, neutron hooks its services, e.g. DHCP into LB or OVS | 16:35 |
*** lucas-hungry is now known as lucasagomes | 16:35 | |
sambetts | JayF: thats how it can run isolated network services like DHCP or routers etc on one machine | 16:36 |
dougs1 | sambetts, agreed. I had heard rumours of better performance with Linux Bridges vs Open vSwitch | 16:36 |
*** watanabeisao has joined #openstack-ironic | 16:38 | |
sambetts | dougs1: I have also heard that, I'm not sure if its true | 16:39 |
*** watanabe_isao has quit IRC | 16:39 | |
*** rbrud has quit IRC | 16:41 | |
*** jaosorior has quit IRC | 16:42 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Correctly count the number of primary partitions https://review.openstack.org/381893 | 16:45 |
*** trown is now known as trown|lunch | 16:47 | |
jlvillal | lucasagomes: Darn! I was too late by 4 minutes https://review.openstack.org/#/c/381893/3/ironic_lib/disk_utils.py :) | 16:50 |
patchbot | patch 381893 - ironic-lib - Correctly count the number of primary partitions | 16:50 |
lucasagomes | jlvillal, haha lemme see | 16:50 |
lucasagomes | jlvillal, o the typo!? :-/ | 16:50 |
jlvillal | lucasagomes: Yeah. Not a big deal. Not your typo. | 16:51 |
* jlvillal does a git blame to make sure ;) | 16:51 | |
lucasagomes | jlvillal, yeah, I can fix it. 1 sec | 16:51 |
* lucasagomes have the code open here | 16:51 | |
jlvillal | lucasagomes: Thanks! | 16:51 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Correctly count the number of primary partitions https://review.openstack.org/381893 | 16:53 |
lucasagomes | jlvillal, ^ | 16:53 |
jlvillal | lucasagomes: Thanks! | 16:53 |
lucasagomes | jlvillal, ty for reviewing | 16:53 |
jlvillal | lucasagomes: -1 | 16:54 |
* jlvillal is known for his dark cold heart... | 16:54 | |
*** yonglihe has quit IRC | 16:55 | |
*** lucasagomes is now known as lucas-brb | 16:55 | |
*** derekh has quit IRC | 17:01 | |
* jlvillal was late getting QA meeting started :( | 17:03 | |
*** e0ne has quit IRC | 17:06 | |
*** vsaienko has joined #openstack-ironic | 17:07 | |
rajinir | o/ | 17:10 |
*** strigazi is now known as strigazi_AFK | 17:10 | |
rajinir | sorry I was late | 17:11 |
*** lucas-brb is now known as lucasagomes | 17:11 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 17:11 |
* sambetts -> home | 17:13 | |
sambetts | night team! o/ | 17:13 |
jlvillal | sambetts: ciao | 17:13 |
jlvillal | sambetts: Or toodle-loo :) | 17:14 |
*** sambetts is now known as sambetts|afk | 17:14 | |
sambetts|afk | jlvillal: haha | 17:14 |
sambetts|afk | jlvillal: cya o/ | 17:14 |
jlvillal | :) | 17:14 |
openstackgerrit | Michael Turek proposed openstack/python-ironicclient: Add --wait to OSC provisioning commands https://review.openstack.org/354806 | 17:15 |
*** baoli has joined #openstack-ironic | 17:18 | |
*** rbrud has joined #openstack-ironic | 17:19 | |
watanabeisao | sambetts|afk, oh afk.. | 17:19 |
watanabeisao | jlvillal, sorry sir. I just want to confirm. | 17:20 |
*** amotoki has quit IRC | 17:21 | |
jlvillal | watanabeisao: Confirm what? | 17:21 |
watanabeisao | so currently, we don't know who is going to check Ironic's 3rd party CIs' state at Newton release. Like returning result within 8 hours. As same as when and how, am I right? | 17:21 |
jlvillal | watanabeisao: Yeah. I think you need to talk to krtaylor. He is the one that knows all that. I do not. | 17:22 |
jlvillal | watanabeisao: I don't know the answer to your question. | 17:22 |
watanabeisao | jlvillal, I see, thank you. | 17:22 |
jlvillal | watanabeisao: Sorry :( | 17:22 |
watanabeisao | jlvillal, it's ok.:) | 17:23 |
*** MattMan has quit IRC | 17:23 | |
*** MattMan has joined #openstack-ironic | 17:23 | |
*** mgoddard has quit IRC | 17:23 | |
*** vsaienko has quit IRC | 17:23 | |
*** ccamacho|wllbbck is now known as ccamacho | 17:23 | |
*** mgould is now known as mgould|afk | 17:25 | |
*** watanabeisao has quit IRC | 17:27 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Correctly count the number of primary partitions https://review.openstack.org/381893 | 17:31 |
lucasagomes | jlvillal, ^ thanks for the review. try...except added + unittest | 17:31 |
JayF | looking as well | 17:32 |
jlvillal | lucasagomes: Thanks. I wasn't sure but thought I would point it out. | 17:32 |
lucasagomes | it was a good point! Thanks guys for looking at it | 17:32 |
JayF | lucasagomes: curious, why with use_standard_locale=true | 17:33 |
JayF | lucasagomes: we don't seem to be doing that on many other utils.execute() calls | 17:33 |
lucasagomes | JayF, most because we are checking the output string, we check if "msdos" is present. I think we don't need standard locale for that, but just in case | 17:34 |
JayF | oh, I get you | 17:34 |
JayF | you wanna make sure it's not translated | 17:34 |
JayF | even though that's unliked | 17:34 |
lucasagomes | not sure if some languages like japanese, korean etc might translate that | 17:34 |
lucasagomes | yeah | 17:34 |
JayF | *unlikely | 17:34 |
JayF | makes sense | 17:34 |
*** ohamada has quit IRC | 17:34 | |
jlvillal | lucasagomes: Still reviewing but I posted a question. | 17:35 |
jlvillal | JayF: I remember someone posting a patch adding the 'use_standard_locale' thing for some execute call because it failed in Japanese. | 17:35 |
lucasagomes | jlvillal, I think it's a good to have as much information as we can in the messages | 17:36 |
lucasagomes | the string was there already, I can update it tho... | 17:36 |
jlvillal | JayF: lucasagomes https://review.openstack.org/#/c/231367/ | 17:36 |
patchbot | patch 231367 - ironic - Use standard locale when executing 'parted' command (MERGED) | 17:36 |
jlvillal | lucasagomes: Up to you. Thanks. | 17:38 |
jlvillal | lucasagomes: I was about to +2 it. Okay with you? | 17:41 |
lucasagomes | jlvillal, I'm fixing it 1 sec | 17:41 |
lucasagomes | better now that I've all the files opened | 17:41 |
jlvillal | lucasagomes: Okay thanks. | 17:41 |
JayF | that works, then john and I can land it | 17:41 |
JayF | unless one of you think we should leave it up for more review | 17:41 |
JayF | seems fairly straightforward to me tbh | 17:41 |
*** bjolo_ has joined #openstack-ironic | 17:41 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Correctly count the number of primary partitions https://review.openstack.org/381893 | 17:41 |
jlvillal | agreed. | 17:41 |
lucasagomes | ^^ | 17:41 |
JayF | mat128: put comments on your raid config spec, if you can completely hash out the idea of deploy steps in there I'm OK with it all being in that spec, but there's a lot of detail missing that I think we need | 17:42 |
jlvillal | lucasagomes: Thanks. I hope it passes the tests :) | 17:43 |
jlvillal | lucasagomes: +2 | 17:43 |
JayF | +A, I'll let jenkins do the job of deciding if it passes tests :D | 17:44 |
lucasagomes | jlvillal, ty! Fingers crossed (jk, I ran it locally) | 17:44 |
lucasagomes | JayF, thanks | 17:44 |
*** athomas has quit IRC | 17:45 | |
*** karthiks has quit IRC | 17:46 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 17:50 |
*** moshele has joined #openstack-ironic | 17:54 | |
*** vsaienko has joined #openstack-ironic | 17:56 | |
*** vsaienko has quit IRC | 17:56 | |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 18:01 |
*** moshele has quit IRC | 18:04 | |
*** mtanino has quit IRC | 18:04 | |
*** trown|lunch is now known as trown | 18:04 | |
*** yolanda has quit IRC | 18:08 | |
*** vsaienko has joined #openstack-ironic | 18:12 | |
jlvillal | lucasagomes: The commit message still says "count_MSDOS_partitions" ;) | 18:13 |
jlvillal | lucasagomes: Not critical | 18:13 |
*** yolanda has joined #openstack-ironic | 18:15 | |
*** vsaienko has quit IRC | 18:16 | |
openstackgerrit | Merged openstack/python-ironicclient: Add prefix "$" for command examples https://review.openstack.org/379371 | 18:19 |
lucasagomes | jlvillal, sigh... yeah :-. | 18:19 |
lucasagomes | :-/* | 18:19 |
jlvillal | lucasagomes: No worries :) | 18:20 |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Update reno for stable/newton https://review.openstack.org/374772 | 18:22 |
*** moshele has joined #openstack-ironic | 18:22 | |
*** e0ne has joined #openstack-ironic | 18:23 | |
*** vsaienko has joined #openstack-ironic | 18:27 | |
openstackgerrit | Merged openstack/ironic-lib: Correctly count the number of primary partitions https://review.openstack.org/381893 | 18:32 |
*** sukhdev has quit IRC | 18:34 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Remove Python 2.6 format style https://review.openstack.org/382616 | 18:37 |
*** rpioso has quit IRC | 18:38 | |
*** mkam has quit IRC | 18:45 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Remove Python 2.6 format style https://review.openstack.org/382616 | 18:47 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 18:49 |
*** e0ne has quit IRC | 18:50 | |
*** Sukhdev has joined #openstack-ironic | 18:51 | |
*** vsaienko has quit IRC | 18:52 | |
Sukhdev | jroll: when you got a sec, can you point me to which document should I updated to add the description for Security Groups? | 18:55 |
*** e0ne has joined #openstack-ironic | 18:58 | |
*** vsaienko has joined #openstack-ironic | 19:01 | |
* lucasagomes is getting a "Timeout error, please try again in a few minutes." when trying to post comments on bugs in launchpad | 19:01 | |
*** ijw has quit IRC | 19:03 | |
*** rpioso has joined #openstack-ironic | 19:04 | |
lucasagomes | Sukhdev, jroll is not feeling well today so he took the day off. I'm not 100% sure, but looks like https://github.com/openstack/ironic/blob/master/install-guide/source/include/configure-neutron-networks.rst ? | 19:04 |
lucasagomes | We could add it there and if it's not the right doc we can later move it to the right one in another patch-set | 19:05 |
Sukhdev | lucasagomes : Sounds good - thanks for the pointer. I will work on it and push the patch | 19:06 |
*** vsaienko has quit IRC | 19:07 | |
lucasagomes | yvw | 19:08 |
*** vsaienko has joined #openstack-ironic | 19:10 | |
*** vsaienko has quit IRC | 19:14 | |
*** moshele has quit IRC | 19:21 | |
Sukhdev | lucasagomes : I was browsing through and saw this - http://docs.openstack.org/developer/ironic/deploy/multitenancy.html#multitenancy | 19:26 |
Sukhdev | lucasagomes : Do you think this is better place or the one you one pointed out - either one will be OK - thought I just double check with you | 19:26 |
lucasagomes | Sukhdev, oh the doc you will is about security groups for the provisioning/tenat networks ? If so, the multitenancy doc you found sounds more correct to me | 19:29 |
Sukhdev | lucasagomes : right - I thought the same, hence, I thought I double check | 19:30 |
lucasagomes | ++ | 19:31 |
Sukhdev | lucasagomes : and. moreover, as you mentioned, if we need to move this around later, we can alway push another patch | 19:31 |
lucasagomes | yeah, as long as we have the content, moving it from one doc to another is trivial. People can comment where it should live as part of the review (and expect comments cause doc patches usually get many in Ironic :D) | 19:32 |
*** bjolo_ has quit IRC | 19:33 | |
*** vsaienko has joined #openstack-ironic | 19:34 | |
openstackgerrit | Merged openstack/ironic: Fix docstrings in tempest plugin baremetal json client https://review.openstack.org/377774 | 19:35 |
*** persia has quit IRC | 19:37 | |
*** kirankv has quit IRC | 19:37 | |
*** ishant|away has quit IRC | 19:37 | |
*** lazy_prince has quit IRC | 19:37 | |
openstackgerrit | Merged openstack/ironic: Keep numbering of list in Install Guide https://review.openstack.org/382169 | 19:37 |
*** persia has joined #openstack-ironic | 19:37 | |
*** lazy_prince has joined #openstack-ironic | 19:39 | |
*** ishant|away has joined #openstack-ironic | 19:39 | |
*** kirankv has joined #openstack-ironic | 19:39 | |
*** e0ne has quit IRC | 19:39 | |
*** vsaienko has quit IRC | 19:44 | |
*** sdake has quit IRC | 19:45 | |
*** sdake has joined #openstack-ironic | 19:46 | |
*** vsaienko has joined #openstack-ironic | 19:56 | |
lucasagomes | folks, calling it a day | 20:01 |
lucasagomes | have a great evening all! | 20:01 |
JayF | o/ | 20:02 |
mariojv | have a good evening lucasagomes | 20:02 |
lucasagomes | \o, see ya | 20:02 |
*** lucasagomes is now known as lucas-afk | 20:02 | |
*** ijw has joined #openstack-ironic | 20:03 | |
*** jpich has joined #openstack-ironic | 20:08 | |
*** ijw has quit IRC | 20:10 | |
*** vsaienko has quit IRC | 20:11 | |
*** joprovost has quit IRC | 20:16 | |
*** narasimha_SV has quit IRC | 20:18 | |
*** vsaienko has joined #openstack-ironic | 20:23 | |
*** vsaienko has quit IRC | 20:28 | |
openstackgerrit | Merged openstack/ironic: Cleanup get_ilo_license() https://review.openstack.org/380341 | 20:34 |
openstackgerrit | Merged openstack/ironic: Documentation change for feature updates in iLO drivers. https://review.openstack.org/374060 | 20:34 |
*** penick has joined #openstack-ironic | 20:35 | |
*** devananda is now known as devananda|afk | 20:43 | |
*** vsaienko has joined #openstack-ironic | 20:47 | |
*** trown is now known as trown|outtypewww | 20:48 | |
*** xavierr is now known as xavierr|afk | 20:55 | |
mrda | Morning Ironic | 20:55 |
jlvillal | FYI: Our gate maybe broken on jobs which run on OSIC with devstack and neutron. Which sounds like us. | 20:56 |
jlvillal | Infra seems to be working on it. Basically with Neutron on OSCI, IPv4 connectivity to the outside world gets broken. Is how I understand it. | 20:57 |
jlvillal | s/OSCI/OSIC/ | 20:57 |
JayF | jlvillal: AIUI it's only IPA | 20:57 |
JayF | jlvillal: not all of ironic | 20:57 |
JayF | jroll: ^ right/ | 20:57 |
jlvillal | JayF: That's better! :) | 20:57 |
jroll | yeah, just IPA afaik | 20:58 |
jlvillal | JayF: Shhh. Don't wake him... | 20:58 |
JayF | I noted it on the whiteboard too | 20:58 |
JayF | jroll: is there any action for us to take fixing that? Anything I can do? | 20:58 |
JayF | jroll: the bug seems to be being worked on by $others | 20:58 |
jroll | JayF: uhhhh. lemme see, I put up a patch to test a thing | 20:58 |
jroll | it's really on neutron's plate | 20:58 |
JayF | it failed | 20:59 |
JayF | your patch to test a thing | 20:59 |
jlvillal | Oh darn. That same bug... | 20:59 |
jroll | I guess I clobbered it https://review.openstack.org/#/c/380351/2 | 20:59 |
patchbot | patch 380351 - ironic-python-agent - DNM: testing 380023 | 20:59 |
jroll | just rechecked | 20:59 |
*** Goneri has quit IRC | 21:00 | |
mariojv | heading out, have a good evening ironic | 21:00 |
jlvillal | mariojv: Good night | 21:01 |
*** ijw has joined #openstack-ironic | 21:07 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 21:08 |
jlvillal | We made the YouTube channel :) https://youtu.be/z6ftW7fUdp4?t=2m28s | 21:09 |
*** dprince has quit IRC | 21:10 | |
jlvillal | The second half of the Newton Release Demo video is about Ironic :) | 21:10 |
*** ijw has quit IRC | 21:12 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 21:22 |
*** Vinsh has joined #openstack-ironic | 21:26 | |
Vinsh | Hello ironic team. I have been working on setup and deployment of ironic in our lab here at charter(formerly twc). I'm running mitaka/stable for ironic-api/conductor. The IPA image is the latest dib of fedora. Using the agent_ipmi driver. Yesterday I was able to set the node to "provide" state to kickoff the cleaning steps. The node called back to ironic-api and begain the hearbeat. I loged into the node as a dev | 21:26 |
Vinsh | user and watched it wipe the disk. Now, I can not repeat this. I have tried every possible setting and combination of settings I could think of and it still fails. The node will pxe boot, but immediatly decides "New API is not available" so "falling back to old agent vendor passthru". That fails with a 401 code on the agent and in api logs I see "POST 6385/v1/drivers/agent_ipmitool/vendor_passthru/lookup HTTP/1.1" | 21:26 |
Vinsh | status: 401. I tried upgrading to newton/stable and it appears the "new api" isn't there either.. where does this new api exist? isn't the agent_ipmi driver the driver of choice? Anyway, what might be causing the persistent 401 for the node trying to do a lookup? | 21:26 |
*** aNuposic has joined #openstack-ironic | 21:28 | |
*** aNuposic has quit IRC | 21:31 | |
*** vsaienko has quit IRC | 21:32 | |
JayF | Vinsh: that's a warning, not an error | 21:33 |
*** jpich has quit IRC | 21:33 | |
*** vsaienko has joined #openstack-ironic | 21:35 | |
Vinsh | JayF: Well :) IPA just fails because it gets about 10 401's then gives up. | 21:36 |
*** jcoufal_ is now known as jcoufal | 21:36 | |
Vinsh | So it can never get to the heartbeat phase and begin cleaning. | 21:36 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 21:36 |
jroll | mmm | 21:36 |
JayF | 401 unauthorized | 21:36 |
JayF | hmmm.... | 21:37 |
jroll | I can't remember how DIB pulls IPA in | 21:37 |
jroll | but this commit fixes that problem, if I'm not mistaken https://github.com/openstack/ironic-python-agent/commit/fb1cbbdafc0b0c4d5141733b2eda03fe1f2aa878 | 21:37 |
JayF | oh, nice one jroll | 21:37 |
JayF | but that's a 406, not a 401, is it possible you misread teh rror Vinsh ? | 21:38 |
jroll | oh you're right | 21:38 |
Vinsh | No, it's 401, that's a copy paste | 21:38 |
Vinsh | and this is it already giving up on that "new api" and failing back to "vendor passthrough" | 21:38 |
Vinsh | which just loops at 401 | 21:38 |
JayF | Vinsh: can you verify the state of the node that's trying to do a lookup? We do not permit nodes to do a lookup unless they're in certain states | 21:38 |
JayF | Vinsh: so I'm wondering if you got gotta sync somehow | 21:38 |
*** aNuposic has joined #openstack-ironic | 21:39 | |
JayF | if you can gist a copy of your node object with any secrets removed | 21:39 |
JayF | and of the ipa logs | 21:39 |
JayF | I'll take a look | 21:39 |
Vinsh | Its in state "available" | 21:39 |
*** bnemec has quit IRC | 21:39 | |
JayF | when a node is in available state, lookup is not permitted | 21:39 |
JayF | because the agent is not supposed to be running on nodes when they are in available state | 21:39 |
Vinsh | oh sorry that was target state | 21:39 |
JayF | so for security we do not permit the agent to perform a lookup in our default configuration | 21:39 |
Vinsh | its in "clean wait' now | 21:39 |
patchbot | Error: No closing quotation | 21:39 |
Vinsh | trying to get to available | 21:40 |
JayF | OK, gotcha. So it should work from there. Looking. | 21:40 |
JayF | The other thing that's weird is why you can't hit the new api endpoint | 21:40 |
jroll | so it did work once though. huh. | 21:40 |
Vinsh | I have gone so far as to wipe the database, re-add the node.. everything.. no dice today. | 21:40 |
JayF | are you certain you're running stable/newton? | 21:40 |
Vinsh | I'm on stable/mitaka now. stable/newton acted the same. I am certain as they are built in specific containers. | 21:41 |
Vinsh | but ONE time yesterday, it fired right up and begain to clean fine | 21:41 |
JayF | OK. It's completely expected that you will not have the new agent API on mitaka. So that piece is behaving as expected. | 21:41 |
Vinsh | after that.. never again. | 21:41 |
Vinsh | stable/newton reported back the same way, failed back to vendor passthrough | 21:41 |
JayF | and if you're running mitaka, it's not any of the security pieces we added around the agent lookup api | 21:42 |
JayF | Can you please gist the full log from a fresh IPA boot, when it's failing? | 21:42 |
jroll | I'd be curious to verify the SHA IPA is at, too | 21:42 |
Vinsh | I'll get both items. one min | 21:43 |
Vinsh | dib-manifest-git-ironi-deploy claims ironic-python-agent bc1cf1388902dc9fada0f9cea49fbac63aa7d562 | 21:44 |
jroll | ok cool, that's close enough to HEAD for me | 21:45 |
Vinsh | on a side note, this project is awesome, glad to finally be implementing it. | 21:45 |
JayF | I agree, it is awesome :D | 21:45 |
Vinsh | Hope to contribute back alot. | 21:45 |
JayF | lol | 21:46 |
jroll | :) | 21:46 |
*** mtanino has joined #openstack-ironic | 21:46 | |
Vinsh | Here is the debug log http://paste.openstack.org/show/584569/ of an IPA run | 21:47 |
Vinsh | Our plan here is to run ironic in a hybrid setup, vm/baremetal same cloud. I know there are alot of challenges around that.. but it can be done :) | 21:47 |
JayF | I /really/ wish we logged the url we're trying to hit | 21:49 |
JayF | with those failure status code messages | 21:49 |
JayF | Vinsh: can you verify with ironic-api logs that it's actually failling back to the vendor_passthru API with those requests that are 401'ing? | 21:50 |
Vinsh | From the api log, you see a matching: | 21:50 |
Vinsh | 2016-10-05 21:11:13.942 44 INFO keystonemiddleware.auth_token [-] Rejecting request | 21:50 |
Vinsh | 2016-10-05 21:11:13.943 44 INFO ironic_api [-] 24.26.90.188 "POST 6385/v1/drivers/agent_ipmitool/vendor_passthru/lookup HTTP/1.1" status: 401 len: 251 time: 0.0016301 | 21:50 |
JayF | okay, good | 21:50 |
JayF | I think I have an idea what your problem is, gimme one sec | 21:50 |
jroll | waaaait | 21:50 |
jroll | POST 6385/ | 21:50 |
jroll | O_o | 21:50 |
JayF | oh nice catch | 21:50 |
JayF | Vinsh: ^ ip:6385 somehow turned into ip/6385? | 21:50 |
Vinsh | oh yuck! | 21:51 |
jroll | so let's verify with cat /proc/cmdline on the host running IPA | 21:51 |
jroll | and then try to remember how that would get there :) | 21:51 |
Vinsh | ironic_api_url=24.26.90.172:6385 troubleshoot=0 text coreos.autologin ipa-debug=1 boot_option=netboot ipa-api-url=24.26.90.172:6385 | 21:52 |
jroll | huh, that looks right | 21:52 |
openstackgerrit | Michael Davies proposed openstack/ironic: Update ironic config docs for keystone v3 https://review.openstack.org/382108 | 21:52 |
*** jheroux has quit IRC | 21:52 | |
JayF | Vinsh: do most of your loglines have the 6385/ in front of it? | 21:53 |
JayF | Vinsh: or only the ones from IPA | 21:53 |
Vinsh | Just the ones from IPA, ipa has to hit that control node at that IP. that node has a few ips. | 21:54 |
Vinsh | the bm node is on a network that can talk to that 24.... where as the API I talk to is on another ip on same node. | 21:55 |
JayF | Vinsh: and you're running ironic-api on default port (6385), correct? | 21:55 |
Vinsh | Yeah, default | 21:55 |
Vinsh | I can ironic node-show .. set states etc fine. | 21:55 |
JayF | Vinsh: I just can't seem to figure out why your IPA would be trying to prepend the url with the port in the path | 21:55 |
jroll | not really related but this is interesting O_o http://logs.openstack.org/51/361451/9/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa/7ebc693/logs/screen-ir-api.txt.gz#_2016-10-04_18_39_54_207 | 21:55 |
jroll | the line below | 21:55 |
jroll | Got exception calling lookup(): _lookup() takes at least 3 arguments (2 given) (('_lookup() takes at least 3 arguments (2 given)',)) | 21:55 |
*** rbudden has quit IRC | 21:56 | |
Vinsh | is there some other way to feed IPA a url.. to test? I wonder if I can edit cmdline | 21:58 |
JayF | you absolutely can, but I think there's something else going on | 21:58 |
JayF | there's nowhere in the code that port would be munged | 21:58 |
Vinsh | Oh. maybe it needs to start with http:// ? | 21:58 |
Vinsh | I wonder if omiting that in the conifg causes parsing to be off by one or two | 21:59 |
JayF | Kernel command line: selinux=0 disk= iscsi_target_iqn= deployment_id= deployment_key= ironic_api_url= troubleshoot=0 text nofb nomodeset vga=normal console=ttyS0 systemd.journald.forward_to_console=yes ipa-debug=1 boot_option= ip=10.0.0.7:10.205.239.176:10.0.0.1:255.255.255.0 BOOTIF=52:54:00:d4:a2:ee ipa-api-url=http://10.205.239.176:6385 ipa-driver-name=agent_ssh boot_mode= initrd=deploy_ramdisk | 21:59 |
jroll | yeah, worth a try | 21:59 |
JayF | coreos.configdrive=0 | 21:59 |
JayF | that's from a gate run | 21:59 |
jroll | it's the ipa-api-url | 21:59 |
JayF | nice catch, I think you're exactly right | 21:59 |
JayF | and if that's the case, we should file a bug to handle that more sanely | 22:00 |
Vinsh | I can't edit cmdline.. will just edit the config and re-deploy to test | 22:00 |
JayF | Vinsh: you can pass the correct url on the actual ipa command line | 22:00 |
JayF | Vinsh: and it'll override kernel cli | 22:00 |
jroll | Vinsh: you can also pass --api-url to the... yeah | 22:00 |
Vinsh | ah even better | 22:00 |
jroll | requests.exceptions.InvalidSchema: No connection adapters were found for '127.0.0.1:8080/foo' | 22:00 |
jroll | seems like requests doesn't even try | 22:00 |
jroll | weird | 22:01 |
JayF | https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/config.py#L25 | 22:01 |
JayF | the helptext has http:// in it, so that's good | 22:01 |
*** penick has quit IRC | 22:01 | |
Vinsh | it works! | 22:01 |
Vinsh | Dang. I spent a good 5 hours on that :) | 22:02 |
JayF | it's always the silly stuff | 22:02 |
JayF | Do you mind filing a bug about the lack of a good error message for that, aganist ironic-python-agent? | 22:02 |
Vinsh | Thanks so much for digging in on this. | 22:02 |
Vinsh | I will file that bug now. | 22:02 |
JayF | Please tag it "low-hanging-fruit" | 22:02 |
JayF | that way someone new to contributing knows it's probably an easy fix | 22:02 |
JayF | Vinsh: now you gotta pass it on :) idle in chat and help the next person troubleshoot :D | 22:03 |
*** jcoufal has quit IRC | 22:03 | |
Vinsh | So, to improve the logging in IPA that just says "401" | 22:03 |
JayF | Vinsh: operators very welcome in our dev room | 22:03 |
JayF | Vinsh: I'd just describe the problem, what the logs were, and how you fixed it | 22:03 |
Vinsh | Will do, I'll keep the channel on auto-login now. | 22:03 |
jroll | man this is so weird, idgi | 22:03 |
JayF | Vinsh: expected behavior is that it either works without a protocol or dies early, with a good error, demanding a protocol | 22:03 |
jroll | I can reproduce though \o/ | 22:03 |
JayF | Vinsh: I also expect if you go back to stable/newton, it'll find the new agent api too :D | 22:04 |
jroll | almost feels like a requests bug | 22:04 |
JayF | yeah, I agree | 22:04 |
JayF | we don't parse that api string anywhere | 22:04 |
JayF | I was looking for that the whole time | 22:04 |
Vinsh | when I run newton.. I never see a VIF from neutron added to the ironic-port | 22:04 |
Vinsh | I don't think our liberty neutron plays well with it. | 22:04 |
*** penick has joined #openstack-ironic | 22:04 | |
Vinsh | Need to get neutron up to mitaka or later I think. | 22:05 |
Vinsh | no clue though | 22:05 |
Vinsh | I think there is stuff around ports and bare-metal non vm ports in neutron that newton ironic depends on | 22:06 |
JayF | I /think/ depending on the network interface you set in ironic you can work around that, but I'm not 100% sure. Either way, newton is a lot better | 22:06 |
*** vsaienko has quit IRC | 22:06 | |
JayF | one of the biggest adds is that we now properly support oslo.policy, so if you're deploying ironic as part of a larger cloud you can actually do fine-grained access restrictions using policy.json, like many other projects support | 22:07 |
Vinsh | I would MUCH rather be on newton also. I'll look closer at the interface setting and retest. | 22:07 |
Vinsh | Oh yeah, that policy stuff is helpful.. we use it to do a "bring your own IP" type of thing in netutron where we allow customers to provide us their public subnet.. we add it neutron and they use it that way. | 22:08 |
JayF | this is more about policies around restricting access to certain api endpoints | 22:08 |
JayF | rather than those kind of neutron policies | 22:08 |
*** ijw has joined #openstack-ironic | 22:08 | |
Vinsh | Makes sense | 22:08 |
*** ijw has quit IRC | 22:13 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Skip create_ovs_taps() for multitenancy case https://review.openstack.org/382360 | 22:16 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Update iptables rules and services IPs for multinode https://review.openstack.org/368648 | 22:16 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Add devstack setup_vxlan_network() https://review.openstack.org/368647 | 22:16 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Skip some steps for multinode case https://review.openstack.org/368646 | 22:16 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 22:16 |
Vinsh | What is the difference in ironic_api_url and ipa-api-url ? they seem to be populated the same | 22:17 |
*** vishwanathj has quit IRC | 22:18 | |
Vinsh | They seem to both come from api_url in ironic.conf | 22:18 |
JayF | I honestly don't know | 22:19 |
jroll | I think ironic_api_url is for the old ramdisk and needs to be deprecated, maybe | 22:19 |
JayF | jroll: ^ support for legacy bash ramdisk? | 22:19 |
JayF | that can be completely removed? | 22:19 |
JayF | well it's already been deprecated if that's what it is | 22:19 |
jroll | looks like it https://github.com/openstack/diskimage-builder/blob/master/elements/deploy-ironic/init.d/80-deploy-ironic#L100 | 22:19 |
JayF | no match for ironic_api_url anywhere in IPA | 22:19 |
JayF | and in ironic it's only in the pxe templates | 22:19 |
jroll | support has been removed in ironic, but seems like we missed that piece | 22:19 |
JayF | yeah, I'll push a patch real quick for that | 22:20 |
Vinsh | Very good | 22:20 |
JayF | we offer the best service in all of open source | 22:20 |
JayF | troubleshoot your problem | 22:20 |
JayF | and fix reported bugs, while you wait | 22:21 |
JayF | lol | 22:21 |
Vinsh | oil change too?? | 22:21 |
jroll | yes but I'm obliged to take a joy ride | 22:21 |
Vinsh | free dvd rewinding as well. | 22:21 |
Vinsh | sweet. | 22:21 |
JayF | jroll: is troubleshoot=0 that one too? | 22:21 |
JayF | jroll: that's not an ipa thing for sure | 22:21 |
JayF | jroll: wondering about deployment_key as well... | 22:22 |
*** [1]cdearborn has quit IRC | 22:22 | |
JayF | that's a general dib thing, not even just for our deploy ramdisk | 22:23 |
JayF | hmm | 22:23 |
jroll | JayF: that file is the entire source for the old ramdisk bash script | 22:23 |
JayF | man, how the hell is this passing tests? | 22:23 |
JayF | we can't still be populating these... | 22:23 |
JayF | I guess maybe tests were pulled too? | 22:24 |
*** wajdi has quit IRC | 22:24 | |
JayF | I'm going to bug this, because at a minimum it's bigger than I thought | 22:24 |
jroll | I don't get what you mean | 22:24 |
jroll | but like, pxe template tests are weird | 22:24 |
JayF | I'm saying this wants | 22:24 |
JayF | pxe_options.deployment_id | 22:24 |
jroll | and validate against their own thing | 22:24 |
JayF | there's no way we still populate that | 22:24 |
JayF | if we removed the code supporting the old ramdisk | 22:24 |
JayF | so I'm a little surprised the template build doesn't explode | 22:25 |
JayF | I guess fail safe is better | 22:25 |
Vinsh | https://bugs.launchpad.net/ironic/+bug/1630785 | 22:25 |
openstack | Launchpad bug 1630785 in Ironic "IPA fails with 401 contacting ironic-api when protocol is omitted" [Undecided,New] | 22:25 |
* jlvillal realizes he has been missing a lot of conversation. Goes to read the backscroll | 22:25 | |
jroll | JayF: I think jinja fails safe | 22:25 |
jroll | will just be "" if the thing doesn't exist | 22:25 |
JayF | yeah, and if you look at the kernel command line from test | 22:25 |
JayF | they're blank | 22:25 |
JayF | that I pasted a page or two above | 22:25 |
jroll | mhm | 22:26 |
jroll | I don't get what you're trying to say, sorry :/ | 22:26 |
JayF | I'm saying there's more than just ironic_api_url= in that command line | 22:27 |
JayF | that needs removal | 22:27 |
JayF | a lot more | 22:27 |
jroll | oh, totally | 22:27 |
JayF | added ipa to https://bugs.launchpad.net/ironic/+bug/1630785 along with smoe notes to go with our troubleshooting, and triaged it at low importance | 22:29 |
openstack | Launchpad bug 1630785 in ironic-python-agent "IPA fails with 401 contacting ironic-api when protocol is omitted" [Low,Triaged] | 22:29 |
*** hj-hpe has quit IRC | 22:33 | |
* Vinsh headed home from Broomfield CO, to Boulder CO. back on later. | 22:34 | |
*** Vinsh has quit IRC | 22:37 | |
*** awiddersheim has joined #openstack-ironic | 22:39 | |
*** hamzy has quit IRC | 22:42 | |
*** rbrud has quit IRC | 22:55 | |
JayF | jroll: vinsh: filed that bug about deprecated keys still being sent to the kernel cli: https://bugs.launchpad.net/ironic/+bug/1630799 | 23:07 |
openstack | Launchpad bug 1630799 in Ironic "Deprecated bash ramdisk options still exist in pxe templates" [Undecided,New] | 23:07 |
JayF | aNuposic: vgadiraj: aslezil ^ this is a nice, simple bug if one of you were looking for something | 23:07 |
aNuposic | JayF: thanks i will take a look at it | 23:09 |
*** ijw has joined #openstack-ironic | 23:10 | |
*** penick has quit IRC | 23:15 | |
*** ijw has quit IRC | 23:15 | |
*** Sukhdev has quit IRC | 23:17 | |
*** Sukhdev has joined #openstack-ironic | 23:19 | |
*** izaakk has quit IRC | 23:28 | |
*** logan- has quit IRC | 23:28 | |
*** izaakk has joined #openstack-ironic | 23:33 | |
*** logan- has joined #openstack-ironic | 23:37 | |
*** hamzy has joined #openstack-ironic | 23:44 | |
*** sdake has quit IRC | 23:48 | |
*** aNuposic has quit IRC | 23:54 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!