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

fix(discovery): observed containers should be checked with persisted nodes #423

Merged
merged 25 commits into from
May 3, 2024
Merged

fix(discovery): observed containers should be checked with persisted nodes #423

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
9ce106c
fix(discovery): observed containers should be checked with persisted …
tthvo Apr 26, 2024
3c5398e
fixup(discovery): ensure transactional context
tthvo Apr 26, 2024
6549d42
fixup(discovery): run transaction in worker thread
tthvo Apr 26, 2024
ea3731f
chore(spotless): refactor for cleaner look
tthvo Apr 26, 2024
9c9bfc3
fix(discovery): ensure realm is persisted first
tthvo Apr 26, 2024
6fcfa42
chore(discovery): remove unused fields
tthvo Apr 26, 2024
909aab5
fixup(discovery): check null
tthvo Apr 26, 2024
c01f5f4
fixup(discovery): use persisted target if any
tthvo Apr 26, 2024
79f374d
add synchronized
tthvo Apr 26, 2024
8bec1c9
chore(discovery): use event bus
tthvo Apr 30, 2024
e439b0f
fix(discovery): only handle message if enabled
tthvo May 1, 2024
5ac2e11
fixup(discovery): no need for explicit address
tthvo May 1, 2024
71292ac
fix(discovery): message should be sent now - consumeEvent must be used
tthvo May 1, 2024
87e8d4a
fixup(discovery): separate addresses for podman and docker discovery
tthvo May 1, 2024
701c7a1
chore(discovery): remove unused fieldsc
tthvo May 1, 2024
f44c9d8
fixup(discovery): targets should now be correctly persisted
tthvo May 1, 2024
12c5db5
fixup(discovery): targets should be correctly persisted
tthvo May 1, 2024
b26f9f9
chore(discovery): define common method in parent class
tthvo May 2, 2024
d1086b4
fix(discovery): should return persisted target if any
tthvo May 2, 2024
50d6ae4
fix(discovery): handle constraint violation for duplicate keys
tthvo May 2, 2024
2b5f990
chore(discovery): simplify methods
tthvo May 2, 2024
e6bdc75
fix(compose): do not clean up database if volumes are kept
tthvo May 2, 2024
8ce2731
fix(discovery): require new transactions for discovery handler
tthvo May 2, 2024
d8009ad
fix(discovery): ensure target has the latest snapshot
tthvo May 3, 2024
c52b759
chore(discovery): combine startup event listeners
tthvo May 3, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion compose/db.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ version: "3"
services:
cryostat:
environment:
QUARKUS_HIBERNATE_ORM_DATABASE_GENERATION: drop-and-create
QUARKUS_HIBERNATE_ORM_DATABASE_GENERATION: ${DATABASE_GENERATION:-drop-and-create}
QUARKUS_DATASOURCE_USERNAME: cryostat3
QUARKUS_DATASOURCE_PASSWORD: cryostat3
QUARKUS_DATASOURCE_JDBC_URL: jdbc:postgresql://db:5432/cryostat3
Expand Down
2 changes: 2 additions & 0 deletions smoketest.bash
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,7 @@ while getopts "hs:prGtOVXcbn" opt; do
;;
V)
KEEP_VOLUMES=true
DATABASE_GENERATION=update
;;
X)
FILES+=("${DIR}/compose/db-viewer.yml")
Expand Down Expand Up @@ -111,6 +112,7 @@ fi
export CRYOSTAT_HTTP_HOST
export CRYOSTAT_HTTP_PORT
export GRAFANA_DASHBOARD_EXT_URL
export DATABASE_GENERATION

s3Manifest="${DIR}/compose/s3-${s3}.yml"
STORAGE_PORT="$(yq '.services.*.expose[0]' "${s3Manifest}" | grep -v null)"
Expand Down
Loading
Loading