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

Migrate Security test plugins to KP #66057

Closed
mshustov opened this issue May 11, 2020 · 5 comments · Fixed by #66415
Closed

Migrate Security test plugins to KP #66057

mshustov opened this issue May 11, 2020 · 5 comments · Fixed by #66415
Assignees
Labels
Feature:New Platform Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more!

Comments

@mshustov mshustov added Feature:New Platform Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! labels May 11, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-security (Team:Security)

@mshustov mshustov changed the title Migrate test plugins to KP Migrate Security test plugins to KP May 11, 2020
@jportner jportner self-assigned this May 11, 2020
@jportner
Copy link
Contributor

It looks like @restrry already migrated the plugins for /x-pack/test/ui_capabilities/common/fixtures in #65611.

I added a commit (8a058b6, mentioned above) to migrate the plugins for /x-pack/test/saved_object_api_integration/common/fixtures and /x-pack/test/spaces_api_integration/common/fixtures. However, that commit is in a PR for a feature branch. This will be merged into master with #62553 and then backported to 7.9.

@mshustov
Copy link
Contributor Author

@jportner It seems that your PR is not going to be merged soon. Is it? Does it make sense to merge test plugin migration separately to unblock #65461 ?

@jportner
Copy link
Contributor

@jportner It seems that your PR is not going to be merged soon. Is it? Does it make sense to merge test plugin migration separately to unblock #65461 ?

I spoke to @azasypkin and he indicated it's OK to wait. I'm happy to split that out into a separate PR if circumstances have changed.

@azasypkin
Copy link
Member

azasypkin commented May 13, 2020

I might have confused you all, sorry about that. I meant that it's OK to wait since #65472 (original work that was blocked) is targeting only 7.9. @restrry did I miss the fact that this is just an intermediate dependency for something else that will eventually unblock #65472 and cannot wait?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:New Platform Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants