Skip to content
This repository was archived by the owner on Mar 16, 2021. It is now read-only.

updated the binary name in Dockerfile #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tparikh
Copy link

@tparikh tparikh commented Nov 6, 2020

Signed-off-by: Tejas Parikh tejas@tejasparikh.com

Signed-off-by: Tejas Parikh <tejas@tejasparikh.com>
@tparikh
Copy link
Author

tparikh commented Nov 6, 2020

@wlan0 @brahmaroutu PTAL

@brahmaroutu
Copy link
Contributor

@tparikh we also need to move the code base under cmd/controller-manager for the build tool to be happy.

@tparikh
Copy link
Author

tparikh commented Nov 8, 2020

@brahmaroutu Can you please elaborate on your comment. I am able to build the binary with the current code structure.

@NicolasT
Copy link

NicolasT commented Dec 7, 2020

Given this project is called cosi-controller-manager I'm a bit surprised by this change 😃 What does it bring?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants