-
Notifications
You must be signed in to change notification settings - Fork 82
Create an application/service in the new environment and push configu… #265
Create an application/service in the new environment and push configu… #265
Conversation
test/e2e/features/basic.feature
Outdated
When executing "kam service add --env-name new-env --app-name app-bus --service-name bus --git-repo-url $GIT_REPO_URL --pipelines-folder $GITOPS_PATH" succeeds | ||
Then stderr should be empty | ||
|
||
Scenario: Commit and Push configuration to GitOps repoository |
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 scenario is not required. Please remove it
/retest |
1 similar comment
/retest |
I tested the pr locally and it went fine. I am not sure why its failing in CI. Can you please take a look @amitkrout
|
/retest |
test/e2e/features/basic.feature
Outdated
Then executing "cd .." | ||
When executing "kam environment add --env-name new-env --pipelines-folder servicescenario" succeeds | ||
Then stderr should be empty | ||
When executing "kam service add --env-name new-env --app-name app-bus --service-name bus --git-repo-url $GIT_REPO_URL --pipelines-folder servicescenario" succeeds |
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.
Pass https://github.com/kam-bot/bus.git
instead of $GIT_REPO_URL
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.
Looks like the CI cluster takes longer time to sync the application. Can you please increase timeout to 3mins, for example Wait for "180" seconds
. If it works then we can increase the CI resources and also we have to implement a proper wait logic.
test/e2e/features/basic.feature
Outdated
@@ -31,7 +31,7 @@ Feature: Basic test | |||
Then executing "git push -u origin main" succeeds | |||
Then executing "cd .." succeeds | |||
|
|||
Scenario: Bringing the deployment infrastructure up | |||
Scenario: Bringing the deployment infrastructure up |
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.
Please use proper indention.
…ration to GitOps repository scenarios
/test v4.8-integration-e2e |
I am able to reproduce the failure on 4.8 ocp cluster locally. @amitkrout can you please take a look? |
blocked due to - #266 |
/retest |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: amitkrout The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…ration to GitOps repository scenarios
What type of PR is this?
/kind enhancement
What does this PR do / why we need it:
#136
Have you updated the necessary documentation?
Which issue(s) this PR fixes:
Fixes #?
#136
How to test changes / Special notes to the reviewer: