Skip to content

Manage Delay Server (main)#168

Draft
ganning127 wants to merge 10 commits intoirods:mainfrom
ganning127:141.main.delay-server
Draft

Manage Delay Server (main)#168
ganning127 wants to merge 10 commits intoirods:mainfrom
ganning127:141.main.delay-server

Conversation

@ganning127
Copy link
Copy Markdown
Contributor

Screenshot 2023-06-26 at 2 24 18 PM Screenshot 2023-06-26 at 2 24 27 PM

Copy link
Copy Markdown
Member

@trel trel left a comment

Choose a reason for hiding this comment

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

good first take.

might also consider populating a dropdown first, rather than having them type the hostname...

and one of the options should be 'No Delay Server'... where it sets it to a known not real name... example.org or something more inspiring/clear.

@ganning127
Copy link
Copy Markdown
Contributor Author

I like the dropdown idea, but I'm not really sure what the pre-populated hostnames would be. Also for "no delay server", would we call iadmin and set something like example.org as the delay server hostname? If i recall correctly, before I set the delay server, the successor value was just a blank string.

Hmm, maybe in this case we should let them just input a blank string?

@trel
Copy link
Copy Markdown
Member

trel commented Jun 26, 2023

We just populate with the same listing that we have for the left sidebar for 'Servers'.

And then yes, we'd just set it to a value that can't be real... and then the Delay Server will not run in the zone.

@trel
Copy link
Copy Markdown
Member

trel commented Jun 26, 2023

You can play with the equivalent iadmin in your own test zone.

@ganning127
Copy link
Copy Markdown
Contributor Author

Screen.Recording.2023-06-26.at.6.29.44.PM.mov

@korydraughn
Copy link
Copy Markdown

Consider changing Change the delay server hostname text above the input box.

This is changing where the delay server runs.

@ganning127
Copy link
Copy Markdown
Contributor Author

Okie, it looks like this now:

Screenshot 2023-06-27 at 10 30 09 AM

@ganning127 ganning127 changed the title Manage Delay Server Manage Delay Server (main) Jun 27, 2023
@ganning127
Copy link
Copy Markdown
Contributor Author

ganning127 commented Jun 27, 2023

EDIT: Fixed.

Happens after selecting "custom hostname" and switching it back to another select option. Need to fix:

Screenshot 2023-06-27 at 10 42 34 AM

@trel
Copy link
Copy Markdown
Member

trel commented Jun 27, 2023

I think we leave this as is for now...

and think about how we get the get_delay_server_info() functionality available in the REST / HTTP API.

@trel trel marked this pull request as draft August 10, 2024 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants