Skip to content

Conversation

@tamird
Copy link
Contributor

@tamird tamird commented Jan 8, 2026

  • Dockerfile: use correct gcloud CLI URL for arm
  • Remove docker config bind mount
    This was introduced in d796b10 with a
    TODO that suggested it should be removed when VFS1 was deleted
    (gvisor.dev/issue/1624). 756bc3e
    removed that TODO which broke the link. Now that VFS1 is deleted, we no
    longer need this.
  • bazel: add realpath -m, stat -c compat shims
    This allows make tests to run on macOS.

@tamird
Copy link
Contributor Author

tamird commented Jan 9, 2026

@ayushr2 could you have a look please? seems this is still waiting for import

@ayushr2
Copy link
Collaborator

ayushr2 commented Jan 9, 2026

imported and sent for internal review

copybara-service bot pushed a commit that referenced this pull request Jan 9, 2026
- **Dockerfile: use correct gcloud CLI URL for arm**
- **Remove docker config bind mount**
This was introduced in d796b10 with a
  TODO that suggested it should be removed when VFS1 was deleted
  (gvisor.dev/issue/1624). 756bc3e
  removed that TODO which broke the link. Now that VFS1 is deleted, we no
  longer need this.
- **bazel: add `realpath -m`, `stat -c` compat shims**
  This allows `make tests` to run on macOS.

FUTURE_COPYBARA_INTEGRATE_REVIEW=#12461 from tamird:build-on-mac 18886c8
PiperOrigin-RevId: 854185855
copybara-service bot pushed a commit that referenced this pull request Jan 9, 2026
- **Dockerfile: use correct gcloud CLI URL for arm**
- **Remove docker config bind mount**
This was introduced in d796b10 with a
  TODO that suggested it should be removed when VFS1 was deleted
  (gvisor.dev/issue/1624). 756bc3e
  removed that TODO which broke the link. Now that VFS1 is deleted, we no
  longer need this.
- **bazel: add `realpath -m`, `stat -c` compat shims**
  This allows `make tests` to run on macOS.

FUTURE_COPYBARA_INTEGRATE_REVIEW=#12461 from tamird:build-on-mac 18886c8
PiperOrigin-RevId: 854185855
copybara-service bot pushed a commit that referenced this pull request Jan 9, 2026
- **Dockerfile: use correct gcloud CLI URL for arm**
- **Remove docker config bind mount**
This was introduced in d796b10 with a
  TODO that suggested it should be removed when VFS1 was deleted
  (gvisor.dev/issue/1624). 756bc3e
  removed that TODO which broke the link. Now that VFS1 is deleted, we no
  longer need this.
- **bazel: add `realpath -m`, `stat -c` compat shims**
  This allows `make tests` to run on macOS.

FUTURE_COPYBARA_INTEGRATE_REVIEW=#12461 from tamird:build-on-mac 18886c8
PiperOrigin-RevId: 854185855
copybara-service bot pushed a commit that referenced this pull request Jan 9, 2026
- **Dockerfile: use correct gcloud CLI URL for arm**
- **Remove docker config bind mount**
This was introduced in d796b10 with a
  TODO that suggested it should be removed when VFS1 was deleted
  (gvisor.dev/issue/1624). 756bc3e
  removed that TODO which broke the link. Now that VFS1 is deleted, we no
  longer need this.
- **bazel: add `realpath -m`, `stat -c` compat shims**
  This allows `make tests` to run on macOS.

FUTURE_COPYBARA_INTEGRATE_REVIEW=#12461 from tamird:build-on-mac 18886c8
PiperOrigin-RevId: 854185855
@shailend-g
Copy link
Contributor

The internal Buildkite workflow fails, maybe worth making the Python script backwards compatible:

SyntaxError: future feature annotations is not defined
  File ".../gvisor/internal/tools/compat/realpath.py", line 19
    from __future__ import annotations

tamird added 3 commits January 9, 2026 18:43
This was introduced in d796b10 with a
TODO that suggested it should be removed when VFS1 was deleted
(gvisor.dev/issue/1624). 756bc3e
removed that TODO which broke the link. VFS1 is deleted now, but a new
dependency has emerged. Document it.
This file does not exist on macOS.
This allows `make tests` to run on macOS.
@tamird
Copy link
Contributor Author

tamird commented Jan 9, 2026

The internal Buildkite workflow fails, maybe worth making the Python script backwards compatible:

