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

Support for superuser not having write access #123338

Closed
wants to merge 13 commits into from

Conversation

tylersmalley
Copy link
Contributor

@tylersmalley tylersmalley commented Jan 19, 2022

Caused by elastic/elasticsearch#81400
Caused by elastic/elasticsearch#81970

There is a PR with the same changes, but running against the current HEAD of ES.

Likely root cause: org.elasticsearch.gateway.CorruptStateException: Format version is not supported. Upgrading to [8.1.0] is only supported from version [7.17.0]

Tyler Smalley and others added 5 commits January 18, 2022 22:02
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
@jportner jportner self-requested a review January 20, 2022 15:56
@jportner
Copy link
Contributor

@elasticmachine merge upstream

@jportner
Copy link
Contributor

Triaged the Saved Objects API integration test failures, they are being caused by a failure that we're encountering in this part of the code:

if (checkAliases) {
for (const space of spaces) {
const rawAliasId = serializer.generateRawLegacyUrlAliasId(space, type, id);
docsToBulkGet.push({
_id: rawAliasId,
_index: getIndexForType(LEGACY_URL_ALIAS_TYPE),
_source: [`${LEGACY_URL_ALIAS_TYPE}.disabled`],
});
aliasSpaces.push(space);
}
}
}
const bulkGetResponse = docsToBulkGet.length
? await client.mget<SavedObjectsRawDocSource>(
{ body: { docs: docsToBulkGet } },
{ ignore: [404] }
)
: undefined;

I opened a bug here: elastic/elasticsearch#82891

Tyler Smalley added 6 commits January 20, 2022 23:33
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/test 193 195 +2

Any counts in public APIs

Total count of every any typed public API. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats any for more detailed information.

id before after diff
@kbn/test 4 5 +1
Unknown metric groups

API count

id before after diff
@kbn/test 218 221 +3

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@jportner jportner removed their request for review January 24, 2022 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants