Uploaded image for project: 'SAFe Program'
  1. SAFe Program
  2. SP-2088

Expand Marvin MR Checks and make it proactive in fixing issues

Change Owns to Parent OfsSet start and due date...
    XporterXMLWordPrintable

Details

    • Feature
    • Must have
    • PI13
    • None
    • None
    • Services
    • Hide

      Marvin the Paranoid Android is extended with the changes from SS-82 regarding new templates and guidance and it is more proactive in both giving assistance to the developers in fixing the common issues wherever it can and MR Approval process so that critical issues and divergences are prevented while the overhead of maintenance on developers are reduced

      Show
      Marvin the Paranoid Android is extended with the changes from SS-82 regarding new templates and guidance and it is more proactive in both giving assistance to the developers in fixing the common issues wherever it can and MR Approval process so that critical issues and divergences are prevented while the overhead of maintenance on developers are reduced
      • Marvin Checks are updated to cover the latest SKAO guidelines
      • Marvin can fix issues related to Settings
      • Any outstanding bugs related to Marvin are closed
    • 2
    • 2
    • 0
    • Team_SYSTEM
    • Sprint 3
    • Hide
      • The feature is demoed on DP and OMC ART Sprint #3 Demos
      • The remaning stories that have not been already deployed to production will be deployed in REL-27
      • All the futures in the description are provided
      • There are some new ideas/improvements/suggestions that accumulated in the backlog for future
      Show
      The feature is demoed on DP and OMC ART Sprint #3 Demos The remaning stories that have not been already deployed to production will be deployed in REL-27 All the futures in the description are provided There are some new ideas/improvements/suggestions that accumulated in the backlog for future
    • 13.6
    • Stories Completed, Integrated, Outcomes Reviewed, Demonstrated, Satisfies Acceptance Criteria, Accepted by FO
    • PI23 - UNCOVERED

    Description

      The Marvin is updated so that it can better support the CI/CD and SKAO guidelines:

      Updates:

      • Marvin performs auto fixes (TBD) where it finds thing it can fix
      • Marvin approves the MR if there's no critical issues
      • Critical issues are reviewed with the above item in mind.

      New Checks:

      • Marvin checks for new templates and makefiles
      • Marvin checks for container scanning results (TBD)
      • Marvin checks repo name for ADR-24 compliance
      • Marvin checks for test coverage?

      Bug Fixes:

      • Marvin bug fixing and technical debt (any open bugs are closed)
      • Delete CI_REPOSITORY_URL from manifest check (and integration to manifest) and other variables that have access tokens enabled!
      • Marvin shouldn't check for docker-compose in documentation(changelog, rtd etc.)

       

      There is also an issue about Marvin figuring out which checks to perform as some checks are not applicable to all MRs such as docs update, small fixes etc.

      Attachments

        Issue Links

          Structure

            Activity

              People

                P.Harding Harding, Piers
                U.Yilmaz Yilmaz, Ugur
                Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Feature Progress

                  Story Point Burn-up: (100.00%)

                  Feature Estimate: 2.0

                  IssuesStory Points
                  To Do00.0
                  In Progress   00.0
                  Complete1413.0
                  Total1413.0

                  Dates

                    Created:
                    Updated:
                    Resolved:

                    Structure Helper Panel