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

workorder search updated to only pull complete WO from ES #58

Merged
merged 2 commits into from Jul 27, 2017

Conversation

Projects
None yet
2 participants
@okornev
Contributor

okornev commented Jul 24, 2017

to avoid pulling WO without offerings in some cases.

Added two more fixes:

  • additional search for workorder in existing CIs is workorder is missing from wo index, because we only keep last 6 months of weekly indexes.
  • in case of workorder action "delete" fetch and store offerings from last workorder to assist with cost calculations (new WorkorderMessageProcessor added)

okornev added some commits Jul 24, 2017

- search for workorder in existing CIs is workorder is missing from w…
…o index, because we only keep last 6 months of weeklies.

- in case of workorder action "delete" fetch and store offerings from last workorder to assist with cost calculations (new WorkorderMessageProcessor added)
@lkhusid

This comment has been minimized.

Show comment
Hide comment
@lkhusid

lkhusid Jul 27, 2017

Contributor

Looks good. As we already discussed the most important thing it will take care of the problem of missing work order in Cis in cms-all index as well as missing offerings and "phantom" deletes.

Contributor

lkhusid commented Jul 27, 2017

Looks good. As we already discussed the most important thing it will take care of the problem of missing work order in Cis in cms-all index as well as missing offerings and "phantom" deletes.

@lkhusid

This comment has been minimized.

Show comment
Hide comment
@lkhusid

lkhusid Jul 27, 2017

Contributor

*correction: "phantom" CIs due to missed delete action processing

Contributor

lkhusid commented Jul 27, 2017

*correction: "phantom" CIs due to missed delete action processing

@lkhusid lkhusid merged commit 6b7b84c into oneops:master Jul 27, 2017

lkhusid pushed a commit that referenced this pull request Feb 26, 2018

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