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

[CCXDEV-10215] CI: Use openapi-generator-cli docker image instead of installing it #1088

Merged
merged 1 commit into from
Jul 21, 2023

Conversation

epapbak
Copy link
Collaborator

@epapbak epapbak commented Jul 21, 2023

Description

Use the docker image provided by the OpenApiTools organization instead of installing the openapi-generator-cli every time a PR is verified.

The installation seems to fail quite often due to downtimes of the repository used to verify and download the jar file used by the verification tool.

The issue does not seem to have been fixed yet. See OpenAPITools/openapi-generator-cli#680 and OpenAPITools/openapi-generator#9349. So using the docker image allows us to use the tool without interacting with the "failing" repository.

Fixes CCXDEV-10215

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Testing steps

CI

Checklist

  • make before_commit passes
  • updated documentation wherever necessary
  • added or modified tests if necessary
  • updated schemas and validators in insights-data-schemas in case of input/output change

Copy link
Collaborator

@tisnik tisnik left a comment

Choose a reason for hiding this comment

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

Let's try it

.travis.yml Show resolved Hide resolved
@epapbak epapbak merged commit 3246bed into RedHatInsights:master Jul 21, 2023
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

Successfully merging this pull request may close these issues.

4 participants