Sprint 57 Review (2018-09-19)

Owned by Andy Nash (Unlicensed)
Last updated: 01 Jul 2019
4 min read

Availability

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

Joanne

Alistair

Roope

Panos

Paul

Matt

-2

-2

-2

-3

-1

-2

Sprint Priorities

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

  1. Bulk upload assessments
  2. Programme Membership investigation
  3. AKS and Pipeline
  4. Stabilise Reval and addressing some issues
  5. Trust UI - User management


Contents

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

Item

TIS Ref / Owner

Status

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

Simon Meredith (Unlicensed)

Oladimeji Onalaja (Unlicensed)

Jayanta Saha

John Simmons (Deactivated)

(and anyone else involved)










key summary type assignee status business value story points (effort) priority (bv*sp)
Refresh






Technical debt & Non-functional requirements:

Admins UI security vulnerabilities in dependencies











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
  1. Andy Nash (Unlicensed)



  2. All Dev team / Paul Hoang (Unlicensed) leading
  3. All Dev team
  4. All Dev team
  5. All Dev team

  6. All Dev team
OngoingMaking some progress - but long way still to go...





Resource Planning:

Andy Nash (Unlicensed)

Eniola Agbeti (Unlicensed)


TIS Team #2 recruitment well under way.
Scrum Master and Lead Dev roles being shortlisted for interview yesterday / today.

This Sprint we say goodbye to Sunil and Fred.
Our contractor budget is reducing rapidly now we're recruiting TIS Team #2.
Paul, Panos and Roope remain with us.



Issue keyIssue TypeSummaryStatusDescription of WorkLinks (e.g. Loom)Assignee
TISNEW-1904StorySpike - Post funding bulk uploadDoneTo do: Script the post funding details in bulk
ashley.ransoo
TISNEW-1891StoryAutomatic bumping of package.json versioningDone

Chris.Mills
TISNEW-1939StoryResolve monitoring errors from #dev-monitoring and #monitoring, and ensure good monitoring of downed/failed servicesDone

Chris.Mills
TISNEW-192StoryESR - Produce ESR requested DNF Files for all Local Offices 05/07/2018Done

Run tests in stage

Run the jenkins parameterized job after the last scheduled one

Still no feedback from ESR


fred.randriamora
TISNEW-1943StoryInvestigate splitting Programme Memberships into its own microserviceDoneGoing for PM as a separate microservice would n't be a feasible idea, as all of us agreed. So we need to refactor the code as much as possible.
jayanta.saha
TISNEW-193StoryRemove records with the status of 'delete' on Intrepid from TISDone

jayanta.saha
TISNEW-180BugPost > List view > National Post Number not displaying/sorting correctly in TIS list pageDone

jayanta.saha
TISNEW-1961StorySet new site to CurrentDoneStatus of specified site(s) set to 'CURRENT'
Oladimeji.Onalaja
TISNEW-1928StoryActivate sitesDoneStatus of specified site(s) set to 'CURRENT'
Oladimeji.Onalaja
TISNEW-1487StoryUnit tests for PostsDone

Unit tests have been written for Post component


Oladimeji.Onalaja
TISNEW-1859BugPost autogen throwing an errorDone

Poor data being sent to sutogen (missing site code for example)

Data fixed

Need to handle errors better...


paul.hoang
TISNEW-174BugAssessment programme information not saving correctlyDone
  1. Event - Endpoint used to define 'Programme name' changed  from programmeMembership to programme
  2. Pre-assessment -  Membership Type field dropdown amended to be populated based on membership types associated with trainees programme membership, Curriculum start and end dates auto populated based on selected curriculum, on changing the programme in 'Event', curriculum, curriculum start date, end date fields gets auto-set to null
  3. Unit tests updated

https://www.useloom.com/share/2de8f50acea44e3e91d43ef7666f5a2f

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

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


paul.hoang

Oladimeji.Onalaja

TISNEW-1861BugCannot remove data from 'old post' or 'new post' fieldDone

roope.hakulinen
TISNEW-2016StoryCreate Smoke test pack with required configuration to run against the envDone

Run against Stage and Prod


shivani.rana
TISNEW-1866StoryScripting of adding c.1,000 usersDoneScript created and ready to go
simon.meredith1
TISNEW-1972StoryGet data into dev reval - run mock gmc sync and reval etl in devDoneData updated in Dev
sunil.rochani
TISNEW-113StoryUser/Application Database access restrictionReady for release

Create policies

Assign policy to users

move the credentials from replication to this code

