shahar1 commented on code in PR #61285:
URL: https://github.com/apache/airflow/pull/61285#discussion_r2749560559
##########
.github/ISSUE_TEMPLATE/1-airflow_bug_report.yml:
##########
@@ -113,8 +111,6 @@ body:
- "Amazon (AWS) MWAA"
- "Microsoft ADF Managed Airflow"
- "Other"
- validations:
- required: true
Review Comment:
> I can understand the deployment info being useful for debugging, but I'd
still like push back a bit. When someone has a CC issue, they're going to
mention CC because that's their whole problem. The optional vs required
distinction mostly affects people reporting core bugs where deployment doesn't
matter.
>
> If the worry is losing diagnostic info, maybe we could add help text
explaining when these fields matter? Something like "especially important for
deployment related issues"?
>
> Happy to take the discussion to dev list if needed.
The thing with deployment, IMO, is that it could be related even for issues
that allegedly it seems not :)
However, you got a point that it's not always like that - and I guess that
over that between "simplifying the report process" and "not giving up on
diagnostic info", I could live with making this optional in the cost of a
better description (and again, maybe also with reordered/removed options).
Let's take the discussion to the dev list, I'll try to think in the
meanwhile what else could be simplified.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
