Skip to content

Conversation

@cgwalters
Copy link
Member

I just got this working but I am feeling it's going
to be a dramatic improvement in my development workflow.

To set up, run hack/cluster-push-prep.sh once after doing
an install. Then run hack/cluster-push.sh to deploy a new
MCD. Use env WHAT=machine-config-controller hack/cluster-push.sh
to update the controller.

(Yes, these don't integrate with the Makefile yet...)

@openshift-ci-robot openshift-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Dec 12, 2018
Copy link
Member

@ashcrow ashcrow Dec 12, 2018

Choose a reason for hiding this comment

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

I assume this is to remind people to podman login 😄. NM, later on it occurs and this is a logging line 👍

@cgwalters
Copy link
Member Author

Just to link explicitly, here is a key bit of sorcery required. For me I ran the setcap on /usr/bin/oc since kubectl is a symlink to it.

Copy link
Member

@ashcrow ashcrow left a comment

Choose a reason for hiding this comment

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

Looks good!! Only thing missing is a reference to it's use in one of the in-repo docs.

@cgwalters
Copy link
Member Author

I sort of wonder though whether the real correct thing is to build a local release payload. There's some hints on that in the release payload doc, but I started this from the perpective of "automate the existing HACKING.md".

@ashcrow
Copy link
Member

ashcrow commented Dec 12, 2018

level=fatal msg="Error executing openshift-install: failed to fetch Cluster: failed to generate asset \"Cluster\": failed to create cluster: failed to apply using terraform"

openshift/installer#882 may fix this e2e issue.

/test e2e-aws

@openshift-ci-robot openshift-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Dec 12, 2018
@cgwalters
Copy link
Member Author

OK, I updated the HACKING.md. Can someone give this a try and report how it went?

@ashcrow
Copy link
Member

ashcrow commented Dec 12, 2018

/test e2e-aws

@jlebon
Copy link
Member

jlebon commented Dec 12, 2018

Can someone give this a try and report how it went?

Sure, will give it a try.

@ashcrow
Copy link
Member

ashcrow commented Dec 12, 2018

/test e2e-aws

@cgwalters
Copy link
Member Author

OK updated again with a few bugs fixed; there was a nasty one that took be a bit to track down that the hack/cluster-push-prep.sh had assumed an oc project openshift-machine-config-operator which I had locally. I also integrated with Makefile.

@jlebon
Copy link
Member

jlebon commented Dec 12, 2018

OK, I'm stuck on trying to get a libvirt cluster up: openshift/installer#889.

@cgwalters cgwalters force-pushed the hack-cluster-push branch 3 times, most recently from 390d8ef to fc31f4f Compare December 14, 2018 14:49
@cgwalters
Copy link
Member Author

OK, fixed one more bug. This now WFM from a clean install.

The next bug though I'm hitting is I think there's a race between me patching the images configmap and then the deployment; the deployment seems to get reset back right away. But there's still high latency between the configmap change and the deployment change.

@cgwalters
Copy link
Member Author

/hold

But nope it's deploying an ancient image for some reason; debugging...

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 14, 2018
@cgwalters
Copy link
Member Author

/hold cancel

Well...I dunno how that happened. Seems to work in another try. If this looks useful to people I'd say we merge, we can always do followups.

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 14, 2018
@jlebon
Copy link
Member

jlebon commented Dec 14, 2018

Giving this a try locally!

I just got this working but I am feeling it's going
to be a *dramatic* improvement in my development workflow.

See the updated `HACKING.md` for details.

Closes: openshift#230
@cgwalters
Copy link
Member Author

Updated ⬆️

@jlebon
Copy link
Member

jlebon commented Dec 14, 2018

OK, this works locally for me now. Good stuff!

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Dec 14, 2018
@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cgwalters, jlebon

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 14, 2018
@openshift-merge-robot openshift-merge-robot merged commit 542d610 into openshift:master Dec 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. 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.

5 participants