-
Notifications
You must be signed in to change notification settings - Fork 99
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
Change organization for UCP tests and separate cloud/non-cloud tests #7219
Change organization for UCP tests and separate cloud/non-cloud tests #7219
Conversation
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
7d6dca7
to
467f740
Compare
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
cd0be88
to
cbf9bdc
Compare
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
cbf9bdc
to
d7a91a8
Compare
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
test/functional-portable/testUtil.go
Outdated
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
See the License for the specific language governing permissions and | ||
limitations under the License. | ||
*/ |
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.
I have only moved this file to the new directory structure. No changes to the file itself.
d7a91a8
to
0829f2f
Compare
@@ -60,8 +60,13 @@ test-validate-cli: ## Run cli integration tests | |||
|
|||
test-functional-all: test-functional-ucp test-functional-kubernetes test-functional-shared test-functional-msgrp test-functional-daprrp test-functional-datastoresrp test-functional-samples ## Runs all functional tests | |||
|
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.
The second commit in this branch moves the testUtil.go to test/testutil folder. Might be easier to review it separately as it is only renames of package name
9a67788
to
b9de9e4
Compare
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
b9de9e4
to
f56a2e7
Compare
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
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.
after addressing minor feedback. I left a few comments, but overall this is a big improvement.
Signed-off-by: vinayada1 <[email protected]>
Signed-off-by: vinayada1 <[email protected]>
Signed-off-by: vinayada1 <[email protected]>
f56a2e7
to
c8d2b5c
Compare
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Description
Type of change
Fixes: Fixes part of #6588. This is a part of a multi-PR fix