Skip to content
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

Revert dev Dockerfile to use official devcontainer image #2335

Merged
merged 3 commits into from Feb 9, 2024

Conversation

ykadowak
Copy link
Contributor

@ykadowak ykadowak commented Feb 8, 2024

Description

Current nightly devcontainer cannot be opened with vscode devcontainer extension. This PR fixes the issue.

Related Issue

Versions

  • Go Version: 1.22.0
  • Docker Version: 20.10.8
  • Kubernetes Version: v1.29.1
  • NGT Version: 2.1.6

Checklist

Special notes for your reviewer

I don't know why it's trying to import buildcahe and fail when there is no buildcache because it's the first try. But should be fine in the main branch. Also it should not be a big problem since it succeeds to publish image anyway.
https://github.com/vdaas/vald/actions/runs/7826490855/job/21368714853?pr=2335#step:9:5209

@vdaas-ci
Copy link
Collaborator

vdaas-ci commented Feb 8, 2024

[CHATOPS:HELP] ChatOps commands.

  • 🙆‍♀️ /approve - approve
  • 🍱 /format - format codes and add licenses
  • /gen-test - generate test codes
  • 🏷️ /label - add labels
  • 🔚 2️⃣ 🔚 /label actions/e2e-deploy - run E2E deploy & integration test

@ykadowak ykadowak changed the title [WIP] Revert Dockerfile to use official debian based image for devcontainer [WIP] Revert dev Dockerfile to use official debian based image for devcontainer Feb 8, 2024
Copy link

cloudflare-pages bot commented Feb 8, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: dac10af
Status: ✅  Deploy successful!
Preview URL: https://c9866737.vald.pages.dev
Branch Preview URL: https://fix-devcontainer-baseimage.vald.pages.dev

View logs

@ykadowak ykadowak force-pushed the fix/devcontainer/baseimage branch 8 times, most recently from 3a3d0e2 to b0f1a42 Compare February 9, 2024 04:42
@ykadowak ykadowak marked this pull request as ready for review February 9, 2024 05:25
@ykadowak ykadowak changed the title [WIP] Revert dev Dockerfile to use official debian based image for devcontainer Revert dev Dockerfile to use official devcontainer image Feb 9, 2024
@ykadowak ykadowak requested review from a team, kpango and datelier and removed request for a team and kpango February 9, 2024 05:26
@ykadowak ykadowak merged commit 818790a into main Feb 9, 2024
35 checks passed
@ykadowak ykadowak deleted the fix/devcontainer/baseimage branch February 9, 2024 05:33
This was referenced Feb 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants