Wednesday, 2016-06-08

JayF000000 UTC gerrit outage strikes agani00:02
JayFgrargh00:02
*** ijw has quit IRC00:03
jlvillalJayF: Me too! :(00:04
*** rama_y has quit IRC00:04
jlvillalOkay, looks like short lived pause00:04
openstackgerritJay Faulkner proposed openstack/ironic-specs: Implement rescue mode  https://review.openstack.org/17187800:04
JayFthey do backups daily00:04
JayFthat's why it dies like that00:04
JayFjust because of the hours I keep I'm almost always in gerrit when it happens00:05
*** Sukhdev has quit IRC00:07
jlvillalJayF: Yep. Happens to me00:07
jlvillalJayF: devananda https://review.openstack.org/#/c/313187/  +2 and +A  I left some comments if want to have a follow-up patch, if appropriate.00:07
*** Sukhdev has joined #openstack-ironic00:08
*** ijw has joined #openstack-ironic00:09
*** ijw has quit IRC00:11
*** ijw has joined #openstack-ironic00:11
*** rbrud has quit IRC00:11
JayFmikal: would love you to land these stable backports: - Nova stable backports: https://review.openstack.org/#/c/323477/ https://review.openstack.org/#/c/323196/ when you get a sec00:11
JayFmikal: you had asked me on one of hte original patches to backport, so I went ahead and got mine + the one it depended on. Matt R has already +2'd both00:11
*** ppiela has quit IRC00:13
JayFjroll: [async] https://review.openstack.org/#/c/321246/ appears to have pretty good consensus, you might want to land it before it needs a rebase00:15
*** PollyZ has joined #openstack-ironic00:15
openstackgerritJay Faulkner proposed openstack/ironic-specs: Implement rescue mode  https://review.openstack.org/17187800:16
*** clenimar has quit IRC00:17
openstackgerritJay Faulkner proposed openstack/ironic-specs: Implement rescue mode  https://review.openstack.org/17187800:18
devanandaJayF: omg - I just hid configdrive from the output of node-show00:19
devanandavia policy settings00:19
* JayF applies that patch downstream immediately00:19
JayFlol00:19
devanandathis makes me so happy00:19
JayFwhen you do that00:19
JayFdoes it hide both key and value00:19
JayFor is it like configdrive: ****00:20
devanandano - it masks the value, just like password00:20
devanandabecause i'm usiing the same oslo method00:20
JayFperfecto00:20
*** PollyZ has quit IRC00:20
openstackgerritMerged openstack/ironic: Update the api-ref documentation for Ports  https://review.openstack.org/31318700:20
devanandaoslo_utils.strutils.mask_password() already checks for a property named "configdrive"00:20
devanandawe just never applied it to instance_info -- or created a separate policy line for it00:20
devanandasoooo easy00:20
JayFyou're making all this seem to easy00:21
devanandayah00:21
devanandado you think that should be a separate patch? or rolled in?00:21
JayFhere's the $1,000,000 question:00:21
devananda.....00:21
JayFwould our backwards compat block-password stuff trigger it00:21
devanandahuh?00:21
JayFi.e. existing behavior would change or not00:21
JayFgrr not finding words00:22
JayFhttps://github.com/openstack/ironic/blob/master/etc/ironic/policy.json#L300:22
JayFis that going to be the same rule that blocks configdrive?00:22
JayFor would it be something different; i.e. show_configdrive00:22
devanandai added this as a new rule for now00:23
devanandaso00:23
JayFas long as it's a new rule it's backward compat and awesome and I don't care if it's a different patc00:24
JayF*patch00:24
JayFor not00:24
devanandasomeone updating to this ironic code will NEED to update their policy.json00:24
devanandaI want to ensure that they don't also need to update the roles in their cloud00:24
devanandawhich I think we do by preserving the is_admin rule00:24
devanandaand allowing that override to still override everything else00:24
JayFI'm not sure I understand why you'd be required to update your policy.json00:24
devanandahmm00:25
devanandamaybe you're not. lemme check00:25
JayFshouldn't our default policies be defined in code, re: the oslo spec anyway? Even if not it doesn't seem like you'd need to update it00:25
devanandaI may be mistaken about that00:26
devanandaI think policy.enforce defaults to pass when there is no such rule defined00:26
devanandaor defaults to the default rule (which is "is_admin")00:26
openstackgerritDevananda van der Veen proposed openstack/ironic: [WIP] Mask configdrive contents in API responses  https://review.openstack.org/32676800:26
devanandaJayF: there's the change ^00:26
JayFawesome; first thing on my list tomorrow is to read all that code and write the spec00:27
JayFI'm not sure I should do it in that order, but that's what I'm doing00:27
JayFlol00:27
devanandahah00:27
JayFwhich came first, the spec or the code00:27
JayFdoesn't matter; has policy00:27
devananda:-D00:27
*** mtanino has quit IRC00:27
devanandain this case, I think the spec is going to be more about documenting the impact00:27
devanandaand then turning that into actual docs00:28
JayFall the patches are labelled WIP, but do you want review on them yet at all?00:28
devanandago for it00:28
JayFwondering how much you expect them to mutate; should I read for review or read for knowledge00:28
devanandathey're ready for reviews00:28
JayFa-ok will review them00:28
devanandaknowledge00:28
devanandaI WIP'd them because I don't want them to land until we have grenade00:28
JayFThat's very sensible.00:28
JayFThat's close, isn't it?00:28
devanandayah00:28
devanandaI think jlvillal said there's like 3 patches outstanding at this point00:28
devanandaI'm happy to wait a week or three for these to land00:29
*** PollyZ has joined #openstack-ironic00:29
devanandaif it means better confidence that it won't break the world00:29
JayFThat works. I feel like this cycle has been crazy productive so far.00:29
JayFMaybe because all the attention on the gate has made things go more smoothly00:29
devanandamaybe that is the stuff we've done ;)00:29
*** clenimar has joined #openstack-ironic00:30
JayFAlright I'm getting off IRC for the day. Awesome stuff, seriously thanks for all this.00:31
JayFo/00:31
*** amotoki has joined #openstack-ironic00:31
devanandaciao!00:31
*** PollyZ has quit IRC00:33
*** amotoki has quit IRC00:36
*** Sukhdev has quit IRC00:37
*** PollyZ has joined #openstack-ironic00:42
*** Goneri has joined #openstack-ironic00:42
*** hoangcx has joined #openstack-ironic00:45
*** PollyZ has quit IRC00:47
openstackgerritDevananda van der Veen proposed openstack/ironic: [WIP] Mask configdrive contents in API responses  https://review.openstack.org/32676800:49
openstackgerritDevananda van der Veen proposed openstack/ironic: [WIP] Mask configdrive contents in API responses  https://review.openstack.org/32676800:50
*** Jabadia has joined #openstack-ironic00:50
*** PollyZ has joined #openstack-ironic00:56
*** baoli has joined #openstack-ironic00:57
*** PollyZ has quit IRC01:00
*** jaybeale_ has quit IRC01:07
*** Goneri has quit IRC01:07
*** PollyZ has joined #openstack-ironic01:09
openstackgerritMerged openstack/ironic: Update the api-ref documentation for Drivers  https://review.openstack.org/31370801:12
*** PollyZ has quit IRC01:14
*** PollyZ has joined #openstack-ironic01:23
*** PollyZ has quit IRC01:28
*** phuongnh has joined #openstack-ironic01:29
*** frontrunner has quit IRC01:31
*** amotoki has joined #openstack-ironic01:32
*** ijw has quit IRC01:32
*** amotoki has quit IRC01:36
*** PollyZ has joined #openstack-ironic01:37
*** ChrisAusten has joined #openstack-ironic01:38
*** PollyZ has quit IRC01:41
openstackgerritMerged openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/32545101:46
*** sacharya has joined #openstack-ironic01:59
*** rloo has quit IRC01:59
*** PollyZ has joined #openstack-ironic02:09
devanandaJayF: traced down why all the tests are failing for my patches: http://logs.openstack.org/99/325599/4/check/gate-tempest-dsvm-ironic-pxe_ipa/c9c8a11/logs/screen-ir-api.txt.gz#_2016-06-07_23_17_51_52002:14
devanandano response needed, just interesting for you02:14
*** Jabadia has quit IRC02:16
*** PollyZ has quit IRC02:24
*** PollyZ has joined #openstack-ironic02:28
*** sdake has joined #openstack-ironic02:32
*** amotoki has joined #openstack-ironic02:33
*** PollyZ has quit IRC02:33
*** moshele has joined #openstack-ironic02:34
*** mtanino has joined #openstack-ironic02:34
*** sdake has quit IRC02:34
*** amotoki has quit IRC02:37
*** openstackgerrit has quit IRC02:47
*** openstackgerrit has joined #openstack-ironic02:48
*** gcb has joined #openstack-ironic02:52
*** rama_y has joined #openstack-ironic02:59
*** moshele has quit IRC02:59
*** kalpase has joined #openstack-ironic03:07
*** mtanino has quit IRC03:08
*** PollyZ has joined #openstack-ironic03:09
*** PollyZ has quit IRC03:14
*** fragatina has quit IRC03:19
*** PollyZ has joined #openstack-ironic03:23
*** PollyZ has quit IRC03:28
*** sdake has joined #openstack-ironic03:28
*** spandhe has quit IRC03:32
*** amotoki has joined #openstack-ironic03:34
*** links has joined #openstack-ironic03:35
*** PollyZ has joined #openstack-ironic03:36
*** amotoki has quit IRC03:38
*** PollyZ has quit IRC03:41
*** absubram has joined #openstack-ironic03:41
*** PollyZ has joined #openstack-ironic03:50
*** clenimar has quit IRC03:51
*** sdake has quit IRC03:53
*** PollyZ has quit IRC03:54
*** vmud213 has joined #openstack-ironic04:01
*** devananda has quit IRC04:11
*** fragatina has joined #openstack-ironic04:17
*** fragatina has quit IRC04:22
*** devananda has joined #openstack-ironic04:23
*** SiRiuS__ has joined #openstack-ironic04:26
openstackgerritMerged openstack/proliantutils: Add RIS support for power operations  https://review.openstack.org/28980004:28
*** PollyZ has joined #openstack-ironic04:31
*** Sukhdev has joined #openstack-ironic04:33
*** baoli has quit IRC04:34
*** amotoki has joined #openstack-ironic04:34
*** PollyZ has quit IRC04:35
*** fragatina has joined #openstack-ironic04:38
*** amotoki has quit IRC04:39
*** fragatina has quit IRC04:40
*** fragatina has joined #openstack-ironic04:40
*** fragatin_ has joined #openstack-ironic04:41
*** garthb has joined #openstack-ironic04:42
*** fragatina has quit IRC04:44
*** vishwanathj has joined #openstack-ironic04:51
*** garthb has quit IRC04:51
*** spandhe has joined #openstack-ironic04:53
*** spandhe_ has joined #openstack-ironic04:54
*** PollyZ has joined #openstack-ironic04:58
*** spandhe has quit IRC04:58
*** spandhe_ is now known as spandhe04:58
*** ChubYann has quit IRC04:59
*** sdake has joined #openstack-ironic05:02
*** PollyZ has quit IRC05:02
*** PollyZ has joined #openstack-ironic05:11
openstackgerritNaohiro Tamura proposed openstack/ironic-specs: Enhance Power Interface for Soft Power Off and Inject NMI  https://review.openstack.org/18670005:12
*** PollyZ has quit IRC05:16
*** moshele has joined #openstack-ironic05:20
*** sacharya has quit IRC05:22
openstackgerritAparna proposed openstack/proliantutils: Passing 'process_input' to the running hpssacli process  https://review.openstack.org/32573005:22
*** PollyZ has joined #openstack-ironic05:25
*** PollyZ has quit IRC05:30
*** sdake has quit IRC05:32
*** amotoki has joined #openstack-ironic05:35
*** fragatin_ has quit IRC05:36
*** spandhe has quit IRC05:37
*** PollyZ has joined #openstack-ironic05:39
*** amotoki has quit IRC05:39
*** PollyZ has quit IRC05:43
*** rama_y has quit IRC05:44
*** PollyZ has joined #openstack-ironic05:52
*** mtanino has joined #openstack-ironic05:56
*** PollyZ has quit IRC05:57
*** stendulker has joined #openstack-ironic05:59
*** jtomasek has quit IRC06:01
*** ChrisAusten has quit IRC06:05
*** PollyZ has joined #openstack-ironic06:06
*** PollyZ has quit IRC06:11
*** lifeless has quit IRC06:13
*** rcernin has joined #openstack-ironic06:13
*** PollyZ has joined #openstack-ironic06:19
*** e0ne has joined #openstack-ironic06:21
*** sacharya has joined #openstack-ironic06:22
*** PollyZ has quit IRC06:25
*** sacharya has quit IRC06:27
*** purp has quit IRC06:30
*** PollyZ has joined #openstack-ironic06:33
*** Sukhdev has quit IRC06:33
*** purp has joined #openstack-ironic06:34
*** vishwanathj has quit IRC06:36
*** vishwanathj has joined #openstack-ironic06:36
*** PollyZ has quit IRC06:37
*** loki_ has joined #openstack-ironic06:44
*** openstackgerrit has quit IRC06:48
*** gcb has quit IRC06:48
*** openstackgerrit has joined #openstack-ironic06:48
*** rbartal has joined #openstack-ironic06:53
*** sdake has joined #openstack-ironic06:54
*** itamarl has joined #openstack-ironic06:57
*** priteau has joined #openstack-ironic06:59
*** tesseract has joined #openstack-ironic07:00
*** e0ne has quit IRC07:06
*** sdake has quit IRC07:10
*** rbartal has quit IRC07:13
*** gcb has joined #openstack-ironic07:13
*** PollyZ has joined #openstack-ironic07:14
*** rbartal has joined #openstack-ironic07:14
*** PollyZ has quit IRC07:19
*** mtanino has quit IRC07:19
*** ifarkas has joined #openstack-ironic07:20
*** watanabe_isao has joined #openstack-ironic07:20
*** sacharya has joined #openstack-ironic07:24
*** mjura has joined #openstack-ironic07:25
*** sacharya has quit IRC07:28
*** watanabe_isao has quit IRC07:31
openstackgerritvinay kumar muddu proposed openstack/ironic-python-agent: Wait for at least one interface before node lookup  https://review.openstack.org/30350307:32
*** amotoki has joined #openstack-ironic07:36
*** amotoki has quit IRC07:41
*** dtantsur|afk is now known as dtantsur07:44
dtantsurMorning Ironic07:44
*** openstackgerrit has quit IRC07:48
*** openstackgerrit has joined #openstack-ironic07:48
openstackgerritvinay kumar muddu proposed openstack/ironic-python-agent: Wait for at least one interface before node lookup  https://review.openstack.org/30350307:53
openstackgerritZhenguo Niu proposed openstack/ironic-lib: Include wipefs --force option  https://review.openstack.org/31991807:53
*** PollyZ has joined #openstack-ironic07:55
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Return HTTP 500 from /v1/continue on unexpected exceptions  https://review.openstack.org/32689107:56
*** PollyZ has quit IRC08:00
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:02
*** jtomasek has joined #openstack-ironic08:05
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844008:10
*** lucas-dinner is now known as lucasagomes08:10
lucasagomesmorning all08:10
dtantsurhey morning lucasagomes08:11
*** bcornec1 has joined #openstack-ironic08:12
*** vishwanathj has quit IRC08:14
vmud213morning lucasagomes,dtantsur and all08:19
*** rcarrillocruz has joined #openstack-ironic08:20
bcornec1Hello again !08:22
*** aarefiev_ is now known as aarefiev08:26
aarefievmorning!08:26
*** ohamada has joined #openstack-ironic08:27
*** kromanenko__ has joined #openstack-ironic08:28
*** Romanenko_K has joined #openstack-ironic08:28
*** kromanenko_ has joined #openstack-ironic08:28
pas-hamorning Ironic08:28
rcarrillocruzmorning folks08:33
rcarrillocruzquestion08:33
rcarrillocruzcan i add arbitrary to ironic nodes metadata (or properties) ?08:33
rcarrillocruzor only a set of keys that is validated in the backend?08:33
alinebmorning all08:34
*** PollyZ has joined #openstack-ironic08:36
*** dmk0202 has joined #openstack-ironic08:37
*** amotoki has joined #openstack-ironic08:37
*** kalpase1 has joined #openstack-ironic08:38
*** kalpase1 has left #openstack-ironic08:38
*** kalpase has quit IRC08:39
*** PollyZ has quit IRC08:41
*** amotoki has quit IRC08:42
*** derekh has joined #openstack-ironic08:42
*** kalpase has joined #openstack-ironic08:44
*** openstackgerrit has quit IRC08:48
*** openstackgerrit has joined #openstack-ironic08:48
*** PollyZ has joined #openstack-ironic08:49
*** e0ne has joined #openstack-ironic08:50
*** PollyZ has quit IRC08:54
*** vishwanathj has joined #openstack-ironic08:55
*** dtantsur is now known as dtantsur|bbl08:57
*** _milan_ has joined #openstack-ironic08:59
_milan_morning Ironic! :)09:00
rcarrillocruzwow, i wrote it really badly09:03
rcarrillocruzcan i add arbitrary *metadata/properties* to ironic nodes?09:03
*** jtomasek has quit IRC09:04
*** electrofelix has joined #openstack-ironic09:13
lucasagomesrcarrillocruz, yup, it's a json field so you basically can add whatever you want09:14
lucasagomessame for other json fields (extra, driver_info, etc...)09:14
rcarrillocruzexcellent, thx lucasagomes09:14
*** sdake has joined #openstack-ironic09:16
*** PollyZ has joined #openstack-ironic09:16
lucasagomesyw09:17
*** chenke has joined #openstack-ironic09:17
*** jtomasek has joined #openstack-ironic09:18
*** karthi has joined #openstack-ironic09:18
karthiHi Ironics09:18
karthiguys, i need help in applying the fix https://bugs.launchpad.net/glance/+bug/156194709:20
openstackLaunchpad bug 1561947 in Glance "Glance Store fails to authenticate to Swift with Keystone v3 API" [High,In progress] - Assigned to Kairat Kushaev (kkushaev)09:20
*** PollyZ has quit IRC09:21
*** chenke has quit IRC09:21
lucasagomeskarthi, I think you may want to check the #openstack-glance channel09:24
*** sacharya has joined #openstack-ironic09:24
*** sacharya has quit IRC09:29
*** PollyZ has joined #openstack-ironic09:30
*** PollyZ has quit IRC09:35
*** amotoki has joined #openstack-ironic09:38
sambetts|afkMorning all o/09:41
*** sambetts|afk is now known as sambetts09:41
*** mgould|afk is now known as mgould09:41
mgouldmorning sambetts09:41
mgouldmorning all :-)09:41
sambettshey mgould09:41
*** amotoki has quit IRC09:42
*** PollyZ has joined #openstack-ironic09:44
lucasagomesmgould, sambetts morning09:44
mgouldmorning lucasagomes09:44
vdrokmorning ironic, sambetts mgould lucasagomes09:44
mgouldmorning vdrok09:45
*** PollyZ has quit IRC09:49
*** sdake has quit IRC09:49
openstackgerritVladyslav Drok proposed openstack/ironic: Gracefully degrade start_iscsi_target for Mitaka ramdisk  https://review.openstack.org/31918309:55
*** amotoki has joined #openstack-ironic09:56
*** PollyZ has joined #openstack-ironic09:57
*** PollyZ has quit IRC10:02
*** vishwanathj has quit IRC10:08
*** dtantsur|bbl is now known as dtantsur10:11
dtantsurmorning vmud213, aarefiev, pas-ha, sambetts, vdrok10:11
*** PollyZ has joined #openstack-ironic10:11
vdrokgood morning dtantsur !10:11
sambettsMorning vdrok dtantsur10:11
*** mjura has quit IRC10:12
aarefievmorning dtantsur, sambetts, vdrok10:13
vdroko/ aarefiev10:14
*** mjura has joined #openstack-ironic10:15
*** PollyZ has quit IRC10:16
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Tests for testing node-create command  https://review.openstack.org/26205510:18
*** hoangcx has quit IRC10:22
mgouldmorning dtantsur10:23
dtantsurmorning mgould10:24
*** PollyZ has joined #openstack-ironic10:25
openstackgerritVladyslav Drok proposed openstack/ironic: Fix nits related to Ports api-ref  https://review.openstack.org/32697810:25
*** PollyZ has quit IRC10:29
*** mbound has joined #openstack-ironic10:29
*** lifeless has joined #openstack-ironic10:35
vmud213morning sambetts,vdrok,mgould,aarefiev10:37
mgouldmorning vmud21310:38
*** PollyZ has joined #openstack-ironic10:38
vdrokhi vmud21310:39
*** loki_ has quit IRC10:40
*** PollyZ has quit IRC10:43
sambettsHi aarefiev, vmud21310:46
*** lifeless has quit IRC10:47
*** lifeless_ has joined #openstack-ironic10:47
*** tesseract has quit IRC10:50
*** karthi has quit IRC10:51
*** tesseract has joined #openstack-ironic10:51
*** PollyZ has joined #openstack-ironic10:52
*** PollyZ has quit IRC10:56
*** d0ugal has quit IRC10:59
*** d0ugal has joined #openstack-ironic11:00
openstackgerritDmitry Tantsur proposed openstack/ironic: Document packing and unpacking the deploy ramdisk  https://review.openstack.org/32699511:01
dtantsurlucasagomes, ^^^11:01
dtantsurI've found an easy way to not forget it any more :)11:01
lucasagomesdtantsur, hah good, will take a look11:01
mgoulddtantsur++11:02
sambettsdtantsur, lucasagomes: I'm tempted to make the pack part of the tinyipa build a separate make alias, so you could modify the tinyipa final dir, then just run make pack or something11:03
sambettsso then the manual workflow would be make build, make finalise, make pack11:03
lucasagomessambetts, yeah that's good too11:03
sambettsor something11:03
lucasagomestho the commands that dtantsur uses apply to all ramdisks11:04
lucasagomesI mean, most of them... you may still need to use unsquashfs for some11:04
sambettsthere are a couple of parts of the ramdisk build that kinda apply to all ramdisks, like the make iso part11:04
lucasagomesand I don't think it works with dracut microcode11:04
mgouldsambetts: have `pack` depend on `finalise` depend on `build`, so you can just run `make pack`11:04
mgoulduse the dependency-tracking system for what it's good at :-)11:05
*** PollyZ has joined #openstack-ironic11:05
sambettsmgould: I have it so that the default, "make" does build, finalise right now and "make all" does build, finalise, iso11:06
* mgould nods11:06
*** stendulker has quit IRC11:06
* sambetts hasn't much experience with complex make files 11:07
* mgould has some experience with them and is happy to help11:07
dtantsurI think this documentation update is at least useful for people who already have the ramdisks now11:08
sambetts++11:08
dtantsure.g. I get this question downstream pretty often :)11:08
mgouldso "pack" means "make a cpio.gz file"11:08
mgouldand that's required by `make iso`?11:08
sambettsmgould: yup11:08
*** d0ugal has quit IRC11:09
sambettstbh it should really be "make tinyipa.gz" to take advantage of makes existing file check11:09
*** bcornec1 has quit IRC11:09
mgouldyep11:09
*** phuongnh has quit IRC11:10
mgouldbut we could easily add a `pack` target which depends on tinyipa.gz, if it would make users' lives easier11:10
*** PollyZ has quit IRC11:10
*** e0ne_ has joined #openstack-ironic11:11
sambettsI guess its a more readable targe t11:11
sambettsmgould: tbh I thought about break all the environment variables out of the scripts for the tinyipa build, and having them sourced/configured in the make file or something, because several of them are shared/defined in both scripts11:13
*** e0ne has quit IRC11:13
mgouldright, yep11:15
mgoulddoes the build process depend on the user's environment? That's not good11:15
sambettsmgould: there are defaults for all the variables11:17
sambettsbut they can be overridden by setting env varaibles11:18
mgouldright, cool11:18
sambettsfor example export BUILD_AND_INSTALL_TINYIPA=false will override the default true11:18
dtantsurlucasagomes, what do you mean by "this also doesn't work with ramdisks built with dracut microcode"? this is known to work with our downstream ramdisks based on dracut..11:19
*** PollyZ has joined #openstack-ironic11:19
openstackgerritDmitry Tantsur proposed openstack/ironic: Document packing and unpacking the deploy ramdisk  https://review.openstack.org/32699511:23
dtantsuradded a note about coreos ^^^11:23
* dtantsur <= coffee11:23
dtantsur#pixiesay time for coffee!11:23
PixieBootsʕ•͡ᴥ•ʔ: time for coffee!11:23
*** PollyZ has quit IRC11:24
lucasagomesdtantsur, https://access.redhat.com/solutions/90105311:24
lucasagomesdtantsur, depends on how it's built11:25
lucasagomesif it does use microcode or not11:25
*** sacharya has joined #openstack-ironic11:25
lucasagomesI wouldn't -1 for that, it's not a common practice to build the deploy ramdisks with microcode, but possible11:25
jrollJayF: doh, thought I landed that :/ thanks11:28
jrollmorning everyone11:28
*** sacharya has quit IRC11:30
sambettsMorning jroll11:30
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Negative tests for testing chassis-create command  https://review.openstack.org/32278811:30
lucasagomesjroll, morning11:31
*** d0ugal has joined #openstack-ironic11:32
* lucasagomes goes grab some lunch11:35
*** lucasagomes is now known as lucas-afk11:35
*** baoli has joined #openstack-ironic11:40
vmud213dtantsur: For the packing and unpacking od deploy image, should not it be done with root?11:41
*** baoli_ has joined #openstack-ironic11:41
vmud213dtantsur:Or may be with sudo?11:41
*** Goneri has joined #openstack-ironic11:42
*** sdake has joined #openstack-ironic11:44
*** baoli has quit IRC11:45
*** gcb has quit IRC11:46
*** PollyZ has joined #openstack-ironic11:46
*** kalpase has left #openstack-ironic11:47
*** yolanda has joined #openstack-ironic11:49
openstackgerritMerged openstack/proliantutils: Passing 'process_input' to the running hpssacli process  https://review.openstack.org/32573011:50
*** PollyZ has quit IRC11:51
openstackgerritMerged openstack/ironic: Add 'How to get a decision on something' to FAQ  https://review.openstack.org/32124611:52
dtantsurvmud213, no11:56
dtantsurlucas-afk, thanks! strangely enough, our ramdisks are not like that. I've packed and unpacked them numerous times11:57
dtantsurmorning jroll! do you remember why we didn't release IPA mitaka 1.2.2?11:58
vsaienko1Morning lucasagomes, dtantusr, vmud213, sambetts, vdrok and all Ironic'ers11:58
dtantsurhey vsaienko111:58
*** ifarkas has quit IRC11:58
*** PollyZ has joined #openstack-ironic12:00
openstackgerritSam Betts proposed openstack/ironic: [WIP] Add Network Interface Attach and Detach APIs  https://review.openstack.org/32704612:00
jrolldtantsur: https://review.openstack.org/#/c/324884/12:00
dtantsurah awesome :)12:00
jrollin fact, https://review.openstack.org/#/q/project:openstack/releases+owner:%22Jim+Rollenhagen+%253Cjim%2540jimrollenhagen.com%253E%22+status:open12:01
*** bcornec1 has joined #openstack-ironic12:01
jrollmatt just got to them yesterday so they should go out today I think12:01
dtantsurcool, thanks, jroll12:02
*** vmud213 has quit IRC12:02
dtantsurheh, inspector release features only requirements :) I guess we still have to do it, right?12:02
*** PollyZ has quit IRC12:05
jrolldtantsur: yeah, it doesn't hurt12:05
jrollI just did everything stablel because why not12:05
dtantsuranyway, release often :)12:06
*** ifarkas has joined #openstack-ironic12:06
jroll++12:06
jrollI'm releasing ironic 6.0 as soon as grenade is done12:06
dtantsurwoohoo12:06
jrolls/done/working/12:06
sambetts\o/12:06
dtantsurjroll, do you want to wait for us to drop the old ramdisk? initially we planned it for 6.012:06
*** ayoung has quit IRC12:07
jrolldtantsur: oh yeah ++12:08
jrollare you working on that?12:08
dtantsurjroll, no, but if we want it soon, I can start right now12:09
dtantsurI was wondering if we want grenade first..12:09
jrolldtantsur: idk, I think it's fine without, we're only removing things and we don't test the bash ramdisk12:11
jrollmight be useful though, could base it on that patch12:11
*** clenimar has joined #openstack-ironic12:11
dtantsurjroll, ok, I'll start hacking, and then we decide if we want grenade first or not before landing it12:12
dtantsurwill start today, just let me enjoy my bit of tripleo-heat-templates debugging >_<12:12
jrolldtantsur: "enjoy" :)12:12
jrollthanks12:12
dtantsuroh, I do :D12:12
mgouldsounds like Type 2 fun: http://www.urbandictionary.com/define.php?term=type%202%20fun12:13
dtantsurjroll, actually, I'm working on making tripleo install ironic in overcloud. so it's a bit interesting as well12:13
jrolloh, cool12:13
*** sdake has quit IRC12:13
dtantsuralso expect me to have much more questions about serving bare metals to tenants :D12:13
jrollhehe12:13
dtantsurTIL type 2 fun, looks very related, thanks mgould12:13
*** frontrunner has joined #openstack-ironic12:14
*** dprince has joined #openstack-ironic12:15
rbartaldtantsur, lucas-afk jroll Hi all, I open a bug about the failed tear down after cleaning we talked about yesterday12:16
rbartal #1590408 Node cleaning fails with 'Failed to tear down from cleaning for node UUID'12:16
*** itamarl has quit IRC12:16
dtantsurthnx!12:16
jrollty rbartal12:17
rbartaljroll, BTW, it is happens every time, I tested it on BM and virt12:18
rbartal*it happens12:18
jrollO_o12:18
vdrokmorning jroll, a question, have you had a chance to look at https://review.openstack.org/#/q/topic:ipa-partition-images ?12:19
jrollI wonder if it's a combination of the user in the original context not having access to list networks in neutron, and the bug sambetts mentioned12:19
jrollvdrok: I have not, sorry12:19
vdrokjroll: it's pretty small, the only thing I'm wondering is - is it ok to use ephemeral size to pick the image for tempest tests, eg if ephemeral size == 0, use whole disk, otherwise use partition image12:20
jrollvdrok: hmm, probably? not sure really12:21
vdrokor better to introduce another parameter to request either whole disk or not. but our builder in project-config really becoming huge :)12:22
jrollyeah, I agree, I think I'm fine with it12:22
jrolldtantsur: ^ wdyt12:22
dtantsurjroll, we definitely have to do something about our project-config, but otherwise I'm fine with it12:23
jrollyeah12:23
dtantsurwe already have pxe vs ipxe chosen implicitly12:23
*** jcoufal has joined #openstack-ironic12:23
jrollI have on my todo list to write a plan for that12:23
vdrokdtantsur: jroll OK, will unwip these then12:23
jrolland for splitting api tests to a separate job12:23
jrolland maybe most jobs testing deploys without nova? many of these options shouldn't affect nova12:24
dtantsurjroll, my next question was if we plan to use simple testing (bifrost based?) for most of the jobs :)12:26
*** PollyZ has joined #openstack-ironic12:27
jrolldtantsur: yeah, I'd like to, probably just build a deploy() method into our tempest client12:27
dtantsurcool12:27
vdrokjroll: dtantsur btw I have this old patch - https://review.openstack.org/#/c/293434/1/ironic_tempest_plugin/tests/scenario/test_baremetal_standalone_ops.py@209, maybe need to fix it properly and move it to base baremetal_manager12:29
jrollvdrok: ++12:30
*** trown|outtypewww is now known as trown12:31
*** PollyZ has quit IRC12:32
*** itamarl has joined #openstack-ironic12:33
mat128vdrok: looks pretty much like what we need :)12:33
vdrokmorning mat12812:34
mat128good morning :)12:34
mgouldmorning mat12812:35
*** jjohnson2 has joined #openstack-ironic12:39
*** vishwanathj has joined #openstack-ironic12:41
*** PollyZ has joined #openstack-ironic12:41
TheJuliagood morning12:43
*** lucas-afk is now known as lucasagomes12:44
xavierrgood morning Ironic12:44
xavierrmorning dtantsur vdrok mat128 mgould TheJulia12:45
lucasagomesrbartal, thanks12:45
xavierrgood morning lucasagomes o/12:45
lucasagomesxavierr, hi there, good morning12:45
*** PollyZ has quit IRC12:45
jrollsambetts: neutron still doesn't have wild-card dhcp, right?12:47
sambettsjroll: no, and annoying the reviews on my specs for it have stalled12:48
sambettsannoyingly*12:48
*** thrash has quit IRC12:48
jrollsambetts: naturally12:48
*** thrash has joined #openstack-ironic12:50
*** thrash has quit IRC12:50
*** thrash has joined #openstack-ironic12:50
*** PollyZ has joined #openstack-ironic12:54
*** PollyZ has quit IRC12:59
openstackgerritChuck Carmack proposed openstack/bifrost: Add redeploy-dynamic playbook  https://review.openstack.org/32537912:59
*** ccarmack has joined #openstack-ironic13:00
*** nico-afk is now known as nicodemos13:01
nicodemosgood morning13:01
*** links has quit IRC13:03
openstackgerritLucas Alvares Gomes proposed openstack/ironic-specs: Collect system logs from IPA  https://review.openstack.org/32351113:09
lucasagomesdtantsur, ^ updated to cover the always retrieve case ^13:09
*** rloo has joined #openstack-ironic13:10
TheJuliagood morning rloo13:10
lucasagomesnicodemos, TheJulia morning13:10
rloohi and good morning TheJulia, lucasagomes13:11
*** alexpilotti has joined #openstack-ironic13:11
lucasagomesrloo, hello there, good morning13:12
dtantsurlucasagomes, cool!13:12
dtantsurmorning TheJulia, rloo, xavierr, nicodemos13:12
rloohi dtantsur13:13
*** priteau has quit IRC13:14
nicodemoshey, dtantsur o/13:14
*** priteau has joined #openstack-ironic13:15
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP] Remove support for the old ramdisk (DIB ironic-deploy element)  https://review.openstack.org/32708313:15
dtantsurhere it starts ^^^13:15
dtantsur(now fix/remove all the unit tests)13:15
sambetts:D13:15
dtantsurreviews are still welcome, I'm sure I missed something13:15
*** links has joined #openstack-ironic13:18
*** ppiela has joined #openstack-ironic13:18
*** watanabe_isao has joined #openstack-ironic13:20
*** rbudden has joined #openstack-ironic13:23
*** ayoung has joined #openstack-ironic13:26
*** alexpilo_ has joined #openstack-ironic13:29
*** alexpilotti has quit IRC13:32
*** PollyZ has joined #openstack-ironic13:35
openstackgerritChris Krelle proposed openstack/ironic-python-agent: Add basic troubleshooting doc  https://review.openstack.org/32674313:39
NobodyCamGood morning Ironic13:39
*** PollyZ has quit IRC13:41
dtantsurmorning NobodyCam13:41
lucasagomesNobodyCam, hi there, morning13:43
*** mbound has quit IRC13:43
NobodyCam:) good morning dtantsur and lucasagomes :)13:43
NobodyCamhappy mid week :)13:43
mgouldmorning NobodyCam13:44
NobodyCamgoodmorning mgould :)13:44
* TheJulia needs lots of coffee this morning13:48
mgouldmorning TheJulia13:49
NobodyCam+++ I was woken up at like 5:20 am this morning :p ....13:49
* mgould needs coffee too, and it's mid-afternoon here :-(13:49
mgouldI can beat that13:49
NobodyCammorning TheJulia13:49
mgould033013:49
NobodyCammgould: ugh13:49
mgould<- not very together today13:49
NobodyCammgould: ++++13:50
NobodyCamalso nap's help ! :)13:50
mgouldI did get a bit more sleep after that, but the morning was a cascade of fail13:50
mgouldyeah, I think a nap may be the way forward13:50
*** PollyZ has joined #openstack-ironic13:50
NobodyCam:)13:50
*** watanabe_isao has quit IRC13:54
rloomorning NobodyCam, hi mgould13:56
mgouldhi rloo13:56
openstackgerritDmitry Tantsur proposed openstack/ironic: Remove support for the old ramdisk (DIB ironic-deploy element)  https://review.openstack.org/32708313:59
dtantsurplease review carefully ^^^13:59
dtantsurjroll, ^^^13:59
*** clenimar has quit IRC13:59
NobodyCammorning rloo :)14:01
*** ametts has joined #openstack-ironic14:02
*** clenimar has joined #openstack-ironic14:04
vdrokmorning TheJulia xavierr rloo and NobodyCam14:05
NobodyCammorning vdrok :)14:06
jrolldtantsur: ++14:06
NobodyCammorning jroll :)14:06
jrollhey NobodyCam :)14:06
*** watanabe_isao has joined #openstack-ironic14:06
NobodyCam:)14:07
rloohi vdrok!14:08
*** yolanda_ has joined #openstack-ironic14:08
*** yolanda has quit IRC14:09
*** yolanda_ has quit IRC14:09
*** yolanda has joined #openstack-ironic14:09
*** vsaienko1 has quit IRC14:12
*** vsaienko has joined #openstack-ironic14:13
rbartalTheJulia, HI there, I have a question about "14:13
TheJuliaabout " ?14:13
NobodyCamhehehehe14:13
NobodyCamTheJulia: would that be ' " '?14:13
* TheJulia does not know the the historical context to speak in regards to ' " ' ;)14:14
TheJuliarbartal: Whats up?14:14
NobodyCamit's one of those mornings.14:14
rbartalTheJulia, Sorry, Thick fingers14:14
TheJuliaIndeed!  The caffeine is just hitting my system now14:14
TheJuliarbartal: no worries!14:15
rbartalTheJulia, I'm testing clean steps and fails with Security Freeze Lock14:15
*** notq_ has joined #openstack-ironic14:15
* TheJulia has sudden urge for music of a land she can never really visit.14:15
TheJuliarbartal: when trying to do ?erase_drives? ?14:17
rbartalTheJulia, I did try the SSD with intel toolbox and re try to run the erase_device again and got the same error14:17
TheJuliarbartal: can you collect the hdparm output from the device and share it with us?14:18
rbartalTheJulia, sure, but it will take some time..14:18
TheJuliaAlso, what kind of hardware are you using? what I guess your using a recent IPA :)14:19
TheJuliarbartal: s/what//14:19
TheJuliaat least, the second what14:19
* TheJulia may need more coffee14:19
TheJuliarbartal: I understand it is not the easiest to get that info from a machine, but i would suspect your bios is locking the device upon boot and the password is not NULL :\14:21
rloohi, this has two +2s, does anyone want to take a look? otherwise, I'll +A it: https://review.openstack.org/#/c/319183/14:23
TheJulialooking14:23
*** jcoufal has quit IRC14:25
*** rama_y has joined #openstack-ironic14:26
rbartalTheJulia, yes the bois password is null, the SSD is intel s3510 series 120GB, and the osp is 914:27
TheJuliarbartal: what is the exact error your getting in the last_error field?14:28
dtantsur(me notes that OSP 9 is Mitaka)14:28
TheJuliadtantsur: thank you for the context14:28
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Add config fixture to functional tests  https://review.openstack.org/32713014:29
*** ijw has joined #openstack-ironic14:29
*** links has quit IRC14:29
*** ChrisAusten has joined #openstack-ironic14:30
TheJuliadtantsur: so I take it that IPA patch that was backported while I was on leave is likely in rbartal's build?14:30
*** notq_ has quit IRC14:30
dtantsurTheJulia, no, it definitely is not. it wasn't even released yet14:31
TheJuliaok, that is good to know14:31
dtantsurofc we'll rebase as soon as possible14:31
*** lintan has quit IRC14:31
TheJulianaturally :)14:31
*** alexpilo_ has quit IRC14:32
*** lintan has joined #openstack-ironic14:32
openstackgerritYosef Hoffman proposed openstack/ironic-python-agent: [WIP] Add configuration options for DISK_WAIT  https://review.openstack.org/32490914:32
jrollrloo: +A'd14:34
TheJuliaheh, Well, I just +2'ed it as well :)14:34
dtantsur\o/ was it the last ironic patch before we can proceed with grenade?14:34
*** jaybeale has joined #openstack-ironic14:34
jrolldtantsur: yep!14:35
jrollthen it's just this one https://review.openstack.org/#/c/316662/14:35
rloothx jroll and TheJulia!14:35
TheJuliaHas anyone asked infra regarding that one?14:35
jrollI have not14:36
TheJuliaI'll ask14:36
rlooTheJulia: dunno. I didn't. Seems like a straightforward one.14:36
vsaienkomorning jroll, rloo, TheJulia14:36
*** joprovost has joined #openstack-ironic14:37
jrollTheJulia: oops, too late14:37
rloomorning vsaienko14:37
*** piet has joined #openstack-ironic14:37
TheJuliajroll: no worries, my manager pinged me in another window as I was typing the message in the infra room, thank you14:37
jrollheh, cool14:37
rbartalTheJulia, this is the node-show output http://paste.openstack.org/show/508947/14:38
jrolldtantsur: TheJulia: JayF: you know if we have plans to use this? https://review.openstack.org/#/c/326203/114:39
dtantsurjroll, it's the first time I see it Oo14:40
jrolldtantsur: looks like it's meant for bifrost14:40
jroll6 months ago14:40
jrollbut was never used14:40
* jroll needs TheJulia input on this one14:41
*** jaybeale has quit IRC14:41
*** clenimar has quit IRC14:41
rbartalTheJulia, dtantsur I also have the ironic-conductor.log, how can I share a file? or it's only copy-paste?14:43
*** clenimar has joined #openstack-ironic14:45
openstackgerritDmitry Tantsur proposed openstack/ironic: Remove support for the old ramdisk (DIB deploy-ironic element)  https://review.openstack.org/32708314:45
dtantsurvdrok, fixed the element name ^^14:45
vdrokdtantsur: thanks, otherwise I haven't found anything else, will wait for the gate :)14:46
*** bcornec1 has quit IRC14:46
*** mtanino has joined #openstack-ironic14:49
*** watanabe_isao has quit IRC14:51
rbartalTheJulia, which command line argument should I use for the hdparm?14:54
TheJuliarbartal: as root, or via sudo: hdparm -I /dev/sda14:55
*** jcoufal has joined #openstack-ironic14:55
*** clenimar has quit IRC14:56
vsaienkodtantsur, vdrok, lucasagomes please review https://review.openstack.org/#/c/299229 https://review.openstack.org/#/c/32595214:56
rbartalTheJulia, http://paste.openstack.org/show/508952/14:56
lucasagomesvsaienko, sure thing! 1 sec14:57
*** links has joined #openstack-ironic14:57
*** thiagop has joined #openstack-ironic14:59
thiagopGood morning, Ironicers14:59
NobodyCammorning thiagop14:59
TheJuliajroll: all for removing it for many reasons()14:59
*** clenimar has joined #openstack-ironic15:00
vsaienkomorning thiagop15:00
jrollTheJulia: cool, thanks15:01
*** clenimar has quit IRC15:01
mgouldmorning thiagop15:01
thiagopo/ NobodyCam vsaienko mgould15:01
rbartal15:02
TheJuliarbartal: checking a few things, seems, one moment15:02
rbartalTheJulia, thanks15:02
rbartaluit15:02
*** ijw has quit IRC15:02
*** ijw has joined #openstack-ironic15:03
*** itamarl has quit IRC15:06
*** e0ne_ is now known as e0ne15:11
TheJuliarbartal: https://wiki.archlinux.org/index.php/SSD_memory_cell_clearing may be worth reading, it has refreshed my memory quite well.  It is likely the bios asserting it, and it is a state that IPA is unable to recover from, unless you have two revisions in your build, one which has yet to land... and even then that only allows you to fallback to disk shredding/wiping out with zeros which ultimate15:12
TheJulialy is really bad for an SSD15:12
JayFjroll: I hope someone has plans to do a DIB image in a job, but I'm not aware of anyone actively working on it.15:12
JayFjroll: did someone look at the commit that added it? is ther person around still?15:13
*** dprince has quit IRC15:13
TheJuliaJayF: I started down that path, but all the random dib failures due to mirrors/repos, single distro specific changes that broke other distros just honestly frustated me beyond belief.15:13
*** piet has quit IRC15:14
JayFTheJulia: yeah; I just think at some point we're going to have to document supported vs unsupported15:14
TheJuliaJayF: on a plus side, bifrost has a non voting job for building it with debian so we can see if it is broken and try and provide some visibility15:14
JayFand if the dib image isn't gated; it's not supported15:14
TheJulia++15:14
JayFand even if it's supported, I'd be curious how we'd handle things like new deps as it's not in the repo15:14
TheJuliaJayF: I think part of the conundrum is some operators are going to need specific distros because of specific hardware drivers, so we're going to have to kind of treat it as best effort15:16
JayFI mean, you're talking to someone who runs tools designed for old as hell centos on coreos15:16
TheJuliabut, we truly can't check all the distros unless it is a job in the dib repo that gates that element and tooling15:16
JayFso you can tell which problem I chose, and DIB seemed more intimidating15:16
JayFlol15:16
TheJulialol15:16
TheJuliacinerama: ^^^ hint hint15:16
TheJuliaw/r/t gating/jobs15:17
*** rpioso has joined #openstack-ironic15:18
*** joprovost has quit IRC15:19
*** rama_y has quit IRC15:19
NobodyCammorning JayF15:19
JayFgm15:19
*** davidlenwell has quit IRC15:20
rbartalTheJulia, OK, so the SSD is locked and IPA can not secure delete it?15:20
TheJuliatruthfully I feel really embarassed that I forgot about the postbuild job, but I think there was a solid two month span where the post job would hit a transient failure somewhere along the way of retrieving a package.15:21
TheJuliarbartal: that is basically the gist of your issue.  It might be a bios setting in your hardware that you can control, I know with one of my lab desktops it is actually a setting I can toggle15:21
rbartalTheJulia, how can we tell its not the IPA that trigger the lock?15:22
TheJulias/embarassed/embarassd and ashamed/15:22
JayFrbartal: we have code in IPA that attempts to unlock it15:22
TheJuliaJayF: rbartal does not have the unlock code yet15:23
rbartalTheJulia,  I run this procedure on the disk and it worked http://www.intel.com/content/www/us/en/support/software/000006094.html15:23
TheJuliaand it is not locked http://paste.openstack.org/show/508952/ :\15:23
*** yohoffman has joined #openstack-ironic15:23
JayFJust making it clear; if IPA locked the disk, it will recover on it's own: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/hardware.py#L74715:24
JayFyou added that code and it's muy bueno15:24
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Add config fixture to functional tests  https://review.openstack.org/32713015:24
rbartalTheJulia, OK, so best is to try and run manual lock / unlock procedure and wait for the patch15:25
TheJuliarbartal: try to unfreeze, reboot to another os, and inspect the drive parameters after power cycling.  I suspect you'll see it frozen again15:25
NobodyCamlucasagomes: would you happen to have any time to take a quick look at: https://bugs.launchpad.net/ironic/+bug/159014615:25
openstackLaunchpad bug 1590146 in Ironic "A timed out manual cleaning cannot be retried successfuly" [Undecided,New]15:25
*** sukhdev has joined #openstack-ironic15:25
lucasagomesNobodyCam, yup, 1 sec15:25
NobodyCamlucasagomes: you awesome .... Thank you :)15:26
NobodyCams/you/your/15:26
alinebJayF: regarding your comment on the metric patch, does that mean the get_logger method should go in a separate module file too?15:26
JayFalineb: I mean, it's just a nit.15:26
JayFalineb: I'm just thinking if I was adding some new metrics logger, I'd like to have the already implemented one in a separate module15:27
JayFalineb: but it's seriously just a nit, and if you disagree, ignore it15:27
JayFlol15:27
alinebJayF: it's not that I disagree, more that i'm not sure how to do it :)15:28
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Update terribly outdated installation instructions  https://review.openstack.org/32718715:28
JayFI mean, I was just thinking put the Statsdmetricslogger class in it's own module15:28
dtantsurmgould, could you please take a look ^^^15:28
JayFbasically if I wanted to make a jaymetricslogger, I could copy the file and start there as a template15:28
rbartalTheJulia, Thanks, and have a good day (I'm on UTC +3)15:29
mgoulddtantsur: looking now15:29
*** dmk0202 has quit IRC15:29
alinebJayF: yes, i think i understand what you meant, my problem is that the Statsdmetricslogger would need to import the base class, and that getLogger needs to import Statsdmetricslogger15:31
alinebwhich explains my questions about getlogger in its own module file (or in utils.py)15:31
JayFthat sort of problem is why it's maybe nice to split it out now15:31
JayFso you can make the next one easier before having to worry about backwards compat15:32
lucasagomesNobodyCam, TheJulia interesting, it sounds like it affects automatic cleainng as well?15:32
lucasagomesNobodyCam, TheJulia I think the proposed solution looks sane too, upon moving it to fail state we should clean up the fields so the operation can be retried15:32
alinebJayF: ok, i will do that then15:33
*** rbartal has quit IRC15:33
NobodyCam++15:33
NobodyCamI did not test with automated cleaning the stuff I looked at was all manual15:34
NobodyCamhowever I assume that would be the same15:34
JayFlucasagomes: NobodyCam: context? Do we have a bug up around cleaning?15:35
lucasagomesJayF, yeah, NobodyCam just pointed me out to https://bugs.launchpad.net/ironic/+bug/159014615:35
openstackLaunchpad bug 1590146 in Ironic "A timed out manual cleaning cannot be retried successfuly" [Undecided,New]15:35
lucasagomesNobodyCam, the periodic task that catches the time out is the same, so I assume that too15:35
JayFoh my15:35
JayFthat explains downstream behavior15:35
JayFjcook: ^ bug 1590146 is probably responsbile for some of our stuck cleanings15:36
NobodyCam:)15:36
JayFThat has an even scarier side effect I think, would it ever be possible for that bug to cause a clean step to be skipped?15:37
NobodyCamJayF: the bug or the fix?15:39
JayFThe bug.15:39
JayFI guess at worse it'd cause them to repeat15:39
NobodyCamyea, we had a node wedged to such a point the only fix was edit the DB15:40
JayFNobodyCam: even manage/provide won't clear the clean step either15:41
JayFman this is insideous15:41
NobodyCamyes15:41
JayFthe only reason we haven't gotten wedged this hard downstream is generally our clean steps are idempotent15:41
TheJulialucasagomes: NobodyCam: JayF: Yeah, I think it could impact automated cleaning if you have something that could get it wedged into a timeout situation15:43
JayFabsolutely it impacts automatd cleaning15:43
JayFI already updated the bug to reflect that15:43
TheJulialess likely if it was just doing erase_drives, but you guys have a lot more going on than most15:43
lucasagomesyeah it seems to do that15:43
JayFYep. Like I said the only reason we haven't been crushed by this downstream is b/c our steps are all idempotent15:44
TheJuliaJayF: awesome15:44
JayFbut I've def seen this behavior before, and couldn't figure it out and it sorta "solved itself" (because our steps are a-ok with rerunning)15:44
* NobodyCam looks at flash bios step and shutters just bit.15:45
TheJuliayeah, in this case we found it with with a dib based raid utility build that OOM'ed a machine and we're like "what?" "huh?" "Why is this not working?"15:45
*** sacharya has joined #openstack-ironic15:47
JayFWhat's the rules with bug triage? Like can I JFDI and assign an importance to this bug and mark it confirmed?15:52
JayFor is that dtantsur's domain15:52
dtantsurJayF, you can (and should :)15:52
JayFmarking it as high and confirming it15:54
JayFis someone going to work on this today and fix?15:54
*** davidlenwell has joined #openstack-ironic15:56
*** sabeen has joined #openstack-ironic15:58
*** chlong has quit IRC15:58
TheJuliaJayF: I will16:01
jrollJayF: dtantsur gets bored, let him do it16:01
jroll:D16:01
JayFTheJulia loses the game of chicken :)16:02
dtantsurLOL16:02
JayFif you had waited 5 more minutes I might have volunteered ;)16:02
jrollheh16:03
rlooTheJulia has already thought about the fix, so I'm sure she really wants to fix it! ;)16:04
JayFI think TheJulia has like, 2 weeks of boredom still left to unload16:05
JayFpoint it at the bug and gogogo!16:05
TheJuliaheh, yeah, should be fairly quick since I've been looking at it, just writing tests16:05
TheJulialol16:05
rloothx JayF for volunteering to review TheJulia's patch :)16:05
TheJulialol16:06
JayFA dirty secret: I review almost everything linked to me unless it's very deep in a vendor driver :)16:06
JayFand I review everything in IPA when I'm working16:06
JayFlol16:06
* rloo has to remember to link everything to JayF16:07
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Update terribly outdated installation instructions  https://review.openstack.org/32718716:07
dtantsurrloo++16:07
TheJuliaIs this where we can laugh evilly?16:08
*** Sukhdev_ has joined #openstack-ironic16:08
*** joprovost has joined #openstack-ironic16:09
rlooTheJulia: hey, I'm ecstatic!16:12
ccarmackHi… can anyone give advice on this error when setting the provision state to active:  | last_error             | Timeout reached while waiting for callback for node                      |16:13
ccarmack|                        | 30490c21-bfc8-4b40-9f43-e09894380db816:13
ccarmackThe node stays in wait for callback state for several minutes, then goes to deploy failed16:14
*** Sukhdev_ has quit IRC16:14
thiagopccarmack: your deploy took too much time or your instance is unable to communicate with Ironic conductor16:14
*** links has quit IRC16:15
thiagopccarmack: normally it is one of these two, but can be something more "black magic" (it always can...)16:15
ccarmackok, thanks… how does the node know where to contact the conductor?16:15
dtantsurTheJulia, wdyt about making this change in IPA: https://bugzilla.redhat.com/show_bug.cgi?id=1283436#c29 (hopefully this link is public)16:15
openstackbugzilla.redhat.com bug 1283436 in rhel-osp-director "diskless boot via iSCSI & iBFT not working with osp-director" [Urgent,On_dev] - Assigned to bnemec16:15
*** alexpilotti has joined #openstack-ironic16:15
*** piet has joined #openstack-ironic16:16
JayFdtantsur: I'd be -1 to doing it unconditionally16:16
*** rpioso has quit IRC16:16
thiagopccarmack: when deploying, Ironic creates a port on neutron with it's IP and PXE info. When the node powers on, it would get it on the DHCP request and contact PXE to download its image16:16
JayFdtantsur: as long as that doesn't run in non-iscsi scenarios and all the ramdisks have that, then sure16:16
JayFdtantsur: with the caveat I don't know anything about icscistart to know if -b is safe :)16:16
dtantsurme neither16:17
dtantsurwell, we can create one more configuration option :)16:17
thiagopccarmack: then, if the BM node can do PXE and download the deploy images, normally it should contact conductor16:17
JayFI mean, given that's hardware specific16:17
TheJuliadtantsur: Kind of with JayF on that, Jay and I talked briefly at the summit about maybe creating a secondary hardware manager or something that could identify such things and run appropriate commands16:17
JayFI must ask if it's something ... ^ yep16:17
JayFbingo16:17
JayFway to steal my thunder Julia :P16:18
dtantsurhow far are you from "maybe creating a secondary hardware manager"? :)16:18
dtantsurI get bugged pretty often by people trying things like that (see the whole bug for just one example)16:18
JayFI mean, you can do it. It's trival. You just add a new hardware manager class to IPA that implements the $whatever that the specific hardware needs16:18
JayFand that gives us the benefit of being able to use entrypoints in setup.cfg to determine what runs or not and/or finding the broken hardware in evaluate_hardware_support() so the workarounds are only enabled as needed16:19
TheJuliame, haven't gotten a chance to look at doing it, but as JayF has said, there are other hardware manager examples out there that can be built in with it16:19
ccarmackthiagop: I'm actually using bifrost with stand-alone ironic16:19
dtantsurwell, this particular change looks pretty generic. do you think something like that can go to IPA upstream?16:19
dtantsurotherwise it means I'll have to develop and support a downstream hw manager for these people16:20
TheJuliadtantsur: I think we need like an "advanced storage hardware manager"16:20
thiagopccarmack: then I think TheJulia can answer better then me16:20
*** jtomasek has quit IRC16:20
TheJuliaand we can put all the important things for advanced storage tech into that16:20
* thiagop needs to give a try to bifrost someday16:20
cineramaccarmack: thanks for putting in that change. i haven't had a chance to look at your most recent patchset yet16:20
dtantsurTheJulia, I still struggle to understand how this potential hw manager will detect that it should kick in..16:21
dtantsur(maybe it's because JayF didn't do his tech talk on hw managers yet ;)16:21
ccarmackcinerama:  no problem… the other thing I'm looking at is scoping bifrost_inventory to a subset of the nodes16:21
TheJuliait would have to be built in with ipa, and it would likely have to, upon start-up, detect if there is "supported" hardware maybe and take particular actions?16:21
ccarmackcinerama: we want to deploy to a subset of our nodes at a time16:22
* TheJulia is just tossing the last idea out there16:22
dtantsurTheJulia, yeah, I understand, I just don't know how to define "supported" in this case. do we need to figure out what this "iscsistart" thing supports?16:22
*** rama_y has joined #openstack-ironic16:22
TheJuliaccarmack: so the "where to go talk to" is on the kernel command line that gets passed in ipxe.  Generally when people are stuck where your at, it is something to do with networking, or maybe the deployment ramdisk doesn't support your network cards or something frustratingly silly like that16:22
* NobodyCam is totally out of bagels an muffin's this morning16:22
JayFdtantsur: tl;dr: evaluate_hardware_support() on each loaded manager runs on IPA start, if it returns 0, it's not a valid manager and will never be called to run things16:23
ccarmackTheJulia: this just started happening.. so I'll need to contact the admin16:23
JayFdtantsur: it's hard to see upstream because we implement that method with a static return vs detecting if a given piece of hardware is there (and/or explicitly enabling it if needed)16:23
cineramaccarmack, that sounds potentially interesting16:24
JayFdtantsur: think about it this way; that crazy hardware is what causes you to need to run that iscsistart command16:24
TheJuliadtantsur: I think so yes :\  Perhaps a midcycle topic or maybe a much higher bandwidth discussion is required. I also need to completely read that bug and try and gain full context16:24
JayFdtantsur: so start with detecting if that crazy hardware exists16:24
ccarmackcinerama: I have new env var (BIFROST_IRONIC_SOURCE) that points to a json file with the node list to call ironic for16:24
cinerama(re your subset-of-nodes idea)16:24
dtantsurJayF, not really hardware, rather kernel arguments (that's how I understand this command)16:25
dtantsurbut maybe I'm just wrong, not sure16:25
JayFhmm. That'd be interesting.16:25
JayFYou could totes trigger it based on kernel command line16:25
TheJuliadtantsur: Yeah, I'm kind of reading this as they are trying to implement iscsi targets16:25
TheJuliasoftware iscsi targets at that16:25
*** ijw has quit IRC16:26
dtantsuryeah.. then it's not hardware thing16:26
TheJuliabut, I think it would fall into advanced storage, but yeah :\16:26
*** Sukhdev_ has joined #openstack-ironic16:26
TheJuliaI think we need a higher bandwidth discussion16:26
*** alexpilotti has quit IRC16:27
dtantsurI get it. I don't want to postpone it for too long though, as this people have eaten my brain since Kilo :)16:27
*** daemontool has joined #openstack-ironic16:27
TheJuliadtantsur: let me try and read through the entire bug and see what they are trying to do since there seems ot be some heavy customization taking place16:27
dtantsurcool, thanks million times!16:28
*** sabeen has quit IRC16:28
TheJuliano problem16:28
dtantsurI have to run now, will get back to it tomorrow. will also try to figure out what they actually achieve by doing that16:28
*** sabeen has joined #openstack-ironic16:28
*** Sukhdev_ has quit IRC16:29
NobodyCambrb16:29
*** e0ne has quit IRC16:30
dtantsursee you tomorrow then16:30
TheJuliahave a good evening16:31
*** trown is now known as trown|lunch16:31
*** tesseract has quit IRC16:31
*** mbound has joined #openstack-ironic16:32
*** mjura has quit IRC16:32
*** dtantsur is now known as dtantsur|afk16:32
*** d0ugal has quit IRC16:33
*** sabeen has quit IRC16:36
*** frontrunner has quit IRC16:36
*** mbound has quit IRC16:36
openstackgerritNisha Agarwal proposed openstack/proliantutils: Fix the logic for fetching the gpu device count  https://review.openstack.org/31540216:37
*** yolanda has quit IRC16:37
*** kromanenko_ has quit IRC16:37
*** kromanenko__ has quit IRC16:37
*** Romanenko_K has quit IRC16:38
*** yolanda has joined #openstack-ironic16:40
*** dprince has joined #openstack-ironic16:41
*** rcernin has quit IRC16:42
openstackgerritBruno Cornec proposed openstack/ironic: Fix JSON error in documentation  https://review.openstack.org/32607916:42
openstackgerritVladyslav Drok proposed openstack/ironic: Add node provsioning methods to the tempest plugin  https://review.openstack.org/32723816:42
*** yohoffman has quit IRC16:44
*** jaybeale has joined #openstack-ironic16:46
lucasagomesfolks, calling it a day too16:47
lucasagomeshave a great evening all, talk to you all tomorrow16:47
*** frontrunner has joined #openstack-ironic16:47
sambettso/ lucasagomes16:48
*** moshele has quit IRC16:48
lucasagomessambetts, :D night!16:48
*** lucasagomes is now known as lucas-dinner16:48
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP: Patch to test Grenade.  https://review.openstack.org/31110916:48
openstackgerritAline Bousquet proposed openstack/ironic-lib: Add support for metrics  https://review.openstack.org/30152616:48
devanandamorning, all16:48
*** d0ugal has joined #openstack-ironic16:49
jlvillallucas-dinner: Ciao!16:49
jlvillaldevananda: Good morning16:49
vdrokmorning devananda and jlvillal16:50
* jlvillal does a little happy dance as only one patch left for Grenade to work :)16:50
sambetts\o/16:50
jlvillalBig thanks to vdrok and vsaienko and everyone else for the help!!16:50
devanandajlvillal: nice! the one you just posted above?16:50
jlvillaldevananda: That just tests it. But if you look in the commit message you can see the one patch.16:50
devanandak16:50
devanandaI'll take a look this morning16:51
jlvillalwarning, it will cause your browser to crawl to a halt for a few moments...16:51
devanandaoh?16:51
* jlvillal glances at sambetts ;)16:51
jlvillalLOL16:51
jlvillaldevananda: A lot of messages in that patch set16:51
*** ChubYann has joined #openstack-ironic16:52
sambettsheh16:52
JayFgood lord why did cisco ci explode on that patch16:52
JayFlol16:52
jlvillalI think I suffer more because I had Toggle CI = On16:52
sambettsNot really sure... I think it had something to do with my recheck regex, cauing it to recheck itself or something ;-P16:52
JayFthat's a hilarious bug16:54
JayFI don't mind things being broken sometimes if they break in funny ways :P16:54
sambettsmy Zuul version also had a bug which caused infinite loops, which didn't get fixed because I was on the stable release of Zuul, and infra expects everyone to CD zuul in there CI, so didn't push a new stable :/16:55
* jlvillal investigates changing grenade jobs from experimental to non-voting...16:55
sambetts(I didn't realise they expected a CD'd Zuul at the time :))16:55
JayFah16:56
JayFjlvillal: want me to jfdi?16:56
jlvillalOne +2 for the final patch16:56
*** PollyZ has quit IRC16:56
JayFjlvillal: I know exactly what knobs to turn to make that happen16:56
* jlvillal wonders if he should go beg in #openstack-infra16:56
JayFI just did16:56
JayFthat's why you got a +216:56
sambettsthe same reason broke my CIs ability to process depends on tags when infra upgraded gerrit and the gerrit API changed16:56
JayFlol16:57
*** daemontool has quit IRC16:57
jlvillalJayF: Thanks. Well I begged in #openstack-qa with a ping to sdague16:57
JayFcool16:57
JayFjlvillal: want me to handle the move of the grenade job?16:57
jlvillalJayF: That would be great :)16:57
jlvillalCan you have it depend on the final patch?16:57
JayFproject-config doesn't work like that16:58
JayFdepends-on isn't effective16:58
jlvillalJayF: I think it is effective for getting code merged.16:58
jlvillalJayF: Or blocking it from getting merged16:58
*** cdearborn has joined #openstack-ironic16:58
JayFah, got what you mean16:58
jlvillalJayF: But yes, not effective for testing the code.16:58
JayFyou wanna make sure it doesn't move before the fix merges16:58
jlvillalYep16:58
jlvillalI seriously doubt it would happen that fast, but I want be safe.16:59
* jlvillal knows his wife thinks he is too cautious while she is learning to drive. He keeps saying, slow down, stop, foot over the brake, look, etc... :)16:59
*** rpioso has joined #openstack-ironic16:59
JayFI suspect the dsg change will be merging in minutes16:59
*** PollyZ has joined #openstack-ironic16:59
JayFif it's not core reviewed and landed in 15m, going to ping a couple of folks directly16:59
jlvillalOh did we get a +A too16:59
JayFone line changes should not block things16:59
JayFNo, you just have the shameless review hound on the case now, I'll make sure it gets merged :P17:00
*** sabeen has joined #openstack-ironic17:00
jlvillalJayF: Agreed and thank you.17:00
jlvillalWoot!17:00
*** ijw has joined #openstack-ironic17:00
jlvillalOh meeting time17:00
jrollyay meetings17:00
krtaylor:)17:00
*** ayoung has quit IRC17:01
*** sacharya_ has joined #openstack-ironic17:02
*** sacharya has quit IRC17:02
*** fragatina has joined #openstack-ironic17:03
*** fragatin_ has joined #openstack-ironic17:04
*** PollyZ has quit IRC17:04
*** rbartal has joined #openstack-ironic17:07
*** fragatina has quit IRC17:08
*** derekh has quit IRC17:08
*** _milan_ has quit IRC17:09
*** sdake has joined #openstack-ironic17:10
*** vishwanathj has quit IRC17:11
*** trown|lunch is now known as trown17:20
*** dprince has quit IRC17:22
*** dprince has joined #openstack-ironic17:23
*** electrofelix has quit IRC17:24
*** PollyZ has joined #openstack-ironic17:26
*** alexpilotti has joined #openstack-ironic17:28
*** PollyZ has quit IRC17:31
*** e0ne has joined #openstack-ironic17:32
*** moshele has joined #openstack-ironic17:32
*** sukhdev has quit IRC17:33
*** ohamada has quit IRC17:34
jlvillalJayF: Feel free to tell me I'm wrong on the idea for a second patch for the ironic grenade job as n-v for grenade.17:38
JayFI think you are :)17:38
jlvillalJayF: I get worried about needing more people to agree. So I lean towards more granularity.17:38
JayFif you look at the list of grenade jobs this seems pretty typical17:38
JayFand when that job votes for us, it'll have to vote for grenade to or else our gate will get wedged17:39
JayFso it's just putting it off17:39
JayFwe'll have to do it, so there's no need to wait17:39
jlvillalJayF: Okay. You convinced me.17:39
jlvillalJayF: Not sure if you want to respin vsaienko's patch?  Unsure if he is still online17:40
jlvillalThough vsaienko seems to always be working...17:40
openstackgerritYosef Hoffman proposed openstack/ironic-python-agent: Add configuration options for DISK_WAIT  https://review.openstack.org/32490917:40
JayFI'll look at that if we get to the point where the other stuff merges17:40
JayFnow I gotta try and get the ds-g patch merged17:40
jlvillalJayF: Thanks for all your help.17:41
JayFnp17:41
*** sabeen has quit IRC17:41
*** mgould is now known as mgould|afk17:42
JayFJust wanted to confirm my suspicion: There's no case where we'd need to run grenade against IPA, right?17:42
*** e0ne has quit IRC17:43
* jlvillal is unsure and needs to leave desk area...17:43
jrollJayF: I don't think so17:43
JayFbut there's a rabbithole there... we do release stable/* of IPA now17:43
jrollan ipa-grenade job might be interesting17:43
JayFdo we have to make sure new IPA works with old ironic or old IPA works with new ironic?17:43
jrollin theory we should do both eventually17:44
jrollgrenade on ironic takes care of the latter17:44
*** ijw has quit IRC17:49
*** e0ne has joined #openstack-ironic17:50
*** sdake has quit IRC17:51
*** rbartal has quit IRC17:51
* sambetts calling it for today 17:52
sambettsNight all17:52
sambettso/17:52
TheJuliagoodnight sambetts17:53
*** sambetts is now known as sambetts|afk17:53
NobodyCamnight sambetts17:53
sambetts|afknight TheJulia, NobodyCam17:53
NobodyCam:)17:53
*** e0ne has quit IRC17:53
*** PollyZ has joined #openstack-ironic17:54
JayFjroll: so really we don't need an IPA grenade job17:54
JayFjroll: as much as we need to test new IPA against latest stable, it sounds like17:54
*** sabeen has joined #openstack-ironic17:54
jrollJayF: yeah, probably17:56
jrollJayF: btw, looking at this e-r bug, this is nasty17:56
JayFjroll: the gate breakage17:56
jrollJayF: though this fixes it in a much simpler way, if you want to rebase on that https://review.openstack.org/#/c/320994/17:56
JayFjroll: and/or the bug breaking e-r unit tests?17:56
jrollthe latter17:56
JayFwell if you read from late yesterday17:57
JayFit appears my query is AFU as well17:57
jrollah, I did but forgot :(17:57
JayFso my patch is basically a steaming pile all the way around17:57
JayFlol17:57
*** PollyZ has quit IRC17:59
*** sabeen has quit IRC18:01
*** ijw has joined #openstack-ironic18:01
*** sabeen has joined #openstack-ironic18:01
*** priteau has quit IRC18:05
*** priteau has joined #openstack-ironic18:05
*** frontrunner has quit IRC18:10
-openstackstatus- NOTICE: Zuul has been restarted to correct an error condition. Events since 17:30 may have been missed; please 'recheck' your changes if they were uploaded since then, or have "NOT_REGISTERED" errors.18:10
*** fragatin_ has quit IRC18:14
*** Sukhdev has joined #openstack-ironic18:16
*** fragatina has joined #openstack-ironic18:19
*** PollyZ has joined #openstack-ironic18:21
*** jaybeale has quit IRC18:23
*** ijw has quit IRC18:24
*** ayoung has joined #openstack-ironic18:28
*** alexpilotti has quit IRC18:32
*** alexpilo_ has joined #openstack-ironic18:32
*** links has joined #openstack-ironic18:32
*** ijw has joined #openstack-ironic18:36
*** rbrud has joined #openstack-ironic18:36
*** frontrunner has joined #openstack-ironic18:36
*** sdake has joined #openstack-ironic18:37
openstackgerritMerged openstack/bifrost: Make ansible installation directory configurable  https://review.openstack.org/32478918:38
*** piet has quit IRC18:47
*** fragatina has quit IRC18:48
*** yolanda has quit IRC18:48
*** krtaylor has quit IRC18:52
*** Sukhdev has quit IRC18:53
*** priteau has quit IRC18:57
*** piet has joined #openstack-ironic18:59
*** jtomasek has joined #openstack-ironic19:01
*** Sukhdev has joined #openstack-ironic19:01
*** alex_xu has quit IRC19:02
*** alexpilotti has joined #openstack-ironic19:04
*** alexpilo_ has quit IRC19:05
*** krtaylor has joined #openstack-ironic19:05
*** alex_xu has joined #openstack-ironic19:06
*** harlowja has quit IRC19:12
*** milan has joined #openstack-ironic19:12
*** e0ne has joined #openstack-ironic19:13
*** harlowja has joined #openstack-ironic19:15
*** frontrunner has quit IRC19:21
*** piet has quit IRC19:24
*** rcernin has joined #openstack-ironic19:25
*** flwang has quit IRC19:29
*** ifarkas has quit IRC19:31
*** MattMan has quit IRC19:31
*** MattMan has joined #openstack-ironic19:31
*** links has quit IRC19:31
*** alexpilotti has quit IRC19:34
*** flwang has joined #openstack-ironic19:34
*** sdake_ has joined #openstack-ironic19:47
*** sdake has quit IRC19:48
openstackgerritChris Krelle proposed openstack/ironic-python-agent: Add basic troubleshooting doc  https://review.openstack.org/32674319:49
*** yolanda has joined #openstack-ironic19:50
*** e0ne has quit IRC20:04
*** Sukhdev has quit IRC20:08
*** rbrud has quit IRC20:16
*** rbrud has joined #openstack-ironic20:17
*** piet has joined #openstack-ironic20:31
*** joprovost has quit IRC20:33
*** rbrud has quit IRC20:34
*** devananda has quit IRC20:35
*** devananda has joined #openstack-ironic20:36
*** mbound has joined #openstack-ironic20:39
rloohi NobodyCam, wrt https://review.openstack.org/#/c/326066/, we had agreed in our weekly meeting, on that change. I don't think it requires waiting for others to review.20:46
rlooNobodyCam: I'm going to +A it if you don't want to.20:46
*** openstackgerrit has quit IRC20:48
NobodyCamI can it hit the button20:48
NobodyCam:)20:48
*** openstackgerrit has joined #openstack-ironic20:48
NobodyCamDone20:49
openstackgerritMerged openstack/python-ironicclient: Add test for 'port-show --field' command  https://review.openstack.org/30400820:49
*** devananda has quit IRC20:51
*** devananda has joined #openstack-ironic20:53
*** SiRiuS__ has quit IRC20:53
*** PollyZ has quit IRC20:54
*** SiRiuS__ has joined #openstack-ironic20:56
*** rama_y has quit IRC20:59
*** PollyZ has joined #openstack-ironic21:01
*** fragatina has joined #openstack-ironic21:01
*** sdake_ has quit IRC21:01
openstackgerritMerged openstack/ironic-specs: Update active node creation spec based on feedback  https://review.openstack.org/32606621:05
*** chlong has joined #openstack-ironic21:06
rloothx NobodyCam :)21:07
*** jaybeale has joined #openstack-ironic21:09
NobodyCam:)21:12
*** SiRiuS__ has quit IRC21:14
*** ametts has quit IRC21:14
*** trown is now known as trown|outtypewww21:18
JayFAnyone else getting randomly signed out of gerrit?21:18
JayFor is my session AFU :/21:18
*** jjohnson2 has quit IRC21:19
*** alexpilotti has joined #openstack-ironic21:19
NobodyCamGerrit has been supper slow for me today21:20
*** lifeless_ is now known as lifeless21:30
jcookJayF bug 1590146 ack21:36
openstackbug 1590146 in Ironic "A timed out cleaning cannot be retried successfully" [High,Confirmed] https://launchpad.net/bugs/1590146 - Assigned to Julia Kreger (juliaashleykreger)21:36
*** baoli_ has quit IRC21:38
*** piet has quit IRC21:38
*** sdake has joined #openstack-ironic21:49
*** jcoufal has quit IRC21:51
*** moshele has quit IRC21:51
mrdaMorning Ironic21:51
*** nicodemos is now known as nico-afk21:52
xavierrmorning mrda :)21:52
mrdahi xavierr21:53
*** rcernin is now known as rcernin|off21:54
*** xavierr is now known as xavierr_afk21:54
*** sdake has quit IRC21:54
*** r-daneel has joined #openstack-ironic21:55
*** sdake has joined #openstack-ironic21:55
*** rpioso has quit IRC21:56
NobodyCammorning mrda21:59
*** dprince has quit IRC22:01
mrdao/22:07
*** rama_y has joined #openstack-ironic22:08
*** ChrisAusten has quit IRC22:08
*** cdearborn has quit IRC22:11
*** thiagop has quit IRC22:17
*** rcernin|off has quit IRC22:21
*** gumbyblake has joined #openstack-ironic22:36
*** rbudden has quit IRC22:37
JayFdevananda: fyi https://review.openstack.org/#/c/327399/ is to make your api-ref changes effective by linking to them22:44
JayFdevananda: thanks very much for helping take that on22:44
devanandaJayF: ah! thanks for that22:47
devanandaJayF: have all my api-ref changes landed?22:47
JayFall of them on that topic did22:47
JayFit looked complete to me but that question makes me worry22:48
JayFlol22:48
JayFThere are definately no more api-ref patchesowned by you22:48
JayFhttps://review.openstack.org/#/q/topic:api-ref+and+project:openstack/ironic are all merged22:49
mikalJayF: hey, I am expecting to batch review backports this week22:49
mikalJayF: I expect tonyb to say sarastic things aobut that ot me this morning22:49
JayFmikal: I already bribed mrda to get your attention22:49
openstackgerritJulia Kreger proposed openstack/ironic: Add cleanwait timeout cleanup process  https://review.openstack.org/32740322:49
JayFmikal: so I'm going to have to get a refund from him then!22:49
TheJuliaJayF: ^^^22:49
JayFmikal: thanks, just wanted to make sure it's on someone's radar22:49
* JayF has had patches sit for long-ass times in nova22:50
mikalJayF: lies. Nova is perfect! Perfect I say!22:51
mikalJayF: also, yes ask mrda for a refund22:51
mikal:P22:51
TheJulialol22:51
JayFI'm a little sore about patches being ignored, I started tracking my outstanding patches more carefully22:51
JayFand that seems to make it worse22:52
JayFlol22:52
JayFwell, more like any patches I care about, not just my outstanding ones22:52
*** piet has joined #openstack-ironic22:53
*** jaybeale has quit IRC22:54
mrdamikal: why do you hang out here and not our team channel?  Do you hate us?22:54
mrdamikal: Also, I asked tonyb, because he is less likely to ignore me. HINT HINT :-P22:54
*** yolanda has quit IRC22:55
mrdamikal: Also, I have a Pixie Boots sticker for you as a bribe. So +2 those reviews.22:55
*** amotoki has quit IRC22:56
JayFActually, that's more of a proxy bribe22:57
JayFsince I bought and shipped those out to you22:57
JayFlol22:57
mrdaJayF: sssh!22:57
mrda:)22:58
*** Jabadia has joined #openstack-ironic23:00
*** amotoki has joined #openstack-ironic23:02
*** PollyZ has quit IRC23:02
*** sabeen has quit IRC23:07
*** bnemec has quit IRC23:11
*** snizar has joined #openstack-ironic23:13
openstackgerritJulia Kreger proposed openstack/ironic: Add cleanwait timeout cleanup process  https://review.openstack.org/32740323:14
TheJuliaJayF: revised ^ based on feedback23:14
*** sacharya_ has quit IRC23:17
*** bnemec has joined #openstack-ironic23:19
*** snizar has quit IRC23:30
*** ppiela has quit IRC23:38
*** mbound has quit IRC23:39
*** alexpilotti has quit IRC23:40
*** mbound has joined #openstack-ironic23:40
*** Goneri has quit IRC23:40
*** mbound has quit IRC23:44
*** Sukhdev has joined #openstack-ironic23:47
* tonyb would take a pixie boot sticker23:49
* tonyb +W's patches23:49
JayFtonyb: you're in Australia, right? I shipped quite a few that way to mrda23:58
JayFtonyb: if he's all out lmk and I'll ship ya one23:58
JayFFWIW that goes for any Ironic contributor; if you want a pixie boots sticker and don't have one just ask me nicely (And land my patches)23:58
tonybJayF: Thanks.  I'll grab one from mrda the next time I see him.23:58
tonybJayF: I feel like it's appropriate that i get a proxy sticker for being a proxy contributor ;P23:59

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