Skip to content
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

[SYSE-370 release-5.4] June template application #6420

Closed
wants to merge 1 commit into from

Conversation

alephnull
Copy link
Contributor

@alephnull alephnull commented Jul 17, 2024

User description

  • Images not being pushed to dockerhub on tag when distroless feature is missing
  • Policy match fix
  • Fix github ref name for test-controller ui on push trigger
  • Test controller UI missing conf for push event and other missconfigs
  • Test controller UI duplicated lines under pump & sink
  • Quality gates for tyk-analitycs UI and API regression tests
  • release-5.3 is not treated correctly in test-controller logic
  • Implement custom rules for enterprise artifacts in package promotion
  • Test controller UI
  • Accomodate templates for new controller UI
  • Add mongo7 and redis7 with cache_DB config field

PR Type

Enhancement, Bug fix


Description

  • Updated docker/build-push-action from v5 to v6 to ensure compatibility and leverage new features.
  • Fixed the GitHub ref name for the test-controller UI on push trigger to ensure correct references.
  • Removed redundant environment variables to streamline the workflow.
  • Consolidated test execution and metadata generation steps to improve efficiency and readability.

Changes walkthrough 📝

Relevant files
Enhancement
release.yml
Update Docker build action and fix test-controller UI configurations

.github/workflows/release.yml

  • Updated docker/build-push-action from v5 to v6.
  • Fixed GitHub ref name for test-controller UI on push trigger.
  • Removed redundant environment variables.
  • Consolidated test execution and metadata generation steps.
  • +13/-36 

    💡 PR-Agent usage:
    Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

    @alephnull alephnull requested a review from a team as a code owner July 17, 2024 08:12
    @alephnull alephnull enabled auto-merge (squash) July 17, 2024 08:12
    Copy link
    Contributor

    API Changes

    no api changes detected

    Copy link
    Contributor

    PR Reviewer Guide 🔍

    ⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Key issues to review

    Version Upgrade
    The PR upgrades docker/build-push-action from v5 to v6. Ensure that all features used in v5 are compatible with v6 and that no deprecated features are being used.

    Version Upgrade
    The PR upgrades docker/build-push-action from v5 to v6. Ensure that all features used in v5 are compatible with v6 and that no deprecated features are being used.

    Version Upgrade
    The PR upgrades docker/build-push-action from v5 to v6. Ensure that all features used in v5 are compatible with v6 and that no deprecated features are being used.

    Version Upgrade
    The PR upgrades docker/build-push-action from v5 to v6. Ensure that all features used in v5 are compatible with v6 and that no deprecated features are being used.

    Copy link
    Contributor

    PR Code Suggestions ✨

    CategorySuggestion                                                                                                                                    Score
    Possible issue
    Add error handling to the docker compose command to enhance robustness

    Ensure that the docker compose command includes error handling to manage potential
    failures during the setup of environments.

    .github/workflows/release.yml [299]

    -docker compose -p auto -f pro-ha.yml -f deps_pro-ha.yml -f ${{ matrix.envfiles.db }}.yml -f ${{ matrix.envfiles.cache }}.yml --env-file versions.env --profile master-datacenter up --quiet-pull -d
    +docker compose -p auto -f pro-ha.yml -f deps_pro-ha.yml -f ${{ matrix.envfiles.db }}.yml -f ${{ matrix.envfiles.cache }}.yml --env-file versions.env --profile master-datacenter up --quiet-pull -d || echo "Error during docker compose up"
     
    Suggestion importance[1-10]: 10

    Why: Adding error handling to the docker compose command is crucial for managing potential failures during the setup of environments, thereby improving the robustness of the CI/CD pipeline.

    10
    Maintainability
    Simplify the conditional assignment of BASE_REF for clarity

    Replace the complex conditional expression for BASE_REF with a simpler or more
    explicit approach to improve readability and maintainability.

    .github/workflows/release.yml [280]

    -BASE_REF: ${{startsWith(github.event_name, 'pull_request') && github.base_ref || github.ref_name}}
    +BASE_REF: ${{ github.event_name == 'pull_request' ? github.base_ref : github.ref_name }}
     
    Suggestion importance[1-10]: 9

    Why: Simplifying the conditional expression improves readability and maintainability, making the code easier to understand and less error-prone.

    9
    Enhancement
    Update the Docker build-push-action to the latest version for enhanced security and features

    Update the docker/build-push-action to the latest version (v7) to ensure
    compatibility with new features and security patches.

    .github/workflows/release.yml [141]

    -uses: docker/build-push-action@v6
    +uses: docker/build-push-action@v7
     
    Suggestion importance[1-10]: 8

    Why: Updating to the latest version of the Docker build-push-action ensures compatibility with new features and security patches, which is important for maintaining the CI/CD pipeline's reliability and security.

    8
    Best practice
    Use a more specific runner version to ensure consistent CI behavior

    Consider using a more specific tag than ubuntu-latest-m-2 for the runs-on attribute
    to avoid potential issues with unexpected updates.

    .github/workflows/release.yml [229]

    -runs-on: ubuntu-latest-m-2
    +runs-on: ubuntu-20.04-m-2
     
    Suggestion importance[1-10]: 7

    Why: Using a specific runner version helps avoid potential issues with unexpected updates, ensuring consistent behavior in the CI environment.

    7

    @buger buger force-pushed the releng/release-5.4 branch from 15f9f2a to 120a326 Compare July 17, 2024 08:16
    Copy link
    Contributor

    💥 CI tests failed 🙈

    git-state

    all ok

    Please look at the run or in the Checks tab.

    @buger buger force-pushed the releng/release-5.4 branch 4 times, most recently from 2481aa9 to 05c112e Compare July 17, 2024 12:18
    Copy link

    @buger buger force-pushed the releng/release-5.4 branch 2 times, most recently from eba4cd4 to cf0053c Compare July 18, 2024 06:42
    Copy link

    Please retry analysis of this Pull-Request directly on SonarCloud

    @buger buger force-pushed the releng/release-5.4 branch from cf0053c to 0d06e7b Compare July 18, 2024 07:11
    Copy link
    Contributor

    💥 CI tests failed 🙈

    git-state

    all ok

    Please look at the run or in the Checks tab.

    @buger buger force-pushed the releng/release-5.4 branch 2 times, most recently from 7adfdca to 01153d1 Compare July 18, 2024 19:05
    Copy link

    Please retry analysis of this Pull-Request directly on SonarCloud

    @buger buger force-pushed the releng/release-5.4 branch from 01153d1 to 6f5ef5f Compare July 19, 2024 22:55
    Copy link
    Contributor

    💥 CI tests failed 🙈

    git-state

    all ok

    Please look at the run or in the Checks tab.

    @buger buger force-pushed the releng/release-5.4 branch 5 times, most recently from c51b726 to 6e834b0 Compare July 22, 2024 11:32
    @buger buger force-pushed the releng/release-5.4 branch 2 times, most recently from 62d0100 to 5788483 Compare July 24, 2024 13:54
    Copy link
    Contributor

    💥 CI tests failed 🙈

    git-state

    all ok

    Please look at the run or in the Checks tab.

    @buger buger force-pushed the releng/release-5.4 branch 2 times, most recently from 541f368 to ac9f684 Compare July 25, 2024 02:03
    Copy link
    Contributor

    💥 CI tests failed 🙈

    git-state

    all ok

    Please look at the run or in the Checks tab.

    @buger buger force-pushed the releng/release-5.4 branch 3 times, most recently from afbcb1d to 75e8f6d Compare August 6, 2024 19:27
    @buger buger force-pushed the releng/release-5.4 branch 2 times, most recently from 3273cc2 to 3fbdb8c Compare August 8, 2024 17:22
    Copy link
    Contributor

    github-actions bot commented Aug 8, 2024

    💥 CI tests failed 🙈

    git-state

    all ok

    Please look at the run or in the Checks tab.

    @buger buger force-pushed the releng/release-5.4 branch 4 times, most recently from 740abbc to 580b777 Compare August 9, 2024 19:57
    Copy link
    Contributor

    github-actions bot commented Aug 9, 2024

    💥 CI tests failed 🙈

    git-state

    all ok

    Please look at the run or in the Checks tab.

    @buger buger force-pushed the releng/release-5.4 branch from 580b777 to 2f58813 Compare August 9, 2024 20:22
    Copy link

    sonarqubecloud bot commented Aug 9, 2024

    @buger buger force-pushed the releng/release-5.4 branch 2 times, most recently from c23c0a5 to e3651ba Compare August 13, 2024 20:35
    Copy link
    Contributor

    💥 CI tests failed 🙈

    git-state

    all ok

    Please look at the run or in the Checks tab.

    @buger buger force-pushed the releng/release-5.4 branch from e3651ba to 79c0f04 Compare August 14, 2024 17:11
    Copy link

    @buger buger force-pushed the releng/release-5.4 branch from 79c0f04 to 3749d7f Compare August 15, 2024 18:45
    Copy link
    Contributor

    💥 CI tests failed 🙈

    git-state

    all ok

    Please look at the run or in the Checks tab.

    @ermirizio
    Copy link
    Collaborator

    closing in favor of release5.5

    @ermirizio ermirizio closed this Aug 29, 2024
    auto-merge was automatically disabled August 29, 2024 16:33

    Pull request was closed

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    2 participants