-
Notifications
You must be signed in to change notification settings - Fork 6
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
Adding expo-datadog breaks my build on expo 49 #29
Comments
Hi @wcastand, thanks for reaching out!
The error message you get means that the path to the Can you check if it's applied locally? |
will keep you inform but looks like i fixed the version and the path, now for some reason he says i don't have a valid api_key when it's the one from datadog application setup. So must have missed something. if i still can't build after fixing this env thing, i'll comeback to you with hopefully more info :) EDIT: i don't get why i check by console.log what the api key the script receive and it's the one from the datadog setup RUM page, but it sayds it's invalid so not sure what is happening |
Hi @wcastand, You indeed need to specify an API key to upload the sourcemaps to Datadog. We recommend to run Note: This is an API key, which is different from the RUM application id. You can check our docs on how to get an API key. If you have already done that and your key is still invalid, I would recommend you to reach out to our support team so they can have a look into what's wrong here. |
hmm ok let me try but that's quite confusing :/ same for the patch for monorepo, it should probably be in the docs, a lot of expo app are on monorepo as far as i know. let me try to setup that then |
I think i fixed ios build but android is still failing on the sourcemaps
i don't know if it's the monorepo setup again that make the path fail. i can't find where yo modify the path since it's dynamic with
and it looks like the issue is maybe in the datadog-ci cli and not the expo-datadog this time but not sure. looks like it's looking for datadog-ci in from what i see the issue is in
looks like but it seems to return the wrong path, getting : looking at how sentry is doing it, in the plugin for expo they add a their cli path as a project.ext to get the right absolute path it seems. in my bbuild.gradle generated by expo i have this from sentry-expo which i assumed give the right path to use for the script later on when it comes to uploading sourcemaps in android. maybe you need to do something similar to handle monorepo in android expo app |
Hi @wcastand, thanks for taking the time to investigate into this, I think what you describe is very likely the root cause. I believe we should handle monorepo setups for Expo apps better, I'll bump the priority for this in our backlog. The approach from Sentry seems like a good one indeed, thanks for looking into it! However we likely won't have the bandwidth to prioritize this straight away, so to get you unblocked in the meantime, can you try to apply this patch to
Let me know if that unblocks you, that would also validate the root cause of the issue at the same time :) |
@louiszawadzki i can confirm, this fixes the issue on eas, i managed to build without fail once i made the patch. |
worth noting just in case, for ios the path change was a bit different than the one you proposed in the other issue for me it was because there is the so it would depend on people setup for sure, can't be an constant or absolute value anymore if you want to handle monorepos |
Ran into the exact same issue and confirmed it was fixed by a similar patch, although my directory structure was slightly different from @wcastand 's. We use yarn workspaces, and recently upgraded from yarn classic to yarn berry. It looks like the It seems like a long-term fix needs to be smart enough to traverse up the workspace tree to figure out where to find the Edit: unfortunately am running into a similar issue for iOS, trying to find the same place where it's being modified:
|
I'm using expo sdk 49
my build fails on android and ios as soon as i try to add
expo-datadog
I'm in a monorepo so according to the issue about monorepo i added the patch. #17
datadog works fine if i don't add expo-datadog or set to false the sourcemaps, dSym config options.
On ios it complains avec about minimum version being set to ios 11 when most dependencies and expo itself only support minimum version of 12 or 13.
It looks like i can build the app if i set to false all the sourcemaps and dSym config.
On android, i can build locally for dev but on
eas
it fails.i tried to use
expo-build-properties
to force :but it didn't help no matter the order of the plugins.
Expo SDK: 49
"@datadog/mobile-react-native": "^2.2.0",
"expo-datadog": "^50.2.0",
"@datadog/datadog-ci": "^2.32.0",
on android the error is
ios logs (they are so long they don't git into my terminal history so i just grabed the last bit, if you need more let me know but it's like really long)
The text was updated successfully, but these errors were encountered: