Skip to content

Conversation

@jcaianirh
Copy link
Member

@jcaianirh jcaianirh commented Jan 29, 2019

Remove unused Jenkins and e2e scripts, and adjust the README.md. Also remove logging to console and add to a file called browser.log which resides in the gui_test_screenshots folder and gets copied to artifacts.

@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jan 29, 2019
@jcaianirh
Copy link
Member Author

@TheRealJon moved the console logging to a file as we spoke about.

onComplete: async() => {
console.log('BEGIN BROWSER LOGS');
const consoleLogStream = createWriteStream('gui_test_screenshots/browser.log', { flags: 'a' });
this.logToFile = function() {
Copy link
Member

Choose a reason for hiding this comment

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

Could probably just move this logic inline since it's only used in one place.

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 Jan 30, 2019
@jcaianirh
Copy link
Member Author

/test console-e2e

@jcaianirh
Copy link
Member Author

/retest

@jcaianirh
Copy link
Member Author

/test console-e2e

1 similar comment
@jcaianirh
Copy link
Member Author

/test console-e2e

@openshift-merge-robot openshift-merge-robot merged commit 582967c into openshift:master Jan 31, 2019
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/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants