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

vdk-plugins: bump docker version and resuse common dind #1701

Merged
merged 5 commits into from
Mar 10, 2023

Conversation

antoniivanov
Copy link
Collaborator

Last change upgraded the shared plugin-ci to use latest version of docker. 3 plugins do not use the shared build-plugin-dind which means they are still using an old version.
Switch the 3 plugins to use build-plugin-dind

Testing Done: this pr ci

Last change upgraded the shared plugin-ci to use latest version of
docker. 3 plugins do not use the shared build-plugin-dind which mean
they are still using old version.

Switch the 3 plugins to use build-plugin-dind

 this pr ci

Signed-off-by: Antoni Ivanov <aivanov@vmware.com>
Copy link
Contributor

@ivakoleva ivakoleva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

vdk-trino seems to use .build-plugin still, https://github.com/vmware/versatile-data-kit/blob/main/projects/vdk-plugins/vdk-trino/.plugin-ci.yml#L6

Maybe reuse and override the tls, is needed?

Basically, any docker-compose.yml would have to be migrated? Should we add a ticket

@antoniivanov
Copy link
Collaborator Author

vdk-trino seems to use .build-plugin still, https://github.com/vmware/versatile-data-kit/blob/main/projects/vdk-plugins/vdk-trino/.plugin-ci.yml#L6

Maybe reuse and override the tls, is needed?

Basically, any docker-compose.yml would have to be migrated? Should we add a ticket

disable tls is something I am not sure is needed now that I look at it. I added while trying to get tests running but it did not solve the issue.

Otherwise added trino to use build-plugin-dind

@antoniivanov antoniivanov enabled auto-merge (squash) March 7, 2023 22:13
@antoniivanov antoniivanov enabled auto-merge (squash) March 10, 2023 12:25
@antoniivanov antoniivanov merged commit fd53929 into main Mar 10, 2023
@antoniivanov antoniivanov deleted the person/aivanov/vdk-plugin branch March 10, 2023 13:02
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