Wednesday, 2016-10-05

*** rama_y has quit IRC00:01
openstackgerritMerged openstack/ironic-lib: Remove tests and common.i18n from autodoc  https://review.openstack.org/38109000:08
*** aNuposic has quit IRC00:10
*** penick has quit IRC00:13
*** penick has joined #openstack-ironic00:14
*** sdake_ has quit IRC00:16
*** thrash is now known as thrash|g0ne00:25
*** pester has quit IRC00:30
openstackgerritMerged openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/37613600:31
*** mtanino has quit IRC00:32
*** penick has quit IRC00:33
*** hj-hpe has quit IRC00:37
*** jvgrant_ has joined #openstack-ironic00:39
*** jvgrant has quit IRC00:43
*** hoangcx has joined #openstack-ironic00:48
*** sdake has joined #openstack-ironic00:53
openstackgerritMerged openstack/ironic: Fix iLO drivers to not clear local_gb if its not detected  https://review.openstack.org/34175200:58
openstackgerritMerged openstack/ironic: Add more details to MIGRATIONS_TIMEOUT note  https://review.openstack.org/37504800:59
openstackgerritMerged openstack/ironic: Change links to point to new install guide  https://review.openstack.org/37864000:59
*** BadCub has quit IRC01:00
*** phuongnh has joined #openstack-ironic01:00
*** Ng has quit IRC01:01
*** wajdi has quit IRC01:01
*** wajdi has joined #openstack-ironic01:02
*** kun_huang has quit IRC01:04
*** Sukhdev has quit IRC01:04
*** syed_ has quit IRC01:05
*** wajdi has quit IRC01:06
*** Ng has joined #openstack-ironic01:08
*** baoli has quit IRC01:09
*** BadCub has joined #openstack-ironic01:14
*** kun_huang has joined #openstack-ironic01:17
*** syed_ has joined #openstack-ironic01:17
*** baoli has joined #openstack-ironic01:20
*** tuanluong has joined #openstack-ironic01:21
*** rloo has quit IRC01:23
*** rama_y has joined #openstack-ironic01:24
*** baoli has quit IRC01:27
*** [1]cdearborn has quit IRC01:29
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/37744601:29
*** rbudden has joined #openstack-ironic01:35
openstackgerritMerged openstack/ironic: Fixing Typo  https://review.openstack.org/37656401:38
*** bdeschenes has quit IRC01:48
*** rama_y has quit IRC01:50
*** tiendc has joined #openstack-ironic01:58
*** rbudden has quit IRC02:00
*** mtanino has joined #openstack-ironic02:01
*** rbudden has joined #openstack-ironic02:03
*** syed_ has quit IRC02:04
openstackgerritMerged openstack/ironic-python-agent: Fixed typo in doc file  https://review.openstack.org/37813702:15
openstackgerritMerged openstack/ironic-lib: Add prefix "$" for command examples  https://review.openstack.org/37861102:17
*** sdake has quit IRC02:18
*** amotoki has joined #openstack-ironic02:19
*** sdake has joined #openstack-ironic02:20
*** amotoki has quit IRC02:31
*** rbudden has quit IRC02:33
*** yuanying_ has quit IRC02:39
*** amotoki has joined #openstack-ironic02:43
*** amotoki_ has joined #openstack-ironic03:04
*** amotoki has quit IRC03:07
*** Sukhdev has joined #openstack-ironic03:09
openstackgerritMerged openstack/ironic-inspector: TrivialFix: Fix typo in the configuration file  https://review.openstack.org/37910803:10
openstackgerritMerged openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/37881903:10
*** Sukhdev has quit IRC03:13
*** appu103_ has joined #openstack-ironic03:25
*** vikrant has joined #openstack-ironic03:30
*** amotoki has joined #openstack-ironic03:35
*** amotoki has quit IRC03:36
*** amotoki_ has quit IRC03:36
*** sdake has quit IRC03:43
*** sdake has joined #openstack-ironic03:43
*** links has joined #openstack-ironic03:45
*** vmud213 has joined #openstack-ironic03:46
*** vishwanathj has quit IRC03:50
*** yuanying has joined #openstack-ironic04:02
*** wajdi has joined #openstack-ironic04:02
*** amotoki has joined #openstack-ironic04:06
*** yuanying has quit IRC04:12
*** yuanying has joined #openstack-ironic04:13
*** amotoki_ has joined #openstack-ironic04:19
*** amotoki has quit IRC04:23
*** sdake has quit IRC04:25
*** wajdi has quit IRC04:36
*** yuanying has quit IRC04:36
*** wajdi has joined #openstack-ironic04:36
*** yuanying has joined #openstack-ironic04:42
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/37744604:47
*** syed_ has joined #openstack-ironic04:50
*** sdake has joined #openstack-ironic04:51
*** HenryG has quit IRC04:51
*** HenryG has joined #openstack-ironic04:52
*** bjolo_ has joined #openstack-ironic04:58
*** amotoki_ has quit IRC04:59
*** jaosorior has joined #openstack-ironic05:07
*** wajdi has quit IRC05:11
*** penick has joined #openstack-ironic05:18
*** jaosorior has quit IRC05:35
*** jaosorior has joined #openstack-ironic05:35
*** moshele has joined #openstack-ironic05:36
*** bks has joined #openstack-ironic05:40
*** amotoki has joined #openstack-ironic05:43
*** amotoki has quit IRC05:57
*** wajdi has joined #openstack-ironic06:02
*** mjura has joined #openstack-ironic06:06
*** vsaienko has joined #openstack-ironic06:08
*** mjura has quit IRC06:12
*** mjura has joined #openstack-ironic06:12
*** yuanying has quit IRC06:16
*** bjolo_ has quit IRC06:24
*** jtomasek_ has joined #openstack-ironic06:36
*** milan has joined #openstack-ironic06:36
milanmorning Ironic! :)06:39
*** vsaienko has quit IRC06:39
*** pcaruana has joined #openstack-ironic06:46
*** jtomasek_ has quit IRC06:50
*** rcernin has joined #openstack-ironic06:53
openstackgerritHironori Shiina proposed openstack/ironic: Keep numbering of list in Install Guide  https://review.openstack.org/38216907:00
*** fxpester has joined #openstack-ironic07:04
*** AlexeyAbashkin has quit IRC07:07
*** tesseract- has joined #openstack-ironic07:07
*** rbartal has joined #openstack-ironic07:08
*** rcernin has quit IRC07:12
*** rcernin has joined #openstack-ironic07:12
*** athomas has joined #openstack-ironic07:15
*** amotoki has joined #openstack-ironic07:20
*** hogepodge has quit IRC07:21
*** amotoki has quit IRC07:32
*** jpich has joined #openstack-ironic07:33
*** syed_ has quit IRC07:34
openstackgerritMilan Kováčik proposed openstack/ironic-inspector-specs: Endpoint to list introspection statuses  https://review.openstack.org/37504507:35
*** e0ne has joined #openstack-ironic07:36
*** amotoki has joined #openstack-ironic07:43
*** AlexeyAbashkin has joined #openstack-ironic07:46
*** tuanluong has quit IRC07:54
*** amotoki has quit IRC07:56
pas-hamorning Ironic :) and milan :)07:59
*** yolanda has joined #openstack-ironic07:59
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
yolandahi, i'm trying some iscsi deployment on ironic, and i'm getting following error: http://paste.openstack.org/show/584381/08:00
yolandainvalid error code 65280 on Re-reading iSCSI luns, anyone has experimented it?08:01
*** yuanying has joined #openstack-ironic08:01
*** dnikishov has joined #openstack-ironic08:08
*** dnikishov has quit IRC08:10
*** dnikishov has joined #openstack-ironic08:10
*** hogepodge has joined #openstack-ironic08:11
*** appu103_ has quit IRC08:14
*** appu103_ has joined #openstack-ironic08:16
*** mgoddard has joined #openstack-ironic08:16
*** ccamacho has joined #openstack-ironic08:16
*** amotoki has joined #openstack-ironic08:17
*** ohamada has joined #openstack-ironic08:18
*** Romanenko_K has joined #openstack-ironic08:25
*** dtantsur|afk is now known as dtantsur08:26
dtantsurmorning Ironic, pas-ha, milan, yolanda08:26
dtantsurmilan, you're an early bird today, aren't you?08:26
dtantsuryolanda, no, never... frankly speaking, the iscsi deployment is more fragile than it should be08:27
yolandadtantsur, so we are testing some use case now, that is to start using full disk images, and partitioned with swap, logical partitions... in ironic08:28
yolandaso trying to debug a bit more08:28
dtantsuryolanda, I'd recommend using agent_* drivers with whole disk images for now. using pxe_* drivers with them is pretty new.08:28
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Tests for testing port-create command  https://review.openstack.org/29180208:33
yolandadtantsur, i'm testing under tripleo scope, can it be customized?08:36
yolandahow do you specify pxe_ or agent_ ?08:36
dtantsuryolanda, hmm, I think tripleo will need additional configuration to use agent_* drivers...08:37
yolandadtantsur, 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 well08:38
dtantsuryolanda, in theory, whole disk images work with pxe drivers too. your error does not seem to be directly related though08:39
yolandayep, i'm adding more debug messages to get exactly the point of failure08:40
yolandai'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
yolandaso i was wondering if the first error was a red herring08:40
dtantsuryolanda, int problem was fixed in ironic-lib master, but we can't do a release yet til next Monday :(08:41
dtantsurbecause of #openstack-release processes08:42
yolandaah... so i can install from source08:42
yolandaso i better install ironic and ironic-lib from git to test?08:43
dtantsuryolanda, you don't need ironic from git, you just need to pick https://github.com/openstack/ironic-lib/commit/7323bf50e60383fb05f0655062cb4b830174888f08:43
dtantsuralso it was backported to stable/newton, so no need for master, if you use newton08:43
yolandaah thx08:43
dtantsur(still, not released, so RDO won't pick it, for example)08:44
yolandayou saved my day :)08:44
*** strigazi_AFK is now known as strigazi08:44
dtantsurI'm not sure it fixes the iscsi problem, but the int() problem is real and breaks deployments with config drives08:44
yolandayep, hitting that08:44
dtantsur(and tripleo forces config drives)08:44
yolandaso i'll apply that fix and retest to see if i arrive further08:45
dtantsurthanks!08:45
*** derekh has joined #openstack-ironic08:46
pas-hamilan: '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 manager08:47
openstackgerritYuriy Zveryanskyy proposed openstack/ironic-specs: Add CRUD and node states notifications to ironic  https://review.openstack.org/34724208:47
pas-haand AFAIU we use testtools' version everywhere, as our TestCases are inherited from testtools.TestCase08:48
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with node-get command  https://review.openstack.org/34206008:48
openstackgerritNaohiro Tamura proposed openstack/ironic: Adds the default implementation of get_supported_power_state  https://review.openstack.org/38219408:50
*** jaosorior is now known as jaosorior_lunch08:50
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with Chassis.  https://review.openstack.org/34379608:55
yolandadtantsur, that worked! i have ironic nodes on active state now08:58
dtantsur\o/08:58
milanmorning dtantsur pas-ha yolanda! :)08:58
milanpas-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 IRC08:59
milanbut never mind, I've updated my patch not to depend on the ctx manager anymore :-/09:00
*** amotoki has quit IRC09:00
pas-haIMO that's a breaking change that should be reported09:00
milanpas-ha, maybe09:01
*** awiddersheim has quit IRC09:06
*** awiddersheim has joined #openstack-ironic09:11
*** skazi has quit IRC09:11
*** dnikishov has quit IRC09:13
fxpesterhi 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/147717909:13
openstackLaunchpad bug 1477179 in diskimage-builder "centos 6 image doesn't boot after creation" [Medium,Triaged]09:13
*** skazi has joined #openstack-ironic09:14
dtantsurfxpester, 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#L709:16
milandtantsur, would you mind review on: https://review.openstack.org/#/c/375045/09:17
patchbotpatch 375045 - ironic-inspector-specs - Endpoint to list introspection statuses09:17
dtantsurmilan, sure, I'll get to it today09:18
milanjust 312 changes ;)09:18
milanthx, would sort of unblock me09:18
dtantsurmilan, if you have nothing else to do, I have a lot of exciting stuff in backlog to keep you busy ;)09:20
milandtantsur, I'd love to finish what's pending atm ;)09:20
dtantsurhehe09:20
*** skazi has quit IRC09:22
fxpesterdtantsur: I use minimum elements: `rhel7 baremetal dhcp-all-interfaces grub2` tripeo use the same, will try to get latest dib from git...09:26
dtantsuryes please.. your list looks correct. maybe the image is not to blame then.09:26
aarefievmorning!09:27
dtantsurhey-hey aarefiev09:27
*** amotoki has joined #openstack-ironic09:31
*** amotoki has quit IRC09:31
*** amotoki has joined #openstack-ironic09:31
milanmorning aarefiev! :)09:32
aarefievmorning dtantsur, milan09:32
milanaarefiev, lemme fix the sorting keys! :)09:32
aarefievmilan: thanks, I like proposed filtering09:33
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal node commands  https://review.openstack.org/30168709:33
aarefievgood work09:33
aarefievdtantsur: what do you think about job with radosgw  backend in inspector09:34
dtantsuraarefiev, I'm not against, even though I'm not sure what our benefit is09:35
aarefievdtantsur: catching possible bugs, to be sure it works, there are a lot of envs with ceph09:36
aarefievwe can switch discovery job for example09:36
aarefievand not adding new one09:36
dtantsurmakes sense09:36
aarefievit will test  ironic as well09:38
dtantsuronly if we use agent_* drivers there, right?09:39
* dtantsur is unsure which driver we use09:39
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-python-agent: [tinyipa] remove trailing slash from TC repo URL  https://review.openstack.org/38222709:40
aarefievdtantsur: I think so, but unsure too09:40
*** skazi has joined #openstack-ironic09:41
*** mgould|afk is now known as mgould09:44
mgouldmorning Ironic!09:44
*** jaosorior_lunch is now known as jaosorior09:46
milanaarefiev, thx! :)09:50
milanmorning mgould09:50
mgouldmorning milan aarefiev dtantsur fxpester jaosorior09:56
aarefievmorning mgould09:56
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with Chassis.  https://review.openstack.org/34379609:56
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with chassis-update  https://review.openstack.org/34471509:56
jaosoriormorning mgould09:56
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with node-list command  https://review.openstack.org/36054909:58
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with node-port-list.  https://review.openstack.org/36063610:00
vdrokgood morning ironic, milan pas-ha dtantsur aarefiev mgould10:04
milanmorning vdrok! :)10:05
aarefievhey vdrok10:05
mgouldmorning vdrok10:06
fxpesterok, no luck even with fresh dib...10:09
fxpestermaybe 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
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal port commands  https://review.openstack.org/36569210:10
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add tests for 'baremetal port create' command  https://review.openstack.org/33288010:11
openstackgerritMilan Kováčik proposed openstack/ironic-inspector-specs: Endpoint to list introspection statuses  https://review.openstack.org/37504510:13
milanaarefiev, ^ I've updated the API accordingly too10:14
*** sambetts|afk is now known as sambetts10:21
sambettsMorning all10:21
milanmorning sambetts! :)10:21
*** mgoddard_ has joined #openstack-ironic10:22
*** mgoddard has quit IRC10:25
aarefievmilan: thanks10:30
aarefievmorning sambetts10:31
*** hoangcx has quit IRC10:31
milanaarefiev, my pleasure ;)10:31
milanaarefiev, 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 directions10:33
dtantsurmorning vdrok, sambetts10:33
dtantsurfxpester, this rings some bells.. lemme try to remember10:33
dtantsurhmm, no, it was for Fedora10:36
dtantsurfxpester, try asking on #tripleo, there are folks much more experienced with DIB there10:36
*** athomas has quit IRC10:40
vdrokmorning sambetts10:44
*** thrash|g0ne is now known as thrash10:54
*** awiddersheim has quit IRC10:57
aarefievmilan: why do we need to return multiple queries11:01
*** athomas has joined #openstack-ironic11:03
*** tiendc has quit IRC11:04
milanaarefiev, so you can specify e.g multiple states as a query11:10
aarefievmilan: yeah, I specify multiple states, and return one query with provided states, no?11:11
milanaarefiev, yes, but HTTP doesn't support types over queries11:11
aarefievor your mean provide multiple queries11:12
milanso multiplicity is usually expressed like repetition11:12
openstackgerritYuriy Zveryanskyy proposed openstack/ironic-specs: Add CRUD and node states notifications to ironic  https://review.openstack.org/34724211:12
milanaarefiev, so a sample query would be "?state=waiting&state=processing"11:12
milanin python: {'state': ['waiting', 'processing'], ...}11:13
aarefievahh, ok, I thought about other query11:13
aarefievyeah, nice11:13
milanaarefiev, so in the code I'd reduce the lists with an (sql) 'or' statement and reduce the dict of lists with an 'and' statement11:14
*** ccamacho is now known as ccamacho|lunch11:15
* milan 's coffee is ready \o/ :D11:16
openstackgerritMerged openstack/bifrost: Add read and execute perms to httpboot and tftpboot folder  https://review.openstack.org/37401511:16
TheJuliaGood morning everyone11:19
aarefievmilan: yeah, that make sense, I understand you wrong first time, thanks11:19
aarefievmorning TheJulia11:19
milanmorning TheJulia! :)11:19
*** bdeschenes has joined #openstack-ironic11:19
*** marksei has joined #openstack-ironic11:21
openstackgerritMerged openstack/bifrost: Allow users to change the default provisioning timeout value  https://review.openstack.org/37630311:22
mgould  morning TheJulia11:23
openstackgerritMathieu Mitchell proposed openstack/ironic-specs: RAID configuration during deployment  https://review.openstack.org/38209111:26
mat128good morning TheJulia11:26
mat128dtantsur: ^11:26
mgouldmorning mat12811:26
milanmorning mat128! :)11:27
dtantsurmat128, \o/ thanks11:30
dtantsurmorning TheJulia, mat12811:31
*** ohamada has quit IRC11:37
*** ohamada has joined #openstack-ironic11:37
*** bdeschenes has quit IRC11:39
*** vsaienko has joined #openstack-ironic11:40
*** vsaienko has quit IRC11:42
*** mgoddard_ has quit IRC11:44
milansambetts, dtantsur we've got +2 on https://review.openstack.org/#/c/375045/7 mind a review? ;)11:46
patchbotpatch 375045 - ironic-inspector-specs - Endpoint to list introspection statuses11:46
dtantsurstill on my way to it, sorry11:46
sambettsmilan: reading it now11:46
milancool thx, I'd proceed w/ https://review.openstack.org/#/c/344921/11:47
patchbotpatch 344921 - ironic-inspector - Add API for listing all introspection statuses11:47
vdrokmorning TheJulia and mat12811:49
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal chassis commands  https://review.openstack.org/36615811:49
mat128o/ vdrok11:51
*** appu103_ has quit IRC11:52
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add negative test-cases for openstack node create command  https://review.openstack.org/38194311:53
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Tests for OSC baremetal node create command with options  https://review.openstack.org/38235211:54
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit  https://review.openstack.org/36902111:55
lucasagomesmorning all11:56
milanmorning lucasagomes! :)11:56
vdrokmorning lucasagomes11:56
*** bdeschenes has joined #openstack-ironic11:57
mat128morning lucasagomes11:57
mgouldmorning lucasagomes11:57
mat128Anyone 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_updates11:57
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with Chassis.  https://review.openstack.org/34379611:57
*** trown|outtypewww is now known as trown12:02
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: [install-guide] describe service clients auth  https://review.openstack.org/38235812:02
openstackgerritVasyl Saienko proposed openstack/ironic: Skip create_ovs_taps() for multitenancy case  https://review.openstack.org/38236012:03
*** appu103_ has joined #openstack-ironic12:03
*** bdeschenes has quit IRC12:04
*** [1]cdearborn has joined #openstack-ironic12:04
*** jcoufal has joined #openstack-ironic12:09
*** wajdi has quit IRC12:09
*** wajdi has joined #openstack-ironic12:10
mat128lucasagomes: thank you :)12:17
*** phuongnh has quit IRC12:17
lucasagomesmat128, :-) yw12:17
*** appu103_ has quit IRC12:18
*** sdake has joined #openstack-ironic12:19
openstackgerritMerged openstack/ironic: Update external links to developer documentation  https://review.openstack.org/37899012:19
*** vmud213 has quit IRC12:20
openstackgerritMerged openstack/ironic: Update configuration reference link to latest draft  https://review.openstack.org/37899112:21
*** ccamacho|lunch is now known as ccamacho12:24
openstackgerritLucas Alvares Gomes proposed openstack/ironic-lib: Correctly count the number of primary partitions  https://review.openstack.org/38189312:26
sambettsmilan: +2 from me12:27
milansambetts, awesome, thanks! :)12:27
sambettsmilan: I wonder if we should wait for dtantsur's opinion too12:27
milanAFAIK dtantsur wanted to review it today ;)12:28
sambettsok :) I'll wait to +A it until he's had a look, but I hope to merge it today12:29
milansambetts, yeah, that'd be nice12:29
milanbtw12:29
milanw/r the list api patch (code),12:29
*** krtaylor has quit IRC12:30
milanI wonder how to split it12:30
*** mgoddard has joined #openstack-ironic12:30
*** [1]cdearborn has quit IRC12:31
*** dnikishov has joined #openstack-ironic12:31
openstackgerritLucas Alvares Gomes proposed openstack/ironic-lib: Correctly count the number of primary partitions  https://review.openstack.org/38189312:31
sambettsmilan: 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
milansambetts, yeah, would be my preference too: pagination(with the default sorting)&time filtering according to the spec12:35
milanstates filter & custom sorting as separate patches12:36
milanto minimise the work needed12:36
milan*rework12:36
mat128lucasagomes: found this in vbmc: https://review.openstack.org/#/c/373687/12:36
patchbotpatch 373687 - virtualbmc - Updated from global requirements12:36
lucasagomesmat128, yay o/ the bot is now working!12:37
mat128:)12:37
lucasagomesmat128, why it;s adding a HP copyright line at setup.py12:37
mat128lucasagomes: I think it was supposed to be there. Maybe the script just copies the file from somewhere else?12:38
mat128Let me check if we have the HP copyright in Ironic12:38
lucasagomesyeah probably just diff it... it's odd anyway12:38
mat128lucasagomes: agreed12:38
mat128lucasagomes: https://github.com/openstack/ironic/blob/master/setup.py12:38
mat128theres nothing much to have rights on anyway12:38
*** [1]cdearborn has joined #openstack-ironic12:39
lucasagomesheh true12:39
lucasagomes+2;d12:39
openstackgerritVasyl Saienko proposed openstack/ironic: Skip create_ovs_taps() for multitenancy case  https://review.openstack.org/38236012:41
*** baoli has joined #openstack-ironic12:44
*** Goneri has joined #openstack-ironic12:45
openstackgerritMerged openstack/virtualbmc: Updated from global requirements  https://review.openstack.org/37368712:48
*** ohamada has quit IRC12:48
*** ohamada has joined #openstack-ironic12:48
*** wajdi has quit IRC12:50
*** wajdi has joined #openstack-ironic12:50
*** milan is now known as milan|bbl12:52
*** vikrant has quit IRC12:54
*** wajdi has quit IRC12:55
*** makowals has joined #openstack-ironic12:56
mariojvgood morning ironic12:57
sambettso/ mariojv12:57
mgouldmorning mariojv12:57
*** makowals_ has quit IRC12:57
*** iurygregory has joined #openstack-ironic12:58
*** jheroux has joined #openstack-ironic12:59
*** nicodemos has joined #openstack-ironic13:00
*** bks has quit IRC13:00
*** wajdi has joined #openstack-ironic13:02
*** vsaienko has joined #openstack-ironic13:02
*** jcoufal_ has joined #openstack-ironic13:03
*** jcoufal has quit IRC13:03
jrollmorning y'all13:04
*** baoli_ has joined #openstack-ironic13:05
nicodemosmorning jroll13:05
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: [install-guide] describe service clients auth  https://review.openstack.org/38235813:07
lucasagomesmariojv, sambetts mgould jroll nicodemos morning13:07
openstackgerritMathieu Mitchell proposed openstack/ironic-specs: RAID configuration during deployment  https://review.openstack.org/38209113:08
*** baoli has quit IRC13:08
mat128thanks mgould and wajdi for the quick feedback13:09
*** hshiina has joined #openstack-ironic13:09
nicodemoshey, lucasagomes13:09
wajdimat128: +113:10
*** ccamacho has quit IRC13:16
*** dprince has joined #openstack-ironic13:16
*** mjturek has joined #openstack-ironic13:17
*** mjturek has left #openstack-ironic13:17
*** mjturek has joined #openstack-ironic13:18
*** wajdi has quit IRC13:19
mgouldmat128: np13:21
mgouldmorning jroll nicodemos13:21
openstackgerritGalyna Zholtkevych proposed openstack/python-ironicclient: Add command `driver properties` to osc  https://review.openstack.org/38115313:21
*** zhenguo has joined #openstack-ironic13:32
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification  https://review.openstack.org/34843713:33
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: devstack portgroup support  https://review.openstack.org/38174313:34
openstackgerritMichael Turek proposed openstack/python-ironicclient: Refactor provision state so all actions can use inherited take_action  https://review.openstack.org/36455213:34
*** ccamacho has joined #openstack-ironic13:34
*** jaosorior has quit IRC13:36
*** jaosorior has joined #openstack-ironic13:36
openstackgerritVadim Hmyrov proposed openstack/python-ironicclient: Add option "--split" to command "ironic node-list"  https://review.openstack.org/37557913:37
openstackgerritHironori Shiina proposed openstack/ironic: Fix undisplayed notes in Quick-Start  https://review.openstack.org/38241613:38
*** bnemec has joined #openstack-ironic13:40
*** marksei has quit IRC13:41
*** mgoddard_ has joined #openstack-ironic13:45
*** moshele has quit IRC13:46
*** mgoddard has quit IRC13:48
*** rbudden has joined #openstack-ironic13:48
*** mgoddard has joined #openstack-ironic13:52
*** mgoddard_ has quit IRC13:53
jrollI'm off sick again today, ping me on hangouts if you desperately need anything13:53
sambettsjroll: get better soon dude13:54
*** alex_xu has quit IRC13:55
lucasagomesjroll, :-/ hope you get better soon!13:57
openstackgerritJulia Kreger proposed openstack/bifrost: Add ability to write clouds.yaml  https://review.openstack.org/38022713:57
openstackgerritJulia Kreger proposed openstack/bifrost: Add support to update keystone for bifrost  https://review.openstack.org/38022613:57
*** vsaienko has quit IRC13:57
openstackgerritJulia Kreger proposed openstack/bifrost: Add keystone installation playbook  https://review.openstack.org/38022513:57
openstackgerritJulia Kreger proposed openstack/bifrost: Update playbooks to support os_client_config  https://review.openstack.org/38022413:57
*** vsaienko has joined #openstack-ironic13:58
xavierrgood morning Ironic14:00
openstackgerritJulia Kreger proposed openstack/bifrost: Update Ansible version to stable-2.1  https://review.openstack.org/37804114:00
openstackgerritJulia Kreger proposed openstack/bifrost: Add keystone install and clouds.yaml to testing  https://review.openstack.org/38022814:00
openstackgerritJulia Kreger proposed openstack/bifrost: Add ability to write clouds.yaml  https://review.openstack.org/38022714:00
openstackgerritJulia Kreger proposed openstack/bifrost: Add support to update keystone for bifrost  https://review.openstack.org/38022614:00
openstackgerritJulia Kreger proposed openstack/bifrost: Add keystone installation playbook  https://review.openstack.org/38022514:00
openstackgerritJulia Kreger proposed openstack/bifrost: Update playbooks to support os_client_config  https://review.openstack.org/38022414:00
*** alex_xu has joined #openstack-ironic14:02
openstackgerritJulia Kreger proposed openstack/bifrost: Convert ironic.conf to template generated  https://review.openstack.org/30563414:02
*** vsaienko has quit IRC14:04
*** rbrud has joined #openstack-ironic14:04
*** makowals has quit IRC14:04
*** vsaienko has joined #openstack-ironic14:04
*** joprovost has joined #openstack-ironic14:05
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: devstack portgroup support  https://review.openstack.org/38174314:06
mgouldjroll: get well soon!14:06
*** makowals has joined #openstack-ironic14:06
*** vsaienko has quit IRC14:07
dtantsurmorning jroll, get better!14:07
openstackgerritMerged openstack/ironic-inspector: LLC Hook: Fix patching Ironic ports  https://review.openstack.org/38040014:07
openstackgerritMario Villaplana proposed openstack/ironic: Add power state change notifications  https://review.openstack.org/32186514:07
*** wajdi has joined #openstack-ironic14:08
*** absubram has quit IRC14:08
*** dprince has quit IRC14:10
*** dims has quit IRC14:11
*** links has quit IRC14:14
*** jaosorior has quit IRC14:16
*** vsaienko has joined #openstack-ironic14:16
*** dims has joined #openstack-ironic14:17
*** vsaienko has quit IRC14:17
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Move build_instance_info_for_deploy to deploy_utils  https://review.openstack.org/37760814:21
dtantsurfolks, what are the requirements for running our tempest plugin? nodes enrolled, images, networks? do we have it documented?14:21
TheJuliaafaik we do not14:21
*** vsaienko has joined #openstack-ironic14:23
lucasagomesdtantsur, I think I've added some docs to run with devstack14:23
* lucasagomes finds14:23
lucasagomesdtantsur, http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#running-tempest-tests14:24
lucasagomesnot much info there, but may be a starting point14:24
*** rpioso has joined #openstack-ironic14:24
dtantsurwell, devstack does a lot of preparation itself, so unfortunately it doesn't help much14:24
lucasagomesyeah, I guess the best source is the devstack script itself14:25
*** hj-hpe has joined #openstack-ironic14:28
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification  https://review.openstack.org/34843714:29
openstackgerritMerged openstack/ironic-inspector: LLC Hook: Do not assume interfaces are added to Ironic  https://review.openstack.org/38040114:33
*** baoli_ has quit IRC14:35
*** moshele has joined #openstack-ironic14:35
*** vsaienko has quit IRC14:36
*** moshele has quit IRC14:38
*** hj-hpe has quit IRC14:38
*** hogepodge has quit IRC14:38
*** tesseract- has quit IRC14:38
*** fxpester has quit IRC14:38
*** milan|bbl has quit IRC14:38
*** mjura has quit IRC14:38
*** BadCub has quit IRC14:38
*** Ng has quit IRC14:38
*** jjohnson2 has quit IRC14:38
*** pmannidi has quit IRC14:38
*** clenimar has quit IRC14:38
*** davidlenwell has quit IRC14:38
*** ipukha has quit IRC14:38
*** marlinc has quit IRC14:38
*** pmannidi has joined #openstack-ironic14:39
*** hj-hpe has joined #openstack-ironic14:39
*** mjura has joined #openstack-ironic14:39
*** fxpester has joined #openstack-ironic14:39
*** milan|bbl has joined #openstack-ironic14:39
*** clenimar has joined #openstack-ironic14:39
*** jjohnson2 has joined #openstack-ironic14:39
*** hogepodge has joined #openstack-ironic14:39
*** bnemec has quit IRC14:39
*** tesseract- has joined #openstack-ironic14:39
*** davidlenwell has joined #openstack-ironic14:40
*** mkam has joined #openstack-ironic14:40
*** ipukha has joined #openstack-ironic14:40
openstackgerritJulia Kreger proposed openstack/bifrost: Add Keystone support to ironic.conf  https://review.openstack.org/38246914:42
*** mkam has quit IRC14:43
*** bnemec has joined #openstack-ironic14:44
*** penick has quit IRC14:44
*** rpioso has quit IRC14:44
*** mkam has joined #openstack-ironic14:45
*** Ng has joined #openstack-ironic14:46
openstackgerritGalyna Zholtkevych proposed openstack/python-ironicclient: Add command `driver properties` to osc  https://review.openstack.org/38115314:46
*** marlinc has joined #openstack-ironic14:48
*** BadCub has joined #openstack-ironic14:49
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: tempest tests for bonding  https://review.openstack.org/38247614:50
vsaienk0jroll, dtantsur, sambetts: could you please let me know your opinion about: https://review.openstack.org/#/c/382438/1/nova/virt/ironic/driver.py@68514:52
patchbotpatch 382438 - nova - WIP: Add portgroups support to ironic virt driver14:52
*** narasimha_SV has joined #openstack-ironic14:52
narasimha_SVhttp://paste.openstack.org/show/584478/14:52
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: [install-guide] describe pxe.ipxe_swift_tempurl  https://review.openstack.org/38247814:52
narasimha_SVin ironic-api i am getting these logs14:53
narasimha_SVit is unable to get the token and not able to validate those tokens14:53
*** mkam has quit IRC14:53
*** wajdi_ has joined #openstack-ironic14:53
narasimha_SVhttp://paste.openstack.org/show/584479/ there are the configurations in my ironic.conf14:54
*** wajdi has quit IRC14:56
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Clean exceptions handling in conductor manager  https://review.openstack.org/38155414:57
openstackgerritDevananda van der Veen proposed openstack/ironic-specs: API Evolution - Error Responses  https://review.openstack.org/38198914:57
*** wajdi_ has quit IRC15:01
*** jistr is now known as jistr|call15:01
openstackgerritDevananda van der Veen proposed openstack/ironic-specs: API Evolution - Implement Tasks Resource  https://review.openstack.org/38201715:01
*** wajdi has joined #openstack-ironic15:01
*** mjura has quit IRC15:05
*** mkam has joined #openstack-ironic15:06
*** rcernin has quit IRC15:12
*** pcaruana has quit IRC15:13
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add tests for OSC baremetal chassis create command  https://review.openstack.org/38249615:13
*** jistr|call is now known as jistr15:13
*** hshiina has quit IRC15:15
*** ijw has joined #openstack-ironic15:16
openstackgerritMathieu Mitchell proposed openstack/ironic-specs: RAID configuration during deployment  https://review.openstack.org/38209115:18
*** tesseract- has quit IRC15:24
openstackgerritMerged openstack/ironic: Use six.StringIO instead of six.moves.StringIO  https://review.openstack.org/37529715:26
jlvillalvsaienk0: 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
vdrokmorning jlvillal15:30
vdrokdevananda: around?15:30
jlvillalHi vdrok :)15:30
jlvillalvdrok: 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
patchbotpatch 364830 - openstack-infra/devstack-gate - Setup ssh-key on subnodes for Ironic15:31
*** vishwanathj has joined #openstack-ironic15:31
vdrokjlvillal: heh, yep, happens sometimes :)15:32
jlvillalvdrok: I hope it gets merged! I worked on that patch too :)15:32
vdrokjlvillal: bring in intel folks then :)15:32
jlvillalheh15:33
*** rbartal has quit IRC15:34
*** lucasagomes is now known as lucas-hungry15:35
*** jaosorior has joined #openstack-ironic15:37
sambettsvsaienk0: I don't mind it, however you are missing some of the logic around vif to pif mapping15:38
*** baoli has joined #openstack-ironic15:39
*** dprince has joined #openstack-ironic15:39
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add tests for maintenance mode commands  https://review.openstack.org/33725915:39
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add negative tests for baremetal node commands  https://review.openstack.org/33771915:40
sambettsvsaienk0: 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 portgroup15:40
*** rama_y has joined #openstack-ironic15:40
sambettss/loadbalanced/round robined15:40
openstackgerritGalyna Zholtkevych proposed openstack/python-ironicclient: Add command `driver properties` to osc  https://review.openstack.org/38115315:42
*** zhenguo has quit IRC15:43
mariojvJayF: was this the BFV spec you were talking about? https://review.openstack.org/#/c/294995/15:45
patchbotpatch 294995 - ironic-specs - Boot from Volume - Reference Drivers15:45
JayFyes15:45
mat128JayF: you might be interested by this spec: https://review.openstack.org/38209115:45
patchbotpatch 382091 - ironic-specs - RAID configuration during deployment15:45
mat128it's what we discussed during the midcycle, re: deploy_steps15:45
milan|bblsambetts, 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
TheJuliadtantsur: I take it with roles that inspector would need to be a baremetal admin?15:47
dtantsurTheJulia, iirc yes15:48
TheJuliaexcellent15:48
TheJuliathank you15:48
* TheJulia raises eyebrow at BFV chatter15:48
milan|bblnext time we'll get a turing-complete language over queries ;) ?size=gt:815:49
mariojvTheJulia: was just asking for things that need reviews, jay recommended a couple specs15:49
*** milan|bbl is now known as milan15:49
TheJuliamariojv: Ahh, excellent!15:50
* milan feels like no adopter unless there's oslo support for parsing those funky bits15:54
openstackgerritAnnie Lezil proposed openstack/ironic: Adding Timing metrics for DRAC drivers.  https://review.openstack.org/37795715:55
sambettsmilan: oh dear...15:56
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Tests for OSC baremetal node create command with options  https://review.openstack.org/38235215:56
milansambetts, I'd postpone compatibility to APIv2 ;)15:57
JayFmat128: do you think we need a more generic spec for deploy steps before your spec?15:57
dtantsurmilan, I'd prefer ignoring operators for now15:57
*** sukhdev has joined #openstack-ironic15:57
JayFmat128: like I'm left with a lot of questions: can I define additional deploy steps in a hardwaremanager, like I can with clean steps? etc15:58
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add tests for OSC baremetal chassis create command  https://review.openstack.org/38249615:58
milandtantsur, but we do have time interval and state sets in the spec15:58
mat128JayF: I was hoping not, but can see it happening if needed15:58
mat128JayF: yes you will be able , I think I mentioned it15:58
mat128let me find it15:58
milanall of those are specified as multi-queries atm15:58
dtantsurugh...15:58
sambettsmilan, dtantsur: are you suggesting we do things like finished_at=gt:<time> ??15:58
mat128JayF: L21915:58
milansambetts, not me15:59
dtantsursambetts, I'm not suggesting, but the API guidelines do15:59
sambetts:/15:59
* milan facepalms15:59
dtantsurassuming that we care about what API-WG produces, of course15:59
*** Sukhdev_ has joined #openstack-ironic15:59
dtantsurif we don't, we just copy what Ironic is doing15:59
dtantsurwe have to be as compatible with either of these two as possible15:59
milanIronic, what are you doing? :D15:59
* milan digs in code16:00
*** jpich has quit IRC16:00
sambettsyeah... 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 cases16:01
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Migrate to keystone session glance and swift  https://review.openstack.org/35228916:01
* milan desperately needs a beer16:01
* sambetts adds a note to buy milan a beer in spain 16:01
* jlvillal pulls the trigger!!!16:02
milansambetts, you're my pal16:02
openstackgerritJohn L. Villalovos proposed openstack/ironic: Update iptables rules and services IPs for multinode  https://review.openstack.org/36864816:02
openstackgerritJohn L. Villalovos proposed openstack/ironic: Fix setting custom IRONIC_VM_NETWORK_BRIDGE  https://review.openstack.org/36511616:02
openstackgerritJohn L. Villalovos proposed openstack/ironic: Add devstack setup_vxlan_network()  https://review.openstack.org/36864716:02
openstackgerritJohn L. Villalovos proposed openstack/ironic: Skip some steps for multinode case  https://review.openstack.org/36864616:02
openstackgerritJohn L. Villalovos proposed openstack/ironic: Update ironic node names for multinode case  https://review.openstack.org/36864516:02
openstackgerritJohn L. Villalovos proposed openstack/ironic: Update devstack provision net config for multihost  https://review.openstack.org/36864416:02
openstackgerritJohn L. Villalovos proposed openstack/ironic: Source openrc on subnode in multinode topology  https://review.openstack.org/35330216:02
openstackgerritJohn L. Villalovos proposed openstack/ironic: Skip db configuration on subnodes  https://review.openstack.org/35330316:02
openstackgerritJohn L. Villalovos proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817316:02
openstackgerritJohn L. Villalovos proposed openstack/ironic: Ignore required_services for multinode topology  https://review.openstack.org/35279316:02
TheJuliawow16:02
sambettsits a tsunami run!!!16:02
* jlvillal has hopefully completed a successful rebase to update a var name change :)16:03
milan:D16:03
sambettsjlvillal: thats one long chain of patches :-P16:03
jlvillalYeah. The multi-node stuff16:03
*** rpioso has joined #openstack-ironic16:03
jlvillalWe 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|willbeb16:05
*** ccamacho|willbeb is now known as ccamacho|willbbc16:05
openstackgerritMerged openstack/bifrost: Fix a typo in documentation  https://review.openstack.org/38138316:05
*** rbudden has quit IRC16:05
*** ccamacho|willbbc is now known as ccamacho|wllbbck16:05
*** Sukhdev_ has quit IRC16:06
*** baoli has quit IRC16:06
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add tests for 'baremetal port create' command  https://review.openstack.org/33288016:06
jlvillalvsaienk0: I rebased the multi-node patches for the var name change in devstack-gate16:08
milansambetts, 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, sry16:11
* milan gone16:11
milangood night Ironic16:11
dtantsurg'night!16:11
TheJuliagoodnight milan16:12
dtantsurugh, 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 already16:12
JayFit can take a /real long time/ in a vm16:13
JayFlol16:13
* dtantsur applies his head to the table16:13
JayFI know the ssh driver is dying16:13
JayFbut if it wasn't, it'd be awesome to implement "out of band" erase devices on it16:13
JayFlike just deleting the disk image on disk or something, lol16:13
dtantsurlol16:14
dtantsurfile an RFE against virtualbmc16:14
dtantsur(although it will require a specialized driver)16:14
JayFyeah, which defeats the point of virtualbmc16:14
JayFmainly just a throwaway comment :P16:14
dtantsurbut yeah, this is good :) why wouldn't qemu implement ATA secure erase btw?16:15
*** milan has quit IRC16:15
*** vsaienko has joined #openstack-ironic16:15
JayFI have no idea, and don't know enough C to make it happen :P16:16
dtantsurI regret I was too lazy to rerun puppet on all conductors to disable the full erase... it would be some much faster :D16:16
*** vsaienko has quit IRC16:16
*** rajinir has joined #openstack-ironic16:18
dtantsurI guess I'll just leave it finish overnight. see you all tomorrow16:18
sambettscya dtantsur16:19
*** dtantsur is now known as dtantsur|afk16:19
JayFo/16:19
mat128dtantsur|afk, JayF: same kind of issue: SMART doesnt work on virtio16:20
mat128or bonding doesnt work on virtio-net because it has no link speed and duplex16:20
mat128fake problems :)16:21
*** syed_ has joined #openstack-ironic16:22
*** rbudden has joined #openstack-ironic16:23
*** Romanenko_K has quit IRC16:25
NobodyCamGood Morning Ironic'ers16:27
jlvillalHi NobodyCam16:27
JayFo/16:28
NobodyCammorning jlvillal JayF :)16:28
*** dougs1 has joined #openstack-ironic16:29
dougs1Anyone know offhand if Ironic supports Linux Bridges (instead of Open vSwitch) in the current release?16:30
JayFHow would linux bridges do anything for you? There's no hypervisor to bridge to in Ironic16:31
sambettsdougs1: in devstack no, but in production you can use what ever you want16:31
jlvillaldougs1: For devstack I think we only support OVS. But in a real bare-metal environment, what JayF said.16:31
dougs1ok, thanks guys16:32
JayFAnd if you're using Ironic outside of devstack with VMs instead of bare metal, please don't :P16:32
sambettsjlvillal, JayF: it can affect network node configuration16:32
sambettsneutron config16:32
JayFwhat do you mean?16:32
jlvillalsambetts: What's that mean?16:32
jlvillalJayF: Jinx!16:32
mat128JayF, dtantsur|afk: just checked for fun, qemu has an "ide_cmd_table" in hw/ide/core.c that wouldn't be too hard to extend16:32
JayFmat128: I can write just slightly more than hello world in qemu; but you could write the patch :P16:33
sambettsJayF, 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 one16:33
JayFmat128: just remember that standard secure erase = drive is zeros, enhanced secure erase = drive is random bits16:34
mat128I'm afraid my C is not cool enough for my feature to land in qemu, but who knows16:34
JayFsambetts: but the LB setup would be completely incompatible with ironic ... right?16:34
*** watanabe_isao has joined #openstack-ironic16:34
JayFor do I not understand?16:34
JayFmat128: It's easier to get patches into things than you think. I have code in systemd-nspawn, and I don't even know C :P16:34
mat128JayF: unless you run the linuxbridge plugin on your TOR :)16:34
JayF*blink*16:35
sambettsJayF: on the network/controller node, neutron hooks its services, e.g. DHCP into LB or OVS16:35
*** lucas-hungry is now known as lucasagomes16:35
sambettsJayF: thats how it can run isolated network services like DHCP or routers etc on one machine16:36
dougs1sambetts, agreed. I had heard rumours of better performance with Linux Bridges vs Open vSwitch16:36
*** watanabeisao has joined #openstack-ironic16:38
sambettsdougs1: I have also heard that, I'm not sure if its true16:39
*** watanabe_isao has quit IRC16:39
*** rbrud has quit IRC16:41
*** jaosorior has quit IRC16:42
openstackgerritLucas Alvares Gomes proposed openstack/ironic-lib: Correctly count the number of primary partitions  https://review.openstack.org/38189316:45
*** trown is now known as trown|lunch16:47
jlvillallucasagomes: Darn! I was too late by 4 minutes https://review.openstack.org/#/c/381893/3/ironic_lib/disk_utils.py  :)16:50
patchbotpatch 381893 - ironic-lib - Correctly count the number of primary partitions16:50
lucasagomesjlvillal, haha lemme see16:50
lucasagomesjlvillal, o the typo!? :-/16:50
jlvillallucasagomes: Yeah. Not a big deal. Not your typo.16:51
* jlvillal does a git blame to make sure ;)16:51
lucasagomesjlvillal, yeah, I can fix it. 1 sec16:51
* lucasagomes have the code open here16:51
jlvillallucasagomes: Thanks!16:51
openstackgerritLucas Alvares Gomes proposed openstack/ironic-lib: Correctly count the number of primary partitions  https://review.openstack.org/38189316:53
lucasagomesjlvillal, ^16:53
jlvillallucasagomes: Thanks!16:53
lucasagomesjlvillal, ty for reviewing16:53
jlvillallucasagomes: -116:54
* jlvillal is known for his dark cold heart...16:54
*** yonglihe has quit IRC16:55
*** lucasagomes is now known as lucas-brb16:55
*** derekh has quit IRC17:01
* jlvillal was late getting QA meeting started :(17:03
*** e0ne has quit IRC17:06
*** vsaienko has joined #openstack-ironic17:07
rajiniro/17:10
*** strigazi is now known as strigazi_AFK17:10
rajinirsorry I was late17:11
*** lucas-brb is now known as lucasagomes17:11
openstackgerritJohn L. Villalovos proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817317:11
* sambetts -> home 17:13
sambettsnight team! o/17:13
jlvillalsambetts: ciao17:13
jlvillalsambetts: Or toodle-loo :)17:14
*** sambetts is now known as sambetts|afk17:14
sambetts|afkjlvillal: haha17:14
sambetts|afkjlvillal: cya o/17:14
jlvillal:)17:14
openstackgerritMichael Turek proposed openstack/python-ironicclient: Add --wait to OSC provisioning commands  https://review.openstack.org/35480617:15
*** baoli has joined #openstack-ironic17:18
*** rbrud has joined #openstack-ironic17:19
watanabeisaosambetts|afk, oh afk..17:19
watanabeisaojlvillal, sorry sir. I just want to confirm.17:20
*** amotoki has quit IRC17:21
jlvillalwatanabeisao: Confirm what?17:21
watanabeisaoso 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
jlvillalwatanabeisao: Yeah. I think you need to talk to krtaylor. He is the one that knows all that. I do not.17:22
jlvillalwatanabeisao: I don't know the answer to your question.17:22
watanabeisaojlvillal, I see, thank you.17:22
jlvillalwatanabeisao: Sorry :(17:22
watanabeisaojlvillal, it's ok.:)17:23
*** MattMan has quit IRC17:23
*** MattMan has joined #openstack-ironic17:23
*** mgoddard has quit IRC17:23
*** vsaienko has quit IRC17:23
*** ccamacho|wllbbck is now known as ccamacho17:23
*** mgould is now known as mgould|afk17:25
*** watanabeisao has quit IRC17:27
openstackgerritLucas Alvares Gomes proposed openstack/ironic-lib: Correctly count the number of primary partitions  https://review.openstack.org/38189317:31
lucasagomesjlvillal, ^ thanks for the review. try...except added + unittest17:31
JayFlooking as well17:32
jlvillallucasagomes: Thanks. I wasn't sure but thought I would point it out.17:32
lucasagomesit was a good point! Thanks guys for looking at it17:32
JayFlucasagomes: curious, why with use_standard_locale=true17:33
JayFlucasagomes: we don't seem to be doing that on many other utils.execute() calls17:33
lucasagomesJayF, 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 case17:34
JayFoh, I get you17:34
JayFyou wanna make sure it's not translated17:34
JayFeven though that's unliked17:34
lucasagomesnot sure if some languages like japanese, korean etc might translate that17:34
lucasagomesyeah17:34
JayF*unlikely17:34
JayFmakes sense17:34
*** ohamada has quit IRC17:34
jlvillallucasagomes: Still reviewing but I posted a question.17:35
jlvillalJayF: I remember someone posting a patch adding the 'use_standard_locale' thing for some execute call because it failed in Japanese.17:35
lucasagomesjlvillal, I think it's a good to have as much information as we can in the messages17:36
lucasagomesthe string was there already, I can update it tho...17:36
jlvillalJayF: lucasagomes https://review.openstack.org/#/c/231367/17:36
patchbotpatch 231367 - ironic - Use standard locale when executing 'parted' command (MERGED)17:36
jlvillallucasagomes: Up to you. Thanks.17:38
jlvillallucasagomes: I was about to +2 it. Okay with you?17:41
lucasagomesjlvillal, I'm fixing it 1 sec17:41
lucasagomesbetter now that I've all the files opened17:41
jlvillallucasagomes: Okay thanks.17:41
JayFthat works, then john and I can land it17:41
JayFunless one of you think we should leave it up for more review17:41
JayFseems fairly straightforward to me tbh17:41
*** bjolo_ has joined #openstack-ironic17:41
openstackgerritLucas Alvares Gomes proposed openstack/ironic-lib: Correctly count the number of primary partitions  https://review.openstack.org/38189317:41
jlvillalagreed.17:41
lucasagomes^^17:41
JayFmat128: 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 need17:42
jlvillallucasagomes: Thanks. I hope it passes the tests :)17:43
jlvillallucasagomes: +217:43
JayF+A, I'll let jenkins do the job of deciding if it passes tests :D17:44
lucasagomesjlvillal, ty! Fingers crossed (jk, I ran it locally)17:44
lucasagomesJayF, thanks17:44
*** athomas has quit IRC17:45
*** karthiks has quit IRC17:46
openstackgerritJohn L. Villalovos proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817317:50
*** moshele has joined #openstack-ironic17:54
*** vsaienko has joined #openstack-ironic17:56
*** vsaienko has quit IRC17:56
openstackgerritMario Villaplana proposed openstack/ironic: Add power state change notifications  https://review.openstack.org/32186518:01
*** moshele has quit IRC18:04
*** mtanino has quit IRC18:04
*** trown|lunch is now known as trown18:04
*** yolanda has quit IRC18:08
*** vsaienko has joined #openstack-ironic18:12
jlvillallucasagomes: The commit message still says "count_MSDOS_partitions" ;)18:13
jlvillallucasagomes: Not critical18:13
*** yolanda has joined #openstack-ironic18:15
*** vsaienko has quit IRC18:16
openstackgerritMerged openstack/python-ironicclient: Add prefix "$" for command examples  https://review.openstack.org/37937118:19
lucasagomesjlvillal, sigh... yeah :-.18:19
lucasagomes:-/*18:19
jlvillallucasagomes: No worries :)18:20
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Update reno for stable/newton  https://review.openstack.org/37477218:22
*** moshele has joined #openstack-ironic18:22
*** e0ne has joined #openstack-ironic18:23
*** vsaienko has joined #openstack-ironic18:27
openstackgerritMerged openstack/ironic-lib: Correctly count the number of primary partitions  https://review.openstack.org/38189318:32
*** sukhdev has quit IRC18:34
openstackgerritJohn L. Villalovos proposed openstack/ironic-python-agent: Remove Python 2.6 format style  https://review.openstack.org/38261618:37
*** rpioso has quit IRC18:38
*** mkam has quit IRC18:45
openstackgerritJohn L. Villalovos proposed openstack/ironic-python-agent: Remove Python 2.6 format style  https://review.openstack.org/38261618:47
openstackgerritVasyl Saienko proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817318:49
*** e0ne has quit IRC18:50
*** Sukhdev has joined #openstack-ironic18:51
*** vsaienko has quit IRC18:52
Sukhdevjroll: 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-ironic18:58
*** vsaienko has joined #openstack-ironic19:01
* lucasagomes is getting a "Timeout error, please try again in a few minutes." when trying to post comments on bugs in launchpad19:01
*** ijw has quit IRC19:03
*** rpioso has joined #openstack-ironic19:04
lucasagomesSukhdev, 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
lucasagomesWe could add it there and if it's not the right doc we can later move it to the right one in another patch-set19:05
Sukhdevlucasagomes : Sounds good - thanks for the pointer. I will work on it and push the patch19:06
*** vsaienko has quit IRC19:07
lucasagomesyvw19:08
*** vsaienko has joined #openstack-ironic19:10
*** vsaienko has quit IRC19:14
*** moshele has quit IRC19:21
Sukhdevlucasagomes : I was browsing through and saw this - http://docs.openstack.org/developer/ironic/deploy/multitenancy.html#multitenancy19:26
Sukhdevlucasagomes : 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 you19:26
lucasagomesSukhdev, 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 me19:29
Sukhdevlucasagomes : right - I thought the same, hence, I thought I double check19:30
lucasagomes++19:31
Sukhdevlucasagomes : and. moreover, as you mentioned, if we need to move this around later, we can alway push another patch19:31
lucasagomesyeah, 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 IRC19:33
*** vsaienko has joined #openstack-ironic19:34
openstackgerritMerged openstack/ironic: Fix docstrings in tempest plugin baremetal json client  https://review.openstack.org/37777419:35
*** persia has quit IRC19:37
*** kirankv has quit IRC19:37
*** ishant|away has quit IRC19:37
*** lazy_prince has quit IRC19:37
openstackgerritMerged openstack/ironic: Keep numbering of list in Install Guide  https://review.openstack.org/38216919:37
*** persia has joined #openstack-ironic19:37
*** lazy_prince has joined #openstack-ironic19:39
*** ishant|away has joined #openstack-ironic19:39
*** kirankv has joined #openstack-ironic19:39
*** e0ne has quit IRC19:39
*** vsaienko has quit IRC19:44
*** sdake has quit IRC19:45
*** sdake has joined #openstack-ironic19:46
*** vsaienko has joined #openstack-ironic19:56
lucasagomesfolks, calling it a day20:01
lucasagomeshave a great evening all!20:01
JayFo/20:02
mariojvhave a good evening lucasagomes20:02
lucasagomes\o, see ya20:02
*** lucasagomes is now known as lucas-afk20:02
*** ijw has joined #openstack-ironic20:03
*** jpich has joined #openstack-ironic20:08
*** ijw has quit IRC20:10
*** vsaienko has quit IRC20:11
*** joprovost has quit IRC20:16
*** narasimha_SV has quit IRC20:18
*** vsaienko has joined #openstack-ironic20:23
*** vsaienko has quit IRC20:28
openstackgerritMerged openstack/ironic: Cleanup get_ilo_license()  https://review.openstack.org/38034120:34
openstackgerritMerged openstack/ironic: Documentation change for feature updates in iLO drivers.  https://review.openstack.org/37406020:34
*** penick has joined #openstack-ironic20:35
*** devananda is now known as devananda|afk20:43
*** vsaienko has joined #openstack-ironic20:47
*** trown is now known as trown|outtypewww20:48
*** xavierr is now known as xavierr|afk20:55
mrdaMorning Ironic20:55
jlvillalFYI: Our gate maybe broken on jobs which run on OSIC with devstack and neutron. Which sounds like us.20:56
jlvillalInfra 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
jlvillals/OSCI/OSIC/20:57
JayFjlvillal: AIUI it's only IPA20:57
JayFjlvillal: not all of ironic20:57
JayFjroll: ^ right/20:57
jlvillalJayF: That's better! :)20:57
jrollyeah, just IPA afaik20:58
jlvillalJayF: Shhh. Don't wake him...20:58
JayFI noted it on the whiteboard too20:58
JayFjroll: is there any action for us to take fixing that? Anything I can do?20:58
JayFjroll: the bug seems to be being worked on by $others20:58
jrollJayF: uhhhh. lemme see, I put up a patch to test a thing20:58
jrollit's really on neutron's plate20:58
JayFit failed20:59
JayFyour patch to test a thing20:59
jlvillalOh darn. That same bug...20:59
jrollI guess I clobbered it https://review.openstack.org/#/c/380351/220:59
patchbotpatch 380351 - ironic-python-agent - DNM: testing 38002320:59
jrolljust rechecked20:59
*** Goneri has quit IRC21:00
mariojvheading out, have a good evening ironic21:00
jlvillalmariojv: Good night21:01
*** ijw has joined #openstack-ironic21:07
openstackgerritJohn L. Villalovos proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817321:08
jlvillalWe made the YouTube channel :)  https://youtu.be/z6ftW7fUdp4?t=2m28s21:09
*** dprince has quit IRC21:10
jlvillalThe second half of the Newton Release Demo video is about Ironic :)21:10
*** ijw has quit IRC21:12
openstackgerritJohn L. Villalovos proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817321:22
*** Vinsh has joined #openstack-ironic21:26
VinshHello 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 dev21:26
Vinshuser 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
Vinshstatus: 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-ironic21:28
*** aNuposic has quit IRC21:31
*** vsaienko has quit IRC21:32
JayFVinsh: that's a warning, not an error21:33
*** jpich has quit IRC21:33
*** vsaienko has joined #openstack-ironic21:35
VinshJayF: Well :)  IPA just fails because it gets about 10 401's then gives up.21:36
*** jcoufal_ is now known as jcoufal21:36
VinshSo it can never get to the heartbeat phase and begin cleaning.21:36
openstackgerritJohn L. Villalovos proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817321:36
jrollmmm21:36
JayF401 unauthorized21:36
JayFhmmm....21:37
jrollI can't remember how DIB pulls IPA in21:37
jrollbut this commit fixes that problem, if I'm not mistaken https://github.com/openstack/ironic-python-agent/commit/fb1cbbdafc0b0c4d5141733b2eda03fe1f2aa87821:37
JayFoh, nice one jroll21:37
JayFbut that's a 406, not a 401, is it possible you misread teh rror Vinsh ?21:38
jrolloh you're right21:38
VinshNo, it's 401, that's a copy paste21:38
Vinshand this is it already giving up on that "new api" and failing back to "vendor passthrough"21:38
Vinshwhich just loops at 40121:38
JayFVinsh: 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 states21:38
JayFVinsh: so I'm wondering if you got gotta sync somehow21:38
*** aNuposic has joined #openstack-ironic21:39
JayFif you can gist a copy of your node object with any secrets removed21:39
JayFand of the ipa logs21:39
JayFI'll take a look21:39
VinshIts in state "available"21:39
*** bnemec has quit IRC21:39
JayFwhen a node is in available state, lookup is not permitted21:39
JayFbecause the agent is not supposed to be running on nodes when they are in available state21:39
Vinshoh sorry that was target state21:39
JayFso for security we do not permit the agent to perform a lookup in our default configuration21:39
Vinshits in "clean wait' now21:39
patchbotError: No closing quotation21:39
Vinshtrying to get to available21:40
JayFOK, gotcha. So it should work from there. Looking.21:40
JayFThe other thing that's weird is why you can't hit the new api endpoint21:40
jrollso it did work once though. huh.21:40
VinshI have gone so far as to wipe the database, re-add the node.. everything.. no dice today.21:40
JayFare you certain you're running stable/newton?21:40
VinshI'm on stable/mitaka now.  stable/newton acted the same.  I am certain as they are built in specific containers.21:41
Vinshbut ONE time yesterday, it fired right up and begain to clean fine21:41
JayFOK. It's completely expected that you will not have the new agent API on mitaka. So that piece is behaving as expected.21:41
Vinshafter that.. never again.21:41
Vinshstable/newton reported back the same way, failed back to vendor passthrough21:41
JayFand if you're running mitaka, it's not any of the security pieces we added around the agent lookup api21:42
JayFCan you please gist the full log from a fresh IPA boot, when it's failing?21:42
jrollI'd be curious to verify the SHA IPA is at, too21:42
VinshI'll get both items. one min21:43
Vinshdib-manifest-git-ironi-deploy claims ironic-python-agent bc1cf1388902dc9fada0f9cea49fbac63aa7d56221:44
jrollok cool, that's close enough to HEAD for me21:45
Vinshon a side note, this project is awesome, glad to finally be implementing it.21:45
JayFI agree, it is awesome :D21:45
VinshHope to contribute back alot.21:45
JayFlol21:46
jroll:)21:46
*** mtanino has joined #openstack-ironic21:46
VinshHere is the debug log http://paste.openstack.org/show/584569/ of an IPA run21:47
VinshOur 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
JayFI /really/ wish we logged the url we're trying to hit21:49
JayFwith those failure status code messages21:49
JayFVinsh: 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
VinshFrom the api log, you see a matching:21:50
Vinsh2016-10-05 21:11:13.942 44 INFO keystonemiddleware.auth_token [-] Rejecting request21:50
Vinsh2016-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.001630121:50
JayFokay, good21:50
JayFI think I have an idea what your problem is, gimme one sec21:50
jrollwaaaait21:50
jrollPOST 6385/21:50
jrollO_o21:50
JayFoh nice catch21:50
JayFVinsh: ^ ip:6385 somehow turned into ip/6385?21:50
Vinshoh yuck!21:51
jrollso let's verify with cat /proc/cmdline on the host running IPA21:51
jrolland then try to remember how that would get there :)21:51
Vinshironic_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:638521:52
jrollhuh, that looks right21:52
openstackgerritMichael Davies proposed openstack/ironic: Update ironic config docs for keystone v3  https://review.openstack.org/38210821:52
*** jheroux has quit IRC21:52
JayFVinsh: do most of your loglines have the 6385/ in front of it?21:53
JayFVinsh: or only the ones from IPA21:53
VinshJust the ones from IPA, ipa has to hit that control node at that IP.  that node has a few ips.21:54
Vinshthe 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
JayFVinsh: and you're running ironic-api on default port (6385), correct?21:55
VinshYeah, default21:55
VinshI can ironic node-show .. set states etc fine.21:55
JayFVinsh: I just can't seem to figure out why your IPA would be trying to prepend the url with the port in the path21:55
jrollnot 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_20721:55
jrollthe line below21:55
jrollGot exception calling lookup(): _lookup() takes at least 3 arguments (2 given) (('_lookup() takes at least 3 arguments (2 given)',))21:55
*** rbudden has quit IRC21:56
Vinshis there some other way to feed IPA a url.. to test? I wonder if I can edit cmdline21:58
JayFyou absolutely can, but I think there's something else going on21:58
JayFthere's nowhere in the code that port would be munged21:58
VinshOh. maybe it needs to start with http:// ?21:58
VinshI wonder if omiting that in the conifg causes parsing to be off by one or two21:59
JayFKernel 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_ramdisk21:59
jrollyeah, worth a try21:59
JayFcoreos.configdrive=021:59
JayFthat's from a gate run21:59
jrollit's the ipa-api-url21:59
JayFnice catch, I think you're exactly right21:59
JayFand if that's the case, we should file a bug to handle that more sanely22:00
VinshI can't edit cmdline.. will just edit the config and re-deploy to test22:00
JayFVinsh: you can pass the correct url on the actual ipa command line22:00
JayFVinsh: and it'll override kernel cli22:00
jrollVinsh: you can also pass --api-url to the... yeah22:00
Vinshah even better22:00
jrollrequests.exceptions.InvalidSchema: No connection adapters were found for '127.0.0.1:8080/foo'22:00
jrollseems like requests doesn't even try22:00
jrollweird22:01
JayFhttps://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/config.py#L2522:01
JayFthe helptext has http:// in it, so that's good22:01
*** penick has quit IRC22:01
Vinshit works!22:01
VinshDang. I spent a good 5 hours on that :)22:02
JayFit's always the silly stuff22:02
JayFDo you mind filing a bug about the lack of a good error message for that, aganist ironic-python-agent?22:02
VinshThanks so much for digging in on this.22:02
VinshI will file that bug now.22:02
JayFPlease tag it "low-hanging-fruit"22:02
JayFthat way someone new to contributing knows it's probably an easy fix22:02
JayFVinsh: now you gotta pass it on :) idle in chat and help the next person troubleshoot :D22:03
*** jcoufal has quit IRC22:03
VinshSo, to improve the logging in IPA that just says "401"22:03
JayFVinsh: operators very welcome in our dev room22:03
JayFVinsh: I'd just describe the problem, what the logs were, and how you fixed it22:03
VinshWill do, I'll keep the channel on auto-login now.22:03
jrollman this is so weird, idgi22:03
JayFVinsh: expected behavior is that it either works without a protocol or dies early, with a good error, demanding a protocol22:03
jrollI can reproduce though \o/22:03
JayFVinsh: I also expect if you go back to stable/newton, it'll find the new agent api too :D22:04
jrollalmost feels like a requests bug22:04
JayFyeah, I agree22:04
JayFwe don't parse that api string anywhere22:04
JayFI was looking for that the whole time22:04
Vinshwhen I run newton.. I never see a VIF from neutron added to the ironic-port22:04
VinshI don't think our liberty neutron plays well with it.22:04
*** penick has joined #openstack-ironic22:04
VinshNeed to get neutron up to mitaka or later I think.22:05
Vinshno clue though22:05
VinshI think there is stuff around ports and bare-metal non vm ports in neutron that newton ironic depends on22:06
JayFI /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 better22:06
*** vsaienko has quit IRC22:06
JayFone 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 support22:07
VinshI would MUCH rather be on newton also.  I'll look closer at the interface setting and retest.22:07
VinshOh 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
JayFthis is more about policies around restricting access to certain api endpoints22:08
JayFrather than those kind of neutron policies22:08
*** ijw has joined #openstack-ironic22:08
VinshMakes sense22:08
*** ijw has quit IRC22:13
openstackgerritJohn L. Villalovos proposed openstack/ironic: Skip create_ovs_taps() for multitenancy case  https://review.openstack.org/38236022:16
openstackgerritJohn L. Villalovos proposed openstack/ironic: Update iptables rules and services IPs for multinode  https://review.openstack.org/36864822:16
openstackgerritJohn L. Villalovos proposed openstack/ironic: Add devstack setup_vxlan_network()  https://review.openstack.org/36864722:16
openstackgerritJohn L. Villalovos proposed openstack/ironic: Skip some steps for multinode case  https://review.openstack.org/36864622:16
openstackgerritJohn L. Villalovos proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817322:16
VinshWhat is the difference in ironic_api_url and ipa-api-url ? they seem to be populated the same22:17
*** vishwanathj has quit IRC22:18
VinshThey seem to both come from api_url in ironic.conf22:18
JayFI honestly don't know22:19
jrollI think ironic_api_url is for the old ramdisk and needs to be deprecated, maybe22:19
JayFjroll: ^ support for legacy bash ramdisk?22:19
JayFthat can be completely removed?22:19
JayFwell it's already been deprecated if that's what it is22:19
jrolllooks like it https://github.com/openstack/diskimage-builder/blob/master/elements/deploy-ironic/init.d/80-deploy-ironic#L10022:19
JayFno match for ironic_api_url anywhere in IPA22:19
JayFand in ironic it's only in the pxe templates22:19
jrollsupport has been removed in ironic, but seems like we missed that piece22:19
JayFyeah, I'll push a patch real quick for that22:20
VinshVery good22:20
JayFwe offer the best service in all of open source22:20
JayFtroubleshoot your problem22:20
JayFand fix reported bugs, while you wait22:21
JayFlol22:21
Vinshoil change too??22:21
jrollyes but I'm obliged to take a joy ride22:21
Vinshfree dvd rewinding as well.22:21
Vinshsweet.22:21
JayFjroll: is troubleshoot=0 that one too?22:21
JayFjroll: that's not an ipa thing for sure22:21
JayFjroll: wondering about deployment_key as well...22:22
*** [1]cdearborn has quit IRC22:22
JayFthat's a general dib thing, not even just for our deploy ramdisk22:23
JayFhmm22:23
jrollJayF: that file is the entire source for the old ramdisk bash script22:23
JayFman, how the hell is this passing tests?22:23
JayFwe can't still be populating these...22:23
JayFI guess maybe tests were pulled too?22:24
*** wajdi has quit IRC22:24
JayFI'm going to bug this, because at a minimum it's bigger than I thought22:24
jrollI don't get what you mean22:24
jrollbut like, pxe template tests are weird22:24
JayFI'm saying this wants22:24
JayFpxe_options.deployment_id22:24
jrolland validate against their own thing22:24
JayFthere's no way we still populate that22:24
JayFif we removed the code supporting the old ramdisk22:24
JayFso I'm a little surprised the template build doesn't explode22:25
JayFI guess fail safe is better22:25
Vinshhttps://bugs.launchpad.net/ironic/+bug/163078522:25
openstackLaunchpad 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 backscroll22:25
jrollJayF: I think jinja fails safe22:25
jrollwill just be "" if the thing doesn't exist22:25
JayFyeah, and if you look at the kernel command line from test22:25
JayFthey're blank22:25
JayFthat I pasted a page or two above22:25
jrollmhm22:26
jrollI don't get what you're trying to say, sorry :/22:26
JayFI'm saying there's more than just ironic_api_url= in that command line22:27
JayFthat needs removal22:27
JayFa lot more22:27
jrolloh, totally22:27
JayFadded ipa to https://bugs.launchpad.net/ironic/+bug/1630785 along with smoe notes to go with our troubleshooting, and triaged it at low importance22:29
openstackLaunchpad 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 IRC22:33
* Vinsh headed home from Broomfield CO, to Boulder CO. back on later.22:34
*** Vinsh has quit IRC22:37
*** awiddersheim has joined #openstack-ironic22:39
*** hamzy has quit IRC22:42
*** rbrud has quit IRC22:55
JayFjroll: vinsh: filed that bug about deprecated keys still being sent to the kernel cli: https://bugs.launchpad.net/ironic/+bug/163079923:07
openstackLaunchpad bug 1630799 in Ironic "Deprecated bash ramdisk options still exist in pxe templates" [Undecided,New]23:07
JayFaNuposic: vgadiraj: aslezil ^ this is a nice, simple bug if one of you were looking for something23:07
aNuposicJayF: thanks i will take a look at it23:09
*** ijw has joined #openstack-ironic23:10
*** penick has quit IRC23:15
*** ijw has quit IRC23:15
*** Sukhdev has quit IRC23:17
*** Sukhdev has joined #openstack-ironic23:19
*** izaakk has quit IRC23:28
*** logan- has quit IRC23:28
*** izaakk has joined #openstack-ironic23:33
*** logan- has joined #openstack-ironic23:37
*** hamzy has joined #openstack-ironic23:44
*** sdake has quit IRC23:48
*** aNuposic has quit IRC23:54

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