Skip to content

[Feat] Document agreed RASI chart #19

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 9 commits into from
Jul 15, 2021
Merged

[Feat] Document agreed RASI chart #19

merged 9 commits into from
Jul 15, 2021

Conversation

@gmacario
Copy link
Collaborator

@gmacario gmacario commented Jul 15, 2021

Fix #9

gmacario added 6 commits July 15, 2021 15:40
@gmacario
docs: First attempt at RASI chart
4abc5ed 
See #9 (comment)

Signed-off-by: Macario Gianpaolo <[email protected]>
@gmacario
Add link to Wikipedia page about RASI
0d18b89 
Signed-off-by: Macario Gianpaolo <[email protected]>
@gmacario
Format codes and descriptions into a table
6573175 
Signed-off-by: Macario Gianpaolo <[email protected]>
@gmacario
Better formatting of RASI table
16818f3 
Signed-off-by: Macario Gianpaolo <[email protected]>
@gmacario
Convert RACI table into RASI
deafea8 
See description of #9

Signed-off-by: Macario Gianpaolo <[email protected]>
@gmacario
Normalized task names and spelling
08dedef 
Signed-off-by: Macario Gianpaolo <[email protected]>
@gmacario gmacario requested review from GGn0, Raffone17, alv67 and gteti July 15, 2021 13:52
@gmacario gmacario added the help wanted Extra attention is needed label Jul 15, 2021
@gmacario gmacario added this to the First deployment to DO milestone Jul 15, 2021
@gmacario
Add section title
f5a7c10 
Signed-off-by: Gianpaolo Macario <[email protected]>
gmacario
gmacario commented Jul 15, 2021
View reviewed changes
docs/rasic.md
|Ops |R| | |I|I|
|Deploy |R| | |I|I|
|Documentation |I| | |I|R|

Copy link
Collaborator Author

@gmacario gmacario Jul 15, 2021

Choose a reason for hiding this comment

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

I do not think that @gteti should take an "R".

I may be "R" as long as someone else takes an "S".
@alv67 , @GGn0 ?

alv67
alv67 approved these changes Jul 15, 2021
View reviewed changes
Copy link
Collaborator

@alv67 alv67 left a comment

Choose a reason for hiding this comment

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

Ok, let start with this

@gmacario
Changing RASI for Documentation
0868d53 
As proposed in #19 (comment)

Signed-off-by: Gianpaolo Macario <[email protected]>
@gmacario gmacario mentioned this pull request Jul 15, 2021
Closed
@gmacario gmacario self-assigned this Jul 15, 2021
@gmacario
Copy link
Collaborator Author

gmacario commented Jul 15, 2021

Unless I hear any disagreements before today 23:59 CEST I plan to merge the PR as-is.

@gteti
Copy link
Member

gteti commented Jul 15, 2021

Okey dokey

@gmacario gmacario merged commit 39c8384 into main Jul 15, 2021
@gmacario gmacario deleted the feat/9-define-rasic branch July 15, 2021 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alv67 alv67 alv67 approved these changes

@GGn0 GGn0 GGn0 approved these changes

@gteti gteti gteti approved these changes

@Raffone17 Raffone17 Awaiting requested review from Raffone17

Assignees

@gmacario gmacario

Labels

help wanted Extra attention is needed

Projects

None yet

Milestone

M1. First deployment to Digital Ocean

Development

Successfully merging this pull request may close these issues.

Define RASI Chart

5 participants

@gmacario @gteti @alv67 @GGn0