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

cm add script is failing on new CM repository #1252

Closed
arjunsuresh opened this issue May 22, 2024 · 10 comments
Closed

cm add script is failing on new CM repository #1252

arjunsuresh opened this issue May 22, 2024 · 10 comments
Labels

Comments

@arjunsuresh
Copy link
Contributor

arjun@arjun-spr:~/CM/repos/arjunsuresh@personal-configs/script$ cm add script .:set-arjun-configs --tags=arjun,configs,set

CM error: CM object arjunsuresh@personal-configs,9b2008637a5b4caa:.:set-arjun-configs is not recognized!
@anandhu-eng
Copy link
Contributor

Hi, same issue here.

anandhu@anandhu-VivoBook-ASUSLaptop-X515UA-M515UA:~/CM/repos/mlcommons@cm4mlops$ cm add script .:hello-world --tags=hello-world,display,test

CM error: CM object mlcommons@cm4mlops,9e97bb72b0474657:.:hello-world is not recognized!

@arjunsuresh
Copy link
Contributor Author

Hi @gfursin I believe this is the most common command for anyone to start using CM in their own CM repository. Do you have any suggestion how to do this?

@gfursin
Copy link
Contributor

gfursin commented Sep 16, 2024

Hi @anandhu-eng and @arjunsuresh .

I have found that the "add script" was not working properly with repositories and I tried to fixed it on Friday but I think I did a mistake somewhere and it was not caught by the tests.

I will try to fix it today ;) ...

Thanks!

@arjunsuresh
Copy link
Contributor Author

Thanks a lot @gfursin . I added this repository to CM to automate the pipeline.

@gfursin
Copy link
Contributor

gfursin commented Sep 16, 2024

I believe I fixed "cm add script" and committed to "dev" branch. Can you please check it? Thanks!

@arjunsuresh
Copy link
Contributor Author

Thank you @gfursin I added a test for this as its easier to test on github. Can you please check and merge this?
#1285

@arjunsuresh
Copy link
Contributor Author

Hi @gfursin I don't see any changes in dev 😕 The below test commands are fine right?

https://github.com/mlcommons/ck/actions/runs/10901463603/job/30251271963?pr=1286

@gfursin
Copy link
Contributor

gfursin commented Sep 17, 2024

Hi @gfursin I don't see any changes in dev 😕 The below test commands are fine right?

https://github.com/mlcommons/ck/actions/runs/10901463603/job/30251271963?pr=1286

The change was in our cm4mlops repo in automation:script (add function)

@arjunsuresh
Copy link
Contributor Author

Thank you @gfursin . I have added some more test commands and they are working fine on Ubuntu

@arjunsuresh
Copy link
Contributor Author

Working fine now :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

No branches or pull requests

3 participants