Skip to content

Commit 757f171

Browse files
jingkaiheRoyceDavison
authored andcommitted
Remove obsolete cpu_count runtime config from documentations
Remove obsolete `cpu_count` runtime config from documentations, since the option has already been removed from the runtime in firecracker-microvm#372. Signed-off-by: Jingkai He <[email protected]>
1 parent 54e0edc commit 757f171

File tree

5 files changed

+15
-24
lines changed

5 files changed

+15
-24
lines changed

config/config_test.go

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -39,17 +39,15 @@ func TestLoadConfigOverrides(t *testing.T) {
3939
overrideKernelArgs := "OVERRIDE KERNEL ARGS"
4040
overrideKernelPath := "OVERRIDE KERNEL PATH"
4141
overrideRootfsPath := "OVERRIDE ROOTFS PATH"
42-
overrideCPUCount := 42
4342
overrideCPUTemplate := "OVERRIDE CPU TEMPLATE"
4443
configContent := fmt.Sprintf(
4544
`{
4645
"kernel_args":"%s",
4746
"kernel_image_path":"%s",
4847
"root_drive":"%s",
49-
"cpu_count": %d,
50-
"cpu_template": "%s",
48+
"cpu_template": "%s",
5149
"log_levels": ["debug"]
52-
}`, overrideKernelArgs, overrideKernelPath, overrideRootfsPath, overrideCPUCount, overrideCPUTemplate)
50+
}`, overrideKernelArgs, overrideKernelPath, overrideRootfsPath, overrideCPUTemplate)
5351
configFile, cleanup := createTempConfig(t, configContent)
5452
defer cleanup()
5553
cfg, err := LoadConfig(configFile)

docs/getting-started.md

Lines changed: 11 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ Thanks for your interest in using containerd and Firecracker together! This
44
project is currently in a very early state, and things are not yet as
55
plug-and-play as we'd like. This guide contains general instructions for
66
getting started, but is more descriptive than a concrete set of step-by-step
7-
instructions.
7+
instructions.
88

99
If you prefer to follow a script, you can use the
1010
[quickstart guide](quickstart.md) instead.
@@ -16,12 +16,12 @@ You need to have the following things in order to use firecracker-containerd:
1616
* A computer with a recent-enough version of Linux (4.14+), an Intel x86_64
1717
processor (AMD and Arm are on the roadmap, but not yet supported), and KVM
1818
enabled. An i3.metal running Amazon Linux 2 is a good candidate.
19-
19+
2020
<details>
21-
21+
2222
<summary>Click here to see a bash script that will check if your system meets
2323
the basic requirements to run Firecracker.</summary>
24-
24+
2525
```bash
2626
#!/bin/bash
2727
err=""; \
@@ -35,12 +35,12 @@ You need to have the following things in order to use firecracker-containerd:
3535
&& echo "WARNING: you are running in a virtual machine. Firecracker is not well tested under nested virtualization."; \
3636
[ -z "$err" ] && echo "Your system looks ready for Firecracker!" || echo -e "$err"
3737
```
38-
38+
3939
</details>
4040
* git
4141
* A root filesystem image (you can use the one
4242
[described here](https://github.com/firecracker-microvm/firecracker/blob/main/docs/getting-started.md#running-firecracker)
43-
as `hello-rootfs.ext4`).
43+
as `hello-rootfs.ext4`).
4444
* A recent installation of [Docker CE](https://docker.com).
4545
* Go 1.13 or later, which you can download from [here](https://golang.org/dl/).
4646

@@ -226,8 +226,6 @@ configuration file has the following fields:
226226
* `root_drive` (optional) - A path where the root drive image file is located. A
227227
fully-qualified path is recommended. If left undefined, the runtime looks for
228228
a file named `/var/lib/firecracker-containerd/runtime/default-rootfs.img`.
229-
* `cpu_count` (optional) - The number of vCPUs to make available to a microVM.
230-
If left undefined, the default is 1.
231229
* `cpu_template` (required) - The Firecracker CPU emulation template. Supported
232230
values are "C3" and "T2".
233231
* `additional_drives` (unused)
@@ -295,7 +293,7 @@ $ sudo firecracker-ctr --address /run/firecracker-containerd/containerd.sock \
295293
docker.io/library/busybox:latest busybox-test
296294
```
297295

298-
Alternatively you can specify `--runtime` and `--snapshotter` just once when
296+
Alternatively you can specify `--runtime` and `--snapshotter` just once when
299297
creating a new namespace using containerd's default labels:
300298

301299
```bash
@@ -313,11 +311,11 @@ $ sudo firecracker-ctr --address /run/firecracker-containerd/containerd.sock \
313311
docker.io/library/busybox:latest busybox-test
314312
```
315313

316-
## Networking support
317-
Firecracker-containerd supports the same networking options as provided by the
314+
## Networking support
315+
Firecracker-containerd supports the same networking options as provided by the
318316
Firecracker Go SDK, [documented here](https://github.com/firecracker-microvm/firecracker-go-sdk#network-configuration).
319317
This includes support for configuring VM network interfaces both with
320-
pre-created tap devices and with tap devices created automatically by
318+
pre-created tap devices and with tap devices created automatically by
321319
[CNI](https://github.com/containernetworking/cni) plugins.
322320

323321
### CNI Setup
@@ -330,7 +328,7 @@ Production deployments should be sure to choose a network configuration suitale
330328
to the specifics of the environment and workloads being hosting, with particular
331329
attention being given to network isolation between tasks.
332330

333-
To install the required CNI dependencies, run the following make target from the
331+
To install the required CNI dependencies, run the following make target from the
334332
previously cloned firecracker-containerd repository:
335333

336334
```bash

docs/logging.md

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ firecracker-runtime.json file.
1111
"kernel_image_path": "/var/lib/firecracker-containerd/runtime/default-vmlinux.bin",
1212
"kernel_args": "ro console=ttyS0 noapic reboot=k panic=1 pci=off nomodules systemd.journald.forward_to_console systemd.unit=firecracker.target init=/sbin/overlay-init",
1313
"root_drive": "/var/lib/firecracker-containerd/runtime/default-rootfs.img",
14-
"cpu_count": 1,
1514
"cpu_template": "T2",
1615
"log_levels": ["debug"],
1716
"jailer": {
@@ -40,7 +39,7 @@ firecracker-runtime.json file.
4039
| firecracker-containerd:info | Logs any error, warn, and info level logs |
4140
| firecracker-containerd:debug | Most verbose logging for firecracker-containerd |
4241

43-
The firecracker:XX are mutually exclusive with other firecracker-YY meaning only one of the log levels can be set at a time.
42+
The firecracker:XX are mutually exclusive with other firecracker-YY meaning only one of the log levels can be set at a time.
4443
However, firecracker:output may be set with other firecracker:YY settings.
4544
The firecracker-containerd:XX are also mutually exclusive with other firecracker-containerd-YY levels
4645
info, error, warning, and debug are mutually exclusive and only one can be set at a time.
@@ -51,7 +50,6 @@ info, error, warning, and debug are mutually exclusive and only one can be set a
5150
"kernel_image_path": "/var/lib/firecracker-containerd/runtime/default-vmlinux.bin",
5251
"kernel_args": "ro console=ttyS0 noapic reboot=k panic=1 pci=off nomodules systemd.journald.forward_to_console systemd.unit=firecracker.target init=/sbin/overlay-init",
5352
"root_drive": "/var/lib/firecracker-containerd/runtime/default-rootfs.img",
54-
"cpu_count": 1,
5553
"cpu_template": "T2",
5654
"log:levels": ["info","firecracker:debug","firecracker-containerd:error"],
5755
"jailer": {

examples/etc/containerd/firecracker-runtime.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
"kernel_image_path": "/var/lib/firecracker-containerd/runtime/default-vmlinux.bin",
44
"kernel_args": "ro console=ttyS0 noapic reboot=k panic=1 pci=off nomodules systemd.journald.forward_to_console systemd.unit=firecracker.target init=/sbin/overlay-init",
55
"root_drive": "/var/lib/firecracker-containerd/runtime/default-rootfs.img",
6-
"cpu_count": 1,
76
"cpu_template": "T2",
87
"log_levels": ["debug"],
98
"jailer": {

runtime/README.md

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,6 @@ configuration file has the following fields:
4848
* `root_drive` (optional) - A path where the root drive image file is located. A
4949
fully-qualified path is recommended. If left undefined, the runtime looks for
5050
a file named `/var/lib/firecracker-containerd/runtime/default-rootfs.img`.
51-
* `cpu_count` (optional) - The number of vCPUs to make available to a microVM.
52-
If left undefined, the default is 1.
5351
* `cpu_template` (required) - The Firecracker CPU emulation template. Supported
5452
values are "C3" and "T2".
5553
* `additional_drives` (unused)
@@ -61,5 +59,5 @@ configuration file has the following fields:
6159
* `debug` (optional) - Enable debug-level logging from the runtime.
6260

6361
## Usage
64-
See our [Getting Started Guide](../docs/getting-started.md) for details on how to use
62+
See our [Getting Started Guide](../docs/getting-started.md) for details on how to use
6563
the aws.firecracker runtime.

0 commit comments

Comments
 (0)