User description

Thanks for contributing to the Docker-Selenium project!
A PR well described will help maintainers to quickly review and merge it

Before submitting your PR, please check our contributing guidelines, applied for this repository.
Avoid large PRs, help reviewers by making them as simple and short as possible.

Description

This pull request includes several changes to the Selenium Grid Helm chart to enhance its configuration and scalability. The key changes involve renaming deployment names for consistency, adding support for cross-browser nodes, and updating templates to accommodate these new configurations.

Example Usage

crossBrowsers:
  chromeNode:
    # Keep the first iteration with latest version of Chrome
    - nameOverride:
    - nameOverride: '{{ $.Release.Name }}-node-chrome-130'
      imageTag: '130.0'
      hpa:
        browserVersion: '130.0'
    - nameOverride: '{{ $.Release.Name }}-node-chrome-129'
      imageTag: '129.0'
      hpa:
        browserVersion: '129.0'
    - nameOverride: '{{ $.Release.Name }}-node-chrome-128'
      imageTag: '128.0'
      hpa:
        browserVersion: '128.0'
  firefoxNode:
    # Keep the first iteration with latest version of Firefox
    - nameOverride:
    - nameOverride: '{{ $.Release.Name }}-node-firefox-130'
      imageTag: '130.0'
      hpa:
        browserVersion: '130.0'
    - nameOverride: '{{ $.Release.Name }}-node-firefox-129'
      imageTag: '129.0'
      hpa:
        browserVersion: '129.0'
    - nameOverride: '{{ $.Release.Name }}-node-firefox-128'
      imageTag: '128.0'
      hpa:
        browserVersion: '128.0'
  edgeNode:
    # Keep the first iteration with latest version of Edge
    - nameOverride:
    - nameOverride: '{{ $.Release.Name }}-node-edge-130'
      imageTag: '130.0'
      hpa:
        browserVersion: '130.0'
    - nameOverride: '{{ $.Release.Name }}-node-edge-129'
      imageTag: '129.0'
      hpa:
        browserVersion: '129.0'
    - nameOverride: '{{ $.Release.Name }}-node-edge-128'
      imageTag: '128.0'
      hpa:
        browserVersion: '128.0'

Cross-Browser Support:

• Added support for additional Chrome, Firefox, and Edge nodes in the Helm chart configuration. (charts/selenium-grid/CONFIGURATION.md: [1] charts/selenium-grid/cross-browsers-values.yaml: [2]

Template Updates:

• Updated templates to handle the new cross-browser node configurations, including deployment, HPA, ScaledJob, and service templates. (charts/selenium-grid/templates/chrome-node-deployment.yaml: [1] charts/selenium-grid/templates/chrome-node-hpa.yaml: [2] charts/selenium-grid/templates/chrome-node-scaledjobs.yaml: [3] charts/selenium-grid/templates/chrome-node-service.yaml: [4] charts/selenium-grid/templates/edge-node-deployment.yaml: [5]

Name Helper Template:

• Modified the name helper template to generate consistent names for cross-browser nodes. (charts/selenium-grid/templates/_nameHelpers.tpl: charts/selenium-grid/templates/_nameHelpers.tplL126-R155)

Motivation and Context

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist

• I have read the contributing document.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.

PR Type

enhancement, tests

Description

• Enhanced Selenium Grid Helm chart to support multiple nodes for Chrome, Firefox, Edge, and Relay.
• Updated test scripts to reflect new naming conventions for browser nodes.
• Added cross-browser configurations in values.yaml and cross-browsers-values.yaml.
• Updated deployment, HPA, ScaledJob, and service templates to support cross-browser configurations.
• Documented new configurations in CONFIGURATION.md.

Changes walkthrough 📝

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information