openshift / openshift/oadp-operator
remove `build/` folder
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 92
- Forks
- 93
- Avg merge
- 1d 23h
- Merged PRs (30d)
- 43
Description
CI files use Dockerfiles in build/ folders (ci-Dockerfile and Dockerfile.bundle).
build/Dockerfile.bundle is a duplication of bundle.Dockerfile in project root. Along with removing it, also remove logic from Makefile
https://github.com/openshift/oadp-operator/blob/b7ad0e47d01bb4759fba5b6895924f5d4989a3d0/Makefile#L314-L316
for standardization, remove stable from channels in oadp-1.1 branch?
https://github.com/openshift/oadp-operator/blob/9e615af37bace792eee315a2934de1e0e56452a5/bundle.Dockerfile#L8
build/ci-Dockerfile should be renamed to ci.Dockerfile (so IDEs and GitHub recognize it as a Dockerfile). As a suggestion, move it to hack/ folder.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by inspecting build/Dockerfile.bundle, build/ci-Dockerfile, bundle.Dockerfile, and the referenced Makefile lines, then search CI files for references to the build/ paths. Confirm the intended location and filename for ci-Dockerfile and whether the oadp-1.1 stable-channel change is in scope; done means obsolete files and references are removed and CI/build checks still pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- docker
- Domain
- build-system, ci-cd
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100