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

[kitsune] Update enriched fields #1137

Merged
merged 2 commits into from
Mar 11, 2024
Merged

Conversation

jjmerchante
Copy link
Contributor

This PR allows to execute the demography study for Kitsune backend.

The enriched index now includes a unique identifier for questions and answers (id), this was causing an error when trying to autorefresh because uuid was missing.

Include the default raw fields in answers items that were missing: metadata__updated_on , metadata__timestamp, offset, origin, tag, and uuid.

This commit allows to execute the demography study for Kitsune
backend (SUMO).

Signed-off-by: Jose Javier Merchante <jjmerchante@bitergia.com>
This commit updates the Kitsune enriched index. It was not
including a unique identifier in the enriched index for questions
and answers, this was causing an error when trying to autorefresh.

Include also the default raw fields in answers items that were missing:
"metadata__updated_on", "metadata__timestamp","offset",
"origin", "tag", and "uuid".

Signed-off-by: Jose Javier Merchante <jjmerchante@bitergia.com>
@jjmerchante jjmerchante changed the title Kitsune updated enriched fields [kitsune] Update enriched fields Mar 11, 2024
@coveralls
Copy link

coveralls commented Mar 11, 2024

Pull Request Test Coverage Report for Build 8230580222

Details

  • 7 of 7 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.02%) to 85.561%

Totals Coverage Status
Change from base Build 8112037918: 0.02%
Covered Lines: 7472
Relevant Lines: 8733

💛 - Coveralls

Copy link
Member

@sduenas sduenas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sduenas sduenas merged commit 724efeb into chaoss:master Mar 11, 2024
8 checks passed
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.

None yet

3 participants