Skip to content

Revert "Emergency fix: remove username/password in Elastic Agent (#637)"#640

Merged
mtojek merged 1 commit intoelastic:masterfrom
mtojek:revert-fix
Jan 17, 2022
Merged

Revert "Emergency fix: remove username/password in Elastic Agent (#637)"#640
mtojek merged 1 commit intoelastic:masterfrom
mtojek:revert-fix

Conversation

@mtojek
Copy link
Copy Markdown
Contributor

@mtojek mtojek commented Jan 10, 2022 •
edited
Loading

This reverts commit 0e97cbe.

Related: elastic/beats#29603

@mtojek mtojek self-assigned this Jan 10, 2022
@mtojek mtojek mentioned this pull request Jan 10, 2022
Closed
@elasticmachine
Copy link
Copy Markdown
Collaborator

elasticmachine commented Jan 10, 2022 •
edited
Loading

💔 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:24.553+0000

  • Duration: 28 min 56 sec

  • Commit: 6c8b35d

Test stats 🧪

Test Results
Failed 0
Passed 459
Skipped 0
Total 459

Steps errors 1

Expand to view the steps failures

Check
  • Took 3 min 15 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.

@mtojek
Copy link
Copy Markdown
Contributor Author

mtojek commented Jan 11, 2022

/test

6 similar comments
@mtojek
Copy link
Copy Markdown
Contributor Author

mtojek commented Jan 12, 2022

/test

@mtojek
Copy link
Copy Markdown
Contributor Author

mtojek commented Jan 12, 2022

/test

@mtojek
Copy link
Copy Markdown
Contributor Author

mtojek commented Jan 12, 2022

/test

@mtojek
Copy link
Copy Markdown
Contributor Author

mtojek commented Jan 13, 2022

/test

@mtojek
Copy link
Copy Markdown
Contributor Author

mtojek commented Jan 14, 2022

/test

@mtojek
Copy link
Copy Markdown
Contributor Author

mtojek commented Jan 16, 2022

/test

@mtojek
Copy link
Copy Markdown
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 61411f4 into elastic:master Jan 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@mtojek mtojek

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mtojek @elasticmachine