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 EGI pulsar endpoint #1

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

sebastian-luna-valero
Copy link
Contributor

Based on: usegalaxy-eu/infrastructure-playbook#872

Please configure the same credentials as the ones used for the production instance of Galaxy.

@sanjaysrikakulam
Copy link
Owner

Hey Sebastian!

A pulsar instance can only be connected to one Galaxy instance. If you would like to move your short-lived pulsar test instance to this one then I will share with you the mq server details and a new password.

@sebastian-luna-valero
Copy link
Contributor Author

Hi Sanjay!

ok, let's have separate credentials for this test instance of Galaxy.

Thanks!

@sebastian-luna-valero
Copy link
Contributor Author

xref: sebastian-luna-valero#1

@sebastian-luna-valero
Copy link
Contributor Author

Here is my plan for testing the TPV API metascheduler.

Please let me know if you agree (when you get back from your holidays) and we proceed exchanging credentials.

@bgruening
Copy link
Collaborator

bgruening commented Dec 15, 2023

Please ignore this comment when I understand something wrong.
For me, the idea of the endpoint was that we can test the endpoint simply via CURL, without Galaxy. We can through all the different parameters (fake S3, fake Pulsar geo locations etc) and should get a meaningful destination back.

This gives us a lot of flexibility, without having real deployments of all of that and we can iterate much faster. So no one should ever feel blocked.

Of course in the end we want to have those real deployments, but for simple testing of the API this should not be needed. Imho. Does that make sense?

@bgruening
Copy link
Collaborator

There is also a merge conflict here in this PR.

@sebastian-luna-valero
Copy link
Contributor Author

Hi @bgruening

Thanks for your feedback! Yes, it makes absolute sense.

I do like the idea of the standalone metascheduler endpoint to avoid blocking other people/activities, but I also see a benefit on integrating it now into the ESG Galaxy instance to confirm that everything works as expected and iterate fast otherwise to avoid last minute surprises down the road.

Happy to pause this activity if you think it's the best option at the moment.

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