Skip to content

Retire the liveness_checking_required flag on service providers#6263

Merged
jmhooper merged 2 commits intomainfrom
jmhooper-retire-liveness-checking-required
Apr 28, 2022
Merged

Retire the liveness_checking_required flag on service providers#6263
jmhooper merged 2 commits intomainfrom
jmhooper-retire-liveness-checking-required

Conversation

@jmhooper
Copy link
Contributor

Liveness checking specific functionality is being discarded in favor of a full IAL2 compliant flow which may include additional deltas. That makes this flag confusing and hopefully unnecessary so this commit removes it.

A follow on commit will be required to drop the column.

Liveness checking specific functionality is being discarded in favor of a full IAL2 compliant flow which may include additional deltas. That makes this flag confusing and hopefully unnecessary so this commit removes it.

A follow on commit will be required to drop the column.
… for setting liveness checking requirements on an SP was removed
Copy link
Contributor

@orenyk orenyk left a comment

Choose a reason for hiding this comment

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

LGTM!

@jmhooper jmhooper merged commit fbe8c73 into main Apr 28, 2022
@jmhooper jmhooper deleted the jmhooper-retire-liveness-checking-required branch April 28, 2022 14:17
peggles2 pushed a commit that referenced this pull request May 3, 2022
Liveness checking specific functionality is being discarded in favor of a full IAL2 compliant flow which may include additional deltas. That makes this flag confusing and hopefully unnecessary so this commit removes it.

A follow on commit will be required to drop the column.

* changelog: Internal, Service provider management, An unnecessary flag for setting liveness checking requirements on an SP was removed
peggles2 pushed a commit that referenced this pull request May 5, 2022
Liveness checking specific functionality is being discarded in favor of a full IAL2 compliant flow which may include additional deltas. That makes this flag confusing and hopefully unnecessary so this commit removes it.

A follow on commit will be required to drop the column.

* changelog: Internal, Service provider management, An unnecessary flag for setting liveness checking requirements on an SP was removed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants