Skip to content
This repository was archived by the owner on Feb 5, 2020. It is now read-only.

Conversation

@squeed
Copy link

@squeed squeed commented Feb 8, 2018

This moves the ingress controller to its own namespace.

@coreosbot
Copy link

Can one of the admins verify this patch?

@enxebre
Copy link
Contributor

enxebre commented Feb 9, 2018

hey @squeed can you terraform fmt

@cpanato
Copy link
Contributor

cpanato commented Feb 9, 2018

this is related to this? #2921

@squeed squeed force-pushed the ingress-controller-operator branch from 21408e5 to 759a6a9 Compare February 9, 2018 12:52
@squeed
Copy link
Author

squeed commented Feb 9, 2018

Updated, PTAL. This replaces #2921

@cpanato
Copy link
Contributor

cpanato commented Feb 9, 2018

retest this please. aws

@cpanato
Copy link
Contributor

cpanato commented Feb 9, 2018

retest this please. removed the wrong label :p

@squeed squeed force-pushed the ingress-controller-operator branch 2 times, most recently from a907ce3 to 26a6c53 Compare February 16, 2018 11:05
@squeed
Copy link
Author

squeed commented Feb 16, 2018

Updated the PR to bump the utility operator as well.

@squeed squeed force-pushed the ingress-controller-operator branch from 26a6c53 to 584f47a Compare February 19, 2018 11:16
@squeed
Copy link
Author

squeed commented Feb 19, 2018

retest this please. aws

@squeed
Copy link
Author

squeed commented Feb 20, 2018

Just need to merge coreos-inc/tectonic-operators#285, then we tests shoooould pass.

@cpanato
Copy link
Contributor

cpanato commented Feb 20, 2018

ok, will wait for that to retrigger this one

@squeed squeed force-pushed the ingress-controller-operator branch from 584f47a to cbce76f Compare February 21, 2018 18:49
@squeed squeed changed the title operator: configure and use tectonic-ingress-controller-operator tectonic: pull in default network policy (utility), enable ingress controller operator Feb 21, 2018
@squeed
Copy link
Author

squeed commented Feb 21, 2018

Okay, fixed the other random problems that were keeping this from going green. As part of the process, I needed to bump the utility operator. So this PR also pulls in a default network policy for tectonic-system

@squeed
Copy link
Author

squeed commented Feb 21, 2018

Nice, most tests pass. What's with the "exp" failure? Not sure how to diagnose.

@cpanato
Copy link
Contributor

cpanato commented Feb 21, 2018

@squeed i retriggered this test lets see if is not a flaky

@squeed
Copy link
Author

squeed commented Feb 22, 2018

all green on aws! yessss!

@cpanato
Copy link
Contributor

cpanato commented Feb 22, 2018

retest this please. azure/ metal / govcloud

@cpanato
Copy link
Contributor

cpanato commented Feb 22, 2018

for govcloud I need to do some cleanup there

@squeed
Copy link
Author

squeed commented Feb 22, 2018

azure failure looks like a flake.

@cpanato
Copy link
Contributor

cpanato commented Feb 22, 2018

govcloud will not work for now :/

@squeed
Copy link
Author

squeed commented Feb 22, 2018

Is this ready to merge?

Casey Callendrello added 2 commits February 23, 2018 11:53
@squeed squeed force-pushed the ingress-controller-operator branch from cbce76f to 00ef75a Compare February 23, 2018 10:53
Copy link
Contributor

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

lgtm

@cpanato cpanato merged commit 950c253 into coreos:master Feb 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants