[release-v1.8][BACKPORT] Don't default Revision values when BYO name is unchanged.

[openshift-cherrypick-robot]

This is an automated cherry-pick of #131

/assign ReToCode

[ReToCode]

/cc @skonto , @nak3 #131 (comment)

[skonto]

@ReToCode @nak3 yes let's add this for consistency for 1.8 (but not for 1.9+).

So in 1.7 we added this because when we have sec pod defaults on and we do an upgrade (1.6 -> 1.7) revisions are modified during defaulting. Thus the BYO (bring your own) name services dont fail.
The BYO name service requires you to change the name of the revision manually if you do an update to its spec and without this an update happens and test fail. The normal services update the revision name automatically by adding a suffix (if the change requires a new revision). Revisions are immutable.

For 1.8 I suspect user will be on 1.7 anyway so things will not fail but let's have it to avoid any inconsistencies.

[ReToCode]

For 1.8 I suspect user will be on 1.7 anyway so things will not fail but let's have it to avoid any inconsistencies.

/lgtm
/approve

[nak3]

/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: nak3, openshift-cherrypick-robot, ReToCode

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:

• OWNERS [nak3]

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