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

U/rbovill/dm36343 - Added the integrationtests application #135

Merged
merged 1 commit into from Mar 9, 2023
Merged

U/rbovill/dm36343 - Added the integrationtests application #135

merged 1 commit into from Mar 9, 2023

Conversation

rbovill
Copy link
Contributor

@rbovill rbovill commented Mar 9, 2023

No description provided.

@rbovill rbovill requested a review from mareuter March 9, 2023 21:50
@rbovill rbovill force-pushed the u/rbovill/DM-36343 branch 5 times, most recently from 400e1e1 to 58125b1 Compare March 9, 2023 22:31
mareuter
mareuter approved these changes Mar 9, 2023
View reviewed changes
Copy link
Collaborator

@mareuter mareuter left a comment

Choose a reason for hiding this comment

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

One comment to handle but overall this looks fine as a first pass. I think we need to take a step back and harmonize the names on the robot control scripts and the application test names to make things more uniform, but that can be done separately.

apps/integrationtests/Chart.yaml Outdated Show resolved Hide resolved
@rbovill
TTS: Added the integrationtests application.
b49babb 
Added the configurations for the AT Prepare for Flat and AT and CC Image Taking tests.

Added the values-auxtel-prepare-onsky.yaml app config.

Added the values-love-stress-test.yaml app config.

Added the values-auxtel-latiss-cwfs-align.yaml app config.

Updated the integrationtests app to use the centralized values.yaml file.

Updated README.md.

Chart.yaml clarification.
@rbovill rbovill merged commit 948b054 into main Mar 9, 2023
@rbovill rbovill deleted the u/rbovill/DM-36343 branch March 9, 2023 23:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mareuter mareuter mareuter approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@rbovill @mareuter