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

assets on file but not in index will now save to index on submission … #588

Merged
merged 1 commit into from
Oct 30, 2017

Conversation

jsnoble
Copy link
Member

@jsnoble jsnoble commented Oct 11, 2017

resolves #586

@kstaken
Copy link
Member

kstaken commented Oct 30, 2017

Integration tests pass. Manual tests pass.

Manual test was to copy the asset directory, remove the asset from the system, put the asset directory back in place and then redeploy the asset ensuring it used the same asset ID.

Log messages in this situation

{"name":"teraslice","hostname":"kteramac.local","pid":95926,"module":"assets_storage","level":30,"msg":"asset: c7809e8fcde24c0df1610d6bb1c6d717270c98a2 exists on disk but not in index, saving asset to index","time":"2017-10-30T18:43:49.240Z","v":0}
{"name":"teraslice","hostname":"kteramac.local","pid":95926,"module":"assets_storage","level":30,"msg":"assets: ieee_oui, id: c7809e8fcde24c0df1610d6bb1c6d717270c98a2 has been saved to elasticsearch","time":"2017-10-30T18:43:49.326Z","v":0}

@kstaken kstaken merged commit bdbe05a into terascope:master Oct 30, 2017
@jsnoble jsnoble deleted the loading_assets_fix branch November 21, 2017 14:37
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.

Asset not stored in ES
2 participants