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.

Sign up for GitHub

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

Jump to bottom

Setting up baseline POC of internal CI #4063

Merged
merged 75 commits into from
Feb 8, 2021
Merged

Setting up baseline POC of internal CI #4063

merged 75 commits into from
Feb 8, 2021

Conversation

mohammedzee1000
Copy link
Contributor

@mohammedzee1000 mohammedzee1000 commented Oct 1, 2020 •
edited
Loading

@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. Required by Prow. label Oct 1, 2020
@mohammedzee1000 mohammedzee1000 mentioned this pull request Oct 1, 2020
Closed
4 tasks
@mohammedzee1000
Copy link
Contributor Author

/test ci/prow/v4.5-integration-e2e

@openshift-ci-robot
Copy link
Collaborator

@mohammedzee1000: The specified target(s) for /test were not found.
The following commands are available to trigger jobs:

  • /test unit
  • /test v4.2-images
  • /test v4.3-images
  • /test v4.4-images
  • /test v4.5-images
  • /test v4.5-integration-e2e

Use /test all to run the following jobs:

  • pull-ci-openshift-odo-master-unit
  • pull-ci-openshift-odo-master-v4.5-integration-e2e

In response to this:

/test ci/prow/v4.5-integration-e2e

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.

@mohammedzee1000
Copy link
Contributor Author

/test v4.5-integration-e2e

4 similar comments
@mohammedzee1000
Copy link
Contributor Author

/test v4.5-integration-e2e

@mohammedzee1000
Copy link
Contributor Author

/test v4.5-integration-e2e

@mohammedzee1000
Copy link
Contributor Author

/test v4.5-integration-e2e

@mohammedzee1000
Copy link
Contributor Author

/test v4.5-integration-e2e

@mohammedzee1000
Copy link
Contributor Author

/test v4.5-integration-e2e

@mohammedzee1000
Copy link
Contributor Author

/retest

@mohammedzee1000
Copy link
Contributor Author

/test v4.5-integration-e2e

@mohammedzee1000
Copy link
Contributor Author

/retest

1 similar comment
@mohammedzee1000
Copy link
Contributor Author

/retest

@mohammedzee1000
Copy link
Contributor Author

/test v4.5-integration-e2e

@mohammedzee1000
Copy link
Contributor Author

/refresh

@mohammedzee1000
Copy link
Contributor Author

/test all

@mohammedzee1000
Copy link
Contributor Author

/test v4.5-integration-e2e

@mohammedzee1000
Copy link
Contributor Author

/test all

@mohammedzee1000
Copy link
Contributor Author

/retest

1 similar comment
@mohammedzee1000
Copy link
Contributor Author

/retest

@mohammedzee1000
Adding remaining tests
a7483c7 
Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@mohammedzee1000
Skipping user login tests
78612d9 
Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@mohammedzee1000
Removing coloration for ginkgo
e0036d2 
Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@mohammedzee1000
Adding helper for oc get pods and using it in operator hub tests
c60cb9e 
oc get pods succeeds, even if there are no resources. So to account
for delays in pods becoming available the fallthough from above mentioned
issue, this was created

Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@mohammedzee1000
Moving noColor to GINKGO_TEST_ARGS
732c063 
Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@mohammedzee1000
Adding justbeforeeach and justafter each to a context
0981fbc 
Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@mohammedzee1000
Fixup for newer image
e8253b8 
Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@mohammedzee1000
Removing Access information from file stat in pod.
efcf039 
1. Touching a file on mounts without `noatime` changes the access time.
2. This includes use by run script in container, hence resulting in errors.
3. We are not interested in access information anyway.

Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@mohammedzee1000
Fixing after rebase
cbb363e 
Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@mohammedzee1000
Increasing curl timeout
f966d87 
Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@mohammedzee1000
Rebasing and addressing comments by @dharmit
c9040a7 
Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@mohammedzee1000
Fixing after rebase
8b6c001 
Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@mohammedzee1000
Readjusting time and order
3f570f1 
Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@mohammedzee1000
Moving operator hub tests to top
46a7456 
Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@mohammedzee1000
Removing ginko nocolor as ansi stripping is available in ci-firewall
64d64cd 
Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@mohammedzee1000
Fixing typo
579eb08 
Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@mohammedzee1000
Adding back old openshift presubmit script with a -old
075c158 
Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@mohammedzee1000
Fixing random project for nodejs-14
5a12670 
Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@mohammedzee1000
Fixing after rebase
7f10941 
Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@mohammedzee1000
Fixing odo command
bb5ec12 
Signed-off-by: Mohammed Zeeshan Ahmed <mohammed.zee1000@gmail.com>
@anandrkskd
Copy link
Contributor

@mohammedzee1000 can we try with a longer timeout? just to be sure if time out is not actually an issue
As the log says it failed due to timeout.

@dharmit
Copy link
Member

dharmit commented Feb 8, 2021

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. Required by Prow. label Feb 8, 2021
@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

4 similar comments
@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-merge-robot openshift-merge-robot merged commit b2483a5 into redhat-developer:master Feb 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sarveshtamba sarveshtamba sarveshtamba left review comments

@anandrkskd anandrkskd anandrkskd left review comments

@girishramnani girishramnani girishramnani left review comments

@dharmit dharmit dharmit requested changes

@prietyc123 prietyc123 prietyc123 requested changes

@cdrage cdrage Awaiting requested review from cdrage

@GeekArthur GeekArthur Awaiting requested review from GeekArthur

Assignees

@dharmit dharmit

@prietyc123 prietyc123

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. Required by Prow. lgtm Indicates that a PR is ready to be merged. Required by Prow.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Run PR tests on internal PSI resources
10 participants
@mohammedzee1000 @openshift-ci-robot @kadel @dharmit @prietyc123 @openshift-merge-robot @openshift-bot @anandrkskd @girishramnani @sarveshtamba