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

[Fleet] Without clicking on upgrade package button, Integration gets updated by itself on just adding it to a policy. #99649

Closed
dikshachauhan-qasource opened this issue May 10, 2021 · 6 comments · Fixed by #101542
Assignees
Labels
bug Fixes for quality problems that affect the customer experience Team:Fleet Team label for Observability Data Collection Fleet team

Comments

@dikshachauhan-qasource
Copy link

Kibana version:
7.13 BC4 Kibana Cloud QA environment

Host OS and Browser version:
Integrations, All

Preconditions

  1. 7.13 BC4 Kibana Cloud QA environment should be available.
Kibana version: 7.13 BC4
BUILD 40749
COMMIT 5a6bad454ffe263aafed54cbd3f764253694bf37
Artifact link: https://staging.elastic.co/7.13.0-5c4bc719/summary-7.13.0.html
  1. Managed host should be available under Agents tab.

Steps to reproduce:

  1. Login to Kibana cloud environment.
  2. Go to Integrations tab and click on Nginx integration
  3. Use below API to see the indicator next to the version number that the package is outdated.
    curl -X POST -u $USER:$PASSWORD http://$KIBANA_URL:$KIBANA_PORT/api/fleet/epm/packages/nginx-0.3.7 -H 'kbn-xsrf: xyz' -H "Content-Type: application/json" -d '{"force": true}'
  4. Navigate to the Settings Tab and verify that you see a button to upgrade the package.
  5. Without clicking on it, add Nginx integration to any policy.
  6. Navigate back to settings page.
  7. Observe,

Reference Link:
#93712

Actual Result:

  • Adding integration to a policy makes it upgrade to latest package version available.
  • Without clicking on button to upgrade the package, Integration gets updated by itself on just adding it to a policy.

Screenshots:
image

Expected result:
Package should be updated to latest package version available after clicking on button to upgrade the package.

@elasticmachine
Copy link
Contributor

Pinging @elastic/integrations (Team:Integrations)

@dikshachauhan-qasource
Copy link
Author

@manishgupta-qasource Please review

@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@EricDavisX EricDavisX transferred this issue from elastic/integrations May 10, 2021
@EricDavisX EricDavisX added the Team:Fleet Team label for Observability Data Collection Fleet team label May 10, 2021
@EricDavisX
Copy link
Contributor

@jen-huang I moved this from "integrations' repo, believe it is on Fleet team Kibana side to review. I don't think we've changed the spec here, unless I'm out of date. It is interesting... and may end up as a 'good thing' that it is doing? Tagging @mostlyjason to confirm the spec / intention around it too, we may just need to update test steps and documentation?

@ruflin
Copy link
Member

ruflin commented May 11, 2021

This sounds indeed like a bug and I would argue is not the expected behaviour.

@ruflin ruflin added bug Fixes for quality problems that affect the customer experience v7.13.0 labels May 11, 2021
@jen-huang
Copy link
Contributor

Yep it's a bug, due to the different code path that the two install paths take :(

@jen-huang jen-huang changed the title Without clicking on upgrade package button, Integration gets updated by itself on just adding it to a policy. [Fleet] Without clicking on upgrade package button, Integration gets updated by itself on just adding it to a policy. May 11, 2021
@jen-huang jen-huang removed their assignment May 11, 2021
@jen-huang jen-huang removed the v7.13.0 label May 11, 2021
@jen-huang jen-huang self-assigned this May 19, 2021
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 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