Skip to content

Commit fa917db

Browse files
fix: push cpp container to gabrielfrasantos/embedded-devcontainer-cpp (#62)
Agent-Logs-Url: https://github.com/embedded-pro/embedded-devcontainer/sessions/c48a1a5c-d49d-4db9-af7b-886392f5bce6 Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: gabrielfrasantos <21131318+gabrielfrasantos@users.noreply.github.com>
1 parent 912505e commit fa917db

9 files changed

Lines changed: 14 additions & 21 deletions

File tree

.devcontainer/cpp-test/devcontainer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
{
2-
"image": "gabrielfrasantos/embedded-devcontainer:${localEnv:IMAGE_VERSION}",
2+
"image": "gabrielfrasantos/embedded-devcontainer-cpp:${localEnv:IMAGE_VERSION}",
33
"workspaceFolder": "/workspaces/embedded-devcontainer/test/cpp/workspace",
44
"containerEnv": {
55
"NODE_EXTRA_CA_CERTS": "/etc/ssl/certs/ca-certificates.crt"

.github/ISSUE_TEMPLATE/bug_report.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ A clear and concise description of what you expected to happen.
2424
If applicable, add screenshots to help explain your problem.
2525

2626
**Environment (please complete the following information):**
27-
- Container: [e.g. gabrielfrasantos/embedded-devcontainer:1.0.0]
27+
- Container: [e.g. gabrielfrasantos/embedded-devcontainer-cpp:1.0.0]
2828
- OS: [e.g. Windows, Linux, OSX]
2929
- Architecture: [e.g. x86_64, Apple M2]
3030
- Docker Version: [e.g. Docker Desktop 4.25.2]

.github/RELEASE_TEMPLATE.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,6 @@
1111
| Container | Full identifier |
1212
|----------------------------|---------------------------------------------------------------------------------------------------------------------------|
1313
| embedded-devcontainer-base | gabrielfrasantos/embedded-devcontainer-base:{{ embedded-devcontainer-base-version }}@{{ embedded-devcontainer-base-sha }} |
14-
| embedded-devcontainer-cpp | gabrielfrasantos/embedded-devcontainer:{{ embedded-devcontainer-cpp-version }}@{{ embedded-devcontainer-cpp-sha }} |
14+
| embedded-devcontainer-cpp | gabrielfrasantos/embedded-devcontainer-cpp:{{ embedded-devcontainer-cpp-version }}@{{ embedded-devcontainer-cpp-sha }} |
1515
| embedded-devcontainer-rust | gabrielfrasantos/embedded-devcontainer-rust:{{ embedded-devcontainer-rust-version }}@{{ embedded-devcontainer-rust-sha }} |
1616

.github/copilot-instructions.md

Lines changed: 6 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -37,13 +37,9 @@ upstream sync must re-apply the deltas listed below.
3737
| Flavor | Upstream image | This fork |
3838
| ------ | ----------------------------------------------------------- | --------------------------------------------------------------- |
3939
| base | `ghcr.io/philips-software/amp-devcontainer-base` | `docker.io/gabrielfrasantos/embedded-devcontainer-base` |
40-
| cpp | `ghcr.io/philips-software/amp-devcontainer-cpp` | `docker.io/gabrielfrasantos/embedded-devcontainer` (no suffix) |
40+
| cpp | `ghcr.io/philips-software/amp-devcontainer-cpp` | `docker.io/gabrielfrasantos/embedded-devcontainer-cpp` |
4141
| rust | `ghcr.io/philips-software/amp-devcontainer-rust` | `docker.io/gabrielfrasantos/embedded-devcontainer-rust` |
4242

43-
Note: the `cpp` flavor is intentionally the "default" image (no `-cpp` suffix)
44-
to preserve the historical Docker Hub repo name. The base and rust images keep
45-
their `-base` / `-rust` suffix.
46-
4743
### Workflow deltas (vs upstream)
4844

4945
- `build-push-test.yml` overrides `registry: docker.io` and pins each flavor's
@@ -52,11 +48,9 @@ their `-base` / `-rust` suffix.
5248
reusable `wc-build-push-test.yml`.
5349
- `continuous-integration.yml` and `release-build.yml` forward the same two
5450
Docker Hub secrets.
55-
- `vulnerability-scan.yml` scans the Docker Hub images (with the cpp
56-
no-suffix special case) and uses the correct per-flavor Dockerfile path
51+
- `vulnerability-scan.yml` scans the Docker Hub images and uses the correct per-flavor Dockerfile path
5752
`.devcontainer/${{ matrix.flavor }}/Dockerfile`.
58-
- `update-dependencies.yml` runs in the published Docker Hub image at the
59-
`:latest` tag (same no-suffix cpp special case).
53+
- `update-dependencies.yml` runs in the published Docker Hub image at the `:latest` tag.
6054
- `image-cleanup.yml` / `pr-image-cleanup.yml` still target ghcr.io via
6155
`dataaxiom/ghcr-cleanup-action`. They are effectively no-ops in this fork
6256
(we publish to Docker Hub). Leave them or remove them — do not point them
@@ -72,12 +66,11 @@ Run a global rebrand on every upstream sync. The mapping is:
7266

7367
- `philips-software/amp-devcontainer``embedded-pro/embedded-devcontainer`
7468
- `amp-devcontainer-base` / `-rust``embedded-devcontainer-base` / `-rust`
75-
- `amp-devcontainer-cpp``embedded-devcontainer-cpp` (text) **but**
76-
`gabrielfrasantos/embedded-devcontainer` (image, no suffix)
69+
- `amp-devcontainer-cpp``embedded-devcontainer-cpp` (text and image)
7770
- `amp-devcontainer` (bare) → `embedded-devcontainer`
7871
- `/workspaces/amp-devcontainer``/workspaces/embedded-devcontainer`
7972
- `https://github.com/orgs/philips-software/packages/container/package/amp-devcontainer-X`
80-
`https://hub.docker.com/r/gabrielfrasantos/embedded-devcontainer[-X]`
73+
`https://hub.docker.com/r/gabrielfrasantos/embedded-devcontainer-X`
8174

8275
**Preserve:**
8376

@@ -93,7 +86,7 @@ For CI to actually publish, the following must be set in this fork:
9386

9487
- Repository secrets `DOCKER_REGISTRY_USERNAME` and `DOCKER_REGISTRY_PASSWORD`
9588
(Docker Hub PAT for the `gabrielfrasantos` account).
96-
- The Docker Hub repos `gabrielfrasantos/embedded-devcontainer`,
89+
- The Docker Hub repos `gabrielfrasantos/embedded-devcontainer-cpp`,
9790
`gabrielfrasantos/embedded-devcontainer-base`, and
9891
`gabrielfrasantos/embedded-devcontainer-rust` must exist.
9992

.github/workflows/build-push-test.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ jobs:
6767
devcontainer-metadata-file: .devcontainer/${{ matrix.flavor }}/devcontainer-metadata.json
6868
dockerfile: .devcontainer/${{ matrix.flavor }}/Dockerfile
6969
enable-edge-tag: ${{ github.event_name == 'merge_group' }}
70-
image-name: ${{ matrix.flavor == 'cpp' && 'gabrielfrasantos/embedded-devcontainer' || format('gabrielfrasantos/embedded-devcontainer-{0}', matrix.flavor) }}
70+
image-name: gabrielfrasantos/embedded-devcontainer-${{ matrix.flavor }}
7171
registry: docker.io
7272
integration-test-file: test/${{ matrix.flavor }}/integration-tests.bats
7373
integration-test-podman: true

.github/workflows/update-dependencies.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ jobs:
1919
flavor: ["base", "cpp", "rust"]
2020
# Using our own container is required since we need all package sources
2121
# set-up correctly.
22-
container: ${{ matrix.flavor == 'cpp' && 'gabrielfrasantos/embedded-devcontainer' || format('gabrielfrasantos/embedded-devcontainer-{0}', matrix.flavor) }}:latest
22+
container: gabrielfrasantos/embedded-devcontainer-${{ matrix.flavor }}:latest
2323
permissions:
2424
contents: write # is needed by peter-evans/create-pull-request to create branches and push commits
2525
pull-requests: write # is needed by peter-evans/create-pull-request to create a PR

.github/workflows/vulnerability-scan.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ jobs:
2424
- uses: crazy-max/ghaction-container-scan@a0a3900b79d158c85ccf034e5368fae620a9233a # v4.0.0
2525
id: scan
2626
with:
27-
image: ${{ matrix.flavor == 'cpp' && 'gabrielfrasantos/embedded-devcontainer' || format('gabrielfrasantos/embedded-devcontainer-{0}', matrix.flavor) }}:latest
27+
image: gabrielfrasantos/embedded-devcontainer-${{ matrix.flavor }}:latest
2828
dockerfile: .devcontainer/${{ matrix.flavor }}/Dockerfile
2929
- uses: github/codeql-action/upload-sarif@e46ed2cbd01164d986452f91f178727624ae40d7 # v4.35.3
3030
if: steps.scan.outputs.sarif != ''

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ This repository is under active development; see [pulse](https://github.com/embe
5555
The following devcontainers are published towards the [GitHub Container Registry](https://ghcr.io/):
5656

5757
- [embedded-devcontainer-base](https://hub.docker.com/r/gabrielfrasantos/embedded-devcontainer-base); shared base image used by the other flavors
58-
- [embedded-devcontainer-cpp](https://hub.docker.com/r/gabrielfrasantos/embedded-devcontainer); the C++ container
58+
- [embedded-devcontainer-cpp](https://hub.docker.com/r/gabrielfrasantos/embedded-devcontainer-cpp); the C++ container
5959
- [embedded-devcontainer-rust](https://hub.docker.com/r/gabrielfrasantos/embedded-devcontainer-rust); the Rust container
6060

6161
All containers include a full [Visual Studio Code](https://code.visualstudio.com/) configuration that is compatible with [GitHub Codespaces](https://github.com/features/codespaces).
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
1-
FROM gabrielfrasantos/embedded-devcontainer:${templateOption:imageVersion}
1+
FROM gabrielfrasantos/embedded-devcontainer-cpp:${templateOption:imageVersion}
22

33
HEALTHCHECK NONE

0 commit comments

Comments
 (0)