Sprint 53 Review (2018-07-25)

Availability

(assume Team Availability Calendar is up to date, otherwise, everyone is available for 9 days):

Panos

Ash

Fred

Ify

Reubs

Al

Jay

Matt

Simon

-3

-3

-1

N/A

-1

-2

-1

-5

-3




Sprint Priorities

The following list was the prioritisation output from Sprint Planning - largely a stabilisation Sprint:

  1. BAU. To include:
    • PO Group to refine priorities in the backlogAlistair Pringle (Unlicensed)Joanne Watson (Unlicensed)Former user (Deleted)Ashley Ransoo, This is the #1 priority piece of work for the POs during the Sprint.
    • Jayanta and Hamed, supported by John will form a BAU team this Sprint to focus on Bugs.
      - Multiple FundingType per Posts are duplicating Posts in the post list 
      - Placements created have 'disappeared'
      - Placements is allowing users to select Inactive posts Recording Placements
      - TIS has reverted changes made by users (Ophthalmology)
      - Cannot update site of Post when post is linked to a rotation Record Creation (Post)
      - TIS_NDW ETL error handling is looping and filling logs
      - Changes to rotation are not reflected in ProgrammeMembership Record Programme Memberships
      - Training Descriptions not displayed in Placements Recording Placements
      - Duplicate Main Site / Approved Grade / Primary Specialty created when editing a post with auto-generate NPN set
    • Help devops on the env alignment for the anonymiser
      You may not have all the skills and knowledge to handle all the priority Bugs, but this is a great way of:
      1. getting exposure to the widest degree of the codebase / infrastructure
      2. pair with other members of the team who have the knowledge / skills needed
      3. instigate knowledge transfer sessions when encountering an area of the codebase / infrastructure with which there is unfamiliarity
      4. encouraging liaison with the PO Group in order to confirm business value for each Bug and the extent (speed of turnaround, breadth of solution required, longevity for the solution, etc) needed
  2. Technical debt +
    • Each functional group - FED, BED, Ops, Everyone else (I do not want to discount others in the team), to determine their top 'n' number of high priority tech debt, size it, allocate it, and timebox days in the Sprint to address it. Come to Stand up on Monday morning with that list to socialise what you are doing and when you are doing it, so where there might be knock on impacts on others, they have the right to feed in. Particular attention to supporting Shivani with addressing the test strategy.
  3. Knowledge transfer (assessment of what's left - who could feasibly do this?)
    • Confluence housekeeping - especially deleting documentation that is out of date. Ongoing
    • Dev standards (Dev meeting / series of meetings). Ongoing
    • Pairing / Mobbing
    • Git-commit messages
    • Walk-throughs (prep as you go - links?). 
    • Architectural level diagrams (Reuben leading - update?)
  4. Development (P1s - POs to review what are P1s in Jira as a matter of urgency)
    1. Document management - accessing existing documents (better dataset to work from, but work needed to be done on reviewing metadata before dev work should commence), and creating, updates and deletes of new ones (easier to achieve). Chris / Luis
    2. Placement comments - Update please?
    3. Removal of deleted records - records marked as "Delete" in DR2, but brought over to TIS as "Current"
    4. NDW UAT missing data analysis - Fields that don't exist in TIS that NDW want in TIS. And fields that don't exist in NDW but do exist in TIS.
    5. ESR - Assisting ESR / NDW with live ESR issue - TISDEV-5089 - GMC/GDC issue, TISDEV-5081 - firstnames/middleName issue & TISDEV-5062
    6. New Listing component
  5. Non Dev (PO & BA)
    1. NDW - analysis to finish off - How to delete going forward (Ify)
  6. Ops
    1. restrain the DB / productionising the DB
    2. Deployment pipeline
  7. Testing
    1. E2E automation pipeline
    2. E2E regression pack stablisation
    3. Relese Testing




Contents

The following items to be shared by team members during this sprint review


Item

Owner

Description of work

Link. e.g. to working software
https://apps.tis.nhs.uk/
(note: use Dev for
obfuscated data)
Loom video of completed work
https://www.useloom.com/

Production Application releases / updates


TIS-52


BAU. To include:

  • PO Group to refine priorities in the backlogAlistair Pringle (Unlicensed)Joanne Watson (Unlicensed)Former user (Deleted)Ashley Ransoo
    This is the #1 priority piece of work for the POs during the Sprint.
  • Jayanta and Hamed, supported by John will form a BAU team this Sprint to focus on Bugs.
    TISDEV-5030: Multiple FundingType per Posts are duplicating Posts in the post list -TO DO
    TISDEV-5003: Placements created have 'disappeared' - DONE
    TISDEV-4879: Placements is allowing users to select Inactive posts Recording Placements - IN PROGRESS
    - TISDEV-4961: TIS has reverted changes made by users (Ophthalmology) - DONE
    TISDEV-4855: Cannot update site of Post when post is linked to a rotation Record Creation (Post) - READY FOR RELEASE
    TISDEV-4631: TIS_NDW ETL error handling is looping and filling logs - DONE
    TISDEV-4679: Changes to rotation are not reflected in ProgrammeMembership Record Programme Memberships - NOT REQUIRED
    TISDEV-4351: Training Descriptions not displayed in Placements Recording Placements - NOT REQUIRED
    TISDEV-5051: Duplicate Main Site / Approved Grade / Primary Specialty created when editing a post with auto-generate NPN set - READY FOR RELEASE.
    You may not have all the skills and knowledge to handle all the priority Bugs, but this is a great way of:
    1. getting exposure to the widest degree of the codebase / infrastructure
    2. pair with other members of the team who have the knowledge / skills needed
    3. instigate knowledge transfer sessions when encountering an area of the codebase / infrastructure with which there is unfamiliarity
    4. encouraging liaison with the PO Group in order to confirm business value for each Bug and the extent (speed of turnaround, breadth of solution required, longevity for the solution, etc) needed

Jayanta Saha

Former user (Deleted)

John Simmons (Deactivated)

Oladimeji Onalaja (Unlicensed)


Status of work:

Total 9 tickets were allocated to the BAU team. 5 of them are successfully done by Jayanta Saha, Oladimeji Onalaja (Unlicensed)Chris Mills (Unlicensed) Luís Matos (Unlicensed) and Sunil Rochani (Unlicensed). Help and advice were taken from Paul Hoang (Unlicensed) and John Simmons (Deactivated).

Hamed' s ticket is in progress and ticket 5030 is in 'to do' status. Rest of the 2 tickets are marked 'Not required' during sprint planning meeting.

Some other tickets which are not in this area  but also were done or work in progress is TISDEV-5179, TISDEV - 4794, TISDEV-5068, TISDEV-4864  and TISDEV - 5125.



TISDEV-4855

https://www.useloom.com/share/ae1fd4c2301c4ace99e8d40dfb2c407c


TISDEV-4854

https://www.useloom.com/share/ae3240672b1c49a1b8c1ada1028c47ac

TISDEV-4452

https://www.useloom.com/share/045675ecba234464856d9f87bf1964cb

TISDEV-5190

https://www.useloom.com/share/0b8b36d869934e6eb057c6b62b81737b

Technical debt +

  1. Each functional group - FED, BED, Ops, Everyone else (I do not want to discount others in the team), to determine their top 'n' number of high priority tech debt, size it, allocate it, and timebox days in the Sprint to address it.

All team

Testing (Simon)

Gap analysis for TCS unit test coverage. Story + tasks created for starting to increase coverage in the next sprint

Pact tests POC. Starting to look at TCS and its interactions with other services

Testing (Shivani)

  • Fixed Programme e2e tests in regression test pack.
  • Fixed Post e2e tests in regression test pack.
  • Supported release testing by testing the release stories in staging and raised the defects.
  • Worked on Contract testing strategy for TCS with Simon.Contract Testing Strategy for TCS
  • KT Session with Luis for Document Management different flows. (Upload, Edit, Search)
  • A session with Jag and Simon for ESR-ETL Pact testing implemented by Jag.

Panos: Increased FE unit test coverage (Programme Memberships)

Prometheus/Grafana

End points added for Prometheus in the following services:

Revalidation, Concerns, Connection-Discrepancies

Profile, TCS, Reference, Assessments, Generic Upload

Chris:

  • Docker and compose upgrade
  • Prometheus endpoints
  • Health checks
  • Max log sizes for containers
  • Creating graphs
  • Statusboard
  • PoC status page
  • Standardise hprof/gc files







Grafana live demo (in Dev)

https://dev-monitoring.tis.nhs.uk/


Knowledge transfer

  1. Confluence housekeeping - especially deleting documentation that is out of date. Ongoing
  2. Dev standards (Dev meeting / series of meetings). Ongoing
  3. Pairing / Mobbing
  4. Git-commit messages
  5. Walk-throughs (prep as you go - links?). 
  6. Architectural level diagrams (Reuben leading - update?)


  1. Andy Nash (Unlicensed)Reuben Noot (Deactivated)
  2. All Dev team / Paul leading
  3. All Dev team
  4. All Dev team
  5. All Dev team
  6. Reuben Noot (Deactivated) / All Dev team
  7. Oladimeji Onalaja (Unlicensed) / John G

Ola and John G

  • Knowledge transfer sessions on RxJs (Reactive programming) which is heavily used in the development of generic upload, document manger and new listing module.
  • Walk-through on document manager and new  Listings
  • BE walk through on Document Management and ESR Pact testing


6. Application architecture old

TIS - Dev Tools Landscape

Document Manager (First pass)

https://www.useloom.com/share/e35583768650428dabe33478985f34e5

Development (P1s - POs to review what are P1s in Jira as a matter of urgency)

  1. Document management - accessing existing documents (better dataset to work from, but work needed to be done on reviewing metadata before dev work should commence), and creating, updates and deletes of new ones (easier to achieve). Chris / Luis
  2. Assessments Bulk Upload - Update please?
  3. Placement comments - Update please?
  4. Removal of deleted records - records marked as "Delete" in DR2, but brought over to TIS as "Current"
  5. NDW UAT missing data analysis - Fields that don't exist in TIS that NDW want in TIS. And fields that don't exist in NDW but do exist in TIS.
  6. ESR - Assisting ESR / NDW with live ESR issue - update please?
  7. DR planning - incl. geo-replication, roll back and build again scenarios (John S and Ops) - update please?

Placement comments (Panos)

Comments related work has been completed. We have also added all required tests, but we ll have to improve the e2e test in terms of structure (test does work but it is not merged as per above requirement). 

6. ESR:

Jag:

  • High Priority middle names issue causing duplicates has been fixed (TISDEV-5081) and released (Release: TIS-52.2). (Jag)
  • The above uncovered a break in contract between TCS and ESR-ETL because of rotation data being sent (Unplanned task). This was fixed (TISDEV-5187) and released (Release: TIS-52.3) 
  • ESR interface was switched back on. (TISDEV-5092)
  • Full Notification files for all the local offices have been sent barring WMD which is planned during next week. (TISDEV-5062)
  • Errored applicant records re-processing is in progress (TISDEV-4784)

Fred/Jag:

  • Fixed and enabled jenkins ftp syn cron job in prod getting rid off crontab. 

Ashley/Jag:

  • Some BAU queries were looked into.


Resource Planning