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

More refactoring to image import controller #3

Merged
merged 1 commit into from
May 17, 2017

Conversation

mfojtik
Copy link
Collaborator

@mfojtik mfojtik commented May 16, 2017

No description provided.

@mfojtik
Copy link
Collaborator Author

mfojtik commented May 16, 2017

@soltysh i did some more cleanup in the code to make this controller more readable for humans...

@soltysh
Copy link
Owner

soltysh commented May 17, 2017

Not needed, this is entirely re-written in openshift#13835.

@mfojtik
Copy link
Collaborator Author

mfojtik commented May 17, 2017

@soltysh i based it on top of that PR (saving time making comments ;-)

@soltysh
Copy link
Owner

soltysh commented May 17, 2017

OMG, I'm blind, now I've noticed it's against my branch. Sorry ;)

@soltysh soltysh merged commit 6091735 into soltysh:image_shared_informers May 17, 2017
soltysh pushed a commit that referenced this pull request Jun 16, 2017
Refactor controller initialization (round #3)
soltysh pushed a commit that referenced this pull request Jun 29, 2017
Automatic merge from submit-queue (batch tested with PRs 3, 4)

more work for prow
soltysh pushed a commit that referenced this pull request Oct 18, 2017
Suppress Strict-Transport-Security header for http requests
It is only emitted for https.

bug 1501133
https://bugzilla.redhat.com/show_bug.cgi?id=1501133
see comment #3
soltysh pushed a commit that referenced this pull request Oct 18, 2017
Automatic merge from submit-queue (batch tested with PRs 16888, 16911, 16913, 16904).

Router - hsts for "edge" or "reencrypt" only

Suppress Strict-Transport-Security header for http requests
It is only emitted for https.

bug 1501133
https://bugzilla.redhat.com/show_bug.cgi?id=1501133
see comment #3
@mfojtik mfojtik deleted the image_controllers branch September 5, 2018 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants