Uploaded image for project: 'Network Edge'
  1. Network Edge
  2. NE-554

[Tech Debt] [Maint] Remove go-bindata dep from Ingress and DNS operators

XMLWordPrintable

    • Icon: Epic Epic
    • Resolution: Done
    • Icon: Minor Minor
    • openshift-4.14
    • None
    • None
    • [Maint] Remove go-bindata dep from Ingress and DNS operators
    • False
    • False
    • Done
    • OCPPLAN-7878 - NetEdge - Maintainability and Debugability & Tech Backlog
    • OCPPLAN-7878NetEdge - Maintainability and Debugability & Tech Backlog
    • 0% To Do, 0% In Progress, 100% Done
    • N/A
    • Release Note Not Required
    • 0
    • 0

      Go 1.16 added the new embed directive to go. This embed directive lets you natively (and trivially) compile your binary with static asset files.

      The current go-bindata dependency that's used in both the Ingress and DNS operator's for yaml asset compilation could be dropped in exchange for the new go embed functionality. This would reduce our dependency count, remove the need for `bindata.go` (which is version controlled and constantly updated), and make our code easier to read. This switch would also reduce the overall lines of code in our repos.

      Note that this may be applicable to OCP 4.8 if and when images are built with go 1.16.

            mmasters1@redhat.com Miciah Masters
            sgreene@redhat.com Stephen Greene (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: