ereuse/IdHub
8
0
Fork 0
Code Issues 1 Pull requests 1 Packages Projects Releases Wiki Activity

WIP: idhub-docker #5

Draft
pedro wants to merge 3 commits from idhub-docker into release
pull from: idhub-docker
merge into: ereuse:release
Conversation 2 Commits 3 Files changed 8 +279 -2
pedro commented 2025-01-08 21:40:00 +00:00
Owner
Copy link

added docker files so localhost could be deployed

@cayop, what do you think on:

  1. bugfix on requirement error
  2. new env var ENABLE_DOMAIN_CHECKER, so on localhost deployment we can use http://localhost:9001 without assert error ; @cayop what do you think?
added docker files so localhost could be deployed @cayop, what do you think on: 1. bugfix on requirement error 2. new env var `ENABLE_DOMAIN_CHECKER`, so on localhost deployment we can use http://localhost:9001 without assert error ; @cayop what do you think?
pedro added 3 commits 2025-01-08 21:40:00 +00:00 
when using localhost and standalone idhub instance we could avoid this
checker to facilitate testing and deployment
error was

```
idhub-1  | ValueError: numpy.dtype size changed, may indicate binary incompatibility. Expected 96 from C header, got 88 from PyObject
idhub-1 exited with code 1
```
pedro commented 2025-01-08 21:51:18 +00:00
Author
Owner
Copy link

ups, no está con la branch cred_snapshots falta hacer rebase de esa parte; la hago en otro momento

ENABLE_DOMAIN_CHECKER quizá no tiene sentido y probablemente descartaré del todo ese commit porque esta solución implica que en DEVELOPMENT no hay este checker y está arreglado. Aunque nuevamente puede ser discutible, porque en DEVELOPMENT nos puede interesar comprobar si hay tricks con dominios o no 🤷, lo comentamos

baa4d87a11

ups, no está con la branch `cred_snapshots` falta hacer rebase de esa parte; la hago en otro momento `ENABLE_DOMAIN_CHECKER` quizá no tiene sentido y probablemente descartaré del todo ese commit porque esta solución implica que en DEVELOPMENT no hay este checker y está arreglado. Aunque nuevamente puede ser discutible, porque en DEVELOPMENT nos puede interesar comprobar si hay tricks con dominios o no 🤷, lo comentamos https://farga.pangea.org/ereuse/IdHub/commit/baa4d87a1194ff1462d00f9d51f47359eb097e9f
pedro force-pushed idhub-docker from abfdf59205 to 913180b8fd 2025-01-09 08:15:09 +00:00 Compare
pedro commented 2025-01-09 08:16:03 +00:00
Author
Owner
Copy link

con @cayop acordamos que los cambios son ok, y que ENABLE_DOMAIN_CHECKER aporta más granularidad a las pruebas

con @cayop acordamos que los cambios son ok, y que `ENABLE_DOMAIN_CHECKER` aporta más granularidad a las pruebas
pedro force-pushed idhub-docker from 913180b8fd to 6f9a06faa1 2025-01-09 08:24:31 +00:00 Compare
This pull request is marked as a work in progress.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin idhub-docker:idhub-docker
git checkout idhub-docker

Merge

Merge the changes and update on Forgejo.
git checkout release
git merge --no-ff idhub-docker
git checkout release
git merge --ff-only idhub-docker
git checkout idhub-docker
git rebase release
git checkout release
git merge --no-ff idhub-docker
git checkout release
git merge --squash idhub-docker
git checkout release
git merge --ff-only idhub-docker
git checkout release
git merge idhub-docker
git push origin release
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: ereuse/IdHub#5
No description provided.
Delete branch "idhub-docker"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?