Uploaded image for project: 'VOLTHA'
  1. VOLTHA
  2. VOL-5091

repo:voltha-northbound-bbf-adapter edits

    XMLWordPrintable

    Details

    • Type: Task
    • Status: To Do (View Workflow)
    • Priority: Medium
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Unknown
    • Labels:
      None
    • Story Points:
      0

      Description

      Misc edits needed to support consistent behavior across repositories:

      1. Remove vendor/ directory from revision control:
        1. augment to retrieve and install based on dependencies (existence)
        2. vendor/ existence in VCS contributes to a variety of problems.
        3. building locally, target mod-update.  mod-update will remove the vendor directory as step one.  If mod-update (~tidy) fails the sandbox is left in an unbuildable state due to git status showing vendor/ has been deleted (manual fix: git checkout vendor/).
        4. This problem disappears when vendor is removed.
        5. Makefile bootstrap logic should retrieve and install the vendor/ directory when it does not exist.
      2. update copyright notice in source to std notice (may be done already)
      3. run gofmt -s -w on all golang sources
      4. Copy in library makefiles/ logic from onf-make
        1. https://github.com/opencord/onf-make
        2. inlined docker logic contains a problem that inhibits jenkins logging container output during a job run.
        3. targets covered: lint, lint-dockerfile, lint-golang,
        4. targets pending/may already have been added in onf-make:
          1. lint-mod
          2. sca
          3. mod-update
          4. ...

       

      Oh also several repositories share this identical problem.

      Dependency logic should be library based/reusable not anchored to a single repository.

        Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

            Assignee:
            Unassigned
            Reporter:
            joey Joey Armstrong
            Watchers:
            1 Start watching this issue

              Dates

              Created:
              Updated:

                Gerrit Reviews

                There are no open Gerrit changes