Skip to content

fix(ironic): allow lessee to see the last error in a build #635

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
Merged
merged 1 commit into from
Feb 20, 2025
Merged

fix(ironic): allow lessee to see the last error in a build #635

merged 1 commit into from
Feb 20, 2025

Conversation

nicholaskuechler
Copy link
Collaborator

@nicholaskuechler nicholaskuechler commented Jan 24, 2025 •
edited by cardoe
Loading

Issues we're seeing:

{'code': 500, 'created': '2025-01-23T15:24:50Z', 'message': 'Build of instance
953691fd-1dde-4b01-9010-e0d45d1a9dfa aborted: Failed to provision instance
953691fd-1dde-4b01-9010-e0d45d1a9dfa: ** Value Redacted - Requires
baremetal:node:get:last_error permission. **'}

@nicholaskuechler nicholaskuechler marked this pull request as ready for review January 27, 2025 21:56
skrobul
skrobul reviewed Jan 30, 2025
View reviewed changes
@nicholaskuechler nicholaskuechler force-pushed the PUC-698-20250124-1 branch 2 times, most recently from 409e4d5 to fda8b5f Compare January 30, 2025 22:25
cardoe
cardoe reviewed Jan 31, 2025
View reviewed changes
@cardoe cardoe changed the title fix(ironic): Update ironic policies for better integration with other OpenStack services fix(ironic): allow lessee to see the last error in a build Feb 20, 2025
@cardoe cardoe added this pull request to the merge queue Feb 20, 2025
Merged via the queue into main with commit 0b070c0 Feb 20, 2025
16 checks passed
@cardoe cardoe deleted the PUC-698-20250124-1 branch February 20, 2025 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@skrobul skrobul skrobul left review comments

@cardoe cardoe cardoe approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nicholaskuechler @cardoe @skrobul