Skip to content

Update run_local_ironic.sh to support HTTP basic auth#643

Merged
metal3-io-bot merged 6 commits intometal3-io:masterfrom
Nordix:auth3-mael
Oct 6, 2020
Merged

Update run_local_ironic.sh to support HTTP basic auth#643
metal3-io-bot merged 6 commits intometal3-io:masterfrom
Nordix:auth3-mael

Conversation

@maelk
Copy link
Copy Markdown
Member

@maelk maelk commented Sep 21, 2020

Based on #642

@metal3-io-bot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: maelk

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@metal3-io-bot metal3-io-bot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Sep 21, 2020
@maelk
Copy link
Copy Markdown
Member Author

maelk commented Sep 21, 2020

/test-integration

3 similar comments
@maelk
Copy link
Copy Markdown
Member Author

maelk commented Sep 21, 2020

/test-integration

@maelk
Copy link
Copy Markdown
Member Author

maelk commented Sep 21, 2020

/test-integration

@maelk
Copy link
Copy Markdown
Member Author

maelk commented Sep 21, 2020

/test-integration

@maelk
Copy link
Copy Markdown
Member Author

maelk commented Sep 22, 2020

/test-integration

@maelk
Copy link
Copy Markdown
Member Author

maelk commented Sep 22, 2020

/test-integration

1 similar comment
@maelk
Copy link
Copy Markdown
Member Author

maelk commented Sep 22, 2020

/test-integration

@maelk maelk force-pushed the auth3-mael branch 4 times, most recently from 78a8b56 to 5d2d3be Compare September 24, 2020 06:20
@maelk
Copy link
Copy Markdown
Member Author

maelk commented Sep 24, 2020

/test-integration

1 similar comment
@maelk
Copy link
Copy Markdown
Member Author

maelk commented Sep 24, 2020

/test-integration

@dhellmann
Copy link
Copy Markdown
Member

I would like to freeze go code changes for a few days to try to land #650 without having to rebase it, because rebasing will mean redoing the work from scratch.

/hold

@metal3-io-bot metal3-io-bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 24, 2020
@dhellmann
Copy link
Copy Markdown
Member

#655 has merged

/hold cancel

@metal3-io-bot metal3-io-bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 1, 2020
maelk added 6 commits October 5, 2020 15:38
If IRONIC_CACERT_FILE is set as environment variable, BMO will
use that file as CA certificate when connecting to Ironic.
This commit adds the possibility to deploy Ironic and BMO with
TLS enabled
Support the full matrix of cases for :
- Keepalived enabled/disabled
- TLS enabled/disabled
- basic authentication enabled/disabled

Please refer to docs/deploying.md for additional information
that configures the kustomization properly and builds them.
@maelk
Copy link
Copy Markdown
Member Author

maelk commented Oct 6, 2020

/test-integration

@kashifest
Copy link
Copy Markdown
Member

/lgtm

@metal3-io-bot metal3-io-bot added the lgtm Indicates that a PR is ready to be merged. label Oct 6, 2020
@maelk
Copy link
Copy Markdown
Member Author

maelk commented Oct 6, 2020

/test-integration
The previous tests failed to start

@maelk
Copy link
Copy Markdown
Member Author

maelk commented Oct 6, 2020

/test-integration
again...

@metal3-io-bot metal3-io-bot merged commit 56a49dc into metal3-io:master Oct 6, 2020
@fmuyassarov fmuyassarov deleted the auth3-mael branch December 11, 2020 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants