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.

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

[ftr] enable visualRegression jobs #42989

Merged
merged 2 commits into from
Aug 12, 2019

Conversation

spalger
Copy link
Contributor

@spalger spalger commented Aug 8, 2019

After a bunch of testing we are ready to enable the visualRegression jobs in CI 🎉

@spalger spalger added Team:Operations Team label for Operations Team v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.4.0 labels Aug 8, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations

@spalger spalger requested review from liza-mae and nreese August 8, 2019 21:50
Copy link
Contributor

@liza-mae liza-mae left a comment

Choose a reason for hiding this comment

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

LGTM

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@liza-mae
Copy link
Contributor

liza-mae commented Aug 9, 2019

Looks like the Percy integration into Github works.

@liza-mae liza-mae self-requested a review August 9, 2019 00:54
Copy link
Contributor

@liza-mae liza-mae left a comment

Choose a reason for hiding this comment

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

Looks like the security login tests are showing up as blank in Percy, either they need to be fixed or disabled, it is best not to use up the snapshots we currently have on our plan. For the same reason,
the console test is showing some non meaningful diffs, maybe we should disable for now until we can get this resolved either on our side or Percy.

@liza-mae
Copy link
Contributor

liza-mae commented Aug 9, 2019

@nreese I noticed your map test show a diff on the logo in bottom left, this is the first time I see this diff on your test.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@nreese nreese left a comment

Choose a reason for hiding this comment

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

lgtm

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@liza-mae liza-mae left a comment

Choose a reason for hiding this comment

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

LGTM

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@spalger spalger merged commit c29a17c into elastic:master Aug 12, 2019
spalger pushed a commit to spalger/kibana that referenced this pull request Aug 12, 2019
* enable visualRegression jobs

* skip the console and login visual tests
spalger pushed a commit that referenced this pull request Aug 12, 2019
* enable visualRegression jobs

* skip the console and login visual tests
@spalger
Copy link
Contributor Author

spalger commented Aug 12, 2019

7.x/7.4: 2441453

@spalger spalger deleted the implement/run-percy-in-prs branch August 12, 2019 22:30
jloleysens added a commit to jloleysens/kibana that referenced this pull request Aug 13, 2019
…p-metrics-selectall

* 'master' of github.com:elastic/kibana: (27 commits)
  [ML] Data Frames: Analytics job creation. (elastic#43102)
  [Vis Default editor] Fix issue with Rollup (elastic#42430)
  [Vis: Default editor] EUIficate Markdown tab (elastic#42677)
  [New Platform Migration Phase I]: update dateHisogramInterval & parseEsInterval imports (elastic#42917)
  [Infra UI] Add AWS metrics to node detail page (elastic#42153)
  update apm index pattern (elastic#43106)
  [SIEM] Toggle Column / Code Coverage and Cypress (elastic#42766)
  skip failing test (elastic#43163)
  [code] Add option to turn the go dependency download on/off. (elastic#43096)
  disable visual regression jobs
  Removed dead code (elastic#42774)
  fixes csv export of saved searches that have _source field (elastic#43123)
  Export missing Context types (elastic#43051)
  Update dependency supports-color to v7 (elastic#43064)
  switch to icon type string instead of node (elastic#43111)
  [Maps] Enable borders for icon symbols (elastic#43066)
  [ftr] enable visualRegression jobs (elastic#42989)
  [ML] Converting single to multi metric job (elastic#42532)
  fix(NA): dont clean dll module if it is a package json file (elastic#42904)
  [Logs UI] Add link from the sample web logs to the Logs UI (elastic#42635)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes Team:Operations Team label for Operations Team v7.4.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants