Skip to content

Add CI workflow with smoke tests for installer scripts in Docker containers#15

Closed
Copilot wants to merge 11 commits intomainfrom
copilot/add-ci-setup
Closed

Add CI workflow with smoke tests for installer scripts in Docker containers#15
Copilot wants to merge 11 commits intomainfrom
copilot/add-ci-setup

Conversation

Copy link
Copy Markdown

Copilot AI commented Oct 21, 2025

  • Analyze repository structure and installer scripts
  • Create CI workflow file with:
    • Smoke tests that run installer scripts inside Ubuntu 24.04 Docker containers
    • Non-root user (testuser) inside containers
    • Docker-in-Docker support for software installer/upgrade scripts
    • --privileged flag to fix Docker permission issues
    • Clean Ubuntu 24.04 environment avoiding GitHub Runner customizations
    • Updated actions/checkout to v5
    • Runs on pull_request and push events to main branch only
    • Explicit permissions (contents: read) for security
  • Run CodeQL security check (no vulnerabilities)
  • Commit and push updated CI workflow
Original prompt

Add CI

Created from VS Code via the GitHub Pull Request extension.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 3 commits October 21, 2025 01:42
Co-authored-by: MtkN1 <51289448+MtkN1@users.noreply.github.com>
Co-authored-by: MtkN1 <51289448+MtkN1@users.noreply.github.com>
Co-authored-by: MtkN1 <51289448+MtkN1@users.noreply.github.com>
Copilot AI changed the title [WIP] Add continuous integration setup for project Add CI workflow with ShellCheck, Python validation, and permissions checks Oct 21, 2025
Copilot AI requested a review from MtkN1 October 21, 2025 01:48
Copy link
Copy Markdown
Owner

@MtkN1 MtkN1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot

  • Rework the job. Use smoke tests, not lint.
  • Target branch name “master” is unnecessary.
  • The .gitignore file is unnecessary.

Co-authored-by: MtkN1 <51289448+MtkN1@users.noreply.github.com>
Copy link
Copy Markdown
Author

Copilot AI commented Oct 21, 2025

@copilot

  • Rework the job. Use smoke tests, not lint.
  • Target branch name “master” is unnecessary....

Done. The workflow now uses smoke tests (bash -n for shell scripts, py_compile for Python), targets only main branch, and .gitignore has been removed. Changes in commit: reworked CI workflow with syntax validation smoke tests.

Copilot AI changed the title Add CI workflow with ShellCheck, Python validation, and permissions checks Add CI workflow with smoke tests for script syntax validation Oct 21, 2025
Copilot AI requested a review from MtkN1 October 21, 2025 02:15
Copy link
Copy Markdown
Owner

@MtkN1 MtkN1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot

  • The smoke test job simply needs to run the Software installer (install/upgrade) and dotfiles installer. (See README.md)
  • Update the version in actions/checkout to v5.

Co-authored-by: MtkN1 <51289448+MtkN1@users.noreply.github.com>
Copy link
Copy Markdown
Author

Copilot AI commented Oct 21, 2025

@copilot

  • The smoke test job simply needs to run the Software installer (install/upgrade) and dotfiles installer. (See README.md)
  • Update the version in actions/checkout to v5.

Done. The smoke test now runs the software installer (install.sh, upgrade.sh) and dotfiles installer as specified in README.md. Also updated actions/checkout to v5. Changes in commit 0e2b05e.

Copilot AI changed the title Add CI workflow with smoke tests for script syntax validation Add CI workflow with smoke tests for installer scripts Oct 21, 2025
Copilot AI requested a review from MtkN1 October 21, 2025 02:27
Copy link
Copy Markdown
Owner

@MtkN1 MtkN1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot
Run Software installer/upgrade and Dotfiles installer inside a non-root Ubuntu 24.04 Docker container with Docker available inside.
GitHub Runner’s customized Ubuntu 24.04 environment causes these scripts to fail.

Co-authored-by: MtkN1 <51289448+MtkN1@users.noreply.github.com>
Copy link
Copy Markdown
Author

Copilot AI commented Oct 21, 2025

@copilot
Run Software installer/upgrade and Dotfiles installer inside a non-root Ubuntu 24.04 Docker container with Docker available inside.
GitHub Runner’s customized Ubuntu 24.04 environme...

Done. The workflow now runs the software installer/upgrade and dotfiles installer inside a non-root Ubuntu 24.04 Docker container. Docker-in-Docker is available for the installer scripts via mounted Docker socket. Changes in commit f3a9e12.

Copilot AI changed the title Add CI workflow with smoke tests for installer scripts Add CI workflow with smoke tests for installer scripts in Docker containers Oct 21, 2025
Copilot AI requested a review from MtkN1 October 21, 2025 16:18
Copy link
Copy Markdown
Owner

@MtkN1 MtkN1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot The CI is failing due to Docker permission issues. Fix this. --privileged option may be required.

Co-authored-by: MtkN1 <51289448+MtkN1@users.noreply.github.com>
Copy link
Copy Markdown
Author

Copilot AI commented Oct 22, 2025

@copilot The CI is failing due to Docker permission issues. Fix this. --privileged option may be required.

Fixed. Added --privileged flag to the Docker containers for software installer and upgrade steps to resolve permission issues. Changes in commit 9c3f7b8.

Copilot AI requested a review from MtkN1 October 22, 2025 09:03
@MtkN1 MtkN1 closed this Nov 26, 2025
@MtkN1 MtkN1 deleted the copilot/add-ci-setup branch November 27, 2025 05:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants