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

Re-enable Postgres tests and remove MariaDB #2370

Merged
merged 5 commits into from
Oct 4, 2023

Conversation

pavannd1
Copy link
Contributor

@pavannd1 pavannd1 commented Oct 2, 2023

This reverts commit 06be966.

Additionally, this PR removes MariaDB tests. We will add it back once we have enough resources to run all the tests.

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

@infraq infraq added this to In Progress in Kanister Oct 2, 2023
@pavannd1
Copy link
Contributor Author

pavannd1 commented Oct 2, 2023

Looks like the mongo-db tests are failing now.

Signed-off-by: Akanksha Kumari <akanksha.kumari@veeam.com>
@akankshakumari393
Copy link
Contributor

if we disable one of the app, the integration test works, I tried disabling MySQL, the test passed commit . https://github.com/kanisterio/kanister/actions/runs/6390938725/job/17345776852, maybe we should upgrade the test infrastructure.

@akankshakumari393
Copy link
Contributor

I have reverted the commit, that I made in this PR to test the CI.

build/integration-test.sh Outdated Show resolved Hide resolved
@pavannd1 pavannd1 changed the title Re-enable Postgres tests Re-enable Postgres tests and remove MariaDB Oct 3, 2023
@pavannd1 pavannd1 added the kueue label Oct 3, 2023
Kanister automation moved this from In Progress to Reviewer approved Oct 4, 2023
@mergify mergify bot merged commit 3eeb8d8 into master Oct 4, 2023
14 checks passed
Kanister automation moved this from Reviewer approved to Done Oct 4, 2023
@mergify mergify bot deleted the revert-2367-temp-disable-pg-16 branch October 4, 2023 03:17
leuyentran pushed a commit that referenced this pull request Oct 18, 2023
* 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
Labels
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants