-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add DomainInternal to status for Route and Service #1586
Merged
google-prow-robot
merged 9 commits into
knative:master
from
scothis:incluster-domain-status
Jul 17, 2018
Merged
Add DomainInternal to status for Route and Service #1586
google-prow-robot
merged 9 commits into
knative:master
from
scothis:incluster-domain-status
Jul 17, 2018
Commits on Jul 15, 2018
-
Add ServiceName to status for resources that create services
For resoruces that own a core Kubernetes Services, the name of the service should be exposed on the resoruce's status. The Revision resources already exposes `.status.serviceName`, this PR adds the same property for the Route and Knative Service resources. Yes, it's a bit strange to have a Serivce include the Service name in it's status, ¯\_(ツ)_/¯ Fixes: #1584
Configuration menu - View commit details
-
Copy full SHA for 3029a48 - Browse repository at this point
Copy the full SHA 3029a48View commit details -
Configuration menu - View commit details
-
Copy full SHA for ef5d583 - Browse repository at this point
Copy the full SHA ef5d583View commit details -
Configuration menu - View commit details
-
Copy full SHA for bc23435 - Browse repository at this point
Copy the full SHA bc23435View commit details
Commits on Jul 16, 2018
-
Configuration menu - View commit details
-
Copy full SHA for 19e5aaf - Browse repository at this point
Copy the full SHA 19e5aafView commit details -
Configuration menu - View commit details
-
Copy full SHA for aa4ae46 - Browse repository at this point
Copy the full SHA aa4ae46View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4fa9d06 - Browse repository at this point
Copy the full SHA 4fa9d06View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3be8173 - Browse repository at this point
Copy the full SHA 3be8173View commit details -
Configuration menu - View commit details
-
Copy full SHA for c37655b - Browse repository at this point
Copy the full SHA c37655bView commit details
Commits on Jul 17, 2018
-
Configuration menu - View commit details
-
Copy full SHA for 80d5f73 - Browse repository at this point
Copy the full SHA 80d5f73View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.