raft-tech / TANF-app

Repo for development of a new TANF Data Reporting System
Other
16 stars 3 forks source link

Kibana Dashboard MVP #2993

Open elipe17 opened 1 month ago

elipe17 commented 1 month ago

Summary of Changes

Note: I did not create dashboards/visualizations for every single table in the linked document. If that is what we want, I can do so. Just didn't want to do too much.

How to Test

cd tdrs-frontend && docker-compose up --build
cd tdrs-backend && docker-compose up --build
  1. Open http://localhost:3000/ and sign in as a Sys admin.
  2. In an incognito window: Open http://localhost:3000/ and sign in as a data analyst with a different STT.
  3. For Both STTs submit TANF S1 and S3 data for FY 2021 Quarter 1. For Section 1, use ADS.E2J.FTP1.TS06 and for Section 3, use the attached file below. Note, this file is a modified version of the original: ADS.E2J.FTP3.TS06. ADS.E2J.FTP3.TS06.txt
  4. [Local only] Make sure to run source commands.sh and then execute the new command: upload_kibana_objs
  5. As the sys admin user, navigate to Kibana->Stack Management->Saved Objects and verify you see have several new saved objects.
  6. From there, navigate through the Visualizations and Dashboards. Make sure your time range includes the datetime that you submitted your S1/S3 files or there will be no data to see.

Deliverables

More details on how deliverables herein are assessed included here.

Deliverable 1: Accepted Features

Checklist of ACs:

Deliverable 2: Tested Code

Deliverable 3: Properly Styled Code

Deliverable 4: Accessible

Deliverable 5: Deployed

Deliverable 6: Documented

Deliverable 7: Secure

Deliverable 8: User Research

Research product(s) clearly articulate(s):

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.99%. Comparing base (b5f8bd7) to head (599f628).

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/raft-tech/TANF-app/pull/2993/graphs/tree.svg?width=650&height=150&src=pr&token=BA04YXPAL9&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=raft-tech)](https://app.codecov.io/gh/raft-tech/TANF-app/pull/2993?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=raft-tech) ```diff @@ Coverage Diff @@ ## develop #2993 +/- ## =========================================== + Coverage 92.97% 92.99% +0.01% =========================================== Files 276 277 +1 Lines 7316 7320 +4 Branches 646 646 =========================================== + Hits 6802 6807 +5 Misses 413 413 + Partials 101 100 -1 ``` | [Flag](https://app.codecov.io/gh/raft-tech/TANF-app/pull/2993/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=raft-tech) | Coverage Δ | | |---|---|---| | [dev-backend](https://app.codecov.io/gh/raft-tech/TANF-app/pull/2993/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=raft-tech) | `93.03% <100.00%> (+<0.01%)` | :arrow_up: | | [dev-frontend](https://app.codecov.io/gh/raft-tech/TANF-app/pull/2993/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=raft-tech) | `92.70% <ø> (+0.09%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=raft-tech#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files](https://app.codecov.io/gh/raft-tech/TANF-app/pull/2993?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=raft-tech) | Coverage Δ | | |---|---|---| | [tdrs-backend/tdpservice/parsers/transforms.py](https://app.codecov.io/gh/raft-tech/TANF-app/pull/2993?src=pr&el=tree&filepath=tdrs-backend%2Ftdpservice%2Fparsers%2Ftransforms.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=raft-tech#diff-dGRycy1iYWNrZW5kL3RkcHNlcnZpY2UvcGFyc2Vycy90cmFuc2Zvcm1zLnB5) | `100.00% <100.00%> (ø)` | | | [...ations/0029\_alter\_tribal\_tanf\_t4\_closure\_reason.py](https://app.codecov.io/gh/raft-tech/TANF-app/pull/2993?src=pr&el=tree&filepath=tdrs-backend%2Ftdpservice%2Fsearch_indexes%2Fmigrations%2F0029_alter_tribal_tanf_t4_closure_reason.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=raft-tech#diff-dGRycy1iYWNrZW5kL3RkcHNlcnZpY2Uvc2VhcmNoX2luZGV4ZXMvbWlncmF0aW9ucy8wMDI5X2FsdGVyX3RyaWJhbF90YW5mX3Q0X2Nsb3N1cmVfcmVhc29uLnB5) | `100.00% <100.00%> (ø)` | | | [...backend/tdpservice/search\_indexes/models/tribal.py](https://app.codecov.io/gh/raft-tech/TANF-app/pull/2993?src=pr&el=tree&filepath=tdrs-backend%2Ftdpservice%2Fsearch_indexes%2Fmodels%2Ftribal.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=raft-tech#diff-dGRycy1iYWNrZW5kL3RkcHNlcnZpY2Uvc2VhcmNoX2luZGV4ZXMvbW9kZWxzL3RyaWJhbC5weQ==) | `100.00% <100.00%> (ø)` | | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/raft-tech/TANF-app/pull/2993/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=raft-tech) ------ [Continue to review full report in Codecov by Sentry](https://app.codecov.io/gh/raft-tech/TANF-app/pull/2993?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=raft-tech). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=raft-tech) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://app.codecov.io/gh/raft-tech/TANF-app/pull/2993?dropdown=coverage&src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=raft-tech). Last update [b5f8bd7...599f628](https://app.codecov.io/gh/raft-tech/TANF-app/pull/2993?dropdown=coverage&src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=raft-tech). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=raft-tech).
robgendron commented 1 month ago

3 points remaining.

robgendron commented 1 month ago

Finishing creation of dashboards. Aiming to publish PR today - 2 point remaining.

elipe17 commented 1 month ago

Note, we can't deploy this again unless we update the elastic instance to 7.10.x manually. I did that once, but it seems other deployments reverted it back and the terraform refuses to update it in the pipeline.

raftmsohani commented 1 month ago

Note, we can't deploy this again unless we update the elastic instance to 7.10.x manually. I did that once, but it seems other deployments reverted it back and the terraform refuses to update it in the pipeline.

@elipe17 You might have to add the version as a variable to terraform

elipe17 commented 1 month ago

Note, we can't deploy this again unless we update the elastic instance to 7.10.x manually. I did that once, but it seems other deployments reverted it back and the terraform refuses to update it in the pipeline.

@elipe17 You might have to add the version as a variable to terraform

@raftmsohani the terraform does specify the version, but it bombs out when it tries to update elastic in the pipeline. I could try deleting the elastic instance and see if it recreates it all together. But that would kill the qasp environment too for a little while.

elipe17 commented 1 month ago

Adding this reference to the PR for an annoying feature of Date based aggregations. I also added it in the comments of the scripted field I created for RPT_MONTH_YEAR.