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.

Sign up for GitHub

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

Jump to bottom

[patch] release v1.7.5 #2013

Merged
merged 1 commit into from
Apr 25, 2023
Merged

[patch] release v1.7.5 #2013

merged 1 commit into from
Apr 25, 2023

Conversation

ykadowak
Copy link
Contributor

@ykadowak ykadowak commented Apr 24, 2023 •
edited by vankichi
Loading

✨ New features

  • Add index with timestamp #1984

♻️ Refactor

  • Improve errors.Join performance #2010
  • Refactor error returning timing of doSearch function #1996
  • Refactor makefile for non GOPATH strict environment 🚨 Fix lint. #2 #1998
  • Refactor makefile for non GOPATH strict environment #1997

🐛 Bugfix

  • Correction of a bug that returned NotFound as success 0 when balancedUpdate is disabled and all ReplicaAgents are AlreadyExists (already have the exact same Index). #2011
  • Refactor replace errors wrap with join #2001
  • Remove nvimlog #1994

💚 CI

  • Fix chatops format workflow #2007
  • Fix incorrect error output of gen-test chatopts command #2004
  • Fix Makefile bug and update deps for checking bugfix #2002
  • Output error to chatops comment #1999
  • Fix ChatOpts /gen-test command error #1993

✏️ Document

  • Update unit test guideline for unimplemented test #1983

✅ Testing

  • Implement generic function tests #2008
  • Generate empty test using /gen-test ChatOpts command #2005
  • Update internal/info test and add new case for coverage #2003

⬆️ Update dependencies

  • Update go module and libs #2012

🤝 Contributor

  • Add ykadowak as a contributor for code, and test #2009

@ykadowak
[patch] release v1.7.5
468530e 
Signed-off-by: ykadowak <yusuke.kadowaki.1231@gmail.com>
@vdaas-ci
Copy link
Collaborator

[CHATOPS:HELP] ChatOps commands.

  • 🙆‍♀️ /approve - approve
  • 💌 /changelog - replace the PR body by changelog details
  • 🍱 /format - format codes and add licenses
  • ✅ /gen-test - generate test codes
  • 🏷️ /label - add labels
  • ⏪ /rebase - rebase main
  • 🔚 2️⃣ 🔚 /label actions/e2e-deploy - run E2E deploy & integration test

@github-actions
Copy link
Contributor

[FOSSA] The scan result will be available at https://app.fossa.com/projects/custom%2B21465%2Fvald/refs/branch/release%2Fv1.7.5/a575e391a72e098ba9e0021c81cf18f1f8c2f407

@vdaas-ci
Copy link
Collaborator

Profile Report

typevald-agent-ngtvald-lb-gatewayvald-discoverervald-manager-index
cpu
heap
 other images

hlts2
hlts2 approved these changes Apr 25, 2023
View reviewed changes
Copy link
Contributor

@hlts2 hlts2 left a comment

Choose a reason for hiding this comment

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

LGTM!

@kpango kpango merged commit 80ff64b into main Apr 25, 2023
@kpango kpango deleted the release/v1.7.5 branch April 25, 2023 02:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kevindiu kevindiu kevindiu approved these changes

@kpango kpango kpango approved these changes

@vankichi vankichi vankichi approved these changes

@hlts2 hlts2 hlts2 approved these changes

@datelier datelier datelier approved these changes

Assignees
No one assigned
Labels
actions/e2e-chaos actions/e2e-deploy actions/e2e-max-dim actions/e2e-profiling actions/fossa priority/low size/S
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@ykadowak @vdaas-ci @kevindiu @kpango @vankichi @hlts2 @datelier