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.

Sign up for GitHub

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

Jump to bottom

Temporarily disable postgres integration tests #2367

Merged
merged 1 commit into from
Oct 2, 2023
Merged

Temporarily disable postgres integration tests #2367

merged 1 commit into from
Oct 2, 2023

Conversation

pavannd1
Copy link
Contributor

Change Overview

Need to temporarily disable the test to unblock #2363

Pull request type

Please check the type of change your PR introduces:

  • 🚧 Work in Progress
  • 🌈 Refactoring (no functional changes, no api changes)
  • 🐹 Trivial/Minor
  • 🐛 Bugfix
  • 🌻 Feature
  • 🗺️ Documentation
  • 🤖 Test

Issues

  • fixes #issue-number

Test Plan

  • 💪 Manual
  • ⚡ Unit test
  • 💚 E2E

@infraq infraq added this to In Progress in Kanister Sep 29, 2023
@pavannd1 pavannd1 added the kueue label Sep 29, 2023
mabhi
mabhi reviewed Oct 2, 2023
View reviewed changes
Copy link
Contributor

@mabhi mabhi left a comment

Choose a reason for hiding this comment

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

LGTM

Kanister automation moved this from In Progress to Reviewer approved Oct 2, 2023
@mergify mergify bot merged commit 06be966 into master Oct 2, 2023
15 checks passed
Kanister automation moved this from Reviewer approved to Done Oct 2, 2023
@mergify mergify bot deleted the temp-disable-pg-16 branch October 2, 2023 05:44
pavannd1 added a commit that referenced this pull request Oct 2, 2023
@pavannd1
Revert "Temporarily disable Postgres tests (#2367)"
12eb869 
This reverts commit 06be966.
mergify bot added a commit that referenced this pull request Oct 4, 2023
@pavannd1 @akankshakumari393 @mergify
Re-enable Postgres tests and remove MariaDB (#2370)
3eeb8d8 
* Revert "Temporarily disable Postgres tests (#2367)"

This reverts commit 06be966.

* Disable mysql app from integration test

Signed-off-by: Akanksha Kumari <akanksha.kumari@veeam.com>

* Revert "Disable mysql app from integration test"

This reverts commit 7eb9ec4.

* Remove Maria DB

---------

Signed-off-by: Akanksha Kumari <akanksha.kumari@veeam.com>
Co-authored-by: Akanksha Kumari <akanksha.kumari@veeam.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
leuyentran pushed a commit that referenced this pull request Oct 18, 2023
@pavannd1
Temporarily disable Postgres tests (#2367)
c3a257f
leuyentran pushed a commit that referenced this pull request Oct 18, 2023
@pavannd1 @akankshakumari393 @mergify
Re-enable Postgres tests and remove MariaDB (#2370)
70de405 
* Revert "Temporarily disable Postgres tests (#2367)"

This reverts commit 06be966.

* Disable mysql app from integration test

Signed-off-by: Akanksha Kumari <akanksha.kumari@veeam.com>

* Revert "Disable mysql app from integration test"

This reverts commit 7eb9ec4.

* Remove Maria DB

---------

Signed-off-by: Akanksha Kumari <akanksha.kumari@veeam.com>
Co-authored-by: Akanksha Kumari <akanksha.kumari@veeam.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mabhi mabhi mabhi left review comments

@r4rajat r4rajat r4rajat approved these changes

@redgoat650 redgoat650 Awaiting requested review from redgoat650

@julio-lopez julio-lopez Awaiting requested review from julio-lopez

@akankshakumari393 akankshakumari393 Awaiting requested review from akankshakumari393

Assignees
No one assigned
Labels
kueue
Projects
Kanister
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@pavannd1 @mabhi @r4rajat