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

Telemetry service directory override. #636

Merged
merged 1 commit into from
May 2, 2024

Conversation

jshort
Copy link
Collaborator

@jshort jshort commented May 2, 2024

In addition to opt-in for telemetry (see #553) this uses the log directory override from #556 to place the sentry/telemetry logs.

In addition to opt-in for telemetry (see MisterTea#553) this uses the log
directory override from MisterTea#556 to place the sentry/telemetry logs.
@jshort
Copy link
Collaborator Author

jshort commented May 2, 2024

Updated the release instructions to appease CI job.

@jshort
Copy link
Collaborator Author

jshort commented May 2, 2024

❯ ./etserver --port 8099 --serverfifo=./test/fifo --telemetry -l test &                                                                                             09:27:55  05.02.24  100% █
[1] 43384
 ✔  ⚙  jwshort@LAPTOP  ~/gitworkspaces/EternalTerminal/build   telemetry_dir 
 ❯ ls test                                                                                                                                                           09:28:33  05.02.24  100% █
etserver-2024-05-02_09-28-33_43384.log         etserver-stderr-2024-05-02_09-28-33_43384.log  fifo=
 ✔  ⚙  jwshort@LAPTOP  ~/gitworkspaces/EternalTerminal/build   telemetry_dir 
 ❯ ls -la test                                                                                                                                                       09:28:37  05.02.24  100% █
total 16
drwxr-xr-x   6 jwshort  staff  192 May  2 09:28 ./
drwxr-xr-x  26 jwshort  staff  832 May  2 09:23 ../
drwx------   5 jwshort  staff  160 May  2 09:28 .sentry-native-etserver/
-rw-------   1 jwshort  staff  393 May  2 09:28 etserver-2024-05-02_09-28-33_43384.log
-rw-------   1 jwshort  staff   31 May  2 09:28 etserver-stderr-2024-05-02_09-28-33_43384.log
srwxrwxrwx   1 jwshort  staff    0 May  2 09:28 fifo=

@jshort
Copy link
Collaborator Author

jshort commented May 2, 2024

@MisterTea Any idea on this? I saw you touched the codecov.yml recently and maybe we need to do some configuration/setup?

{'detail': ErrorDetail(string='Rate limit reached. Please upload with the Codecov repository upload token to resolve issue. Expected available in 3556 seconds.', code='throttled')}{'detail': ErrorDetail(string='Rate limit reached. Please upload with the Codecov repository upload token to resolve issue. Expected available in 0 seconds.', code='throttled')}{'detail': ErrorDetail(string='Rate limit reached. Please upload with the Codecov repository upload token to resolve issue. Expected available in 3598 seconds.', code='throttled')}{'detail': ErrorDetail(string='Rate limit reached. Please upload with the Codecov repository upload token to resolve issue. Expected available in 0 seconds.', code='throttled')}{'detail': ErrorDetail(string='Rate limit reached. Please upload with the Codecov repository upload token to resolve issue. Expected available in 3599 seconds.', code='throttled')}{'detail': ErrorDetail(string='Rate limit reached. Please upload with the Codecov repository upload token to resolve issue. Expected available in 0 seconds.', code='throttled')}

@MisterTea MisterTea merged commit 2ff796a into MisterTea:master May 2, 2024
23 of 24 checks passed
@MisterTea
Copy link
Owner

@jshort looks like we are being throttled by codecov?

@MisterTea
Copy link
Owner

@jshort Fixed in #637

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants