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

updatecli: pass the username to fix the issue with auth #10739

Merged
merged 2 commits into from
Aug 8, 2024

Conversation

jsoriano
Copy link
Member

@jsoriano jsoriano commented Aug 8, 2024

Apply elastic/elastic-package#2009 to this repository.

Pin version v2.64.0
Set username to solve the issues with the authentication needed.
Simplify GitHub action to use a folder with all the pipelines
Restructure the folder layout for the updatecli, updatecli.d for pipelines and values.d for configuration files.

@elasticmachine
Copy link

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@jsoriano jsoriano requested review from v1v and mrodm August 8, 2024 14:38
@jsoriano jsoriano enabled auto-merge (squash) August 8, 2024 14:43
@elasticmachine
Copy link

elasticmachine commented Aug 8, 2024

@jsoriano jsoriano disabled auto-merge August 8, 2024 15:08
@jsoriano jsoriano merged commit ae4f22f into elastic:main Aug 8, 2024
3 of 4 checks passed
@jsoriano jsoriano deleted the updatecli-username branch August 8, 2024 15:09
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

jvalente-salemstate pushed a commit to jvalente-salemstate/integrations that referenced this pull request Aug 21, 2024
Pin version v2.64.0
Set username to solve the issues with the authentication needed.
Simplify GitHub action to use a folder with all the pipelines
Restructure the folder layout for the updatecli, updatecli.d for pipelines and values.d for configuration files.

Co-authored-by: Victor Martinez <[email protected]>
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.

4 participants