Friday, 2016-06-03

*** david-lyle has quit IRC00:25
*** david-lyle has joined #openstack-trove00:26
*** david-lyle has quit IRC00:30
*** khushbu has quit IRC00:42
*** tqtran has quit IRC00:46
*** cheneydc has joined #openstack-trove00:57
*** imandhan has quit IRC01:09
*** imandhan has joined #openstack-trove01:09
*** imandhan has quit IRC01:14
*** stewie925 has quit IRC01:45
*** pmackinn has quit IRC02:20
*** links has joined #openstack-trove04:38
*** sgotliv has joined #openstack-trove04:40
*** rcernin has joined #openstack-trove04:58
*** rcernin has quit IRC05:04
*** johnma has quit IRC05:10
*** johnma has joined #openstack-trove05:11
*** sgotliv has quit IRC05:35
*** rcernin has joined #openstack-trove05:46
*** imandhan has joined #openstack-trove06:07
*** openstackgerrit has quit IRC06:17
*** openstackgerrit has joined #openstack-trove06:17
*** openstackgerrit has quit IRC07:03
*** openstackgerrit has joined #openstack-trove07:03
openstackgerritIshita Mandhan proposed openstack/trove-dashboard: Add backups table to ng-backups panel  https://review.openstack.org/27696607:38
*** nikhil has quit IRC07:58
*** sgotliv has joined #openstack-trove08:04
*** imandhan has quit IRC08:06
*** imandhan has joined #openstack-trove08:06
*** imandhan has quit IRC08:11
*** fesp has joined #openstack-trove08:11
*** fesp has quit IRC08:16
*** nikhil_ has joined #openstack-trove09:17
*** nikhil_ is now known as Guest2723109:17
*** rcernin is now known as rcernin|lunch09:31
*** imandhan has joined #openstack-trove09:37
-openstackstatus- NOTICE: CI is experiencing issues with test logs, all jobs are currently UNSTABLE as a result. No need to recheck until this is fixed! Thanks for your patience.09:38
*** pcaruana has joined #openstack-trove09:44
*** logan_ has joined #openstack-trove09:57
*** logan_ is now known as Guest6297109:58
*** logan- has quit IRC09:59
*** cheneydc has quit IRC10:00
*** hferenc has quit IRC10:00
*** hogepodge has quit IRC10:01
*** Guest62971 is now known as logan-10:01
*** Guest27231 has quit IRC10:02
*** imandhan has quit IRC10:08
-openstackstatus- NOTICE: CI is experiencing issues with test logs, all jobs are currently UNSTABLE as a result. No need to recheck until this is fixed! Thanks for your patience.10:10
*** ChanServ changes topic to "CI is experiencing issues with test logs, all jobs are currently UNSTABLE as a result. No need to recheck until this is fixed! Thanks for your patience."10:10
*** SlickN1k has joined #openstack-trove10:13
*** sgotliv has quit IRC10:16
*** SlickNik has quit IRC10:16
*** SlickN1k is now known as SlickNik10:16
*** rcernin|lunch is now known as rcernin10:20
*** Guest27231 has joined #openstack-trove10:41
*** hogepodge has joined #openstack-trove10:49
*** sgotliv has joined #openstack-trove10:49
*** orwell.freenode.net changes topic to "Trove Openstack Database as a Service | Docs https://github.com/openstack/trove | Channel Logs http://eavesdrop.openstack.org/irclogs/ | IRC must read http://sackheads.org/~bnaylor/spew/away_msgs.html"10:49
*** imandhan has joined #openstack-trove11:06
*** tesseract has joined #openstack-trove11:33
-openstackstatus- NOTICE: CI is experiencing issues with test logs, all jobs are currently UNSTABLE as a result. No need to recheck until this is fixed! Thanks for your patience.11:43
*** ChanServ changes topic to "CI is experiencing issues with test logs, all jobs are currently UNSTABLE as a result. No need to recheck until this is fixed! Thanks for your patience."11:43
*** pcaruana has quit IRC11:45
*** imandhan has quit IRC11:55
*** pmackinn has joined #openstack-trove12:10
*** imandhan has joined #openstack-trove12:52
*** ewilson has joined #openstack-trove12:58
*** miqui has joined #openstack-trove13:14
openstackgerritPetr Malik proposed openstack/trove: Cleanup guestagent models  https://review.openstack.org/27807413:34
*** Guest27231 has quit IRC13:45
*** Guest27231 has joined #openstack-trove13:46
*** Guest27231 is now known as nikhil13:48
*** ChanServ changes topic to "Trove Openstack Database as a Service | Docs https://github.com/openstack/trove | Channel Logs http://eavesdrop.openstack.org/irclogs/ | IRC must read http://sackheads.org/~bnaylor/spew/away_msgs.html"14:01
-openstackstatus- NOTICE: Cleanup from earlier block storage disruption on static.openstack.org has been repaired, and any jobs which reported an "UNSTABLE" result or linked to missing logs between 08:00-14:00 UTC can be retriggered by leaving a "recheck" comment.14:02
*** imandhan has quit IRC14:02
*** links has quit IRC14:36
*** rcernin has quit IRC14:46
*** imandhan has joined #openstack-trove14:59
*** rcernin has joined #openstack-trove15:31
*** rcernin has quit IRC15:39
*** imandhan has quit IRC15:54
*** imandhan has joined #openstack-trove16:07
*** tesseract has quit IRC16:19
openstackgerritPetr Malik proposed openstack/trove: Cleanup guestagent models  https://review.openstack.org/27807416:29
*** rcernin has joined #openstack-trove16:31
*** grapex has joined #openstack-trove16:51
openstackgerritPetr Malik proposed openstack/trove: Cleanup guestagent models  https://review.openstack.org/27807417:02
*** twm2016 has joined #openstack-trove17:43
*** khushbu has joined #openstack-trove17:57
*** tqtran has joined #openstack-trove18:06
openstackgerritPetr Malik proposed openstack/trove: Cleanup guestagent models  https://review.openstack.org/27807418:09
*** ewilson has quit IRC18:52
*** ewilson has joined #openstack-trove18:52
*** mfisch has quit IRC19:06
*** stewie925 has joined #openstack-trove19:09
stewie925hi tellesnobrega19:09
tellesnobregahey stewie92519:10
stewie925hey I am currently working on committing a patch for review19:10
stewie925but I am getting a gerrit error:19:10
stewie925 ! [remote rejected] HEAD -> refs/publish/master/bug/1529622 (you are not allowed to upload merges)19:10
stewie925error: failed to push some refs to 'https://review.openstack.org/openstack/trove'19:10
stewie925this was the gerrit patch that I took over from a colleague (there was a merge conflict in it)19:11
stewie925so yesterday, from our discussion, I git cloned trove to get the latest trove repo, and then applied the changes he made19:12
stewie925ran the tests successfully...19:14
tellesnobregahmm, weird. Don't really know what is going on19:15
stewie925here were your suggestions:  pull in another branch (master) the current code from repo , go back to the branch with this code and rebase it against master19:16
dougshelley66stewie925 did you use the Download..Checkout function in gerrit to setup your local repo with the change19:17
*** johnma has quit IRC19:17
stewie925hi dougshelley6619:17
dougshelley66hi19:17
stewie925no, I didnt....19:17
dougshelley66the easiest way to grab a change from gerrit and work on it19:17
dougshelley66is to use the "Download...Checkout...copy to clipboard" thing19:18
tellesnobregastewie925, you applied every change by hand?19:18
dougshelley66it will put a big git command into your clipboard19:18
dougshelley66you paste that into a terminal19:18
dougshelley66on your local19:18
dougshelley66that will put you on a branch with the changeset19:18
dougshelley66then to rebase, you do this19:19
dougshelley66get fetch origin master19:19
*** arist has quit IRC19:19
dougshelley66sorry "git"19:19
dougshelley66git rebase FETCH_HEAD19:19
*** arist has joined #openstack-trove19:19
dougshelley66fix up the conflicts19:19
dougshelley66git add each file as you fix it19:19
dougshelley66then when you are done19:19
dougshelley66git rebase --continue19:19
dougshelley66then git commit -a --amend to update the commit19:20
tellesnobregastewie925, dougshelley66 is completely right, that will work for you19:20
dougshelley66then git review19:20
stewie925hi tellesnobrega and dougshelley66  - thank you very much19:20
tellesnobreganp19:20
dougshelley66no prob19:20
stewie925and tellesnobrega - yes I did update the eight modules manually - let me try this new approach - thanks again!19:21
tellesnobregacool, give it a try, if you run into problems let us know19:21
stewie925so at this point - I have a git commit that is "hanging"  because of the gerrit issue19:21
tellesnobregayou can git stash to drop the changes19:22
stewie925unfortunately i ran the git commit amend though... :(19:22
tellesnobregayou could go git log19:23
tellesnobregaget the latest change on the master19:23
tellesnobregaand hard reset your branch to it19:23
tellesnobregathis way it would lose your changes and go back to master version19:23
stewie925I see- let me go ahead and do the git hard reset19:24
stewie925git reset --hard HEAD on the branch that I did my last commit19:25
dougshelley66run git reflog19:29
dougshelley66and pick the HEAD reference that makes sense for your git reset command19:30
openstackgerritIshita Mandhan proposed openstack/trove-dashboard: Add backups table to ng-backups panel  https://review.openstack.org/27696619:37
*** grapex has left #openstack-trove19:44
stewie925thank you19:46
*** amrith is now known as _amrith_19:49
-openstackstatus- NOTICE: The infrastructure team is taking Gerrit offline for maintenance this afternoon, beginning shortly after 20:00 UTC. We aim to have it back online around 00:00 UTC.20:00
*** _amrith_ is now known as amrith20:04
*** khushbu has quit IRC20:06
-openstackstatus- NOTICE: Gerrit is offline for maintenance until 00:00 UTC20:11
*** ChanServ changes topic to "Gerrit is offline for maintenance until 00:00 UTC"20:11
*** rcernin has quit IRC20:15
stewie925just my luck -gerrit is down for maintenance hehe20:18
*** grapex has joined #openstack-trove20:47
*** amrith is now known as _amrith_21:15
*** _amrith_ is now known as amrith21:27
*** ewilson has quit IRC22:11
*** twm2016 has quit IRC22:29
*** grapex has quit IRC22:40
*** stewie925 has quit IRC22:52
*** khushbu_ has joined #openstack-trove22:58
*** stewie925 has joined #openstack-trove23:03
stewie925hi dougshelley6623:03
dougshelley66hi23:03
stewie925so i just cloned the trove repo, and then i ran the checkout url of the patch with the merge conflict23:03
dougshelley66yes23:03
dougshelley66well not a url23:04
stewie925sorry i mean a git fetch command not a url :)23:04
dougshelley66right23:04
dougshelley66so you should be on detached head23:04
stewie925yes!!! a detached head23:04
dougshelley66so that is ok23:05
dougshelley66normally what i do23:05
dougshelley66is git checkout -b <topic from gerrit)23:05
stewie925so do I run both 'git fetch origin/master'   and 'git rebase fetch_head'  at this point?23:05
stewie925ohhh ok23:05
dougshelley66but that isn't required23:05
dougshelley66so no slash23:05
dougshelley66git fetch origin master23:05
stewie925yeah, it ran successfully :)  then I run the git rebase FETCH_HEAD (in caps)23:07
stewie925git rebase FETCH_HEAD23:08
dougshelley66yes23:08
dougshelley66did you have conflicts?23:08
dougshelley66git status will tell you too23:08
stewie925ahhh now i see conflicts yeah23:09
dougshelley66so fix them23:09
dougshelley66make sure all the chevrons are sorted out (<<<< >>>>)23:09
dougshelley66then do git add on each file23:09
dougshelley66at that point i would run tox -epep8 and tox -epy2723:09
dougshelley66just to be sure23:09
dougshelley66then git rebase --continue23:10
stewie925thank you dougshelley6623:10
stewie925ok I need to fix the two conflicts (out of 8 modules)23:10
dougshelley66right that is why the "push button" rebase didnt' work23:12
stewie925ok I see some '<<<<<'  and '>>>>>>'23:13
dougshelley66yes those are the conflict markers23:14
amrithstewie925, here's something that may help you get some context to all this magic. https://www.youtube.com/watch?v=1ffBJ4sVUb423:17
amrithit is a great 45 minute video.23:17
amrithit'll help you understand all this stuff that dougshelley66 is talking about23:17
stewie925thank you amrith23:17
stewie925and thank you dougshelley66 !23:17
stewie925its a 1 hr 34 min video :D23:19
dougshelley66np - so if i look at that you have a conflict in cfg.py23:19
dougshelley66right23:19
dougshelley66it looks like you need to keep all of the ocde23:19
dougshelley66code23:19
dougshelley66default_password_length was added in master and you are trying to add the timeouts23:20
dougshelley66so just take the chevrons and equal signs out23:20
stewie925that is right, I need to add the timeouts23:20
dougshelley66so the conflict is because default_password_length was added in the master23:21
dougshelley66and it is in the same place as where the timeouts were added23:22
dougshelley66so you just need to take both23:22
*** sgotliv has quit IRC23:29
stewie925whew - finally resolved the conflicts23:34
stewie925:)23:34
*** khushbu_ has quit IRC23:36
stewie925very interesting though....  after conflicts are fixed, I ran 'git rebase --continue'23:37
stewie925then I ran git status and no modules are listed23:37
dougshelley66thats ok23:40
dougshelley66do git log23:40
dougshelley66you will see the commit at the top23:40
stewie925yes!23:40
dougshelley66so run tox23:40
dougshelley66tox -epep823:40
dougshelley66tox -epy2723:41
stewie925fixing some pep8 structural errors23:55

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