Skip to content
This repository has been archived by the owner on Oct 12, 2023. It is now read-only.

Improve and document the developer deployment process. #37

Merged
merged 1 commit into from Sep 14, 2020

Conversation

dgoodwin
Copy link
Contributor

@dgoodwin dgoodwin commented Sep 4, 2020

Rather than require developers to change on disk files in the git index,
or create custom patching solutions, introduce a make target that allows
us to build, push, and deploy manifests in a single command.

Adds namespace to the ServiceAccount subject in the ClusterRoleBinding,
as far as I can tell this was invalid yaml.

Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
manifests/cluster-rbac/rbac.yaml Outdated Show resolved Hide resolved
Rather than require developers to change on disk files in the git index,
or create custom patching solutions, introduce a make target that allows
us to build, push, and deploy manifests in a single command.
@dgoodwin
Copy link
Contributor Author

dgoodwin commented Sep 4, 2020

PR updated, thanks!

Copy link
Contributor

@mgoodness mgoodness left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@xianlubird xianlubird merged commit a705945 into argoproj:master Sep 14, 2020
mgoodness pushed a commit to mgoodness/applicationset that referenced this pull request Sep 14, 2020
Rather than require developers to change on disk files in the git index,
or create custom patching solutions, introduce a make target that allows
us to build, push, and deploy manifests in a single command.
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.

None yet

3 participants