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

Clean lxd code #423

Open
wants to merge 75 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
75 commits
Select commit Hold shift + click to select a range
a97b699
increate image builder revision
javierdelapuente Dec 3, 2024
0615076
First try for integ tests in openstack
javierdelapuente Dec 3, 2024
414827d
no need for lxd profile. log for proxy
javierdelapuente Dec 4, 2024
c514428
Merge branch 'main' into integration-tests-lxd-to-openstack
javierdelapuente Dec 5, 2024
053b57d
Fix integ tests
javierdelapuente Dec 5, 2024
1a3c2c8
put the same name to the integ tests
javierdelapuente Dec 5, 2024
32de0e4
Disable snap autorefresh to see if it does not fail
javierdelapuente Dec 5, 2024
6fd2d9d
use autorefresh for all
javierdelapuente Dec 5, 2024
3e4ffb2
empty commit
javierdelapuente Dec 5, 2024
7d692b8
add sleep for tests
javierdelapuente Dec 5, 2024
d2883dd
remove sleep
javierdelapuente Dec 5, 2024
894888f
use reverse tunnel
javierdelapuente Dec 6, 2024
62f335c
Merge branch 'main' into integration-tests-lxd-to-openstack
javierdelapuente Dec 16, 2024
3956940
Do run all the tests without waiting
javierdelapuente Dec 16, 2024
1918cb4
run against op workflows main
javierdelapuente Dec 16, 2024
414bc89
remove linting exception
javierdelapuente Dec 16, 2024
c1bc3c1
Merge branch 'main' into integration-tests-lxd-to-openstack
javierdelapuente Dec 17, 2024
4d2beda
Update tests/integration/test_debug_ssh.py
javierdelapuente Dec 17, 2024
7ea902f
rename fixture for tmate
javierdelapuente Dec 17, 2024
b5c5339
fix fixture name
javierdelapuente Dec 17, 2024
9a1d25e
improve comment
javierdelapuente Dec 17, 2024
5f04542
Starting to remove code, let's see if it passes tests
javierdelapuente Dec 5, 2024
29fc1a4
some more cleaning
javierdelapuente Dec 4, 2024
19da916
Add back test_charm_fork_repo
javierdelapuente Dec 5, 2024
975e6bf
remove loop devices in integration testing
javierdelapuente Dec 16, 2024
50b1581
forgotten code back
javierdelapuente Dec 16, 2024
bfcafae
Remove non-existant fixture
javierdelapuente Dec 16, 2024
277072d
fix linting
javierdelapuente Dec 16, 2024
b96064f
big removal, fingers crossed
javierdelapuente Dec 18, 2024
ab9c4ad
Nothing insteresint in scripts now
javierdelapuente Dec 17, 2024
e3662cc
remove reference to lxd
javierdelapuente Dec 17, 2024
2d7be0b
More code cleaning
javierdelapuente Dec 18, 2024
237435e
one more try. doc and denylist...
javierdelapuente Dec 18, 2024
7ca5b19
clean lxd code 3 after first rebase
javierdelapuente Dec 18, 2024
db91cd1
Merge branch 'main' into clean-lxd-code3
javierdelapuente Dec 18, 2024
4352850
remove old comment
javierdelapuente Dec 18, 2024
f34d63a
remove tutorial directory link
javierdelapuente Dec 18, 2024
50fd711
Put message in readme file
javierdelapuente Dec 18, 2024
899bee5
remove link to the tutorial and lxd references in the readme
javierdelapuente Dec 18, 2024
f14998d
many more changes and removing of lxd references
javierdelapuente Dec 18, 2024
ad0c348
a few more things to delete
javierdelapuente Dec 18, 2024
1632def
update md files
javierdelapuente Dec 18, 2024
d6433e6
PR review suggestions
javierdelapuente Dec 18, 2024
a7ec455
suggestions from pr review
javierdelapuente Dec 18, 2024
ebe6169
ditch InstanceType
javierdelapuente Dec 18, 2024
1d5797d
Update config.yaml
javierdelapuente Dec 18, 2024
975f70b
remove unneded templates
javierdelapuente Dec 18, 2024
28e0aa5
Apply suggestions from code review
javierdelapuente Dec 19, 2024
8eba9eb
Add local-lxd docs directories
javierdelapuente Dec 19, 2024
27e396b
let's see if this works
javierdelapuente Dec 19, 2024
ea32be7
Apply suggestions from code review
javierdelapuente Dec 19, 2024
67c7ad5
suggestions from code review
javierdelapuente Dec 19, 2024
d730e67
remove storage (the rebase readded it...)
javierdelapuente Dec 19, 2024
14dbff1
use main branch from action-tmate
javierdelapuente Dec 19, 2024
ff027ac
put back storage for now as no upgrade possible
javierdelapuente Dec 19, 2024
033b6a1
no need for generic InstanceHelper
javierdelapuente Dec 19, 2024
1ec4f62
Add some logging to integration tests
javierdelapuente Dec 19, 2024
96cd50e
pr review suggestions
javierdelapuente Dec 19, 2024
776aad8
Apply suggestions from code review
javierdelapuente Dec 19, 2024
928c9d8
make linter happy
javierdelapuente Dec 19, 2024
e6b688b
recover deleted tests thatn could run in openstack
javierdelapuente Dec 19, 2024
dc4da20
update test
javierdelapuente Dec 19, 2024
91e8934
fix test
javierdelapuente Dec 19, 2024
157c88e
make the no runner app an openstack one
javierdelapuente Dec 19, 2024
e68e5a8
fix test
javierdelapuente Dec 19, 2024
98b42ba
clean the test
javierdelapuente Dec 19, 2024
fe92a6a
clean the test
javierdelapuente Dec 19, 2024
1068c18
fix names
javierdelapuente Dec 19, 2024
8a4bc4b
image builder build interval to 12
javierdelapuente Dec 20, 2024
176015a
fix linting
javierdelapuente Dec 20, 2024
5cfcaa8
fail faster
javierdelapuente Dec 20, 2024
a5d4a81
suggested changes
javierdelapuente Dec 20, 2024
8da8b33
Merge branch 'main' into clean-lxd-code3
javierdelapuente Dec 20, 2024
b22a3f7
logs
javierdelapuente Dec 21, 2024
02be48c
fix lint
javierdelapuente Dec 21, 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
1 change: 0 additions & 1 deletion .github/pull_request_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,5 @@ Applicable spec: <link>
- [ ] The documentation for charmhub is updated.
- [ ] The PR is tagged with appropriate label (`urgent`, `trivial`, `complex`).
- [ ] The changelog is updated with changes that affects the users of the charm.
- [ ] The changes do not introduce any regression in code or tests related to LXD runner mode.

<!-- Explanation for any unchecked items above -->
25 changes: 0 additions & 25 deletions .github/workflows/e2e_test_run.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,37 +12,20 @@ on:
description: The e2e test runner tag to run the workflow on.
type: string
required: true
runner-virt-type:
description: The e2e test runner virtualization type. E.g. lxd, or openstack.
# workflow_call does not support choice type.
type: string
required: true
workflow_dispatch:
inputs:
runner-tag:
description: The e2e test runner tag to run the workflow on.
type: string
required: true
runner-virt-type:
description: The e2e test runner virtualization type.
type: choice
required: true
options:
- lxd
- openstack

jobs:
e2e-test:
name: End-to-End Test Run
runs-on: [self-hosted, linux, "${{ inputs.runner-tag }}"]
steps:
- name: Hostname is set to "github-runner"
if: "${{ github.event.inputs.runner-virt-type == 'openstack' }}"
run: sudo hostnamectl hostname | grep github-runner
# Snapd can have some issues in privileged LXD containers without setting
# security.nesting=True and this.
- name: Fix snap issue in privileged LXD containers
run: ln -s /bin/true /usr/local/bin/udevadm
# Below is a series of simple tests to assess the functionality of the newly spawned runner.
- name: Echo hello world
run: echo "hello world"
Expand Down Expand Up @@ -104,11 +87,3 @@ jobs:
# ~/.local/bin is added to path runner env through in scripts/env.j2
- name: test check-jsonschema
run: check-jsonschema --version
- name: Test Firewall
javierdelapuente marked this conversation as resolved.
Show resolved Hide resolved
if: "${{ github.event.inputs.runner-virt-type == 'lxd' }}"
run: |
HOST_IP=$(ip route | grep default | cut -f 3 -d" ")
[ $((ping $HOST_IP -c 5 || :) | grep "Destination Port Unreachable" | wc -l) -eq 5 ]
- name: Test sctp support
if: "${{ github.event.inputs.runner-virt-type == 'lxd' }}"
run: sudo apt-get install lksctp-tools -yq && checksctp
6 changes: 3 additions & 3 deletions .github/workflows/integration_test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ jobs:
provider: lxd
test-tox-env: integration-juju3.1
modules: '["test_charm_scheduled_events", "test_debug_ssh", "test_charm_upgrade"]'
extra-arguments: "-m openstack"
extra-arguments: '-m openstack --log-format="%(asctime)s %(levelname)s %(message)s"'
self-hosted-runner: true
self-hosted-runner-label: stg-private-endpoint
openstack-interface-tests-private-endpoint:
Expand All @@ -47,8 +47,8 @@ jobs:
pre-run-script: scripts/setup-lxd.sh
provider: lxd
test-tox-env: integration-juju3.6
modules: '["test_charm_metrics_failure", "test_charm_metrics_success", "test_charm_fork_repo", "test_charm_runner", "test_reactive"]'
extra-arguments: "-m openstack"
modules: '["test_charm_metrics_failure", "test_charm_metrics_success", "test_charm_fork_repo", "test_charm_fork_path_change", "test_charm_no_runner", "test_charm_runner", "test_reactive"]'
extra-arguments: '-m openstack --log-format="%(asctime)s %(levelname)s %(message)s"'
self-hosted-runner: true
self-hosted-runner-label: stg-private-endpoint
allure-report:
Expand Down
Loading
Loading