Page MenuHomePhorge

Phorge
PublicGeneral discussions about Phorge. For long and complex questions, you may want to use Ponder.

Cigaryno14:39

I had to revert to my old (insecure) VCS password to run arc land to D25874. Still not sure on making Git ask for credentials for a Phorge install.

RhinosF114:47

If it's windows, search for credential manager in the start menu and then delete / change the relevant credential. There may be more than one.

Feb 15th, 2025
avivey09:44

why didn't you set the Reviewers field on it?

Cigaryno09:48

I can't land as the push policy for rHP is Blessed Committers. Could it be lowered to Trusted Contributors?

avivey11:17

yeah, sure

Feb 17th, 2025
aklapper00:00

I'd appreciate if someone could accept https://we.phorge.it/D25885 . Looks like I again screwed up somehow when rebasing (Would love to see "bin/celerity map" run somehow automagically)

valerio.bozzolan09:19

Are you still able to run ./bin/diviner generate in latest master arcanist/phorge locally?

valerio.bozzolan09:21

Using --trace I see that it remains stuck after the Phorge Contributor Documentation. Then I get a Lock wait timeout exceeded

valerio.bozzolan09:45

OK nothing, I've just a monster query running since hours in my laptop on diviner_liveatom and stucking everything (even if it does not seem a good news lol)

Cigaryno18:56

Do you know how often should this install be updated to the latest master commits (T15092)? How would Almanac, Drydock, and/or Harbormaster be used for this?

aklapper20:08

I'd welcome your opinionated, heated arguments on https://we.phorge.it/T15998 (adding an "Authored Tasks" item to the user profile menu panel) in the spectrum between "omg yesh lol" and "hell no!!!1". TIA!

Feb 18th, 2025
avivey09:21

Do you know how often should this install be updated to the latest master commits

Whenever I feel like doing it, which is usually the same time as we promote to stable.

Feb 19th, 2025
Cigaryno18:52

Should the .gitignore file in both rP and rARC have the entry ./vscode/settings.json by default to help users with the CRLF end of line problem (like me)?

amybones22:42

I'd be in favor of actually the whole .vscode directory and maybe even .code-workspace being in .gitignore. Alternatively, including an .editorconfig or even some a settings file for vscode (and other editors) might be really helpful, too.

Feb 20th, 2025
avivey07:28

The gitignore files include this message:

# NOTE: Thinking about adding files created by your operating system, IDE,
# or text editor here? Don't! Add them to your per-user .gitignore instead.

which addresses this issue.
The idea is, since .vscode, __MACOSX, .idea or whatever are machine- or user-specific, they should be in the machine- or user-specific gitignore file.
The default location is $XDG_CONFIG_HOME/git/ignore or $HOME/.config/git/ignore. see here for more.

p.s. we do have an .editorconfig file in both repos.

avivey07:29

Also not sure how adding it to gitignore will help with CRLF problem.

Cigaryno19:00

Whenever I feel like doing it, which is usually the same time as we promote to stable.

Should the frequency of updating this install be rediscussed? I think it's preferable to have a install to diagnose the latest of master for bugs without having to install Phorge yourself.

aklapper19:24

I don't think that using a production instance for testing is a good idea.

Cigaryno19:34

I think neither was it to always run the master branch on Phacility SaaS instances.

Feb 21st, 2025
avivey06:49

It might make sense to have a separate install for testing, or something like that. But if a bug happens that makes the site not load, or makes it hard to fix the bug, we'd be in a problem.

Feb 22nd, 2025
Cigaryno15:52

The number of accounts created every day on this install with no activity seems to be suspiciously high in my opinion (but some clearly look to be created using GitHub OAuth given their profile pictures). What about you? Could there be accounts created by spambots?

aklapper17:50

Sure, some could be, as in any system which allows registration. :) We've also seen lots of accounts created with no activity on our instance, for years. We face some spam and vandalism but not on the level I'd expect given the number of created accounts.

Feb 23rd, 2025
• l2dy03:36

