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

Avoid removing from brand media relay and application server sets in use #2795

Open
wants to merge 5 commits into
base: bleeding
Choose a base branch
from

Conversation

danigargar
Copy link
Contributor

Add lifecycle services checking if a MediaRelaySet or ApplicationServerSet is in use by a brand before removing the relation.

Type Of Change

  • Small bug fix
  • New feature or enhancement
  • Breaking change

Checklist:

  • Commits are named and have tag following commit rules
  • Commits are split per component (schema, portal/platform, kamusers, agis, ..)
  • Changes have been tested locally
  • Fixes an existing issue (Fixes #XXXX)
  • Upport from existing Pull request #XXXX

Description

Additional information

@danigargar danigargar added +enhancement Enhancement core Core Libraries ci-force-tests-back Run all backend pipeline tests in this Pull Request *tempest* IvozProvider 4.x Tempest Release labels Oct 22, 2024
@danigargar danigargar self-assigned this Oct 22, 2024
@danigargar danigargar changed the base branch from bleeding to PROVIDER-2053-adds-schema-relations-between-entities October 22, 2024 07:42
@danigargar danigargar force-pushed the PROVIDER-2067-avoid-removing-from-brand-sets-in-use branch from aaa4a36 to 21d7c07 Compare October 22, 2024 07:43
@danigargar danigargar force-pushed the PROVIDER-2053-adds-schema-relations-between-entities branch from d21f6ab to 8ee3cbd Compare October 22, 2024 13:44
@danigargar danigargar force-pushed the PROVIDER-2067-avoid-removing-from-brand-sets-in-use branch 6 times, most recently from 8c50290 to 3c1f115 Compare October 22, 2024 16:20
@danigargar danigargar marked this pull request as ready for review October 22, 2024 16:20
Copy link
Contributor

@ironArt3mis ironArt3mis left a comment

Choose a reason for hiding this comment

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

Functional review required

@danigargar danigargar force-pushed the PROVIDER-2053-adds-schema-relations-between-entities branch 2 times, most recently from 0ce89b8 to 5ec744e Compare October 23, 2024 09:21
@danigargar danigargar force-pushed the PROVIDER-2067-avoid-removing-from-brand-sets-in-use branch from 3c1f115 to baf00d6 Compare October 23, 2024 09:34
Base automatically changed from PROVIDER-2053-adds-schema-relations-between-entities to bleeding October 23, 2024 10:27
@danigargar danigargar force-pushed the PROVIDER-2067-avoid-removing-from-brand-sets-in-use branch from baf00d6 to 55b3bca Compare October 23, 2024 10:28
@danigargar danigargar force-pushed the PROVIDER-2067-avoid-removing-from-brand-sets-in-use branch from 55b3bca to 5dab9e5 Compare October 23, 2024 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-force-tests-back Run all backend pipeline tests in this Pull Request core Core Libraries +enhancement Enhancement functional-review *tempest* IvozProvider 4.x Tempest Release
Development

Successfully merging this pull request may close these issues.

3 participants