The environment cannot be switched anymore between "Prod" and "Stage" using the "XPCSHELL_TEST_PROFILE_DIR=foo" variable
Categories
(Firefox :: Remote Settings Client, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr91 | --- | unaffected |
firefox98 | --- | unaffected |
firefox99 | --- | verified |
firefox100 | --- | verified |
People
(Reporter: mcoman, Assigned: jdescottes)
References
(Regression)
Details
(Keywords: regression)
Attachments
(2 files)
469.75 KB,
image/gif
|
Details | |
48 bytes,
text/x-phabricator-request
|
dmeehan
:
approval-mozilla-beta+
|
Details | Review |
[Notes]:
- We received a workaround for this issue, more exactly we need to enter the
XPCOMUtils.defineLazyPreferenceGetter(ChromeUtils.import("resource://services-settings/Utils.jsm").Utils, "SERVER_URL", "services.settings.server")
code in the "Browser Console" before changing the environment. However, this workaround does not help us in the case of first-run experiments where we need to change the environment before opening the browser.
[Affected versions]:
- Firefox Beta 99.0b1 - Build ID: 20220308094525
[Affected Platforms]:
- Windows 10 x64
- macOS 11.6.2
- Linux Mint 20.2 x64
[Steps to reproduce]:
- Open a "Command Prompt" window ("Terminal" on macOS and Linux).
- Enter the `set XPCSHELL_TEST_PROFILE_DIR=foo" variable ("export XPCSHELL_TEST_PROFILE_DIR=foo" on macOS and Linux) and press the "Enter" key.
- Open the browser from the above "Command Prompt/Terminal".
- Install the "Remote Settings Devtools" addon.
- Click the "Remote Settings Devtools" toolbar button.
- Select the "Stage" option from the "Environment" dropdown placed in the top-right part of the page.
- Observe the behavior.
[Expected result]:
- The "Stage" environment is successfully selected.
[Actual result]:
- The "Prod" environment is still selected.
[Regression Window]:
- According to Ed Lee it seems that bug 1272255 has caused this behavior.
[Additional Notes]:
- Attached a screen recording of the issue:
Reporter | ||
Comment 1•2 years ago
|
||
Julian, could you please take a look over this issue?
Assignee | ||
Comment 2•2 years ago
|
||
Right, I mentioned this in https://bugzilla.mozilla.org/show_bug.cgi?id=1757741
The workaround to set XPCSHELL_TEST_PROFILE_DIR in order to be able to override the URL was not documented anywhere, and we switched to another environment variable: MOZ_DISABLE_NONLOCAL_CONNECTIONS and there is no workaround possible.
IMO we need another environment variable, here with a proper name and documentation.
Assignee | ||
Updated•2 years ago
|
Assignee | ||
Comment 3•2 years ago
|
||
Assignee | ||
Comment 4•2 years ago
|
||
Heads up, I am landing a fix which introduces a new env variable called MOZ_REMOTE_SETTINGS_DEVTOOLS
.
If you have existing documentation which refers to "set XPCSHELL_TEST_PROFILE_DIR=foo" or similar, it should be updated to say "set MOZ_REMOTE_SETTINGS_DEVTOOLS=1".
We will need to uplift the patch to Beta 99 before you can use it for testing though.
Reporter | ||
Comment 5•2 years ago
|
||
Great news! Thank you, Julian! We will update our documentation and we will verify this issue as soon as it will be uplifted to Beta 99.
Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f43486fc8a95 Check new env variable MOZ_REMOTE_SETTINGS_DEVTOOLS to allow overriding the remote settings server URL r=Gijs,leplatrem
Comment 7•2 years ago
|
||
bugherder |
Assignee | ||
Comment 8•2 years ago
|
||
Cross posting here: a workaround applicable to FF99 was shared at https://bugzilla.mozilla.org/show_bug.cgi?id=1757741#c6
(In reply to Ed Lee :Mardak from Bug 1757741 comment #6)
With bug 1757744 fixed, "Browser Toolbox" now shows up as "XPCShell Toolbox" when setting
XPCSHELL_TEST_PROFILE_DIR
. To get Remote Settings Devtools to switch to "Stage" environment on beta/release, here's a Browser Console workaround:XPCOMUtils.defineLazyPreferenceGetter( ChromeUtils.import("resource://services-settings/Utils.jsm").Utils, "SERVER_URL", "services.settings.server" )
https://github.com/mozilla-extensions/remote-settings-devtools/issues/44#issuecomment-1062060893
That workaround seems to be sufficient for testing with stage remote settings, but if not, we can file a new Remote Settings Client bug.
Assignee | ||
Comment 9•2 years ago
|
||
Marius, just want to check before requesting the uplift. Since there was a workaround shared that works with 99 (cf previous comment), do you think an uplift here is worth it? Or are you fine with using the workaround when testing with 99?
Reporter | ||
Comment 10•2 years ago
|
||
Hi Julian, I think we will need an uplift to Beta since we are using the "Stage" environment to test experiments and if a first-run experiment will arrive in Fx 99 we will need to change the environment before opening the browser. Thanks for all the help!
Assignee | ||
Comment 11•2 years ago
|
||
Makes sense! Will do
Assignee | ||
Updated•2 years ago
|
Assignee | ||
Comment 12•2 years ago
|
||
Comment on attachment 9267006 [details]
Bug 1758645 - Check new env variable MOZ_REMOTE_SETTINGS_DEVTOOLS to allow overriding the remote settings server URL
Beta/Release Uplift Approval Request
- User impact if declined: QA is unable to use the Remote Settings DevTools extension with Firefox 99.
I don't think it needs extra verifications since it's going to be used by QA when it lands, but the STRs are in the initial comment, just replace XPCSHELL_TEST_PROFILE_DIR=foo
with MOZ_REMOTE_SETTINGS_DEVTOOLS=1
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Small javascript change.
The only risky part is that the code checks the RELEASE_OR_BETA flag. I manually tested it on Nightly by "flipping" the RELEASE_OR_BETA
check in my patch and it worked as expected.
- String changes made/needed:
Updated•2 years ago
|
Comment 13•2 years ago
|
||
Set release status flags based on info from the regressing bug 1272255
Updated•2 years ago
|
Comment 14•2 years ago
|
||
Comment on attachment 9267006 [details]
Bug 1758645 - Check new env variable MOZ_REMOTE_SETTINGS_DEVTOOLS to allow overriding the remote settings server URL
Approved for 99.0b4. Thanks.
Comment 15•2 years ago
|
||
bugherder uplift |
Updated•2 years ago
|
Reporter | ||
Comment 16•2 years ago
|
||
I have verified that this issue is no longer reproducible with the latest Firefox Beta (99.0b4 Build ID - 20220315185755) installed on Windows 10 x64, macOS 11.6.4, and Ubuntu 20.04 x64. Now the environment can be successfully changed between "Prod" and "Stage" if the MOZ_REMOTE_SETTINGS_DEVTOOLS=1
variable is used.
Description
•