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

Simpler checkin endpoint(s) #170

Closed
jasontibbitts opened this issue May 27, 2016 · 4 comments
Closed

Simpler checkin endpoint(s) #170

jasontibbitts opened this issue May 27, 2016 · 4 comments

Comments

@jasontibbitts
Copy link

I would like to be able to do a checkin from a shell script. I know the current checkin xmlrpc checkin endpoint is there for compatibility with existing report_mirror checkins, but it would be nice to have something that's a bit simpler to use.

I propose to add two API endpoints to api.py: checkin-json and checkin-text. These will take json or a text file and parse that into a structure suitable for passing to read_host_config. Compressed endpoints could be added, too. The json should be trivial to generate from the regular report_mirror data structure that gets pickled. The text file should be easy to generate from a shell script.

Given the fact that you should really shouldn't unpickle untrusted data, I'd figure that this would be something you wouldn't mind adding. I think that doing this should be pretty easy for me, but I'm not sure how I'm going to test it. Any hints as to how you'd want to see this done would be appreciated, of course.

@adrianreber
Copy link
Member

The new interfaces you propose sound reasonable. Not entirely sure what to expect from checkin-text. Compressing the transmitted data would be nice I think.

Testing it can be done using the staging instance I would say.

@jasontibbitts
Copy link
Author

So puiterwijk add a json checkin mechanism to the regular checkin endpoint as part of getting rid of the unpickle operation. He also indicated that he would go ahead an add a regular json submission endpoint that doesn't involve dealing with xmlrpc. (And also add a way to send gzipped data instead of requiring bzip2.) So I'm going to see if it's workable to just generate the json directly in the quick-fedora-mirror shell script and go with that. If not, then I'll work on some format.

@adrianreber
Copy link
Member

I guess this is resolved with #173

@jasontibbitts
Copy link
Author

Not 100%, since puiterwijk was going to add a regular, non-xmlrpc endpoint for me to use. I can follow up with him on that.

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

No branches or pull requests

2 participants