Skip to content

[tests-only][full-ci][k8s] test: run apiNotification test suite on k8s deployment - #12922

Merged
ishabaral merged 1 commit into
owncloud:masterfrom
Ashim-Stha:test/apiNotifications-k8
Sep 16, 2026
Merged

ishabaral merged 1 commit into
owncloud:masterfrom
Ashim-Stha:test/apiNotifications-k8

Conversation

@Ashim-Stha

@Ashim-Stha Ashim-Stha commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor

Description

apiNotification tests were failing due to the missing translations in the k8s deployment, de translation for word Ansehen differed in the k8s causing related tests to fail see, so created the configMap for the translations, and defined volume from the configMap and then mounted the translations for notifications in it.

Related Issue

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

@kw-security

kw-security commented Sep 10, 2026 •

Copy link
Copy Markdown

✅ Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues
✅ Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@Ashim-Stha
Ashim-Stha force-pushed the test/apiNotifications-k8 branch 7 times, most recently from adcd257 to e3fc2a2 Compare September 14, 2026 04:10
@Ashim-Stha
Ashim-Stha marked this pull request as ready for review September 14, 2026 04:23
@Ashim-Stha
Ashim-Stha requested a review from a team as a code owner September 14, 2026 04:24
PrajwolAmatya
PrajwolAmatya previously approved these changes Sep 14, 2026

@PrajwolAmatya PrajwolAmatya left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Comment thread tests/config/k8s/templates/extra.tpl Outdated
Signed-off-by: Ashim Shrestha <ashimshrestha2384@gmail.com>
@Ashim-Stha
Ashim-Stha force-pushed the test/apiNotifications-k8 branch from 775fe2c to 217650d Compare September 16, 2026 06:30
@ishabaral
ishabaral merged commit 13583ab into owncloud:master Sep 16, 2026
147 of 150 checks passed
@Ashim-Stha
Ashim-Stha deleted the test/apiNotifications-k8 branch September 16, 2026 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants