Skip to content
Snippets Groups Projects

Remove column ignore rules for migrated PKs

Merged Krasimir Angelov requested to merge remove-column-ignores-for-migrated-pk-2 into master

What does this MR do and why?

This MR removes column ignore rules for the following tables:

All columns were removed in 14.4.

We also remove IgnorableColumns module for models that no more have columns ignored, and had rules removed with !71161 (merged).

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Krasimir Angelov

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
  • Krasimir Angelov
  • Krasimir Angelov
  • Krasimir Angelov
  • Krasimir Angelov
  • Krasimir Angelov
  • Krasimir Angelov marked this merge request as draft

    marked this merge request as draft

  • Krasimir Angelov changed the description

    changed the description

  • Allure report

    allure-report-publisher generated test report for 27c9a5e7!

    review-qa-smoke: :pencil: test report

  • Setting label groupdatabase based on @krasio's group.

  • Krasimir Angelov added 1271 commits

    added 1271 commits

    • ef4c1844...438c3b27 - 1269 commits from branch master
    • 3aa9b3d5 - Remove column ignore rules for migrated PKs
    • 72d4e3b4 - Remove IgnorableColumns module for models that have no ignored columns

    Compare with previous version

  • Krasimir Angelov mentioned in merge request !72378 (merged)

    mentioned in merge request !72378 (merged)

  • Krasimir Angelov marked this merge request as ready

    marked this merge request as ready

  • Krasimir Angelov changed the description

    changed the description

  • requested review from @dfrazao-gitlab

  • Diogo Frazão requested review from @pbair and removed review request for @dfrazao-gitlab

    requested review from @pbair and removed review request for @dfrazao-gitlab

  • Diogo Frazão approved this merge request

    approved this merge request

  • :wave: @dfrazao-gitlab, thanks for approving this merge request.

    This is the first time the merge request is approved. To ensure full test coverage, a new pipeline has been started.

    For more info, please refer to the following links:

  • Patrick Bair approved this merge request

    approved this merge request

  • Patrick Bair resolved all threads

    resolved all threads

  • Patrick Bair enabled an automatic merge when the pipeline for 292bf16c succeeds

    enabled an automatic merge when the pipeline for 292bf16c succeeds

  • Patrick Bair aborted the automatic merge because source branch was updated

    aborted the automatic merge because source branch was updated

  • Patrick Bair added 416 commits

    added 416 commits

    • 72d4e3b4...62ff6345 - 414 commits from branch master
    • d9708d6c - Remove column ignore rules for migrated PKs
    • 27c9a5e7 - Remove IgnorableColumns module for models that have no ignored columns

    Compare with previous version

  • Patrick Bair enabled an automatic merge when the pipeline for 542d40aa succeeds

    enabled an automatic merge when the pipeline for 542d40aa succeeds

  • merged

  • Patrick Bair mentioned in commit 6f343281

    mentioned in commit 6f343281

  • Krasimir Angelov resolved all threads

    resolved all threads

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading