Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
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
Draft PR for review comments of infraclouds contributions #2
base: infracloud/cassandra-5.0
Are you sure you want to change the base?
Draft PR for review comments of infraclouds contributions #2
Changes from all commits
7d2b552
c337a82
96799ad
b90db0a
9e0848e
9d5c38e
6a1cdf6
66b153d
c0b12dc
9f86207
fae1c01
0a5f6af
be11cf6
f4b7897
f5e1ae1
82a1589
f61f449
e1f2333
bb28801
e2cbcbe
108d2b0
24fb78d
ecd22b2
3b5eb23
6ac257f
647c1c1
fcf2a7c
f60f1e8
9e0633f
1556417
7e0d4aa
7f0b241
10d6323
e5565f4
cb2a0b7
2e8d9e1
ae5c475
fdbf9cf
ac463d7
fa3f81a
d7b7e4d
446ca14
58c36f6
431385f
379d325
f1dbace
dcb8726
faac81a
99cc876
61bcad7
82224fe
3051892
499af2f
13ff92f
23bb558
d951683
68df97f
c3830b3
218e983
217c0c3
bbcb26d
08ee5d4
87fb545
1626567
9b10365
006c593
2bff62b
7c48fe3
d486827
245ed22
7b3b36a
2fb9d30
65a8835
bd837ed
886d7eb
c28eb0a
bcba87d
0111793
1e595bc
09283df
c341abc
ec091fe
3d6e2bd
078ca8e
395b9be
f2a2504
ed247ae
059b2c7
fbbbbe4
d6ec04d
0f3e13e
04dde10
62909cb
4ff1bce
0bc5d09
d603628
6e8efd9
c95bd00
5cf5a81
7bd388f
b73b5d0
1adb11c
63694b5
d5db683
e5c45fc
b978768
08189f1
5ca5833
7d35fb3
bafeb6c
b7405df
8e3ffd1
c6ede46
aef256f
3f37ec5
ffd55f6
7e68e82
7de2a55
17a797d
5c0ac21
1753aa3
4148183
ff86cf1
68dceab
7d900e8
291c042
729f8cd
b5adf2f
d822ffe
5cfd2a9
83ba764
1862029
a171ab4
c2d0643
8b2c14d
d08eded
6d9b24e
7cf493c
8b3eb0d
8165818
c2ef545
17e1e36
6e094ac
bc787be
5b0a0d9
0445a3e
9b51d62
77e3f44
420232f
e957b1a
579b71b
0ee7f7a
8801460
704b80f
0c3d81a
b380716
d34b6de
517f17b
9b2d1f1
9b4980c
546cf95
2109074
84e6595
2207f9f
fe31a27
18295df
b4cbd6a
ad4d06a
3f99f2f
d98613e
3d97537
98c3ede
c3a1a32
a59258b
8da7257
1942c8d
d87111d
9112b04
238b21d
46838ce
392c486
837df0b
6ac16ab
385eaec
b0f4083
baa58fd
668755b
dd547a6
dd34428
e8377c8
9549f12
3f33e1f
ebd05b3
159e867
ae5fb90
a2f5fe7
4a42255
b77830b
90937ff
ebb332f
37d4a19
b961834
3c5840e
38eadd2
68ededd
6ab307d
7dfb474
5a88e4f
7060a8d
fb4122b
d37ffb8
7773805
d5c0e25
015c9e5
c9328f9
20eb15a
b2e3ab0
c2974b6
b1faf35
d43af09
2a3d6c2
13b89a5
4f9f82d
45d8aa2
88ad70c
1862109
d6ee789
acaed3e
d84b73b
38c27a1
d714963
3e94a1c
93482ef
a3f7b1a
2bfe665
be5f02e
49c15df
742e923
b696aee
b8f9e3a
9d1765c
3490fdd
80a2e43
fa39706
653e3b1
6d47a1e
c0572a2
7613a52
2b89e51
3710ca5
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we're missing here the teardown of the jenkins stack.
and this needs to be optional via a parameter.
something along the lines of
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can leave this, i will tackle it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since for now, the jenkins deployment is working with default namespace only (the limitation we mentioned earlier), so there is no need to delete the default namespace, we can remove the delete command (for namespace)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have added the teardown code, also parameterised it as well