Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Few fixes to the local deploy templates #568

Merged
merged 1 commit into from Nov 27, 2017

Conversation

rthallisey
Copy link
Contributor

Describe what this PR does and why we need it:
A bunch of fixes when running locally.

Changes proposed in this pull request

  • Deleting the asb service causes issues. The service-ca (asb-tls) gets deleting causing later prep_local_dev_env runs to fail and the catalog fails to connect to the broker.
  • Use the proper path when deleting ci resources
  • Use a deployment config for asb-etcd
  • Point the asb-etcd resources at the etcd app and service
  • Remove the BROKER_PORT variable since it was causing an issue with template rendering and isn't needed

Does this PR depend on another PR (Use this to track when PRs should be merged)
depends-on

Which issue this PR fixes (This will close that issue when PR gets merged)
fixes <issue_number>

@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Nov 21, 2017
@coveralls
Copy link

coveralls commented Nov 21, 2017

Coverage Status

Changes Unknown when pulling b243fa2 on rthallisey:fix-local-deploy into ** on openshift:master**.

Copy link
Contributor

@shawn-hurley shawn-hurley left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@djzager djzager left a comment

Choose a reason for hiding this comment

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

LGTM

@rthallisey
Copy link
Contributor Author

CI is down right now. Going to merge without a green CI pass since it doesn't use this code path because it's only for running the broker locally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants