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.

Sign up for GitHub

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

Jump to bottom

[CI] PackageTests test60Reinstall failing #71762

Closed
mark-vieira opened this issue Apr 15, 2021 · 3 comments · Fixed by #71764
Closed

[CI] PackageTests test60Reinstall failing #71762

mark-vieira opened this issue Apr 15, 2021 · 3 comments · Fixed by #71764
Assignees
@probakowski
Labels
:Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP Team:Data Management Meta label for data/management team >test-failure Triaged test failures from CI

Comments

@mark-vieira
Copy link
Contributor

I think this might be fallout from #71505

Build scan:
https://gradle-enterprise.elastic.co/s/uht66ddl3ftic/tests/:qa:os:destructiveDistroTest.default-deb/org.elasticsearch.packaging.test.PackageTests/test60Reinstall

Reproduction line:
null

Applicable branches:
master, 7.x

Reproduces locally?:
Didn't try

Failure history:
https://gradle-enterprise.elastic.co/scans/tests?tests.container=org.elasticsearch.packaging.test.PackageTests&tests.test=test60Reinstall

Failure excerpt:

org.elasticsearch.packaging.util.Shell$ShellException: Command was not successful: [bash -c sudo -u elasticsearch /usr/share/elasticsearch/bin/elasticsearch-keystore list]
   result: exitCode = [1] stdout = [] stderr = [Exception in thread "main" org.elasticsearch.common.settings.SettingsException: Failed to load settings from [elasticsearch.yml]
	at org.elasticsearch.common.settings.Settings$Builder.loadFromStream(Settings.java:1071)
	at org.elasticsearch.common.settings.Settings$Builder.loadFromPath(Settings.java:1044)
	at org.elasticsearch.node.InternalSettingsPreparer.prepareEnvironment(InternalSettingsPreparer.java:64)
	at org.elasticsearch.cli.EnvironmentAwareCommand.createEnv(EnvironmentAwareCommand.java:89)
	at org.elasticsearch.cli.EnvironmentAwareCommand.createEnv(EnvironmentAwareCommand.java:80)
	at org.elasticsearch.cli.EnvironmentAwareCommand.execute(EnvironmentAwareCommand.java:75)
	at org.elasticsearch.cli.Command.mainWithoutErrorHandling(Command.java:116)
	at org.elasticsearch.cli.MultiCommand.execute(MultiCommand.java:80)
	at org.elasticsearch.cli.Command.mainWithoutErrorHandling(Command.java:116)
	at org.elasticsearch.cli.Command.main(Command.java:79)
	at org.elasticsearch.common.settings.KeyStoreCli.main(KeyStoreCli.java:32)
Caused by: com.fasterxml.jackson.core.JsonParseException: Duplicate field 'ingest.geoip.downloader.enabled'
 at [Source: (sun.nio.ch.ChannelInputStream); line: 84, column: 32]
	at com.fasterxml.jackson.core.json.JsonReadContext._checkDup(JsonReadContext.java:204)
	at com.fasterxml.jackson.core.json.JsonReadContext.setCurrentName(JsonReadContext.java:198)
	at com.fasterxml.jackson.dataformat.yaml.YAMLParser.nextToken(YAMLParser.java:399)
	at org.elasticsearch.common.xcontent.json.JsonXContentParser.nextToken(JsonXContentParser.java:49)
	at org.elasticsearch.common.settings.Settings.fromXContent(Settings.java:603)
	at org.elasticsearch.common.settings.Settings.fromXContent(Settings.java:578)
	at org.elasticsearch.common.settings.Settings$Builder.loadFromStream(Settings.java:1067)
	... 10 more]

  at __randomizedtesting.SeedInfo.seed([30ABA2F80D8441D9:DA3B2FE65DAB1E48]:0)
  at org.elasticsearch.packaging.util.Shell.runScript(Shell.java:147)
  at org.elasticsearch.packaging.util.Shell.run(Shell.java:73)
  at org.elasticsearch.packaging.util.Packages.verifyOssInstallation(Packages.java:181)
  at org.elasticsearch.packaging.util.Packages.verifyPackageInstallation(Packages.java:152)
  at org.elasticsearch.packaging.test.PackageTests.test60Reinstall(PackageTests.java:219)
  at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(NativeMethodAccessorImpl.java:-2)
  at jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
  at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
  at java.lang.reflect.Method.invoke(Method.java:566)
  at com.carrotsearch.randomizedtesting.RandomizedRunner.invoke(RandomizedRunner.java:1758)
  at com.carrotsearch.randomizedtesting.RandomizedRunner$8.evaluate(RandomizedRunner.java:946)
  at com.carrotsearch.randomizedtesting.RandomizedRunner$9.evaluate(RandomizedRunner.java:982)
  at com.carrotsearch.randomizedtesting.RandomizedRunner$10.evaluate(RandomizedRunner.java:996)
  at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
  at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
  at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
  at com.carrotsearch.randomizedtesting.ThreadLeakControl$StatementRunner.run(ThreadLeakControl.java:375)
  at com.carrotsearch.randomizedtesting.ThreadLeakControl.lambda$forkTimeoutingTask$0(ThreadLeakControl.java:831)
  at java.lang.Thread.run(Thread.java:834)
@mark-vieira mark-vieira added :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP >test-failure Triaged test failures from CI labels Apr 15, 2021
@elasticmachine elasticmachine added the Team:Data Management Meta label for data/management team label Apr 15, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (Team:Core/Features)

@mark-vieira
Copy link
Contributor Author

@probakowski do you mind taking a look at this?

@probakowski
Copy link
Contributor

yes, I'll check it out

@probakowski probakowski self-assigned this Apr 15, 2021
@probakowski probakowski mentioned this issue Apr 15, 2021
Merged
probakowski added a commit that referenced this issue Apr 15, 2021
@probakowski
Remove assertion from DatabaseRegistry (#71764)
2b81d72 
This change removes assertion from DatabaseRegistry - we can easily loose .geoip_databases index with persistent task state still in cluster state, this is not assertion failing, this is usual failure and should be signalled as one.

This also tries to fix packaging tests by avoiding duplicates in elasticsearch.yml.

Closes #71762
probakowski added a commit to probakowski/elasticsearch that referenced this issue Apr 15, 2021
@probakowski
Remove assertion from DatabaseRegistry (elastic#71764)
3c1c79e 
This change removes assertion from DatabaseRegistry - we can easily loose .geoip_databases index with persistent task state still in cluster state, this is not assertion failing, this is usual failure and should be signalled as one.

This also tries to fix packaging tests by avoiding duplicates in elasticsearch.yml.

Closes elastic#71762
probakowski added a commit that referenced this issue Apr 15, 2021
@probakowski
Remove assertion from DatabaseRegistry (#71764) (#71767)
9f458b6 
This change removes assertion from DatabaseRegistry - we can easily loose .geoip_databases index with persistent task state still in cluster state, this is not assertion failing, this is usual failure and should be signalled as one.

This also tries to fix packaging tests by avoiding duplicates in elasticsearch.yml.

Closes #71762
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@probakowski probakowski

Labels
:Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP Team:Data Management Meta label for data/management team >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove assertion from DatabaseRegistry probakowski/elasticsearch
3 participants
@probakowski @mark-vieira @elasticmachine