SyntaxError: future feature annotations is not defined
  File ".../gvisor/internal/tools/compat/realpath.py", line 19
    from __future__ import annotations

Removed! Wasn't needed.

copybara-service bot pushed a commit that referenced this pull request Jan 10, 2026
- **Dockerfile: use correct gcloud CLI URL for arm**
- **Remove docker config bind mount**
This was introduced in d796b10 with a
  TODO that suggested it should be removed when VFS1 was deleted
  (gvisor.dev/issue/1624). 756bc3e
  removed that TODO which broke the link. Now that VFS1 is deleted, we no
  longer need this.
- **bazel: add `realpath -m`, `stat -c` compat shims**
  This allows `make tests` to run on macOS.

FUTURE_COPYBARA_INTEGRATE_REVIEW=#12461 from tamird:build-on-mac f9d0da5
PiperOrigin-RevId: 854185855
copybara-service bot pushed a commit that referenced this pull request Jan 10, 2026
- **Dockerfile: use correct gcloud CLI URL for arm**
- **Remove docker config bind mount**
This was introduced in d796b10 with a
  TODO that suggested it should be removed when VFS1 was deleted
  (gvisor.dev/issue/1624). 756bc3e
  removed that TODO which broke the link. Now that VFS1 is deleted, we no
  longer need this.
- **bazel: add `realpath -m`, `stat -c` compat shims**
  This allows `make tests` to run on macOS.

FUTURE_COPYBARA_INTEGRATE_REVIEW=#12461 from tamird:build-on-mac f9d0da5
PiperOrigin-RevId: 854185855
copybara-service bot pushed a commit that referenced this pull request Jan 10, 2026
- **Dockerfile: use correct gcloud CLI URL for arm**
- **Remove docker config bind mount**
This was introduced in d796b10 with a
  TODO that suggested it should be removed when VFS1 was deleted
  (gvisor.dev/issue/1624). 756bc3e
  removed that TODO which broke the link. Now that VFS1 is deleted, we no
  longer need this.
- **bazel: add `realpath -m`, `stat -c` compat shims**
  This allows `make tests` to run on macOS.

FUTURE_COPYBARA_INTEGRATE_REVIEW=#12461 from tamird:build-on-mac f9d0da5
PiperOrigin-RevId: 854185855
copybara-service bot pushed a commit that referenced this pull request Jan 11, 2026
- **Dockerfile: use correct gcloud CLI URL for arm**
- **Remove docker config bind mount**
This was introduced in d796b10 with a
  TODO that suggested it should be removed when VFS1 was deleted
  (gvisor.dev/issue/1624). 756bc3e
  removed that TODO which broke the link. Now that VFS1 is deleted, we no
  longer need this.
- **bazel: add `realpath -m`, `stat -c` compat shims**
  This allows `make tests` to run on macOS.

FUTURE_COPYBARA_INTEGRATE_REVIEW=#12461 from tamird:build-on-mac f9d0da5
PiperOrigin-RevId: 854185855
copybara-service bot pushed a commit that referenced this pull request Jan 11, 2026
- **Dockerfile: use correct gcloud CLI URL for arm**
- **Remove docker config bind mount**
This was introduced in d796b10 with a
  TODO that suggested it should be removed when VFS1 was deleted
  (gvisor.dev/issue/1624). 756bc3e
  removed that TODO which broke the link. Now that VFS1 is deleted, we no
  longer need this.
- **bazel: add `realpath -m`, `stat -c` compat shims**
  This allows `make tests` to run on macOS.

FUTURE_COPYBARA_INTEGRATE_REVIEW=#12461 from tamird:build-on-mac f9d0da5
PiperOrigin-RevId: 854185855
copybara-service bot pushed a commit that referenced this pull request Jan 11, 2026
- **Dockerfile: use correct gcloud CLI URL for arm**
- **Remove docker config bind mount**
This was introduced in d796b10 with a
  TODO that suggested it should be removed when VFS1 was deleted
  (gvisor.dev/issue/1624). 756bc3e
  removed that TODO which broke the link. Now that VFS1 is deleted, we no
  longer need this.
- **bazel: add `realpath -m`, `stat -c` compat shims**
  This allows `make tests` to run on macOS.

FUTURE_COPYBARA_INTEGRATE_REVIEW=#12461 from tamird:build-on-mac f9d0da5
PiperOrigin-RevId: 854185855
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.

4 participants