-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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] update svl search config and custom role test #196126
Conversation
`--xpack.cloud.id=ES3_FTR_TESTS:ZmFrZS1kb21haW4uY2xkLmVsc3RjLmNvJGZha2Vwcm9qZWN0aWQuZXMkZmFrZXByb2plY3RpZC5rYg==`, | ||
`--xpack.cloud.serverless.project_id=fakeprojectid`, | ||
`--xpack.cloud.serverless.project_name=ES3_FTR_TESTS`, | ||
`--xpack.cloud.serverless.project_type=elasticsearch`, | ||
`--xpack.cloud.base_url=https://fake-cloud.elastic.co`, | ||
`--xpack.cloud.profile_url=/user/settings/`, | ||
`--xpack.cloud.billing_url=/billing/overview/`, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pheyos we probably need to move these arguments to shared config as it is relevant for all projects. But I'm not sure if we want to generate id/name of have a static value based on project type.
// Playground should be also hidden, probably a bug | ||
// await testSubjects.missingOrFail('~nav-item-id-searchPlayground'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@elastic/search-kibana
It seems like application privileges are not properly applied to searchPlayground
. The role I defined has only access to Discover, but I still see and able to open Playground
link, later to see error message
Similar question is if Connectors
should be visible by default?
Role descriptors:
{
elasticsearch: {
indices: [{ names: ['logstash-*'], privileges: ['read', 'view_index_metadata'] }],
},
kibana: [
{
feature: {
discover: ['read'],
},
spaces: ['*'],
},
],
}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this is known and something that is being worked on currently
💚 Build Succeeded
Metrics [docs]
cc @dmlemeshko |
Summary
This PR removes
xpack.cloud.serverless.project_type
from FTR config, the value breaks mockIdpPlugin that expects it to be defined assearch
. We will address project type unification separately.Closes #195976