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

Production 2024-11-19_01 #2714

Merged
merged 10 commits into from
Nov 20, 2024
Merged

Production 2024-11-19_01 #2714

merged 10 commits into from
Nov 20, 2024

Conversation

pavelbrm and others added 10 commits November 6, 2024 16:48
* chore: update x_net dep

* chore: use local toolchain
* chore: Update hashicorp/vault deps, use openbao sdk for tools

* Makefile: update golangci-lint to match CI

* go mod tidy

* chore: run go mod tidy

* chore: backport some changes from 2662

* chore: sync with master

* improve caching, fix docker user

* remove unnecessary step

---------

Co-authored-by: PavelBrm <pburmistrov@brave.com>
* feat: handle stripe invoice.payment_failed events

* review: increment numPaymentFailed in-place

* fix: fix test
When I ran the grants container I got these errors:
> DBG failed to dial: x509: certificate has expired or is not yet valid: current time 2024-10-29T18:07:57Z is after 2024-04-02T16:18:51Z module=kafka.NewKafkaReader
> DBG Unable to establish connection to consumer group coordinator for group grant-bat-skus-local: failed to dial: x509: certificate has expired or is not yet valid: current time 2024-10-29T18:08:02Z is after 2024-04-02T16:18:51Z module=kafka.NewKafkaReader
> DBG failed to dial: x509: certificate has expired or is not yet valid: current time 2024-10-29T18:08:02Z is after 2024-04-02T16:18:51Z module=kafka.NewKafkaReader

I verified that the certificates were expired:
```
$ openssl x509 -in test/secrets/snakeoil-ca-1.crt -noout -text | grep 'Not'
            Not Before: Apr  3 16:18:51 2023 GMT
            Not After : Apr  2 16:18:51 2024 GMT
```

To regenerate the certificates I used `test/secrets/README.md`
…ns (#2685)

* Update setup instructions

- Update minimum Go version (all setups require at least 1.22)
- `go get` is no longer supported outside a module, update to `go install`
- Update outdated path for payments in README and remove superfluous redirection character
- Update a couple of oudated paths in the `services/skus/README.md` file

* Remove unused docker-refresh-skus and docker compose files

- The Makefile was referencing a yml file that didn't exist.
- The skus README was specifying a command which was previously renamed.

I removed the Makefile entry and instead specify to run `make docker-dev`
and speicfy to run `cd main && go run main.go serve grant` in the
services/sku readme.

* Remove unused docker-refresh-dev and docker compose file

* Add typical development steps

Credit: Kit
@pavelbrm pavelbrm self-assigned this Nov 19, 2024
@pavelbrm pavelbrm requested a review from clD11 November 19, 2024 07:10
@pavelbrm pavelbrm marked this pull request as ready for review November 19, 2024 07:10
@pavelbrm pavelbrm added this pull request to the merge queue Nov 20, 2024
Merged via the queue into prod with commit e2745c6 Nov 20, 2024
18 checks passed
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.

4 participants