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.

Sign up for GitHub

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

Jump to bottom

Deployment #1677

Merged
merged 55 commits into from
Aug 2, 2024
Merged

Deployment #1677

merged 55 commits into from
Aug 2, 2024

Conversation

hugotiburtino
Copy link
Member

@hugotiburtino hugotiburtino commented Jul 16, 2024 •
edited
Loading

hugotiburtino and others added 30 commits July 12, 2024 11:50
@hugotiburtino
feat(cache): do not try to connect at creation and add error handling
f2f8c26
@hugotiburtino
feat(cache): hit the database while trying to reconnect
d13ce47 
The retries were reduced in order that the user get the data more
quickly.

Solves #1430
@AndreasHuber
push image to GitHub Container Registry instead of Google
324e0fa
@AndreasHuber
format
40485c6
@hugotiburtino
fix(cache): connect at init to fix tests but it is also unnecessary i…
573951d 
…n deploy
@hugotiburtino
test(create-cache): add test to show that api does not crash if redis…
742e096 
… is down
@hugotiburtino
style: format
9587c91
@hugotiburtino
test(create-cache): remove test that leads to error when running para…
8e0dfad 
…llel
@hugotiburtino
refactor!(api): remove automatic migration while locally developing
2ff9da9
@hugotiburtino
refactor: remove unused import and minor styling changes
ccb94f6
@hugotiburtino
chore(docker): fix remove with yarn install immutable cache
8722f1e
@hugotiburtino
Merge pull request #1676 from serlo/no-automatic-db-migration
244613f 
refactor!(api): remove automatic migration while locally developing
@hugotiburtino
Merge pull request #1675 from serlo/github-container-registry-poc
54f28bf 
push db-migration image to GitHub Container Registry instead of Google
@AndreasHuber
make Docker default repo name the same as given in the env variable w…
cf8422f 
…hen pushing via Github Action
@hugotiburtino
Merge pull request #1669 from serlo/redis-harderning
1f27bb7 
feat: Use the database while trying to reconnect to Redis
@hugotiburtino
Revert "feat: Use the database while trying to reconnect to Redis"
30c0f27
@hugotiburtino
Merge pull request #1684 from serlo/revert-1669-redis-harderning
c4992ad 
Revert "feat: Use the database while trying to reconnect to Redis"
@hugotiburtino
Revert "Revert "feat: Use the database while trying to reconnect to R…
20d4325 
…edis""
@dependabot
chore(deps-dev): bump @graphql-codegen/typescript-resolvers
e37d088 
Bumps [@graphql-codegen/typescript-resolvers](https://github.com/dotansimha/graphql-code-generator/tree/HEAD/packages/plugins/typescript/resolvers) from 4.1.0 to 4.2.1.
- [Release notes](https://github.com/dotansimha/graphql-code-generator/releases)
- [Changelog](https://github.com/dotansimha/graphql-code-generator/blob/master/packages/plugins/typescript/resolvers/CHANGELOG.md)
- [Commits](https://github.com/dotansimha/graphql-code-generator/commits/@graphql-codegen/[email protected]/packages/plugins/typescript/resolvers)

---
updated-dependencies:
- dependency-name: "@graphql-codegen/typescript-resolvers"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot
chore(deps-dev): bump esbuild from 0.21.5 to 0.23.0
4529daf 
Bumps [esbuild](https://github.com/evanw/esbuild) from 0.21.5 to 0.23.0.
- [Release notes](https://github.com/evanw/esbuild/releases)
- [Changelog](https://github.com/evanw/esbuild/blob/main/CHANGELOG.md)
- [Commits](evanw/esbuild@v0.21.5...v0.23.0)

---
updated-dependencies:
- dependency-name: esbuild
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot
chore(deps-dev): bump @types/node from 18.19.34 to 18.19.41
7cb4c9e 
Bumps [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) from 18.19.34 to 18.19.41.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node)

---
updated-dependencies:
- dependency-name: "@types/node"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot
chore(deps-dev): bump msw from 2.3.1 to 2.3.2
703f3c6 
Bumps [msw](https://github.com/mswjs/msw) from 2.3.1 to 2.3.2.
- [Release notes](https://github.com/mswjs/msw/releases)
- [Changelog](https://github.com/mswjs/msw/blob/main/CHANGELOG.md)
- [Commits](mswjs/msw@v2.3.1...v2.3.2)

---
updated-dependencies:
- dependency-name: msw
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot
chore(deps-dev): bump fp-ts from 2.16.6 to 2.16.8
14aab8e 
Bumps [fp-ts](https://github.com/gcanti/fp-ts) from 2.16.6 to 2.16.8.
- [Release notes](https://github.com/gcanti/fp-ts/releases)
- [Changelog](https://github.com/gcanti/fp-ts/blob/master/CHANGELOG.md)
- [Commits](https://github.com/gcanti/fp-ts/commits)

---
updated-dependencies:
- dependency-name: fp-ts
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
@hugotiburtino
feat(health): couple the health check with a operation that depends o…
4e99b8b 
…n redis
@hugotiburtino
Merge pull request #1690 from serlo/dependabot/npm_and_yarn/fp-ts-2.16.8
7df4477 
chore(deps-dev): bump fp-ts from 2.16.6 to 2.16.8
@hugotiburtino
Merge pull request #1688 from serlo/dependabot/npm_and_yarn/types/nod…
8c1bc59 
…e-18.19.41

chore(deps-dev): bump @types/node from 18.19.34 to 18.19.41
@hugotiburtino
Merge pull request #1691 from serlo/health-check-coupled-graphql
7af171a 
feat(health): couple the health check with a operation that depends on redis
@hugotiburtino
refactor(swr-queue): reset to autoconnect
37260b2 
After a first attempt, we got the error `this._ready.then is not a function`
which it related to `Object.queue`. We couldn't unfortunatelly reproduce
it locally, but since this was the only change made in Queue I hope
reverting it will avoid the error.
@AndreasHuber
possible improvements for enmeshed
bd9d533
@hugotiburtino
Merge pull request #1689 from serlo/dependabot/npm_and_yarn/msw-2.3.2
07dd78c 
chore(deps-dev): bump msw from 2.3.1 to 2.3.2
dependabot bot and others added 25 commits July 29, 2024 06:39
@dependabot
chore(deps-dev): bump typescript from 5.5.3 to 5.5.4
bab937d 
Bumps [typescript](https://github.com/Microsoft/TypeScript) from 5.5.3 to 5.5.4.
- [Release notes](https://github.com/Microsoft/TypeScript/releases)
- [Changelog](https://github.com/microsoft/TypeScript/blob/main/azure-pipelines.release.yml)
- [Commits](microsoft/TypeScript@v5.5.3...v5.5.4)

---
updated-dependencies:
- dependency-name: typescript
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot
chore(deps-dev): bump fp-ts from 2.16.8 to 2.16.9
67dacb8 
Bumps [fp-ts](https://github.com/gcanti/fp-ts) from 2.16.8 to 2.16.9.
- [Release notes](https://github.com/gcanti/fp-ts/releases)
- [Changelog](https://github.com/gcanti/fp-ts/blob/master/CHANGELOG.md)
- [Commits](https://github.com/gcanti/fp-ts/commits)

---
updated-dependencies:
- dependency-name: fp-ts
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot
chore(deps-dev): bump prettier from 3.3.2 to 3.3.3
df3b665 
Bumps [prettier](https://github.com/prettier/prettier) from 3.3.2 to 3.3.3.
- [Release notes](https://github.com/prettier/prettier/releases)
- [Changelog](https://github.com/prettier/prettier/blob/main/CHANGELOG.md)
- [Commits](prettier/prettier@3.3.2...3.3.3)

---
updated-dependencies:
- dependency-name: prettier
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
@hugotiburtino
Merge pull request #1695 from serlo/dependabot/npm_and_yarn/prettier-…
e76ca01 
…3.3.3

chore(deps-dev): bump prettier from 3.3.2 to 3.3.3
@hugotiburtino
Merge pull request #1693 from serlo/dependabot/npm_and_yarn/typescrip…
28c9e3e 
…t-5.5.4

chore(deps-dev): bump typescript from 5.5.3 to 5.5.4
@hugotiburtino
Merge pull request #1694 from serlo/dependabot/npm_and_yarn/fp-ts-2.16.9
a92490a 
chore(deps-dev): bump fp-ts from 2.16.8 to 2.16.9
@AndreasHuber
push server and swr images to GHCR instead of Google
3d89af2
@hugotiburtino
Merge pull request #1692 from serlo/possible-enmeshed-improvements
616546c 
possible improvements for enmeshed
@AndreasHuber
move spreadsheet 'model' into 'schema'
d47e59c
@hugotiburtino
Merge pull request #1697 from serlo/move-spreadsheet-model-into-schema
4d0b48f 
move spreadsheet 'model' into 'schema'
@hugotiburtino
chore(types): regenerate types with codegen
4e1fb50
@hugotiburtino
Merge pull request #1686 from serlo/dependabot/npm_and_yarn/graphql-c…
683332b 
…odegen/typescript-resolvers-4.2.1

chore(deps-dev): bump @graphql-codegen/typescript-resolvers from 4.1.0 to 4.2.1
@hugotiburtino
Merge pull request #1696 from serlo/push-server-and-swr-images-to-ghcr
c1fc70b 
push server and swr images to GHCR instead of Google
@hugotiburtino @AndreasHuber
chore(docker): adapt name of image tag
f15b4e9
@AndreasHuber
format
c6d9681
@dependabot @AndreasHuber
chore(deps): bump fast-xml-parser from 4.4.0 to 4.4.1
465431d 
Bumps [fast-xml-parser](https://github.com/NaturalIntelligence/fast-xml-parser) from 4.4.0 to 4.4.1.
- [Release notes](https://github.com/NaturalIntelligence/fast-xml-parser/releases)
- [Changelog](https://github.com/NaturalIntelligence/fast-xml-parser/blob/master/CHANGELOG.md)
- [Commits](NaturalIntelligence/fast-xml-parser@v4.4.0...v4.4.1)

---
updated-dependencies:
- dependency-name: fast-xml-parser
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
@AndreasHuber
pull Serlo MySQL database from GHCR
e89d379
@hugotiburtino
chore(deploy): rewrite to bash to stop after errors
4a00bd8 
Unfortunately, handling error in typescript script was quite complicated
and sometimes the CI passed, even though there was an error.
The bash script fails fast.

coauthor: ChatGPT for rewriting the code so fast.
@hugotiburtino
chore(deploy): minor dry in script
973cc1c
@hugotiburtino
Merge pull request #1687 from serlo/dependabot/npm_and_yarn/esbuild-0…
eb11547 
….23.0

chore(deps-dev): bump esbuild from 0.21.5 to 0.23.0
@hugotiburtino
Merge pull request #1702 from serlo/script-deploy-bash
e6c315c 
chore(deploy): rewrite to bash to stop after errors
@hugotiburtino
chore(ci): authenticate to github to push images
89843ab
@hugotiburtino
chore(ci): use serlo node configuration to be able again to use scrip…
47265fe 
…ts with yarn
@hugotiburtino
Merge pull request #1701 from serlo/pull-serlo-mysql-database-from-ghcr
aa6fda7 
pull Serlo MySQL database from GHCR
@hugotiburtino
Merge pull request #1685 from serlo/revert-1684-revert-1669-redis-har…
0a099e4 
…derning

feat: Use the database while trying to reconnect to Redis - Second Attempt
@hugotiburtino hugotiburtino requested a review from kulla August 2, 2024 08:44
@hugotiburtino hugotiburtino merged commit df5da0c into production Aug 2, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kulla kulla Awaiting requested review from kulla

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hugotiburtino @AndreasHuber