Skip to content

Commit

Permalink
Drop references to system_setup from the subiquity tree
Browse files Browse the repository at this point in the history
Signed-off-by: Olivier Gayot <[email protected]>
  • Loading branch information
ogayot committed May 30, 2024
1 parent 1a4bdaa commit adbec9d
Show file tree
Hide file tree
Showing 68 changed files with 49 additions and 3,886 deletions.
10 changes: 4 additions & 6 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,17 +43,15 @@ jobs:
- uses: psf/black@stable
with:
version: "~= 23.0"
src: "console_conf subiquity subiquitycore system_setup"

src: "console_conf subiquity subiquitycore"
format-isort:
runs-on: ubuntu-20.04
steps:
- uses: actions/checkout@v4
- uses: isort/isort-action@v1
with:
isort-version: "5.12.0"
sort-paths: "console_conf subiquity subiquitycore system_setup"

sort-paths: "console_conf subiquity subiquitycore"
static-typing:
# In this job, we compare the output of mypy before and after the PR.
if: github.event_name == 'pull_request'
Expand All @@ -74,7 +72,7 @@ jobs:
# If it does not work, we can set fetch-depth: 0
fetch-depth: $(( ${{ github.event.pull_request.commits }} + 1 ))
- name: Run mypy on pull request branch
run: python3 -m mypy --ignore-missing-imports --check-untyped-defs subiquity subiquitycore system_setup console_conf scripts/replay-curtin-log.py | tee /tmp/mypy-head.out
run: python3 -m mypy --ignore-missing-imports --check-untyped-defs subiquity subiquitycore console_conf scripts/replay-curtin-log.py | tee /tmp/mypy-head.out
- name: Determine base commit (most recent common ancestor)
id: determine_base_commit
run: |
Expand All @@ -88,7 +86,7 @@ jobs:
ref: ${{ steps.determine_base_commit.outputs.base_commit }}
- run: git show
- name: Run mypy on base commit
run: python3 -m mypy --ignore-missing-imports --check-untyped-defs subiquity subiquitycore system_setup console_conf scripts/replay-curtin-log.py | tee /tmp/mypy-base.out
run: python3 -m mypy --ignore-missing-imports --check-untyped-defs subiquity subiquitycore console_conf scripts/replay-curtin-log.py | tee /tmp/mypy-base.out
- name: Produce the diff between the two mypy runs
run: diff --color=always --unified=0 /tmp/mypy-base.out /tmp/mypy-head.out
continue-on-error: true
6 changes: 3 additions & 3 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
@@ -1,16 +1,16 @@
files: "(console_conf|subiquity|subiquitycore|system_setup|doc)"
files: "(console_conf|subiquity|subiquitycore|doc)"
repos:
- repo: https://github.com/psf/black
rev: 23.7.0
hooks:
- id: black
files: "(console_conf|subiquity|subiquitycore|system_setup)"
files: "(console_conf|subiquity|subiquitycore)"
- repo: https://github.com/pycqa/isort
rev: 5.12.0
hooks:
- id: isort
name: isort
files: "(console_conf|subiquity|subiquitycore|system_setup)"
files: "(console_conf|subiquity|subiquitycore)"
- repo: local
hooks:
- id: doc-spelling
Expand Down
20 changes: 1 addition & 19 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -9,11 +9,10 @@ PROBERT_REPO=https://github.com/canonical/probert
DRYRUN?=--dry-run --bootloader uefi --machine-config examples/machines/simple.json \
--source-catalog examples/sources/install.yaml \
--postinst-hooks-dir examples/postinst.d/
SYSTEM_SETUP_DRYRUN?=--dry-run
export PYTHONPATH
CWD := $(shell pwd)

CHECK_DIRS := console_conf subiquity subiquitycore system_setup
CHECK_DIRS := console_conf subiquity subiquitycore
PYTHON := python3

ifneq (,$(MACHINE))
Expand Down Expand Up @@ -52,22 +51,6 @@ dryrun-serial ui-view-serial:
dryrun-server:
$(PYTHON) -m subiquity.cmd.server $(DRYRUN)

.PHONY: dryrun-system-setup
dryrun-system-setup:
$(PYTHON) -m system_setup.cmd.tui $(SYSTEM_SETUP_DRYRUN)

.PHONY: dryrun-system-setup-server
dryrun-system-setup-server:
$(PYTHON) -m system_setup.cmd.server $(SYSTEM_SETUP_DRYRUN)

.PHONY: dryrun-system-setup-recon
dryrun-system-setup-recon:
DRYRUN_RECONFIG=true $(PYTHON) -m system_setup.cmd.tui $(SYSTEM_SETUP_DRYRUN)

.PHONY: dryrun-system-setup-server-recon
dryrun-system-setup-server-recon:
DRYRUN_RECONFIG=true $(PYTHON) -m system_setup.cmd.server $(SYSTEM_SETUP_DRYRUN)

.PHONY: lint
lint: flake8

Expand Down Expand Up @@ -106,7 +89,6 @@ gitdeps: curtin probert
.PHONY: schema
schema: gitdeps
@$(PYTHON) -m subiquity.cmd.schema > autoinstall-schema.json
@$(PYTHON) -m system_setup.cmd.schema > autoinstall-system-setup-schema.json

.PHONY: format black isort
format:
Expand Down
148 changes: 0 additions & 148 deletions autoinstall-system-setup-schema.json

This file was deleted.

11 changes: 0 additions & 11 deletions examples/answers/system-setup-init.yaml

This file was deleted.

12 changes: 0 additions & 12 deletions examples/answers/system-setup-reconf.yaml

This file was deleted.

25 changes: 0 additions & 25 deletions examples/autoinstall/system-setup-full.yaml

This file was deleted.

16 changes: 0 additions & 16 deletions examples/autoinstall/system-setup-no-shutdown.yaml

This file was deleted.

17 changes: 0 additions & 17 deletions examples/autoinstall/system-setup.yaml

This file was deleted.

6 changes: 0 additions & 6 deletions examples/prefill-system-setup-complete.yaml

This file was deleted.

2 changes: 0 additions & 2 deletions examples/prefill-system-setup-missing-identity.yaml

This file was deleted.

4 changes: 0 additions & 4 deletions examples/prefill-system-setup-missing-locale.yaml

This file was deleted.

4 changes: 0 additions & 4 deletions examples/prefill-system-setup-missing-realname.yaml

This file was deleted.

4 changes: 0 additions & 4 deletions examples/prefill-system-setup-missing-username.yaml

This file was deleted.

3 changes: 0 additions & 3 deletions examples/prefill-system-setup-missing-welcome.yaml

This file was deleted.

2 changes: 1 addition & 1 deletion scripts/run-mypy.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ def run_mypy(cwd: str | None, stdout: TextIO | None) -> None:
mypy_cmd = [
'python3', '-m', 'mypy',
'--ignore-missing-imports', '--check-untyped-defs',
'subiquity', 'subiquitycore', 'system_setup', 'console_conf',
'subiquity', 'subiquitycore', 'console_conf',
'scripts/replay-curtin-log.py',
]

Expand Down
Loading

0 comments on commit adbec9d

Please sign in to comment.