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

Add values.yaml for DREAM & ATBuilding on summit #342

Merged
merged 2 commits into from
Nov 20, 2024
Merged

Conversation

bbrondel
Copy link
Contributor

No description provided.

@bbrondel bbrondel requested a review from tribeiro November 12, 2024 19:59
@@ -10,3 +10,6 @@ cscs:
- atspectrograph
- atheaderservice
- pmd1
spec:
source:
targetRevision: tickets/DM-47527
Copy link
Member

Choose a reason for hiding this comment

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

do you really want to commit with the target being a ticket branch? I think probably not. Also, I think this is something you can edit easily on argo-cd

@@ -41,3 +41,6 @@ renameMap:
ess204: auxtel-ess204
ess301: calibhill-ess301
ess302: earthquake-ess302
spec:
source:
targetRevision: tickets/DM-47527
Copy link
Member

Choose a reason for hiding this comment

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

same comment as above.

tribeiro
tribeiro previously approved these changes Nov 13, 2024
@tribeiro tribeiro dismissed their stale review November 13, 2024 15:39

I think I missed something..

Copy link
Member

@tribeiro tribeiro left a comment

Choose a reason for hiding this comment

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

Make sure you add atbuilding to the list of auxtel apps at the summit here, and dream to the list of apps in the obssys apps at the summit here.

@bbrondel bbrondel force-pushed the tickets/DM-47527 branch 2 times, most recently from 0046b3c to 9536f4e Compare November 13, 2024 17:52
@mareuter
Copy link
Collaborator

PR is missing the addition of those CSCs to both the kafka-producers and the love-producer applications for the summit configuration.

@bbrondel
Copy link
Contributor Author

PR is missing the addition of those CSCs to both the kafka-producers and the love-producer applications for the summit configuration.

Thanks for checking. Now fixed.

@bbrondel bbrondel merged commit d14d863 into main Nov 20, 2024
2 checks passed
@bbrondel bbrondel deleted the tickets/DM-47527 branch November 20, 2024 14:50
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