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

User-configurable stack name in init-odkx-sync-endpoint.py #194

Open
linl33 opened this issue Mar 5, 2021 · 16 comments
Open

User-configurable stack name in init-odkx-sync-endpoint.py #194

linl33 opened this issue Mar 5, 2021 · 16 comments

Comments

@linl33
Copy link
Member

linl33 commented Mar 5, 2021

init-odkx-sync-endpoint.py hard-coded the stack name syncldap. A user-configurable stack name would be useful.

Repository https://github.com/odk-x/sync-endpoint-default-setup

@adityaofficial10
Copy link

adityaofficial10 commented Mar 5, 2021

Hey!
@linl33
So we need to ask the user for the stack name right?
Can I work on this issue?

@linl33
Copy link
Member Author

linl33 commented Mar 5, 2021

Hi @adityaofficial10!

Yes, you can work on this issue.
The idea is that the script should prompt for a stack name but default to syncldap (the current name). The script should also store the chosen stack name to its settings file.

@adityaofficial10
Copy link

@linl33
Sure, I'll follow this up with a PR shortly.

@adityaofficial10
Copy link

@linl33
I have made the changes as shown below:
Screenshot 2021-03-05 at 11 59 35 PM
Is this fine?
Also, you were talking about the settings file.
Which file is that exactly?

@linl33
Copy link
Member Author

linl33 commented Mar 6, 2021

The function is correct but a few notes on the implementation,

  • Please move the prompt into run_interactive_config, to keep it consistent with the rest of the configuration
  • Please put the default value into [()], e.g. Please enter a name for the Docker Swarm stack [(syncldap)]
  • Please use String interpolation instead of concatenation

For the settings file, see https://github.com/odk-x/sync-endpoint-default-setup/blob/0d97f322b6f3c48f86cdb47c9c1c7acf3f7bfb93/init-odkx-sync-endpoint.py#L134

Let me know if you have questions. Thanks for the work so far!

@adityaofficial10
Copy link

adityaofficial10 commented Mar 6, 2021

@linl33
I've made the required changes.
I'm sending you a link to the file in my fork. Kindly review and inform if I can improve something.
https://github.com/adityaofficial10/sync-endpoint-default-setup/blob/fix/stack-name-added/init-odkx-sync-endpoint.py
Can I open a PR?

@linl33
Copy link
Member Author

linl33 commented Mar 7, 2021

Yes, please send a pull request against the development branch.

@adityaofficial10
Copy link

@linl33
I've linked the PR.

@wbrunette wbrunette added this to the Release Version 2.1.8 milestone Mar 9, 2021
@wbrunette wbrunette removed this from the Release Version 2.1.8 milestone Apr 11, 2021
@wbrunette
Copy link
Member

For future reference for someone taking on this issue. Take a look at the previous incomplete PR. odk-x/sync-endpoint-default-setup#26

The comments in the PR give additional direction

@wsmmxmm
Copy link

wsmmxmm commented Mar 29, 2022

Hi team! I am an outreachy applicant. Please can I work on this issue if it is still open?

@wbrunette
Copy link
Member

@wsmmxmm it is still open. I will assign it to you. Check out the PR I commented about as much of the work has been attempted once, just not fully completed.

@Ndacyayisenga-droid
Copy link

Ndacyayisenga-droid commented Feb 12, 2023

@wbrunette @linl33 @elmps2018
Can I finish up with this if its still open?
Regards,
Noah

@MariaAngella
Copy link

Hello team,

Can I take this on please, I'm an Outreachy applicant.

Thank you and regards,
Maria Angella

@elmps2018
Copy link

Hi @MariaAngella! @wsmmxmm is currently assigned but does not look like they worked on it lately. @wbrunette @wsmmxmm is it okay if I reassign?

@wbrunette
Copy link
Member

You should first ask the question if they want to have a custom stack name. Because if they change it the directions will not match. So really it's two questions.

@MariaAngella
Copy link

Thank you for this assignment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants