Skip to content

Add the troubleshoot item for "Calling Remote API Errors"#45

Open
jinhli wants to merge 6 commits into
redhat-openshift-ecosystem:mainfrom
jinhli:main
Open

Add the troubleshoot item for "Calling Remote API Errors"#45
jinhli wants to merge 6 commits into
redhat-openshift-ecosystem:mainfrom
jinhli:main

Conversation

@jinhli

@jinhli jinhli commented May 10, 2022

Copy link
Copy Markdown

For some old projects, the ospid and project id are different. If the older ospid is using in the preflight test, it will cause "calling remote api errors"

@acornett21 acornett21 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@jinhli It looks like two files were included in the PR, I think you only wanted to update the troubleshooting guide.


## <a id="callingremoteapierrors"></a>Calling Remote API Errors
Possible causes:
- You are using the old opsid in --certification-project-id, you could find the project id in project link https://connect.redhat.com/projects/1234567890/overview

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think the wording here should be consistend with the --help output of preflight You can see that verbiage here

@openshift-ci

openshift-ci Bot commented Jun 7, 2022

Copy link
Copy Markdown

@jinhli: PR needs rebase.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci openshift-ci Bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants