Friday, 2019-02-08

« Thursday, 2019-02-07 Index Saturday, 2019-02-09 »
*** sdake has quit IRC00:01
*** mbeierl has quit IRC00:19
*** etingof has quit IRC00:20
*** sdake has joined #openstack-ironic00:44
*** mjturek has joined #openstack-ironic00:56
*** mjturek has quit IRC01:00
*** mjturek has joined #openstack-ironic01:01
*** sdake has quit IRC01:06
*** mjturek has quit IRC01:24
*** dustinc has quit IRC01:27
*** rh-jelabarre has quit IRC01:29
*** mjturek has joined #openstack-ironic01:41
*** mjturek has quit IRC01:49
*** mjturek has joined #openstack-ironic01:51
*** mjturek has quit IRC01:54
*** sdake has joined #openstack-ironic02:06
*** sdake has quit IRC02:13
*** hwoarang has quit IRC03:00
*** hwoarang has joined #openstack-ironic03:02
*** gkadam has joined #openstack-ironic03:18
*** gkadam_ has joined #openstack-ironic03:28
*** gkadam has quit IRC03:31
*** rloo has quit IRC03:41
*** ianychoi has quit IRC03:49
*** ianychoi has joined #openstack-ironic03:55
*** hwoarang has quit IRC04:19
*** hwoarang has joined #openstack-ironic04:20
*** whoami-rajat has joined #openstack-ironic04:24
*** dustinc has joined #openstack-ironic04:26
*** dustinc has quit IRC05:00
*** penguinepimple has quit IRC05:16
openstackgerritMerged openstack/ironic master: Optionally preserve original system boot order upon instance deployment  https://review.openstack.org/63277405:16
*** S4ren has joined #openstack-ironic05:26
*** S4ren has quit IRC05:29
*** S4ren has joined #openstack-ironic05:30
*** S4ren has quit IRC05:31
*** S4ren has joined #openstack-ironic05:31
*** S4ren has quit IRC05:33
*** S4ren has joined #openstack-ironic05:34
*** S4ren has quit IRC05:35
*** gyee has quit IRC05:36
*** S4ren has joined #openstack-ironic05:37
*** S4ren has quit IRC05:37
*** S4ren has joined #openstack-ironic05:39
*** S4ren has quit IRC05:41
*** etingof has joined #openstack-ironic05:42
*** moshele has joined #openstack-ironic06:34
*** moshele has quit IRC06:46
*** e0ne has joined #openstack-ironic06:50
openstackgerritMerged openstack/ironic master: Add description field to node: api-ref  https://review.openstack.org/63287707:03
*** e0ne has quit IRC07:06
arne_wiebalckgood morning ironic!07:21
*** pcaruana has joined #openstack-ironic07:23
*** Emine has joined #openstack-ironic08:12
*** amoralej|off is now known as amoralej08:27
*** sburette has joined #openstack-ironic08:33
iurygregorygood morning \o/08:39
iurygregorymorning arne_wiebalck o/08:39
*** _fragatina has joined #openstack-ironic08:40
*** hjensas has quit IRC08:42
*** hjensas has joined #openstack-ironic08:43
*** ianychoi has quit IRC08:50
openstackgerritHarald Jensås proposed openstack/networking-baremetal stable/rocky: Set amqp_auto_delete=true for notifications transport.  https://review.openstack.org/63515808:51
*** rpittau has joined #openstack-ironic08:56
rpittaugood morning ironic! o/08:59
iurygregorymorning again =P09:02
rpittauhi iurygregory :D09:04
iurygregoryo/09:04
*** Emine has quit IRC09:05
openstackgerritHarald Jensås proposed openstack/networking-baremetal stable/rocky: Ensure notifications are consumed from non-pool queue  https://review.openstack.org/63572109:09
hjensasmorning ironic!09:10
rpittauciao hjensas :)09:12
*** whoami-rajat has quit IRC09:14
arne_wiebalckhey rpittau iurygregory hjensas o/09:19
rpittauhi arne_wiebalck :)09:19
*** e0ne has joined #openstack-ironic09:20
*** tssurya has joined #openstack-ironic09:25
*** whoami-rajat has joined #openstack-ironic09:28
openstackgerritHarald Jensås proposed openstack/networking-baremetal stable/queens: Ensure notifications are consumed from non-pool queue  https://review.openstack.org/63572409:29
*** dougsz has joined #openstack-ironic09:33
*** derekh has joined #openstack-ironic09:36
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-lib master: Refactor zuulv3 based on ironic-base  https://review.openstack.org/63572609:39
*** dtantsur|afk is now known as dtantsur09:40
dtantsurmorning ironic09:40
rpittauhey dtantsur :)09:40
arne_wiebalckgood morning dtantsur o/09:43
*** dougsz has quit IRC09:45
hjensasgood morning dtantsur09:48
hjensasdtantsur: I picked up an old change you abandoned, https://review.openstack.org/602273 (The stable/queens job in the repo are broken without it, as tox_install.sh would get neutron from master.)09:49
patchbotpatch 602273 - networking-baremetal (stable/queens) - Avoid tox_install.sh - 2 patch sets09:49
dtantsurgreat09:50
hjensasdtantsur: not sure it's ok to change requirements like that? wdyt?09:50
openstackgerritIury Gregory Melo Ferreira proposed openstack/virtualbmc master: Move to zuulv3  https://review.openstack.org/63245309:50
iurygregorymorning dtantsur o/ feeling better?09:50
dtantsuriurygregory: morning, thanks. yeah, much better.09:50
iurygregory\o/09:51
dtantsurhjensas: I think it's fine since networking-baremetal de facto requires neutron to run09:51
etingof\o/09:51
dtantsurhjensas: but let's ask #openstack-release?09:51
hjensasdtantsur: yeah, will do.09:52
dtantsurrpittau: no code changes required for https://github.com/redhat-cip/hardware/pull/65 ?09:54
*** manjeets_ has joined #openstack-ironic09:55
*** manjeets has quit IRC09:57
rpittaudtantsur, I checked the tests and compiled/ran it with py37, didn't complain09:58
dtantsurok09:58
openstackgerritIury Gregory Melo Ferreira proposed openstack/sushy master: Move to zuulv3  https://review.openstack.org/63269210:04
openstackgerritIury Gregory Melo Ferreira proposed openstack/sushy-tools master: Move to zuulv3  https://review.openstack.org/63347310:09
*** tssurya has quit IRC10:15
openstackgerritArne Wiebalck proposed openstack/ironic-python-agent master: [WIP] Support Software RAID  https://review.openstack.org/63468110:19
openstackgerritHarald Jensås proposed openstack/ironic master: API - Implement /events endpoint  https://review.openstack.org/63194610:25
openstackgerritHarald Jensås proposed openstack/ironic master: WIP: Implement process_events  https://review.openstack.org/63372910:25
openstackgerritHarald Jensås proposed openstack/python-ironicclient master: Add Events support  https://review.openstack.org/34593410:26
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Enable ironic notifications  https://review.openstack.org/57687310:26
*** v12aml has quit IRC10:35
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-python-agent master: Move to zuulv3  https://review.openstack.org/63403810:50
*** dougsz has joined #openstack-ironic11:02
*** Emine has joined #openstack-ironic11:02
*** v12aml has joined #openstack-ironic11:02
*** moshele has joined #openstack-ironic11:15
*** tssurya has joined #openstack-ironic11:29
*** moshele has quit IRC11:31
*** moshele has joined #openstack-ironic11:34
*** sdake has joined #openstack-ironic11:34
*** sdake has quit IRC11:41
*** v12aml has quit IRC11:43
*** sdake has joined #openstack-ironic11:45
*** v12aml has joined #openstack-ironic11:49
*** Emine has quit IRC11:51
*** moshele has quit IRC11:54
openstackgerritDmitry Tantsur proposed openstack/ironic master: Allocation API: REST API implementation  https://review.openstack.org/62926011:59
*** v12aml has quit IRC11:59
*** dougsz has quit IRC12:00
dtantsurmgoddard, TheJulia, fixed and rebased ^^ please re-review when you have a minute12:04
*** sdake has quit IRC12:11
*** sdake has joined #openstack-ironic12:13
*** v12aml has joined #openstack-ironic12:14
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Zuulv3 - Use ironic-base job  https://review.openstack.org/63582212:21
jrollmorning ironic12:24
*** whoami-rajat has quit IRC12:24
dtantsurmorning jroll12:27
*** v12aml has quit IRC12:27
iurygregorymorning jroll12:28
*** e0ne has quit IRC12:29
hjensasmorning jroll12:29
openstackgerritIury Gregory Melo Ferreira proposed openstack/networking-baremetal master: Zuulv3 - Use ironic-base job  https://review.openstack.org/63582412:29
jrollhey hjensas, iurygregory, dtantsur :)12:30
*** _fragatina has quit IRC12:31
openstackgerritIury Gregory Melo Ferreira proposed openstack/networking-generic-switch master: Zuulv3 - Use ironic-base job  https://review.openstack.org/63582712:34
*** v12aml has joined #openstack-ironic12:40
*** whoami-rajat has joined #openstack-ironic12:44
*** mjturek has joined #openstack-ironic12:44
*** v12aml has quit IRC12:45
*** Emine has joined #openstack-ironic12:47
*** jtomasek has joined #openstack-ironic12:48
*** sdake has quit IRC12:48
*** v12aml has joined #openstack-ironic12:52
*** _fragatina has joined #openstack-ironic12:55
*** mjturek has quit IRC12:59
*** mjturek has joined #openstack-ironic13:01
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Building tinyipa with python3 by default  https://review.openstack.org/63548713:15
*** e0ne has joined #openstack-ironic13:17
*** v12aml has quit IRC13:17
arne_wiebalckmorning jroll13:17
arne_wiebalckjroll: let me know once you have a moment to discuss potential next steps for sardonic13:18
* iurygregory sardonic to looks like sad ironic XD13:19
*** ianychoi has joined #openstack-ironic13:27
*** moshele has joined #openstack-ironic13:31
arne_wiebalckiurygregory: :-D13:31
*** sturi has quit IRC13:33
*** v12aml has joined #openstack-ironic13:33
*** rh-jelabarre has joined #openstack-ironic13:34
*** _fragatina has quit IRC13:35
TheJuliadtantsur: and anyone else interested in saving a reboot when discovery is in use. mgoddard you may be interested in this for your deployment cases https://storyboard.openstack.org/#!/story/200496513:37
hjensasTheJulia: is there a "I relly like it" tag? :)13:39
TheJuliawe call that rfe-approved :)13:39
*** rpittau has quit IRC13:40
TheJuliaI looked through all the code, it should be super simple to actually implement.. lots of unit tests, but yeah13:40
*** rpittau has joined #openstack-ironic13:40
*** moshele has quit IRC13:43
arne_wiebalckTheJulia: IIUC, the proposal is after inspection, the node would stay powered on, bascially waiting to be deployed … correct?13:49
TheJuliayup13:49
arne_wiebalcksweet13:49
hjensasTheJulia: if we could trust power states we should clean nodes and go to S3 mode with the ramdisk loaded. To speed up deployments.13:49
dtantsurTheJulia: morning, is it the option you're looking for? https://github.com/openstack/ironic/blob/master/ironic/conf/api.py#L5313:49
arne_wiebalckTheJulia: How about using that concept after deletion?13:49
dtantsurTheJulia: I wonder how this all plays with cleaning. We may need to do some partprobe dance to make sure we can deploy after cleaning.13:49
TheJuliadtantsur: no, that is a general blocking lookup for the api for secure endpoints13:50
TheJuliahjensas: or if cleaning is disabled, we would literally could just step right into deployment in seconds13:50
arne_wiebalckTheJulia: I’d think that while there may be some time between inspecttion and first deployment, there may be less time between deletion and re-instantiation.13:50
TheJuliaarne_wiebalck: not sure about deletion, I think that would be a separate case and flow....13:51
arne_wiebalckTheJulia: Basically, users recreating their machine.13:51
dtantsurTheJulia: I get the idea, but I also don't want this feature to require people to disable cleaning.13:51
TheJuliaarne_wiebalck: tl;dr dtantsur are working with some folks trying to get the whole thing down to <5 minutes for an initial run. Power cycling 2-3 extra times seems nowhere near ideal for that13:52
dtantsuryeah13:52
arne_wiebalckTheJulia: I see, thx.13:52
TheJuliaI guess we might actually be able to run clean and rescan disks13:52
TheJuliaI feel like that would be distinct flow logic that we would need to look at13:53
dtantsuryeah, I'd like this working with cleaning enabled if no steps require reboot13:53
TheJuliabut I do suspect we could leave the power on possibly if just disk scrubbing13:53
mgoddarddtantsur: should be able to check the allocations patch later today13:53
dtantsurthanks mgoddard13:53
TheJuliabefore I look at review, has anyone by chance looked at the smartnic code?13:53
dtantsurnot me13:53
dtantsurTheJulia: what I'd prefer not see in our docs is "This cool feature only works with automated_clean=False" :)13:54
dtantsurwe can postpone dealing with post-tear-down cleaning, although it's probably the same code path13:54
TheJuliaI think we could just put the logic in the sanity checker that only cycles the power for reboot/clean->provide transaction if the universe is aligned, no reboot is needed13:54
*** rloo has joined #openstack-ironic13:54
TheJuliayeah, it is the same code path13:55
arne_wiebalckFrom my experience, users are patient when they get their machines (“sorry, not ready yet”), but are less patient when freeing up/reclaiming resources takes “surprisingly” long.13:55
arne_wiebalckaka: “no valid host” :)13:55
hjensasI think I saw clean steps have a require reboot flag. If the last step requires a reboot we could "just" do that reboot, and then not power off the node?13:55
dtantsurhjensas: yep13:56
TheJuliathis whole idea, for at least flat/noop networking could really.... really change a lot if we let it... and could scope creep like crazy13:56
* arne_wiebalck stops taking about doing this during deletion13:56
TheJuliaarne_wiebalck: so, yeah... there has been talk of ironic publishing into placement......13:56
TheJuliaarne_wiebalck: no, don't!13:56
arne_wiebalckTheJulia: “publishing into placement“? isn’t that the case already?13:57
TheJuliaI think that is a great idea! I think my fear and hesitation to the discussion is the security impact, but if we're saying the networking is already static....13:57
TheJuliaarne_wiebalck: ironic publishing into placement directly. Right now nova takes our inventory, massages it, and loads it into placement for us13:58
arne_wiebalckTheJulia: Right, that’s what I meant.13:58
* TheJulia envisions a placement API as like a kitchen chef making a pizza of ironic inventory13:58
* TheJulia may need to make a pizza today13:58
arne_wiebalckTheJulia: LOL13:58
rpittauplease stop talking about pizza :D13:59
mgoddardTheJulia: nice idea. We could even make a new verb, 'openstack baremetal node inspeploy'14:00
arne_wiebalckmgoddard: :-D14:00
TheJuliarpittau: hehe14:00
TheJuliarpittau: If I do, I'll try to resist posting photos to twitter14:00
iurygregorypizza <314:01
TheJuliamgoddard: maybe instdeploy because discovery would have already handled the inspection portion of things()14:01
* dtantsur just had pizza14:01
TheJulialol14:02
TheJuliarpittau: sorry :(14:02
rpittauTheJulia lol14:02
rpittauoh no no, it's just very hard to find good pizza around here :/14:02
TheJulia:( Should I make a video or something on making pizza from scratch?14:03
iurygregoryrpittau, what defines a good pizza? XD14:03
rpittaulol14:03
rpittauwe do make a great homemade pizza :)14:03
TheJuliaExcellent!14:03
rpittauiurygregory, hard to say, it's really personal taste, but for me most important things are the base and the tomato sauce :)14:04
iurygregoryok now you owe us a pizza =P14:04
rpittauahah anytime you're around!14:04
* rpittau feels like hijacked a very serious talk :P14:05
TheJuliaI feel like the channel topic should change for today "Pizza lovers and Bare Metal Provisioning"14:05
rpittaurotfl14:05
iurygregoryhehe14:05
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Add unit tests for _get_notification_transport_url()  https://review.openstack.org/63584414:05
*** jcoufal has joined #openstack-ironic14:06
TheJuliadtantsur: yeah, so if memory serves, the same guarding logic could be used. we wouldn't need to really power cycle into clean (at least in most cases, and the cleaning settings could just be "hey, scrub metadata" for smaller deploys or environments that dont' need disk shredding14:07
TheJulias/shredding/erasing/14:07
TheJuliaor wiping14:07
TheJuliaconsider we have all 314:07
rpittaudtantsur, TheJulia, I have the impression the smartnic and the allocation patches are "competing" for the same minor version ?14:07
TheJuliarpittau: they are14:07
TheJuliarpittau: but changing the version is not a huge deal with rebasing14:08
TheJuliaIt is one of those "Okay, ugh, give me 30 minutes" kind of tasks14:08
rpittauTheJulia, thanks14:09
*** _fragatina has joined #openstack-ironic14:10
dtantsurrpittau: this is indeed a problem of our microversioning..14:15
dtantsurmmm, is our standalone job broken? I see 2-3 patches failing it14:16
TheJuliaI saw it passed on a recheck yesterday, so I'm guessing unstable14:17
iurygregorytimeouts <314:18
* TheJulia gets a security notification that for some reason she is the security contact for but has no actual rights to edit the repo....14:19
rpittaudtantsur, I see, was wondering if it would make sense to decide on a priority in such a case14:19
jrollarne_wiebalck: morning, sardonic is way toward the bottom of my priority list at the moment, but happy to discuss14:19
TheJuliarpittau: I typically try to get people to stack api changes this time of the cycle so we can just sequence through them without rebasing... but yeah14:20
TheJuliamy cat seems determined to drive me crazy this morning14:21
jrollon the pizza topic: this is the ultimate way to make a pizza: https://slice.seriouseats.com/2013/01/the-pizza-lab-the-worlds-easiest-pizza-no-knead-no-stretch-pan-pizza.html14:21
iurygregoryjroll, tks for sharing14:22
jrollTheJulia: your latest RFE very much looks like https://review.openstack.org/#/c/351943/2/specs/approved/long-running-ramdisk-support.rst14:23
patchbotpatch 351943 - ironic-specs - Spec for Long Running Ramdisk Support - 2 patch sets14:23
TheJuliajroll: that is super close to how I make my deep dish pizzas14:23
jroll\o/14:24
rpittauuhmmmm..... no :D14:24
jrollI'm about to get yelled at, I should rephrase14:24
jrollthis is the ultimate way to make a pizza without a dedicated wood fired pizza oven.14:24
*** jcoufal has quit IRC14:24
TheJulia:)14:24
TheJulialol14:24
TheJuliaOkay, I've reached full on-silly14:25
*** TheJulia is now known as needssleep14:25
jroll:D14:25
*** jcoufal has joined #openstack-ironic14:25
needssleephttp://logs.openstack.org/89/635089/2/gate/ironic-standalone/3e2b6b9/controller/logs/ironic-bm-logs/node-4_no_ansi_2019-02-08-13:13:09_log.txt.gz#_2019-02-08_13_02_25_41614:29
needssleeplooks like an intermittent connectivity issue to the external IP :(14:29
openstackgerritMerged openstack/ironic-python-agent master: Deprecate CoreOS ramdisk support  https://review.openstack.org/63184714:33
needssleepOo14:33
*** v12aml has quit IRC14:35
needssleepit looks like it may have never gotten an address...14:36
openstackgerritJim Rollenhagen proposed openstack/ironic master: Allow case-insensitivity when setting conductor_group via API  https://review.openstack.org/63504414:36
arne_wiebalckjroll: Oh, ok. As mentioned earlier, we’ve planned to dedicate some time to this. But I’d like to avoid a situation where we go off in the wrong direction :)14:36
jrollneedssleep: dtantsur: with release note ^14:37
*** jtomasek has quit IRC14:37
dtantsurthanks!14:37
needssleepdtantsur: did you see anything like no ip addrss found on other jobs?14:37
dtantsurneedssleep: I'm pretty sure I've seen this before14:37
*** dtantsur is now known as creepy_owlet14:37
jrollarne_wiebalck: right, cool. afaik the existing doc is still the goal, unless penick or someone else has changed their mind :)14:38
*** jtomasek has joined #openstack-ironic14:38
arne_wiebalckjroll: ok, as we wrote the doc, we’re at least clear on what we’d like to achieve … ideally :)14:39
needssleepugh "Command execution failed: Unable to stat device /dev/vda1 after attempting to verify 3 times." again14:39
jrollarne_wiebalck: cool :)14:39
*** sdake has joined #openstack-ironic14:39
needssleepshiv approved the smartnic support patch and it looks like it hit failures on other jobs. I'm suspecting we're hitting odd CI load again or something14:41
*** sdake has quit IRC14:42
*** sdake has joined #openstack-ironic14:42
*** amoralej is now known as amoralej|lunch14:43
creepy_owletneedssleep: is it that configdrive problem again? :(14:47
needssleepcreepy_owlet: it looked like post-partition statting over iscsi14:47
needssleepI only glanced though14:47
needssleepwe've seen that with IO latency14:47
* needssleep used to see it a lot in HPE's downstream CI14:47
*** sdake has quit IRC14:48
*** v12aml has joined #openstack-ironic14:49
*** sdake has joined #openstack-ironic14:52
creepy_owletah, iscsi14:54
*** tzumainn has joined #openstack-ironic14:56
tzumainnhi! can someone tell me if my understanding of the documentation is correct - that it's not possible to have an ironic node scheduled through nova through a traditional memory/disk/cpu flavor?14:58
*** mbeierl has joined #openstack-ironic14:59
arne_wiebalcktzumainn: that depends on the release, but for recent releases your understanding is correct, you should use resource classes instead15:01
tzumainnarne_wiebalck, okay, thanks!15:01
*** whoami-rajat has quit IRC15:04
*** Emine has quit IRC15:05
creepy_owletright15:08
*** amoralej|lunch is now known as amoralej15:29
*** whoami-rajat has joined #openstack-ironic15:31
*** pcaruana has quit IRC15:31
*** baha has joined #openstack-ironic15:48
*** priteau has joined #openstack-ironic15:51
*** sdake has quit IRC15:52
*** Emine has joined #openstack-ironic15:52
*** tssurya has quit IRC15:52
*** sdake has joined #openstack-ironic15:53
openstackgerritIlya Etingof proposed openstack/ironic master: Prioritize sloppy nodes for power sync  https://review.openstack.org/63587115:55
*** sdake has quit IRC15:56
*** sdake has joined #openstack-ironic15:58
*** priteau has quit IRC16:00
*** _fragatina has quit IRC16:08
*** TxGirlGeek has joined #openstack-ironic16:13
*** e0ne has quit IRC16:17
openstackgerritIlya Etingof proposed openstack/ironic master: Make power sync unit test operational  https://review.openstack.org/63587816:19
openstackgerritIlya Etingof proposed openstack/ironic master: Prioritize sloppy nodes for power sync  https://review.openstack.org/63587116:22
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: [WIP] Allocation API: tests  https://review.openstack.org/63588216:25
*** gyee has joined #openstack-ironic16:38
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: [WIP] Allocation API: tests  https://review.openstack.org/63588216:42
*** hamzy has quit IRC16:47
*** sburette has quit IRC16:48
*** iurygregory has quit IRC16:49
*** mjturek has quit IRC16:53
*** manjeets_ is now known as manjeets17:00
*** Emine has quit IRC17:03
*** dustinc has joined #openstack-ironic17:13
*** rpittau has quit IRC17:17
needssleepcreepy_owlet: I need to go to the doctor... again... so this afternoon/evening I'll likely try and hack out the code that it would be for the flow.17:20
creepy_owletneedssleep: ugh, good luck (and with coding as well)17:20
needssleepthanks, the coding will be the fun part of the day :)17:21
*** jcoufal has quit IRC17:21
NobodyCamGood Morning Ironic'ers17:22
NobodyCamand ...17:22
NobodyCamofc ..17:22
NobodyCamTGIF!17:22
arne_wiebalckhey NobodyCam o/17:23
NobodyCamhey hey happy Friday arne_wiebalck17:23
creepy_owletmorning NobodyCam, TGIF indeed (oh, that was a week)17:23
NobodyCamHey hey creepy_owlet, oh I see it casual Nic Friday17:24
creepy_owletit is indeed :)17:24
needssleepNobodyCam: Feel free to join us17:26
needssleepjoin us... it is what all the cool irc addicts do on friday...17:26
needssleep;)17:26
*** ijw has quit IRC17:26
needssleeparne_wiebalck: do you have a list of raid patches up for review by chance?17:27
arne_wiebalckneedssleep: I currently have parts of the IPA patches as a WIP17:27
needssleeparne_wiebalck: okay, I just want to make sure that we don't forget about your work there17:28
arne_wiebalckneedssleep: give me 2 mins17:28
arne_wiebalckneedssleep: it’s too early for review, I think17:28
arne_wiebalckneedssleep: 1 min17:28
openstackgerritArne Wiebalck proposed openstack/ironic-python-agent master: [WIP] Support Software RAID  https://review.openstack.org/63468117:31
arne_wiebalckneedssleep: a looong minute :)17:31
needssleeparne_wiebalck: no rush!17:31
arne_wiebalckneedssleep: so this ^ is the current state for the IPA17:31
arne_wiebalckneedssleep: I wouldn’t ask for reviews yet17:32
*** gkadam_ has quit IRC17:32
arne_wiebalckneedssleep: I’ll give that a spin early next week to see if that actually works, since it is quite different from the patches we use in prod17:32
arne_wiebalckneedssleep: I should be able to test this relatively easily17:33
needssleeparne_wiebalck: awesome, becuase breaking what you guys are doing in prod is something I want to minimize17:33
arne_wiebalckneedssleep: no, there should be no risk17:34
arne_wiebalckneedssleep: I’ll test in our qa env17:34
arne_wiebalckneedssleep: it’ll use the other (ironic) patches plus this new GenericHardwareManager17:34
arne_wiebalckneedssleep: once the IPA patches work, I move fwd to ironic17:35
needssleepthat is a great way to test it actually17:35
arne_wiebalckneedssleep: feel free to have a lok at the code if you have time, maybe there is sth fundamentally flawed17:36
arne_wiebalckneedssleep: next week I should be able to say if that patch ^works (here at least)17:36
arne_wiebalckI have to run, good night everyone, have a good weekend!17:39
*** Emine has joined #openstack-ironic17:40
*** jaypipes is now known as leakypipes17:43
creepy_owlethave a great weekend17:43
*** creepy_owlet is now known as dtantsur|afk17:43
*** sdake has quit IRC17:47
openstackgerritDmitry Tantsur proposed openstack/ironic master: Allocation API: REST API implementation  https://review.openstack.org/62926017:47
dtantsur|afkthis ^^^ will teach me to test things locally with auth_strategy=noauth >_<17:48
*** derekh has quit IRC17:53
openstackgerritDmitry Tantsur proposed openstack/ironic master: [Trivial] Allocation API: correct syntax in API version history docs  https://review.openstack.org/63590417:54
needssleepdoh!17:57
needssleepgoodnight dtantsur|afk17:57
needssleepI'll look at it later tonight17:57
*** mjturek has joined #openstack-ironic18:03
*** trown is now known as trown|lunch18:04
*** Emine has quit IRC18:12
JayFarne_wiebalck: just a small piece of feedback, from looking at that change: I'd suggest investigating putting the raid stuff in a separate hardware manager, (still inside IPA), so it's trivial for people to completely ensure they don't run the RAID related code if they don't want to.18:13
*** hamzy has joined #openstack-ironic18:14
*** Emine has joined #openstack-ironic18:16
*** hamzy has quit IRC18:22
*** hamzy has joined #openstack-ironic18:25
*** ijw has joined #openstack-ironic18:29
*** whoami-rajat has quit IRC18:34
*** Emine has quit IRC18:38
*** dougsz has joined #openstack-ironic18:40
*** dougsz has quit IRC18:48
*** hamzy has quit IRC18:50
*** Emine has joined #openstack-ironic18:55
*** mjturek has quit IRC18:58
*** hamzy has joined #openstack-ironic18:58
*** mjturek has joined #openstack-ironic19:04
*** Emine has quit IRC19:06
mjturekhjensas: sorry for not responding http://lists.openstack.org/pipermail/openstack-discuss/2019-February/002381.html but was taking some time to look at the job again. Mind taking a minute to walk through your theory with me?19:10
*** hamzy has quit IRC19:10
*** hamzy has joined #openstack-ironic19:11
*** sdake has joined #openstack-ironic19:28
*** trown|lunch is now known as trown19:30
*** hamzy has quit IRC19:37
*** hamzy has joined #openstack-ironic19:38
openstackgerritMerged openstack/ironic master: Add a voting CI job running unit tests with driver-requirements  https://review.openstack.org/63523719:38
openstackgerritMerged openstack/ironic master: Log a warning for Gen8 Inspection  https://review.openstack.org/63508919:38
*** mgoddard has quit IRC19:50
*** mgoddard has joined #openstack-ironic19:51
*** mbeierl has quit IRC19:52
*** ijw has quit IRC19:54
*** sdake has quit IRC19:59
*** sdake has joined #openstack-ironic20:01
*** ijw has joined #openstack-ironic20:08
*** sdake has quit IRC20:09
*** sdake has joined #openstack-ironic20:10
*** sdake has quit IRC20:13
*** sdake has joined #openstack-ironic20:14
*** hamzy has quit IRC20:21
*** trident has quit IRC20:21
*** trident has joined #openstack-ironic20:22
*** e0ne has joined #openstack-ironic20:29
*** ijw has quit IRC20:32
*** trident has quit IRC20:34
*** trident has joined #openstack-ironic20:35
*** ijw has joined #openstack-ironic20:36
*** sdake has quit IRC20:44
*** sdake has joined #openstack-ironic20:46
*** whoami-rajat has joined #openstack-ironic20:48
*** hamzy has joined #openstack-ironic20:59
*** e0ne has quit IRC21:06
*** e0ne has joined #openstack-ironic21:11
*** e0ne has quit IRC21:24
*** mjturek has quit IRC21:26
*** mjturek has joined #openstack-ironic21:27
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: [WIP] Allocation API: tests  https://review.openstack.org/63588221:29
needssleepmjturek: hjensas is in something like CEST time zone, so he may be asleep by now21:32
mjturekthanks needssleep :) I'll ping him at a more reasonable hour21:32
*** mjturek has quit IRC21:52
*** mjturek has joined #openstack-ironic21:53
*** mjturek has quit IRC21:56
*** sdake has quit IRC21:58
*** mjturek has joined #openstack-ironic22:00
*** trown is now known as trown|outtypewww22:05
*** TxGirlGeek has quit IRC22:11
*** mjturek has quit IRC22:19
*** baha has quit IRC22:26
*** hwoarang has quit IRC22:31
*** hwoarang has joined #openstack-ironic22:32
*** dustinc has quit IRC22:40
*** dustinc has joined #openstack-ironic22:44
*** dustinc has quit IRC22:49
*** MattMan has quit IRC22:53
*** MattMan has joined #openstack-ironic22:53
*** sw3 has quit IRC22:58
*** ijw has quit IRC23:07
*** ijw_ has joined #openstack-ironic23:07
*** whoami-rajat has quit IRC23:08
*** sdake has joined #openstack-ironic23:10
*** sdake has quit IRC23:11
openstackgerritMerged openstack/ironic master: Allocation API: REST API implementation  https://review.openstack.org/62926023:21
openstackgerritMerged openstack/ironic master: [Trivial] Allocation API: correct syntax in API version history docs  https://review.openstack.org/63590423:21
openstackgerritMerged openstack/ironic master: Add support for Smart NICs  https://review.openstack.org/58357323:21
*** sdake has joined #openstack-ironic23:33
needssleep\o/23:34
*** rh-jelabarre has quit IRC23:39
*** ijw_ has quit IRC23:39
« Thursday, 2019-02-07 Index Saturday, 2019-02-09 »

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