Skip to content

DOCSP-51041: drop debian 9 & 10#129

Merged
rustagir merged 4 commits into
mongodb:masterfrom
rustagir:DOCSP-51041-debian-9-10-drop
Jul 3, 2025
Merged

DOCSP-51041: drop debian 9 & 10#129
rustagir merged 4 commits into
mongodb:masterfrom
rustagir:DOCSP-51041-debian-9-10-drop

Conversation

@rustagir
Copy link
Copy Markdown
Contributor

@rustagir rustagir commented Jul 1, 2025

Pull Request Info

PR Reviewing Guidelines

JIRA - https://jira.mongodb.org/browse/DOCSP-51041

Staging Links

  • compatibility
  • upgrade
  • whats-new
  • Self-Review Checklist

    • Is this free of any warnings or errors in the RST?
    • Did you run a spell-check?
    • Did you run a grammar-check?
    • Are all the links working?
    • Are the facets and meta keywords accurate?
    • Are the page titles greater than 20 characters long and SEO relevant?

    @netlify
    Copy link
    Copy Markdown

    netlify Bot commented Jul 1, 2025

    Deploy Preview for docs-c ready!

    Name Link
    🔨 Latest commit fa56bcd
    🔍 Latest deploy log https://app.netlify.com/projects/docs-c/deploys/6866ced8cb4be2000818f484
    😎 Deploy Preview https://deploy-preview-129--docs-c.netlify.app
    📱 Preview on mobile
    Toggle QR Code...

    QR Code

    Use your smartphone camera to open QR code link.

    To edit notification comments on pull requests, go to your Netlify project configuration.

    @docs-builder-bot
    Copy link
    Copy Markdown

    docs-builder-bot commented Jul 1, 2025

    🔄 Deploy Preview for docs-c processing

    Item Details
    🔨 Latest Commit 094401118a10fbc7b662adb0ea4e925df0fe79bc
    😎 Deploy Preview https://deploy-preview-129--docs-c.netlify.app
    🔍 Build Logs View Logs

    @rustagir rustagir requested review from a team and mdb-ad and removed request for a team July 1, 2025 15:07
    Copy link
    Copy Markdown
    Collaborator

    @norareidy norareidy left a comment

    Choose a reason for hiding this comment

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

    LGTM with a suggestion to move this info to Upgrade Driver Versions

    Comment thread source/whats-new.txt
    @rustagir rustagir requested a review from kevinAlbs July 3, 2025 15:08
    @kevinAlbs kevinAlbs removed the request for review from mdb-ad July 3, 2025 17:11
    Copy link
    Copy Markdown
    Collaborator

    @kevinAlbs kevinAlbs left a comment

    Choose a reason for hiding this comment

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

    Removal LGTM, but have a suggestion to simplify / fix the compatibility matrix.

    Comment thread source/compatibility.txt Outdated
    - Windows variants of the same generation are supported

    * - Windows Server 2008 and Windows Server 2016
    - Windows variants of the same generation are supported.
    Copy link
    Copy Markdown
    Collaborator

    Choose a reason for hiding this comment

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

    Some of this section is out-dated. Suggest instead rewording the sentence above to:

    The following operating systems are expected to be supported with the {+driver-short+}:
    

    That helps separate the list of exact OS versions we test (which may change more frequently) from what we expect to support.

    Then this table can be updated and simplified to:

      * - Debian
        - Versions 11.0 and newer
    
      * - RHEL
        - Versions 7.0 and newer. RHEL derivatives
          (for example, CentOS, Rocky Linux, or AlmaLinux) of the same release version are
          supported. Fedora is also supported.
    
      * - Ubuntu
        - Versions 20.04 and newer. Ubuntu derivatives based on supported Ubuntu versions are also supported.
    
      * - Arch Linux
        -
    
      * - macOS
        - Version 11.0 and newer
        
      * - Windows
        - Windows Server 2008 and newer.
    

    Copy link
    Copy Markdown
    Contributor Author

    Choose a reason for hiding this comment

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

    I'll use earlier/later terminology per our style guide but this SGTM!

    @rustagir rustagir requested a review from kevinAlbs July 3, 2025 17:58
    Copy link
    Copy Markdown
    Collaborator

    @kevinAlbs kevinAlbs left a comment

    Choose a reason for hiding this comment

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

    LGTM with minor comments

    Comment thread source/compatibility.txt Outdated
    - Versions 7.0, 7.1, 8.1, 8.2, and 8.3. RHEL derivatives
    (for example, CentOS, Rocky Linux, or AlmaLinux) of the same release version are
    supported. Fedora is also supported, but not continually tested.
    - Versions 7.0 and newer. RHEL derivatives (such as CentOS,
    Copy link
    Copy Markdown
    Collaborator

    Choose a reason for hiding this comment

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

    Suggested change
    - Versions 7.0 and newer. RHEL derivatives (such as CentOS,
    - Versions 7.0 and later. RHEL derivatives (such as CentOS,

    Copy link
    Copy Markdown
    Contributor Author

    Choose a reason for hiding this comment

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

    copypasta facepalm

    Comment thread source/compatibility.txt Outdated
    - Versions 16.04, 18.04, and 20.04. Subsequent minor releases
    are also supported. Ubuntu 22.04 and newer is not yet tested. Ubuntu
    derivatives based on supported Ubuntu versions are also supported.
    - Versions 20.04 and newer. Ubuntu derivatives based on supported
    Copy link
    Copy Markdown
    Collaborator

    Choose a reason for hiding this comment

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

    Suggested change
    - Versions 20.04 and newer. Ubuntu derivatives based on supported
    - Versions 20.04 and later. Ubuntu derivatives based on supported

    @rustagir rustagir merged commit 4cd97ff into mongodb:master Jul 3, 2025
    2 checks passed
    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.

    4 participants