Setup Sentry secrets on CI

[igor-sirotin]

Sentry integration is done here: status-im/status-go#6054

Description

We want to keep Sentry DSN private, so we need to provide some environment variables on CI.

The following env vars need to be defined on PRODUCTION CI only.
Not for PR-level or develop builds.

• set SENTRY_DSN_STATUS_GO to the DSN from here
• set SENTRY_PRODUCTION to true

[jakubgs]

Depends on:

• Configure Sentry for Mobile #21682

So for now it's on hold.

[ilmotta]

Depends on:

• [DRAFT] Configure Sentry for Mobile #21682

So for now it's on hold.

@jakubgs @markoburcul the implementation in PR #21682 should work because I tested it locally pointing to a cloud Sentry instance. So we can proceed with this issue.

In status-desktop the make target building status-go is passing two new env vars SENTRY_CONTEXT_NAME and SENTRY_CONTEXT_VERSION. We need to do the same to build status-go for mobile. Could you assist me in how to implement this in status-go/mobile/build.nix? Actually, I would greatly appreciate if you could do the implementation on the nix layer 😅 Please, let me know if this is reasonable on your end. Thanks in advance 🙏🏼

The value for SENTRY_CONTEXT_NAME should be status-mobile and SENTRY_CONTEXT_VERSION should be whatever value is in the VERSION file, just as was implemented in status-desktop AFAIU.

Once this issue #21706 is resolved I will proceed to rebase develop onto #21682

---

@igor-sirotin in status-desktop it appears that SENTRY_ENVIRONMENT is not being used, only the env var SENTRY_PRODUCTION. In status-mobile I opted to follow the table documented in status-go to define the values of SENTRY_ENVIRONMENT for each .env.* file. Is that the idea or have I misunderstood something?

[markoburcul]

Could you assist me in how to implement this in status-go/mobile/build.nix?

I don't see that file, do you want to create it or? Since in status-desktop as I can see those two env variables are passed in the Makefile.

[ilmotta]

Could you assist me in how to implement this in status-go/mobile/build.nix?

I don't see that file, do you want to create it or? Since in status-desktop as I can see those two env variables are passed in the Makefile.

Thanks @markoburcul. The file is in this status-mobile repo at nix/status-go/mobile/build.nix, but when I copied I accidentally removed the parent path nix/.

[markoburcul]

Thanks @markoburcul. The file is in this status-mobile repo at nix/status-go/mobile/build.nix, but when I copied I accidentally removed the parent path nix/.

could you give me a link to this file on Github?

[ilmotta]

Thanks @markoburcul. The file is in this status-mobile repo at nix/status-go/mobile/build.nix, but when I copied I accidentally removed the parent path nix/.

could you give me a link to this file on Github?

@markoburcul it's this file

status-mobile/nix/status-go/mobile/build.nix

Line 59 in d0deb94

gomobile bind \

[igor-sirotin]

in status-desktop it appears that SENTRY_ENVIRONMENT is not being used, only the env var SENTRY_PRODUCTION.

@ilmotta indeed, yes. I didn't bother with other environments for now, because we don't yet report to sentry from e2e tests.
I'll do the same later when we will set it up for tests.

I opted to follow the table documented in status-go

❤