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

Update OpenAPI Schema's file path in README.md #134

Merged

Conversation

upadhyeammit
Copy link
Collaborator

No description provided.

Copy link
Collaborator

@kgaikwad kgaikwad left a comment

Choose a reason for hiding this comment

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

Change looks good to me but jobs are not happy, failed with errors. Please look into failures.

@subpop
Copy link
Collaborator

subpop commented Oct 25, 2023

These tests sometimes fail. I have not looked into why, but I have found that rerunning the test often ends up succeeding.

{"level":"error","error":"Get \"http://localhost:56998/api/inventory/v1/hosts?fields%5Bsystem_profile%5D=rhc_client_id%2Crhc_config_state&filter%5Bsystem_profile%5D%5Brhc_client_id%5D=not_nil&page=1&registered_with=cloud-connector\": dial tcp [::1]:56998: connect: connection refused","time":"2023-09-28T06:58:50Z","message":"error during request to inventory"}

@subpop
Copy link
Collaborator

subpop commented Oct 25, 2023

The Lint commits workflow checks commits for conformance to the Conventional Commits format.

@upadhyeammit upadhyeammit force-pushed the update-openapi-file-path branch from 083e06a to a128690 Compare November 1, 2023 08:21
@kgaikwad kgaikwad merged commit 5187258 into RedHatInsights:master Feb 1, 2024
4 checks passed
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