Use common logic to handle observedgeneration for Revision

[whaught]

Issue #4937

Proposed Changes

• Use the generated code logic for automatically bumping generation
  • Note: I'm writing these on a per-resource basis so we can see e2e tests separately for each resource. This might make it easier than doing one larger PR for all serving resources.

Release Note

[knative-prow-robot]

@whaught: 0 warnings.

In response to this:

Issue knative/pkg#1226

Proposed Changes

• Use the generated code logic for automatically bumping generation
  • Note: I'm writing these on a per-resource basis so we can see e2e tests separately for each resource. This might make it easier than doing one larger PR for all serving resources.

Release Note

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[knative-metrics-robot]

The following is the coverage report on the affected files.
Say /test pull-knative-serving-go-coverage to re-run this coverage report

File	Old Coverage	New Coverage	Delta
pkg/apis/serving/v1/revision_types.go	0.0%	100.0%	100.0
pkg/reconciler/revision/revision.go	98.1%	97.9%	-0.2

[whaught]

/assign @mattmoor @dprotaso

[mattmoor]

This looks good to me, but I'm wondering if we should hold off on landing more of these until after the cut on Tuesday, to avoid introducing instability.

Definitely looking forward to the auto-obsgen management though!

[whaught]

This looks good to me, but I'm wondering if we should hold off on landing more of these until after the cut on Tuesday, to avoid introducing instability.

Definitely looking forward to the auto-obsgen management though!

No rush! I know we just made the cut so if we want to hold, that's okay too

[whaught]

/hold waiting for release to get cut

[mattmoor]

We did the PKG cut, the serving cut is next week. thanks!

[markusthoemmes]

/unhold

release has cut.

[dprotaso]

/lgtm
/approve

[knative-prow-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dprotaso, whaught

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

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [dprotaso]

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