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

[Agent Upgrade]: Under Maintenance window available Immediately option is not displaying for bulk agent upgrade when agent is <=10. #133113

Closed
ghost opened this issue May 30, 2022 · 13 comments · Fixed by #133369
Assignees
Labels
bug Fixes for quality problems that affect the customer experience impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. QA:Validated Issue has been validated by QA Team:Fleet Team label for Observability Data Collection Fleet team

Comments

@ghost
Copy link

ghost commented May 30, 2022

Describe the bug:
Under Maintenance window available Immediately option is not displaying for bulk agent upgrade when agent is <=10 agent.

Build Details:
VERSION: 8.3.0-SNAPSHOT
COMMIT: c993ff2
BUILD: 53035
ARTIFACT LINK: https://snapshots.elastic.co/8.3.0-a497a6ef/summary-8.3.0-SNAPSHOT.html

Preconditions:

  1. Elastic 8.3.0-SNAPSHOT environment should be deployed.

Steps to Reproduce:

  1. Navigate to the Fleet tab under Management.
  2. Now, under Agents tab select the Agents which are available for upgrade. (Agents should be less then <=10)
  3. Click on Action button and select the upgrade version and upgrade Maintenance window available for the upgrade.
  4. Observe that under Maintenance window available Immediately option is not displaying for bulk agent upgrade when agent is <=10 agent.

Expected Result:
Under Maintenance window available Immediately option should display for bulk agent upgrade when agent is <=10 agent.

What's working:
Previously Immediately option was displaying for bulk agent upgrade if agent is <=10.
Capture4
Capture2

Screenshot:
Capture
Capture1

Refrenece ticket:
#132421

@ghost ghost added bug Fixes for quality problems that affect the customer experience impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. Team:Fleet Team label for Observability Data Collection Fleet team labels May 30, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@ghost
Copy link
Author

ghost commented May 30, 2022

@manishgupta-qasource Please review.

@manishgupta-qasource
Copy link

Secondary review for this ticket is Done

@ghost ghost changed the title [Agent Upgrade]: Under Maintenance window available Immediately option is not displaying for bulk agent upgrade when agent is <=10 agent. [Agent Upgrade]: Under Maintenance window available Immediately option is not displaying for bulk agent upgrade when agent is <=10. May 30, 2022
@jen-huang
Copy link
Contributor

@nchaulet @criamico Could you clarify if this is expected behavior?

@nchaulet
Copy link
Member

nchaulet commented Jun 1, 2022

Re-reading the slack discussion where we decided to add Immediatly, I think it should be available even when we have more than 10 agents, but it should be the default when we have less than 10 agents.

@criamico
Copy link
Contributor

criamico commented Jun 1, 2022

@nchaulet I think that the reason why we decided to have it available only under 10 agents was to avoid that the user could use it when trying to update a large number of agents. I would still keep it hidden under a certain threshold for this reason, but it should definitely be there if <=10 agents.

@nchaulet
Copy link
Member

nchaulet commented Jun 1, 2022

@nchaulet I think that the reason why we decided to have it available only under 10 agents was to avoid that the user could use it when trying to update a large number of agents. I would still keep it hidden under a certain threshold for this reason, but it should definitely be there if <=10 agents.

From the slack discussion we had in the fleet channel it seems we did not talk about hiding this, but it's not clear :), I think it make sense to allow user to do it immediately even with more than 10 agents, but immediately should not be the default in this case @nimarezainia @ph what do you think?

@jen-huang
Copy link
Contributor

jen-huang commented Jun 1, 2022

@criamico @nchaulet I believe this bug report is that this option is not showing even when agent count is <= 10. It sounds like we expect the Immediate option to be present for a small amount of agents, so could one of you try to replicate this isue?

@nchaulet
Copy link
Member

nchaulet commented Jun 1, 2022

@jen-huang I just tested and the option is available for agents <= 10 we just need to decide if that option should be available for more agents

@jen-huang
Copy link
Contributor

@nchaulet So this bug is invalid? Did you get to test on Cloud?

@nimarezainia
Copy link
Contributor

@nchaulet the decision was to set it to "immediately" as the default when the agent count is less than 10. We never discussed forcing that by hiding the immediately option. But I think by hiding it we are saying it is enforced.

Could we at least show the option? because the way it is right now for <10 agents we are asking the user to choose a window of time to do it in, which is wrong.

Let's do this:

  1. When agent =< 10 : default to immediately in the drop down. User can choose if they want a window
  2. When agent > 10 : default to 1hr in the drop down. User can choose if they want it immediately or a bigger window

question is whether we have this flexibility in the drop down.

(fyi @joshdover )

@nchaulet
Copy link
Member

nchaulet commented Jun 2, 2022

@nchaulet So this bug is invalid? Did you get to test on Cloud?

I was finally able to reproduce it by selecting all agents, and not individually some agents. I am going to work on this:

  • fixing the bug that should set Immediately as the default for less than 10 agents
  • showing Immediately option for agent > 10

@nchaulet nchaulet self-assigned this Jun 2, 2022
@jen-huang jen-huang added the QA:Ready for Testing Code is merged and ready for QA to validate label Jun 7, 2022
@ghost
Copy link
Author

ghost commented Jun 8, 2022

Hi @nchaulet
We have validated this issue on the latest 8.3.0-SNAPSHOT build and observed that the issue is fixed. Please find below the testing details:

  • Under Maintenance window available Immediately option is displaying for bulk agent upgrade when agent is <=10 agent.

Screen-Recording:

Agents.-.Fleet.-.Elastic.-.Google.Chrome.2022-06-08.11-43-48.mp4

Build Details:
VERSION: 8.3.0-SNAPSHOT
COMMIT: 6ef4570
BUILD: 53295
ARTIFACT LINK: https://snapshots.elastic.co/8.3.0-b94e381c/summary-8.3.0-SNAPSHOT.html

Hence, marking this issue as QA Validated.

Thanks!

@ghost ghost added QA:Validated Issue has been validated by QA and removed QA:Ready for Testing Code is merged and ready for QA to validate labels Jun 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. QA:Validated Issue has been validated by QA Team:Fleet Team label for Observability Data Collection Fleet team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants