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

Verify stack boot up for 8.x and 8.0.0 #643

Merged
merged 1 commit into from
Jan 17, 2022
Merged

Conversation

mtojek
Copy link
Contributor

@mtojek mtojek commented Jan 12, 2022

This PR expands the testing scope to cover 8.0.0 and 8.x separately.

@mtojek mtojek self-assigned this Jan 12, 2022
@elasticmachine
Copy link
Collaborator

elasticmachine commented Jan 12, 2022

💔 Build Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-01-16T23:28:14.550+0000

  • Duration: 28 min 8 sec

  • Commit: 6ee2368

Test stats 🧪

Test Results
Failed 0
Passed 459
Skipped 0
Total 459

Steps errors 2

Expand to view the steps failures

Check
  • Took 4 min 31 sec . View more details here
  • Description: make build test-stack-command-800 check-git-clean
Check
  • Took 4 min 33 sec . View more details here
  • Description: make build test-stack-command-8x check-git-clean

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@jsoriano
Copy link
Member

What is the reason to consider 8.0 a special case?

@mtojek
Copy link
Contributor Author

mtojek commented Jan 12, 2022

I suppose it's temporary.

Last week we faced an issue against 8.0.0 due to a missed backport and it would be good to catch some issues as soon as possible as we're heading for release.

EDIT:

BTW in the 8.1.0-SNAPSHOT we're observing this one, so it's a totally different root cause/problem.

@mtojek
Copy link
Contributor Author

mtojek commented Jan 13, 2022

/test

1 similar comment
@mtojek
Copy link
Contributor Author

mtojek commented Jan 14, 2022

/test

@mtojek
Copy link
Contributor Author

mtojek commented Jan 16, 2022

/test

@mtojek
Copy link
Contributor Author

mtojek commented Jan 17, 2022

I think we can merge this one, as there is another, more serious problem:

Attaching to elastic-package-stack_kibana_1
�[36mkibana_1                     |�[0m [2022-01-16T23:40:05.855+00:00][INFO ][plugins-service] Plugin "metricsEntities" is disabled.
�[36mkibana_1                     |�[0m [2022-01-16T23:40:05.910+00:00][FATAL][root] Error: [config validation of [elasticsearch].username]: value of "elastic" is forbidden. This is a superuser account that cannot write to system indices that Kibana needs to function. Use a service account token instead. Learn more: https://www.elastic.co/guide/en/elasticsearch/reference/8.0/service-accounts.html
�[36mkibana_1                     |�[0m     at ensureValidConfiguration (/usr/share/kibana/src/core/server/config/ensure_valid_configuration.js:25:11)
�[36mkibana_1                     |�[0m     at Server.preboot (/usr/share/kibana/src/core/server/server.js:160:5)
�[36mkibana_1                     |�[0m     at Root.preboot (/usr/share/kibana/src/core/server/root/index.js:48:14)
�[36mkibana_1                     |�[0m     at bootstrap (/usr/share/kibana/src/core/server/bootstrap.js:99:9)
�[36mkibana_1                     |�[0m     at Command.<anonymous> (/usr/share/kibana/src/cli/serve/serve.js:216:5)
�[36mkibana_1                     |�[0m 
�[36mkibana_1                     |�[0m  FATAL  Error: [config validation of [elasticsearch].username]: value of "elastic" is forbidden. This is a superuser account that cannot write to system indices that Kibana needs to function. Use a service account token instead. Learn more: https://www.elastic.co/guide/en/elasticsearch/reference/8.0/service-accounts.html
�[36mkibana_1                     |�[0m 

@mtojek mtojek merged commit c26b86e into elastic:master Jan 17, 2022
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.

3 participants