Last modified: 2014-05-06 15:39:52 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 T45956, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 43956 - rename $wmfRealm to $wmgRealm in mediawiki-config.git
rename $wmfRealm to $wmgRealm in mediawiki-config.git
Status: NEW
Product: Wikimedia
Classification: Unclassified
Site requests (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: ---
Assigned To: Aravind K N
:
Depends on: 39106
Blocks:
  Show dependency treegraph
 
Reported: 2013-01-14 12:23 UTC by Antoine "hashar" Musso (WMF)
Modified: 2014-05-06 15:39 UTC (History)
7 users (show)

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


Attachments
Renamed $wmf to $wmg in mediawiki-config (65.33 KB, patch)
2013-11-08 14:03 UTC, Aravind K N
Details
Renamed $wmf to $wmg in mediawiki-config (9.02 KB, patch)
2013-11-09 15:55 UTC, Aravind K N
Details

Description Antoine "hashar" Musso (WMF) 2013-01-14 12:23:08 UTC
With https://gerrit.wikimedia.org/r/#/c/39056/ we renamed the old $realm to $wmfRealm. Tim mentioned:

>  It should be $wmgRealm, for consistency with other globals.
> wmg = Wikimedia global, wmf = Wikimedia function.


We probably want to do that after the EQIAD migration though.
Comment 1 Dereckson 2013-01-14 16:30:15 UTC
[ Set tracking bug 39106 (EQIAD migration) as blocker. ]
Comment 2 Brad Jorsch 2013-01-14 18:34:15 UTC
If you do this, you should also rename the following:

$wmfConfigDir
$wmfDatacenter
$wmfExtendedVersionNumber
$wmfHostnames
$wmfSwiftBigWikis
$wmfSwiftConfig
$wmfSwiftShardCommon
$wmfSwiftShardLocal
$wmfUdp2logDest
$wmfVersionNumber
$wmfThrottlingExceptions
$wmfStandardAutoPromote
$wmfUseArticleCreationWorkflow

It looks to me like "$wmg" variables are used for particular features, e.g. which extensions to enable or setting a particular "$wg" variable. And except for the last three, it looks like "$wmf" variables are used for global configuration.

Don't forget to check extensions/WikimediaMaintenance and /usr/local/bin stuff too.
Comment 3 Aravind K N 2013-11-05 11:48:16 UTC
I have already mailed people who are on the cc list of this bug. One of them told me to comment on this so that this bug would be assigned to me. So, could someone assign this bug to me?
Comment 4 Andre Klapper 2013-11-05 12:32:01 UTC
(In reply to comment #3)
> I have already mailed people who are on the cc list of this bug.

That's not a good idea if there is nothing private or off-topic to discuss - instead, comment on this bug report. Assigned to you now.
Comment 5 Aravind K N 2013-11-06 13:14:15 UTC
Could you please mentor me for this bug fixing?
Comment 6 Dereckson 2013-11-06 13:35:01 UTC
Hello,

I can mentor you for this bug.

Please poke me on IRC. Dereckson (I'm on #wikimedia-tech and #wikimedia-dev, I figure the first is more indicated for this kind of configuration bug).
Comment 7 Aravind K N 2013-11-06 13:36:39 UTC
Is it okay if I mail you?
Comment 8 Andre Klapper 2013-11-06 14:27:31 UTC
I don't see much sense in relying on one person only in private, when there's
an unlimited number of people who can help you in public. 
The knowledge of the crowd is one of the advantages of free software. :)
Comment 9 Dereckson 2013-11-06 14:38:19 UTC
I concur.

A little explanation for the bug status. We use:
* UNCONFIRMED for bugs still needing clarification
* NEW for (confirmed) new bugs
* ASSIGNED for bug someone is working on (so if there is an assignee, status should be ASSIGNED, to avoid someone else meanwhile work on it)
* PATCH_TO_REVIEW when bug has a code solution, pending code review
* RESOLVED when the bug has been solved
* VERIFIED for the areas using quality assurance, after the QA occurred
Comment 10 Aravind K N 2013-11-08 14:03:33 UTC
Created attachment 13733 [details]
Renamed $wmf to $wmg in mediawiki-config

Renamed $wmf* to $wmg* for consistency with other global variables.
$wmf= 	Wikimedia function
wmg=   Wikimedia global
Comment 11 Gerrit Notification Bot 2013-11-09 04:57:49 UTC
Change 94354 had a related patch set uploaded by TTO:
Renamed $wmfConfigDir to $wmgConfigDir in mediawiki-config

https://gerrit.wikimedia.org/r/94354
Comment 12 Gerrit Notification Bot 2013-11-09 14:33:07 UTC
Change 94503 had a related patch set uploaded by Arav93:
Renamed $wmf* to $wmg* to improve consistency

https://gerrit.wikimedia.org/r/94503
Comment 13 Gerrit Notification Bot 2013-11-09 15:33:19 UTC
Change 94503 abandoned by Arav93:
Renamed $wmf* to $wmg* to improve consistency

Reason:
Revision added : https://gerrit.wikimedia.org/r/#/c/94354/

https://gerrit.wikimedia.org/r/94503
Comment 14 Aravind K N 2013-11-09 15:55:10 UTC
Created attachment 13753 [details]
Renamed $wmf to $wmg in mediawiki-config
Comment 15 Andre Klapper 2013-11-09 17:52:03 UTC
(In reply to comment #14)
> Created attachment 13753 [details]
> Renamed $wmf to $wmg in mediawiki-config

Patch is as broken as the patch that was uploaded to Gerrit, including lots of noise from conflicts like "<<<<<<< HEAD". 
Please follow http://www.mediawiki.org/wiki/Gerrit/Tutorial and upload patches to Gerrit instead of Bugzilla, and fix potential issues in them. Thanks.
Comment 16 Aravind K N 2013-11-10 07:44:12 UTC
Ok, Sure.
Comment 17 Gerrit Notification Bot 2013-11-10 07:45:58 UTC
Change 94354 abandoned by Arav93:
Renamed $wmfConfigDir to $wmgConfigDir in mediawiki-config

Reason:
Uploading new patch.

https://gerrit.wikimedia.org/r/94354
Comment 18 Gerrit Notification Bot 2013-11-10 08:17:31 UTC
Change 94598 had a related patch set uploaded by Arav93:
Renamed $wmf* to $wmg*

https://gerrit.wikimedia.org/r/94598
Comment 19 Aravind K N 2013-11-10 08:58:25 UTC
Could someone review? :)
Comment 20 Andre Klapper 2013-11-10 10:30:46 UTC
Please don't ask for review after 40 minutes. Every comment creates bugmail and it is obvious to everybody anyway that there is a patch due to comment 18.

Also, as I had written several times on IRC, please AMEND changes instead of creating completely new numbers in Gerrit: https://www.mediawiki.org/wiki/Git/Tutorial#Amending_a_change
Comment 21 Andre Klapper 2013-11-10 10:36:27 UTC
As mentioned three times now (comment 2, and twice in Gerrit), this change also requires a corresponding change to the mediawiki/extensions/WikimediaMaintenance project.
Comment 22 Aravind K N 2013-11-10 12:53:34 UTC
How am I supposed to do that?
Comment 23 Aravind K N 2013-11-10 12:54:07 UTC
And I'm sorry about spamming.
Comment 24 Gerrit Notification Bot 2013-11-10 13:58:51 UTC
Change 94609 had a related patch set uploaded by Arav93:
Renamed $wmf* to $wmg* for consistency. Bug 43956 required a change in this repository as well.

https://gerrit.wikimedia.org/r/94609
Comment 25 Gerrit Notification Bot 2013-12-16 07:37:11 UTC
Change 94598 abandoned by Arav93:
Renamed $wmf* to $wmg*

Reason:
Adding an improved patch.

https://gerrit.wikimedia.org/r/94598
Comment 26 Aravind K N 2014-01-08 13:31:20 UTC
Could someone tell why this (https://gerrit.wikimedia.org/r/#/c/101820/) cannot be merged?
Comment 27 Andre Klapper 2014-01-08 14:28:59 UTC
Somebody still needs to review the patch...
Comment 28 Aravind K N 2014-01-09 12:20:37 UTC
Are you sure that's the only reason?

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


Navigation
Links