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

Update manifests and dashboard deploy script #128

Merged
merged 1 commit into from
Sep 9, 2022

Conversation

sallyom
Copy link
Collaborator

@sallyom sallyom commented Sep 2, 2022

Signed-off-by: Sally O'Malley somalley@redhat.com

For #126

@sallyom sallyom force-pushed the manifests-sa-token branch 3 times, most recently from ae99de4 to 3e9cade Compare September 8, 2022 19:10
@sallyom sallyom changed the title WIP: Update manifests and dashboard deploy script Update manifests and dashboard deploy script Sep 8, 2022
@rootfs
Copy link
Contributor

rootfs commented Sep 8, 2022

@williamcaban can you take a look? thanks

@sallyom
Copy link
Collaborator Author

sallyom commented Sep 8, 2022

(the bash script to define grafana dashboard I'm still working on)

@marceloamaral
Copy link
Collaborator

@sallyom in this case, could you please update the title with the work in progress status? [WIP]

@sallyom sallyom force-pushed the manifests-sa-token branch 2 times, most recently from a6914f8 to 75c6d81 Compare September 8, 2022 20:31
@sallyom
Copy link
Collaborator Author

sallyom commented Sep 8, 2022

@sallyom in this case, could you please update the title with the work in progress status? [WIP]

@marceloamaral (will do) - all seems to be working now, ready for review, thanks!

Copy link
Collaborator

@williamcaban williamcaban left a comment

Choose a reason for hiding this comment

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

Consider the comment for 01-kepler-install.yaml, any of the option should work. The rest LGTM.

@rootfs
Copy link
Contributor

rootfs commented Sep 9, 2022

thank you @williamcaban

Signed-off-by: Sally O'Malley <somalley@redhat.com>
@rootfs
Copy link
Contributor

rootfs commented Sep 9, 2022

thank you @sallyom

@rootfs rootfs merged commit 826e988 into sustainable-computing-io:main Sep 9, 2022
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.

None yet

4 participants