releases set-commits <commit> --auto --ignore-missing fails with error: could not find repository at '.'; class=Repository (6); code=NotFound (-3)
#2157
Comments
|
Hi, the |
|
@lforst I'm pretty sure we're just printing the error generated by the library here. We can likely format it better |
This used to work in the context of a docker build and suddenly stopped working after we renamed the repository. See https://github.com/kentcdodds/kentcdodds.com/blob/34014471facce523e1401299431ac4a5d0bd1943/Dockerfile#L57 for an open source example of a very similar setup. We also tried setting auto to false and manually specifying the commit but it also fails with the same error. |
|
@felangel I don't entirely see how the kentcdodds repo is similar. They're not collecting commit data for Sentry as far as I can see. My guess is that you used to copy your |
They’re also using the vite plugin and specifying setCommits to auto: https://github.com/kentcdodds/kentcdodds.com/blob/34014471facce523e1401299431ac4a5d0bd1943/vite.config.ts#L49 Our Dockerfile hasn’t changed unfortunately but I’ll take a closer look at the container in a few minutes. I can also try to make a minimal reproduction sample later today, thanks! |
|
@lforst the likely culprit seems to be the fact that we moved to a monorepo with multiple apps which means the directory we build from no longer contains a |
|
@felangel The vite plugin needs to run inside a git folder to be able to associate commits. It is an abstraction around Sentry CLI. You can work around this restriction by using Sentry CLI and associating the commits after the fact: https://docs.sentry.io/product/releases/associate-commits/ |
CLI Version
2.36.1
Operating System and Architecture
Operating System Version
debian:12
Link to reproduction repository
No response
CLI Command
sentry-cli releases set-commits --auto --ignore-missing
Exact Reproduction Steps
apps/my-app)npx @sentry/wizard@latest -i remix --url https://sentry.ioDockerfile.sentry_auth.txtdocker build --build-arg COMMIT_SHA=<COMMIT> --secret id=SENTRY_AUTH_TOKEN,src=.sentry_auth.txt .Additional Context
vite.config.tspackage.json{ "private": true, "sideEffects": false, "type": "module", "scripts": { "build": "remix vite:build", "dev": "remix vite:dev --port 3000", "lint": "eslint --ignore-path .gitignore --cache --cache-location ./node_modules/.cache/eslint .", "start": "remix-serve ./build/server/index.js", "typecheck": "tsc", "format": "prettier --write .", "format:check": "prettier --check .", "test": "vitest" }, "eslintIgnore": [ "/node_modules", "/build", "/public/build" ], "dependencies": { "@remix-run/node": "^2.12.0", "@remix-run/react": "^2.12.0", "@remix-run/serve": "^2.12.0", "@sentry/remix": "^8.30.0", "apexcharts": "^3.48.0", "currency.js": "^2.0.4", "decimal.js": "^10.4.3", "glob": "^11.0.0", "highlight.js": "^11.9.0", "isbot": "^5.1.3", "react": "^18.2.0", "react-apexcharts": "^1.4.1", "react-dom": "^18.2.0", "rehype-highlight": "^7.0.0", "rehype-raw": "^7.0.0", "rehype-stringify": "^10.0.0", "remark-gfm": "^4.0.0", "remark-parse": "^11.0.0", "remark-rehype": "^11.1.0", "remix-auth": "^3.4.0", "remix-auth-google": "^2.0.0", "remix-auth-microsoft": "^2.0.0", "remix-redis-session": "^0.0.12", "remix-typedjson": "^0.4.1", "remix-utils": "^7.5.0", "stripe": "^14.23.0", "unified": "^11.0.4" }, "devDependencies": { "@remix-run/dev": "^2.12.0", "@remix-run/eslint-config": "^2.12.0", "@remix-run/testing": "^2.12.0", "@sentry/vite-plugin": "^2.22.4", "@tailwindcss/typography": "^0.5.12", "@testing-library/jest-dom": "^6.4.2", "@testing-library/react": "^14.2.2", "@types/react": "^18.2.20", "@types/react-dom": "^18.2.7", "@vitejs/plugin-react": "^4.3.1", "@vitest/coverage-v8": "^2.0.5", "autoprefixer": "^10.4.19", "daisyui": "^3.9.2", "eslint": "^8.57.0", "eslint-config-prettier": "^9.1.0", "eslint-import-resolver-typescript": "^3.6.1", "eslint-plugin-import": "^2.28.1", "eslint-plugin-jsx-a11y": "^6.7.1", "eslint-plugin-react": "^7.33.2", "eslint-plugin-react-hooks": "^4.6.0", "jsdom": "^25.0.0", "postcss": "^8.4.38", "prettier": "^3.2.5", "prettier-plugin-tailwindcss": "^0.5.13", "tailwindcss": "^3.4.4", "typescript": "^5.4.3", "vite": "^5.1.0", "vite-tsconfig-paths": "^4.2.1", "vitest": "^2.0.5" }, "optionalDependencies": { "@rollup/rollup-linux-x64-gnu": "^4.21.2" }, "engines": { "node": ">=20.0.0" } }Expected Results
I expect the build to succeed and successfully upload source maps and commit information to sentry.
Actual Results
Build fails with error:
Additional Context
The build succeeds if we remove
setCommitsfrom the build step:import { glob } from 'glob'; import { vitePlugin as remix } from '@remix-run/dev'; import { defineConfig } from 'vite'; import tsconfigPaths from 'vite-tsconfig-paths'; import { sentryVitePlugin } from '@sentry/vite-plugin'; const MODE = process.env.NODE_ENV; export default defineConfig({ plugins: [ remix({ future: { v3_fetcherPersist: true, v3_relativeSplatPath: true, v3_throwAbortReason: true, }, }), tsconfigPaths(), process.env.SENTRY_AUTH_TOKEN ? sentryVitePlugin({ debug: true, disable: MODE !== 'production', authToken: process.env.SENTRY_AUTH_TOKEN, org: process.env.SENTRY_ORG, project: process.env.SENTRY_PROJECT, release: { name: process.env.COMMIT_SHA, - setCommits: { - auto: true, - ignoreMissing: true, - }, }, sourcemaps: { filesToDeleteAfterUpload: await glob([ './build/**/*.map', '.server-build/**/*.map', ]), }, }) : null, ], build: { sourcemap: true, }, });Logs
The text was updated successfully, but these errors were encountered: