Skip to content

Conversation

@spadgett
Copy link
Member

@spadgett spadgett commented Feb 18, 2019

Previously we were running the bridge binary locally. Instead test the in-cluster console, which tests the built console image.

/cc @jcaianirh @alecmerdler @benjaminapetersen

https://jira.coreos.com/browse/CONSOLE-1267
https://jira.coreos.com/browse/CONSOLE-1266

@openshift-ci-robot openshift-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Feb 18, 2019
Copy link
Contributor

@alecmerdler alecmerdler left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 18, 2019
Copy link
Member

@jcaianirh jcaianirh left a comment

Choose a reason for hiding this comment

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

lgtm

@benjaminapetersen
Copy link
Contributor

level=info msg="Waiting up to 30m0s for the cluster to initialize..."
level=fatal msg="failed to initialize the cluster: Cluster operator network has not yet reported success"

@benjaminapetersen
Copy link
Contributor

/retest

@spadgett
Copy link
Member Author

/retest

level=info msg="Creating cluster..."
level=error
level=error msg="Error: Error applying plan:"
level=error
level=error msg="1 error occurred:"
level=error msg="\t* module.vpc.aws_route_table_association.route_net[3]: 1 error occurred:"
level=error msg="\t* aws_route_table_association.route_net.3: timeout while waiting for state to become 'success' (timeout: 5m0s)"

@spadgett spadgett force-pushed the test-in-cluster-console branch from 1090ad3 to 8e49b16 Compare February 19, 2019 00:06
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Feb 19, 2019
@spadgett
Copy link
Member Author

+ ./test-gui.sh e2e
+ copyArtifacts
++ pwd
+ echo 'Copying artifacts from /go/src/github.com/openshift/console...'
Copying artifacts from /go/src/github.com/openshift/console...
+ cp -rv ./frontend/gui_test_screenshots /tmp/artifacts/gui_test_screenshots
cp: cannot stat './frontend/gui_test_screenshots': No such file or directory

Not sure what failed in this run. It looks like the e2e tests didn't start.

I added a guard in copyArtifacts to avoid that error and added set -x to test-gui.sh for better logging.

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 19, 2019
Copy link
Contributor

@alecmerdler alecmerdler left a comment

Choose a reason for hiding this comment

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

/lgtm

@spadgett
Copy link
Member Author

level=fatal msg="failed to initialize the cluster: Cluster operator network has not yet reported success"

/retest

@spadgett
Copy link
Member Author

/retest

@spadgett spadgett force-pushed the test-in-cluster-console branch from 8e49b16 to cb77dd2 Compare February 19, 2019 02:16
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Feb 19, 2019
@spadgett
Copy link
Member Author

Looks like the call to yarn run webdriver-update is failing. We send output to /dev/null so it's not clear why.

@spadgett spadgett changed the title Tests in-cluster console for e2e tests Test in-cluster console for e2e tests Feb 19, 2019
@spadgett spadgett force-pushed the test-in-cluster-console branch from cb77dd2 to a007524 Compare February 19, 2019 02:37
@spadgett
Copy link
Member Author

Ah, I forgot to add yarn install since we removed build-frontend.sh

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 19, 2019
Copy link
Contributor

@alecmerdler alecmerdler left a comment

Choose a reason for hiding this comment

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

/lgtm

@spadgett
Copy link
Member Author

/retest

1 similar comment
@spadgett
Copy link
Member Author

/retest

@spadgett
Copy link
Member Author

/retest

@spadgett spadgett closed this Feb 22, 2019
@spadgett spadgett reopened this Feb 22, 2019
@spadgett
Copy link
Member Author

Got all the way to the etcd operator tests

Interacting with the etcd OCS creates etcd Operator `Deployment` 1m41s
Error: Timeout - Async callback was not invoked within timeout specified by jasmine.DEFAULT_TIMEOUT_INTERVAL.

/retest

@spadgett
Copy link
Member Author

/retest

8 similar comments
@spadgett
Copy link
Member Author

/retest

@kyoto
Copy link
Member

kyoto commented Feb 22, 2019

/retest

@kyoto
Copy link
Member

kyoto commented Feb 22, 2019

/retest

@kyoto
Copy link
Member

kyoto commented Feb 22, 2019

/retest

@kyoto
Copy link
Member

kyoto commented Feb 22, 2019

/retest

@kyoto
Copy link
Member

kyoto commented Feb 22, 2019

/retest

@kyoto
Copy link
Member

kyoto commented Feb 22, 2019

/retest

@kyoto
Copy link
Member

kyoto commented Feb 22, 2019

/retest

@spadgett
Copy link
Member Author

@kyoto Thanks for helping me retest this 👍

@spadgett spadgett force-pushed the test-in-cluster-console branch from 0194cb9 to 85b8d6d Compare February 22, 2019 14:02
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Feb 22, 2019
@spadgett
Copy link
Member Author

/retest

Previously we were running the bridge binary locally. Instead test the
in-cluster console, which tests the built console image.
@spadgett spadgett force-pushed the test-in-cluster-console branch from 85b8d6d to 427060a Compare February 22, 2019 14:21
@spadgett
Copy link
Member Author

Rate limited

/retest

level=fatal msg="failed to fetch Terraform Variables: failed to fetch dependency of \"Terraform Variables\": failed to fetch dependency of \"Bootstrap Ignition Config\": failed to fetch dependency of \"Common Manifests\": failed to generate asset \"DNS Config\": getting public zone for \"origin-ci-int-aws.dev.rhcloud.com\": listing hosted zones: Throttling: Rate exceeded\n\tstatus code: 400, request id: 329198a7-36af-11e9-8cbf-abfcb2fdb5d0"

Copy link
Contributor

@alecmerdler alecmerdler left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 22, 2019
@spadgett
Copy link
Member Author

backend/frontend failures were infrastructure flakes (test-bin pod was evicted)

@spadgett
Copy link
Member Author

/test backend

@spadgett
Copy link
Member Author

/refresh

@openshift-merge-robot openshift-merge-robot merged commit afb2707 into openshift:master Feb 22, 2019
@spadgett spadgett deleted the test-in-cluster-console branch February 22, 2019 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants