Uploaded image for project: 'OpenShift Bugs'
  1. OpenShift Bugs
  2. OCPBUGS-22976

S2I Build Wizard should check for Containerfile in addition to Dockerfile

    XMLWordPrintable

Details

    • Moderate
    • No
    • ODC Sprint 3248
    • 1
    • Rejected
    • False
    • Hide

      None

      Show
      None
    • Hide
      * Previously, the console searched only for the file name `Dockerfile` in a repository to identify the repository suitable for the `Container` strategy in the Import Flows. Since other containerization tools are available, support for the the `Containerfile` file name is now suitable for the `Container` strategy. (link:https://issues.redhat.com/browse/OCPBUGS-22976[*OCPBUGS-22976*])
      Show
      * Previously, the console searched only for the file name `Dockerfile` in a repository to identify the repository suitable for the `Container` strategy in the Import Flows. Since other containerization tools are available, support for the the `Containerfile` file name is now suitable for the `Container` strategy. (link: https://issues.redhat.com/browse/OCPBUGS-22976 [* OCPBUGS-22976 *])
    • Enhancement
    • Done

    Description

      Description of problem:

      A Github project with a Containerfile instead of a Dockerfile is not seen as a Buildah target, and the wizard falls through to templating as a standard (language) project.
      
      

      Version-Release number of selected component (if applicable):

      
      Server Version: 4.13.18
      Kubernetes Version: v1.26.9+c7606e7
      
      

      How reproducible:

      Always
      
      

      Steps to Reproduce:

      1. Create a git application with Containerfile, e.g. https://github.com/cwilkers/jumble-c
      2. Use the Developer view to add the app as a git repo
      3. Observe failure as project is not built properly due to ignoring Containerfile
      

      Actual results:

      Build failure
      

      Expected results:

      Buildah includes Containerfile which includes html and other resources required for app
      

      Additional info:

      https://github.com/cwilkers/jumble-c
      

      Attachments

        Issue Links

          Activity

            People

              avik6028 Avik Kundu
              rh_cwilkers Chandler Wilkerson
              Sanket Pathak Sanket Pathak
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: