Skip to content

Set domain type in domain resource templates #1135

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

Merged
merged 3 commits into from
May 25, 2022

Conversation

rakillen
Copy link
Member

Hide the spec / configuration / model section unless the domain source type is model-in-image.
Move target configuration validation to target_configuration.py .
Assign default extract target type before processing arguments.

Internal JIRA WDT-613

@rakillen rakillen changed the title WIP: Set domain type in domain resource templates Set domain type in domain resource templates May 25, 2022

def uses_wdt_model(self):
"""
Determine if this configuration uses a WDT model to build the domain.
Copy link
Member

Choose a reason for hiding this comment

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

You might want to clarify this comment. From the way that the code is written, I assume what you really mean is whether operator is using a model to build the domain (since Domain in Image is also using a model to build the domain).

Copy link
Member Author

Choose a reason for hiding this comment

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

Revised that

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@robertpatrick robertpatrick merged commit 39b2b69 into main May 25, 2022
@robertpatrick robertpatrick deleted the JIRA-WDT-613-source-type branch May 25, 2022 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants