Skip to content

Document initial placement semantics in architecture and object PUT API#115

Merged
roman-khimov merged 8 commits into
masterfrom
copilot/describe-initial-placement
May 15, 2026
Merged

Document initial placement semantics in architecture and object PUT API#115
roman-khimov merged 8 commits into
masterfrom
copilot/describe-initial-placement

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 14, 2026 •
edited
Loading

  • Inspect the reviewed files and the recent commit history to identify changes that must be reverted or updated
  • Revert the autogenerated API file change and confirm it matches the pre-change state
  • Rewrite the architecture section so initial placement comes after CBF and documents each control clearly
  • Run targeted validation for the documentation changes and review the final diff
  • Reply to the actionable review comments with the addressing commit hashes

Copilot AI linked an issue May 14, 2026 that may be closed by this pull request
Describe initial placement #113
Closed
Copilot AI and others added 6 commits May 14, 2026 15:06
@roman-khimov
Document initial placement semantics
2fc9847 
Agent-Logs-Url: https://github.com/nspcc-dev/neofs-spec/sessions/4a24a865-b2f1-4177-b3d5-d5ec544f3cd3

Co-authored-by: roman-khimov <22092804+roman-khimov@users.noreply.github.com>
@roman-khimov
Clarify initial placement description
d8d422b 
Agent-Logs-Url: https://github.com/nspcc-dev/neofs-spec/sessions/4a24a865-b2f1-4177-b3d5-d5ec544f3cd3

Co-authored-by: roman-khimov <22092804+roman-khimov@users.noreply.github.com>
@roman-khimov
Polish initial placement wording
ba4af57 
Agent-Logs-Url: https://github.com/nspcc-dev/neofs-spec/sessions/4a24a865-b2f1-4177-b3d5-d5ec544f3cd3

Co-authored-by: roman-khimov <22092804+roman-khimov@users.noreply.github.com>
@roman-khimov
Refine initial placement explanation
a2b6e49 
Agent-Logs-Url: https://github.com/nspcc-dev/neofs-spec/sessions/4a24a865-b2f1-4177-b3d5-d5ec544f3cd3

Co-authored-by: roman-khimov <22092804+roman-khimov@users.noreply.github.com>
@roman-khimov
Finalize initial placement docs
2aaba48 
Agent-Logs-Url: https://github.com/nspcc-dev/neofs-spec/sessions/4a24a865-b2f1-4177-b3d5-d5ec544f3cd3

Co-authored-by: roman-khimov <22092804+roman-khimov@users.noreply.github.com>
@roman-khimov
Clarify initial placement constraints
a47772a 
Agent-Logs-Url: https://github.com/nspcc-dev/neofs-spec/sessions/4a24a865-b2f1-4177-b3d5-d5ec544f3cd3

Co-authored-by: roman-khimov <22092804+roman-khimov@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix initial placement as per linked PR Document initial placement semantics in architecture and object PUT API May 14, 2026
Copilot AI requested a review from roman-khimov May 14, 2026 15:16
roman-khimov
roman-khimov reviewed May 14, 2026
View reviewed changes
Comment thread 20-api-v2/object.md Outdated
Comment thread 01-arch/02-policy.md
Comment thread 01-arch/02-policy.md Outdated
@roman-khimov
Copy link
Copy Markdown
Member

roman-khimov commented May 14, 2026

@copilot, please fix review comments

@roman-khimov
Address initial placement review comments
7aa177b 
Agent-Logs-Url: https://github.com/nspcc-dev/neofs-spec/sessions/54eb7d97-a3db-4a1b-a0f1-73f6784b5a94

Co-authored-by: roman-khimov <22092804+roman-khimov@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 14, 2026

@copilot, please fix review comments

Addressed in 7aa177b.

Copilot AI requested a review from roman-khimov May 14, 2026 16:47
roman-khimov
roman-khimov approved these changes May 14, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@roman-khimov roman-khimov left a comment

Choose a reason for hiding this comment

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

OK for now.

@roman-khimov roman-khimov marked this pull request as ready for review May 14, 2026 19:27
@roman-khimov roman-khimov merged commit 37e0eb3 into master May 15, 2026
1 check passed
@roman-khimov roman-khimov deleted the copilot/describe-initial-placement branch May 15, 2026 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@End-rey End-rey End-rey approved these changes

+1 more reviewer

@roman-khimov roman-khimov roman-khimov approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@roman-khimov roman-khimov

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Describe initial placement

3 participants

@roman-khimov @End-rey