-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Remove invalid versions #9441
Remove invalid versions #9441
Conversation
@@ -338,7 +338,6 @@ | |||
"11-21h2-e", | |||
"10-21h2-e-lts", | |||
"10-21h2-iot-lts", | |||
"10-20h2-e-lts", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Windows 10 20H2 was only a semi-anual version and never a LTS. Therefore there was also no information on EndOfLife.date.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are right. Not sure what happened there. Likely a copy/paste error.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't 10-20h2-e
and 10-20h2-2
be added to the unsupported list?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
10-20h2-e and 10-20h2-w are already on the unsupported list. Not sure what you mean by 10-20h2-2.
@@ -393,11 +392,9 @@ | |||
"x86" | |||
], | |||
"supported-versions": [ | |||
"23H2", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to https://mcr.microsoft.com/en-us/product/windows/servercore and https://learn.microsoft.com/en-us/virtualization/windowscontainers/deploy-containers/base-image-lifecycle there was never a release of 23H2 or 2012.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And besides https://learn.microsoft.com/en-us/virtualization/windowscontainers/deploy-containers/base-image-lifecycle stating that 1607 is only Semi-Anual and released on Nano Server, whereas 2016 is the only Long-Term on Server Core:
https://mcr.microsoft.com/en-us/product/windows/servercore shows that it's instead handled like the Server-Core case with 2019 and 1809:
So in my opinion, https://learn.microsoft.com/en-us/virtualization/windowscontainers/deploy-containers/base-image-lifecycle should be fixed to be like this:
If 1607 and 1809 are considered LTSC as well, we should update the EOL dates on https://endoflife.date/windows-server-core as well to show they are still maintained (as shown by the image tags on MCR). As of now, both are considered EOL while the LTSC variants are supported:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Server 2012: https://learn.microsoft.com/en-us/lifecycle/products/windows-server-2012
Containers started with 2016, so the container-related lifecycle page is providing a partial view of Windows Server lifecycle.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But since there are no images available, there is nothing to have support for 🤔 Or did there exist a non-containered installation option for 2012? In that case, I will re-add it here and have EoL.dates updated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, I found https://learn.microsoft.com/en-us/previous-versions/windows/desktop/legacy/hh846313(v=vs.85) so I missed to add 2012 to EoL.date.
Will have a PR there and fix it here tomorrow once in office.
LGTM. Add 2012 back and then we can merge. Thanks for noticing this! |
My point was that Windows Containers started with 2016 so this search isn't testing anything. Server Core pre-dates containers. |
My point (in the reply, not the comment above which was created earlier) was about which version server core was introduced with as installation option. Seems to be 2012, right? So adding that to EoL.date should be enough to cover all versions? |
Got it. Sorry about that. |
I will add 2012 here in 1H once I'm on the office and make a PR for it on EoL.date :) |
It was available as installation option, but not as containered variant
@richlander I re-added it to .NET 6 and 8 and added it to .NET 7 and 9 where it had been missing in the first place. I did so, because I requested to add 2008 and 2012 to EoL.date as I missed them earlier: endoflife-date/endoflife.date#5643 Do we need to add 2008 as |
That all tracks for me. We don't add OSes to unsupported if we are still supporting them, like the ESU case you call out. For clarity, the decision to support an OS longer than standard EOL is arbitrary (not documented by policy) and not specific to MS-owned OSes. We have on occasion chosen to support Ubuntu longer than standard EOL since we had multiple customers who needed that. I am sure this will happen again. That makes these files all the more valuable as a source of truth. |
"2022", | ||
"2019", | ||
"2016" | ||
"2016", | ||
"2012-R2", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm asking for insight on 2012/R2 on .NET 9 support, from @jamshedd.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same for 2008/R2 then since it's in ESU4: https://learn.microsoft.com/de-de/lifecycle/products/windows-server-2008-r2
Was never listed as supported in the past (also not on the markdown before having the JSON).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I got confirmation that we still support 2012/R2, including for .NET 9. It was the lowest Server OS we've supported, so no need to add 2008.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Question about 2008 is still open.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was saying it is correct that 2008 was never listed. We never supported it, so it should not be in either supported or unsupported lists. This one isn't a function of ESU. It's that we don't test it and it may not work. We do test Server 2012.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the clarification. Then this should be fine. EoL.date was merged, so I will have another run of distrossed and the core-support tooling to see if any more might be off. But should be fine to merge otherwise :)
Is that it? Read to merge? |
On those 2 releases (with the R2 variants), it only existed as build option but not as container image. They share the lifecycle / EoS dates of the windows-server variant. See the following links to prove their existance: https://learn.microsoft.com/en-us/previous-versions/windows/desktop/legacy/ee391645(v=vs.85) https://learn.microsoft.com/en-us/previous-versions/windows/desktop/legacy/ee391631(v=vs.85) https://learn.microsoft.com/en-us/previous-versions/windows/desktop/legacy/hh846323(v=vs.85) This came up during dotnet/core#9441
Merging. There will also be opportunity for more changes! Thanks for the contribution. |
I removed some invalid versions that sadly found their way into the JSON files.
/CC @richlander