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

Extend policy probationary period until May 31st, 2021. #2656

Merged
merged 1 commit into from
Apr 19, 2021

Conversation

KPrasch
Copy link
Member

@KPrasch KPrasch commented Apr 19, 2021

Type of PR:
Other

Required reviews:
1

What this does:
Extends policy probationary period until May 31st, 2021.

Why it's needed:
Prevent CLI CI failures and give additional development runway before permitting long-lived and high-value policies.

Copy link
Member

@derekpierre derekpierre left a comment

Choose a reason for hiding this comment

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

🎸 - Do we want to do a newsfagment (done for #2585 ) or since this is only temporary until we have a discussion, no need for a newsfragment...? It does affect app developers.

I guess after an in-depth discussion it is more likely to extend the probationary period than reduce 😅 , so a newsfragment this time would not be a bad thing.

@KPrasch
Copy link
Member Author

KPrasch commented Apr 19, 2021

Yes, let's do a newsfragment, it's exactly the kind of thing it's good at! Standby.

KPrasch added a commit to KPrasch/nucypher that referenced this pull request Apr 19, 2021
@KPrasch
Copy link
Member Author

KPrasch commented Apr 19, 2021

Done.

@derekpierre
Copy link
Member

One nitpick - May has 31 days 😅 , we should do the 31st. After that merge at will.

@@ -73,4 +73,4 @@
NUCYPHER_EVENTS_THROTTLE_MAX_BLOCKS = 'NUCYPHER_EVENTS_THROTTLE_MAX_BLOCKS'

# Probationary period (see #2353, #2584)
END_OF_POLICIES_PROBATIONARY_PERIOD = MayaDT.from_iso8601('2021-04-30T23:59:59.0Z')
END_OF_POLICIES_PROBATIONARY_PERIOD = MayaDT.from_iso8601('2021-05-30T23:59:59.0Z')
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
END_OF_POLICIES_PROBATIONARY_PERIOD = MayaDT.from_iso8601('2021-05-30T23:59:59.0Z')
END_OF_POLICIES_PROBATIONARY_PERIOD = MayaDT.from_iso8601('2021-05-31T23:59:59.0Z')

@@ -0,0 +1 @@
Extends policy probationary period until May 30th, 2021. No policies may be created on the network beyond this date.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Extends policy probationary period until May 30th, 2021. No policies may be created on the network beyond this date.
Extends policy probationary period until May 31st, 2021. No policies may be created on the network beyond this date.

@KPrasch KPrasch changed the title Extend policy probationary period until May 30th, 2021. Extend policy probationary period until May 31st, 2021. Apr 19, 2021
@KPrasch KPrasch merged commit e96b1d7 into nucypher:main Apr 19, 2021
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.

None yet

2 participants