Friday, 2019-02-15

*** whoami-rajat has quit IRC00:00
*** jamesmcarthur has quit IRC00:22
*** jamesmcarthur has joined #storyboard00:26
*** jamesmcarthur has quit IRC00:31
*** jamesmcarthur has joined #storyboard00:47
*** jamesmcarthur has quit IRC00:58
*** jamesmcarthur has joined #storyboard01:15
*** jamesmcarthur has quit IRC01:20
*** jamesmcarthur has joined #storyboard01:34
*** jamesmcarthur has quit IRC01:46
*** jamesmcarthur has joined #storyboard02:06
*** jamesmcarthur has quit IRC02:11
*** jamesmcarthur has joined #storyboard02:27
*** jamesmcarthur has quit IRC02:32
*** jamesmcarthur has joined #storyboard02:48
*** jamesmcarthur has quit IRC02:53
*** jamesmcarthur has joined #storyboard03:00
*** jamesmcarthur has quit IRC03:18
*** jamesmcarthur has joined #storyboard03:19
*** diablo_rojo has quit IRC03:48
*** jamesmcarthur has quit IRC04:49
*** jamesmcarthur has joined #storyboard04:50
*** jamesmcarthur has quit IRC04:54
*** jamesmcarthur has joined #storyboard05:10
*** jamesmcarthur has quit IRC05:15
*** jamesmcarthur has joined #storyboard05:31
*** jamesmcarthur has quit IRC05:35
*** jamesmcarthur has joined #storyboard05:51
*** jamesmcarthur has quit IRC05:56
*** whoami-rajat has joined #storyboard06:08
*** jamesmcarthur has joined #storyboard06:12
*** jamesmcarthur has quit IRC06:16
*** jamesmcarthur has joined #storyboard06:33
*** jamesmcarthur has quit IRC06:38
*** jamesmcarthur has joined #storyboard06:54
*** jamesmcarthur has quit IRC06:59
*** jtomasek has joined #storyboard07:00
*** jamesmcarthur has joined #storyboard07:15
*** jamesmcarthur has quit IRC07:20
*** jamesmcarthur has joined #storyboard07:37
*** jamesmcarthur has quit IRC07:42
*** jamesmcarthur has joined #storyboard07:58
*** ivoline has joined #storyboard07:59
*** jamesmcarthur has quit IRC08:03
*** jamesmcarthur has joined #storyboard08:19
*** jamesmcarthur has quit IRC08:24
*** tosky has joined #storyboard08:38
*** jamesmcarthur has joined #storyboard08:41
*** jamesmcarthur has quit IRC08:45
*** jamesmcarthur has joined #storyboard09:02
*** jamesmcarthur has quit IRC09:07
*** jamesmcarthur has joined #storyboard09:23
*** jamesmcarthur has quit IRC09:28
*** jamesmcarthur has joined #storyboard09:45
*** jamesmcarthur has quit IRC09:49
*** jamesmcarthur has joined #storyboard10:06
*** dtantsur|afk is now known as dtantsur10:09
*** jamesmcarthur has quit IRC10:11
*** ivoline has quit IRC10:18
*** jamesmcarthur has joined #storyboard10:28
*** jamesmcarthur has quit IRC10:32
*** jamesmcarthur has joined #storyboard10:49
*** jamesmcarthur has quit IRC10:54
*** jamesmcarthur has joined #storyboard11:10
*** jamesmcarthur has quit IRC11:15
*** jamesmcarthur has joined #storyboard11:31
*** jamesmcarthur has quit IRC11:36
*** jamesmcarthur has joined #storyboard11:53
*** jamesmcarthur has quit IRC11:57
*** jamesmcarthur has joined #storyboard12:14
*** jamesmcarthur has quit IRC12:19
*** tosky has quit IRC12:30
*** jamesmcarthur has joined #storyboard12:35
*** tosky has joined #storyboard12:36
*** jamesmcarthur has quit IRC12:40
*** jamesmcarthur has joined #storyboard12:57
*** whoami-rajat has quit IRC12:57
*** jamesmcarthur has quit IRC13:01
*** jamesmcarthur has joined #storyboard13:18
*** jamesmcarthur has quit IRC13:24
*** tosky has quit IRC13:25
*** tosky has joined #storyboard13:26
*** jamesmcarthur has joined #storyboard13:40
*** whoami-rajat has joined #storyboard13:41
*** jamesmcarthur has quit IRC13:44
*** jamesmcarthur has joined #storyboard13:45
*** jamesmcarthur has quit IRC13:50
*** jamesmcarthur has joined #storyboard14:06
*** jamesmcarthur has quit IRC14:10
*** jamesmcarthur has joined #storyboard14:27
*** jamesmcarthur has quit IRC14:31
openstackgerritNgong Ivoline-Clarisse proposed openstack-infra/storyboard master: Fixed hyperlinks to worklist and boards documents  https://review.openstack.org/63719314:46
*** ivoline has joined #storyboard14:48
ivolinehello guys, I was able to figure out what was wrong with my installation. It was permission issues. Now my local webclient works :) .14:58
ivolineI also just pushed a patch to the remote. Will be glad if someone can review.15:01
*** jamesmcarthur has joined #storyboard15:08
*** jamesmcarthur has quit IRC15:13
fungithanks ivoline! i'll take a look15:27
fungiand glad you got your file perms sorted15:27
fungithat was certainly a strange error you were getting15:27
*** jamesmcarthur has joined #storyboard15:29
fungiivoline: looks like you somehow managed to also reset file permissions on every file in the repository in the process. your proposed change adds execute permissions on every file in the repository15:29
fungithe links you fixed look great though! http://logs.openstack.org/93/637193/1/check/tox-docs/9ea46c0/html/gui/theory.html#worklists-and-boards15:32
*** jamesmcarthur has quit IRC15:34
ivolineI'll try to change the execute permissions that were added15:34
ivolineAfter changing these, do I need to create a different branch to push the changes or can I just use the same branch?15:38
fungiit's all the same branch, you just need to amend your original commit switching back the file permissions to their original state15:40
ivolineOkay, thanks15:41
fungiyou can reset the permissions with something like `git show -p -R --no-color | grep -E "^(diff|(old|new) mode)" --color=never | git apply` and then amend the commit with `git commit -a --amend`15:41
ivolineokay15:42
fungiafter that, `git show` should give you just the two changed lines and no messages about permission changes on other files15:43
fungiif so, then `git review` again and a new revision will be pushed up15:43
openstackgerritNgong Ivoline-Clarisse proposed openstack-infra/storyboard master: Reverted execute permissions on files  https://review.openstack.org/63720915:47
ivolineThanks so much fungi. Worked like a charm15:47
ivoline:)15:47
*** jamesmcarthur has joined #storyboard15:51
*** jamesmcarthur has quit IRC15:53
ivolineFrom the documentation, I guess it might be good practice to report bugs on storyboard. And then link to the stories/task for the issue in the commit messages. But given that this was a minor change. I didn't do that.15:53
*** jamesmcarthur has joined #storyboard15:53
ivolineIs it important to always report the bugs on storyboard?15:53
persiaI think it is important to report bugs if a) you aren't fixing it right away (as a way of leaving a note), b) someone else might want to work on the same thing (as a way of using assignment to tell others about your work), or c) you aren't sure about the solution (to provide a forum to discuss the issue).15:55
persiaOtherwise, probably not as much.15:55
fungiagreed, if you're fixing something you spotted, there's no real need to do extra "paperwork"15:56
fungithe commit message serves as a suitable means of explaining what was wrong15:56
SotK+115:57
ivolineOkay. Good to know15:57
fungiivoline: it seems you did a new commit instead of amending your original commit, so it was pushed up as a separate change15:59
fungiyou can get back the earlier change from gerrit if needed with `git review -d 637193`16:00
fungithen run the command to fix up the permissions and follow up with `git commit -a --amend`16:01
fungithe --amend makes it update the current commit rather than creating a new one16:01
fungithis is why i suggested `git show` afterward should show your fixed lines in the document and not mention anything about permissions being changed16:02
ivolineOh okay. When I run the command  `git commit -a --amend`, I am asked to enter a commit message. So, I ran this command instead 'git commit -m "Reverted execute permissions on files" -a --amend' but it was rather pushed up as a seperate change16:04
fungioh, you don't have a suitable editor to enter commit messages interactively?16:05
fungialso, normally when amending a commit there's no need to mention changes you're making to correct previous mistakes in the commit itself16:06
ivolineNo. I just use the command line16:06
fungiso the original commit message in 637193 is fine16:07
ivolineWhich editor can I use?16:07
fungiyou can just do `git commit -a --amend --no-edit` in this case to not alter the original commit message16:08
ivolineOkay16:08
fungibut generally, git users will rely on some console-based text editor (vim, emacs, nano, pico, joe...) to create or alter commit messages interactively16:09
openstackgerritNgong Ivoline-Clarisse proposed openstack-infra/storyboard master: Fixed hyperlinks to worklist and boards documents  https://review.openstack.org/63719316:10
ivolineOkay. I should get used to using an editor then. Thanks :)16:11
fungiyou're welcome! in the end it's much easier than trying to compose detailed commit messages through the command-line interface16:12
ivolineOkay16:13
ivolineI have learnt alot today. Now,I have learnt how to do changes in the documentation. I think I can now pick one of the low-hanging-fruit bugs on StoryBoard and start working on it. Hopefully, it can help me to get used to the codebase16:20
ivolineWhat do you think?16:21
ivolineWould that be a good next step?16:26
SotKI think it would be yeah16:27
fungiivoline: and yeah, that docs fix looks great now. i've abandoned the extra change you accidentally made16:28
fungiwhat happened there, when you used -m, was that it replaced the original commit message entirely including the change-id footer previously added by the gerrit commit hook, so a new change-id footer was generated resulting in a separate change in gerrit16:29
ivolineaahh I see now.16:29
fungithe way gerrit knows a new git push is a revision for an existing change is by matching on the change-id footer that commit hook adds to the commit message for you16:30
ivolineOkay. Lesson learned :)16:30
ivolineNow, I can move to the forward.16:34
ivolineWhen a patch is pushed and reviewed, how long after that is it merged ?16:38
persiaDepends on the load on the merge infrastructure, but usually within a couple hours after final review approval.  Range is probably something like 10 minutes to 10 hours.16:43
ivolineOk16:45
*** dtantsur is now known as dtantsur|afk17:08
fungiivoline: if you visit https://zuul.openstack.org/ you can see it being tested. once the jobs for it complete successfully, it should merge automatically17:28
openstackgerritMerged openstack-infra/storyboard master: Fixed hyperlinks to worklist and boards documents  https://review.openstack.org/63719317:28
fungiand now it's done17:28
diablo_rojo_phonCongratulations ivoline :) I know it's not your first patch ever, but it's your first to storyboard :)17:29
ivoline:) Thank you guys17:41
fungiplease don't thank us--we're greatful to have new folks getting involved!17:42
fungier, grateful17:43
ivolineI'm happy to be part of the community. You guys are super helpful.17:45
fungithe only way to get new people to help us is to help them too. glad to know it works! ;)17:47
ivoline:)17:51
*** ivoline has quit IRC18:08
*** ivoline has joined #storyboard19:13
*** ivoline has quit IRC19:14
*** jamesmcarthur has quit IRC19:22
*** jamesmcarthur has joined #storyboard19:23
*** jamesmcarthur has quit IRC19:27
*** jamesmcarthur has joined #storyboard19:33
*** jamesmcarthur has quit IRC19:37
*** jamesmcarthur has joined #storyboard19:38
*** jamesmcarthur has quit IRC19:40
*** jamesmcarthur has joined #storyboard19:48
-openstackstatus- NOTICE: The StoryBoard service on storyboard.openstack.org is offline momentarily for maintenance: http://lists.openstack.org/pipermail/openstack-discuss/2019-February/002666.html20:05
fungithat maintenance is done now, btw20:11
fungiseems everything is working as expected20:11
SotKNice, thanks fungi20:21
fungiyou bet. sorry to take so long to make it happen20:24
fungijust wanted to be sure it had minimal impact20:25
*** jamesmcarthur has quit IRC20:31
*** jamesmcarthur has joined #storyboard20:38
*** jamesmcarthur has quit IRC20:48
*** jamesmcarthur has joined #storyboard20:52
*** jamesmcarthur_ has joined #storyboard20:55
*** jamesmcarthur_ has quit IRC20:57
*** jamesmcarthur has quit IRC20:58
*** whoami-rajat has quit IRC21:30

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