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

control-service: upgrade docker container used in cicd #911

Merged
merged 15 commits into from
Jul 26, 2022

Conversation

mrMoZ1
Copy link
Contributor

@mrMoZ1 mrMoZ1 commented Jul 25, 2022

what: Upgraded cicd docker image.

why: The cicd (integration tests) were failing because a version with an older jdk was used in the container. Container couldn't install a newer jdk version, even though we tried various ways of pinning the dependency. Only a newer docker allowed us to do this.

testing: integration test

Signed-off-by: Momchil Zhivkov [email protected]

mrMoZ1 added 12 commits July 25, 2022 13:46
Signed-off-by: mrMoZ1 <[email protected]>
Signed-off-by: mrMoZ1 <[email protected]>
Signed-off-by: mrMoZ1 <[email protected]>
Signed-off-by: mrMoZ1 <[email protected]>
Signed-off-by: mrMoZ1 <[email protected]>
Signed-off-by: mrMoZ1 <[email protected]>
jdk
Signed-off-by: mrMoZ1 <[email protected]>
jdk
Signed-off-by: mrMoZ1 <[email protected]>
jdk
Signed-off-by: mrMoZ1 <[email protected]>
jdk
Signed-off-by: mrMoZ1 <[email protected]>
Signed-off-by: mrMoZ1 <[email protected]>
Signed-off-by: mrMoZ1 <[email protected]>
@mrMoZ1 mrMoZ1 changed the title control-service: pin tls version control-service: upgrade docker container used in cicd Jul 25, 2022
@mrMoZ1 mrMoZ1 enabled auto-merge (squash) July 25, 2022 16:08
Signed-off-by: mrMoZ1 <[email protected]>
@mrMoZ1 mrMoZ1 force-pushed the person/mzhivkov/fix-cicd branch from 72d4155 to ff8bac8 Compare July 26, 2022 07:24
@mrMoZ1 mrMoZ1 merged commit 5776f5c into main Jul 26, 2022
@mrMoZ1 mrMoZ1 deleted the person/mzhivkov/fix-cicd branch July 26, 2022 08:24
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.

3 participants