Skip to content

wifi: enterprise: Pull support for runtime certificates #21228

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
Apr 3, 2025
Merged

wifi: enterprise: Pull support for runtime certificates #21228

merged 1 commit into from
Apr 3, 2025

Conversation

@krish2718
Copy link
Contributor

@krish2718 krish2718 commented Mar 29, 2025

Pull support and update docs.

@krish2718 krish2718 requested review from a team as code owners March 29, 2025 16:18
@krish2718 krish2718 added this to the 3.0.0 milestone Mar 29, 2025
@github-actions github-actions bot added doc-required PR must not be merged without tech writer approval. manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Mar 29, 2025
@krish2718 krish2718 requested a review from krga2022 March 29, 2025 16:18
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Mar 29, 2025 •
edited
Loading

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff

✅ All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Mar 29, 2025 •
edited
Loading

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 7

Inputs:

Sources:

sdk-nrf: PR head: d6764fd96bf6bbb5e05877f6abfa75a67e762d50

more details

sdk-nrf:

PR head: d6764fd96bf6bbb5e05877f6abfa75a67e762d50
merge base: 1e37cd29e42b2a3f5f3d75914957dacf4e6197cf
target head (main): 3367f9249d718e6c9bf61f625585ca53210e8517
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (1) 
doc
│  ├── nrf
│  │  ├── app_dev
│  │  │  ├── device_guides
│  │  │  │  ├── nrf70
│  │  │  │  │  │ wifi_advanced_security_modes.rst

Outputs:

Toolchain

Version:
Build docker image:

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain
  • ◻️ Build twister
  • ◻️ Integration tests
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-boot
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_mosh
    • test-fw-nrfconnect-nrf-iot_positioning
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

@github-actions
Copy link

github-actions bot commented Mar 29, 2025

You can find the documentation preview for this PR here.

Preview links for modified nRF Connect SDK documents:

https://ncsdoc.z6.web.core.windows.net/PR-21228/nrf/app_dev/device_guides/nrf70/wifi_advanced_security_modes.html

richabp
richabp requested changes Mar 31, 2025
View reviewed changes
@krish2718 krish2718 force-pushed the wifi_runtime_certs branch from f29d04b to e4ba073 Compare April 1, 2025 14:16
@krish2718 krish2718 requested a review from richabp April 1, 2025 14:16
@krish2718 krish2718 force-pushed the wifi_runtime_certs branch from e4ba073 to 25aeaf6 Compare April 1, 2025 19:31
@nordicjm nordicjm mentioned this pull request Apr 2, 2025
Closed
@krish2718 krish2718 force-pushed the wifi_runtime_certs branch from 25aeaf6 to dfeba9b Compare April 2, 2025 08:39
@NordicBuilder NordicBuilder removed the DNM label Apr 2, 2025
@krish2718
doc: nrf70: enterprise: Point to the upstream docs
d6764fd 
Upstream already has the proper instructions, so, remove duplication and
point to the upstream docs.

Signed-off-by: Chaitanya Tata <[email protected]>
@krish2718 krish2718 force-pushed the wifi_runtime_certs branch from dfeba9b to d6764fd Compare April 3, 2025 07:11
@github-actions github-actions bot removed the manifest label Apr 3, 2025
@rlubos rlubos merged commit c93ea09 into nrfconnect:main Apr 3, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nordicjm nordicjm nordicjm approved these changes

@richabp richabp richabp approved these changes

@rado17 rado17 rado17 approved these changes

@sachinthegreen sachinthegreen sachinthegreen approved these changes

@krga2022 krga2022 Awaiting requested review from krga2022

+1 more reviewer

@VivekUppunda VivekUppunda VivekUppunda approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. doc-required PR must not be merged without tech writer approval.

Projects

None yet

Milestone

3.0.0

Development

Successfully merging this pull request may close these issues.

8 participants

@krish2718 @NordicBuilder @nordicjm @richabp @rado17 @sachinthegreen @VivekUppunda @rlubos