Monday, 2021-10-11

iurygregorygood morning janders_ and Ironic o/05:48
janders_hey iurygregory o/06:01
iurygregoryo/06:05
arne_wiebalckGood morning janders_ iurygregory and Ironic!06:13
iurygregoryarne_wiebalck, o/06:15
rpittaugood morning ironic! o/08:04
iurygregorymorning rpittau o/08:15
rpittauhey iurygregory :)08:15
janders_hey rpittau iurygregory o/08:16
iurygregoryjanders_, o/08:17
rpittauhey janders_ :)08:17
janders_oh, the underscore is back08:17
janders_let me sort that out08:17
*** janders_ is now known as janders08:17
janders^ the one and only08:18
janderswatch out for cheap low quality imitations 08:18
janders(and I need to set up my new lab server so this stops once and for all)08:18
iurygregorycheap low quality imitations LOL08:19
opendevreviewRiccardo Pittau proposed openstack/metalsmith master: Set explicit boot mode to bios for legacy job  https://review.opendev.org/c/openstack/metalsmith/+/81340009:51
rpittauthis ^ should fix the issue with the metalsmith jobs09:51
iurygregoryjust me or the gerrit UI is a bit different?09:52
rpittauit is different indeed! :)09:53
rpittauthere was an update09:53
iurygregoryoh :D09:53
* dtantsur rubs eyes09:57
dtantsurhappy Monday! how's everyone?09:57
rpittauhey dtantsur :)09:57
iurygregorymorning dtantsur o/ doing good, how was the long weekend? 10:02
dtantsurpretty relaxing, although I woke up tired again :)10:05
dtantsurwe did an unusual thing: went to a restaurant!10:05
iurygregoryoh nice!10:12
dtantsurmmm, do we have a new gerrit version? it looks differently10:32
rpittauyes we do! at least the interface10:36
dtantsurI need to figure out how the new "attention set" feature works10:40
opendevreviewVerification of a change to openstack/ironic master failed: Update the list of supported database filters  https://review.opendev.org/c/openstack/ironic/+/81297911:20
janderssee you tomororow Ironic o/11:29
iurygregoryo/11:29
fmuyassarovHi Ironic! I would like to write a custom hardware manager that would clean the root disks only when IPA is started. Do you know if there is such an example code for cleaning root disks ?  11:31
dtantsurfmuyassarov: you could do something based on https://opendev.org/openstack/ironic-python-agent/src/branch/master/examples/custom-disk-erase/example_disk_eraser.py11:40
fmuyassarovthanks dtantsur , I will try based on that!11:53
opendevreviewRiccardo Pittau proposed openstack/metalsmith master: Set explicit boot mode to bios for legacy job  https://review.opendev.org/c/openstack/metalsmith/+/81340012:47
TheJuliagood morning13:10
iurygregorygood morning TheJulia =)13:12
rpittaugood morning TheJulia :)13:13
dtantsurTheJulia: good morning13:14
TheJuliasooo many emails this morning and meetings this week13:15
dtantsurI can relate, I haven't seen a free inbox for a while13:16
opendevreviewIury Gregory Melo Ferreira proposed openstack/ironic-lib master: Add initial privsep structure  https://review.opendev.org/c/openstack/ironic-lib/+/74553614:15
opendevreviewMerged openstack/ironic master: Update the list of supported database filters  https://review.opendev.org/c/openstack/ironic/+/81297914:30
*** pmannidi is now known as pmannidi|AFK14:47
iurygregory#startmeeting ironic15:00
opendevmeetMeeting started Mon Oct 11 15:00:00 2021 UTC and is due to finish in 60 minutes.  The chair is iurygregory. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
opendevmeetThe meeting name has been set to 'ironic'15:00
dtantsuro/15:00
TheJuliao/15:00
erbarro/15:00
iurygregoryWelcome to our weekly meeting! o/15:00
iurygregoryOur agenda can be found on the wiki. 15:00
stendulkero/15:00
ajyao/15:00
iurygregory#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:00
rpittauo/15:00
rpioso\o15:01
* iurygregory wondering if we should try to summon arne_wiebalck (because of the SIG updates =P)15:01
arne_wiebalcko/15:01
iurygregory#topic Announcements / Reminders15:01
iurygregory#info Yoga PTG is inext week! Our schedule is in the ML and in the etherpad15:01
iurygregory#link http://lists.openstack.org/pipermail/openstack-discuss/2021-October/025249.html15:01
iurygregory#link https://etherpad.opendev.org/p/ironic-yoga-ptg15:02
TheJuliaI see we have powerful arne_wiebalck summoning spells in place in IRC :)15:02
iurygregory:D15:02
* arne_wiebalck was summoned again15:02
iurygregory#info No upstream meeting on Oct 18 because of the PTG15:02
iurygregory#action iurygregory to send email to the ML15:02
iurygregoryjust and fyi, I probably won't be able to join the first day of the PTG Oct18, I will be on PTO =)15:03
iurygregorybut Tue-Fri I will be around =)15:04
iurygregoryAnyone has something to add for announcements/reminders?15:04
TheJuliaOfficial Xena release was last week15:05
iurygregoryoh right! :D15:05
TheJuliaAs well as press releases on the growth of openstack15:05
TheJuliaUse of bare metal as a back-end to nova grew 9% year over year.15:06
iurygregory#info Xena was officially released last week15:06
TheJuliaWell, actually...15:06
TheJuliait is now 20% of all respondants15:06
iurygregorythe Open Infra Live showed some interesting data from the survey15:06
dtantsursweet!15:06
TheJuliait is like 82% growth overall15:07
rpittaunice15:07
iurygregory\o/15:07
TheJuliawhich is kind of epic!15:07
TheJuliaAnyway, I'll be quiet now :)15:07
iurygregorywell, congrats to everyone who contributed during the Xena cycle :D15:08
dtantsurironic++15:08
iurygregory#topic Review action items from previous meeting15:08
iurygregorywe don't have any actions items, moving on15:08
iurygregory#topic Review subteam status reports15:09
iurygregory#link https://etherpad.opendev.org/p/IronicWhiteBoard15:09
iurygregorystarting on L6215:09
dtantsurwe've done some good job cleaning up obsolete RFEs.. but bugs are piling up :(15:10
dtantsurI don't find 300+ bugs and 200+ RFEs comfortable numbers15:11
CharlotteHello, my name is Charlotte. A software engineer and python lover. I am glad to be here15:12
arne_wiebalckmaybe we need a bug jam session to do the triaging15:12
iurygregorydtantsur, agree 15:12
rpittauarne_wiebalck: was thinking the same15:12
iurygregory++ I think it makes sense15:12
TheJuliadtantsur:  We need to scrub the bugs just like RFEs I think15:12
arne_wiebalckto see which are valid, how to reproduce, how difficult to fix, who can have a look, ...15:13
TheJuliaI went through some and found them to be out dated or already fixed15:13
TheJuliait just takes time :(15:13
arne_wiebalckTheJulia: exactly15:13
arne_wiebalckTheJulia: exactly again15:13
dtantsurHi Charlotte! We're in a meeting right now, so may delay saying welcome to you a bit :)15:13
TheJuliaAnyway, I'm all for the week after the PTG15:13
TheJuliaWelcome Charlotte15:13
iurygregorywelcome Charlotte =)15:13
TheJulialets just not cause that time window to collide with a board or committee meeting15:14
iurygregoryif we know the at what time is the board/committee meeting we can avoid this :D15:14
TheJuliaiurygregory: tuesdays. Avoid tuesdays :)15:15
iurygregoryTheJulia, awesome! so we should probably try to change the review jam day?15:15
TheJuliaso luckilly enough, the 26th is clear as the board meeting is 2pm-4pm my time15:15
rpittauwhat about wednesdays?15:16
TheJuliawednesdays are actually a really good day for me15:16
JayFTime to ruin it with more meetings? ;) 15:16
dtantsur\o/15:16
iurygregoryJayF, lol!15:16
arne_wiebalckJayF: heh15:16
rpittaud'oh! yeah15:16
TheJuliaJayF: presently meeting free, but don't tell anyone ;)15:16
TheJuliaand of course I look at it this week and I do have one15:16
TheJuliad'oh15:17
dtantsurmy Wednesday slots are 12pm and 2pm (UTC)15:17
iurygregorymetal3 meeting is on wed we can try to avoid the same slot and we should be good I think...15:17
rpittau2pm utc looks good15:18
TheJuliaLooks like I can do 2pm-3pm utc15:18
iurygregoryworks for me also15:20
iurygregorywho will be the good soul that will update the wiki? :D15:20
iurygregoryshall we go to the next topic?15:21
arne_wiebalckthis will be new review/bug jam slot or is this a one-off?15:21
TheJuliahttps://c.tenor.com/odyVsZbC-OYAAAAM/why-not-both-why-not.gif15:21
iurygregorylol ^15:22
iurygregoryI love it15:22
TheJuliadoing it weekly will be a much better habit for us 15:22
* TheJulia signs in to change the wiki15:22
iurygregorythank you TheJulia ++15:23
iurygregoryok, next topic :D15:23
iurygregory#topic Deciding on priorities for the coming week15:23
iurygregory#link https://review.opendev.org/q/status:open+hashtag:ironic-week-prio15:23
iurygregorywe don't have a lot of topics in the list15:24
iurygregoryanyone with patches that needs reviews?15:24
TheJuliahttps://wiki.openstack.org/wiki/Meetings/Ironic#Related_meetings updated15:24
rpittauthis one https://review.opendev.org/c/openstack/metalsmith/+/81340015:25
iurygregory++15:25
dtantsurI have a couple as well15:26
opendevreviewDmitry Tantsur proposed openstack/ironic master: Add and document high-level helpers for async steps  https://review.opendev.org/c/openstack/ironic/+/80729515:26
opendevreviewDmitry Tantsur proposed openstack/ironic master: Add a helper for node-based periodics  https://review.opendev.org/c/openstack/ironic/+/81249515:26
dtantsurhere they are ^^15:26
iurygregorydtantsur, awesome tks!15:26
iurygregoryhashtags added15:27
TheJuliaI'm good with what is there, largely CI fixes. I did post and tag a nova fix for the long grumbled about vif attach on nova-compute restart15:27
iurygregoryyeah, I think we won't have much patches till the PTG probably15:27
TheJuliaYeah, unlikely15:27
TheJuliaWe also have the initial round for outreachy starting up15:28
iurygregoryoh \o/ nice15:28
iurygregoryshall we move to the next topic?15:29
TheJulia++15:29
iurygregory#topic Discussion15:29
iurygregoryI have one topic for discussion, but I'm not sure if we will be able to get much information :D15:29
iurygregory#topic PTG schedule15:30
iurygregoryI would like the community thoughts about the proposed schedule for the PTG, if we should change something etc15:30
iurygregorywe have 18 topics and 11 slots (1hr each)15:31
TheJuliaso I'm wondering if we should drop "Posting to placement ourselves" since it is bypassing the resource tracker and there is a double interaction there that seems problematic. At least, that is the impression I got from NobodyCam who experimented with doing so15:31
dtantsur"Stopping use of wiki.openstack.org" is something we should likely decide in a small circle15:32
dtantsur(I'm all for it btw)15:32
rpittaufine by me too15:32
iurygregory^ we can probably handle in our upstream meetings I think15:32
iurygregoryor review jam15:33
dtantsuror just sit down one evening and Just Do It15:33
iurygregoryyeah :D15:33
dtantsurI don't think it needs a voice call15:33
TheJuliabut where do we move interactive stuff like meeting agendas?15:33
iurygregory^ this15:33
dtantsuretherpad?15:33
TheJuliacould work!15:33
TheJuliaworth a try I think15:33
dtantsuri.e. the whiteboard, which is actively used for the meeting anyway15:33
TheJulia++15:34
rpittaufaster and easier to add content o nthe fly15:34
dtantsuryep15:34
iurygregorysounds like a plan =)15:34
dtantsurTheJulia: it looks like you're half-way through it. are there specific areas where you need help?15:35
dtantsurmigrate something specific to the docs or anything?15:35
iurygregoryif we think we have a lot of info in the WhiteBoard we can have a specific etherpad for the meeting agenda also15:35
TheJuliadtantsur: w/r/t wiki.o.o ?15:35
dtantsuryep15:35
TheJuliastendulker: you around?15:35
dtantsur(I also agree re placement btw)15:35
TheJuliaI need clarity w/r/t all of the ilo docs on the wiki15:36
stendulkerTheJulia: yes15:36
TheJuliathe *rest* of it including the release notes being moved is easy15:36
stendulkerwiki is not updated off late15:36
TheJuliastendulker: I noticed a lot of it duplicated the existing docs, is it safe to just abandon those wiki pages on the iLODrivers ?15:36
dtantsurhttps://wiki.openstack.org/wiki/ThirdPartySystems/* is an openstack-wide thing, I assume, maybe we shouldn't touch it15:37
stendulkeri think so, its error prone and mostly duplicate15:37
TheJuliadtantsur: yeah15:37
TheJuliastendulker: Okay, I'll go back grhough it after moving the release notes15:37
TheJuliasoon()15:37
dtantsur"Certificate Management (dtantsur)" - I think we can delay this item, unless anybody has anything for it15:38
TheJuliaiurygregory: don't delete the text, I spent a ton of time digging up all the pages15:38
iurygregoryTheJulia, ack =)15:38
stendulkerTheJulia: Thanks15:38
dtantsurVirtualMedia 1.4.0 is still not widely used, and after it is, this may be a Just Do It item15:38
TheJulia++15:39
dtantsur"Self-configuring Ironic service" and "Eliminate manual commands" should probably go back-to-back15:39
dtantsur(if not merged together)15:40
TheJuliathey are rather tied together15:40
TheJuliasame general theme, improving ease of use and operability15:40
TheJuliaMaybe that is a theme for the community's during the Yoga cycle15:40
dtantsurthen we should probably do it on one day15:40
TheJuliaIn addition to everyone learning yoga.15:40
dtantsur(and they may occupy most of it)15:40
dtantsur++15:40
iurygregoryack I will move them to the same day15:41
Charlotte".. so bring that into base.py is appropriate." Hello TheJulia do you mind elaborating further what this means? I found this in this story https://storyboard.openstack.org/#!/story/200925715:41
iurygregoryEliminate manual-commands as sub-topic for Self-configuring Ironic service?15:41
TheJuliaCharlotte: We're still meeting, although approaching the end, I can switch to your question once the meeting is over15:42
CharlotteThank you so much TheJulia15:42
TheJuliaiurygregory: yes, I think the idea is more "can we we have times we determine we should just do the needful instead of requiring a human to populate db schema or run the db upgrade15:42
iurygregoryTheJulia, got it =)15:43
TheJuliasome of that is totally navigatable too, we have most of the substrate needed already there, just not wired for ease of use15:43
TheJuliaits all wired for "human run command"15:43
* dtantsur would love to reduce the amount of bash in metal3-io/ironic-image15:44
iurygregoryI will be updating the agenda locally and will past a link tomorrow to see what everyone think, I plan to send a new email on Thursday/Friday with the new proposal (after we get a consensus :D )15:44
TheJuliadtantsur: ++15:44
TheJuliadtantsur: if you propose env variable loading/checking, I won't object ;)15:44
TheJuliaso onward?15:46
iurygregoryI think we can move to the next topic, since  I got enough feedback to update the schedule, tks everyone15:46
iurygregory#topic Baremetal SIG15:46
iurygregory#link https://etherpad.opendev.org/p/bare-metal-sig15:46
* iurygregory summons arne_wiebalck 15:46
* arne_wiebalck is summoned :)15:47
arne_wiebalckTwo meetings tomorrow, again on ops feedback.15:47
arne_wiebalckAll details on the etherpad.15:47
iurygregoryawesome \o/15:47
arne_wiebalck#link https://etherpad.opendev.org/p/bare-metal-sig15:47
arne_wiebalck2 PM UTC (EMEA friendly) and 6 PM UTC (AMER friendly)15:47
iurygregorytks arne_wiebalck =)15:47
iurygregory#topic RFE review15:48
arne_wiebalcktimeu: jcosmao_ if you're interested ^^15:48
iurygregoryskipping since we don't have any RFE's =)15:48
TheJuliaNobodyCam: you may also be interseted ^15:48
TheJuliazer0c00l: And you as well ^15:48
iurygregoryguilhermesp, please try to join the meeting tomorrow =P you or maybe mnaser_ =)15:49
TheJulia++15:49
NobodyCamme? ... Good Morning Ironic'ers15:49
TheJuliaYassss, you.15:49
NobodyCam:)15:49
TheJulia:)15:49
iurygregoryonward =D15:50
iurygregory#topic Open discussion15:50
iurygregorydo we have more things to discuss? 15:50
TheJuliaCoffee(), lets discuss it ;)15:50
TheJuliaJoking, mostly. :)15:50
iurygregory=D15:50
dtantsurwe picked up a habit of ordering a completely new coffee flavor each time15:51
dtantsurworks very well so far :)15:51
iurygregorywow =O15:51
rpittaummm I do that with tea :)15:52
dtantsurwith tea we're quite conservative :)15:52
rpittauopposite with coffee here :D15:52
NobodyCammmmm Coffee15:52
TheJuliaI do need a fresh cup of coffee15:52
TheJuliaToday is one of those days that if I actually touch code I'll be surprised15:53
TheJuliaI've done nothing but reply to emails really15:53
iurygregorywell, it's monday :D 15:53
TheJuliaTrue15:53
* dtantsur gets surprised when touching code as well.. quite often15:53
iurygregorylet's go to the last topic and we can enjoy coffee :D15:54
iurygregory#topic Who is going to run the next meeting?15:54
TheJuliahttps://twitter.com/ashinclouds/status/1447590940718235651 <-- of course has appropriate gif for 82% growth.15:54
iurygregorythis would be for the meeting in Oct 25th15:54
iurygregoryTheJulia, ++15:55
rpittauI can run that15:55
TheJuliaI'll just be back from traveling so I don't think it would be a good idea15:55
iurygregorythanks rpittau =D15:55
rpittau:)15:55
TheJuliaI'm only working half time next week, fwiw15:55
iurygregoryTheJulia, the ++ was for the link :D15:55
TheJuliaack15:55
iurygregoryThanks everyone!15:55
iurygregory#endmeeting15:55
opendevmeetMeeting ended Mon Oct 11 15:55:53 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:55
opendevmeetMinutes:        https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-10-11-15.00.html15:55
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-10-11-15.00.txt15:55
opendevmeetLog:            https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-10-11-15.00.log.html15:55
TheJuliaCharlotte: Greetings!15:56
CharlotteGreetings! TheJulia15:57
TheJuliaCharlotte: looking at what stevebaker posted, it looks like his goal is to eliminate https://opendev.org/openstack/ironic/src/branch/master/ironic/tests/unit/api/base.py#L28 which would require possibly copying the method or duplicating the functionality in that same file so the unit tests can pass and have an base API framework they can interact with, without firing up a fully working service which would be more along the lines of 15:59
TheJuliafunctional testing.15:59
TheJuliaCharlotte: does that make sense?16:00
iurygregorywow this new feature "attention" in gerrit is funny16:00
TheJuliaiurygregory: attention?!16:01
iurygregoryyeah16:01
TheJuliaLike where I have a jetpack on the moon?16:01
iurygregorymy patch failed and I got in the email " Attention is currently required from: Iury Gregory Melo Ferreira. "16:01
TheJulia(My status field)16:01
iurygregoryand in the UI there is an Icon LOL16:01
TheJuliajoy16:01
iurygregorywoot status field? :O16:01
TheJuliaoh! you didn't know about the status field?!?16:01
iurygregoryno =O16:01
iurygregoryomg16:02
TheJuliahttps://review.opendev.org/q/owner:juliaashleykreger%2540gmail.com <-- if you look right under my name16:02
iurygregoryOn the moon! With a jetpack! :D16:02
iurygregoryNice!16:03
CharlotteTheJulia yes a bit. Its like refactoring the base.py file to use another testing framework that doesn't need pecan?16:05
opendevreviewVerification of a change to openstack/ironic-inspector master failed: CI: Make inspector configuration UEFI aware  https://review.opendev.org/c/openstack/ironic-inspector/+/81213716:07
TheJuliaugh, inspector16:10
TheJuliaCharlotte: basically, yes16:10
Charlottethank you =D 16:11
CharlotteTheJulia16:11
rpittaubye everyone! o/16:16
iurygregoryo/16:18
opendevreviewDmitry Tantsur proposed openstack/ironic master: node_periodics: encapsulate the interface class check  https://review.opendev.org/c/openstack/ironic/+/81345816:30
opendevreviewMerged openstack/metalsmith master: Set explicit boot mode to bios for legacy job  https://review.opendev.org/c/openstack/metalsmith/+/81340016:48
dtantsursee you tomorrow folks16:51
* TheJulia pokes the calendar app17:05
TheJuliao/ kedar_h 17:34
ChapiGuys I can't still push my commit to the ironic repository :(19:49
ChapiI followed the steps but still get an authentification error19:49
ChapiI created an account, pasted my SSH key and installed git review19:50
ChapiI also accepted the license19:51
TheJuliaChapi: are you using git review?20:29
stevebakerotherstevebaker: hi22:00
otherstevebakerstevebaker: yop22:00
iurygregorywoot?22:01
stevebakerI'm just trying web based chat clients to recommend to outreachy folk22:08
iurygregoryohhhh :D22:09
stevebakerpas-ha[m]: how are you bridging from Matrix? I was hoping #openstack-ironic would just be in app.element.io OFTC rooms but it isn't22:10
stevebakerhttps://webchat.oftc.net/?channels=%23openstack-ironic seems easiest for now22:11
stevebakerstevebak[m]: hi!22:22
Chapi@TheJulia I am using it, yes. I still get the authentification error, I will try again.22:47
TheJuliaChapi: so generally that seems like the wrong key pair is being used?23:11
TheJuliastevebaker: hmmmm23:12
TheJuliaor wrong username from your shell vs gerrit?23:12
ChapiShould I use `git clone "ssh://luiscgomez@review.opendev.org:29418/openstack/ironic"` instead of cloning it directly from the repository?23:12
TheJuliaChapi: no23:12
Chapi:(23:12
TheJuliaChapi: you can glone https://opendev.org/openstack/ironic23:12
TheJuliaChapi: and then run `git review -s` which sets up the post-commit hook and sets up the remote it pushes the changes to when you submit a change set into gerrit23:13
ChapiI get this23:14
Chapi`ssh://luiscgomez@review.opendev.org:29418/openstack/ironic.git did not work. Description: luiscgomez@review.opendev.org: Permission denied (publickey).23:14
Chapifatal: Could not read from remote repository.23:14
ChapiPlease make sure you have the correct access rights23:14
Chapiand the repository exists.23:14
ChapiCould not connect to gerrit.23:14
ChapiEnter your gerrit username:23:14
Chapi`23:14
ChapiI use my username but still gives me an error23:14
TheJuliawhat email address did you use for gerrit?23:17
Chapideveloperchapi@gmail.com23:18
TheJuliaChapi: and you uploaded a key into gerrit?23:20
TheJuliawell, a public key for ssh authentication?23:21
ChapiYes, I did, is the .pub file?23:21
Chapithe information in it23:21
Chapior the other one?23:21
TheJuliathe one in the .pub file23:22
ChapiYes, I did23:24
ChapiMaybe it has to do with this? https://usercontent.irccloud-cdn.com/file/EAbbxx6I/image.png23:24
TheJulianot likely... Whta was the command which resulted in the Permission denied error?23:25
Chapigit review -s23:28
TheJuliadid you generate a new ssh keypair or use an existing one?23:29
ChapiI generated a new one23:29
TheJuliadid you hve a pre-existing keypair?23:29
ChapiYes, maybe I should delete all of them and create a new one from zero.23:30
TheJuliaChapi: did you add an ssh config file entry directing ssh to use your new keypair files?23:31
TheJuliafor the review.opendev.org hostname specifically23:31
Chapihttps://www.irccloud.com/pastebin/3IaBVSav/23:31
ChapiI did :(23:31
ChapiI am sorry for causing so much trouble23:31
TheJuliaand ~/.ssh/openstack is the private key?23:32
ChapiYes, it is23:33
ChapiMaybe I should create another key because I followed all the steps23:33
TheJuliawhat are the file permissions on that file?23:34
jandersgood morning Ironic o/23:35
Chapio/23:35
ChapiI am not sure @TheJulia, I am on Windows. Should I switch to Linux?23:37
TheJuliaChapi: I'd recommend using linux. I don't know if anyone has ever tried `git review` on windows23:37
TheJuliaand ssh will not load files if they are world readable23:37
ChapiI will try23:41
ChapiSorry23:41
TheJuliano worries23:42
ChapiSo it worked on Linux23:50
ChapiAfter git review -s23:50
Chapiwhat should I do?23:50
Chapinow*23:51
TheJuliaChapi: well, if you have a commit staged, you just type `git review`23:54
TheJuliaand it creates a changeset in gerrit23:54

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!