-
Notifications
You must be signed in to change notification settings - Fork 244
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
Doc automation:Command Reference: odo create namespace #6703
Doc automation:Command Reference: odo create namespace #6703
Conversation
@ritudes: The label(s) In response to this:
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. |
✅ Deploy Preview for odo-docusaurus-preview ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good! Thank you for working on this :)
Apart from the changes I have suggested, there is one more change I'd like i.e. Restore the yarn.lock
file
tests/documentation/command-reference/doc_command_reference_createnamespace_test.go
Outdated
Show resolved
Hide resolved
tests/documentation/command-reference/doc_command_reference_createnamespace_test.go
Outdated
Show resolved
Hide resolved
docs/website/docs/command-reference/docs-mdx/create-namespace/create_namespace.mdx
Outdated
Show resolved
Hide resolved
docs/website/docs/command-reference/docs-mdx/create-namespace/create_project.mdx
Outdated
Show resolved
Hide resolved
tests/documentation/command-reference/doc_command_reference_createnamespace_test.go
Show resolved
Hide resolved
tests/documentation/command-reference/doc_command_reference_createnamespace_test.go
Show resolved
Hide resolved
@ritudes Can you rebase your branch on top of the latest main branch? That should fix the doc test failure. [FAILED] registry_list_output.mdx
Expected
<string>: strings.Join({
... // 101 identical bytes
" VERSIONS\nnodejs-react StagingReg",
"istry React is a free and open-source front-en... 2.0.2",
+ ", 2.1.0",
"\nnodejs-react DefaultDevfileRegistry React is a free and open-",
"source front-en... 2.0.2\n```\n",
}, "")
to be empty
Summarizing 1 Failure:
[FAIL] doc command reference odo init Non Interactive Mode fetching devfile from a registry when setting up the registry [It] Fetch Devfile from any registry of the list [nocluster]
/go/odo_1/tests/documentation/command-reference/doc_command_reference_init_test.go:189 Philippe has already fixed this failure in #6710. |
…create_namespace.mdx Co-authored-by: Parthvi Vala <[email protected]>
…create_project.mdx Co-authored-by: Parthvi Vala <[email protected]>
…eatenamespace_test.go Co-authored-by: Parthvi Vala <[email protected]>
Signed-off-by: Ritu Deshmukh <[email protected]>
/override Kubernetes-Integration-Tests/Kubernetes-Docs-Integration-Tests [FAILED] user-guides/quickstart/docs-mdx/java/java_odo_dev_output.mdx
Expected
<string>: (
"""
... // 10 identical lines
✓ Added storage m2 to component
âš Pod is Pending
- ✓ Pod is Running
✓ Syncing files into the container [1s]
✓ Building your application in container (command: build) [1s]
... // 9 identical lines
[Ctrl+c] - Exit and delete resources from the cluster
[p] - Manually apply local changes to the application on the cluster
+ ✓ Pod is Running
```
"""
)
to be empty
In [It] at: /go/odo_1/tests/documentation/user-guides/doc_user_guides_quickstart_test.go:256 @ 04/06/23 13:23:11.307
------------------------------
Summarizing 1 Failure:
[FAIL] User guides: Quickstart test Java quickstart guide [It] s |
@valaparthvi: Overrode contexts on behalf of valaparthvi: Kubernetes-Integration-Tests/Kubernetes-Docs-Integration-Tests In response to this:
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. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
/override windows-integration-test/Windows-test [FAILED] Expected
<*url.Error | 0xc00057e720>: {
Op: "Post",
URL: "http://127.0.0.1:55919/api/newuser",
Err: <*errors.errorString | 0xc00008a130>{s: "EOF"},
}
to be nil
In [It] at: C:/Users/Administrator.ANSIBLE-TEST-VS/3654/tests/e2escenarios/e2e_test.go:306 @ 04/10/23 00:48:11.831
There were additional failures detected. To view them in detail run ginkgo -vv
------------------------------
Summarizing 1 Failure:
[FAIL] E2E Test starting with non-empty Directory add Binding [It] should verify developer workflow of using binding as env in innerloop
C:/Users/Administrator.ANSIBLE-TEST-VS/3654/tests/e2escenarios/e2e_test.go:306 /override OpenShift-Integration-tests/OpenShift-Integration-tests Summarizing 1 Failure:
[FAIL] odo dev command tests when Starting a PostgreSQL service when creating local files and dir and running odo dev - without metadata.name [AfterEach] when deleting local files and dir and waiting for sync should not list deleted dir and file in container
/go/odo_1/tests/helper/helper_dev.go:207
Overriding the failures because they are not related to the PR. |
@valaparthvi: Overrode contexts on behalf of valaparthvi: OpenShift-Integration-tests/OpenShift-Integration-tests, windows-integration-test/Windows-test In response to this:
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. |
What type of PR is this:
/area documentation
What does this PR do / why we need it:
Which issue(s) this PR fixes:
Fixes #6675
PR acceptance criteria:
Unit test
Integration test
Documentation
How to test changes / Special notes to the reviewer: