Friday, 2016-11-18

*** ijw has joined #openstack-ironic00:00
*** syed__ has quit IRC00:05
*** penick has joined #openstack-ironic00:15
*** penick_ has joined #openstack-ironic00:18
*** penick has quit IRC00:19
*** penick_ is now known as penick00:19
*** causten has joined #openstack-ironic00:21
*** ijw has quit IRC00:23
*** ijw has joined #openstack-ironic00:25
*** godp1301 has joined #openstack-ironic00:28
*** rloo has quit IRC00:30
*** godp1301 has quit IRC00:33
*** tuanluong has joined #openstack-ironic00:35
*** hoangcx has joined #openstack-ironic00:54
*** phuongnh has joined #openstack-ironic00:56
*** SerenaFeng has joined #openstack-ironic00:57
*** ijw has quit IRC00:58
*** ijw has joined #openstack-ironic01:01
*** ijw has quit IRC01:06
*** chlong has quit IRC01:11
*** ijw has joined #openstack-ironic01:12
*** ijw has quit IRC01:14
*** penick has quit IRC01:17
openstackgerritNaohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off  https://review.openstack.org/21673001:19
openstackgerritNaohiro Tamura proposed openstack/ironic: iRMC power driver for soft reboot and soft power off  https://review.openstack.org/21674301:21
*** SerenaFeng has quit IRC01:27
*** zhangjl has joined #openstack-ironic01:32
*** SerenaFeng has joined #openstack-ironic01:33
*** ijw has joined #openstack-ironic01:35
*** rpioso has quit IRC01:36
*** ijw has quit IRC01:40
*** ijw has joined #openstack-ironic01:46
openstackgerritNaohiro Tamura proposed openstack/ironic: Generic management I/F for Inject NMI  https://review.openstack.org/34819101:47
*** zhugaoxiao has quit IRC01:47
*** zhangyufei has joined #openstack-ironic01:48
openstackgerritNaohiro Tamura proposed openstack/ironic: iRMC management driver for Inject NMI  https://review.openstack.org/34873201:49
*** ijw has quit IRC01:52
*** aNuposic has quit IRC01:55
*** ijw has joined #openstack-ironic02:02
*** Sukhdev has quit IRC02:06
*** causten has quit IRC02:09
*** causten has joined #openstack-ironic02:13
*** zhangyufei has quit IRC02:19
*** zhangyufei has joined #openstack-ironic02:27
*** fragatina has quit IRC03:03
*** fragatina has joined #openstack-ironic03:05
*** causten has quit IRC03:09
*** fragatin_ has joined #openstack-ironic03:09
*** fragatina has quit IRC03:09
*** causten has joined #openstack-ironic03:10
*** tuanluong has quit IRC03:10
*** causten_ has joined #openstack-ironic03:11
*** fragatin_ has quit IRC03:14
*** Nisha has joined #openstack-ironic03:15
*** causten has quit IRC03:15
*** Nisha is now known as Guest2117903:15
*** tuanluong has joined #openstack-ironic03:15
*** causten_ has quit IRC03:15
*** Goneri has joined #openstack-ironic03:17
*** zhangyufei has quit IRC03:18
*** awiddersheim has joined #openstack-ironic03:31
*** ijw has quit IRC03:32
*** ijw has joined #openstack-ironic03:35
*** bks has joined #openstack-ironic03:39
*** ijw has quit IRC03:39
*** SerenaFeng has quit IRC03:55
*** godp1301 has joined #openstack-ironic03:56
*** g3ek has quit IRC03:59
*** ijw has joined #openstack-ironic04:05
*** godp1301 has quit IRC04:19
*** aparnav has joined #openstack-ironic04:20
*** baoli has joined #openstack-ironic04:22
*** Goneri has quit IRC04:27
*** baoli has quit IRC04:31
*** SerenaFeng has joined #openstack-ironic04:31
*** SerenaFeng has quit IRC04:42
*** zhangjl has quit IRC05:01
*** vishwanathj has joined #openstack-ironic05:05
*** vmud213 has joined #openstack-ironic05:07
*** zhangjl has joined #openstack-ironic05:09
openstackgerritNisha Agarwal proposed openstack/ironic: SNMP agent support for OOB inspection for iLO Drivers  https://review.openstack.org/37242805:14
*** SerenaFeng has joined #openstack-ironic05:36
openstackgerritAparna proposed openstack/proliantutils: Add Sanitize disk erase support using SSA  https://review.openstack.org/39820205:54
*** zhangyufei has joined #openstack-ironic05:59
*** Sukhdev has joined #openstack-ironic06:00
*** g3ek has joined #openstack-ironic06:05
*** rama_y has quit IRC06:13
*** g3ek has quit IRC06:14
*** e0ne has joined #openstack-ironic06:16
*** ijw has quit IRC06:18
*** mgagne has quit IRC06:18
*** mgagne has joined #openstack-ironic06:21
*** mgagne is now known as Guest5228506:21
*** g3ek has joined #openstack-ironic06:26
*** zhangyufei has quit IRC06:29
*** sabeen has quit IRC06:35
*** pc_m has quit IRC06:38
*** zhangyufei has joined #openstack-ironic06:40
*** shuquan has joined #openstack-ironic06:40
tuanluongMorning Ironic06:41
*** mjura has joined #openstack-ironic06:53
*** fxpester has joined #openstack-ironic06:59
*** mjura has quit IRC07:00
*** mjura has joined #openstack-ironic07:00
*** Sukhdev has quit IRC07:03
*** ChubYann has quit IRC07:04
openstackgerritNisha Agarwal proposed openstack/ironic: SNMP agent support for OOB inspection for iLO Drivers  https://review.openstack.org/37242807:08
*** causten has joined #openstack-ironic07:13
*** zhangyufei has quit IRC07:15
*** causten has quit IRC07:17
*** tesseract has joined #openstack-ironic07:18
*** tesseract is now known as Guest9031307:18
*** zhangyufei has joined #openstack-ironic07:21
*** ijw has joined #openstack-ironic07:21
*** shuquan_ has joined #openstack-ironic07:22
*** ijw has quit IRC07:22
*** shuquan has quit IRC07:23
openstackgerritMoshe Levi proposed openstack/ironic-python-agent: Add a new Hardware Manager for Mellanox NICs  https://review.openstack.org/33481607:28
*** jaosorior has joined #openstack-ironic07:39
*** SerenaFeng has quit IRC07:44
*** pcaruana has joined #openstack-ironic07:45
*** vsaienko has joined #openstack-ironic07:45
*** e0ne has quit IRC07:48
*** lukl has quit IRC07:49
*** Guest21179 has quit IRC07:52
*** Nisha has joined #openstack-ironic07:54
*** Nisha is now known as Guest9010107:55
*** phuongnh has quit IRC07:56
*** phuongnh has joined #openstack-ironic07:57
*** _milan_ has joined #openstack-ironic07:59
*** shuquan_ has quit IRC07:59
*** tuanluong has quit IRC08:01
*** openstackgerrit has quit IRC08:03
*** vishwana_ has joined #openstack-ironic08:03
*** openstackgerrit has joined #openstack-ironic08:03
*** vishwanathj has quit IRC08:05
*** SerenaFeng has joined #openstack-ironic08:07
*** pc_m has joined #openstack-ironic08:09
*** shuquan has joined #openstack-ironic08:09
*** hogepodge has quit IRC08:15
*** shuquan has quit IRC08:19
*** ralonsoh has joined #openstack-ironic08:21
openstackgerritMerged openstack/bifrost: Create new fact for dib_env_vars  https://review.openstack.org/39764908:28
*** zhangyufei has quit IRC08:32
*** shuquan has joined #openstack-ironic08:35
*** zhangyufei has joined #openstack-ironic08:35
openstackgerritVasyl Saienko proposed openstack/ironic: Remove deprecated Neutron DHCP provider methods  https://review.openstack.org/39886108:39
*** betherly has joined #openstack-ironic08:40
*** shuquan_ has joined #openstack-ironic08:40
*** zhangyufei has quit IRC08:41
*** shuquan has quit IRC08:41
*** lifeless has quit IRC08:45
*** pmannidi has quit IRC08:45
*** aparnav has quit IRC08:46
*** ishant|away has quit IRC08:46
*** lifeless has joined #openstack-ironic08:47
*** lazy_prince has quit IRC08:47
*** vmud213 has quit IRC08:49
*** kirankv has quit IRC08:49
*** zhangyufei has joined #openstack-ironic08:50
*** vmud213 has joined #openstack-ironic08:51
*** ishant|away has joined #openstack-ironic08:51
*** kirankv has joined #openstack-ironic08:51
*** lazy_prince has joined #openstack-ironic08:52
*** jpich has joined #openstack-ironic08:55
*** pmannidi has joined #openstack-ironic08:58
*** hoangcx has quit IRC08:59
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-ironic09:00
*** zhangyufei has quit IRC09:00
*** phuongnh has quit IRC09:06
*** moshele has joined #openstack-ironic09:07
openstackgerritqinchunhua proposed openstack/python-ironicclient: Use uuidutils instead of uuid.uuid4()  https://review.openstack.org/39945309:09
*** derekh has joined #openstack-ironic09:14
*** zhangyufei has joined #openstack-ironic09:15
*** zhangyufei has quit IRC09:15
*** zhangyufei has joined #openstack-ironic09:15
*** strigazi_AFK is now known as strigazi09:15
*** shuquan_ has quit IRC09:18
*** shuquan has joined #openstack-ironic09:19
*** Guest90101 has quit IRC09:22
*** ijw has joined #openstack-ironic09:24
*** ijw has quit IRC09:25
*** ijw has joined #openstack-ironic09:26
*** hshiina has joined #openstack-ironic09:26
openstackgerritafazekas proposed openstack/ironic-inspector: Remove the generic eventlet monkey patch from test  https://review.openstack.org/39946909:27
*** zhangyufei has quit IRC09:29
*** moshele has quit IRC09:30
*** lucas-afk is now known as lucasagomes09:31
lucasagomesmorning all09:31
*** vsaienko has quit IRC09:31
*** ijw has quit IRC09:31
_milan_morning Ironic! :)09:32
_milan_morning lucasagomes! :)09:32
lucasagomeso/09:32
openstackgerritafazekas proposed openstack/ironic-inspector: Remove the generic eventlet monkey patch from test  https://review.openstack.org/39946909:38
*** aarefiev has quit IRC09:39
*** aarefiev has joined #openstack-ironic09:41
vdrokgood morning lucasagomes and _milan_ !09:41
aarefievmorning!09:42
vdrokmorning aarefiev09:42
vdrok_milan_: nothing was decided yesterday on the api-wg meeting, seems that they are ok with both not_in and nin :)09:43
*** shuquan has quit IRC09:44
vdrokah, _milan_ https://review.openstack.org/#/c/399131/09:44
patchbotpatch 399131 - api-wg - Add the operator for "not in" to the filter guideline09:44
openstackgerritYushiro FURUKAWA proposed openstack/ironic: Align cleanning behaivor in maintaenace b/w auto and manual  https://review.openstack.org/29909709:45
aarefievmorning vdrok, _milan_09:45
*** shuquan has joined #openstack-ironic09:46
*** afazekas has joined #openstack-ironic09:47
*** e0ne has joined #openstack-ironic09:47
* afazekas looking for reviewers for: https://review.openstack.org/#/c/399469/09:48
patchbotpatch 399469 - ironic-inspector - Remove the generic eventlet monkey patch from test09:48
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Test grenade  https://review.openstack.org/39947809:50
aarefievafazekas: I'm waiting for results of ^09:51
*** AlexeyAbashkin has joined #openstack-ironic09:53
_milan_morning aarefiev, vdrok! :)09:54
_milan_vdrok, I see :-/09:55
_milan_I sort of expected this :D09:55
vdrok_milan_ the patch i referenced adds nin, so you don't even have to propose ;)09:56
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Fix PXE setup for fresh Ubuntu Xenial  https://review.openstack.org/39948509:56
* _milan_ +109:56
_milan_vdrok, thanks for the link09:56
*** dtantsur|afk is now known as dtantsur09:56
dtantsurMorning Ironic09:56
vdrokgood morning dtantsur09:56
_milan_morning dtantsur! :)09:56
aarefievmorning dtantsur09:57
_milan_TGIF? ;)09:57
*** aparnav has joined #openstack-ironic09:57
*** jtomasek has joined #openstack-ironic09:57
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Fix PXE setup for fresh Ubuntu Xenial  https://review.openstack.org/39948509:58
dtantsurTGIF!09:58
_milan_:)09:58
dtantsur_milan_, aarefiev, looks like afazekas has tracked down the issue with us and tempest: https://review.openstack.org/#/c/399469/09:58
patchbotpatch 399469 - ironic-inspector - Remove the generic eventlet monkey patch from test09:58
dtantsurplease review09:58
* dtantsur wishes we used asyncio instead of eventlet...09:59
_milan_afazekas, thanks!09:59
aarefievon way  https://review.openstack.org/39947809:59
patchbotpatch 399478 - ironic-inspector - Test grenade09:59
* _milan_ check09:59
dtantsuraarefiev, iirc depends-on does not work within one repo.. maybe I'm wrong09:59
*** amotoki has joined #openstack-ironic10:00
aarefievlets see, I thought it doesn't too, but someone says it does :)10:00
aarefievI think about running all tests on grenade again10:00
dtantsur++ if it fixes neutron tests, I'm all for it10:01
openstackgerritVladyslav Drok proposed openstack/ironic: Update the alembic migration section in the developer FAQ  https://review.openstack.org/39848610:02
_milan_Oops I might have a+ the previous patch10:05
_milan_aarefiev, did you want to first check with ^?10:06
aarefiev_milan_: doesn't matter, I don't like monkey patch there10:08
*** SerenaFeng has quit IRC10:08
openstackgerritMerged openstack/ironic-inspector: Remove the generic eventlet monkey patch from test  https://review.openstack.org/39946910:09
* _milan_ anticipates a wave of raised eyebrows /o\10:09
* dtantsur backports10:10
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Remove the generic eventlet monkey patch from test  https://review.openstack.org/39949110:11
dtantsurI'll hold the backport until we confirm our gate still works ^^^10:11
dtantsuraarefiev, wanna try removing the network_basic_ops workaround?10:11
*** shuquan has quit IRC10:11
aarefievdtantsur: sure10:12
*** shuquan has joined #openstack-ironic10:13
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Fix PXE setup for fresh Ubuntu Xenial  https://review.openstack.org/39948510:13
pas-hamorning all10:13
pas-hahow can one re-open a "Fix released" bug? https://bugs.launchpad.net/ironic/+bug/163894010:14
openstackLaunchpad bug 1638940 in Ironic "DevStack: Standard PXE setup broken: pxelinux.0 not found" [Medium,Fix released] - Assigned to Lucas Alvares Gomes (lucasagomes)10:14
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Add API for listing all introspection statuses  https://review.openstack.org/34492110:14
lucasagomespas-ha, I think you can just change the status of it. Or, open a new ticket10:15
pas-hatried changing status - can not (but may be I lack some privelege/group on LP for that)10:15
lucasagomespas-ha, urgh so pxelinux package doesn't exist on trusty ?10:16
pas-hayep10:16
*** AlexeyAbashkin has quit IRC10:16
pas-hathus code does look ugly I admit10:16
lucasagomes>.<10:17
*** sharatss has joined #openstack-ironic10:17
_milan_dtantsur, 2016-11-18 10:12:42.724853 | ERROR: InterpreterNotFound: python3.410:17
*** sharatss has left #openstack-ironic10:17
pas-haalthough my patch above still does not fix the thing, will investigate today10:17
* _milan_ +210:17
dtantsurpas-ha, marked as in progress10:18
lucasagomespas-ha, gotcha, at least now we have a gate job testing pxe linux10:18
lucasagomes(it was merged yesterday)10:18
lucasagomespxelinux.0*10:18
pas-hadtantsur: thanks10:19
pas-halucasagomes: yep, we still need to test it if we support it :)10:19
_milan_aarefiev, mind a+ https://review.openstack.org/#/c/399491 ?10:19
patchbotpatch 399491 - ironic-inspector (stable/newton) - Remove the generic eventlet monkey patch from test10:19
*** SerenaFeng has joined #openstack-ironic10:19
* _milan_ doesn't have a+ on stable10:19
lucasagomespas-ha, ++10:19
dtantsur_milan_, please hold until we confirm our CI still works10:20
* dtantsur suspects only the release team has +A on stable10:20
_milan_dtantsur, OK10:20
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Test grenade  https://review.openstack.org/39947810:22
*** shuquan has quit IRC10:22
aarefievyeah, looks like depends-on catch up that patch10:22
*** ishant|away has quit IRC10:23
*** shuquan has joined #openstack-ironic10:23
* pas-ha wonders how he can check the output of failed job while others (grenade...) is still running10:23
*** ishant|away has joined #openstack-ironic10:24
_milan_aarefiev, dtantsur what about this one: https://review.openstack.org/#/c/399123/ ?10:25
patchbotpatch 399123 - ironic-inspector - Fixed incorrect string type checking10:25
_milan_got stuck on merge :-/10:25
dtantsurpas-ha, check zuul10:26
pas-hadtantsur: well, it only gives a telnet link, not sure how I can scroll back in that..10:27
aarefiev_milan_: tempest job fails on clean up10:27
dtantsurpas-ha, this is for running jobs, for finished ones there'll be a link to logs10:27
*** shuquan has quit IRC10:28
_milan_aarefiev, saw the same w/ my list-api-statuses patch... but posted new revision there w/ updated commit message10:28
pas-hadtantsur: nope, still a telnet link for already failed job10:29
dtantsurstrange..10:29
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification  https://review.openstack.org/34843710:29
*** fragatina has joined #openstack-ironic10:35
*** fragatina has quit IRC10:36
*** fragatina has joined #openstack-ironic10:37
_milan_aarefiev, so what is the procedure? just recheck? (it already had jenkins +1 on it before)10:38
*** zhangjl has left #openstack-ironic10:38
*** hshiina has quit IRC10:40
*** mgould|afk is now known as mgould10:46
openstackgerritGalyna Zholtkevych proposed openstack/ironic-specs: API Evolution - ETAG identifiers  https://review.openstack.org/38199110:51
mgouldmorning Ironic!10:52
jrolldtantsur: pas-ha: when infra moved off of jenkins to zuul only, we only ended up with telnet, but I think they have or are making a way to get the logs before you connected10:54
jrollmorning everyone10:54
dtantsurmorning jroll10:54
mgouldmorning jroll aarefiev dtantsur _milan_ pas-ha lucasagomes10:54
aarefiev_milan_: let me take a look on grenade also10:55
aarefievmorning jroll, mgould10:55
*** moshele has joined #openstack-ironic10:57
aarefievdtantsur: what happens with tempest on newton10:58
aarefievjob I mean10:59
dtantsuraarefiev, mmm? what exactly do you mean?10:59
*** moshele has quit IRC10:59
aarefievdoes it broken?11:00
vdrokgood morning pas-ha mgould and jroll11:01
*** vsaienko has joined #openstack-ironic11:02
*** SerenaFeng has quit IRC11:03
dtantsuraarefiev, do you have a failed job at hand? I'm not aware of any breakages11:03
aarefievdtantsur: ok, thanks, I'll check11:04
*** jaosorior is now known as jaosorior_lunch11:04
sergek_morning pas-ha mgould jroll aarefiev vdrok11:05
*** moshele has joined #openstack-ironic11:05
*** vsaienko has quit IRC11:06
*** vsaienko has joined #openstack-ironic11:06
aarefievmorning sergek_11:06
*** sergek_ is now known as sergek11:08
vdrokmorning sergek11:10
*** jkilpatr has quit IRC11:11
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add ironic resources CRUD notifications  https://review.openstack.org/35654111:13
*** jtomasek has quit IRC11:13
*** bks has quit IRC11:15
*** vsaienko has quit IRC11:20
pas-hamorning jroll mgould11:21
openstackgerritJim Rollenhagen proposed openstack/ironic-tempest-plugin: Initial commit from cookiecutter  https://review.openstack.org/39953211:22
jroll\o/11:22
jrollmorning dtantsur mgould aarefiev pas-ha vdrok sergek :)11:22
sergek\o/11:23
lucasagomeso/11:23
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic-specs: Support rolling upgrades  https://review.openstack.org/29924511:28
openstackgerritJim Rollenhagen proposed openstack/ironic-tempest-plugin: Initial commit from cookiecutter  https://review.openstack.org/39953211:31
openstackgerritJim Rollenhagen proposed openstack/ironic-inspector-tempest-plugin: Initial commit from cookiecutter  https://review.openstack.org/39953711:32
jrollthose should be ready to go, I believe11:34
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add node maintenance notifications  https://review.openstack.org/39623911:34
* jroll looks over inspector plugin again real quick11:35
*** khamtamtun has joined #openstack-ironic11:35
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Update to newer hacking  https://review.openstack.org/39954611:37
*** amotoki_ has joined #openstack-ironic11:39
dtantsurhuh, have we broken inspector CI?11:40
jrolldtantsur: we did for a bit yesterday, should be okay now11:40
jrollunless there's new problems11:40
dtantsurI'm afraid we could break it again11:40
jrollthe inspecotr devstack plugin was loading before the ironic plugin11:40
*** jtomasek has joined #openstack-ironic11:41
aarefievdtantsur: it's not about inspector only11:41
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Rework handling requirements for functional tests  https://review.openstack.org/39955111:41
jrolloh no11:41
jrollplease don't do this to me friday11:41
aarefiev:)11:41
*** amotoki has quit IRC11:42
aarefievI see Failed to establish a new connection11:42
dtantsur\o/11:42
aarefievhappy Friday :)11:43
openstackgerritJim Rollenhagen proposed openstack/ironic-tempest-plugin: Initial commit from cookiecutter  https://review.openstack.org/39953211:43
openstackgerritJim Rollenhagen proposed openstack/ironic-inspector-tempest-plugin: Initial commit from cookiecutter  https://review.openstack.org/39953711:43
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add node console notifications  https://review.openstack.org/39781211:44
*** jaosorior_lunch is now known as jaosorior11:44
jrollaarefiev: dtantsur: got a link?11:47
aarefievhttps://review.openstack.org/#/c/399485/11:48
*** Guest90101 has joined #openstack-ironic11:48
patchbotpatch 399485 - ironic - Fix PXE setup for fresh Ubuntu Xenial11:48
aarefievall last runs on ironic or inspector :)11:48
dtantsurok, I'm approving https://review.openstack.org/#/c/399491/ as it does not seem to break things11:50
patchbotpatch 399491 - ironic-inspector (stable/newton) - Remove the generic eventlet monkey patch from test11:50
aarefiev+111:50
* dtantsur will update his release request with this change11:51
openstackgerritMerged openstack/ironic-inspector: Remove the generic eventlet monkey patch from test  https://review.openstack.org/39949111:55
*** Guest90101 has quit IRC11:59
*** vsaienko has joined #openstack-ironic12:01
*** amotoki_ has quit IRC12:02
ricardoasmorning!12:07
*** amotoki has joined #openstack-ironic12:11
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Fix PXE setup for fresh Ubuntu Xenial  https://review.openstack.org/39948512:12
*** khamtamtun has quit IRC12:12
lucasagomesdtantsur, hmm yeah it might be broken: http://logs.openstack.org/37/348437/25/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial/8014958/logs/screen-ir-cond.txt.gz#_2016-11-18_11_16_35_76112:13
*** amotoki has quit IRC12:15
dtantsurOo12:16
*** jkilpatr has joined #openstack-ironic12:22
*** aparnav has quit IRC12:24
*** ccamacho is now known as ccamacho|lunch12:25
*** hogepodge has joined #openstack-ironic12:31
_milan_morning jroll mgould pas-ha sergek! :)12:35
_milan_aarefiev, ack,12:36
* _milan_ checks patches12:37
*** lucasagomes is now known as lucas-hungry12:37
*** dtantsur is now known as dtantsur|brb12:39
*** gcb has quit IRC12:40
*** yufei has joined #openstack-ironic12:41
*** vsaienko has quit IRC12:52
*** vsaienko has joined #openstack-ironic12:53
openstackgerritMoshe Levi proposed openstack/ironic-inspector: Fix passing incorrect args to logging  https://review.openstack.org/39958412:58
mosheledtantsur: ^ trivial fix12:59
mosheledtantsur: ^ trivial fix12:59
*** yufei has quit IRC13:02
*** jkilpatr has quit IRC13:03
*** vmud213 has quit IRC13:06
moshelesambetts_:  hi, in the ironic-inspctor local_link_connection plugin it don't parse the system name ? I thought it is the switch_info http://docs.openstack.org/developer/ironic/deploy/multitenancy.html can I add it the the plugin?13:11
_milan_moshele, thx for the patch13:13
*** yufei has joined #openstack-ironic13:13
openstackgerritoctopuszhang proposed openstack/ironic-inspector: Fix passing incorrect args to logging  https://review.openstack.org/39958413:13
moshele_milan_: no, saw the issue will I was testing the inspector with lldp :)13:13
_milan_moshele, ack :)13:14
*** jkilpatr has joined #openstack-ironic13:14
*** trown|outtypewww is now known as trown13:15
yufeihi, moshele , I just modify your commit message just now for two trailing e. Sorry for break the +2 :O13:17
mosheleyufei: np :)13:17
_milan_although the gate is broken :-/13:20
TheJuliaGood morning13:25
_milan_morning TheJulia! :)13:25
*** vsaienko has quit IRC13:26
vdrokgood morning TheJulia13:29
*** lucas-hungry is now known as lucasagomes13:31
*** jkilpatr has quit IRC13:31
_milan_lucasagomes, so what is actually failing in the gate? IPA heartbeat?13:32
lucasagomes_milan_, just came back from lunch, lemme take a quick look at the gate13:33
_milan_lucasagomes, sure, thanks!13:34
lucasagomes_milan_, yeah it's def broken :-/13:34
lucasagomesis someone already looking into it ?13:34
*** ccamacho|lunch is now known as ccamacho13:36
_milan_I've just checked couple of commits all seems to suffer the same error in inspector tempest test13:36
_milan_aarefiev was looking too13:36
* _milan_ dunno how to check gate IPA logs13:36
_milan_lucasagomes, ^ hints?13:37
jroll_milan_: ironic-bm-logs/13:37
openstackgerritJulia Kreger proposed openstack/ironic: [WIP] Add Cinder storage driver  https://review.openstack.org/36619713:37
jrollsorry I'm here but not really here yet :/13:37
_milan_jroll, ack, thx13:38
*** godp1301 has joined #openstack-ironic13:40
*** mjura has quit IRC13:40
TheJuliaInteresting....13:40
TheJuliaI'm kind of wondering if the virsh instance is getting powered off or something.  The logs don't show it up for that long.  That or there is a firewall issue reaching the VM13:42
*** jaosorior has quit IRC13:42
TheJuliathe first heartbeat is received, the failure is when the conductor goes to call back13:44
TheJuliahttp://logs.openstack.org/41/356541/20/check/gate-tempest-dsvm-ironic-ipa-partition-agent_ipmitool-tinyipa-ubuntu-xenial/8e72a8c/logs/screen-ir-cond.txt.gz#_2016-11-18_11_35_34_87713:44
* _milan_ opens the same logs13:45
TheJuliaNothing in the syslog indicating dropped packets13:47
_milan_TheJulia, it's like an abrupt end of the VM isn't it?13:47
TheJuliaat least, back to the VM13:47
TheJuliayeah, conductor should be failing the deploy I think13:47
TheJuliaand powering off the VM13:47
_milan_but instead it says it successfully powered off the node. maybe I'm wrong.13:48
*** dtantsur|brb is now known as dtantsur13:48
*** baoli has joined #openstack-ironic13:48
*** baoli has quit IRC13:48
TheJuliayeah, networking is borked13:50
*** dmellado is now known as dmellado|lunch13:50
*** sabeen has joined #openstack-ironic13:51
TheJuliavm boots with a 10.0.0.x address. Default kernel networking namespace:13:52
TheJuliadefault via 10.0.0.1 dev ens3 onlink13:52
TheJulia10.0.0.0/8 dev ens3  proto kernel  scope link  src 10.7.181.12913:52
patchbotError: 'supybot.via' is not a valid configuration variable.13:52
*** vsaienko has joined #openstack-ironic13:52
TheJuliaoh Patchbot....13:52
TheJuliabut, namespace for neutron is the same address space13:53
*** baoli has joined #openstack-ironic13:53
TheJuliahttps://www.irccloud.com/pastebin/5giQ29VS/13:53
*** e0ne_ has joined #openstack-ironic13:53
*** baoli_ has joined #openstack-ironic13:53
TheJuliatl;dr, unless ironic is operating in the namespace, it can't know how to reply, it will attempt out ens3, it will fail because the arp fails, and *boom*13:54
TheJuliaWasn't this an issue like 3 weeks ago?13:54
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal chassis commands  https://review.openstack.org/36615813:55
*** causten has joined #openstack-ironic13:55
*** e0ne has quit IRC13:56
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal port commands  https://review.openstack.org/36569213:56
*** krtaylor has quit IRC13:57
*** baoli has quit IRC13:57
TheJulia_milan_: http://logs.openstack.org/41/356541/20/check/gate-tempest-dsvm-ironic-ipa-partition-agent_ipmitool-tinyipa-ubuntu-xenial/8e72a8c/logs/worlddump-2016-11-18-114527.txt.gz13:59
_milan_TheJulia, I'm wondering http://logs.openstack.org/41/356541/20/check/gate-tempest-dsvm-ironic-ipa-partition-agent_ipmitool-tinyipa-ubuntu-xenial/8e72a8c/logs/ironic-bm-logs/node-0_console_18-11-2016-11:36:50.txt.gz#_2016-11-18_11_36_41_104 so the first heartbeat passes13:59
* _milan_ doesn't get it13:59
*** lindycoder has joined #openstack-ironic14:00
TheJuliaheartbeat reports to the conductor the IP address to connect back to14:00
TheJuliathat connection back fails in the conductor log14:00
_milan_ah14:01
_milan_so it worked one way but didn't work it's way back?14:01
_milan_odd14:01
TheJuliaI saw mention of this same issue on OSIC recently in the last few weeks, but I just don't remember off-hand14:01
_milan_TheJulia, yeah I see it now in the route output is 10.0.0.0/8 dev ens3  proto kernel  scope link  src 10.7.181.12914:03
TheJuliayup14:03
jrollI'm going to take today off, I'll be mostly on the computer if you need me though. or hangouts.14:09
_milan_jroll, o/ have a nice weekend! :)14:09
jrollyou all do the same :) see you monday14:10
TheJuliaThe only thing I can think is use something else besides 10/8 for that testing network14:10
TheJuliahave a good weekend jroll14:10
_milan_too bad I don't follow how the namespace thing works here14:11
_milan_I feel like there should be an interface connected to the same net on the conductor side14:11
*** syed__ has joined #openstack-ironic14:12
_milan_although it's routing14:12
* _milan_ dunno14:12
*** dmellado|lunch is now known as dmellado14:12
_milan_forwarding would have to be enabled?14:12
*** daemontool has joined #openstack-ironic14:13
* _milan_ wonders where are the network guys when they're needed ;)14:13
_milan_also why IPA listens on 10.0.0.5 if the src addr is 10.0.0.714:15
*** causten has quit IRC14:15
*** causten has joined #openstack-ironic14:16
TheJuliaeven with forwarding on, the default namespace shouldn't be routing packets down to the namespace that it doesn't really know about14:18
*** Goneri has joined #openstack-ironic14:18
TheJulia_milan_: so, first attempt was with 10.0.0.5.  Second was with 10.0.0.7 from what I saw14:18
_milan_TheJulia, you're right w/ the arp: http://paste.openstack.org/show/589716/14:19
_milan_TheJulia, ack14:19
*** cdearborn has joined #openstack-ironic14:20
TheJuliaI'm not in a position to test unborking the config, not enough bandwidth to stand-up a full devstack in a vm like OSIC14:20
*** daemontool has quit IRC14:21
*** daemontool has joined #openstack-ironic14:21
*** causten has quit IRC14:23
openstackgerritVasyl Saienko proposed openstack/ironic: Rely on portgroup standalone_ports_supported  https://review.openstack.org/36064714:25
*** jheroux has joined #openstack-ironic14:28
*** ccamacho has quit IRC14:29
*** srobert has joined #openstack-ironic14:29
*** ccamacho has joined #openstack-ironic14:30
_milan_is there some pic how this devstack thing is wired together w/r networking?14:37
* _milan_ feels like visiting a nuclear power plant 14:37
_milan_wires everywhere14:37
*** amotoki has joined #openstack-ironic14:37
*** bnemec has joined #openstack-ironic14:42
*** e0ne_ has quit IRC14:42
*** dansmith is now known as superdan14:42
*** amotoki_ has joined #openstack-ironic14:45
openstackgerritVladyslav Drok proposed openstack/ironic-specs: Add bonding configuration fields to portgroups  https://review.openstack.org/39661014:47
vsaienk0is there anyone who investigating CI failures?14:47
*** amotoki has quit IRC14:48
*** e0ne has joined #openstack-ironic14:48
*** bnemec is now known as beekneemech-semi14:48
*** beekneemech-semi is now known as bnemec-semi-here14:49
*** rook is now known as rook-baby14:51
TheJuliavsaienk0: I have to a degree, if you look at discussion you should get some context14:53
*** chlong has joined #openstack-ironic14:53
*** causten_ has joined #openstack-ironic14:54
TheJulia_milan_: no diagram, that I'm really aware of14:54
vdrokjust tried the logstash query14:54
vdrokthe first failure is 2016-11-18T02:21:40.338Z14:54
vdrokhttps://github.com/openstack-dev/devstack/commit/1493bdeba24674f6634160d51b8081c571df401714:54
*** amotoki_ has quit IRC14:55
TheJulia_milan_: part of it http://git.openstack.org/cgit/openstack/ironic/tree/devstack/tools/ironic/scripts/setup-network.sh14:56
TheJuliavdrok: time to request a revert? :)14:57
*** rpioso has joined #openstack-ironic14:58
*** rama_y has joined #openstack-ironic14:58
TheJuliaor add it to our stuff :\14:59
vdrokTheJulia: pointed it to vsaienk0, he tries to find a place where it should be15:00
vdrokTheJulia: not sure if revert will be accepted15:00
TheJulia:(15:02
TheJuliabetherly: papilla: our call is in an hour right?15:02
_milan_TheJulia, thx for the link15:02
*** shuquan has joined #openstack-ironic15:03
_milan_well, the commit message from that patch doesn't say much ^^^15:03
_milan_at least it seems it all happens via L315:04
betherlyTheJulia: ppiela: yup indeed15:05
*** shuquan has quit IRC15:07
openstackgerritVasyl Saienko proposed openstack/ironic: Unblock broken gates  https://review.openstack.org/39963215:09
vsaienk0sambetts: are you planning to finish https://review.openstack.org/#/c/392959/ in near future? It is a real shame that we using vxlan network in our 'flat' scenario...15:12
patchbotpatch 392959 - ironic - Rework Ironic devstack baremetal network simulation15:12
_milan_vsaienk0, hey :) I dunno what I'm doing, just curious: https://github.com/openstack-dev/devstack/commit/1493bdeba24674f6634160d51b8081c571df4017#diff-553937b39616c8a41282299a514dfdfcL373 missing in your patch but I guess that's OK15:20
_milan_https://review.openstack.org/399632 < ^15:20
patchbotpatch 399632 - ironic - Unblock broken gates15:20
vsaienk0_milan_ it is just informational thing, if we failed to get router IP, ip route replace command will fail15:24
_milan_vsaienk0, OK cool :)15:24
* _milan_ +115:24
vsaienk0TheJulia do we have a bug  already?15:25
vsaienk0I just want to make sure it works :) and reupload patch with bug ID and die "" in the code15:25
TheJuliano, we don't have a bug yet15:25
TheJuliaI can create one15:26
openstackgerritMario Villaplana proposed openstack/ironic: Add notification documentation to install guide  https://review.openstack.org/39578715:26
vsaienk0TheJulia I think we can reuse old bug: https://bugs.launchpad.net/openstack-ci/+bug/139309915:26
openstackLaunchpad bug 1393099 in OpenStack-Gate "test_baremetal_server_ops fails with timeout waiting for provision_state to change" [Undecided,Confirmed]15:26
*** godp1301_ has joined #openstack-ironic15:26
* TheJulia looks15:26
mariojvJayF: I responded to your comments there ^ fixed all the issues you mentioned, and expanded the commit message + added Related-Bug tag15:26
* _milan_ fingers crossed for jenkins acking this fixes us :)15:27
vsaienk0TheJulia: or create a new one, the one I provided is not related :(15:27
TheJuliavsaienk0: reusing that one seems... like not a great idea ;)15:27
TheJuliaagreed, creating, give me five and I'll update the commit message on that change15:28
JayFmariojv: +215:28
*** godp1301 has quit IRC15:29
* _milan_ bbib15:32
mariojvthanks!15:33
mariojvJayF: i think i remember you mentioning a weekly list of priority patches to review based on the ironic meeting15:34
*** jcoufal has joined #openstack-ironic15:34
mariojvJayF: are you just talking about the patches listed under the subteam status reports on the whiteboard?15:34
*** fxpester has quit IRC15:37
JayFmariojv: yeah, it's in the whiteboard15:37
JayFmariojv: under "weekly priorities"15:37
mariojvthanks15:37
*** rook-baby is now known as rook15:37
TheJuliavsaienk0: https://bugs.launchpad.net/ironic/+bug/164298615:38
openstackLaunchpad bug 1642986 in Ironic "Ironic gate failure - unable to reach test vm network" [Critical,In progress]15:38
TheJuliavsaienk0: if you could assign that to yourself, since I don't know your launchpad id off the top of my head15:39
openstackgerritJulia Kreger proposed openstack/ironic: Unblock broken gates  https://review.openstack.org/39963215:40
JayFTheJulia: mind updating the whiteboard about gate issue while I review your patch?15:41
TheJuliavsaienk0's patch, I just added the bug #15:41
*** yufei has quit IRC15:41
vdrokJayF: morning, could you remove -2 on https://review.openstack.org/#/c/295371/ &15:42
patchbotpatch 295371 - ironic - Add user and project domains to ironic context15:42
openstackgerritVasyl Saienko proposed openstack/ironic: Unblock broken gates  https://review.openstack.org/39963215:43
JayFvdrok: done15:43
vdrokty15:43
*** yufei has joined #openstack-ironic15:43
JayFvsaienk0: would it be reasonable for us to ask devstack to revert their patch?15:43
vsaienk0TheJulia: I've reuploaded patch, the way Neutron folks get router IP doesn't work, because if using both ipv4 and ipv6 IP is not determined correctly15:44
TheJuliaheh15:44
TheJuliaI just pulled it up15:44
openstackgerritDmitry Tantsur proposed openstack/ironic: Add node (database and objects) fields for all interfaces  https://review.openstack.org/39668115:44
dtantsurno rescue no cry ^^^15:44
vsaienk0JayF: I don't think that reverting devstack patch is proper fix, because it is a shame that we using VXLAN network in our setup at the gates15:45
JayFvsaienk0: cool :). I reviewed your patch, +2 but a couple of nits on the commit message if you have to push another patchset15:46
vsaienk0JayF: sambetts has a patch that reworks network setup https://review.openstack.org/#/c/392959/ we need to follow that way15:46
patchbotpatch 392959 - ironic - Rework Ironic devstack baremetal network simulation15:46
JayFYeah, I've looked at that. It's hard to grok how it's all supposed to work15:46
* JayF wants a devstack+ironic network diagram15:46
*** vsaienko has quit IRC15:46
TheJuliaI could actually make one at this point, I may just cry15:47
JayFlol'15:47
TheJuliait is not too complex, but you have to have wrapped your head around namespaces and how neutron works...15:47
*** causten_ has quit IRC15:48
*** mat128_ has joined #openstack-ironic15:48
*** [1]cdearborn has joined #openstack-ironic15:48
mat128_Update: we're at the hospital!15:48
TheJuliaIf somebody ships me a bottle of Nikka, I'd make a diagram :)15:49
JayFmat128_: do whatever you can to provision that child, remember, you're an expert15:49
TheJuliamat128_: :)15:50
*** pcaruana has quit IRC15:50
*** HenryG has quit IRC15:50
*** HenryG has joined #openstack-ironic15:51
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP] Support defining and loading hardware types  https://review.openstack.org/33662615:56
*** ijw has joined #openstack-ironic15:58
*** anush has joined #openstack-ironic15:59
*** ijw_ has joined #openstack-ironic15:59
*** ijw_ has quit IRC15:59
NobodyCamGood Morning Ironic'ers :) ... And....16:00
*** ijw_ has joined #openstack-ironic16:00
NobodyCamTGIF!!!!16:00
lucasagomesNobodyCam, hi there16:01
NobodyCamhey hey lucasagomes :) happy Friday :)16:01
lucasagomeso/16:01
NobodyCam:)16:01
*** causten has joined #openstack-ironic16:01
*** cdearborn has quit IRC16:03
dtantsurmorning NobodyCam16:03
*** ijw has quit IRC16:03
NobodyCamGood Morning dtantsur :)16:03
*** daemontool has quit IRC16:05
*** yufei has quit IRC16:06
*** Guest90313 has quit IRC16:09
*** rajinir has joined #openstack-ironic16:10
*** anush has quit IRC16:10
*** anush has joined #openstack-ironic16:12
*** anush has quit IRC16:12
*** anush has joined #openstack-ironic16:13
*** anush has quit IRC16:13
*** anush has joined #openstack-ironic16:14
*** anush has quit IRC16:14
vdrokmorning NobodyCam and mat12816:15
vsaienk0Morning NobodyCam16:15
NobodyCamMorning vdrok and vsaienk0 :)16:15
*** rama_y has quit IRC16:15
mgouldmorning NobodyCam16:16
vsaienk0morning mgould :)16:16
NobodyCammorning mgould :)16:16
*** anush has joined #openstack-ironic16:16
mgouldmat128_: gosh, good luck!16:16
mgouldTheJulia: make a diagram and artistically stain it with tears16:17
mat128_mgould: ty16:17
*** amotoki has joined #openstack-ironic16:18
*** penick has joined #openstack-ironic16:18
mgouldTheJulia: alternatively, put lots of "here be dragons" labels on it16:19
openstackgerritMoshe Levi proposed openstack/ironic-inspector: add lldp system name to switch_info  https://review.openstack.org/39967116:20
*** rbrud has joined #openstack-ironic16:20
*** anush has quit IRC16:21
*** godp1301 has joined #openstack-ironic16:21
*** godp1301_ has quit IRC16:25
mariojvvdrok: i'm curious about why the default portgroup mode is "active-backup" in https://review.openstack.org/#/c/39661016:27
patchbotpatch 396610 - ironic-specs - Add bonding configuration fields to portgroups16:27
mariojvvdrok: the default linux mode is round robin it seems16:27
JayFmariojv: what line?16:28
mariojvJayF: L134: https://review.openstack.org/#/c/396610/7/specs/approved/ironic-ml2-integration.rst16:29
JayFHmm. I see it.16:29
patchbotpatch 396610 - ironic-specs - Add bonding configuration fields to portgroups16:29
JayFThat's certainly not the mode most people run their bonds on16:29
mariojvit seems like there are only advantages to balance-rr mode, compared to active-backup16:29
mariojvJayF: active-backup is not or balance-rr is not?16:29
JayFyes.16:29
JayFI would pick lacp.16:29
JayFbecause that's most commonly used16:30
JayFaka 802.1ad16:30
mariojvthat requires a particular physical configuration right?16:30
JayFaka mode "4"16:30
JayFHmm. Yeah, it requires switch cooperation16:30
mariojvyeah, switch has to support 802.3ad16:30
mariojvi think it's better to have our default be the linux default16:31
mariojvbenefits of balance-rr seem to encompass all the active-backup benefits, plus more, and no extra disadvantages16:31
mariojvfrom what i can gather16:31
vdrokhere is sambetts_ reasoning - http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2016-11-17.log.html#t2016-11-17T12:44:3616:32
mariojvi'll leave a comment about that for now16:32
mariojvthanks vdrok16:32
vdrokmy personal opinion is - to allow None, but most of people want some default so...16:32
mariojvvdrok: sambetts_: imbw, but i don't think round robin requires any particular switch support16:33
openstackgerritVasyl Saienko proposed openstack/ironic: Add route to Neutron private network  https://review.openstack.org/39963216:33
mariojvi think it'll work even with 1 switch, there would just be only 1 available switch on the round robin16:34
mariojvhmm, well the linux docs don't say anything about it, but netgear says it requires static link aggregation to be enabled https://www.readynas.com/wp-content/uploads/2008/09/ReadyNAS-Teaming.pdf16:36
mariojvyeah, many switches don't support balance-rr, according to other docs16:36
mariojvnever mind then16:37
mariojvvdrok: also i think None could be allowed, even if it isn't the default16:41
mariojvthen it could default to linux default16:42
mariojvi guess that wouldn't be very useful though, since if you're changing it to None, you may as well make it explicitly balance-rr16:42
vdrokmariojv: yeah, having two defaults, one explicit and one implicit would be confusing16:45
JayFOne thing to note here16:46
JayFfrom having used various bonding modes in the real world16:46
JayFsome of those are really, really unstable16:46
JayFand iirc balance-rr is one of those16:46
JayF*note, last I tried it was 5y+ ago, lol16:46
mariojvyeah, some references note that16:47
mariojvespecially for udp16:47
mariojvprobably poorly coded uses of udp... but yeah16:47
JayFI mean like, kernel panic unstable16:49
*** makowals has quit IRC16:49
JayFlol16:49
mariojvouch16:49
*** makowals has joined #openstack-ironic16:50
*** rama_y has joined #openstack-ironic16:52
*** makowals has quit IRC16:53
vdrokmariojv: the str_or_none is mentioned because you can actually create a node in the api with mode=None, it will go down all the way to dbapi and only there it will be changed to the conf option value. the table iiuc describes the object level fields. will add more clarifications tho16:53
mariojvoh, i see16:54
mariojvthank you16:54
*** makowals has joined #openstack-ironic16:54
mariojv-1 is only for the default justification then16:54
*** godp1301 has quit IRC16:54
vdrokyup ,thanks for review! :) seems like i'll update it only on monday16:54
*** ccamacho has quit IRC16:58
*** makowals has quit IRC16:58
*** ralonsoh has quit IRC17:05
openstackgerritPeter Piela proposed openstack/ironic-ui: Incorporate driver-validation into node-detail panels  https://review.openstack.org/39969517:06
*** moshele has quit IRC17:12
*** sabeen has quit IRC17:16
*** pester has quit IRC17:19
*** rbrud has quit IRC17:20
*** e0ne has quit IRC17:20
*** sabeen has joined #openstack-ironic17:24
*** derekh has quit IRC17:24
dtantsurhave a nice weekend, see you on Monday17:26
*** dtantsur is now known as dtantsur|afk17:26
JayFo/17:26
*** srobert has quit IRC17:28
lucasagomesdtantsur|afk, g'night17:30
* lucasagomes is heading too17:30
lucasagomeshave a great weekend on, enjoy it!17:31
lucasagomesweekend all*17:31
*** lucasagomes is now known as lucas-afk17:32
*** Guest52285 is now known as mgagne17:33
*** mgagne has quit IRC17:33
*** mgagne has joined #openstack-ironic17:33
*** amotoki has quit IRC17:34
vdrokgood night dtantsur|afk and lucas-afk17:40
*** vsaienko has joined #openstack-ironic17:42
vsaienkoJayF, TheJulia, vdrok patch to unblock gate passed CI, I've resolved comments https://review.openstack.org/#/c/399632/17:43
patchbotpatch 399632 - ironic - Add route to Neutron private network17:43
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Fix PXE setup for fresh Ubuntu Xenial  https://review.openstack.org/39948517:43
*** trown is now known as trown|lunch17:44
*** vsaienko has quit IRC17:48
*** godp1301 has joined #openstack-ironic17:48
*** vsaienko has joined #openstack-ironic17:49
*** rbrud has joined #openstack-ironic17:51
*** bcornec1 has quit IRC17:52
*** vsaienko has quit IRC17:53
*** jpich has quit IRC17:53
mariojvhey, question about how node lookup works w/ policy17:56
mariojvhow does passing instance_info even happen during a deploy?17:57
mariojvthe lookup convert_with_links calls the node subcontroller convert_with_links17:57
mariojvhttps://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/ramdisk.py#L7817:57
*** vsaienko has joined #openstack-ironic17:57
mariojvthat checks for show_instance_secrets in policy: https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/node.py#L83517:58
mariojvby default, that's false in policy: https://github.com/openstack/ironic/blob/master/etc/ironic/policy.json.sample#L817:58
mariojvso, how can you even deploy with a configdrive with instance password?17:59
*** aNuposic has joined #openstack-ironic17:59
mariojvJayF found it, there's a separate prepare_image call18:00
*** vsaienko has quit IRC18:00
* mgould emails the Kyrgyz Alpine Club to say "my colleagues found a whole load of mistakes in your Russian..."18:10
mgouldI did however add "but I constantly nitpick their English, so this may just be revenge" :-)18:11
*** ipukha has quit IRC18:29
*** Sukhdev has joined #openstack-ironic18:33
*** vsaienko has joined #openstack-ironic18:37
*** mgould is now known as mgould|afk18:39
vdrokgood night everyone!18:41
_milan_night vdrok! :)18:41
*** vsaienko has quit IRC18:42
*** krtaylor has joined #openstack-ironic18:46
*** vsaienko has joined #openstack-ironic18:46
*** _milan_ has quit IRC18:50
*** vsaienko has quit IRC18:53
openstackgerritMerged openstack/ironic: Add route to Neutron private network  https://review.openstack.org/39963218:54
*** glonlas has joined #openstack-ironic18:54
glonlasHello, Do you know where I can download the new mascot logo for Ironic? (even if it's not approved yet)18:54
*** milan has joined #openstack-ironic18:55
JayF^^ gate is unblocked, going to check at the whiteboard18:59
JayFglonlas: no idea; but I know a lot of folks had feedback about it18:59
JayFglonlas: I'd suggest continuing to use pixie boots for whatever logo needs you have until it's officialized :)18:59
glonlasI will :) Thank you19:00
*** glonlas_ has joined #openstack-ironic19:01
*** vsaienko has joined #openstack-ironic19:01
*** glonlas_ has quit IRC19:02
*** jkilpatr has joined #openstack-ironic19:02
*** glonlas_ has joined #openstack-ironic19:02
vsaienkogood night Ironic'ers! gates are unblocked, safe to do rechecks now :)19:03
JayFvsaienko: tyvm for that :D19:04
JayFvsaienko: I updated whiteboard and logged the failure in the gate debugging etherpad19:04
cloudnullhey all. is the ironic cluster manager recommended in mitaka ? right now we're running the nova ComputeManager and while it's working we're seeing some limitations w/ scheduling, mainly it's just a little slow, and I'm curious if the ironic ClusteredComputeManager is better?19:04
*** glonlas has quit IRC19:05
*** milan has quit IRC19:05
*** milan has joined #openstack-ironic19:05
cloudnullTheJulia: if you need an instance on OSIC to test things I'd be happy to give you one. just let me know.19:06
*** godp1301 has quit IRC19:06
* cloudnull was reading scrollback19:06
JayFcloudnull: very much no19:06
JayFcloudnull: you're better off running active/passive with cold failover than using CCM19:06
cloudnullok. is that better in Newton ?19:06
JayFYes, very much, I can link you the release note19:07
JayFjroll landed real HA for nova-compute with ironic driver19:07
*** trown|lunch is now known as trown19:07
JayFcloudnull: http://docs.openstack.org/releasenotes/nova/newton.html ^f 'ironic virt driver'19:07
cloudnullah cool . so the consistent hashing make it all better.19:08
*** aNuposic has quit IRC19:08
JayFYeah, basically unprovisioned nodes automatically distribute as nova-computes are added/removed19:08
*** vsaienko has quit IRC19:09
JayFBUT deployed instances are "sticky" to the nova-compute that deployed them19:09
cloudnullvery cool19:09
JayFso you can't perform instance actions on an instance if it's associated compute is not up19:09
JayFso it's not "true HA', but it follows the general nova/openstack model of having limited failure domains19:09
patchbotError: No closing quotation19:09
*** aNuposic has joined #openstack-ironic19:09
cloudnullthat's all good things IMO.19:10
cloudnullmore reason I can go yell at people such that they let me upgrade to newton19:10
JayFcloudnull: I suspect we may have already won that argument for you downstream, fwiw. You might wanna re-check-in with product :)19:10
*** penick has quit IRC19:11
*** vsaienko has joined #openstack-ironic19:11
cloudnullyea, I think I'm good with product people. I just need to convice OSIC that is what we're going to do.19:12
*** snozhka has quit IRC19:12
cloudnullalso using the RAID configs let me get around my /boot issues.19:12
cloudnullso thanks again TheJulia and JayF for all of the pointers there.19:13
*** snozhka has joined #openstack-ironic19:13
JayFno problem :)19:14
*** penick has joined #openstack-ironic19:14
*** betherly has quit IRC19:18
*** milan has quit IRC19:20
*** milan has joined #openstack-ironic19:21
*** vsaienko has quit IRC19:22
*** vsaienko has joined #openstack-ironic19:29
*** milan has quit IRC19:36
*** glonlas_ has quit IRC19:38
*** glonlas has joined #openstack-ironic19:39
*** bnemec-semi-here has quit IRC19:39
*** milan has joined #openstack-ironic19:41
*** vsaienko has quit IRC19:49
*** milan has quit IRC19:50
*** vsaienko has joined #openstack-ironic19:51
*** moshele has joined #openstack-ironic19:51
*** glonlas_ has joined #openstack-ironic19:55
*** mat128_ has quit IRC19:58
*** glonlas has quit IRC19:59
*** milan has joined #openstack-ironic20:01
*** milan has quit IRC20:05
JayFI think I have evidence that our gate is broken again20:09
JayFat least IPA's gate20:09
*** penick has quit IRC20:09
JayFdevstack fails to come up here: http://logs.openstack.org/16/334816/21/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-coreos-src-ubuntu-xenial/95f5e9e/logs/devstacklog.txt.gz#_2016-11-18_19_41_00_39420:09
JayFand the error printed seems to indicate they're validing the cidr mask as if it's an IPv4, not IPv6, subnet: http://logs.openstack.org/16/334816/21/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-coreos-src-ubuntu-xenial/95f5e9e/logs/screen-q-svc.txt.gz#_2016-11-18_19_41_00_38920:10
*** milan has joined #openstack-ironic20:21
JayFlooks like that is breaking our gate, working with infra to find a fix20:22
*** glonlas_ has quit IRC20:26
*** milan has quit IRC20:35
JayF^^^ gate is broken, whiteboard updated20:37
JayFdevstack patch already in place to fix it, awaiting merge20:37
JayFhttps://review.openstack.org/#/c/399762/ and https://bugs.launchpad.net/devstack/+bug/1643055 if anyone is curious20:38
openstackLaunchpad bug 1643055 in devstack "FIXED_RANGE_V6 not set to sane value in neutron_plugins/services/l3" [Undecided,In progress] - Assigned to Clark Boylan (cboylan)20:38
patchbotpatch 399762 - openstack-dev/devstack - Fix default ipv6 fixed range var20:38
mariojvif anyone wants an easy review, this one's short and has been waiting for a while: https://review.openstack.org/#/c/385459/20:38
patchbotpatch 385459 - ironic - Fix stable states validation in FSM20:38
mariojvmat128: you might want to reconsider your -1 on that - the docs follow up patch has been approved and will be auto merged as soon as that one is merged20:39
*** aNuposic has quit IRC20:39
JayFmariojv: he's off being a dad in a delivery room, aiui20:40
JayFmariojv: so might be ssomething we have to merge over a -120:40
JayFdtantsur|afk: gate is broken, ^^^ please do not recheck20:40
mariojvah! congrats!20:40
*** milan has joined #openstack-ironic20:41
*** penick has joined #openstack-ironic20:44
*** godp1301 has joined #openstack-ironic20:48
*** glonlas has joined #openstack-ironic20:49
*** glonlas has quit IRC20:49
*** milan has quit IRC20:51
*** glonlas has joined #openstack-ironic20:52
*** glonlas has quit IRC20:52
*** chlong has quit IRC20:53
*** godp1301 has quit IRC20:53
*** clenimar has quit IRC20:56
*** godp1301 has joined #openstack-ironic20:56
openstackgerritMerged openstack/ironic-inspector: Fix passing incorrect args to logging  https://review.openstack.org/39958421:00
*** glonlas has joined #openstack-ironic21:00
*** milan has joined #openstack-ironic21:01
*** MattMan has quit IRC21:03
*** MattMan has joined #openstack-ironic21:04
*** ijw_ has quit IRC21:04
*** glonlas has quit IRC21:04
*** glonlas_ has joined #openstack-ironic21:04
*** milan has quit IRC21:05
*** moshele has quit IRC21:09
*** jheroux has quit IRC21:11
*** fragatina has quit IRC21:11
JayFthe fix for the gate is landing now in devstack, just got +a'd21:14
*** milan has joined #openstack-ironic21:17
*** jkilpatr has quit IRC21:17
openstackgerritOpenStack Proposal Bot proposed openstack/bifrost: Updated from global requirements  https://review.openstack.org/39790321:23
*** anush has joined #openstack-ironic21:28
*** godp1301 has quit IRC21:34
*** fragatina has joined #openstack-ironic21:41
*** causten has quit IRC21:44
*** rbrud has quit IRC21:45
*** e0ne has joined #openstack-ironic21:51
*** trown is now known as trown|outtypewww21:54
*** rbrud has joined #openstack-ironic21:57
*** causten has joined #openstack-ironic21:58
*** nicodemos has quit IRC22:00
*** nicodemos has joined #openstack-ironic22:00
*** penick has quit IRC22:01
*** penick has joined #openstack-ironic22:03
*** glonlas_ has quit IRC22:12
*** e0ne has quit IRC22:21
*** jcoufal has quit IRC22:23
*** milan has quit IRC22:27
openstackgerritMerged openstack/ironic-inspector: Fixed incorrect string type checking  https://review.openstack.org/39912322:28
*** lindycoder has quit IRC22:31
*** aNuposic has joined #openstack-ironic22:40
*** penick has quit IRC22:52
openstackgerritJesse J. Cook proposed openstack/ironic: Add API and conductor methods for [un]rescue  https://review.openstack.org/35083122:54
openstackgerritJesse J. Cook proposed openstack/ironic: Add API and conductor methods for [un]rescue  https://review.openstack.org/35083122:55
*** [1]cdearborn has quit IRC22:59
*** baoli_ has quit IRC22:59
*** baoli has joined #openstack-ironic23:01
*** causten has quit IRC23:01
*** causten has joined #openstack-ironic23:02
*** baoli_ has joined #openstack-ironic23:03
*** vsaienko has quit IRC23:05
*** baoli has quit IRC23:06
*** causten has quit IRC23:06
*** anush has quit IRC23:07
*** anush has joined #openstack-ironic23:09
openstackgerritJulia Kreger proposed openstack/bifrost: Add support to update keystone for bifrost  https://review.openstack.org/38022623:11
*** anush has quit IRC23:12
TheJuliaThank you NobodyCam for finding that!23:13
NobodyCam*(blush)*23:14
*** ijw has joined #openstack-ironic23:14
*** ijw has quit IRC23:15
*** ijw has joined #openstack-ironic23:15
*** zackf has quit IRC23:17
*** baoli has joined #openstack-ironic23:23
*** baoli_ has quit IRC23:26
*** bfournie has quit IRC23:28
*** vsaienko has joined #openstack-ironic23:30
*** vsaienko has quit IRC23:32
*** mjturek has quit IRC23:40
*** fragatin_ has joined #openstack-ironic23:50
*** rpioso has quit IRC23:51
*** fragatina has quit IRC23:53

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