BasPH commented on code in PR #24680:
URL: https://github.com/apache/airflow/pull/24680#discussion_r907476222


##########
README.md:
##########
@@ -410,6 +410,27 @@ For example this means that by default we upgrade the 
minimum version of Airflow
 to 2.3.0 in the first Provider's release after 11th of October 2022 (11th of 
October 2021 is the date when the
 first `PATCHLEVEL` of 2.2 (2.2.0) has been released.
 
+Since providers are often connected with some stakeholders that are vitally 
interested in maintaining
+their integrations (for example cloud providers, ore specific service 
providers), and we are also bound with
+the Apache Software Foundation release policies. The provider's governance 
model is something we name
+"mixed governance".
+
+The Airflow Community and release manager still decides when to release those 
providers.
+This is fully managed by the community and the usual release-management 
process.
+
+Usually, we release only the most recent version of the provider and rather 
aggressively
+remove deprecations in "major" versions of the providers, however the 
stakeholder in a given provider might
+agree and make their effort on cherry-picking the non-breaking changes to a 
selected previous major branch
+of the provider which results in releasing more (usually two) versions of such 
provider when we release it:
+potentially breaking "latest" major version, and selected past major version 
with non-breaking changes
+applied by the stakeholder (cherry-picked changes have to be merged by the 
committer following the usual
+rules of the community).
+
+The community continues to release such older versions of the providers for as 
long as there is an effort
+of the stakeholder to maintain the cherry-picked changes. The availability of 
the stakeholder that can
+manage "service-oriented" maintenance and agrees to such a responsibility, 
will also drive our willingness
+to accept future, new providers to become community managed.

Review Comment:
   Not sure what message you want to convey here. Is this paragraph necessary?



##########
README.md:
##########
@@ -410,6 +410,27 @@ For example this means that by default we upgrade the 
minimum version of Airflow
 to 2.3.0 in the first Provider's release after 11th of October 2022 (11th of 
October 2021 is the date when the
 first `PATCHLEVEL` of 2.2 (2.2.0) has been released.
 
+Since providers are often connected with some stakeholders that are vitally 
interested in maintaining
+their integrations (for example cloud providers, ore specific service 
providers), and we are also bound with
+the Apache Software Foundation release policies. The provider's governance 
model is something we name
+"mixed governance".
+
+The Airflow Community and release manager still decides when to release those 
providers.
+This is fully managed by the community and the usual release-management 
process.
+
+Usually, we release only the most recent version of the provider and rather 
aggressively
+remove deprecations in "major" versions of the providers, however the 
stakeholder in a given provider might
+agree and make their effort on cherry-picking the non-breaking changes to a 
selected previous major branch
+of the provider which results in releasing more (usually two) versions of such 
provider when we release it:
+potentially breaking "latest" major version, and selected past major version 
with non-breaking changes
+applied by the stakeholder (cherry-picked changes have to be merged by the 
committer following the usual
+rules of the community).

Review Comment:
   - "however the stakeholder in a given provider might agree" agree with what?
   - Can you give an example of when this happens, e.g. critical bug or 
security fix?



##########
README.md:
##########
@@ -410,6 +410,27 @@ For example this means that by default we upgrade the 
minimum version of Airflow
 to 2.3.0 in the first Provider's release after 11th of October 2022 (11th of 
October 2021 is the date when the
 first `PATCHLEVEL` of 2.2 (2.2.0) has been released.
 
+Since providers are often connected with some stakeholders that are vitally 
interested in maintaining
+their integrations (for example cloud providers, ore specific service 
providers), and we are also bound with
+the Apache Software Foundation release policies. The provider's governance 
model is something we name
+"mixed governance".
+
+The Airflow Community and release manager still decides when to release those 
providers.
+This is fully managed by the community and the usual release-management 
process.

Review Comment:
   IMO this paragraph doesn't hold any useful information. As a reader, in case 
I would have a question; what are useful resources or channels to ask?



##########
README.md:
##########
@@ -410,6 +410,27 @@ For example this means that by default we upgrade the 
minimum version of Airflow
 to 2.3.0 in the first Provider's release after 11th of October 2022 (11th of 
October 2021 is the date when the
 first `PATCHLEVEL` of 2.2 (2.2.0) has been released.
 
+Since providers are often connected with some stakeholders that are vitally 
interested in maintaining
+their integrations (for example cloud providers, ore specific service 
providers), and we are also bound with
+the Apache Software Foundation release policies. The provider's governance 
model is something we name
+"mixed governance".

Review Comment:
   - Please clarify "we are also bound with the Apache Software Foundation 
release policies"
   - "providers are often connected with some stakeholders" are those 
stakeholders documented somewhere?
   - Please clarify "The provider's governance model"
   - You throw in the term "mixed governance" but don't refer to it anywhere. 
Can you elaborate on it?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to