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

[Bug]: Logs Not available in test-report #4335

Closed
derek-ho opened this issue Jan 10, 2024 · 5 comments
Closed

[Bug]: Logs Not available in test-report #4335

derek-ho opened this issue Jan 10, 2024 · 5 comments
Labels
bug Something isn't working

Comments

@derek-ho
Copy link
Contributor

Describe the bug

Coming from @peternied , logs from failed autocuts do not contain actual links or logs, instead showing as "Not Available"

To reproduce

Download test report from autocut, example: https://ci.opensearch.org/ci/dbc/integ-test/2.12.0/9167/linux/x64/tar/test-results/6950/integ-test/test-report.yml](https://ci.opensearch.org/ci/dbc/integ-test/2.12.0/9167/linux/x64/tar/test-results/6950/integ-test/test-report.yml), you will see "Not available", instead of a link to the logs or failures

Expected behavior

No response

Screenshots

If applicable, add screenshots to help explain your problem.

Host / Environment

No response

Additional context

No response

Relevant log output

No response

@derek-ho derek-ho added bug Something isn't working untriaged Issues that have not yet been triaged labels Jan 10, 2024
@gaiksaya
Copy link
Member

@gaiksaya gaiksaya removed the untriaged Issues that have not yet been triaged label Jan 10, 2024
@rishabh6788
Copy link
Collaborator

rishabh6788 commented Jan 17, 2024

The reason for status showing N/A is that OpenSearch process failed to come up. We are now uploading the OpenSearch logs to S3 bucket for users to be able to download and see why OS process failed to come up. For the e.g. below is the log for test status not showing:

Exception in thread "main" SettingsException[Failed to load settings from [opensearch.yml]]; nested: JsonParseException[Duplicate field 'node.name'
 at [Source: (sun.nio.ch.ChannelInputStream); line: 166, column: 10]];
	at org.opensearch.common.settings.Settings$Builder.loadFromStream(Settings.java:1146)
	at org.opensearch.common.settings.Settings$Builder.loadFromPath(Settings.java:1117)
	at org.opensearch.node.InternalSettingsPreparer.prepareEnvironment(InternalSettingsPreparer.java:96)
	at org.opensearch.cli.EnvironmentAwareCommand.createEnv(EnvironmentAwareCommand.java:118)
	at org.opensearch.cli.EnvironmentAwareCommand.createEnv(EnvironmentAwareCommand.java:109)
	at org.opensearch.cli.EnvironmentAwareCommand.execute(EnvironmentAwareCommand.java:104)
	at org.opensearch.cli.Command.mainWithoutErrorHandling(Command.java:138)
	at org.opensearch.cli.MultiCommand.execute(MultiCommand.java:104)
	at org.opensearch.cli.Command.mainWithoutErrorHandling(Command.java:138)
	at org.opensearch.cli.Command.main(Command.java:101)
	at org.opensearch.common.settings.KeyStoreCli.main(KeyStoreCli.java:56)
Caused by: com.fasterxml.jackson.core.JsonParseException: Duplicate field 'node.name'
 at [Source: (sun.nio.ch.ChannelInputStream); line: 166, column: 10]
	at com.fasterxml.jackson.core.json.JsonReadContext._checkDup(JsonReadContext.java:250)
	at com.fasterxml.jackson.core.json.JsonReadContext.setCurrentName(JsonReadContext.java:244)
	at com.fasterxml.jackson.dataformat.yaml.YAMLParser.nextToken(YAMLParser.java:482)
	at org.opensearch.common.xcontent.json.JsonXContentParser.nextToken(JsonXContentParser.java:66)
	at org.opensearch.common.settings.Settings.fromXContent(Settings.java:660)
	at org.opensearch.common.settings.Settings.fromXContent(Settings.java:629)
	at org.opensearch.common.settings.Settings$Builder.loadFromStream(Settings.java:1142)
	... 10 more
WARNING: A terminally deprecated method in java.lang.System has been called
WARNING: System::setSecurityManager has been called by org.opensearch.bootstrap.OpenSearch (file:/tmp/tmpx9mwdyf7/1/local-test-cluster/opensearch-2.12.0/lib/opensearch-2.12.0.jar)
WARNING: Please consider reporting this to the maintainers of org.opensearch.bootstrap.OpenSearch
WARNING: System::setSecurityManager will be removed in a future release
Exception in thread "main" SettingsException[Failed to load settings from [opensearch.yml]]; nested: JsonParseException[Duplicate field 'node.name'
 at [Source: (sun.nio.ch.ChannelInputStream); line: 166, column: 10]];
	at org.opensearch.common.settings.Settings$Builder.loadFromStream(Settings.java:1146)
	at org.opensearch.common.settings.Settings$Builder.loadFromPath(Settings.java:1117)
	at org.opensearch.node.InternalSettingsPreparer.prepareEnvironment(InternalSettingsPreparer.java:96)
	at org.opensearch.cli.EnvironmentAwareCommand.createEnv(EnvironmentAwareCommand.java:118)
	at org.opensearch.cli.EnvironmentAwareCommand.createEnv(EnvironmentAwareCommand.java:109)
	at org.opensearch.cli.EnvironmentAwareCommand.execute(EnvironmentAwareCommand.java:104)
	at org.opensearch.cli.Command.mainWithoutErrorHandling(Command.java:138)
	at org.opensearch.cli.Command.main(Command.java:101)
	at org.opensearch.bootstrap.OpenSearch.main(OpenSearch.java:137)
	at org.opensearch.bootstrap.OpenSearch.main(OpenSearch.java:103)
Caused by: com.fasterxml.jackson.core.JsonParseException: Duplicate field 'node.name'
 at [Source: (sun.nio.ch.ChannelInputStream); line: 166, column: 10]
	at com.fasterxml.jackson.core.json.JsonReadContext._checkDup(JsonReadContext.java:250)
	at com.fasterxml.jackson.core.json.JsonReadContext.setCurrentName(JsonReadContext.java:244)
	at com.fasterxml.jackson.dataformat.yaml.YAMLParser.nextToken(YAMLParser.java:482)
	at org.opensearch.common.xcontent.json.JsonXContentParser.nextToken(JsonXContentParser.java:66)
	at org.opensearch.common.settings.Settings.fromXContent(Settings.java:660)
	at org.opensearch.common.settings.Settings.fromXContent(Settings.java:629)
	at org.opensearch.common.settings.Settings$Builder.loadFromStream(Settings.java:1142)
	... 9 more

The error logs can be downloaded from https://ci.opensearch.org/ci/dbc/integ-test/2.12.0/9167/linux/x64/tar/test-results/6950/integ-test/security/with-security/local-cluster-logs/id-0/stderr.txt

@gaiksaya Can we add this link in the documentation for users to be able to see the cluster logs.

@peternied
Copy link
Member

@rishabh6788 Can we fix the manifest to show those files even if the server failed to come up? Its difficult to infer the paths and associated files that are available.

@rishabh6788
Copy link
Collaborator

@rishabh6788 Can we fix the manifest to show those files even if the server failed to come up? Its difficult to infer the paths and associated files that are available.

It should be doable. Will look into it further.

@rishabh6788
Copy link
Collaborator

The PR to add local cluster logs to test report has been merged. You should be able to see the stdout and stderr log files in the newly generated test-report.yml files.
Closing this issue, feel free to reopen if you have any other concerns.
@peternied @derek-ho

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants