Skip to content

Commit 0d24dba

Browse files
authored
fix upgrade and delete manifests->elements (#391)
1 parent 76c4ba0 commit 0d24dba

3 files changed

Lines changed: 21 additions & 13 deletions

File tree

.github/workflows/build.yml

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ env:
1313
PROJECT_NAME: exordos_core
1414
REPO_ENDPOINT_PUT: http://10.20.0.1:8082
1515
PACKER_VERSION: "1.9.2"
16-
CORE_ENDPOINT_URL: http://10.20.0.2:11010
16+
CORE_ENDPOINT_URL: http://10.20.0.2:80/api/core
1717
CORE_USERNAME: "admin"
1818
CORE_PASSWORD: "admin"
1919

@@ -34,7 +34,6 @@ jobs:
3434
set -eux
3535
mkdir -p ${ARTIFACTS_PATH}
3636
37-
source ~/.profile
3837
VERSION="$(exordos get-version .)"
3938
4039
# Build Exordos Core
@@ -64,13 +63,15 @@ jobs:
6463
with:
6564
version: ${{ env.PACKER_VERSION }}
6665
- name: Install exordos
67-
run: curl -fsSL https://repo.exordos.com/install.sh | sudo sh
66+
run: curl -fsSL https://repo.exordos.com/install.sh | sh
6867
- name: set exordos config
6968
run: |
7069
exordos settings set-realm default --endpoint $CORE_ENDPOINT_URL
7170
exordos settings set-context default --name default --user $CORE_USERNAME --password $CORE_PASSWORD
7271
- name: Install hypervisor
73-
run: sudo exordos compute hypervisors init
72+
run: |
73+
EXORDOS_BIN=$(which exordos)
74+
sudo $EXORDOS_BIN compute hypervisors init
7475
- name: Download build artifacts
7576
uses: actions/download-artifact@v8
7677
with:
@@ -83,14 +84,14 @@ jobs:
8384
- name: check exordos core
8485
timeout-minutes: 10
8586
run: |
86-
# set -eux
87+
set -eux
8788
88-
for i in {1..10}; do
89+
for i in {1..20}; do
8990
if exordos ready_api; then
9091
echo "Exordos Core API is ready"
9192
break
9293
fi
93-
echo "Waiting for Exordos Core API... ($i/10)"
94+
echo "Waiting for Exordos Core API... ($i/20)"
9495
sleep 5
9596
done
9697

Makefile

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,5 +15,8 @@ build_core:
1515
bootstrap:
1616
exordos bootstrap -i output -f -m core --admin-password admin --cidr 10.20.0.0/22
1717

18+
delete_core:
19+
exordos realms d local
20+
1821
mdlint:
1922
markdownlint-cli2 --config .markdownlint.yaml "**/*.md" "#node_modules" "#!.venv" "#!.tox" --fix

exordos_core/elements/dm/models.py

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -205,6 +205,7 @@ def upgrade(self) -> "Manifest":
205205
element.version = self.version
206206
element.api_version = self.api_version
207207
element.description = self.description
208+
element.manifest = self
208209
element.save()
209210
return self.apply_element(element)
210211

@@ -397,13 +398,16 @@ def _check_no_dependents(element: "Element") -> None:
397398

398399
def uninstall(self) -> "Manifest":
399400
element_engine.load_from_database()
400-
401-
elements = Element.objects.get_all(
402-
filters={
403-
"name": ra_filters.EQ(self.name),
404-
"version": ra_filters.EQ(self.version),
405-
}
401+
filters = ra_filters.AND(
402+
ra_filters.OR(
403+
{
404+
"name": ra_filters.EQ(self.name),
405+
"version": ra_filters.EQ(self.version),
406+
},
407+
{"manifest": ra_filters.EQ(self.uuid)},
408+
)
406409
)
410+
elements = Element.objects.get_all(filters=filters)
407411
for element in elements:
408412
self._check_no_dependents(element)
409413
for element in elements:

0 commit comments

Comments
 (0)