Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

It should be easier for system admins to debug startup failures as a result of a misplaced config file or incorrect config values. #217

Closed
Gnuxie opened this issue Jan 24, 2024 · 4 comments
Labels
L3 Some Users Likelihood P3 Outrageous / Cannot continue to or start to use Priority level - includes missing documentation leading to same outrage etc T6 Crash Bug causes crash OR data loss

Comments

@Gnuxie
Copy link
Member

Gnuxie commented Jan 24, 2024

https://matrix.to/#/!IaWNErZAgQUhGqJXjX:matrix.org/$yQISu-7x69vp8tq2mn5cRRNL6JVO_WdLNXUE4WCg5YM?via=matrix.org&via=envs.net&via=conduit.rory.gay

They should be able to get feedback on what config file is actually being seen and loaded by Draupnir

It should go without saying that documentation for figuring out how to do this should be included and visible directly from the setup instructions.

The config shown in output should have the password/access token and other sensitive fields redacted so that it can be pasted directly into our support room when we ask for logs.

Related: #218

@Gnuxie
Copy link
Member Author

Gnuxie commented Apr 12, 2024

it's also going to be essential to be able to read out the appdata #333. Though note, this is before Draupnir is created, so I'm not sure how we'd go about that.

@Gnuxie
Copy link
Member Author

Gnuxie commented Apr 12, 2024

Maybe we could have something that uses https://nodejs.org/api/process.html#event-beforeexit to try and dump account data, but we'd need another hook for appservice bots.

You also have to remember that there can be startup issues that happen before Draupnir or the MatrixClient are constructed.

@Gnuxie
Copy link
Member Author

Gnuxie commented Apr 12, 2024

Maybe the better way to do it would be to add a dependency to the MatrixAccountData utility per Draupnir that interns them and reads them all out at the "stopping due to fatal error" hook, whatever that might be.

Gnuxie added a commit that referenced this issue Apr 16, 2024
Part of #217.
I don't think this is right yet.
Gnuxie added a commit that referenced this issue Apr 16, 2024
* Update for MPS 0.17.1.

Part of #333
And #345

Fixes #337

* Add loggableConfigTracker to Draupnir.

Part of #217.
I don't think this is right yet.
@Gnuxie Gnuxie moved this to Planned in Draupnir planning May 18, 2024
@Gnuxie Gnuxie moved this from Planned to Scheduled in Draupnir planning May 18, 2024
@Gnuxie Gnuxie moved this from Scheduled to Planned in Draupnir planning Jul 29, 2024
@Gnuxie Gnuxie moved this from Planned to In progress in Draupnir planning Oct 4, 2024
@Gnuxie
Copy link
Member Author

Gnuxie commented Oct 8, 2024

Account data concerns have been covered by the-draupnir-project/planning#29

@Gnuxie Gnuxie closed this as completed Oct 11, 2024
@github-project-automation github-project-automation bot moved this from In progress to Done in Draupnir planning Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L3 Some Users Likelihood P3 Outrageous / Cannot continue to or start to use Priority level - includes missing documentation leading to same outrage etc T6 Crash Bug causes crash OR data loss
Projects
Status: Done
Development

No branches or pull requests

1 participant