@Cigaryno Was Q79 closed because the situation has improved, or simply out of inactivity?

Cigaryno18:18

In Z1#3206, @l2dy wrote:
@Cigaryno Was Q79 closed because the situation has improved, or simply out of inactivity?

Because the answer appears to be answered.

Feb 28th, 2025
Cigaryno14:22

Is it normal for arc unit on rP Phorge to call MySQL?

valerio.bozzolan14:23

Yup. Set it up your local Phorge so you can have extra fun with tests that need a database

Mar 1st, 2025
taavi10:59

any plans to promote a new stable? it's been a while since the last release

aklapper13:12

I'd say at least the regression in T15985 should get fixed before a new stable, and ideally a few more open patches get reviewed (and merged), we just all need more spare time :-/

Cigaryno16:07

Also should we change the naming for releases from YYYY.WW (year and week) to YYYY.MM (year and month) as we don't promote to stable as often as prior to the end of Phabricator maintenance?

PixDeVl16:12

That seems logical considering from taavi’s comment new releases are pretty spread out, if we happen to have 2 releases in one month, we could maybe do YYYY.MMb or something

Cigaryno16:42

What to do to avoid MySQL error 1045 with arc unit (I don't have Phorge install and I am using a Windows computer)?

avivey17:40

@Cigaryno which test/test engine is getting it?

Cigaryno18:58

The default phutil lint engine and likely tests ran at \src\__tests__. In addition, when I run arc lint with the --trace option, I get this warning: Unable to install signal handler, pcntl_signal() unavailable..

Cigaryno18:59

I mean unit engine, not lint engine.

avivey19:05

that's just a warning, most things should still work. it relates to things like ctrl-c, I think.
I can't reproduce the mysql think ( I have linux, but no mysql).

Mar 2nd, 2025
Cigaryno12:40

I accidentally wrote arc lint while I actually mean arc unit. arc unit is where I get the aforementioned pcntl_signal() warning. Could it be the cause of MySQL error 1045 with arc unit?

avivey12:42

I tried with arc unit....
I do not expect any test in rARC to try to connect to MySQL, only tests in rP.

Cigaryno12:43

I am talking about rP.

avivey12:46

that's different, I don't think it relates to the pcntl thing. Probably just not fully set-up in mysql.
1045 is "permissions" issue - the test is trying to create lots of temporary DBs (all of them) and needs access to do that.
It should be using the creds from config/local.json.

Juest21:14

uhhhh hey been a while, i just got myself a really weird bug with the chat system on edge

Juest21:15

now its normal again on small window but bad in full view

Juest21:37

huh suddenly it fixed itself completely

Juest21:38

and it broke again when resizing o_O

Mar 3rd, 2025
aklapper09:38

Would anyone have time to review some one-liner bug fix patches? https://we.phorge.it/differential/?ids=25892,25894,25867,25752,25749,25742,25868#R Thanks in advance!

aklapper13:49

Stupid question: Anyone knows how to get some table representation for burndown charts? /maniphest/report/burn/ seems to only offer a chart but no table. formatBurnRow() in ManiphestReportController computes a lot of rows with stuff like <span class="red">+144</span> and so far I wonder why

valerio.bozzolan14:01

I don't think there is such table right now

aklapper14:27

This entire code is so unperformant, let's see how much time I can spend looking into it

aklapper20:23

Thanks, right, there cannot be a table as that exact variable gets overwritten later with a chart, ahaha. That's quite some dead plus expensive code in there. Time for a patch tomorrow

Juest22:48

ugh, its annoying how i need to reload to clear new chat message notifications

Mar 4th, 2025
valerio.bozzolan07:54

@Juest I agree. It's also strange that that counter updates if there are new unread messages, but does not reset if there aren't anymore. task welcome I guess :D

Mar 5th, 2025
Juest20:59

not sure how to formulate the task tbh @valerio.bozzolan

Mar 6th, 2025
Cigaryno10:53

@Juest on Maniphest, click on the Create Task dropdown and select Create Task.

Cigaryno13:52

The secure.phabricator.com server is out of storage: I can't go to the Welcome page on Phriction or even the changelog. Also when going to a large commit, only some files load and I get the error mkdir(): No space left on device.

Cigaryno13:53

Sorry for posting here, should have posted at Z5.

aklapper13:55

Well we are not phabricator.com but phorge.it, so wrong place :)

Mar 7th, 2025
Cigaryno12:56

Who wants to discuss the policy for contributing to prototype applications?: T16007

jbo21:02

Is there a way to either re-active a revoked SSH key or delete it permanently so it can be re-added?

jbo21:02

and I mean "any way at all".

jbo21:03

A user's SSH public key used to access git repos over SSH that is

avivey21:40

Can't be activated via the UI or any script, but if you have access to the db directly you can edit the row and flip the "is active" bit.
And you can probably delete it completely using bin/remove destroy by its phid.

jbo21:43

yes, I do have access to the DB

jbo21:43

my SQL is very rusty tho

jbo21:43

regarding bin/remove destroy, where can I get the phid from?

avivey21:46

If you have Developer Mode enabled and can see the page of the object, it's under the "Advanced/Developer -> view Hovercard"

avivey21:49

If you have the key number, you can find its phid using conduit auth.querypublickeys

avivey21:51

or you could provide public key to the same method in the keys field.

jbo22:49

yeah, I have full admin access (and SSH access) to the phorge instance.
I enabled developer mode. But I don't know how to get to the SSH key in the web UI anymore (it doesn't show up under my account's "SSH Public Keys" as I revoked it previously.

jbo22:50

I tried conduit's auth.querypublickeys with the keys field.
I only get this as a response:

{
  "data": [],
  "cursor": {
    "limit": 100,
    "after": null,
    "before": null
  }
}
jbo22:50

no error_code or error_info

Mar 8th, 2025
avivey07:45

You can put the user's phid under objectPHID field, I think that should find it too.

Cigaryno13:17

May a Blessed Roots (probably @avivey) configure inbound email (T15036#19958) if possible?

Mar 9th, 2025
jbo00:35

@avivey thanks, I was able to remove the previously revoked SSH key by using the developer mode and bin/remove destroy

jbo00:35

I appreciate the help!

taavi09:14

i can't edit https://we.phorge.it/w/changelog/next_up/ but it has T15988 in the patch column and D25865 in the task column.. could someone please fix that?

valerio.bozzolan10:13

Thanks taavi. 3 more remaining tips until you get "auto-"promoted as "blessed communicator" I guess \o/ lol

Mar 10th, 2025
Ekubischta13:23

All - I am attempting to land my revision D25898 but am getting an error. When running arc land, it asks me for my credentials, after entering them I get fatal: unable to access 'https://we.phorge.it/source/phorge.git/': The requested URL returned error: 403 any ideas?

valerio.bozzolan13:45

You probably cloned by HTTPs instead of SSH. If you like it you need a VCS password. Visit this:
https://we.phorge.it/source/phorge/
Clone → https://we.phorge.it → Manage Password (second URL)

valerio.bozzolan13:46

If you like SSH instead, update your .git/config to set remote URI ssh://git@we.phorge.it/source/phorge.git or something like that instead of HTTP

aklapper17:31

Would anyone know where in the DB Phorge stores the relation between the name of a custom Maniphest field and its indexKey/fieldIndex? I'm too stupid to find it... TIA.

Mar 13th, 2025
Cigaryno16:53

There is really no post on Security Announcements. Would this blog actually be useful for security advisories and urgent stable releases (ie. when a severe security issue has discovered and patched)?

valerio.bozzolan17:29

Short answer I think yes.
If the real answer if if a content is missing, I don't know. Maybe T15965 can be considered a security issue and I should write a note about it?

Content licensed under Creative Commons Attribution-ShareAlike 4.0 (CC-BY-SA) unless otherwise noted; code licensed under Apache 2.0 or other open source licenses. · CC BY-SA 4.0 · Apache 2.0