Remove organismAgeValue term facet from service response (#2824)

[achave11-ucsc]

#2824

Author

• PR title references issue
• PR title matches issue title (preceded by Fix: for bugs) _{or there is a good reason why they're different}
• Title of main commit references issue
• PR is connected to Zenhub issue and description links to issue

Author (reindex)

• Added r tag to commit title _{or this PR does not require reindexing}
• Added reindex label to PR _{or this PR does not require reindexing}

Author (freebies & chains)

• Freebies are blocked on this PR _{or there are no freebies in this PR}
• Freebies are referenced in commit titles _{or there are no freebies in this PR}
• This PR is blocked by previous PR in the chain _{or this PR is not chained to another PR}
• Added chain label to the blocking PR _{or this PR is not chained to another PR}

Author (upgrading)

• Documented upgrading of deployments in UPGRADING.rst _{or this PR does not require upgrading}
• Added u tag to commit title _{or this PR does not require upgrading}
• Added upgrade label to PR _{or this PR does not require upgrading}
• Added announcement to PR description _{or this PR does not require announcement}

Author (requirements, before every review)

• Ran make requirements_update _{or this PR leaves requirements*.txt, common.mk and Makefile untouched}
• Added R tag to commit title _{or this PR leaves requirements*.txt untouched}
• Added reqs label to PR _{or this PR leaves requirements*.txt untouched}

Author (before every review)

• make integration_test passes in personal deployment _{or this PR does not touch functionality that could break the IT}
• Rebased branch on develop, squashed old fixups

Primary reviewer (after approval)

• Commented in issue about demo expectations _{or labelled issue as no demo}
• Decided if PR can be labeled no sandbox
• PR title is appropriate as title of merge commit
• Moved ticket to Approved column
• Assigned PR to an operator

Operator (before pushing merge the commit)

• Checked reindex label and r commit title tag
• Checked that demo expectations are clear _{or issue is labeled as no demo}
• Rebased and squashed branch
• Sanity-checked history
• Pushed PR branch to Github
• Branch pushed to Gitlab and added sandbox label _{or PR is labeled no sandbox}
• Build passed in sandbox _{or PR is labeled no sandbox}
• Started reindex in sandbox _{or this PR does not require reindexing sandbox}
• Checked for failures in sandbox _{or this PR does not require reindexing sandbox}
• Added PR reference to merge commit title
• Collected commit title tags in merge commit title
• Moved linked issue to Merged column
• Pushed merge commit to Github

Operator (after pushing the merge commit)

• Made announcement requested by author _{or PR description does not contain an announcement}
• Moved freebies to Merged column _{or there are no freebies in this PR}
• Shortened the PR chain _{or this PR is not the base of another PR}
• Verified that N reviews labelling is accurate
• Pushed merge commit to Gitlab _{or this changes can be pushed later, together with another PR}
• Deleted PR branch from Github and Gitlab

Operator (reindex)

• Started reindex in dev _{or this PR does not require reindexing or does not target dev}
• Checked for failures in dev _{or this PR does not require reindexing or does not target dev}
• Started reindex in prod _{or this PR does not require reindexing or does not target prod}
• Checked for failures in prod _{or this PR does not require reindexing or does not target prod}

Operator

• Unassigned PR

[codecov]

Codecov Report

Merging #3437 (bdde3d5) into develop (bdde3d5) will not change coverage.
The diff coverage is n/a.

❗ Current head bdde3d5 differs from pull request most recent head 2bd436d. Consider uploading reports for the commit 2bd436d to get more accurate results

@@           Coverage Diff            @@
##           develop    #3437   +/-   ##
========================================
  Coverage    82.55%   82.55%           
========================================
  Files          123      123           
  Lines        14246    14246           
========================================
  Hits         11761    11761           
  Misses        2485     2485           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bdde3d5...2bd436d. Read the comment docs.

[coveralls]

Coverage remained the same at 82.794% when pulling 2bd436d on issues/achave11/2824-rm-organismAgeValue-facet into bdde3d5 on develop.

[dsotirho-ucsc]

Approved.

[hannes-ucsc]

Looks good but we can't merge until blocker DataBiosphere/data-browser#1947 lands in dev and prod.

[achave11-ucsc]

As per Slack conv. this changes can be merged without breaking DB.

[hannes-ucsc]

Sounds good. In that case, rebase and request review, please.

[nadove-ucsc]

Failed in IT. Trying early_reindex.

[nadove-ucsc]

No reindex failures. Queues are all empty.

[nadove-ucsc]

Retrying IT passed after the early_reindex.

[nadove-ucsc]

IT for merge commit failed in develop as expected. Starting early_reindex.

[nadove-ucsc]

No unexpected failures. 5 failed bundles, all expected due to #2873 and #2870:

40733888-3b9f-500e-84fd-1a18a64aadbe
94565940-7b2f-5fd7-beb1-dc309687fe09
9d0f5cd1-0ef6-5eed-8dd7-f15d36a34185
ae338c4e-650f-545b-8631-3c8d755aa2f9
c12a6ca2-3234-5182-ad0b-f8c2fb881ab5

[nadove-ucsc]

Merge build passed post-reindex 👍