-
-
Notifications
You must be signed in to change notification settings - Fork 40
feat: containerd/nerdctl engine backend #445
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
base: main
Are you sure you want to change the base?
Changes from all commits
f1820f7
9d73794
18ecd05
c0bbb00
cad27e7
40000b1
c623d21
1952476
6c05b2d
270c369
9eff2a6
c8c5a14
36923bd
980f6bd
b064ca0
05d105d
da24125
9dca7db
5c38568
6ed6c1a
abd2ef0
440b97e
e6d3afe
7729c16
63516bb
1b395e6
09b9a7e
854cb9f
1be7349
22f5771
4e2e99e
5784ca6
678de49
9f53c2d
a0e6931
73a5b28
f87eb45
82cfe1b
b41d2b6
bc4b1d4
a73dfd5
525fb9a
bcca717
513f08c
abf6ce8
31c853c
5f2f969
7de2358
c1dc204
c376f7c
07014b3
5df6efa
c41b5c0
e43c728
7a68b80
269033f
c08f532
0a2d87c
afb01b9
2444814
d8ccba7
e4fffb7
d78dd06
da922de
e1639d3
47ef985
4a56845
f774732
a1ce102
232ad8f
1eaad1b
af2f1e8
34db526
af1c4e3
4d63559
d4112de
fa7043c
dab5b91
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,62 @@ | ||
| name: Containerd Engine Tests | ||
|
|
||
| on: | ||
| pull_request: | ||
|
|
||
| jobs: | ||
| leia-tests: | ||
| runs-on: ${{ matrix.os }} | ||
| env: | ||
| TERM: xterm | ||
| strategy: | ||
| fail-fast: false | ||
| matrix: | ||
| leia-test: | ||
| - containerd | ||
| node-version: | ||
| - "20" | ||
| os: | ||
| - ubuntu-24.04 | ||
|
|
||
| steps: | ||
| - name: Checkout code | ||
| uses: actions/checkout@v4 | ||
| - name: Install node ${{ matrix.node-version }} | ||
| uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: ${{ matrix.node-version }} | ||
| registry-url: https://registry.npmjs.org | ||
| cache: npm | ||
| - name: Bundle Deps | ||
| uses: lando/prepare-release-action@v3 | ||
| with: | ||
| lando-plugin: true | ||
| version: dev | ||
| sync: false | ||
| - name: Install pkg dependencies | ||
| run: npm clean-install --prefer-offline --frozen-lockfile --production | ||
| - name: Package into node binary | ||
| uses: lando/pkg-action@v6 | ||
| id: pkg-action | ||
| with: | ||
| entrypoint: bin/lando | ||
| filename: lando | ||
| node-version: ${{ matrix.node-version }} | ||
| options: --options dns-result-order=ipv4first | ||
| upload: false | ||
| pkg: "@yao-pkg/pkg@5.16.1" | ||
| - name: Install full deps | ||
| run: npm clean-install --prefer-offline --frozen-lockfile | ||
| - name: Setup lando ${{ steps.pkg-action.outputs.file }} | ||
| uses: lando/setup-lando@v3 | ||
| with: | ||
| auto-setup: false | ||
| lando-version: ${{ steps.pkg-action.outputs.file }} | ||
| telemetry: false | ||
| - name: Run Leia Tests | ||
| uses: lando/run-leia-action@v2 | ||
| with: | ||
| leia-test: "./examples/${{ matrix.leia-test }}/README.md" | ||
| cleanup-header: "Destroy tests" | ||
| shell: bash | ||
| stdin: true |
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,7 +6,7 @@ const _ = require('lodash'); | |
| /* | ||
| * Helper to get core proxy service | ||
| */ | ||
| const getProxy = ({proxyCommand, proxyPassThru, proxyDomain, userConfRoot, version = 'unknown'} = {}) => { | ||
| const getProxy = ({proxyCommand, proxyPassThru, proxyDomain, userConfRoot, dockerSocket, version = 'unknown'} = {}) => { | ||
| return { | ||
| services: { | ||
| proxy: { | ||
|
|
@@ -21,7 +21,7 @@ const getProxy = ({proxyCommand, proxyPassThru, proxyDomain, userConfRoot, versi | |
| }, | ||
| networks: ['edge'], | ||
| volumes: [ | ||
| '/var/run/docker.sock:/var/run/docker.sock', | ||
| `${dockerSocket || '/var/run/docker.sock'}:/var/run/docker.sock`, | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Proxy builder never receives
|
||
| `${userConfRoot}/scripts/proxy-certs.sh:/scripts/100-proxy-certs`, | ||
| 'proxy_config:/proxy_config', | ||
| ], | ||
|
|
||


There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Config YAML indentation breaks
enginesetting in CIHigh Severity
The
configparameter forlando/setup-landouses a YAML multi-line block (|), butengine=${{ matrix.engine }}on line 120 lacks the leading whitespace indentation to be on its own line within the block scalar. It appears to continue aftersetup.skipCommonPlugins=truewithout a newline separator, resulting in a malformed config value likesetup.skipCommonPlugins=trueengine=dockerinstead of two separate config entries.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bugbot Autofix determined this is a false positive.
Both config lines have identical 12-space indentation and parse correctly as a valid YAML block scalar.