Last modified: 2014-08-20 00:51:59 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T67104, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 65104 - MediaWiki-Vagrant is changing my remotes
MediaWiki-Vagrant is changing my remotes
Status: RESOLVED FIXED
Product: MediaWiki-Vagrant
Classification: Unclassified
General (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: Ori Livneh
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-05-09 20:53 UTC by Kunal Mehta (Legoktm)
Modified: 2014-08-20 00:51 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Kunal Mehta (Legoktm) 2014-05-09 20:53:19 UTC
Introduced in I04cdee7b83e2b1c3503f4b73c902d08ec7028904 I believe.

I have a unique-ish setup, in which my `origin` remote points to either the github repo, or the https://gerrit.wikimedia.org repo, and the `gerrit` remote is over ssh.

I do this since my school blocks ssh to gerrit since it uses a non-standard port (that's another bug somewhere), so I always pull through `origin`.

Now whenever I run $ vagrant provision; it "fixes" my remotes, which leaves my setup broken.

There should be some kind of option to leave the remotes alone. I tried leaving my username blank (as the prompt claims this would do it "anonymous"-ly), but that left it in an even more broken state, pointing to "ssh://@gerrit.wikimedia.org:29418/mediawiki/extensions/Flow.git"
Comment 1 Yuri Astrakhan 2014-05-09 21:26:07 UTC
looking... seems the user='' is broken
Comment 2 Yuri Astrakhan 2014-05-09 21:41:21 UTC
Anonymous git user should be fixed with Change-Id: I16eb918f21eee23c0fb933744b1ae58c4813db6f  Would anonymous https://gerrit.wikimedia.org fix your issue?
Comment 3 Kunal Mehta (Legoktm) 2014-05-09 21:42:48 UTC
It would fix my current issue, but really vagrant shouldn't be changing my remotes unless I explicitly ask for it.
Comment 4 Yuri Astrakhan 2014-05-09 22:11:43 UTC
The fix has been merged.

I have altered the code to change remotes only if $remote parameter is not given in the clone parameters. This won't be very useful just yet (all callers of the clone object use default remote), so we could later alter it a bit further to introduce a global flag which also prevents this. I will leave it up to Ori to decide this one.
Comment 5 Kunal Mehta (Legoktm) 2014-05-11 06:31:38 UTC
It makes sense that if you're actually cloning a new repo that vagrant defines the remote to use, but in my case the repositories already exist. I fail to see why changing the remotes is even necessary or should be done at all.
Comment 6 Bryan Davis 2014-08-20 00:51:59 UTC
The manipulation of git remotes was reverted. Yuri continues to work on getting the feature back in, but I'm with Kunal that existing git clones should not be touched. The number of people this will be harmful/confusing for outweighs the number it would help in my opinion.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links