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

point release v1.91.2 #6466

Open
wants to merge 4 commits into
base: release-v1.91
Choose a base branch
from
Open

point release v1.91.2 #6466

wants to merge 4 commits into from
+14 −7

Conversation

andriikotko
Copy link
Contributor

What:

Why:
cherry-pick bugfixes for release v1.91
Please describe the tests:

  • Test 1:
  • Test 2:

Please describe the performance impact:

Code Review Checklist (to be filled out by reviewer)

  • NEW: Are there any Satellite database migrations? Are they forwards and backwards compatible?
  • Does the PR describe what changes are being made?
  • Does the PR describe why the changes are being made?
  • Does the code follow our style guide?
  • Does the code follow our testing guide?
  • Is the PR appropriately sized? (If it could be broken into smaller PRs it should be)
  • Does the new code have enough tests? (every PR should have tests or justification otherwise. Bug-fix PRs especially)
  • Does the new code have enough documentation that answers "how do I use it?" and "what does it do?"? (both source documentation and higher level, diagrams?)
  • Does any documentation need updating?
  • Do the database access patterns make sense?

VitaliiShpital and others added 4 commits October 30, 2023 13:50
@VitaliiShpital @andriikotko
cmd/satellite: copy wasm files into app/static/wasm subfolder
4f575aa 
Fixed docker command to copy files to app/static/wasm subfolder instead of app/static itself.

Change-Id: I50e605b9224d2c643cd5f6ae62a3610ed2a06752
@VitaliiShpital @andriikotko
web/satellite: don't make billing API requests if billing is disabled
09dbd04 
Removed billing API calls on all project dashboard if billing is disabled.

Change-Id: Ia3407c811e599f40670b97e1c2ad9a8f661c0e2a
@VitaliiShpital @andriikotko
web/satellite: update low token balance banner copy
e08d63d 
Updated the copy for low token balance to correspond with a new use-case.

Change-Id: I1e08a039849c2950cdc4b8dc3b4c9f470461b0dd
@thepaul @andriikotko
cmd/tools/segment-verify: fix global race in TestCommandLineTool.
aa14992 
This should stop the failures when TestCommandLineTool is run with
parallel test execution enabled.

Change-Id: Id80d5eacb78fcec886be786ae8b182517b17fbc6
@cla-bot cla-bot bot added the cla-signed label Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@egonelbre egonelbre egonelbre approved these changes

@mniewrzal mniewrzal mniewrzal approved these changes

@VitaliiShpital VitaliiShpital VitaliiShpital approved these changes

@ihaid ihaid Awaiting requested review from ihaid

@neo-cypher neo-cypher Awaiting requested review from neo-cypher

@amwolff amwolff Awaiting requested review from amwolff

@adaRn adaRn Awaiting requested review from adaRn

Assignees
No one assigned
Labels
cla-signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@andriikotko @egonelbre @mniewrzal @VitaliiShpital @thepaul