Skip to content
Snippets Groups Projects

Check if Kubernetes is required when creating pipeline jobs

Merged Grzegorz Bizon requested to merge feature/gb/kubernetes-only-pipeline-jobs into master
All threads resolved!

What does this MR do?

This MR extends only/except policy for CI/CD jobs, to make it possible to require Kubernetes.

deploy:
  only:
    refs:
      - master
    kubernetes: active

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes #34785 (closed)

Edited by Grzegorz Bizon

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • mentioned in issue #34785 (closed)

  • Author Maintainer

    @ayufan Can you do an initial review, please? Thanks!

  • mentioned in merge request gitlab-ci-yml!88 (merged)

  • Grzegorz Bizon changed milestone to %10.0

    changed milestone to %10.0

  • @grzesiek Given that we would focus on service integration, we then would not need an evaluation of variables.

  • Grzegorz Bizon added 1 commit

    added 1 commit

    • 96a7e162 - 1 commit from branch feature/gb/add-complex-jobs-only-except-config-policy

    Compare with previous version

  • Grzegorz Bizon added 3 commits

    added 3 commits

    • 96a7e162...5623cb9a - 3 commits from branch feature/gb/add-complex-jobs-only-except-config-policy

    Compare with previous version

  • Grzegorz Bizon added 3 commits

    added 3 commits

    • 96a7e162...5623cb9a - 3 commits from branch feature/gb/add-complex-jobs-only-except-config-policy

    Compare with previous version

  • Grzegorz Bizon added 426 commits

    added 426 commits

    • 5623cb9a...92673c2c - 426 commits from branch feature/gb/add-complex-jobs-only-except-config-policy

    Compare with previous version

  • Grzegorz Bizon changed target branch from feature/gb/add-complex-jobs-only-except-config-policy to master

    changed target branch from feature/gb/add-complex-jobs-only-except-config-policy to master

  • Grzegorz Bizon added 1 commit

    added 1 commit

    • ef030709 - 1 commit from branch master

    Compare with previous version

  • Grzegorz Bizon added 1 commit

    added 1 commit

    • ef030709 - Change kubernetes job policy allowed values

    Compare with previous version

  • Grzegorz Bizon resolved all discussions

    resolved all discussions

  • mentioned in issue #37280 (closed)

  • Grzegorz Bizon marked the checklist item Tests added for this feature/bug as completed

    marked the checklist item Tests added for this feature/bug as completed

  • Grzegorz Bizon unmarked as a Work In Progress

    unmarked as a Work In Progress

  • Grzegorz Bizon changed the description

    changed the description

  • Author Maintainer

    @ayufan Can you please review? Thanks!

    @bikebilly Can you please approve usage of kubernetes: configured? Thanks!

  • assigned to @ayufan

  • Grzegorz Bizon added 1 commit

    added 1 commit

    • dc4d9386 - Fix only/except policy CI/CD linter

    Compare with previous version

  • Grzegorz Bizon
  • Grzegorz Bizon added 1 commit

    added 1 commit

    • 73d0cafb - Check if service active when using CI/CD kubernetes policy

    Compare with previous version

  • Grzegorz Bizon changed the description

    changed the description

  • Grzegorz Bizon marked the checklist item Has been reviewed by Product as completed

    marked the checklist item Has been reviewed by Product as completed

  • Grzegorz Bizon marked the checklist item Has been reviewed by Backend as completed

    marked the checklist item Has been reviewed by Backend as completed

  • Grzegorz Bizon added 1 commit

    added 1 commit

    • fc4fb6e4 - Change CI/CD kubernetes policy keyword to active

    Compare with previous version

  • Grzegorz Bizon marked the checklist item Changelog entry added, if necessary as completed

    marked the checklist item Changelog entry added, if necessary as completed

  • Grzegorz Bizon added 1 commit

    added 1 commit

    • 6f658392 - Add changelog entry for kubernetes: active job policy

    Compare with previous version

  • Grzegorz Bizon added 1 commit

    added 1 commit

    • 6f658392 - Add changelog entry for kubernetes: active job policy

    Compare with previous version

  • Grzegorz Bizon marked the checklist item Documentation created/updated as completed

    marked the checklist item Documentation created/updated as completed

  • Grzegorz Bizon added 2 commits

    added 2 commits

    • 84827c13 - Add some basic docs about kubernetes: active feature
    • f911dafe - Add disclaimer about alpha kubernetes: active feature

    Compare with previous version

  • Grzegorz Bizon added 1 commit

    added 1 commit

    • ab8e6c80 - Copy-edit kubernetes: active feature docs

    Compare with previous version

  • Grzegorz Bizon added 1 commit

    added 1 commit

    • 2c5a4cf5 - Fix only/except docs disclaimer's markdown

    Compare with previous version

  • assigned to @ayufan

  • Grzegorz Bizon added 1 commit

    added 1 commit

    • a6ead9d1 - Remove an empty line from YAML processor class

    Compare with previous version

  • Author Maintainer

    @ayufan Please review! Thanks!

  • Kamil Trzcińśki
  • assigned to @ayufan

  • mentioned in issue #37384

  • Grzegorz Bizon added 1 commit

    added 1 commit

    • 339a1ad0 - Fix rubocop offense in CI/CD YAML processor specs

    Compare with previous version

  • Kamil Trzcińśki resolved all discussions

    resolved all discussions

  • Kamil Trzcińśki approved this merge request

    approved this merge request

  • Kamil Trzcińśki enabled an automatic merge when the pipeline for 339a1ad0 succeeds

    enabled an automatic merge when the pipeline for 339a1ad0 succeeds

  • Grzegorz Bizon added 561 commits

    added 561 commits

    Compare with previous version

  • Merging this, as the failure is also present on master. Thanks @grzesiek :100:

  • Kamil Trzcińśki approved this merge request

    approved this merge request

  • mentioned in commit a0c13698

  • Please register or sign in to reply
    Loading