Test accesses


fred.randriamora
TISNEW-376StoryComments need to be surfaced in the NDWReady for releaseThis is under test now, as Data team just confirmed yesterday.
jayanta.saha
TISNEW-1204StoryImplement Elasticsearch best practicesDoneApplies some best practices to Elasticsearch and pre configured some of the curator configuration for when we upgrade ES to 5 then 6
john.simmons
TISNEW-2042StoryServer audit logs not being backed upDoneUpgraded the inventory as the audit category had been added to the elasticsearch server and not the apps server. Changed the ansible playbook to handle the fact that there are now multiple apps servers so the files don't get overwritten. Removed some of the ansible command module entries and replaced them with the dedicated module for that process. 
john.simmons
TISNEW-1826StoryAmend current Trust access roleReady for release

E2E test added to cover newly created HEE Trust Observer role


Oladimeji.Onalaja
TISNEW-182BugCall to TCS for programme membership fails when viewing an assessmentReady for release

Whenever a trainees pre-assessment information gets loaded on the UI, a call is made to TCS to get their associated programme memberships.

The specified error arises when we have bad data and any of the programme membership ids of the trainee is 'null'. We have mitigated this and accounted for situations where bad data exists.


Oladimeji.Onalaja
TISNEW-194StoryBulk Upload/Create Assessment eventsReady for release
  1. Added a 'Bulk upload' L2 navigation to the Assessment L1 item.
  2. Added the 'TIS Assessment Import Template.xlsx' file to the TIS asset storage in Azure.
  3. Enabled the download of assessments template
https://www.useloom.com/share/d76b9a691fcb4ca8986ddffec9c726aaOladimeji.Onalaja
TISNEW-1923BugRevalidation known concerns incorrectly displaying values in Assessment recordsReady for release"No" was the default. Now allows blanks

TISNEW-1867BugProgramme membership info not showing correctlyReady for release

panos.paralakis
TISNEW-1981StoryMigrate 'Search & Filters Prototype' to a dev/testing environmentReady for releaseNew listing module
roope.hakulinen
TISNEW-1975StoryFE Unit tests for Bulk uploadReady for release

roope.hakulinen
TISNEW-2017StoryFix hee-users end point in profile serviceReady for releaseThis wasn't being used. Now fixed, added tests and in use by UM service.
simon.meredith1
TISNEW-1593Bug+/- is not searchable within the placement creation fucntion.Ready for release

sunil.rochani
TISNEW-439BugUnable to search for any data (i.e. sites, posts etc) containing hyphen / underscoreReady for release

sunil.rochani
TISNEW-1279StoryCreate Read only role for trust usersReady for releaseFrom last sprint - mainly testing in this sprint
paul.hoang
TISNEW-1595StoryAKS Service on AzureIn Progress

Chris.Mills
TISNEW-1870Storyadd required configuration to trigger soak tests via jenkins pipeline (devops)In Progress

Shivani Rana (Unlicensed) Worked with Fred to provide inputs and help so that soak tests can be added to the pipeline.

FRED: Created the docker image with jemeter setup. Created the ansible code to deploy it and the jenkins job which runs the ansible code. Tested in sandbox and dev

https://build.tis.nhs.uk/jenkins/job/tis-soaktest/fred.randriamora
TISNEW-135StoryStabilise RevalDone

roope.hakulinen
TISNEW-1933StorySupport Release TestingDone

shivani.rana
TISNEW-1962StoryUser Management - Create "read journey" for current usersReady for release

https://dev-apps.tis.nhs.uk/allUsers


simon.meredith1
TISNEW-196StoryBulk Update Placements using TIS_Placement_IDsIn Progress
  1. Added the 'TIS Placements Update Template.xlsx' file to the TIS asset storage in Azure.
  2. Enabled the download of placements template
https://www.useloom.com/share/623a38559e204e47b4f43d7a8ed794fc

sunil.rochani

Oladimeji.Onalaja


TISNEW-1897BugGMC sync failure investigation (incl RCA) - develop actions to mitigate future failureTo DoJVM arguments, Docker memory and custom query needs to be considered here. 
jayanta.saha
TISNEW-1305StoryAdd "missing" vwAssessments fields to NDW ETLIn ProgressMapping query has been written and tested locally.
jayanta.saha
TISNEW-2038BugFix the HATEOAS pagination of posts on backendReady for Release

paul.hoang



Slack: https://hee-nhs-tis.slack.com/
Jira issues: https://hee-tis.atlassian.net/issues/?filter=14213