Skip to content

Windows CI tests are failing with invalid character ',' looking for beginning of object key string #982

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

Closed
mauriciopoppe opened this issue May 11, 2022 · 3 comments

Comments

@mauriciopoppe
Copy link
Member

After a few green runs in CI for Windows I saw that the CI jobs failed with the error invalid character ',' looking for beginning of object key string, I checked recent merged PRs and none of them seem to be modifying the Makefile paths related with Windows (the closest PR was #976).

@mauriciopoppe
Copy link
Member Author

After #986 was merged I saw that the post submit job created the image gcr.io/k8s-staging-cloud-provider-gcp/gcp-compute-persistent-disk-csi-driver:master with these contents for the manifest:

{
   "schemaVersion": 2,
   "mediaType": "application/vnd.docker.distribution.manifest.list.v2+json",
   "manifests": [
      {
         "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
         "size": 957,
         "digest": "sha256:1805f8eb3bcf7b004ec6fa5bb0230cdd0d16e5ef7e121653f9f902d1f7e4279b",
         "platform": {
            "architecture": "amd64",
            "os": "windows",
            "os.version": "10.0.19042.1706"
         }
      },
      {
         "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
         "size": 7009,
         "digest": "sha256:51f90c6cc412ac3ad479b74944aae4d57784bbf53d9f59c67d7d4a36b8105fa2",
         "platform": {
            "architecture": "amd64",
            "os": "linux"
         }
      },
      {
         "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
         "size": 7009,
         "digest": "sha256:5d473a7af3d98c32823eafbf8c6efa7d1b51df70c889d6a8ddcb5ca227f4abb4",
         "platform": {
            "architecture": "arm64",
            "os": "linux"
         }
      },
      {
         "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
         "size": 956,
         "digest": "sha256:7451eb02135eefc8abe6e8ac3810840cb8808c88a7207bf4edcdcabd9e508daa",
         "platform": {
            "architecture": "amd64",
            "os": "windows",
            "os.version": "10.0.17763.2928"
         }
      }
   ]
}

@mauriciopoppe
Copy link
Member Author

@mattcary
Copy link
Contributor

Awesome, thanks for tracking this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants